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
{{ message }}
This repository was archived by the owner on Nov 4, 2026. It is now read-only.
Repository navigation
This repository was archived by the owner on Nov 4, 2026. It is now read-only.
Triage: pm:queue — all four call sites re-verified live; ⚠️line numbers have drifted
Verdict. Queued. No domain:* (objectui is a whole-repo seat). The "越界即停, report don't fix" handling was right and is why this is a clean card rather than a scope creep on #3590.
Premise re-verified against origin/main @ 616353a — your baseline was 03f25f7a3, and #3590 has since merged, so every line number in the table has moved. All four targets are still live, at these coordinates:
your citation
current
still bare /apps/setup?
AppSidebar.tsx:445
:454 (navigate('/apps/setup'))
yes
AppSidebar.tsx:675
:684 (navigate('/apps/setup'))
yes
ConsoleShell.tsx:378
:378 (unmoved)
yes
QuickActions.tsx:49
:49 (unmoved)
yes
⚠️ Whoever claims this must re-anchor by content, not by the table above — AppSidebar.tsx is an actively-edited file and will drift again.
Root cause confirmed as shared with #3590 (now closed, PR landed): AppContent mounts the system hub only when isSystemRoute holds, so bare /apps/setup is the "No Apps Configured" empty state's own URL. #3590 redirected the empty-state CTA and two sys-settings nav items; these four were outside its declared file surface.
Two classes in one card, and the split should survive into the PR:
Three are live defects — reachable in a zero-app deployment today. system-sidebar-header is the sharpest: it renders only when there are no apps, i.e. it is unreachable except in exactly the state where its target is broken.
QuickActions.tsx:49 is dormant — QuickActions has zero JSX call sites repo-wide (exported from console/home/index.ts, rendered by nobody). Fix it anyway, in this pass: it costs one line now, and the alternative is a dead link that silently returns the day someone remounts it on HomePage.
ConsoleShell.tsx:378 deserves the small note you gave it — it is already half right (suffix ? '/apps/setup/system' + suffix : '/apps/setup'), so the fix is making the bare-/system bookmark agree with the branch beside it, not new logic.
Sequencing (not a blocker).#3610 also edits AppSidebar.tsx (:329 / :330). Different lines, same hot file. ⛔ Do not run these two as concurrent independent claims — sequence them or land them together.
Dedup (issues + PRs, three repos): no duplicate. #3590 is the parent finding and is closed; this is its declared remainder, not a re-file.
Activity
claude commentedon Aug 7, 2026
Triage:⚠️ line numbers have drifted
pm:queue— all four call sites re-verified live;Verdict. Queued. No
domain:*(objectui is a whole-repo seat). The "越界即停, report don't fix" handling was right and is why this is a clean card rather than a scope creep on #3590.Premise re-verified against
origin/main@616353a— your baseline was03f25f7a3, and #3590 has since merged, so every line number in the table has moved. All four targets are still live, at these coordinates:/apps/setup?AppSidebar.tsx:445:454(navigate('/apps/setup'))AppSidebar.tsx:675:684(navigate('/apps/setup'))ConsoleShell.tsx:378:378(unmoved)QuickActions.tsx:49:49(unmoved)AppSidebar.tsxis an actively-edited file and will drift again.Root cause confirmed as shared with #3590 (now closed, PR landed):
AppContentmounts the system hub only whenisSystemRouteholds, so bare/apps/setupis the "No Apps Configured" empty state's own URL. #3590 redirected the empty-state CTA and twosys-settingsnav items; these four were outside its declared file surface.Two classes in one card, and the split should survive into the PR:
system-sidebar-headeris the sharpest: it renders only when there are no apps, i.e. it is unreachable except in exactly the state where its target is broken.QuickActions.tsx:49is dormant —QuickActionshas zero JSX call sites repo-wide (exported fromconsole/home/index.ts, rendered by nobody). Fix it anyway, in this pass: it costs one line now, and the alternative is a dead link that silently returns the day someone remounts it on HomePage.ConsoleShell.tsx:378deserves the small note you gave it — it is already half right (suffix ? '/apps/setup/system' + suffix : '/apps/setup'), so the fix is making the bare-/systembookmark agree with the branch beside it, not new logic.Sequencing (not a blocker). #3610 also edits
AppSidebar.tsx(:329/:330). Different lines, same hot file. ⛔ Do not run these two as concurrent independent claims — sequence them or land them together.Dedup (issues + PRs, three repos): no duplicate. #3590 is the parent finding and is closed; this is its declared remainder, not a re-file.
本评论来自分诊座位 Routine(#5474 试点),不构成认领。
Generated by Claude Code
yinlianghui commentedon Aug 7, 2026
认领(PM 派发)。
session_01GTRjn8xBqp75dk7kFupVRtclaude/issue-3611-remaining-setup-linkspackages/app-shell/src/layout/AppSidebar.tsx、packages/app-shell/src/console/ConsoleShell.tsx、packages/app-shell/src/console/home/QuickActions.tsx+ 测试 + changeset。layout/__tests__/systemNavSettingsTarget.test.tsx正被在途 console: /home 的 Administration 组被压平成一条指向 /home 的死链——整簇 9 个系统管理入口在参考 console 上不可达 #3609 翻转,不许碰 — 新断言放新文件或各组件既有测试文件。UnifiedSidebar.tsx(在途 console: /home 的 Administration 组被压平成一条指向 /home 的死链——整簇 9 个系统管理入口在参考 console 上不可达 #3609)、AppContent.tsx(在途 console: 零应用下 sys-objects / sys-datasources 落到 /apps/setup/component/... 渲染空白屏——无-activeApp 分支既无 component/* 路由也无 catch-all #3610)也不许碰。/apps/setup/system— 该语义已由 console: 零应用空态的另一颗按钮 go-to-settings-btn 同样是死键——/apps/setup 渲染的正是这个空态本身 #3590/fix(app-shell): 把 System Settings 指向系统枢纽 /apps/setup/system,而非空态自己的裸 URL #3608 + fix(app-shell): 空态「Create Your First App」CTA 指向已声明的 app 内 create-app 路由 (#3573) #3588 的 app-scoped absolute 契约确立,本单按此收口。休眠的 QuickActions 一并改(顺手成本为零,防止挂回时带死链复活);SystemRedirect裸/system分支改成与自己带后缀的另一半自洽。Generated by Claude Code
yinlianghui commentedon Aug 7, 2026
验收 ACCEPT → PR #3637(session_01GTRjn8xBqp75dk7kFupVRt)。
已核:4 处常量替换零行为分叉;测试隔离遵守(systemNavSettingsTarget.test.tsx / UnifiedSidebar / AppContent 一行未碰,与在途 #3609/#3610 零相交已核对);逆向验证三源回退后 5 红 5 绿逐条方向命中。
裁决记录:
零应用 console 簇(#3590/#3609/#3610/#3611)至此四单全部有 PR;#3610 在途,其余三单落地/在队。
Generated by Claude Code
fix(app-shell): point the remaining four Settings senders at /apps/se…
fix(app-shell): point the three sys-objects producers at the canonica…