[Bug] Agent 工具返回业务失败对象时仍被传输和持久化为 success

[ShallM]

版本

main 源码，commit 320c07f

平台 / 功能

macOS Apple Silicon；AI Agent 对话剪辑 / Server Run。代码路径跨平台。

严重性

高：会让 Agent 在明知工具失败的情况下继续工作并最终声称完成，直接破坏工程质量和审计可信度。

问题描述

OpenChatCut 工具有两种失败方式：抛异常，以及正常返回 { error }、{ ok:false }、{ aborted:true }。后者在浏览器桥、server store 和 runtime ledger 中会被当作成功结果：tool result HTTP 通道不带 error、运行记录写 success，completion gate 也看不到未解决失败。

复现步骤

1. 让 edit_item 返回原子校验失败，例如非法 easing。
2. 或让 analyze_music 返回 { error: "clip is not linked..." }。
3. 检查 browser tool result、server run event、agent runtime ledger 和最终状态。
4. 观察模型仍可能继续后续步骤并输出“完成”。

期望行为

所有业务失败形态在每一层都保持失败语义；没有成功重试前，run 不得以 completed 结束，也不得记录成功 outcome。

实际行为

只要没有 throw，失败对象就走 success 通道；模型和运行记录均可能把失败当成成功。

根因定位

浏览器执行器、server settlement、Codex bridge、runtime projection 和 completion gate 各自只检查异常或 transport error，没有共享业务失败判定。

建议修复

• 提供共享 isFailedToolResult/toolFailureReason，识别 {error}、{ok:false}、{aborted:true} 等。
• 浏览器桥以 error 通道提交业务失败。
• store settlement 拒绝失败形状，并将 ledger 写为 terminal_failure。
• Codex settleToolResult.success=false。
• run 内追踪未解决失败；只有同工具后续成功才能清除。
• completion gate 在仍有失败时将 run 标记 failed，并原样报告工具名和原因。

本地验证

覆盖 browser bridge、store、Codex turn、runtime ledger 和 completion gate 的回归测试均通过；失败操作不会再被记录为成功。

[0xsline]

已在 main 修复，落在 c6f6db7。

• 各层统一使用 isFailedToolResult / toolFailureReason 识别 { error }、{ ok:false }、{ aborted:true } 等业务失败。
• 浏览器桥、server store、Codex settlement 与运行 ledger 均保持失败语义；Codex settle 写入 success=false。
• 未解决失败进入 terminal_failure，后续同工具成功结果才会清除失败记录。
• browser bridge、store、Codex turn、runtime ledger 和 completion gate 均补有回归。

验证：真实 Agent 已观察到工具业务失败只执行一次、明确报告失败且未记录成功；全量 typecheck、lint、test、build 均通过。

[0xsline]

这个在 main 上应该已经修好了，你当时建议的方案基本就是最后落地的方案。

src/agent/toolFailure.ts 提供了你提到的共享判定 isFailedToolResult / toolFailureReason，另外还有一个 ToolFailureTracker 维护未解决失败的集合，用于 completion gate。你列的五层现在都接了同一套判定，共 15 个文件：

• 浏览器执行器：server/agent-runs/browser-tool.ts
• server settlement：server/agent-runs/executor.ts、store-tools.ts
• Codex bridge：src/agent/codex/stream-events.ts、codex/runtime.ts、server/agent-runs/codex-turn.ts
• runtime projection：src/agent/runtime.ts、api-runtime.ts、serverRunToolExecutor.ts
• 持久化：src/agent/useAgentPersistence.ts、src/persist/projectStore.ts

对应的回归测试是 server/agent-runs/store-business-failure.verify.ts 和 src/agent/serverRunToolBusinessFailure.verify.ts，断言就是「业务失败对象走 error 通道而非 success」和「失败结果写入服务端账本」。

需要说明一下为什么这个 issue 拖到现在还开着：这两个测试文件从来没被运行过。测试运行器按 && 切分 package.json 的 test:serial，没写进去的文件就不会跑，而没有任何东西检查这一点。全仓有 55 个 verify 文件处于这个状态，这两个在其中。已在 be5c47ec 全部注册，并加了 scripts/verify-registration.verify.mjs 防止再犯。

所以修复其实早就进去了，只是一直没有人拿到验证信号。麻烦你在 v0.2.12 上按原步骤（非法 easing 的 edit_item、未关联片段的 analyze_music）复测一下，确认后我就关掉。