Skip to content

fix: remove search_and_replace tool from codebase #8892

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
Oct 31, 2025
Merged

fix: remove search_and_replace tool from codebase #8892

Show file tree
Hide file tree
Changes from 6 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
950b958
fix: remove search_and_replace tool and all references
roomote Oct 28, 2025
9b993ef
test: update snapshots after removing search_and_replace tool
roomote Oct 28, 2025
7d42021
fix: remove remaining search_and_replace references from localization…
roomote Oct 28, 2025
b1c1751
fix: remove searchAndReplace case from ChatRow.tsx
roomote Oct 28, 2025
201c1f4
fix(pr-fixer): scrub lingering references to removed search_and_repla…
roomote Oct 28, 2025
06507ff
test: remove obsolete search_and_replace e2e test file
daniel-lxs Oct 30, 2025
2017d37
chore: remove orphaned search_and_replace fields from ClineSayTool (E…
hannesrudolph Oct 31, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion .roo/rules-issue-fixer/1_Workflow.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -191,7 +191,6 @@
Use appropriate tools:
- apply_diff for targeted changes
- write_to_file for new files
- search_and_replace for systematic updates

After each significant change, run relevant tests:
- execute_command to run test suites
Expand Down
2 changes: 1 addition & 1 deletion .roo/rules-merge-resolver/2_best_practices.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@
<principle priority="high">
<name>Escape Conflict Markers</name>
<description>
When using apply_diff or search_and_replace tools, always escape merge
When using apply_diff, always escape merge
conflict markers with backslashes to prevent parsing errors.
</description>
<example><![CDATA[
Expand Down
25 changes: 3 additions & 22 deletions .roo/rules-merge-resolver/3_tool_usage.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
</priority>

<priority level="3">
<tool>apply_diff or search_and_replace</tool>
<tool>apply_diff</tool>
<when>To resolve conflicts by replacing conflicted sections</when>
<why>Precise editing of specific conflict blocks</why>
</priority>
Expand Down Expand Up @@ -127,25 +127,6 @@ function mergedImplementation() {
]]></example>
</tool>

<tool name="search_and_replace">
<best_practices>
<practice>Use for simple conflict resolutions</practice>
<practice>Enable regex mode for complex patterns</practice>
<practice>Always escape special characters</practice>
</best_practices>

<example><![CDATA[
<search_and_replace>
<path>src/config.ts</path>
<search>\<<<<<<< HEAD[\s\S]*?\>>>>>>> \w+</search>
<replace>// Resolved configuration
const config = {
// Merged settings from both branches
}</replace>
<use_regex>true</use_regex>
</search_and_replace>
]]></example>
</tool>
</tool_specific_guidance>

<tool_combination_patterns>
Expand Down Expand Up @@ -215,7 +196,7 @@ const config = {
<scenario name="malformed_conflicts">
<description>Conflict markers are incomplete or nested</description>
<approach>
Use search_and_replace with careful regex patterns
Use apply_diff with precise search blocks; split into multiple targeted edits if needed
Manual inspection may be required
</approach>
</scenario>
Expand All @@ -232,7 +213,7 @@ const config = {
<description>Code contains literal conflict marker strings</description>
<approach>
Extra careful escaping in diffs
Consider using search_and_replace with precise patterns
Prefer apply_diff with precise search blocks
</approach>
</scenario>
</error_handling>
Expand Down
Loading