fix: truncate AWS Bedrock toolUseId to 64 characters

[daniel-lxs]

Problem

AWS Bedrock has a 64-character limit on toolUseId fields, but the Bedrock Converse format converter doesn't truncate these IDs. This causes validation errors when using LiteLLM as a proxy to Bedrock, especially when switching modes with long conversation histories.

Error example:

Value at 'messages.2.member.content.1.member.toolUse.toolUseId' failed to satisfy constraint: 
Member must have length less than or equal to 64

The issue manifests when:

• Using LiteLLM to route to AWS Bedrock
• Switching between modes (e.g., Ask → Orchestrator) with existing tool calls in history
• Tool call IDs exceed 64 characters

Solution

Applied the same ID sanitization approach used in OpenAI providers to the Bedrock Converse format converter:

1. Import sanitizeOpenAiCallId from src/utils/tool-id.ts
2. Apply sanitization to all toolUseId fields in:
   • tool_use blocks
   • tool_result blocks

This ensures IDs are truncated to 64 characters with hash suffixes to maintain uniqueness.

Changes

• Updated src/api/transform/bedrock-converse-format.ts
• Added 7 comprehensive tests in src/api/transform/__tests__/bedrock-converse-format.spec.ts
• All tests pass (17/17)

Related

Fixes EXT-556
Related to issue #10766 which addresses MCP tool name length, but this is a separate issue about tool call ID length.

---

Important

Truncates toolUseId to 64 characters in convertToBedrockConverseMessages() to comply with AWS Bedrock limits, adding tests for verification.

• Behavior:
  • Truncates toolUseId to 64 characters in convertToBedrockConverseMessages() in bedrock-converse-format.ts using sanitizeOpenAiCallId.
  • Handles tool_use and tool_result blocks, ensuring IDs are unique with hash suffixes.
• Tests:
  • Adds 7 tests in bedrock-converse-format.spec.ts to verify ID truncation and uniqueness.
  • Tests cover cases for long IDs, short IDs, consistent truncation, and matching tool_use and tool_result IDs.
• Misc:
  • Imports sanitizeOpenAiCallId from tool-id.ts for ID sanitization.

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

[roomote]

Rooviewer   See task on Roo Cloud

Review complete. No issues found.

This PR correctly applies the existing sanitizeOpenAiCallId utility to all toolUseId fields in the Bedrock Converse format converter, ensuring IDs are truncated to 64 characters with hash suffixes for uniqueness. The implementation is consistent across all 6 locations where toolUseId is set, and the tests comprehensively cover the key behaviors.

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