From 4c04396a701afe965dbb439aaaa17fd68c1302d6 Mon Sep 17 00:00:00 2001 From: weichen Date: Wed, 30 Sep 2026 15:37:23 +0000 Subject: [PATCH] Record review-fix evidence on the chat-process-split-r1 task card Why: the card and PR description are one document; the 2026-09-30 review fixes (symlink untrack, per-turn live interleaving, artifact process mark) need to appear on it. Evidence: card lists frontend 22a8723/754ce83, backend ae8e5d0, Dev 596d02c with the verification outputs (vitest 27/27, go test ./... ok, make test exit 0). Impact: .task documentation only. Refs: feature/chat-process-split-r1 --- .task/feature-chat-process-split-r1.md | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/.task/feature-chat-process-split-r1.md b/.task/feature-chat-process-split-r1.md index 9b1f3e97..8157c708 100644 --- a/.task/feature-chat-process-split-r1.md +++ b/.task/feature-chat-process-split-r1.md @@ -34,3 +34,10 @@ - 影响与剩余项: - 行为变化:持久化消息中工具轮文本进入终态过程折叠区;live 末态与终态同构(对账无跳变)。 - 剩余:模块 PR 与 Dev gitlink PR(面板恢复后自动建并合入);无头走查留卡流 CF2 一并执行。 +- 审查修复(2026-09-30 代码复审,覆盖全部审查项): + - frontend `22a8723`:untrack node_modules 符号链接(曾指向本机绝对路径被提交);`.gitignore` 规则 `node_modules/` → `node_modules`(目录与符号链接同拦)。 + - frontend `754ce83`:LiveState 改为到达序 `items` 流(thinking/text 增量并段、工具按位落位),`liveToMessage` 与落库轮次穿插序同构,多轮对账不再重排;文本段 process 由「本段内是否出现工具调用」判定(对齐落库 `Process: len(calls) > 0`),tool.call 一到解说即入过程区;折叠谓词抽取为 `showProcessInline`;长回复摘要加 100 字摘句并注明按 Markdown 源码计长。 + - backend `ae8e5d0`:`PartArtifactRef` 补 `Process: true`(与同轮 tool_call/tool_result 一致,终态随过程折叠);新增 TestLLMRunnerArtifactPartIsProcess。 + - Dev `596d02c`:gitlink 更新至上述两笔模块提交。 + - 验证:frontend vitest 27/27(live 用例 5→12:多轮部件顺序、即时改标、折叠谓词)+ `tsc -b` 0 错误;backend `go test ./...` 全绿;Dev worktree `make test` 退出码 0(frontend 27/27、ui 35/35、agent-command ok)。 + - 残余边界(live.ts 注释已载):静默纯工具轮(该轮无正文无思考)的调用会并入上一文本段标记,需协议携带轮次归属才能消除。