From 32b1ef37791ef5cf94c1b80941ad899e1924feec Mon Sep 17 00:00:00 2001
From: shamso-goog <shamso@google.com>
Date: Wed, 30 Jul 2025 16:37:51 -0400
Subject: [PATCH] feat(ui): Update tool confirmation cancel button text (#4820)

Co-authored-by: Jacob Richman <jacob314@gmail.com>
---
 .../messages/ToolConfirmationMessage.tsx      | 21 ++++++++++++++-----
 1 file changed, 16 insertions(+), 5 deletions(-)

diff --git a/packages/cli/src/ui/components/messages/ToolConfirmationMessage.tsx b/packages/cli/src/ui/components/messages/ToolConfirmationMessage.tsx
index c1a313d5..197a922c 100644
--- a/packages/cli/src/ui/components/messages/ToolConfirmationMessage.tsx
+++ b/packages/cli/src/ui/components/messages/ToolConfirmationMessage.tsx
@@ -118,7 +118,10 @@ export const ToolConfirmationMessage: React.FC<
         label: 'Modify with external editor',
         value: ToolConfirmationOutcome.ModifyWithEditor,
       },
-      { label: 'No (esc)', value: ToolConfirmationOutcome.Cancel },
+      {
+        label: 'No, suggest changes (esc)',
+        value: ToolConfirmationOutcome.Cancel,
+      },
     );
     bodyContent = (
       <DiffRenderer
@@ -142,10 +145,12 @@ export const ToolConfirmationMessage: React.FC<
         label: `Yes, allow always ...`,
         value: ToolConfirmationOutcome.ProceedAlways,
       },
+      {
+        label: 'No, suggest changes (esc)',
+        value: ToolConfirmationOutcome.Cancel,
+      },
     );
 
-    options.push({ label: 'No (esc)', value: ToolConfirmationOutcome.Cancel });
-
     let bodyContentHeight = availableBodyContentHeight();
     if (bodyContentHeight !== undefined) {
       bodyContentHeight -= 2; // Account for padding;
@@ -180,7 +185,10 @@ export const ToolConfirmationMessage: React.FC<
         label: 'Yes, allow always',
         value: ToolConfirmationOutcome.ProceedAlways,
       },
-      { label: 'No (esc)', value: ToolConfirmationOutcome.Cancel },
+      {
+        label: 'No, suggest changes (esc)',
+        value: ToolConfirmationOutcome.Cancel,
+      },
     );
 
     bodyContent = (
@@ -221,7 +229,10 @@ export const ToolConfirmationMessage: React.FC<
         label: `Yes, always allow all tools from server "${mcpProps.serverName}"`,
         value: ToolConfirmationOutcome.ProceedAlwaysServer,
       },
-      { label: 'No (esc)', value: ToolConfirmationOutcome.Cancel },
+      {
+        label: 'No, suggest changes (esc)',
+        value: ToolConfirmationOutcome.Cancel,
+      },
     );
   }