fix: preserve user images in native tool call results

[daniel-lxs]

Problem

When native tool calls are enabled, user images sent with responses were being converted to placeholder text (image content) instead of being preserved and sent to the LLM. This caused the model to lose important visual context that users were providing.

Solution

Modified the pushToolResult function in presentAssistantMessage.ts to:

• Preserve image blocks in tool_result content as arrays when images are present
• Only convert to string when no images exist (for cleaner representation)
• Maintain full compatibility with Anthropic API's tool_result format

Changes

• Modified: src/core/assistant-message/presentAssistantMessage.ts

  • Updated pushToolResult function to detect and preserve image blocks
  • Images are kept as Array<TextBlockParam | ImageBlockParam> when present
  • Falls back to string conversion when no images for simpler representation

• Added: src/core/assistant-message/__tests__/presentAssistantMessage-images.spec.ts

  • Comprehensive test coverage for image handling in both native and XML protocols
  • 4 test cases covering various scenarios

Testing

All tests pass, including:

• ✅ Image preservation in native protocol tool results
• ✅ String conversion when no images (native protocol)
• ✅ Image preservation in XML protocol (existing behavior)
• ✅ Empty tool result handling

Impact

• Native Protocol: Images are now correctly preserved and sent to the LLM
• XML Protocol: No changes - continues to work as before
• Backwards Compatible: No breaking changes to existing functionality

---

Important

Fixes image preservation in native tool call results by updating pushToolResult in presentAssistantMessage.ts to handle image blocks correctly.

• Behavior:
  • pushToolResult in presentAssistantMessage.ts now preserves image blocks in tool results for native protocol.
  • Converts to string only if no images are present.
  • Ensures compatibility with Anthropic API's tool_result format.
• Testing:
  • Added presentAssistantMessage-images.spec.ts with 4 test cases for image handling in native and XML protocols.
  • Tests cover image preservation, string conversion, XML protocol behavior, and empty tool result handling.
• Impact:
  • Native protocol now correctly preserves images for LLM.
  • XML protocol remains unchanged.
  • No breaking changes to existing functionality.

^{This description was created by for b5dc163. You can customize this summary. It will automatically update as commits are pushed.}

[roomote]

Rooviewer   See task on Roo Cloud

Review completed. No issues found - the implementation correctly preserves images in native protocol tool results while maintaining backward compatibility.

_{Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.}

[roomote]

Rooviewer   See task on Roo Cloud

Review completed. No issues found - the implementation correctly preserves images in native protocol tool results while maintaining backward compatibility.

_{Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.}