Skip to content

feat(chat): rework the top-level turn fold header - #1817

Merged
Tioit-Wang merged 6 commits into
mainfrom
feat/turn-process-header
Oct 23, 2026
Merged

Tioit-Wang merged 6 commits into
mainfrom
feat/turn-process-header

Conversation

@Tioit-Wang

Copy link
Copy Markdown
Collaborator

改动

  • 顶层回合折叠标题改为整行细线(方案 B):状态、耗时、工具数组成一个左对齐序列,折叠指示紧跟工具数。
  • 顶层标题不再显示问题数量;失败由状态本身和错误图标表达。嵌套活动组标题的问题摘要保持不变。
  • 移除顶层标题的背景 hover,改为仅加深文字颜色;嵌套组标题的 hover 行为保持不变。
  • 同步 ADR、组件规格与 E2E 场景的中英文文档。

验证

  • node --test test/turn-process-header.test.mjs test/turn-process-header-style.test.mjs test/turn-process.test.mjs test/transcript-style.test.mjs test/transcript-display-modes-ui.test.mjs:66 passed。
  • 同目录下相关 transcript 测试子集:82 passed。
  • node node_modules/typescript/bin/tsc -p tsconfig.json --noEmit:通过。
  • node scripts/check-style-tokens.mjs:通过。
  • 渲染层生产构建(Vite,复用 electron-vite renderer 配置):通过。
  • 完整 electron-vite build 在本机 Windows 环境的主进程 SSR 打包阶段报 child_process: * 转换错误;该错误与本次纯渲染层改动无关,当前环境为复用其他 worktree 的依赖树。以 CI 结果为准。

风险

仅渲染层展示变更,不涉及持久化、IPC、协议、权限或插件契约。

v_wyxgwang added 6 commits October 22, 2026 17:15
The turn header now reads as a hairline row: status, recorded duration and tool count stay in one left-aligned run, with the disclosure caret trailing the metadata instead of sitting at the far edge of the transcript. Hovering only strengthens the text colour, and the issue count is no longer repeated at this level because the status already names a failure.

Nested activity-group headers and their hover fill stay unchanged so the transcript hierarchy below the turn fold is not flattened.
The top-level row no longer bleeds into the gutter, so its fill starts at the prose edge while nested group and run fills keep their 4px hover bleed. The shared 5px gap keeps the turn title on the same text line as the nested titles, and the transcript E2E probes the new content edge and full-width hairline.
The probe compared the header with the padded fixture root, so a correct full-width row could never match. Measure the header against the turn process that owns it instead.
# Conflicts:
#	docs/adr/turn-process-and-thinking-display.md
@Tioit-Wang
Tioit-Wang merged commit 0a76c8b into main Oct 23, 2026
9 of 10 checks passed
@Tioit-Wang
Tioit-Wang deleted the feat/turn-process-header branch October 23, 2026 03:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant