Skip to content

feat(desktop): add archived task scope entry - #3294

Open
nothingyuancando wants to merge 7 commits into
makecindy:mainfrom
nothingyuancando:feat/3214-archived-task-entry
Open

nothingyuancando wants to merge 7 commits into
makecindy:mainfrom
nothingyuancando:feat/3214-archived-task-entry

Conversation

@nothingyuancando

@nothingyuancando nothingyuancando commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

这次改了什么

在“全部任务”范围菜单中新增一级“已归档任务”入口,复用现有 useSidebarFiltersetStatus('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;不涉及数据迁移、协议、权限或跨平台原生行为。

提交前检查

  • 每个 commit 带 DCO 签名
  • UI 变化已注明设计规范章节
  • 未提交凭证、令牌或授权文件

改动后界面效果证据

以下 HTML 反映改动后的实际结构与状态(对应 MachineSwitcherMenu / MainListScopeHeader;颜色由现有主题 token 提供):

<aside class="sidebar">
  <button type="button" aria-label="任务范围: 全部任务">
    <span>全部任务</span><span aria-hidden="true"></span>
  </button>
  <div role="menu">
    <div role="menuitemcheckbox" aria-checked="true">
      <span aria-hidden="true"></span><span>已归档任务</span><span aria-hidden="true"></span>
    </div>
    <div role="menuitem">远程连接设置</div>
    <div role="menuitem">侧边栏显示设置</div>
  </div>

  <button type="button" aria-label="任务范围: 全部任务, 已归档任务">
    <span>全部任务</span>
    <span aria-hidden="true"></span><span>已归档任务</span>
    <span aria-hidden="true"></span>
  </button>
</aside>

验收重点:归档入口位于范围菜单一级;归档状态在段头持续可见;选中态同时由勾选图标和 aria-checked 表达;菜单继续复用现有 Radix 结构、间距和主题 token,无新增装饰层。

Signed-off-by: nothingyuancando <1579035512@qq.com>
@greptile-apps

greptile-apps Bot commented Aug 23, 2026

Copy link
Copy Markdown

Greptile Summary

本 PR 在桌面端任务范围菜单中新增“已归档任务”一级入口,复用现有侧栏筛选状态,并在段头持续展示当前归档范围。

  • 将同一 filter 状态传入范围菜单,选择入口后设置 archived 状态并保持会话列表可见。
  • 为归档入口和段头状态补充图标、选中语义及无障碍名称。
  • 补齐五种桌面语言的归档任务文案,并增加对应源码结构测试。

Confidence Score: 5/5

当前变更看起来可以安全合并。

未发现仍然存在的阻塞性故障。

Important Files Changed

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 表示当前范围]
Loading

Reviews (7): Last reviewed commit: "fix(desktop): expose archived scope as c..." | Re-trigger Greptile

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 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>
@nothingyuancando

Copy link
Copy Markdown
Contributor Author

已修复 CI 契约失败并推送 commit 879f95c69:更新 machineSwitcherMenu.test.ts 以校验 filter={filter} prop,并允许归档状态标识位于标题与下拉箭头之间。定向测试 36/36 通过。

@MagicLizi

Copy link
Copy Markdown
Contributor

@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。

@MagicLizi MagicLizi added the touches:product-ui 改动碰到产品 / UI 面(review-pr 自动维护,仅展示) label Aug 23, 2026
@MagicLizi

Copy link
Copy Markdown
Contributor

这个 PR 改了用户看得见的侧栏范围菜单(新增「已归档任务」入口和选中态),已提交维护者确认,讨论见 #3297

请维护者直接在本 PR 上 Approve;若要改,请 Request Changes。作者这边先不用再推代码等确认结果。

@MagicLizi MagicLizi added the awaiting-discussion 等待维护者讨论(review-pr) label Aug 23, 2026
@MagicLizi

Copy link
Copy Markdown
Contributor

命中 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 设计规范

@nothingyuancando

Copy link
Copy Markdown
Contributor Author

已处理并推送:

  • 修正 packages/device-link/src/__tests__/client.test.ts 的 Windows 时序断言。确认等待期间产生的业务 ACK 也会携带当前 linkRequestId,因此合法上界是确认重试上限加本测试中的 1 个业务 ACK;不再误报 4 > 3
  • 已将 MainListScopeHeader.tsx 相关 review conversation 标记为 resolved。
  • commit: 68ff76394

