[bug] ToolArgsError 不带工具名，code 模式下 run_code/bash 同名 description 死循环 — 附可 cherry-pick 修复

[ericcaiwx-star]

摘要 / Summary

run_code 和 bash 都把 UI 摘要字段叫 description 且标成 required。校验失败时两边都报同一句 invalid arguments: missing required property "description"。Code Mode 下模型常把内层 bash.description 当成已经传过，外层 JSON 只剩 {"code":"..."}，然后按「harness 丢了字段」死循环重试。

Same class as #558. Minimal fix: put the tool name in ToolArgsError.

复现 / Repro

code preset 下让模型跑需要 bash 的诊断。失败调用几乎都是：

{"code":"const out = await tools.bash({\n command: \"...\",\n description: \"Check port listeners\"\n});\nreturn out.stdout.text;"}

外层没有 description。报错：Error: invalid arguments: missing required property "description"。内层缺字段时文案完全一样。

根因 / Root cause

ToolArgsError 只拼 violations，不带工具名。defineTool 的 run_code 与 bash 校验失败走同一句话。模型无法区分「外层缺字段」和「内层缺字段」，reminder 回显规范化后的 {"code":"..."}，于是换 command 重试，成功率极低。

修复 / Fix (cherry-pick ready)

• 分支：https://github.com/ericcaiwx-star/deepseek-harness/tree/fix/tool-args-error-names-tool
• commit：b5de96e5de — fix(tools): name the tool in ToolArgsError messages
• 对比：master...ericcaiwx-star:deepseek-harness:fix/tool-args-error-names-tool

消息改为 invalid arguments for <tool>: …。外层缺字段报 run_code，内层报 bash。structured_output 同步带上工具名。

未改 description 是否 required（那是更大的产品决策）。本补丁只消歧义，打破错误归因。

验收 / Acceptance

pnpm exec vitest run packages/core/tools/tests/tools.spec.ts packages/subagent/subagent-in-process-driver/tests 通过。缺 run_code.description 与缺 bash.description 的报错不再相同。

[zhou256bug]

我是外面看这条的开发者，看了 fix/tool-args-error-names-tool。

ToolArgsError 只拼 violations、不带工具名，再加上 run_code 和 bash 都把 description 标成 required，模型确实容易把外层校验失败当成内层缺字段。给 message 加上 for <tool> 是消歧义，这个方向我认同。structured_output 那边一并带上名字也合理。

两点想确认：

1. 现有单测主要是改了通用 reader / demo 的文案，没有直接断言「缺 run_code.description」和「缺 bash.description」不再相同。如果这是这条要修的现场，最好补一条对照。
2. 这是模型可见字符串变更。我不会把它写成运行时死循环——更像是归因失败之后的低成功率重试，直到 turn 上限。

description 还要不要 required，我觉得不该和这条绑在一起。

[ericcaiwx-star]

谢谢，两点都对。

1. 已补对照：packages/core/tools/tests/code-mode.spec.ts 在 mode: 'both' 下同时打缺 description 的 run_code 和 bash，断言

   • invalid arguments for run_code: missing required property "description"
   • invalid arguments for bash: missing required property "description"
   • 两条 message 不相等
     commit：c3585f1252（同一分支 fix/tool-args-error-names-tool）。run_code 不能当普通工具注册，所以对照放在真实 transport 旁边，而不是 reader/demo。

2. 同意用词：这不是运行时死循环，是归因失败后的低成功率重试，直到 turn 上限。description 还要不要 required，不绑在这条里。

[zylos-luna]

Verified that ToolArgsError in master carries only the violation list, so a defineTool-validated bash and the run_code dispatcher emit byte-identical messages for a missing required description — the model has no signal to attribute the failure to the outer vs inner call. Putting the tool name in the message (and on the error as toolName) is the minimal disambiguation, and the follow-up contrast test in code-mode.spec.ts pins the two messages as distinct where it matters, next to the real transport. Both in-repo construction sites (schema.ts and the structured-output runtime) are updated. One note for maintainers: the constructor signature change is source-breaking for any out-of-tree code constructing ToolArgsError directly — fine under the pre-release stance, worth a changelog line at release.

[Xilink-Lin]

run_code 和 bash 都把 description 标成 required，校验失败走同一句 invalid arguments: missing required property "description"。Code Mode 里模型把内层 bash.description 当成已经传过，外层只剩 {"code":"..."}，两边报错字节级相同。给 ToolArgsError 加上 for ${toolName}，并在 code-mode.spec.ts 做对照断言，这是最小消歧义。description 还要不要 required，不该绑在这条里。

代价：构造函数从 (violations) 改成 (toolName, violations)，对仓库外直接 new ToolArgsError 的代码是源码破坏。rc 阶段可接受，发版 changelog 写一行即可。若想更软，第二个参数可选、旧签名保留一段时间也行，但没必要为 rc 做。