惯性聚合 高效追踪和阅读你感兴趣的博客、新闻、科技资讯
阅读原文 在惯性聚合中打开

推荐订阅源

阮一峰的网络日志
阮一峰的网络日志
Jina AI
Jina AI
GbyAI
GbyAI
D
DataBreaches.Net
人人都是产品经理
人人都是产品经理
Hugging Face - Blog
Hugging Face - Blog
V
Visual Studio Blog
P
Proofpoint News Feed
The Cloudflare Blog
H
Help Net Security
MyScale Blog
MyScale Blog
T
The Blog of Author Tim Ferriss
量子位
博客园 - 聂微东
Apple Machine Learning Research
Apple Machine Learning Research
T
Tailwind CSS Blog
博客园 - 三生石上(FineUI控件)
钛媒体:引领未来商业与生活新知
钛媒体:引领未来商业与生活新知
让小产品的独立变现更简单 - ezindie.com
让小产品的独立变现更简单 - ezindie.com
MongoDB | Blog
MongoDB | Blog
Last Week in AI
Last Week in AI
大猫的无限游戏
大猫的无限游戏
小众软件
小众软件
月光博客
月光博客

Recent Commits to openclaw:main

test: merge chat side-result checks · openclaw/openclaw@ddd2c2a test: merge cron history checks · openclaw/openclaw@f7eb746 test: merge responsive navigation shell checks · openclaw/openclaw@c2e4b47 docs(changelog): add codex oauth fixes · openclaw/openclaw@628e6cd test: merge navigation routing cases · openclaw/openclaw@5d8cecb Tests: mock channel registry bundled fallback · openclaw/openclaw@2b08233 Secrets: avoid broad web search discovery for single plugin config · openclaw/openclaw@a464f59 test: merge config view browser checks · openclaw/openclaw@20cf511 fix(status): align oauth health with runtime · openclaw/openclaw@eed7116 feat: add macOS screen snapshots for monitor preview (#67954) thanks … · openclaw/openclaw@f377db1 fix: report shared auth scopes in hello-ok (#67810) thanks @BunsDev · openclaw/openclaw@0b6c39b Auto-reply: avoid eager bundled route fallback · openclaw/openclaw@3ea1bf4 Tests: narrow session binding contract setup · openclaw/openclaw@54e4e16 fix(macOS): enable undo/redo in webchat composer text input (#34962) · openclaw/openclaw@00951dc Tests: speed up channel setup promotion · openclaw/openclaw@82b529a Docs: refresh agent instructions · openclaw/openclaw@5775fe2 fix(auth): serialize OAuth refresh across agents to fix #26322 (#67876) · openclaw/openclaw@8e79080 test: allow ollama public surface boundary test · openclaw/openclaw@7d4f1a6 Docs: add test performance guardrails · openclaw/openclaw@89706d3 Tests: restore context-engine usage proof · openclaw/openclaw@e4c4f95 Tests: slim context engine runtime coverage · openclaw/openclaw@74c198f ci: retry failed custom checkouts · openclaw/openclaw@0ee5baf test: trim duplicate provider auth onboarding cases · openclaw/openclaw@1ffc02e matrix: fix sessions_spawn --thread subagent session spawning (#67643) · openclaw/openclaw@1ce2596 test: reduce auth choice fixture churn · openclaw/openclaw@857b9cd test: mock health status config boundaries · openclaw/openclaw@9d5ab4a test: mock onboard config io boundary · openclaw/openclaw@299694d test: mock legacy state plugin boundaries · openclaw/openclaw@2713089 test: mock channel install boundaries · openclaw/openclaw@b945248 test: mock doctor preview channel boundaries · openclaw/openclaw@b1a3ad4
fix(ollama): preserve tool call ids [AI-assisted] (#84855...
IWhatsskill · 2026-05-21 · via Recent Commits to openclaw:main

@@ -693,7 +693,27 @@ describe("convertToOllamaMessages", () => {

693693

expect(result[0].role).toBe("assistant");

694694

expect(result[0].content).toBe("Let me check.");

695695

expect(result[0].tool_calls).toEqual([

696-

{ function: { name: "bash", arguments: { command: "ls" } } },

696+

{ id: "call_1", function: { name: "bash", arguments: { command: "ls" } } },

697+

]);

698+

});

699+700+

it("preserves assistant tool-call ids before Ollama replay", () => {

701+

const messages = [

702+

{

703+

role: "assistant",

704+

content: [

705+

{

706+

type: "toolCall",

707+

id: "fc_ollama_123",

708+

name: "bash",

709+

arguments: { command: "pwd" },

710+

},

711+

],

712+

},

713+

];

714+

const result = convertToOllamaMessages(messages);

715+

expect(result[0].tool_calls).toEqual([

716+

{ id: "fc_ollama_123", function: { name: "bash", arguments: { command: "pwd" } } },

697717

]);

698718

});

699719

@@ -709,8 +729,8 @@ describe("convertToOllamaMessages", () => {

709729

];

710730

const result = convertToOllamaMessages(messages);

711731

expect(result[0].tool_calls).toEqual([

712-

{ function: { name: "exec", arguments: { command: "pwd" } } },

713-

{ function: { name: "read", arguments: { path: "README.md" } } },

732+

{ id: "call_1", function: { name: "exec", arguments: { command: "pwd" } } },

733+

{ id: "call_2", function: { name: "read", arguments: { path: "README.md" } } },

714734

]);

715735

});

716736

@@ -729,9 +749,9 @@ describe("convertToOllamaMessages", () => {

729749

availableToolNames: new Set(["tool_a", "tools_invoke_test", "function-run"]),

730750

});

731751

expect(result[0].tool_calls).toEqual([

732-

{ function: { name: "tool_a", arguments: { value: 1 } } },

733-

{ function: { name: "tools_invoke_test", arguments: { value: 2 } } },

734-

{ function: { name: "function-run", arguments: { value: 3 } } },

752+

{ id: "call_1", function: { name: "tool_a", arguments: { value: 1 } } },

753+

{ id: "call_2", function: { name: "tools_invoke_test", arguments: { value: 2 } } },

754+

{ id: "call_3", function: { name: "function-run", arguments: { value: 3 } } },

735755

]);

736756

});

