From a87f81042aa0d8110dc4034f629a375e54850219 Mon Sep 17 00:00:00 2001
From: Dhanji Prasanna <dhanji@squareup.com>
Date: Thu, 2 Oct 2025 13:58:57 +1000
Subject: [PATCH] remove edit_file

---
 TODO                      |  1 +
 crates/g3-core/src/lib.rs | 41 ++++++++-------------------------------
 2 files changed, 9 insertions(+), 33 deletions(-)

diff --git a/TODO b/TODO
index 7232b2f..c717631 100644
--- a/TODO
+++ b/TODO
@@ -2,6 +2,7 @@ next tasks
 
 x get something working with autonomous mode
 - g3d
+- bug where it prints everything in a conversation turn all over again before final_output
 x ui abstraction from core
 - context token counting bug
 - embedded model
diff --git a/crates/g3-core/src/lib.rs b/crates/g3-core/src/lib.rs
index afd8aa9..d6edca6 100644
--- a/crates/g3-core/src/lib.rs
+++ b/crates/g3-core/src/lib.rs
@@ -299,7 +299,6 @@ impl ContextWindow {
     pub fn create_summary_prompt(&self) -> String {
         "Please provide a comprehensive summary of our conversation so far. Include:
 
-
 1. **Main Topic/Goal**: What is the primary task or objective being worked on?
 2. **Key Decisions**: What important decisions have been made?
 3. **Actions Taken**: What specific actions, commands, or code changes have been completed?
@@ -902,35 +901,6 @@ The tool will execute immediately and you'll receive the result (success or erro
                     "required": ["file_path", "content"]
                 }),
             },
-            Tool {
-                name: "edit_file".to_string(),
-                description: "Edit a specific range of lines in a file. Replaces lines from start_of_range to end_of_range (inclusive, 1-indexed) with new content.".to_string(),
-                input_schema: json!({
-                    "type": "object",
-                    "properties": {
-                        "content": {
-                            "type": "string",
-                            "description": "The content to write to the file"
-                        }
-                    },
-                    "required": ["file_path", "content"]
-                }),
-            },
-            // Commented out edit_file tool to prevent g3 from using it
-            // Tool {
-            //     name: "edit_file".to_string(),
-            //     description: "Edit a specific range of lines in a file. Replaces lines from start_of_range to end_of_range (inclusive, 1-indexed) with new content.".to_string(),
-            //     input_schema: json!({
-            //         "type": "object",
-            //         "properties": {
-            //             "file_path": {"type": "string", "description": "The path to the file to edit"},
-            //             "content": {"type": "string", "description": "The new content to replace the specified range"},
-            //             "start_of_range": {"type": "integer", "description": "The starting line number (1-indexed, inclusive)"},
-            //             "end_of_range": {"type": "integer", "description": "The ending line number (1-indexed, inclusive)"}
-            //         },
-            //         "required": ["file_path", "content", "start_of_range", "end_of_range"]
-            //     }),
-            // },
             Tool {
                 name: "str_replace".to_string(),
                 description: "Apply a unified diff to a file. Supports multiple hunks and context lines. Optionally constrain the search to a [start, end) character range (0-indexed; end is EXCLUSIVE). Useful to disambiguate matches or limit scope in large files.".to_string(),
@@ -1134,7 +1104,9 @@ The tool will execute immediately and you'll receive the result (success or erro
             // Get the summary
             match provider.complete(summary_request).await {
                 Ok(summary_response) => {
-                    self.ui_writer.print_context_status("✅ Summary created successfully. Resetting context window...\n");
+                    self.ui_writer.print_context_status(
+                        "✅ Summary created successfully. Resetting context window...\n",
+                    );
 
                     // Extract the latest user message from the request
                     let latest_user_msg = request
@@ -1151,7 +1123,9 @@ The tool will execute immediately and you'll receive the result (success or erro
                     // Update the request with new context
                     request.messages = self.context_window.conversation_history.clone();
 
-                    self.ui_writer.print_context_status("🔄 Context reset complete. Continuing with your request...\n");
+                    self.ui_writer.print_context_status(
+                        "🔄 Context reset complete. Continuing with your request...\n",
+                    );
                 }
                 Err(e) => {
                     error!("Failed to create summary: {}", e);
@@ -1408,7 +1382,8 @@ The tool will execute immediately and you'll receive the result (success or erro
 
                             // Closure marker with timing
                             if tool_call.tool != "final_output" {
-                                self.ui_writer.print_tool_timing(&Self::format_duration(exec_duration));
+                                self.ui_writer
+                                    .print_tool_timing(&Self::format_duration(exec_duration));
                                 self.ui_writer.print_agent_prompt();
                             }