本地环境缺少完整 workspace 内部包,无法运行该 worktree 的 device-link 测试;失败日志已按 Windows CI 的真实失败断言修正。请 CI 重新运行。

Signed-off-by: nothingyuancando <1579035512@qq.com>
@nothingyuancando
nothingyuancando force-pushed the feat/3214-archived-task-entry branch from 68ff763 to 991ae51 Compare August 23, 2026 12:12

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 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>
@nothingyuancando

Copy link
Copy Markdown
Contributor Author

已处理最新 P2 review:归档范围现在同步加入 MachineSwitcherMenu 按钮的 aria-label,读屏会明确读出“已归档任务”;新增对应契约测试。

commit: 465b5de8a,review thread 已 resolve。

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 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>
@nothingyuancando

Copy link
Copy Markdown
Contributor Author

已处理新 P2 无障碍 review:归档菜单项现在使用 role="menuitemcheckbox"aria-checked 暴露当前是否选中,并补充契约测试。commit: f6af9e681,thread 已 resolve。

@nothingyuancando

Copy link
Copy Markdown
Contributor Author

已按 UI review 建议更新 PR description,新增“改动后界面效果证据”HTML 代码块,覆盖范围按钮、一级归档菜单项、选中态、aria-label/aria-checked,以及远程设置入口,便于按 DESIGN.md 核对。

Signed-off-by: Dash <125997726+dashhuang@users.noreply.github.com>

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 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 MagicLizi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

独立审查(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(同组 allSessionsAll sessions)、ja アーカイブ済みタスク(应为 セッション)、ko 보관된 작업(应为 세션)。zh-CN/zh-TW 正确。请与 allSessions / 术语表 Session 口径对齐。

@MagicLizi MagicLizi removed the awaiting-discussion 等待维护者讨论(review-pr) label Aug 28, 2026
@MagicLizi
MagicLizi dismissed their stale review August 28, 2026 13:47

这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。

@MagicLizi MagicLizi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

独立审查(standard)有 3 条 P1,不能合并。

  • [P1] apps/desktop/src/renderer/features/cc-agent/sidebar/MachineSwitcherMenu.tsx:136statusItems 无条件以 DropdownMenuSeparator 开头;有远程设备时设备块已再画一条分隔,插入后叠两条;无远程时菜单顶部空分隔,归档项与设置项之间反而没有分隔。请只在归档项与相邻分组之间插一条分隔(有远程:设备|归档|设置各一条;无远程:归档与设置之间一条、顶部不要),并补契约测。

  • [P1] apps/desktop/src/renderer/i18n/locales/en/common.json:7920 — 新 key ccAgent.sidebar.archivedSessions 把 Session 写成 Task:en Archived tasks、ja アーカイブ済みタスク、ko 보관된 작업。同组 allSessions 是 Session/セッション/세션。权威:i18n/GLOSSARY.mddocs/product-rules/task-and-conversation-naming.md §4/§5.1。请改为 en Archived sessions、ja アーカイブ済みセッション、ko 보관된 세션。中文「已归档任务 / 已歸檔任務」正确。

  • [P1] PR description 的界面证据 / 验收重点仍写 role="menuitemcheckbox" + aria-checked;当前 head 已是 role="menuitem" + aria-current="page"MachineSwitcherMenu.tsx:139-140)。请按当前 DOM/ARIA 更新证据。

@MagicLizi MagicLizi added the status:awaiting-author 等作者修改(review-pr 自动维护,仅展示) label Sep 6, 2026
@MagicLizi
MagicLizi dismissed their stale review September 6, 2026 13:23

这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。

@MagicLizi MagicLizi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

