DeepSeek-V4-Flash 流式模式下所有工具调用报 unknown tool ""

[wangziwei734]

问题描述

使用 DeepSeek-V4-Flash（或任何以多个 SSE 分块返回工具调用数据的模型）时，所有工具调用都会立即失败，报错：

unknown tool ""

工具名始终为空字符串，导致在 dsh 中无法使用任何 Skill 或工具。

根本原因

packages/llm/llm-deepseek/src/translate.ts 中的 SSE 流式解析逻辑在每个分块到来时直接覆盖赋值 block.name 和 block.callId，而非累加。当后续分块携带空的 function.name 时，会将首个分块已解析好的工具名抹除。

// 当前写法 — 后续空字符串分块会覆盖已解析的工具名
if (call.id !== undefined) block.callId = call.id
if (call.function?.name !== undefined) block.name = call.function.name

修复方案

// 修复写法 — 安全累加所有分块
if (call.id !== undefined) block.callId = (block.callId ?? '') + call.id
if (call.function?.name !== undefined) block.name = (block.name ?? '') + call.function.name

验证结果

应用修复后，dsh 能正确从 DeepSeek-V4-Flash 的流式响应中解析出工具名（如 Glob、Read），并成功执行，不再报错。

环境： Windows 11 · DeepSeek-V4-Flash · SSE 流式传输

[wangziwei734]

工具名被解析成空字符串，导致任何 Skill 和工具都无法运行。

原因

packages/llm/llm-deepseek/src/translate.ts 在解析 SSE 流时，
每个新分块都会覆盖已有的工具名，而不是累加。
当后续分块的 function.name 为空时，就把之前解析好的名字抹掉了。

解决方案

修改 packages/llm/llm-deepseek/src/translate.ts 第 159–160 行：

- if (call.id !== undefined) block.callId = call.id
- if (call.function?.name !== undefined) block.name = call.function.name
+ if (call.id !== undefined) block.callId = (block.callId ?? '') + call.id
+ if (call.function?.name !== undefined) block.name = (block.name ?? '') + call.function.name

改完重新 pnpm run build 再重启 dsh 即可。

我的环境： Windows 11 · DeepSeek-V4-Flash · SSE 流式输出

[chenghu17]

这个解决方案对于hy3和longcat-2.0等模型不太适用，它们SSH流式输出时delta2的id和name会填充null，需要再补充对于null值的处理

[wangziwei734]

@chenghu17 非常感谢指正，这个边界情况确实抓得非常准！

如果部分模型在流式输出（SSE delta）时针对 id 和 name 填充了 null，由于 null !== undefined 在 JS 中为 true，原本的代码会将 null 隐式转换为字符串，最终导致拼接出类似 "Globnull" 这样破坏性的工具名。

为了彻底解决 null 和其他可能的异常类型，我将条件判断优化为通过严格校验 typeof === 'string'，这样无论它是 null、undefined 还是不规范的对象，都能被完美屏蔽，兼容性更好：

// packages/llm/llm-deepseek/src/translate.ts
// 优化前：
if (call.id !== undefined) block.callId = (block.callId ?? '') + call.id
if (call.function?.name !== undefined) block.name = (block.name ?? '') + call.function.name
// 优化后 (Handle null values for models like hy3 & longcat-2.0):
if (typeof call.id === 'string') block.callId = (block.callId ?? '') + call.id
if (typeof call.function?.name === 'string') block.name = (block.name ?? '') + call.function.name

再次感谢提醒！

[bestgopher]

所以我要怎么更新呢？

[wangziwei734]

这个只能自己去对应文件处修改，官方关闭了issue提交和PR。

[wangziwei734]

Just a heads up: this bug also causes severe secondary damage (bricking session history).

Since the tool call was persisted with an empty callId (""), when you restart and try to load the session log, it triggers a SessionPersistenceCorruptionError ("message must have tool source"). This happens because assertMessageEventShape in packages/core/session/src/index.ts strictly rejects callId === ''.

So this bug not only breaks the current execution but also permanently corrupts the session file. To rescue corrupted sessions, we should relax the empty string check in packages/core/session/src/index.ts:

--- packages/core/session/src/index.ts
+++ packages/core/session/src/index.ts
@@ -336,8 +336,7 @@
   if (type !== 'tool/result') return
   if (sourceRecord['kind'] !== 'tool'
-    || typeof sourceRecord['callId'] !== 'string'
-    || sourceRecord['callId'] === '') {
+    || typeof sourceRecord['callId'] !== 'string') {
     throw new Error(`${subject} message must have tool source`)
   }

补充一个次生灾害及修复方案： 这个 Bug 会导致已经保存的历史会话完全损坏、无法加载（触发 SessionPersistenceCorruptionError）。因为带有空 callId 的异常记录被写入后，重启读取时会被 assertMessageEventShape 的严格校验拦截。要拯救已经坏档的会话，只需按上面的 Diff 移除对空字符串的限制，放宽校验即可正常载入历史脏数据。

[Electricitysheep]

这个根因我们也确认过（手册 FAQ 已收录）：translate.ts 里 SSE 分块对 block.name/block.callId 做覆盖赋值而非累加，后续分块带空 function name 时把工具名抹成空串（#161 同族）。

官方修复前的规避：

1. 降级到非流式模式（如果有配置开关）
2. 或者等 rc 修复——模型会反复重试，注意及时中止避免浪费 token

排查思路和同族 bug 汇总见：https://github.com/Electricitysheep/dsh-handbook/blob/main/docs/faq.md

[enjlife]

修复如下：

https://github.com/enjlife/deepseek-harness/pull/new/fix/llm-deepseek-empty-tool-name