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
This commit is contained in:
@@ -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 注释已载):静默纯工具轮(该轮无正文无思考)的调用会并入上一文本段标记,需协议携带轮次归属才能消除。
|
||||
|
||||
Reference in New Issue
Block a user