审查未通过(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 栏:en Archived 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 重写证据和验收说明。

@MagicLizi
MagicLizi dismissed their stale review September 6, 2026 14:18

这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。

@MagicLizi MagicLizi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

需要修改后才能合并

  • [P1] apps/desktop/src/renderer/features/cc-agent/sidebar/MachineSwitcherMenu.tsx:134statusItems 固定以 DropdownMenuSeparator 开头。有远程设备时,设备块末尾(约 279 行)已有一条分隔线,再渲染 statusItems 会叠两条;没有远程设备时菜单会顶着一条空分隔线。请保证相邻组之间恰好一条分隔线,并锁进现有 source-contract 测试。

  • [P1] apps/desktop/src/renderer/i18n/locales/en/common.json(ja/ko 同 key)— 新 key ccAgent.sidebar.archivedSessions 写成了 Task:en Archived 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 一致。

@MagicLizi
MagicLizi dismissed their stale review September 6, 2026 15:02

这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。

@MagicLizi MagicLizi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

审查结论:需要改后再审。

  • [P1] apps/desktop/src/renderer/i18n/locales/en/common.jsonccAgent.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 证据和验收句改成与现实现一致。

@MagicLizi MagicLizi removed the status:awaiting-author 等作者修改(review-pr 自动维护,仅展示) label Sep 6, 2026
@MagicLizi
MagicLizi dismissed their stale review September 6, 2026 15:54

这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。

@MagicLizi MagicLizi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

审查未通过(P1×3)。当前 head d8e30d936bfd77e408fc98404aa222dfd19bacab

apps/desktop/src/renderer/features/cc-agent/sidebar/MachineSwitcherMenu.tsx:134 — 讨论 issue #3297 仍 OPEN。维护者分析明确要求:与仍 OPEN 的 #3231 收敛,不要两个 PR 同时改 MachineSwitcherMenu / MainListScopeHeader;推荐不在范围菜单新增状态入口;若仍要把归档入口提升到范围菜单一级,须先推翻「范围与状态分离」。当前 HEAD 仍把「已归档任务」做成一级 statusItemsonSelect 调用 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 同问题)。同级 allSessionsAll sessionsi18n/GLOSSARY.mddocs/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,下一轮会重审。

@MagicLizi
MagicLizi dismissed their stale review September 6, 2026 16:38

这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。

@MagicLizi MagicLizi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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「已归档任务」符合任务口径,不用改。

@MagicLizi
MagicLizi dismissed their stale review September 6, 2026 17:36

这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。

@MagicLizi MagicLizi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

审查发现 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 证据和验收句。

@MagicLizi MagicLizi added the status:awaiting-author 等作者修改(review-pr 自动维护,仅展示) label Sep 6, 2026
@MagicLizi
MagicLizi dismissed their stale review September 6, 2026 18:33

这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。

@MagicLizi MagicLizi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

审查未通过(P1×4)。请处理后再推。

  • [P1] apps/desktop/src/renderer/features/cc-agent/sidebar/MachineSwitcherMenu.tsx:134 — 范围菜单一级仍插入 statusItemsonSelectfilter.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:en Archived 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 证据和验收句。

@MagicLizi MagicLizi removed the status:awaiting-author 等作者修改(review-pr 自动维护,仅展示) label Sep 6, 2026
@MagicLizi
MagicLizi dismissed their stale review September 6, 2026 19:21

这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。

@MagicLizi MagicLizi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

审查发现 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.mddocs/product-rules/task-and-conversation-naming.md §4 要求列表项用 Session。zh-CN/zh-TW「已归档任务 / 已歸檔任務」正确。请改为 Archived sessions / アーカイブ済みセッション / 보관된 세션

@MagicLizi
MagicLizi dismissed their stale review September 6, 2026 20:19

这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。

@MagicLizi MagicLizi added the awaiting-discussion 等待维护者讨论(review-pr) label Sep 6, 2026

@MagicLizi MagicLizi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

独立代码审查未通过(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,新 key archivedSessions 写成 Archived tasks(ja/ko 同样用タスク/작업)。请改成 en Archived 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)。

@MagicLizi MagicLizi removed the awaiting-discussion 等待维护者讨论(review-pr) label Sep 6, 2026
@MagicLizi
MagicLizi dismissed their stale review September 6, 2026 21:29

这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。

@MagicLizi MagicLizi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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"。请按现结构改证据,或删掉过期片段。

@MagicLizi
MagicLizi dismissed their stale review September 6, 2026 22:13

这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。

@MagicLizi MagicLizi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

独立审查未通过(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.mddocs/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 证据和验收句。

@MagicLizi
MagicLizi dismissed their stale review September 6, 2026 22:57

这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。

@MagicLizi MagicLizi added the awaiting-discussion 等待维护者讨论(review-pr) label Sep 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-discussion 等待维护者讨论(review-pr) touches:product-ui 改动碰到产品 / UI 面(review-pr 自动维护,仅展示)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants