Summary

This PR fixes issue #1610 where Browser Tool + Memory causes ValidationException after the first message in a session.

Problem

When using tools (especially Browser Tool) with session memory, context truncation or summarization may remove toolUse blocks while keeping their corresponding toolResult blocks. This creates "orphaned" toolResult blocks that cause:

ValidationException: The number of toolResult blocks at messages.N.content
exceeds the number of toolUse blocks of previous turn.

Error Scenario

1. User sends message → Browser tool creates toolUse blocks
2. Memory saves conversation history
3. Context is truncated/summarized → toolUse blocks removed, toolResult kept
4. User sends second message → ValidationException

This is a widespread issue affecting:

• Anthropic Claude Code (#13964) - orphaned tool_result blocks after context truncation
• Pydantic AI - similar ValidationException issues
• OpenHands - tool use/result mismatch after summarization
• LiteLLM - same pattern with Bedrock Converse API

Root Cause

The existing _fix_broken_tool_use method only handled:

1. ✅ Orphaned toolResult at the START of conversation (first message)
2. ✅ Orphaned toolUse (missing toolResult)
3. ❌ Orphaned toolResult in the MIDDLE of conversation

Solution

Added _remove_orphaned_tool_results method that:

1. Collects all toolUseId values from toolUse blocks in all messages
2. Scans all messages for toolResult blocks
3. Removes any toolResult whose toolUseId has no matching toolUse
4. Handles mixed-content messages (keeps valid content, removes only orphaned)
5. Drops empty messages that only contained orphaned toolResult blocks

Code Change

def _remove_orphaned_tool_results(self, messages: list[Message]) -> list[Message]:
    """Remove orphaned toolResult blocks that have no matching toolUse."""
    # Collect all toolUse IDs
    all_tool_use_ids: set[str] = set()
    for message in messages:
        for content in message.get("content", []):
            if "toolUse" in content:
                all_tool_use_ids.add(content["toolUse"]["toolUseId"])
    
    # Filter out orphaned toolResult blocks
    # ... (see full implementation)

Testing

Added 5 new tests covering:

All 29 tests pass:

======================= 29 passed in 0.72s =======================

Files Changed

• src/strands/session/repository_session_manager.py - Added _remove_orphaned_tool_results method
• tests/strands/session/test_repository_session_manager.py - Added 5 new tests

Related Issues

• Fixes Browser Tool + Memory: orphaned toolUse causing ValidationException after first message #1610
• Related: Anthropic Claude Code #13964
• Related: browser-use [BUG] Claude 4 models stop at 8192 output tokens #710

Note: This fix is backward compatible and won't affect existing valid conversations. It only removes toolResult blocks that have no matching toolUse block anywhere in the conversation history.