Fix empty list in content on tool calls causing ollama parse error (google#102)

selcukgun · web-flow · commit cfbcc1764413 · 2025-04-12T17:34:57.000-07:00
This is related to google#49
diff --git a/src/google/adk/models/lite_llm.py b/src/google/adk/models/lite_llm.py
@@ -178,7 +178,7 @@ def _content_to_message_param(
 
     return ChatCompletionAssistantMessage(
         role=role,
-        content=_get_content(content.parts),
+        content=_get_content(content.parts) or None,
         tool_calls=tool_calls or None,
     )
 
diff --git a/tests/unittests/models/test_litellm.py b/tests/unittests/models/test_litellm.py
@@ -536,7 +536,7 @@ def test_content_to_message_param_function_call():
   content.parts[0].function_call.id = "test_tool_call_id"
   message = _content_to_message_param(content)
   assert message["role"] == "assistant"
-  assert message["content"] == []
+  assert message["content"] == None
   assert message["tool_calls"][0].type == "function"
   assert message["tool_calls"][0].id == "test_tool_call_id"
   assert message["tool_calls"][0].function.name == "test_function"

Original file line number	Diff line number	Diff line change
`@@ -178,7 +178,7 @@ def _content_to_message_param(`
`178`	`178`
`179`	`179`	`return ChatCompletionAssistantMessage(`
`180`	`180`	`role=role,`
`181`		`- content=_get_content(content.parts),`
	`181`	`+ content=_get_content(content.parts) or None,`
`182`	`182`	`tool_calls=tool_calls or None,`
`183`	`183`	`)`
`184`	`184`