Refactor the Checkpoints command quick pick to use a dedicated button for clearing all checkpoints and clarify the tooltip for running preset groups

Author: robertpiosik

packages/ui/src/components/editor/panel/Presets/Presets.tsx

@@ -597,7 +597,7 @@ export const Presets: React.FC<Presets.Props> = (props) => {
                       >
                         <IconButton
                           codicon_icon="run-all-coverage"
-                          title="Run"
+                          title="Run selected presets"
                           on_click={(e) => {
                             e.stopPropagation()
                             props.on_group_click('Ungrouped')
@@ -763,7 +763,7 @@ export const Presets: React.FC<Presets.Props> = (props) => {
                       {!preset.chatbot && (
                         <IconButton
                           codicon_icon="run-all-coverage"
-                          title="Run"
+                          title="Run selected presets"
                           on_click={(e) => {
                             e.stopPropagation()
                             props.on_group_click(preset.name)

packages/vscode/src/commands/checkpoints-command/checkpoints-command.ts

@@ -68,6 +68,11 @@ export const checkpoints_command = (
         quick_pick.placeholder =
           'Select a checkpoint to restore or add a new one'
 
+        const clear_all_button: vscode.QuickInputButton = {
+          iconPath: new vscode.ThemeIcon('trash'),
+          tooltip: 'Delete all checkpoints'
+        }
+
         let notification_count = 0
         let checkpoints: Checkpoint[] = []
 
@@ -107,6 +112,12 @@ export const checkpoints_command = (
 
           const visible_checkpoints = checkpoints.filter((c) => !c.is_temporary)
 
+          if (visible_checkpoints.length > 0) {
+            quick_pick.buttons = [clear_all_button]
+          } else {
+            quick_pick.buttons = []
+          }
+
           visible_checkpoints.sort((a, b) => {
             return b.timestamp - a.timestamp
           })
@@ -117,15 +128,6 @@ export const checkpoints_command = (
               label: '$(add) New checkpoint...',
               alwaysShow: true
             },
-            ...(visible_checkpoints.length > 0
-              ? [
-                  {
-                    id: 'clear-all',
-                    label: '$(trash) Clear all checkpoints...',
-                    alwaysShow: true
-                  }
-                ]
-              : []),
             ...(revert_item ? [revert_item] : []),
             ...(visible_checkpoints.length > 0
               ? [
@@ -200,7 +202,18 @@ export const checkpoints_command = (
               TEMPORARY_CHECKPOINT_STATE_KEY,
               undefined
             )
-          } else if (selected.id == 'clear-all') {
+          } else if (selected.checkpoint) {
+            quick_pick.hide()
+            await restore_checkpoint({
+              checkpoint: selected.checkpoint,
+              workspace_provider,
+              context
+            })
+          }
+        })
+
+        quick_pick.onDidTriggerButton(async (button) => {
+          if (button === clear_all_button) {
             quick_pick.hide()
             const confirmation = await vscode.window.showWarningMessage(
               'Are you sure you want to clear all checkpoints? This operation cannot be undone.',
@@ -219,13 +232,6 @@ export const checkpoints_command = (
               undefined
             )
             await show_quick_pick()
-          } else if (selected.checkpoint) {
-            quick_pick.hide()
-            await restore_checkpoint({
-              checkpoint: selected.checkpoint,
-              workspace_provider,
-              context
-            })
           }
         })