Skip to content

Conversation

@mrubens
Copy link
Collaborator

@mrubens mrubens commented Jun 1, 2025

Reverts #3654

A test is failing in main, not sure why, but figured better safe.


Important

Reverts recent changes in file editing tools to previous state due to a failing test, removing user edit handling and diff application logic.

  • Revert Changes:
    • Reverts changes in applyDiffTool.ts, insertContentTool.ts, searchAndReplaceTool.ts, and writeToFileTool.ts to previous state.
    • Removes handling of user edits and diff application logic.
  • DiffViewProvider:
    • Removes pushToolWriteResult() method and related logic.
    • Restores previous behavior in saveChanges() method.
  • Misc:
    • Reverts import of addLineNumbers in applyDiffTool.ts and writeToFileTool.ts.

This description was created by Ellipsis for 58e8c36. You can customize this summary. It will automatically update as commits are pushed.

@mrubens mrubens requested a review from cte as a code owner June 1, 2025 20:51
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Jun 1, 2025
@mrubens mrubens merged commit 98f5542 into main Jun 1, 2025
12 of 13 checks passed
@mrubens mrubens deleted the revert-3654-format-file-write-response branch June 1, 2025 20:51
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jun 1, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Jun 1, 2025
SmartManoj pushed a commit to SmartManoj/Raa-Code that referenced this pull request Jun 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L This PR changes 100-499 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants