dsh 工具调用落库时 name/callId 为空，导致 unknown tool 死循环

[dreamor]

dsh 工具调用落库时 name/callId 为空，执行端报 unknown tool "" 并无限重试。

复现、预期与验收

• 复现步骤：
  1. provider 配置为 deepseek-official，baseURL 指向 OpenAI 兼容网关（如 https://ai-hub.ebanma.com/v1），模型 deepseek-v4-flash
  2. 跑一个需要调用工具的回合，观察 session 日志：assistant/chunk 的 tool-call-delta 带 id/name（如 run_code），但落库的 tool/call 事件 callId 与 name 均为空字符串
  3. 执行端返回 Error: unknown tool ""（ToolNotFoundError / UNKNOWN_TOOL），模型拿不到有效结果继续重试 → 单回合出现 161 次空名调用死循环
• 实际结果：所有 tool/call 事件 callId="" name=""，全部被 UNKNOWN_TOOL 拒绝
• 预期结果：工具调用按 name 正常路由执行
• 环境：@deepseek-ai/cordis@4.0.1、@deepseek-ai/dsh-llm-deepseek@0.1.1-rc.2（npm next）、loongsuite-pilot agentshell 1.1.27
• 验收条件：translate.ts 对空 name 的 tool-call block 改为拒绝（MALFORMED_RESPONSE）或发出明确错误，并把现有 unknown-tool snapshot gate 扩展覆盖空 name 变体

补充技术定位：

packages/llm/llm-deepseek/src/translate.ts 只有 !== undefined 才赋值 name，空字符串会被原样接受，收尾 name: block.name ?? ''：

if (call.function?.name !== undefined) block.name = call.function.name
...
case 'tool-call': return {
  type: 'tool-call',
  id: ToolCallId(block.callId ?? ''),
  name: block.name ?? '',   // 空字符串存活
  arguments: block.text,
}

packages/core/agent-loop/src/tool-calls.ts 的 appendToolCall 原样持久化 block.name：

session.append('tool/call', { turn, step, callId: block.id, name: block.name, arguments: block.arguments })

已确认该逻辑在发布版 @deepseek-ai/dsh-llm-deepseek@0.1.1-rc.2 与当前 master（dsh-0.1.2-alpha.1，2026-08-28）中相同，尚未修复。

[denial123789]

The handbook has two relevant operator paths for this failure: streamed tool identity corruption (trace empty callId/name before policy) and runaway-loop containment (cancel first, preserve durable events, and do not retry malformed calls).\n\n- https://github.com/sandbaseai/deepseek-harness-handbook/blob/main/docs/en/troubleshooting/streamed-tool-call-empty-identity.md\n- https://github.com/sandbaseai/deepseek-harness-handbook/blob/main/docs/en/troubleshooting/runaway-agent-loop.md

[weijiafu14]

这个死循环是已被系统测量过的家族，补跨适配器数据（含你要的会话不可读角度——同族 #2674/#4611）：

我们把这一族畸形 delta（空串/null 的 id、name 续块，真身晚到，全程无 id 的并行交错，无 index 按 id 续块）逐一注入真 pi-ai openai-completions 累加器（client/SSE 解析/组装全部未修改，只有线上字节是手造的）：全部正确组装。免疫的机理：块按 index 优先键控，id/name 首个非空胜出——后到的空值覆盖不了已收到的真身。最新 0.84.4 复测同样成立。逐案判决：accumulator-experiment.mjs / 结果；真机侧真 api.deepseek.com 上双工具调用 id 独立非空的活证据：provider-threads-e2e.json。

这条量出来的规则（index 优先 + 首个非空胜出）可以直接当 llm-deepseek 修复的语义蓝本——同族报告 #3090（诊断最深）、#2674（会话历史角度）都收敛到这里。边界说明：这不修 llm-deepseek；落地前，自定义声明走 llm-pi-ai 路由骑的是免疫累加器（写法）。公平起见也说这条路已知的坑：两个调用共用同一 index 时它会合并丢一个——已报上游 earendil-works/pi#8861。

[dreamor]

复测 + 最小补丁（对应验收条件 a）

复测（master / @deepseek-ai/dsh-llm-deepseek@0.1.2-alpha.2，2026-08-30 发版）仍未修，packages/llm/llm-deepseek/src/translate.ts 与原帖引用字节级一致：

81: id: brandString<ToolCallId>(block.callId ?? ''),
82: name: block.name ?? '',                        // 空串存活 → 落库 tool/call{name:""} → unknown tool "" 死循环
…
169: if (call.id !== undefined) block.callId = call.id
170: if (call.function?.name !== undefined) block.name = call.function.name   // 后到的 "" 会写入并覆盖已 latch 的真身

该仓库 pulls 端点返回 404、CONTRIBUTING.md 亦载明早期阶段暂不接受外部 PR，无 PR 通道，故只能在此内联补丁供维护者参考。

与 @weijiafu14 在 #2 量出的免疫规则对齐：本路径缺的正是「按 index 键控 + 首个非空 id/name 胜出 + 后到空值不覆盖」这层——当前 L169-170 只判 !== undefined，空串既会被写入也会覆盖真身。

最小补丁（finalize 前对空 name 直接 MALFORMED_RESPONSE，不再落库 → 不再死循环；LlmError 已在 L12 导入，无需新增 import）：

   for (const call of delta?.tool_calls ?? []) {
     let block = toolBlocks.get(call.index)
     if (!block) {
       block = open('tool-call')
       toolBlocks.set(call.index, block)
       yield { type: 'block-start', index: block.index, blockType: 'tool-call' }
     }
-    if (call.id !== undefined) block.callId = call.id
-    if (call.function?.name !== undefined) block.name = call.function.name
+    // immunity: never write/overwrite with an empty id or name (first non-empty wins; later "" can't clobber)
+    if (call.id !== undefined && call.id.length > 0) block.callId = call.id
+    if (call.function?.name !== undefined && call.function.name.length > 0) block.name = call.function.name

     if (payload === DONE) {
+      // reject any tool-call that finished without a name BEFORE emitting block-ends,
+      // so no empty-name tool/call event is ever persisted (kills the unknown-tool "" death loop)
+      for (const block of order) {
+        if (block.kind === 'tool-call' && (block.name === undefined || block.name.length === 0)) {
+          throw new LlmError(
+            `tool-call at index ${block.index} finished without a tool name`
+              + ` (callId=${block.callId ?? '<empty>'}); aborting as MALFORMED_RESPONSE`,
+            'MALFORMED_RESPONSE',
+          )
+        }
+      }
       for (const block of order) {
         yield { type: 'block-end', index: block.index, block: closeBlock(block) }
       }

可选第二层（验收条件 b，defense-in-depth）：packages/core/tools 的 unknown-tool 判定处把空 name 与 UNKNOWN_TOOL 同等拦截，作 core 侧兜底。

临时止血（落地前）：自定义 OpenAI 兼容网关改走 llm-pi-ai 路由借用其免疫累加器（同类调用共用同一 index 会被合并吞一个，见 earendil-works/pi#8861）。

同族 #3090 / #2674 同根因、同样待修。