fix: preserve HTML entities in apply_diff tool to avoid false identical content errors

[roomote]

This PR attempts to address Issue #9563. Feedback and guidance are welcome.

Problem

The apply_diff tool was incorrectly identifying HTML entities (like ') as identical to their decoded characters (like ') when processing JSX files. This caused valid diff operations to fail with "Search and replace content are identical" errors, preventing agents from fixing ESLint warnings about unescaped entities.

Root Cause

The issue was caused by the unescapeHtmlEntities function being applied to diff content for non-Claude models, which converted HTML entities to their literal characters before the comparison. This made doesn't and doesn't appear identical to the diff strategy.

Solution

• Removed the HTML entity unescaping logic from ApplyDiffTool.ts
• This ensures that diff content is compared using raw strings as provided by the agent
• HTML entities are now correctly recognized as different from their decoded counterparts

Testing

• Added comprehensive test suite in applyDiffTool.htmlentities.spec.ts
• Tests verify that HTML entities are preserved in diff content
• Tests confirm the fix works for both Claude and non-Claude models
• All existing tests continue to pass

Impact

This fix enables agents to properly escape special characters in JSX files when addressing ESLint warnings about unescaped entities.

Fixes #9563

---

Important

Fixes HTML entity handling in apply_diff tool to prevent false identical content errors in JSX files.

• Behavior:
  • Removed unescapeHtmlEntities logic from ApplyDiffTool.ts to preserve HTML entities in diff content.
  • Ensures diff content is compared using raw strings, preventing false identical content errors.
• Testing:
  • Added applyDiffTool.htmlentities.spec.ts to test HTML entity handling.
  • Tests verify preservation of HTML entities and correct behavior for Claude and non-Claude models.
• Impact:
  • Fixes issue [BUG] Roo things ' and ' are the same and fails to apply the change #9563, allowing agents to escape special characters in JSX files correctly.

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

[roomote]

Rooviewer   See task on Roo Cloud

Review complete. This PR correctly fixes the HTML entity handling issue by removing the unescaping logic that was causing false "identical content" errors. The changes are minimal, well-tested, and directly address the root cause described in issue #9563.

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