文档：记录回合错误分类层与线程条目脱敏边界

- decision-log 增加 2026-10-03 条目：TurnError::classify / TurnErrorClassified 取代 terminal_failure - 记录 ThreadManager 不再做字段级脱敏与限长、脱敏搬到前端 directThreadSanitize，失败载荷脱敏保留在 classify
2026-10-03 12:08:25 +08:00
parent 3aef44c54a
commit a8948f27bb
1 changed files with 9 additions and 0 deletions
@@ -1,5 +1,14 @@
 # 决策记录

+## 2026-10-03 回合错误的处置分类层与线程条目脱敏边界
+
+- 背景：`TurnError::terminal_failure -> Option<TurnFailure>` 用一个 `None` 同时表达"没有失败"和"不是失败、要继续跑（返修 / 复核控制流）"，调用方看到 `None` 只会理解成前者，`dispatch.rs` 两处只能 `.expect("回合失败必可投影成失败载荷")`；同时 ThreadManager 在搬运线程条目时做字段级脱敏与限长，前端要被截断，而且与失败载荷的脱敏是两套实现。
+- 决策（分类层）：`TurnError::classify(&self, root) -> TurnErrorClassified{ShouldStop(TurnFailure), ShouldContinue { detail: String }}` 取代 `terminal_failure`——真失败继续投影成 `TurnFailure`（脱敏 + 截断仍在这一处），控制流带自己的说明走 `ShouldContinue`；`dispatch.rs` 两处改成按分类 match，`ShouldContinue` 不写终态、不伪造失败。"这一轮怎么收场"仍只由 `TurnCompletion` 定义，private `SessionOutcome` 并入 `session_completion`。
+- 决策（脱敏边界）：ThreadManager **不做任何字段级脱敏与限长**——`wire/items.rs` 的 `bounded` / `detail_text` / `sanitize_detail_text` / `relativize_project_root_paths` / `thread_delta_text` 与三个字符上限常量删除，条目与流式增量原样透传；脱敏移到前端 `chat/conversation/directThreadSanitize`，在 bootstrap / consume / 历史切片进入聊天状态之前统一做（前端手里有 `projectPath`，能把项目内绝对路径归一成相对路径）。失败载荷（错误文案）的脱敏不属于 ThreadManager，保留在 `TurnError::classify` 投影时。订阅侧的 8 MiB / 8192 事件缓冲上限不动：那是背压，不是字段限长。
+- 边界：错误尚未持久化，`TurnErrorClassified` 是纯宿主内部类型、不加 `Serialize` / `TS`；线上 `TurnFailure` 与 `ThreadEvent` 形状、占位符词表（`<absolute-path>` / `[redacted-secret]` / `[redacted-sensitive-field]` / `[redacted-config]` / `[redacted sensitive context]`）都不变。`TurnFailure` 名字加了 `// TODO badnaming` 待后续改。
+- 影响范围：`apps/ai-game-creator-shell/src-tauri/src/agent/{codex_app_server/{turn_error.rs,mod.rs},thread_manager/{dispatch.rs,turn_completion.rs,wire/{items.rs,failure.rs,tests.rs}},generation/prompt_context.rs}`、`apps/ai-game-creator-shell/src-tauri/src/commands/desktop.rs`、前端 `chat/{controller/useDirectThreadChatSubscription.ts,conversation/directThreadSanitize.ts}`、`chat/generated/{ThreadItem,TurnFailure}.ts`、`tests/directThreadSanitize.test.ts`。
+- 验证：`cargo test -- agent:: --skip export_bindings`（949 passed；删掉一条已无对象的流式脱敏用例）、`npm run check:generated-bindings`、`npm run ai-game-creator-shell:typecheck`、`npx vitest run apps/ai-game-creator-shell/tests/directThreadSanitize.test.ts`（12 passed）、`npm run check:encoding`、`npm run check:doc-index`、`git diff --check`；全量 vitest 的 `gameDistributionPublish*` / `recentProjectsHook` localStorage 失败为既有问题。
+
 ## 2026-10-02 DirectProject 失败载荷两臂改名与 typed 失败原样下发

 - 背景：`turn.completed.failure` 的两个载荷臂把字符串债藏在像正常类型的名字后面（`TurnFailed { stage, detail: String }`、`TurnFailedUnclassified { detail: String }`）；同时 `direct_runtime` 与 `thread_manager::dispatch` 各有一处收口把**任意** `TurnError` 重包成 `TurnFailed { stage: turn_failure_stage(), detail: 预拼收口文案 }`，typed 变体（模型调用 / 超时 / 通道断开）在下发前就被吃掉，前端只能看到 catch-all 与一个伪造的 `code-generation` 阶段。