fix(openai): address comments

Author: Ratish1

src/strands/models/openai.py

@@ -209,14 +209,27 @@ def _split_tool_message_images(
         if not image_content:
             return tool_message, None
 
-        # Create tool message with only text content
-        # If no text content, add a simple success message
+        # Let the user know that we are modifying the messages for OpenAI compatibility
+        logger.warning(
+            "tool_call_id=<%s> | Moving image from tool message to a new user message for OpenAI compatibility",
+            tool_message["tool_call_id"],
+        )
+
+        # Append a message to the text content to inform the model about the upcoming image
+        text_content.append(
+            {
+                "type": "text",
+                "text": (
+                    "Tool successfully returned an image. The image is being provided in the following user message."
+                ),
+            }
+        )
+
+        # Create the clean tool message with the updated text content
         tool_message_clean = {
             "role": "tool",
             "tool_call_id": tool_message["tool_call_id"],
-            "content": text_content
-            if text_content
-            else [{"type": "text", "text": "Tool execution completed successfully."}],
+            "content": text_content,
         }
 
         # Create user message with only images

tests/strands/models/test_openai.py

@@ -192,11 +192,13 @@ def test_split_tool_message_images_with_image():
 
     tool_clean, user_with_image = OpenAIModel._split_tool_message_images(tool_message)
 
-    # Tool message should only have text
+    # Tool message should now have the original text plus the appended informational text
     assert tool_clean["role"] == "tool"
     assert tool_clean["tool_call_id"] == "c1"
-    assert len(tool_clean["content"]) == 1
+    assert len(tool_clean["content"]) == 2
     assert tool_clean["content"][0]["type"] == "text"
+    assert tool_clean["content"][0]["text"] == "Result"
+    assert "Tool successfully returned an image" in tool_clean["content"][1]["text"]
 
     # User message should have the image
     assert user_with_image is not None