From 3aef44c54af877a323f116ec2d7a28ec35377792 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:07:42 +0800 Subject: [PATCH] =?UTF-8?q?=E6=96=87=E6=A1=A3=EF=BC=9A=E5=90=8C=E6=AD=A5?= =?UTF-8?q?=E9=94=99=E8=AF=AF=E6=8A=95=E5=BD=B1=E7=82=B9=E6=94=B9=E5=90=8D?= =?UTF-8?q?=E4=B8=8E=E9=80=90=20delta=20=E8=84=B1=E6=95=8F=E9=80=80?= =?UTF-8?q?=E5=BD=B9=E7=8E=B0=E7=8A=B6?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - ADR 与 decision-log 里的 TurnError::terminal_failure 改成 TurnError::classify - pitfalls 的「逐 delta 脱敏吃掉段尾换行」补一条现状：Rust 逐 delta 脱敏已退役，脱敏搬到前端 directThreadSanitize --- docs/adr/【ADR】AGC命令错误结构化与错误报告口径-2026-10-01.md | 2 +- docs/project-memory/shared-memory/decision-log.md | 2 +- docs/project-memory/shared-memory/pitfalls.md | 1 + 3 files changed, 3 insertions(+), 2 deletions(-) diff --git a/docs/adr/【ADR】AGC命令错误结构化与错误报告口径-2026-10-01.md b/docs/adr/【ADR】AGC命令错误结构化与错误报告口径-2026-10-01.md index 597c4db21..83fa95574 100644 --- a/docs/adr/【ADR】AGC命令错误结构化与错误报告口径-2026-10-01.md +++ b/docs/adr/【ADR】AGC命令错误结构化与错误报告口径-2026-10-01.md @@ -59,7 +59,7 @@ DirectProject 已经解决过同一类问题（[`【ADR】DirectProject命令接 回合终态判定 `TurnCompletion` 是宿主内部判别联合（`agent/thread_manager/turn_completion.rs`），不导出、不进 `wire/`。 `TurnFailure` 里仍带字符串的两臂专门标成"待清的债"：`SuperErrorFromStringPlusStage`（`stage` 是 typed 枚举、 `detail` 仍是产生层字符串）与 `Unclassified`（连阶段都没有），名字故意起丑，`detail` typed 化后即改名； - `TurnError::terminal_failure` 是**唯一**投影点，任何地方都不许再把 typed 失败重包成"阶段失败 + 预拼文案"。 + `TurnError::classify`（返回 `TurnErrorClassified{ShouldStop, ShouldContinue}`）是**唯一**投影点，任何地方都不许再把 typed 失败重包成"阶段失败 + 预拼文案"。 - 变体按**可判定的事实**命名。服务端 400 只提供 `status + message`（`AppError.code` 仍是通用 `BAD_REQUEST`），所以 400 变体按"哪条请求的输入被拒"命名（如 `passwordLoginRejected`）， 不假装能区分密码长度/手机号格式；**任何地方都不允许对错误文案做判断**。 diff --git a/docs/project-memory/shared-memory/decision-log.md b/docs/project-memory/shared-memory/decision-log.md index 319cfea0d..db644ccfa 100644 --- a/docs/project-memory/shared-memory/decision-log.md +++ b/docs/project-memory/shared-memory/decision-log.md @@ -12,7 +12,7 @@ ## 2026-10-02 DirectProject 回合错误命名化与 wire 模块拆分 - 背景：DirectProject 三条错误通道的类型还带着 `Direct` / `DirectCodex` 前缀（`DirectTurnError` / `DirectTurnEnqueueError` / `DirectTurnFailure`），且 `thread_manager/wire.rs`（1600+ 行）把条目投影、事件、失败载荷与终态判定混在一个文件里；`direct_turn_error.rs` 也留在 `agent/` 顶层而不是它服务的 `codex_app_server` 深模块旁边。 -- 决策（命名）：既然类型已经归到具名深模块，去掉冗余前缀——`DirectTurnError → TurnError`、`DirectTurnEnqueueError → EnqueueError`、`DirectTurnFailure → TurnFailure`、`DirectTurnDeadline → Deadline`、`DirectCodexFailureStage → FailureStage`、`DirectCodexNativeKind → NativeKind`、`DirectModelCallKind → ModelCallKind`；`direct_codex_user_item` 里的 `DirectCodexUserItem/UserMessageItem/UserRole/UserContentPart/UserAttachmentReferencePart/UserRuntimeRegionPart → UserItem/UserMessageItem/UserRole/UserContentPart/UserAttachmentReferencePart/UserRuntimeRegionPart`；`codex_app_server` 内部的 `DirectCodexTurnKind/DirectCodexTurnObservation/DirectTurnRunFailure/DirectTurnReport/DirectTurnTerminalContext/DirectTurnCancelView → TurnKind/TurnObservation/RunFailure/TurnReport/TerminalContext/TurnCancelView`；`direct_now_ms → now_ms`。载荷 struct 名仍是"变体裸名"，`TurnFailure` 的投影点仍是 `TurnError::terminal_failure` 一处。 +- 决策（命名）：既然类型已经归到具名深模块，去掉冗余前缀——`DirectTurnError → TurnError`、`DirectTurnEnqueueError → EnqueueError`、`DirectTurnFailure → TurnFailure`、`DirectTurnDeadline → Deadline`、`DirectCodexFailureStage → FailureStage`、`DirectCodexNativeKind → NativeKind`、`DirectModelCallKind → ModelCallKind`；`direct_codex_user_item` 里的 `DirectCodexUserItem/UserMessageItem/UserRole/UserContentPart/UserAttachmentReferencePart/UserRuntimeRegionPart → UserItem/UserMessageItem/UserRole/UserContentPart/UserAttachmentReferencePart/UserRuntimeRegionPart`；`codex_app_server` 内部的 `DirectCodexTurnKind/DirectCodexTurnObservation/DirectTurnRunFailure/DirectTurnReport/DirectTurnTerminalContext/DirectTurnCancelView → TurnKind/TurnObservation/RunFailure/TurnReport/TerminalContext/TurnCancelView`；`direct_now_ms → now_ms`。载荷 struct 名仍是"变体裸名"，`TurnFailure` 的投影点仍是 `TurnError::classify` 一处。 - 决策（模块归位）：`agent/thread_manager/wire.rs` 提升为目录并按职责拆分——`wire/{mod,clock,items,turn,failure,tests}.rs`，`wire/` 只留**线上形状**（`TurnFailure` 载荷与 `ThreadEvent`）；`agent/direct_turn_failure.rs` 的终态判定并入 `agent/thread_manager/turn_completion.rs`；`agent/direct_turn_error.rs`（三条错误表与分类）移到 `agent/codex_app_server/turn_error.rs`。`thread_manager::wire::*` 仍从 `mod.rs` 平铺 re-export，外部路径不变。 - 决策（终态类型）：宿主侧的回合终态从 `struct TurnTerminal { status, failure }` 改为判别联合 `enum TurnCompletion { Completed, Interrupted, Aborted, Failed(TurnFailure) }`——`status` 由变体反推（投影回 `turn.completed` 时才变回字符串），失败必须带载荷、正常收场不许带；「有载荷就一定是失败」这条反推关系不变。`TurnCompletion` **不是线上形状**：不加 `Serialize` / `TS`、不导出、不下发，放在 `thread_manager/turn_completion.rs` 而不是 `wire/`；收尾阶段推出来的 `status` 字符串在这里一次性收进类型，认不出的值 fail closed（按未分类失败），不冒充正常收场。`thread_manager/mod.rs` 的过期回合释放兜底也改走 `TurnCompletion::Aborted.event(...)`。 - 边界：只改类型 / 函数 / 生成文件命名与模块位置，不改线上 JSON 形状（`TurnFailure` 的 `type` 判别值与字段名不变，`TurnCompletion` 只影响宿主内部）；不迁就存量字符串分流，不保留别名或 `Display` 回落。`DirectTurnFailureKind`、`wire_kind()`、`Display for TurnError` 与 `From for String` 已在上一轮删除。 diff --git a/docs/project-memory/shared-memory/pitfalls.md b/docs/project-memory/shared-memory/pitfalls.md index 16dfe9416..cb3bb7e09 100644 --- a/docs/project-memory/shared-memory/pitfalls.md +++ b/docs/project-memory/shared-memory/pitfalls.md @@ -72,6 +72,7 @@ - **仍未修**：`/` 开头的 delta 片段被判成绝对路径（上面 ②）。要修必须把「上一个字符」带进边界判定（正文增量可从 `direct_project_history` 的累计文本取，思考增量目前没有累计器），属于跨 delta 状态的取舍，本次未一并改。 - **判据/取证**：`cargo test --manifest-path apps/ai-game-creator-shell/src-tauri/Cargo.toml --bin genarrative-ai-game-creator-shell direct_thread_delta`——新增 `direct_thread_delta_sanitization_preserves_line_breaks` 钉住「逐段脱敏 == 整段脱敏」，去掉 `split_inclusive` 即红；真实文本的回归用渲染侧夹具复核（修复前 table/li/h2 全 0，修复后与整段脱敏一致：1 个 table / 3 个 th / 4 个 h2）。 - **关联**：`apps/ai-game-creator-shell/src-tauri/src/agent/codex_app_server/mod.rs`、`.../agent/generation/prompt_context.rs`、`.../agent/thread_manager/wire.rs`、`apps/ai-game-creator-shell/src/view/project-development/chat/conversation/directThreadChat.ts`、`#384`。 +- **现状（2026-10-03 更新）**：逐 delta 的 Rust 脱敏已整体退役——ThreadManager 改为原样透传，流式正文与历史条目的脱敏统一移到前端 `directThreadSanitize`，在事件进聊天状态前逐段处理并保留换行。上面两条根因与"仍未修"针对的实现已不存在，保留作历史。 ## 2026-10-01 文件列举被无关临时锁删除打断