more consistent confirmations, TODO to improve write confirmations, drop "description" from execution confirmation, add confirmation to new (still dummy) shell tool (#176)

1 files changed, 5 insertions, 12 deletions

diff --git a/packages/cli/src/ui/components/messages/ToolConfirmationMessage.tsx b/packages/cli/src/ui/components/messages/ToolConfirmationMessage.tsx
index 9f65f1b9..19ccb466 100644
--- a/packages/cli/src/ui/components/messages/ToolConfirmationMessage.tsx
+++ b/packages/cli/src/ui/components/messages/ToolConfirmationMessage.tsx
@@ -60,11 +60,11 @@ export const ToolConfirmationMessage: React.FC<
     question = `Apply this change?`;
     options.push(
       {
-        label: 'Yes',
+        label: 'Yes, allow once',
         value: ToolConfirmationOutcome.ProceedOnce,
       },
       {
-        label: 'Yes (always allow)',
+        label: 'Yes, allow always', // TODO: this is extreme w/o being qualified by file or directory
         value: ToolConfirmationOutcome.ProceedAlways,
       },
       { label: 'No (esc)', value: ToolConfirmationOutcome.Cancel },
@@ -73,29 +73,22 @@ export const ToolConfirmationMessage: React.FC<
     const executionProps =
       confirmationDetails as ToolExecuteConfirmationDetails;
 
-    // For execution, we still need context display and description
-    const commandDisplay = (
-      <Text color={Colors.AccentCyan}>{executionProps.command}</Text>
-    );
-
-    // Combine command and description into bodyContent for layout consistency
     bodyContent = (
       <Box flexDirection="column">
         <Box paddingX={1} marginLeft={1}>
-          {commandDisplay}
+          <Text color={Colors.AccentCyan}>{executionProps.command}</Text>
         </Box>
       </Box>
     );
 
     question = `Allow execution?`;
-    const alwaysLabel = `Yes (always allow '${executionProps.rootCommand}' commands)`;
     options.push(
       {
-        label: 'Yes',
+        label: 'Yes, allow once',
         value: ToolConfirmationOutcome.ProceedOnce,
       },
       {
-        label: alwaysLabel,
+        label: `Yes, allow always for ${executionProps.rootCommand} ...`,
         value: ToolConfirmationOutcome.ProceedAlways,
       },
       { label: 'No (esc)', value: ToolConfirmationOutcome.Cancel },