Repository navigation
Conversation
…list The project list becomes a card grid that reuses the knowledge network list's toolbar, grid and card styles. Each card shows the project's stage (read from the project status), its published or last release and a rollback note, the knowledge network, data sources, owner and update time. Archive and restore move to the card's menu; archived projects stay dimmed and are shown only with "Show archived". Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
There was a problem hiding this comment.
表格改卡片的实现整体正确:请求竞态守卫、游标失效回退、权限判断与 i18n 双语都保留完整,卡片状态全部来自真实 /status 接口,没有占位兜底。只有每张卡片单独拉 status 的请求放大,以及归档后卡片沿用旧 status 两处值得作者看一眼。
影响范围与未覆盖面
- 影响范围:项目列表每页从 1 个请求变成 1 + 最多 24 个 /projects/{id}/status 并发请求,翻页、刷新、搜索时都会重新拉一遍。
待确认项(不阻断放行)
- src/modules/bkn-project/components/ProjectCard.tsx — 第 96 行每张卡片挂载时各自调一次 getProjectStatus,一页 24 张就是 24 个并发 /status 请求。需要确认后端 status 接口的开销能承受列表页这种 N+1 调用,或者考虑由后端在列表接口里直接带上 stage/release。
有异议或想再评一轮,请回复 /review——普通回帖不会触发复评,且回帖人需为本仓 OWNER/MEMBER/COLLABORATOR。
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The BKN project list (
/studio/bkn-project) moves from a table to a card grid that follows the knowledge network list (/studio/knowledge-network).KnowledgeNetworkListScene.module.css; the newProjectCardcomposesKnowledgeNetworkCard.module.css(card frame, accent bar, icon box, title, more button, menu, footer). The card's accent and icon use theme tokens (--color-text-link,--color-border).GET /projects/{id}/status(Not drafted, Drafting, Modeling, Model to confirm, Data check pending, Checking data, Data check needs attention, Ready to publish, Published);cardTitle,cardTip,columns.*,archivedTag) are removed.Each visible card reads its status once (one request per card, at most 24 per page). The list endpoint has no stage field; if that becomes a cost, a stage on
GET /projectswould replace these calls.Project import/export
Not added. There is no project import/export today in the list or in bkn-project. The only project export is requirements export (
/requirements/export, for the workshop and eval). With versioned publish in 0.2.2, what a project produces lives in the knowledge network's versions, and the network list already exports and imports those (JSON / BKN package). A project export would only move work in progress (drafts, decisions, requirement documents) between environments. Nobody has asked for that, it would need new backend endpoints, and it does not fit before the 11/15 freeze.Test plan
tsc -p tsconfig.app.json,pnpm lint,pnpm lint:types, prettiervitest run src/modules/bkn-project src/modules/knowledge-network/scenes src/modules/knowledge-network/components/network: 79 files, 836 tests passed (newProjectCard.test.tsx; list tests updated for cards and menus)🤖 Generated with Claude Code