Skip to content

Conversation

@mrubens
Copy link
Collaborator

@mrubens mrubens commented May 29, 2025

Just noticed this while reading the docs https://docs.roocode.com/features/rooignore#file-editing-tools-potential-write-bypass


Important

Add rooignore checks to insertContentTool and searchAndReplaceTool to prevent operations on ignored files.

  • Behavior:
    • Add rooignore check in insertContentTool() to prevent operations on ignored files.
    • Add rooignore check in searchAndReplaceTool() to prevent operations on ignored files.

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

@mrubens mrubens requested a review from cte as a code owner May 29, 2025 02:04
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label May 29, 2025
@mrubens mrubens merged commit 4cf240f into main May 29, 2025
15 checks passed
@mrubens mrubens deleted the fix_insert_content_search_replace_rooignore branch May 29, 2025 02:36
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap May 29, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap May 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S This PR changes 10-29 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants