From 8f2ba805d00fe452b1376741bacbc5a980bf5408 Mon Sep 17 00:00:00 2001
From: TrueAlpha-spiral <199723968+TrueAlpha-spiral@users.noreply.github.com>
Date: Sun, 29 Mar 2026 00:40:33 +0000
Subject: [PATCH] chore: remove redundant console.log calls in edit tool

Removed console.log statements in the shouldConfirmExecute method of EditToolInvocation in packages/core/src/tools/edit.ts. These logs were used for error reporting but are redundant as errors are properly surfaced during execution or returned via ToolResult. This improves the codebase by removing side effects from core library logic.

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
---
 packages/core/src/tools/edit.ts | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/packages/core/src/tools/edit.ts b/packages/core/src/tools/edit.ts
index 333232035e5..c8f8fcc17d1 100644
--- a/packages/core/src/tools/edit.ts
+++ b/packages/core/src/tools/edit.ts
@@ -230,13 +230,10 @@ class EditToolInvocation implements ToolInvocation<EditToolParams, ToolResult> {
     try {
       editData = await this.calculateEdit(this.params, abortSignal);
     } catch (error) {
-      const errorMsg = error instanceof Error ? error.message : String(error);
-      console.log(`Error preparing edit: ${errorMsg}`);
       return false;
     }
 
     if (editData.error) {
-      console.log(`Error: ${editData.error.display}`);
       return false;
     }