[Bug] llm-pi-ai forwards empty tool id/name, crashing the session write (assertV4ToolResultMessage)

[leo-liushuhong]

摘要

当 DeepSeek 模型通过第三方网关（wdtrip-gateway）以 Anthropic Messages 协议（api: "anthropic-messages"）对外提供、并由 harness 的 pi-ai 适配器（kind: "pi-ai", version: 2）消费时，网关偶尔会下发一个 id 与 name 均为空字符串的 tool_use 块（模型在 reasoning 里输出 DSML 标记、网关误判为工具调用所致）。

llm-pi-ai 适配器把这个空 id/name 原样透传成 harness 的 tool-call 块，于是：

1. tool/call 事件携带 callId: ""；
2. 派发失败（unknown tool ""），ToolCallRecovery 生成 tool/result 时 toolCallId 也为空字符串；
3. V4 会话格式校验 assertV4ToolResultMessage 拒绝 toolCallId.length === 0，报错 format v4 tool/result at seq N requires toolCallId matching its tool source，该 tool/result 事件无法写入会话，本轮强制终止。

关键点：llm-deepseek 适配器早已对空工具身份做了防御（if (!content.id || !content.name) return malformed('empty tool identity')），但 llm-pi-ai 适配器缺失同一道防线——这是两个适配器之间的不一致，正是本次崩溃的根因。

复现条件

项	值
网关 / 协议	wdtrip-gateway，api: "anthropic-messages"
适配器	kind: "pi-ai", version: 2
模型	deepseek-v4-pro-huoshan[1m]（响应模型 deepseek-v4-pro-ga-260813）
会话	session-20fc753c-164e-48af-bf2a-b742355cec42

表现症状（证据）

会话日志尾部展示了「占位名不崩、空 id 才崩」的精确对照：

• seq 112 assistant/message 含 tool-call id="call_8uwxpajwomekefw1vnwkofrj" name="tool_name"（占位名 tool_name、真实非空 id）；
• seq 114 tool/result 正常写入（content 为 Error: unknown tool "tool_name"）——非空 id 走「未知工具」路径，不崩溃；
• seq 117 assistant/message 含 tool-call id="" name=""（空 id 与空 name）；
• seq 118 tool/call callId="" name=""；
• seq 119 的 tool/result 缺失——写入被 assertV4ToolResultMessage 拒绝，没有 step/end、没有 turn/end，会话在此强制终止。

错误信息在本会话中出现 41 次：format v4 tool/result at seq … requires toolCallId matching its tool source。

根因分析

崩溃链路（全部在官方 master 上确认）：

1. pi-ai 适配器不校验工具身份：packages/llm/llm-pi-ai/src/stream.ts 的 toolcall_end 分支直接把 event.toolCall.id / event.toolCall.name 原样写入 harness 的 tool-call 块，id 可以为空字符串。
2. assembler 的兜底 ?? 拦不住空串：packages/llm/llm/src/assembler.ts 里 id: partial.toolCallId ?? brandString(\call-${index}`)用??，只拦 undefined，不拦空串 ""`。
3. agent-loop 沿用空 id：packages/core/agent-loop/src/tool-calls.ts 的 callId = block.id、ToolCallRecovery 的 toolCallId = callId 都直接继承空 id。
4. V4 校验拒绝空 id：packages/session/session-format-v3-to-v4/src/tool-role.ts 的 assertV4ToolResultMessage 里 toolCallId.length === 0 抛 SessionFormatError——报错文案「requires toolCallId matching its tool source」其实由空 id 分支触发，具有误导性。

不一致点（关键）：packages/llm/llm-deepseek/src/translate.ts 在 tool_use 分支已有 if (!content.id || !content.name) return malformed('empty tool identity')（抛 MALFORMED_RESPONSE，且该 code 不在默认重试清单内，不会被重试放大成循环）。llm-pi-ai 缺失同一防线，导致同样类别的畸形响应一个被拒、一个崩溃。

修复方案

在 packages/llm/llm-pi-ai/src/stream.ts 的 toolcall_end 分支、yield block-end 之前加入空工具身份校验，与 llm-deepseek 对齐：

if (event.toolCall.id.length === 0 || event.toolCall.name.length === 0) {
  throw new LlmError('pi-ai tool call has empty tool identity', 'MALFORMED_RESPONSE')
}

MALFORMED_RESPONSE 不在默认重试清单（retryableCodes = [EMPTY_RESPONSE, RATE_LIMIT, SERVER, TIMEOUT, TRANSPORT]），不会把畸形响应放大成重试循环。

说明：本修复基于官方 master（22ddee37）。master 的 stream.ts 相对 rc.2 已新增「argumentsError 延迟到 done 抛 MALFORMED_RESPONSE、并在 stopReason === 'length' 时跳过」的逻辑。本修复的空身份校验放在该分支之前立即抛出——空 id/name 属于 provider 协议违规（wire 载荷畸形），而非 length 截断，应即时拒绝，与新增的 argumentsError 延迟路径可以共存。

修复代码见 fork 分支：leo-liushuhong/deepseek-harness@fix/llm-dsml-leak-and-empty-tool-identity（提交 9a2ced4d）

修复后的验证

1. 单元测试（packages/llm/llm-pi-ai/tests/convert.spec.ts）：新增 it.each 两例（空 id、空 name）断言抛 MALFORMED_RESPONSE，vitest -t "empty tool identity" → 2 例全部通过。
2. 静态检查：tsx scripts/build.ts --scope typecheck 全仓库 5 个 gate 全部通过（0 failed）。
3. 提交：9a2ced4d，pre-commit 门（staged lint、whitespace、vendor manifest）全部通过。

附注：master 上 convert.spec.ts 另有 3 例与本修复无关的既有失败（retains signed thinking … 期望 content: [] 与代码不符），在干净 master 上同样失败，非本修复引入。