feat(desktop): add archived task scope entry - #3294
nothingyuancando wants to merge 7 commits into
Conversation
Signed-off-by: nothingyuancando <1579035512@qq.com>
|
| Filename | Overview |
|---|---|
| apps/desktop/src/renderer/features/cc-agent/sidebar/MachineSwitcherMenu.tsx | 新增归档范围入口,将归档筛选同步到菜单选中语义、段头文本和无障碍名称。 |
| apps/desktop/src/renderer/features/cc-agent/sidebar/MainListScopeHeader.tsx | 将现有侧栏 filter 传给 MachineSwitcherMenu,使入口与列表消费同一状态。 |
| apps/desktop/src/renderer/tests/machineSwitcherMenu.test.ts | 更新静态结构断言,并覆盖归档段头名称及 menuitem/aria-current 语义。 |
| apps/desktop/src/renderer/i18n/locales/en/common.json | 新增英文归档任务文案;其余四种桌面语言同步补齐对应键值。 |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart LR
U[用户打开任务范围菜单] --> A[选择已归档任务]
A --> F[setStatus archived]
F --> L[会话列表按归档状态过滤]
F --> H[段头显示归档状态]
H --> M[菜单项通过 aria-current 表示当前范围]
Reviews (7): Last reviewed commit: "fix(desktop): expose archived scope as c..." | Re-trigger Greptile
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 443c0b949f
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
Signed-off-by: nothingyuancando <1579035512@qq.com>
|
已修复 CI 契约失败并推送 commit |
|
@nothingyuancando 👋 这个 PR 还有 1 条 review conversation 没 resolve(apps/desktop/src/renderer/features/cc-agent/sidebar/MainListScopeHeader.tsx),auto-review 因此暂时跳过、没法继续审查 / 合并。 如果你已经按评论改完或回应了,请到对应 thread 上点 Resolve conversation;全部 resolve 后,下一轮 auto-review 会自动重新审查这个 PR。 |
|
这个 PR 改了用户看得见的侧栏范围菜单(新增「已归档任务」入口和选中态),已提交维护者确认,讨论见 #3297。 请维护者直接在本 PR 上 Approve;若要改,请 Request Changes。作者这边先不用再推代码等确认结果。 |
|
命中 UI 路径(apps/desktop/src/renderer/features/cc-agent/sidebar/MachineSwitcherMenu.tsx / apps/desktop/src/renderer/features/cc-agent/sidebar/MainListScopeHeader.tsx)但 description 未附界面效果证据——建议补充改动后效果:截图/录屏,或改动后界面的 HTML 页面(```html 代码块、.html 附件或在线预览链接),便于确认界面符合 DESIGN.md 设计规范 |
|
已处理并推送:
本地环境缺少完整 workspace 内部包,无法运行该 worktree 的 device-link 测试;失败日志已按 Windows CI 的真实失败断言修正。请 CI 重新运行。 |
Signed-off-by: nothingyuancando <1579035512@qq.com>
68ff763 to
991ae51
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 991ae51e4a
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
Signed-off-by: nothingyuancando <1579035512@qq.com>
|
已处理最新 P2 review:归档范围现在同步加入 commit: |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 465b5de8a3
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
Signed-off-by: nothingyuancando <1579035512@qq.com>
|
已处理新 P2 无障碍 review:归档菜单项现在使用 |
|
已按 UI review 建议更新 PR description,新增“改动后界面效果证据”HTML 代码块,覆盖范围按钮、一级归档菜单项、选中态、aria-label/aria-checked,以及远程设置入口,便于按 DESIGN.md 核对。 |
Signed-off-by: Dash <125997726+dashhuang@users.noreply.github.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c6075aed08
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Signed-off-by: Dash <125997726+dashhuang@users.noreply.github.com>
MagicLizi
left a comment
There was a problem hiding this comment.
独立审查(standard)结论:changes-requested,P1×2。维护者确认门仍在拦(product / 讨论 issue #3297),本轮不合并。
P1 apps/desktop/src/renderer/features/cc-agent/sidebar/MachineSwitcherMenu.tsx:136 — statusItems 无条件以 DropdownMenuSeparator 开头。有远程设备时 279 行设备块末尾已经画过分隔,282 再插入 statusItems 会叠两条分隔;无远程时菜单第一项上方也会多出一条空分隔,归档项与设置项之间反而没有分隔。请只在归档项与相邻分组之间插一条分隔。
P1 apps/desktop/src/renderer/i18n/locales/en/common.json:7920 — 新 key ccAgent.sidebar.archivedSessions 把 Session 写成 Task:en Archived tasks(同组 allSessions 是 All sessions)、ja アーカイブ済みタスク(应为 セッション)、ko 보관된 작업(应为 세션)。zh-CN/zh-TW 正确。请与 allSessions / 术语表 Session 口径对齐。
这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。
MagicLizi
left a comment
There was a problem hiding this comment.
独立审查(standard)有 3 条 P1,不能合并。
-
[P1]
apps/desktop/src/renderer/features/cc-agent/sidebar/MachineSwitcherMenu.tsx:136—statusItems无条件以DropdownMenuSeparator开头;有远程设备时设备块已再画一条分隔,插入后叠两条;无远程时菜单顶部空分隔,归档项与设置项之间反而没有分隔。请只在归档项与相邻分组之间插一条分隔(有远程:设备|归档|设置各一条;无远程:归档与设置之间一条、顶部不要),并补契约测。 -
[P1]
apps/desktop/src/renderer/i18n/locales/en/common.json:7920— 新 keyccAgent.sidebar.archivedSessions把 Session 写成 Task:enArchived tasks、jaアーカイブ済みタスク、ko보관된 작업。同组allSessions是 Session/セッション/세션。权威:i18n/GLOSSARY.md与docs/product-rules/task-and-conversation-naming.md§4/§5.1。请改为 enArchived sessions、jaアーカイブ済みセッション、ko보관된 세션。中文「已归档任务 / 已歸檔任務」正确。 -
[P1] PR description 的界面证据 / 验收重点仍写
role="menuitemcheckbox"+aria-checked;当前 head 已是role="menuitem"+aria-current="page"(MachineSwitcherMenu.tsx:139-140)。请按当前 DOM/ARIA 更新证据。
这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。
MagicLizi
left a comment
There was a problem hiding this comment.
审查未通过(P1×3)。请处理后再推。
-
[P1] apps/desktop/src/renderer/features/cc-agent/sidebar/MachineSwitcherMenu.tsx:136 —
statusItems固定以DropdownMenuSeparator开头。有远程设备时,设备块末尾已有分割线,再渲染{statusItems}会叠两条;无远程时菜单以空分割线开头。相邻组之间应恰好一条分割线(有远程:设备 | 已归档 | 设置;无远程:已归档 | 设置,无开头线)。请在现有 source-contract 测试里锁住数量和位置。 -
[P1] apps/desktop/src/renderer/i18n/locales/en/common.json:7920(ja/ko 同 key) — 新
ccAgent.sidebar.archivedSessions用了 Task 栏:enArchived tasks、jaアーカイブ済みタスク、ko보관된 작업。同级allSessions是 Session / セッション / 세션。GLOSSARY 与 task-and-conversation-naming 要求 Session。请改成Archived sessions/アーカイブ済みセッション/보관된 세션。zh-CN / zh-TW 正确。 -
[P1] PR 描述 vs HEAD — 「改动后界面效果证据」和验收重点仍写
role="menuitemcheckbox"/aria-checked="true"。HEAD 与测试已是role="menuitem"+aria-current="page",并禁止 checkbox/checked。UI 证据与 diff 不符。请按当前 ARIA 重写证据和验收说明。
这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。
MagicLizi
left a comment
There was a problem hiding this comment.
需要修改后才能合并
-
[P1]
apps/desktop/src/renderer/features/cc-agent/sidebar/MachineSwitcherMenu.tsx:134—statusItems固定以DropdownMenuSeparator开头。有远程设备时,设备块末尾(约 279 行)已有一条分隔线,再渲染statusItems会叠两条;没有远程设备时菜单会顶着一条空分隔线。请保证相邻组之间恰好一条分隔线,并锁进现有 source-contract 测试。 -
[P1]
apps/desktop/src/renderer/i18n/locales/en/common.json(ja/ko 同 key)— 新 keyccAgent.sidebar.archivedSessions写成了 Task:enArchived tasks、jaアーカイブ済みタスク、ko보관된 작업。同级allSessions是 Session / セッション / 세션。请按术语表改为Archived sessions/アーカイブ済みセッション/보관된 세션。zh-CN/zh-TW 的「已归档任务」正确。 -
[P1] PR 描述里的 UI 证据仍写
role="menuitemcheckbox"与aria-checked="true",当前 HEAD 是role="menuitem"+aria-current="page"。请把 HTML 证据和验收说明改成与 diff 一致。
这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。
MagicLizi
left a comment
There was a problem hiding this comment.
审查结论:需要改后再审。
-
[P1]
apps/desktop/src/renderer/i18n/locales/en/common.json(ccAgent.sidebar.archivedSessions;ja/ko 同源)— 同一段头里allSessions是 Session 口径(All sessions / すべてのセッション / 모든 세션),新文案却写成 Task 口径(Archived tasks / アーカイブ済みタスク / 보관된 작업)。i18n/GLOSSARY.md:Session 的 ja/ko 是 セッション/세션,Task 是 タスク/작업。请把 en/ja/ko 改成 Archived sessions / アーカイブ済みセッション / 보관된 세션。zh-CN/zh-TW「已归档任务」与「全部任务」一致,不用改。 -
[P1]
apps/desktop/src/renderer/features/cc-agent/sidebar/MachineSwitcherMenu.tsx:137— PR 描述 HTML 与验收句仍写role="menuitemcheckbox"+aria-checked;当前 head 实际是role="menuitem"+aria-current="page"。请把 UI 证据和验收句改成与现实现一致。
这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。
MagicLizi
left a comment
There was a problem hiding this comment.
审查未通过(P1×3)。当前 head d8e30d936bfd77e408fc98404aa222dfd19bacab。
apps/desktop/src/renderer/features/cc-agent/sidebar/MachineSwitcherMenu.tsx:134 — 讨论 issue #3297 仍 OPEN。维护者分析明确要求:与仍 OPEN 的 #3231 收敛,不要两个 PR 同时改 MachineSwitcherMenu / MainListScopeHeader;推荐不在范围菜单新增状态入口;若仍要把归档入口提升到范围菜单一级,须先推翻「范围与状态分离」。当前 HEAD 仍把「已归档任务」做成一级 statusItems,onSelect 调用 filter.setStatus('archived')。请按 #3297 改,或先关闭/改造 #3231 并由维护者书面推翻该口径。
apps/desktop/src/renderer/i18n/locales/en/common.json:7920 — 新 key ccAgent.sidebar.archivedSessions 把 Session 写成 Task:en Archived tasks(ja/ko 同问题)。同级 allSessions 是 All sessions。i18n/GLOSSARY.md 与 docs/product-rules/task-and-conversation-naming.md 要求 en/ja/ko 跟 Session。请改为 Archived sessions / アーカイブ済みセッション / 보관된 세션。zh-CN/zh-TW「已归档任务」正确。
PR 描述 — 「改动后界面效果证据」仍写 role="menuitemcheckbox" + aria-checked="true"。HEAD 与测试已是 role="menuitem" + aria-current="page"。请按现头改写证据和验收说明。
修完后推送新 commit,下一轮会重审。
这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。
MagicLizi
left a comment
There was a problem hiding this comment.
P1×2
-
[P1] apps/desktop/src/renderer/features/cc-agent/sidebar/MachineSwitcherMenu.tsx:134 — 讨论 issue #3297 仍 OPEN,要求与 #3231 收敛:范围菜单只负责设备/范围,状态筛选留在筛选体系。当前 head 仍插入 statusItems,onSelect 调用 filter.setStatus('archived')。请去掉范围菜单状态项,只留段头图标/文案 + aria-label,或由维护者先推翻该口径。
-
[P1] apps/desktop/src/renderer/i18n/locales/en/common.json:7920(ja/ko 同 key)— 新 key ccAgent.sidebar.archivedSessions 把 Session 写成 Task:en Archived tasks、ja アーカイブ済みタスク、ko 보관된 작업。同组 allSessions 是 All sessions / すべてのセッション / 모든 세션。i18n/GLOSSARY.md:Session 的 ja/ko 是 セッション/세션。en/ja/ko 应改为 Archived sessions / アーカイブ済みセッション / 보관된 세션。zh-CN/zh-TW「已归档任务」符合任务口径,不用改。
这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。
MagicLizi
left a comment
There was a problem hiding this comment.
审查发现 P1×3,请先处理后再继续。
-
[P1] apps/desktop/src/renderer/features/cc-agent/sidebar/MachineSwitcherMenu.tsx:134
statusItems 固定以 DropdownMenuSeparator 开头;有远程时设备块末尾已经再画一条,插入 statusItems 会叠两条分隔;无远程时菜单顶部空出一条分隔。归档项与 settingsItems 之间反而没有分隔。
请让相邻分组之间恰好一条分隔(有远程:设备 | 归档 | 设置;无远程:归档 | 设置,顶部不要空线),并在 machineSwitcherMenu.test.ts 锁住条数和位置。 -
[P1] apps/desktop/src/renderer/i18n/locales/en/common.json:7920(ja/ko 同 key)
新 key ccAgent.sidebar.archivedSessions 把 Session 写成 Task:en Archived tasks、ja アーカイブ済みタスク、ko 보관된 작업。同组 allSessions 是 All sessions / すべてのセッション / 모든 세션。请改为 Archived sessions / アーカイブ済みセッション / 보관된 세션。zh-CN/zh-TW 与「全部任务」一致,不用改。 -
[P1] PR 描述与 HEAD 不符
「改动后界面效果证据」和验收重点仍写 role="menuitemcheckbox" + aria-checked="true"。HEAD 与测试已是 role="menuitem" + aria-current="page"。请按现头改写 HTML 证据和验收句。
这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。
MagicLizi
left a comment
There was a problem hiding this comment.
审查未通过(P1×4)。请处理后再推。
- [P1] apps/desktop/src/renderer/features/cc-agent/sidebar/MachineSwitcherMenu.tsx:134 — 范围菜单一级仍插入
statusItems,onSelect调filter.setStatus('archived')。同文件、同需求的 #3231 上 dashhuang 已 Request Changes:状态筛选不要进范围菜单(badge-only)。#3231 已撤出状态项;本 PR 把「已归档任务」重新做成一级入口。请去掉范围菜单状态项,只留段头图标/文案 + aria-label;或维护者先书面推翻「范围与状态分离」并关掉/改写 #3231。 - [P1] apps/desktop/src/renderer/features/cc-agent/sidebar/MachineSwitcherMenu.tsx:136 —
statusItems固定以DropdownMenuSeparator开头。有远程时设备块末尾已再画一条,会叠两条;无远程时菜单顶部空出一条。相邻分组应恰好一条分隔,并在测试里锁条数与位置。 - [P1] apps/desktop/src/renderer/i18n/locales/en/common.json — 新 key
ccAgent.sidebar.archivedSessions把 Session 写成 Task:enArchived tasks、jaアーカイブ済みタスク、ko보관된 작업。同组allSessions是 sessions / セッション / 세션。请改为Archived sessions/アーカイブ済みセッション/보관된 세션。zh-CN/zh-TW 与「全部任务」一致,不必改。 - [P1] PR 描述 vs HEAD — 「改动后界面效果证据」仍写
role="menuitemcheckbox"+aria-checked="true"。HEAD 与测试已是role="menuitem"+aria-current="page"。请按现头改写 HTML 证据和验收句。
这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。
MagicLizi
left a comment
There was a problem hiding this comment.
审查发现 P1×2。
apps/desktop/src/renderer/features/cc-agent/sidebar/MachineSwitcherMenu.tsx:136 — statusItems 固定以 DropdownMenuSeparator 开头。设备块在 279 行 {statusItems} 前已经有一条分隔线,有远程设备时会画出两条相邻分隔线;showDeviceList === false 时菜单顶部会在「已归档任务」上多一条孤儿分隔线,破坏无远程时设置项为第一行的既有契约。请在 devices | archived | settings 组之间只留一条分隔线,不要有前导孤儿线,并在 machineSwitcherMenu.test.ts 锁住条数和位置。
apps/desktop/src/renderer/i18n/locales/en/common.json:7920 — 新增 ccAgent.sidebar.archivedSessions 用了 Task 名词:en Archived tasks、ja アーカイブ済みタスク、ko 보관된 작업。同级 allSessions 用 Session / セッション / 세션;i18n/GLOSSARY.md 与 docs/product-rules/task-and-conversation-naming.md §4 要求列表项用 Session。zh-CN/zh-TW「已归档任务 / 已歸檔任務」正确。请改为 Archived sessions / アーカイブ済みセッション / 보관된 세션。
这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。
MagicLizi
left a comment
There was a problem hiding this comment.
独立代码审查未通过(P1×3)。
- apps/desktop/src/renderer/features/cc-agent/sidebar/MachineSwitcherMenu.tsx:134 —
statusItems固定以分隔线开头;设备列表分支插入前已经画过分隔线。有远程设备时会出现连续两条分隔线,无远程设备时菜单顶部会悬空一条。请只在「设备列表非空」时画一次,statusItems不要自带 leading separator。 - apps/desktop/src/renderer/i18n/locales/en/common.json:7920 — 同级
allSessions用 Session,新 keyarchivedSessions写成Archived tasks(ja/ko 同样用タスク/작업)。请改成 enArchived sessions、jaアーカイブ済みセッション、ko보관된 세션,与 glossary / task-and-conversation-naming 及同菜单文案对齐。zh-CN/zh-TW「已归档任务」可保留。 - apps/desktop/src/renderer/features/cc-agent/sidebar/MachineSwitcherMenu.tsx:137 — 讨论 issue #3297 仍 OPEN,要求先与 #3231 收敛实现(范围菜单只管设备/范围,状态留在筛选体系)。当前仍把
filter.status === 'archived'放进一级,且归档后无法从同一入口回到 active/all。请按 #3297 收成一套,并更新证据(现证据仍写 menuitemcheckbox,代码已是 menuitem)。
这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。
MagicLizi
left a comment
There was a problem hiding this comment.
P1×3:归档入口可以合,但这三处会直接露给用户。
apps/desktop/src/renderer/features/cc-agent/sidebar/MachineSwitcherMenu.tsx:134 — statusItems 开头固定画一条分隔线;有远程设备时设备列表末尾(约 279 行)已经画过一条。有设备时归档项上方双线,无设备时菜单顶部空出一条线。请只在设备列表非空时插一次分隔线,无设备时归档项直接作为首项。
apps/desktop/src/renderer/i18n/locales/en/common.json:7920 — ccAgent.sidebar.archivedSessions 的 en/ja/ko 没用 Session 术语。同屏 allSessions 已是 All sessions / すべてのセッション / 모든 세션,这里写成 Archived tasks / アーカイブ済みタスク / 보관된 작업。请改成 Archived sessions / アーカイブ済みセッション / 보관된 세션。zh-CN/zh-TW 的「已归档任务」正确。
PR 描述里的 UI 证据 HTML 写 role="menuitemcheckbox" + aria-checked="true",实现和测试是 role="menuitem" + aria-current="page"。请按现结构改证据,或删掉过期片段。
这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。
MagicLizi
left a comment
There was a problem hiding this comment.
独立审查未通过(P1×3)。当前 head d8e30d936bfd77e408fc98404aa222dfd19bacab 不能合。
-
[P1] apps/desktop/src/renderer/features/cc-agent/sidebar/MachineSwitcherMenu.tsx:136 —
statusItems固定以DropdownMenuSeparator开头,有远程时设备块已经画过分隔,插入后叠两条;无远程时菜单以空分隔开头。请只在相邻组之间留一条,并在machineSwitcherMenu.test.ts锁分隔数量与位置。 -
[P1] apps/desktop/src/renderer/i18n/locales/en/common.json —
ccAgent.sidebar.archivedSessions英文/日/韩把 Session 写成 Task(Archived tasks/アーカイブ済みタスク/보관된 작업),与同段头allSessions以及i18n/GLOSSARY.md、docs/product-rules/task-and-conversation-naming.md不一致。请改为Archived sessions/アーカイブ済みセッション/보관된 세션。中文「已归档任务」与「全部任务」一致,不用改。 -
[P1] PR description — 「改动后界面效果证据」和验收重点仍写
role="menuitemcheckbox"+aria-checked="true"。HEAD 已是role="menuitem"+aria-current="page"。请按现 DOM 改写 HTML 证据和验收句。
这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。
这次改了什么
在“全部任务”范围菜单中新增一级“已归档任务”入口,复用现有
useSidebarFilter的setStatus('archived');进入归档视图后,段头持续显示归档状态,菜单项显示选中态。五种桌面语言均已补齐。关联 Issue:#3214
明确不包含:新的归档数据模型、IPC、查询接口或第二套筛选状态。
UI 变化
docs/design-rules/DESIGN.md§1 Visual Theme & Atmosphere、§2 Color Palette & Roles、§4 Component Stylings;复用现有菜单结构、语义主题 token、图标与选中态,不新增装饰性视觉层。怎么验证的
自动验证
node scripts/check-i18n.mjs:通过(仅仓库既有警告)。node scripts/check-i18n-glossary.mjs:通过(仅仓库既有 proposed 术语警告)。git diff --check:通过。pnpm test:unit:related:已启动,但新 worktree 缺少完整 workspace 依赖,Desktop 测试收集阶段因缺失@cindy/maker-shared/*、@cindy/voice-input-core等内部包失败;未出现本次改动断言失败。手工验证
未执行:当前环境没有可运行的 Desktop 实例;交互由现有 Radix 菜单和筛选状态路径承载。
风险
风险分类
影响与回滚
影响范围:Desktop 侧栏归档入口与本地化文案。回滚方式:撤销本 PR commit;不涉及数据迁移、协议、权限或跨平台原生行为。
提交前检查
改动后界面效果证据
以下 HTML 反映改动后的实际结构与状态(对应
MachineSwitcherMenu/MainListScopeHeader;颜色由现有主题 token 提供):验收重点:归档入口位于范围菜单一级;归档状态在段头持续可见;选中态同时由勾选图标和
aria-checked表达;菜单继续复用现有 Radix 结构、间距和主题 token,无新增装饰层。