dsh-deepseek-web-login 0.6.34 → 0.6.35

package/CHANGELOG.md

@@ -2,6 +2,40 @@
 
 本项目大致遵循语义化版本；日期为本地时间。
 
+## 0.6.35 — 2026-10-02
+
+**纠正 0.6.34：真正的原因是 DSH 的工具返回是 `role: "tool"`，而我们没有这个分支。**
+
+0.6.34 我按"工具返回被折进一条普通 user 文本"去修的（判据是"assistant 发过工具调用 ⇒ 紧随的
+user 文本就是返回"）。**那个推断是猜的，而且我的证据本身是错的** —— 我用的
+`dev/dump-turn-input.mjs` 只打了 `user/message` / `assistant/message` / `request/header` / `step/end`
+**四种事件**，而工具返回记在第五种里：
+
+```
+type=tool/result  {"message":{"role":"tool",
+  "toolCallId":"call_c2cf288c12734112b11e",
+  "content":[{"type":"text","text":"2026-10-02 18:54:28 星期五\r\n"}],"isError":false}}
+```
+
+**`role: "tool"`** —— 不是 `role:'user'`，也不是 `tool-result` 块（测试夹具里长的是后者，真机不是）。
+我们原来**没有 `tool` 这个分支**，于是它掉进 user 分支 ⇒ 渲染成
+`User: 2026-10-02 18:54:28 星期五` ⇒ 模型以为"用户告诉了我时间"（它的思考原话：
+「**我没拿到工具结果，用户直接给了时间**。那就接受。」）⇒ 回答变成「收到，…」。
+
+**改法（这次是有证据的）**：`serializePromptParts` 增加 `role === 'tool'` 分支，
+用消息**自带的** `toolCallId` 与 `isError` 标成 `[Tool Result for <id>]`，
+并清掉 0.6.34 那条"预期下一个 user 文本是工具返回"的待办。
+0.6.34 的推断**保留为次要防线**（注释已改写，明确它不是主要依据），因为两种形状的期望输出一致、
+留着能覆盖我没能证伪的那种形状；且主分支会先清掉待办，它不会再误触发。
+
+**顺带修掉那个把我带偏的工具**：`dev/dump-turn-input.mjs` 增加 `tool/call` / `tool/result` 分支 ——
+**看不见工具返回的诊断，比没有诊断更危险**（它让我在错误前提上写了一版修复）。
+
+**用例 69 项**（新增 4 条，直接用手机会话日志里的确切形状）。
+⚠️ 写这 4 条时踩了一次**虚守卫**：第一版把 assistant 侧的调用 id 与 `toolCallId` 写成**同一个值**，
+于是关掉主分支时次要防线给出同样结果、用例**照样全绿**（变异只红 2 条而不是 4 条）。
+把两边的 id 改成不同值后，变异才真正红 4 条 —— 又一次"两个来源同一个值就测不出真伪"。
+
 ## 0.6.34 — 2026-10-02
 
 **修复：工具返回被当成「用户说的话」发给模型 —— 所以它回「收到，…」而不是回答问题。**

package/lib/index.js

@@ -1893,16 +1893,15 @@ function serializePromptParts(options) {
 	const protocol = toolSection ? `\n\n${protocolText}${toolSection}` : "";
 	const lines = [];
 	/**
-	* 上一条 assistant 消息里的工具调用 id —— 用来把**紧随其后的那条 user 文本**认成工具返回。
+	* ⚠️ **次要防线**（主路径是下面 `role === 'tool'` 那一段 —— 那才是有证据的权威形状）。
 	*
-	* 🔴 2026-10-02 现场（用户报「提示词把答案直接告诉模型了，所以它回『收到，…』」）：
-	* DSH 执行 `pwsh` 之后，把工具输出当成一条**普通的 user 文本消息**交给我们，于是我们照实
-	* 渲染成 `User: 2026-10-02 18:54:28 星期五` —— 模型读到的是"用户告诉了我时间"，
-	* 它自己的思考原话就是：「**我没拿到工具结果，用户直接给了时间**。那就接受。」
-	* 回答于是从"回答几点"变成"收到，…" —— 看着就像在复述一个已知答案。
+	* 这里兜的是另一种可能存在的形状：**工具返回被折进一条普通 `user` 文本消息**。
+	* 判据：在 agent 循环里 assistant 发出工具调用后**必须先有工具返回**才轮得到下一条用户输入，
+	* 所以紧跟其后的那条 user 文本只可能是工具返回。
 	*
-	* 判据为什么成立：在 agent 循环里，assistant 发出工具调用之后**必须先有工具返回**，
-	* 才轮得到下一条用户输入。所以紧跟其后的那条 user 文本，按构造只能是工具返回。
+	* ⚠️ 不要把它当成主要依据 —— 2026-10-02 我先用这个推断去修，后来读会话日志才发现
+	* 真机用的是 `role:'tool'`。推断能修好只是**碰巧**（两种形状的期望输出一致）。
+	* 主路径落地后 `pendingToolCallIds` 会被清空，这里不会再误触发。
 	*/
 	let pendingToolCallIds = [];
 	for (const message of options.messages ?? []) {
@@ -1921,6 +1920,13 @@ function serializePromptParts(options) {
 			else if (text.trim()) lines.push(`Assistant: ${text}`);
 			continue;
 		}
+		if (message.role === "tool") {
+			const body = flattenText(blocks).join("") || "(no output)";
+			const errorMark = message.isError === true ? " [ERROR]" : "";
+			lines.push(`[Tool Result${errorMark} for ${String(message.toolCallId ?? "")}]\n${body}`);
+			pendingToolCallIds = [];
+			continue;
+		}
 		const toolResults = blocks.filter((block) => block?.type === "tool-result");
 		const text = flattenText(blocks.filter((block) => block?.type !== "tool-result")).join("");
 		const imageMarks = blockImageMarks(blocks, options.keptImageKeys);
@@ -10442,7 +10448,7 @@ async function checkForUpdate(current, fetchImpl) {
 * 兜底常量与 package.json 的一致性由 `tests/check-smoke.mjs` 守着，不会漂。
 */
 /** 与 package.json 保持一致的兜底版本（由测试保证不会漂）。 */
-const FALLBACK_VERSION = "0.6.34";
+const FALLBACK_VERSION = "0.6.35";
 let cached;
 /** 本插件版本（如 `0.1.26`）。 */
 function pluginVersion() {