fix(llm): reject responses function calls without ids

Author: MouseWW

RELEASE_NOTES.md

@@ -1,16 +1,16 @@
-# Anything Analyzer v3.6.36
+# Anything Analyzer v3.6.37
 
 ## 修复
 
-- **Anthropic/MiniMax 非流式文本格式校验** — 避免 text content 不是字符串时被拼成 `[object Object]`
-  - 普通请求和工具调用最终轮次复用同一段 Anthropic 文本块校验逻辑
-  - 新增回归测试覆盖非字符串 text content 的异常路径
+- **Responses API 工具调用格式校验** — 避免缺少 call_id 的 function_call 继续进入工具执行轮次
+  - 工具调用入站阶段直接拒绝缺失 call_id 的畸形响应
+  - 新增回归测试覆盖 Responses API function_call 缺少调用 ID 的异常路径
 
 ## 下载
 
 | 平台 | 文件 |
 |------|------|
-| Windows | Anything-Analyzer-Setup-3.6.36.exe |
-| macOS (Apple Silicon) | Anything-Analyzer-3.6.36-arm64.dmg |
-| macOS (Intel) | Anything-Analyzer-3.6.36-x64.dmg |
-| Linux | Anything-Analyzer-3.6.36.AppImage |
+| Windows | Anything-Analyzer-Setup-3.6.37.exe |
+| macOS (Apple Silicon) | Anything-Analyzer-3.6.37-arm64.dmg |
+| macOS (Intel) | Anything-Analyzer-3.6.37-x64.dmg |
+| Linux | Anything-Analyzer-3.6.37.AppImage |

package.json

@@ -1,6 +1,6 @@
 {
   "name": "anything-analyzer",
-  "version": "3.6.36",
+  "version": "3.6.37",
   "description": "Universal web protocol analyzer with embedded browser and AI-powered analysis",
   "packageManager": "pnpm@10.24.0",
   "main": "./out/main/index.js",

src/main/ai/llm-router.ts

@@ -570,13 +570,14 @@ export class LLMRouter {
           onChunk(`\n\n> 🔧 调用工具: ${toolNames}\n\n`);
         }
 
-        for (const fc of functionCalls) {
-          let result: string;
-          try {
-            if (!fc.name) throw new Error("function_call missing name");
-            const args = JSON.parse(fc.arguments || "{}");
-            result = await callTool(fc.name, args);
-          } catch (err) {
+        for (const fc of functionCalls) {
+          let result: string;
+          if (!fc.id) throw new Error("function_call missing call_id");
+          if (!fc.name) throw new Error("function_call missing name");
+          try {
+            const args = JSON.parse(fc.arguments || "{}");
+            result = await callTool(fc.name, args);
+          } catch (err) {
             result = `Error: ${err instanceof Error ? err.message : String(err)}`;
           }
           input.push({

tests/main/ai/llm-router.test.ts

@@ -537,6 +537,25 @@ describe("LLMRouter", () => {
         ),
       ).rejects.toThrow("Responses API incomplete: max_output_tokens");
     });
+
+    it("should reject Responses API function calls without call id", async () => {
+      const config: LLMProviderConfig = { ...baseConfig, apiType: "responses" };
+      fetchSpy.mockResolvedValueOnce(
+        createJSONResponse({
+          output: [{ type: "function_call", name: "lookup", arguments: "{}" }],
+        }),
+      );
+
+      const router = new LLMRouter(config);
+
+      await expect(
+        router.completeWithTools(
+          [{ role: "user", content: "test" }],
+          [{ name: "lookup", description: "Lookup", inputSchema: { type: "object" } }],
+          async () => "unused",
+        ),
+      ).rejects.toThrow("function_call missing call_id");
+    });
   });
 
   describe("completeWithTools - Anthropic", () => {