fix: remove content from Message if content is an empty array (#1895)

vachillo · collindutter · commit 9c066c3f572f · 2025-04-21T14:11:14.000-07:00
fix: remove  from Message if  is an empty array
diff --git a/griptape/drivers/prompt/openai_chat_prompt_driver.py b/griptape/drivers/prompt/openai_chat_prompt_driver.py
@@ -273,7 +273,7 @@ def __to_openai_messages(self, messages: list[Message]) -> list[dict]:
 
                 # Some OpenAi-compatible services don't accept an empty array for content
                 if not openai_message["content"]:
-                    openai_message["content"] = ""
+                    del openai_message["content"]
 
                 openai_messages.append(openai_message)
 
diff --git a/tests/unit/drivers/prompt/test_openai_chat_prompt_driver.py b/tests/unit/drivers/prompt/test_openai_chat_prompt_driver.py
@@ -385,7 +385,6 @@ def messages(self):
             },
             {"role": "assistant", "content": "assistant-input"},
             {
-                "content": "",
                 "role": "assistant",
                 "tool_calls": [
                     {
@@ -403,7 +402,7 @@ def messages(self):
                 ],
                 "role": "user",
             },
-            {"audio": {"id": "audio-id"}, "content": "", "role": "assistant"},
+            {"audio": {"id": "audio-id"}, "role": "assistant"},
             {"content": [{"type": "text", "text": "assistant-audio-transcription"}], "role": "assistant"},
             {"content": [{"type": "text", "text": "generic-value"}], "role": "user"},
         ]

Original file line number	Diff line number	Diff line change
`@@ -385,7 +385,6 @@ def messages(self):`
`385`	`385`	`},`
`386`	`386`	`{"role": "assistant", "content": "assistant-input"},`
`387`	`387`	`{`
`388`		`- "content": "",`
`389`	`388`	`"role": "assistant",`
`390`	`389`	`"tool_calls": [`
`391`	`390`	`{`
`@@ -403,7 +402,7 @@ def messages(self):`
`403`	`402`	`],`
`404`	`403`	`"role": "user",`
`405`	`404`	`},`
`406`		`- {"audio": {"id": "audio-id"}, "content": "", "role": "assistant"},`
	`405`	`+ {"audio": {"id": "audio-id"}, "role": "assistant"},`
`407`	`406`	`{"content": [{"type": "text", "text": "assistant-audio-transcription"}], "role": "assistant"},`
`408`	`407`	`{"content": [{"type": "text", "text": "generic-value"}], "role": "user"},`
`409`	`408`	`]`