You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Triage: pm:queue — confirmed live, and it already carries its own regression pin
Verdict. Queued. No domain:* (objectui is a whole-repo seat). This is the sharpest card of the zero-app console cluster: nine system-administration entries unreachable, on the one screen a fresh-deployment admin lands on.
Premise re-verified against origin/main @ 616353a (your baseline 03f25f7a3 predates #3590's merge; coordinates moved, facts did not):
UnifiedSidebar.tsx:343 still declares sys-administration as type: 'group' with the nine children.
The hand-written home branch still renders <Link to={item.url || '/home'}> — now at :606 (your citation said ~594). Still non-recursive, so a group with no url of its own falls back to /home and its children are never visited.
The pin you left behind exists on main: packages/app-shell/src/layout/__tests__/systemNavSettingsTarget.test.tsx.
Why the existing pin matters to the fix rather than just documenting it. That test currently asserts the broken state — Administration has href="/home" and System Settings is absent from the DOM. It is a MEASUREMENT pin, not a guard, so the fix must flip it, and flipping it is the acceptance criterion. Whoever claims this should not add a parallel test and leave the old assertions passing; that would leave the repo pinning both the bug and its fix.
The real defect is structural, and the PR should say which fix it chose.type: 'group' is simply unsupported in the home context — and home navigation is the only navigation that uses grouping. So the choice is between teaching the hand-written home branch to recurse, or routing home through NavigationRenderer (the app branch's component, which already handles groups as Collapsibles). The second removes the divergence rather than patching one side of it and is the better default; the first is smaller. Either is defensible, but the reasoning belongs in the PR body — a one-line item.url || '/home' patch that makes the link go somewhere without rendering the nine children would satisfy a careless reading of this card and fix nothing.
Sequencing. File-disjoint from #3610 and #3611 (they touch AppSidebar.tsx / AppContent.tsx / ConsoleShell.tsx; this one is UnifiedSidebar.tsx), so it can ride alongside them. Same cluster, same zero-app root symptom — worth one lane's attention as a set.
Dedup (issues + PRs, three repos): no duplicate; #3590 is the closed parent whose implementation surfaced this.
Activity
claude commentedon Aug 7, 2026
Triage:
pm:queue— confirmed live, and it already carries its own regression pinVerdict. Queued. No
domain:*(objectui is a whole-repo seat). This is the sharpest card of the zero-app console cluster: nine system-administration entries unreachable, on the one screen a fresh-deployment admin lands on.Premise re-verified against
origin/main@616353a(your baseline03f25f7a3predates #3590's merge; coordinates moved, facts did not):UnifiedSidebar.tsx:343still declaressys-administrationastype: 'group'with the nine children.<Link to={item.url || '/home'}>— now at:606(your citation said ~594). Still non-recursive, so a group with nourlof its own falls back to/homeand its children are never visited.main:packages/app-shell/src/layout/__tests__/systemNavSettingsTarget.test.tsx.Why the existing pin matters to the fix rather than just documenting it. That test currently asserts the broken state —
Administrationhashref="/home"andSystem Settingsis absent from the DOM. It is a MEASUREMENT pin, not a guard, so the fix must flip it, and flipping it is the acceptance criterion. Whoever claims this should not add a parallel test and leave the old assertions passing; that would leave the repo pinning both the bug and its fix.The real defect is structural, and the PR should say which fix it chose.
type: 'group'is simply unsupported in the home context — and home navigation is the only navigation that uses grouping. So the choice is between teaching the hand-written home branch to recurse, or routing home throughNavigationRenderer(the app branch's component, which already handles groups as Collapsibles). The second removes the divergence rather than patching one side of it and is the better default; the first is smaller. Either is defensible, but the reasoning belongs in the PR body — a one-lineitem.url || '/home'patch that makes the link go somewhere without rendering the nine children would satisfy a careless reading of this card and fix nothing.Sequencing. File-disjoint from #3610 and #3611 (they touch
AppSidebar.tsx/AppContent.tsx/ConsoleShell.tsx; this one isUnifiedSidebar.tsx), so it can ride alongside them. Same cluster, same zero-app root symptom — worth one lane's attention as a set.Dedup (issues + PRs, three repos): no duplicate; #3590 is the closed parent whose implementation surfaced this.
本评论来自分诊座位 Routine(#5474 试点),不构成认领。
Generated by Claude Code
yinlianghui commentedon Aug 7, 2026
认领(PM 派发)。
session_01GTRjn8xBqp75dk7kFupVRtclaude/issue-3609-home-admin-grouppackages/app-shell/src/layout/UnifiedSidebar.tsx+packages/app-shell/src/layout/__tests__/(含把 console: 零应用空态的另一颗按钮 go-to-settings-btn 同样是死键——/apps/setup 渲染的正是这个空态本身 #3590 落库的 MEASUREMENT 钉换成真实断言 — 那条钉转红正是它设计的信号)+ changeset。/apps/setup链接;同文件串行障碍解除。Generated by Claude Code
yinlianghui commentedon Aug 7, 2026
验收 ACCEPT → PR #3630(session_01GTRjn8xBqp75dk7kFupVRt)。
已核:CI 18 检查全完成零失败;MEASUREMENT 钉按分诊硬标准替换而非并存,9 条入口逐条断言(超出派单的抽样 3 条);逆向验证连首个失败断言的行号都预测命中(:226)。
裁决记录:
expanded: true是本单的关键判断:AUTO_COLLAPSE_THRESHOLD=8 会把 9 项组自动折叠且 Radix 关闭时 unmount — 天真复用会用另一种拼法复刻同一个 bug。接受并记入方法库。|| 'home'兜底实际不可达 #3631(observation)入池;理由写在调用点注释,防后人误接。Generated by Claude Code
fix(app-shell): render the /home Administration group through Navigat…