Avoid requiring confirmation for tools that edit buffers (#27923)

Antonio Scandurra created 8 months ago

It's super easy to undo those changes. In a future PR, we should also
avoid requiring confirmation in the batch tool if all the underlying
tools don't require confirmation.

Release Notes:

- N/A

Change summary

crates/assistant_tools/src/create_file_tool.rs       | 2 +-
crates/assistant_tools/src/edit_files_tool.rs        | 2 +-
crates/assistant_tools/src/find_replace_file_tool.rs | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)

Detailed changes

crates/assistant_tools/src/create_file_tool.rs

@@ -41,7 +41,7 @@ impl Tool for CreateFileTool {
     }
 
     fn needs_confirmation(&self) -> bool {
-        true
+        false
     }
 
     fn description(&self) -> String {

crates/assistant_tools/src/edit_files_tool.rs

@@ -82,7 +82,7 @@ impl Tool for EditFilesTool {
     }
 
     fn needs_confirmation(&self) -> bool {
-        true
+        false
     }
 
     fn description(&self) -> String {

crates/assistant_tools/src/find_replace_file_tool.rs

@@ -130,7 +130,7 @@ impl Tool for FindReplaceFileTool {
     }
 
     fn needs_confirmation(&self) -> bool {
-        true
+        false
     }
 
     fn description(&self) -> String {