dsh-deepseek-web-login 0.6.33 → 0.6.35

package/CHANGELOG.md

@@ -2,6 +2,79 @@
 
 本项目大致遵循语义化版本；日期为本地时间。
 
+## 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
+
+**修复：工具返回被当成「用户说的话」发给模型 —— 所以它回「收到，…」而不是回答问题。**
+
+用户现场（原话）：「提示词还是把答案直接告诉模型了，难怪最后模型的回答有『收到，…』」
+（附分享页 + 截图：问「你知道现在几点嘛」→ 答「**收到**，2026-10-02 星期五 18:54，傍晚快七点了。」）
+
+先排除掉已修的那一半：0.6.33 确实在跑（日志已有 `firstDiff`），而且**增量已经正常** ——
+`18:51:37 chars=17` / `18:54:09 chars=11` / `18:54:26 chars=14` / `18:54:30 chars=31`，
+对比修之前的 **40193**。所以这次的「答案在提示词里」不是回声。
+
+**证据链**（`dev/dump-turn-input.mjs` 读 DSH 自己的会话日志 + 分享页逐行核对）：
+
+```
+turn 3  step 1:  user      「你知道现在几点嘛」
+                 assistant 「pwsh」            ← 工具调用
+         step 2:  assistant 「收到，2026-10-02 星期五 18:54，傍晚快七点了。」
+```
+
+**中间那条工具返回，在 DSH 的消息列表里根本不存在**（不是 `tool-result` 块）。
+而分享页里渲染出来的是 `User: 2026-10-02 18:54:28 星期五` ——
+**`User: ` 前缀只可能由我们的"纯文本分支"产生** ⇒ DSH 把 pwsh 的输出当成一条**普通 user 文本**交给我们。
+
+于是模型读到的是"用户告诉了我时间"。它自己的思考原话（分享页里看得到）：
+
+> 「用户告诉我时间。不需要工具。」
+> 「不过我应该核对一下——**我没拿到工具结果，用户直接给了时间**。那就接受。」
+
+⇒ 回答从"回答几点"变成"收到，…"。**用户看到的"答案被喂进去"是真的，但它的根因不是重发，是错标。**
+
+**修法**：`serializePromptParts` 认出这种形状 —— **assistant 发过工具调用 ⇒ 紧随其后的那条 user 文本
+按构造只能是工具返回**（agent 循环里必须先有工具返回才轮到下一条用户输入），于是标成
+`[Tool Result for <id>]`，不再渲染成 `User: …`。
+一对多时逐个配对（一个调用一条返回）；数量对不齐就合并成一条（宁可少一层对应关系，
+也不能让它看起来像用户说的话）；**带图片的 user 消息不误标**（那是用户真的发的）。
+
+**新增 5 条用例**（`check-context-feed` 65 项）：必须标成工具返回 / 没有工具调用时不许误标 /
+多调用多返回逐个配对 / 数量不齐时合并 / 带图消息不误标。**一处变异确认变红**（关掉这条判据 → 3 红）。
+
 ## 0.6.33 — 2026-10-02
 
 **修复：断链重发时把模型自己刚说的话又喂了回去 —— 一句话走掉 4 万字符。**

package/lib/index.js

@@ -1892,6 +1892,18 @@ function serializePromptParts(options) {
 	const toolSection = buildToolSection(options.tools, toolBudget);
 	const protocol = toolSection ? `\n\n${protocolText}${toolSection}` : "";
 	const lines = [];
+	/**
+	* ⚠️ **次要防线**（主路径是下面 `role === 'tool'` 那一段 —— 那才是有证据的权威形状）。
+	*
+	* 这里兜的是另一种可能存在的形状：**工具返回被折进一条普通 `user` 文本消息**。
+	* 判据：在 agent 循环里 assistant 发出工具调用后**必须先有工具返回**才轮得到下一条用户输入，
+	* 所以紧跟其后的那条 user 文本只可能是工具返回。
+	*
+	* ⚠️ 不要把它当成主要依据 —— 2026-10-02 我先用这个推断去修，后来读会话日志才发现
+	* 真机用的是 `role:'tool'`。推断能修好只是**碰巧**（两种形状的期望输出一致）。
+	* 主路径落地后 `pendingToolCallIds` 会被清空，这里不会再误触发。
+	*/
+	let pendingToolCallIds = [];
 	for (const message of options.messages ?? []) {
 		if (!message || typeof message !== "object") continue;
 		const blocks = Array.isArray(message.content) ? message.content : [];
@@ -1903,14 +1915,31 @@ function serializePromptParts(options) {
 		if (message.role === "assistant") {
 			const text = flattenText(blocks).join("");
 			const renderedCalls = renderToolCalls(blocks);
+			pendingToolCallIds = renderedCalls ? blocks.filter((block) => block?.type === "tool-call").map((block) => String(block.toolCallId ?? block.id ?? "")) : [];
 			if (renderedCalls) lines.push(`Assistant: ${renderedCalls}`);
 			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);
 		const images = imageMarks.length;
+		if (toolResults.length === 0 && pendingToolCallIds.length > 0 && images === 0 && text.trim()) {
+			const textBlocks = blocks.filter((block) => block?.type === "text" && typeof block.text === "string" && block.text.trim()).map((block) => block.text);
+			const ids = pendingToolCallIds;
+			pendingToolCallIds = [];
+			if (textBlocks.length === ids.length && ids.length > 0) for (let i = 0; i < ids.length; i += 1) lines.push(`[Tool Result for ${ids[i]}]\n${textBlocks[i]}`);
+			else lines.push(`[Tool Result for ${ids.filter(Boolean).join(", ")}]\n${text}`);
+			continue;
+		}
+		pendingToolCallIds = [];
 		if (text.trim() || toolResults.length === 0 && images === 0 || images > 0) {
 			const imageNote = imageMarks.length > 0 ? `\n${imageMarks.join(" ")}` : "";
 			lines.push(`User: ${text}${imageNote}`);
@@ -10419,7 +10448,7 @@ async function checkForUpdate(current, fetchImpl) {
 * 兜底常量与 package.json 的一致性由 `tests/check-smoke.mjs` 守着，不会漂。
 */
 /** 与 package.json 保持一致的兜底版本（由测试保证不会漂）。 */
-const FALLBACK_VERSION = "0.6.33";
+const FALLBACK_VERSION = "0.6.35";
 let cached;
 /** 本插件版本（如 `0.1.26`）。 */
 function pluginVersion() {