fix: Remove status field from toolResult for non-claude 3 models in Bedrock model provider

src/strands/models/bedrock.py

@@ -276,10 +276,19 @@ def _format_bedrock_messages(self, messages: Messages) -> Messages:
                     # Create a new content block with only the cleaned toolResult
                     tool_result: ToolResult = content_block["toolResult"]
 
-                    # Keep only the required fields for Bedrock
-                    cleaned_tool_result = ToolResult(
-                        content=tool_result["content"], toolUseId=tool_result["toolUseId"], status=tool_result["status"]
-                    )
+                    model_id = self.config.get("model_id")
+                    if "claude-3" in model_id:
+                        # Keep the status field for Claude models
+                        cleaned_tool_result = ToolResult(
+                            content=tool_result["content"],
+                            toolUseId=tool_result["toolUseId"],
+                            status=tool_result["status"],
+                        )
+                    else:
+                        # For non-Claude models, use ToolResult without status
+                        cleaned_tool_result = ToolResult(
+                            toolUseId=tool_result["toolUseId"], content=tool_result["content"]
+                        )
 
                     cleaned_block: ContentBlock = {"toolResult": cleaned_tool_result}
                     cleaned_content.append(cleaned_block)

src/strands/types/tools.py

@@ -7,7 +7,7 @@
 
 from abc import ABC, abstractmethod
 from dataclasses import dataclass
-from typing import TYPE_CHECKING, Any, AsyncGenerator, Awaitable, Callable, Literal, Protocol, Union
+from typing import TYPE_CHECKING, Any, AsyncGenerator, Awaitable, Callable, Literal, Optional, Protocol, Union
 
 from typing_extensions import TypedDict
 
@@ -91,7 +91,7 @@ class ToolResult(TypedDict):
     """
 
     content: list[ToolResultContent]
-    status: ToolResultStatus
+    status: Optional[ToolResultStatus]
     toolUseId: str
 
 

tests/strands/models/test_bedrock.py

@@ -1233,7 +1233,65 @@ def test_format_request_cleans_tool_result_content_blocks(model, model_id):
 
     # Verify toolResult only contains allowed fields in the formatted request
     tool_result = formatted_request["messages"][0]["content"][0]["toolResult"]
-    expected = {"content": [{"text": "Tool output"}], "toolUseId": "tool123", "status": "success"}
+    expected = {"content": [{"text": "Tool output"}], "toolUseId": "tool123"}
     assert tool_result == expected
     assert "extraField" not in tool_result
     assert "mcpMetadata" not in tool_result
+
+
+def test_format_request_removes_status_field_for_non_claude_models(model, model_id):
+    """Test that format_request removes status field for non-Claude models."""
+    # Update model to use a non-Claude model
+    model.update_config(model_id="us.writer.palmyra-x4-v1:0")
+
+    messages = [
+        {
+            "role": "user",
+            "content": [
+                {
+                    "toolResult": {
+                        "content": [{"text": "Tool output"}],
+                        "toolUseId": "tool123",
+                        "status": "success",
+                    }
+                },
+            ],
+        }
+    ]
+
+    formatted_request = model.format_request(messages)
+
+    # Verify toolResult does not contain status field for non-Claude models
+    tool_result = formatted_request["messages"][0]["content"][0]["toolResult"]
+    expected = {"toolUseId": "tool123", "content": [{"text": "Tool output"}]}
+    assert tool_result == expected
+    assert "status" not in tool_result
+
+
+def test_format_request_keeps_status_field_for_claude_models(model, model_id):
+    """Test that format_request keeps status field for Claude models."""
+    # Update model to use a Claude model
+    model.update_config(model_id="anthropic.claude-3-sonnet-20240229-v1:0")
+
+    messages = [
+        {
+            "role": "user",
+            "content": [
+                {
+                    "toolResult": {
+                        "content": [{"text": "Tool output"}],
+                        "toolUseId": "tool123",
+                        "status": "success",
+                    }
+                },
+            ],
+        }
+    ]
+
+    formatted_request = model.format_request(messages)
+
+    # Verify toolResult contains status field for Claude models
+    tool_result = formatted_request["messages"][0]["content"][0]["toolResult"]
+    expected = {"content": [{"text": "Tool output"}], "toolUseId": "tool123", "status": "success"}
+    assert tool_result == expected
+    assert "status" in tool_result