Skip to content

🤖 refactor: make file edit tool warnings more prominent #502

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 1 commit into from
Nov 3, 2025
Merged

🤖 refactor: make file edit tool warnings more prominent #502

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
8 changes: 4 additions & 4 deletions src/utils/tools/toolDefinitions.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -70,8 +70,8 @@ export const TOOL_DEFINITIONS = {
},
file_edit_replace_string: {
description:
"Apply one or more edits to a file by replacing exact text matches. All edits are applied sequentially. Each old_string must be unique in the file unless replace_count > 1 or replace_count is -1. " +
`IMPORTANT: Edits may fail if old_string is not found or not unique. ${TOOL_EDIT_WARNING}`,
"⚠️ CRITICAL: Always check tool results - edits WILL fail if old_string is not found or unique. Do not proceed with dependent operations (commits, pushes, builds) until confirming success.\n\n" +
"Apply one or more edits to a file by replacing exact text matches. All edits are applied sequentially. Each old_string must be unique in the file unless replace_count > 1 or replace_count is -1.",
schema: z.object({
file_path: z.string().describe("The absolute path to the file to edit"),
old_string: z
Expand All @@ -91,8 +91,8 @@ export const TOOL_DEFINITIONS = {
},
file_edit_replace_lines: {
description:
"Replace a range of lines in a file. Use this for line-based edits when you know the exact line numbers to modify. " +
`IMPORTANT: Edits may fail if line numbers are invalid or file content has changed. ${TOOL_EDIT_WARNING}`,
"⚠️ CRITICAL: Always check tool results - edits WILL fail if line numbers are invalid or file content has changed. Do not proceed with dependent operations (commits, pushes, builds) until confirming success.\n\n" +
"Replace a range of lines in a file. Use this for line-based edits when you know the exact line numbers to modify.",
schema: z.object({
file_path: z.string().describe("The absolute path to the file to edit"),
start_line: z.number().int().min(1).describe("1-indexed start line (inclusive) to replace"),
Expand Down