737757

@@ -750,9 +770,9 @@ describe("convertToOllamaMessages", () => {

750770

availableToolNames: new Set(["exec", "read"]),

751771

});

752772

expect(result[0].tool_calls).toEqual([

753-

{ function: { name: "exec", arguments: { command: "pwd" } } },

754-

{ function: { name: "read", arguments: { path: "." } } },

755-

{ function: { name: "tool_missing", arguments: {} } },

773+

{ id: "call_1", function: { name: "exec", arguments: { command: "pwd" } } },

774+

{ id: "call_2", function: { name: "read", arguments: { path: "." } } },

775+

{ id: "call_3", function: { name: "tool_missing", arguments: {} } },

756776

]);

757777

});

758778

@@ -770,10 +790,10 @@ describe("convertToOllamaMessages", () => {

770790

];

771791

const result = convertToOllamaMessages(messages);

772792

expect(result[0].tool_calls).toEqual([

773-

{ function: { name: "functionshell", arguments: {} } },

774-

{ function: { name: "tooling", arguments: {} } },

775-

{ function: { name: "tools", arguments: {} } },

776-

{ function: { name: "tool_a", arguments: {} } },

793+

{ id: "call_1", function: { name: "functionshell", arguments: {} } },

794+

{ id: "call_2", function: { name: "tooling", arguments: {} } },

795+

{ id: "call_3", function: { name: "tools", arguments: {} } },

796+

{ id: "call_4", function: { name: "tool_a", arguments: {} } },

777797

]);

778798

});

779799

@@ -795,7 +815,7 @@ describe("convertToOllamaMessages", () => {

795815

];

796816

const result = convertToOllamaMessages(messages);

797817

expect(result[0].tool_calls).toEqual([

798-

{ function: { name: "Read", arguments: { file_path: "/tmp/test.txt" } } },

818+

{ id: "call_2", function: { name: "Read", arguments: { file_path: "/tmp/test.txt" } } },

799819

]);

800820

});

801821

@@ -810,7 +830,7 @@ describe("convertToOllamaMessages", () => {

810830

];

811831

const result = convertToOllamaMessages(messages);

812832

expect(result[0].tool_calls).toEqual([

813-

{ function: { name: "exec", arguments: { command: "echo hello" } } },

833+

{ id: "toolu_1", function: { name: "exec", arguments: { command: "echo hello" } } },

814834

]);

815835

});

816836

@@ -831,6 +851,7 @@ describe("convertToOllamaMessages", () => {

831851

const result = convertToOllamaMessages(messages);

832852

expect(result[0].tool_calls).toEqual([

833853

{

854+

id: "call_3",

834855

function: {

835856

name: "read",

836857

arguments: {

@@ -981,6 +1002,47 @@ describe("buildAssistantMessage", () => {

9811002

expect(toolCall.id).toMatch(/^ollama_call_[0-9a-f-]{36}$/);

9821003

});

98310041005+

it("preserves Ollama response tool-call ids", () => {

1006+

const response = {

1007+

model: "gemini-3-flash-preview:cloud",

1008+

created_at: "2026-01-01T00:00:00Z",

1009+

message: {

1010+

role: "assistant" as const,

1011+

content: "",

1012+

tool_calls: [

1013+

{ id: "fc_ollama_real_1", function: { name: "bash", arguments: { command: "pwd" } } },

1014+

],

1015+

},

1016+

done: true,

1017+

};

1018+

const result = buildAssistantMessage(response, modelInfo);

1019+

expectToolCallContent(result.content[0], { name: "bash", arguments: { command: "pwd" } });

1020+

expect((result.content[0] as { id?: string }).id).toBe("fc_ollama_real_1");

1021+

});

1022+1023+

it("preserves parallel Ollama response tool-call ids independently", () => {

1024+

const response = {

1025+

model: "gemini-3-flash-preview:cloud",

1026+

created_at: "2026-01-01T00:00:00Z",

1027+

message: {

1028+

role: "assistant" as const,

1029+

content: "",

1030+

tool_calls: [

1031+

{ id: "fc_ollama_real_1", function: { name: "read", arguments: { path: "a.txt" } } },

1032+

{ id: "fc_ollama_real_2", function: { name: "exec", arguments: { command: "date" } } },

1033+

],

1034+

},

1035+

done: true,

1036+

};

1037+

const result = buildAssistantMessage(response, modelInfo);

1038+

expect(result.content.map((part) => (part as { id?: string }).id)).toEqual([

1039+

"fc_ollama_real_1",

1040+

"fc_ollama_real_2",

1041+

]);

1042+

expectToolCallContent(result.content[0], { name: "read", arguments: { path: "a.txt" } });

1043+

expectToolCallContent(result.content[1], { name: "exec", arguments: { command: "date" } });

1044+

});

1045+9841046

it("normalizes provider-prefixed tool-call names in Ollama responses", () => {

9851047

const response = {

9861048

model: "qwen3:32b",