From a8948f27bb323af35caef105c2d35fa8d9a9757e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E7=8E=8B=E5=BE=B7=E5=AE=87?= Date: Sat, 3 Oct 2026 12:08:25 +0800 Subject: [PATCH] =?UTF-8?q?=E6=96=87=E6=A1=A3=EF=BC=9A=E8=AE=B0=E5=BD=95?= =?UTF-8?q?=E5=9B=9E=E5=90=88=E9=94=99=E8=AF=AF=E5=88=86=E7=B1=BB=E5=B1=82?= =?UTF-8?q?=E4=B8=8E=E7=BA=BF=E7=A8=8B=E6=9D=A1=E7=9B=AE=E8=84=B1=E6=95=8F?= =?UTF-8?q?=E8=BE=B9=E7=95=8C?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - decision-log 增加 2026-10-03 条目：TurnError::classify / TurnErrorClassified 取代 terminal_failure - 记录 ThreadManager 不再做字段级脱敏与限长、脱敏搬到前端 directThreadSanitize，失败载荷脱敏保留在 classify --- docs/project-memory/shared-memory/decision-log.md | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/docs/project-memory/shared-memory/decision-log.md b/docs/project-memory/shared-memory/decision-log.md index db644ccfa..51117eb26 100644 --- a/docs/project-memory/shared-memory/decision-log.md +++ b/docs/project-memory/shared-memory/decision-log.md @@ -1,5 +1,14 @@ # 决策记录 +## 2026-10-03 回合错误的处置分类层与线程条目脱敏边界 + +- 背景：`TurnError::terminal_failure -> Option` 用一个 `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` 形状、占位符词表（`` / `[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` 阶段。