fix: search entire condensed region for tool_use blocks (ROO-520)

[daniel-lxs]

Problem

After multiple condensations, tool_use blocks preserved in earlier summary messages can end up outside the bounded search window of N_MESSAGES_TO_KEEP (3 messages). This causes orphaned tool_result blocks that trigger 400 errors from the API:

messages.0.content.0: unexpected tool_use_id found in tool_result blocks

Root Cause

The bounded search in getKeepMessagesWithToolBlocks was too restrictive:

// Only searches 3 messages before the keep boundary
const searchStart = startIndex - 1
const searchEnd = Math.max(0, startIndex - N_MESSAGES_TO_KEEP)
const messagesToSearch = messages.slice(searchEnd, searchStart + 1)

After multiple condensations, tool_use blocks can be in earlier summary messages that are outside this window.

Solution

Changed the search to cover the entire condensed region:

// Search the entire condensed region (from index 0 to startIndex-1)
const messagesToSearch = messages.slice(0, startIndex)

Tests

Added two new tests:

• should find tool_use even when it is far back in the message history (ROO-520 fix)
• should find tool_use in previous summary message after multiple condensations (ROO-520)

Fixes: https://linear.app/roocode/issue/ROO-520

---

Important

Expands search range for tool_use blocks in condensed regions to prevent API errors, with new tests added.

• Behavior:
  • Expands search range in getKeepMessagesWithToolBlocks to cover entire condensed region, preventing orphaned tool_result blocks.
  • Fixes API 400 errors due to unexpected tool_use_id in tool_result blocks.
• Tests:
  • Adds test should find tool_use even when it is far back in the message history (ROO-520 fix).
  • Adds test should find tool_use in previous summary message after multiple condensations (ROO-520).
• Misc:
  • Minor formatting changes in README.md and page.tsx.
  • Removes unused mocks in useMcpToolTool.spec.ts.

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

[roomote]

Rooviewer   See task on Roo Cloud

Re-review complete for latest update. No issues found.

• No issues found

Previous reviews

• 04fc2a0: Review #1

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

[mrubens]

@roomote can you fix the conflicts here?

[roomote]

Fixaroo   See task on Roo Cloud

Resolved merge conflicts by accepting main's changes. The getKeepMessagesWithToolBlocks function was removed in main, making the ROO-520 fix obsolete. All local checks passed.

_{View commit | Revert commit}

[github-actions]

🚀 Preview deployed!

Your changes have been deployed to Vercel:

Preview URL: https://roo-code-website-dal7uoh3u-roo-code.vercel.app

This preview will be updated automatically when you push new commits to this PR.