Remove all checkpoint-related functionality and associated UI components from panel

- remove history/checkpoints handling from the panel (we're showing it now somewhere else), note that 'response history' is a different feature
- remove panel-related logic from 'checkpoints' feature
- remove 'show progress', 'hide progress', 'SHOW_AUTO_CLOSING_MODAL' message passing and remove related, now unnecessary params
- make native progress required, skip such redundant  assignments const progress = native_progress
- remove panel_provider.send_checkpoints
- restore the use of 'Tabs' [Changes]

Author: robertpiosik

apps/editor/src/commands/apply-chat-response-command/apply-chat-response-command.ts

@@ -428,7 +428,6 @@ export const apply_chat_response_command = (params: {
                   CHECKPOINTS_STATE_KEY,
                   checkpoints
                 )
-                await params.panel_provider.send_checkpoints()
               }
             }
 

apps/editor/src/commands/history-command.ts

@@ -21,7 +21,6 @@ import {
 import { PanelProvider } from '@/views/panel/backend/panel-provider'
 import { get_checkpoint_path } from '@/features/checkpoints/utils'
 import { dictionary } from '@shared/constants/dictionary'
-import { checkpoints_emitter } from '@/features/checkpoints/events'
 import { get_response_preview_promise_resolve } from '@/commands/apply-chat-response-command/utils/preview/preview'
 
 dayjs.extend(localizedFormat)
@@ -268,7 +267,7 @@ export const history_command = (params: {
               checkpoint: temp_checkpoint,
               workspace_provider: params.workspace_provider,
               context: params.context,
-              options: { skip_confirmation: true, use_native_progress: true },
+              options: { skip_confirmation: true },
               panel_provider: params.panel_provider
             })
             // After reverting, delete the temp checkpoint and clear state.
@@ -281,15 +280,13 @@ export const history_command = (params: {
               checkpoint_to_delete: temp_checkpoint,
               panel_provider: params.panel_provider
             })
-            checkpoints_emitter.emit('checkpoints-updated')
           } else if (selected.checkpoint) {
             quick_pick.hide()
             await restore_checkpoint({
               checkpoint: selected.checkpoint,
               workspace_provider: params.workspace_provider,
               context: params.context,
-              panel_provider: params.panel_provider,
-              options: { use_native_progress: true }
+              panel_provider: params.panel_provider
             })
           }
         })
@@ -381,7 +378,6 @@ export const history_command = (params: {
                   CHECKPOINTS_STATE_KEY,
                   checkpoints
                 )
-                params.panel_provider.send_checkpoints()
               }
             }
             await refresh_and_update_view()

apps/editor/src/features/checkpoints/actions/clear-all-checkpoints.ts

@@ -51,7 +51,6 @@ export const clear_all_checkpoints = async (
       undefined
     )
     await context.workspaceState.update(CHECKPOINTS_STATE_KEY, [])
-    await panel_provider.send_checkpoints()
   }
   await vscode.window.withProgress(
     {

apps/editor/src/features/checkpoints/actions/create-checkpoint.ts

@@ -29,7 +29,6 @@ export const create_checkpoint = async (params: {
   panel_provider: PanelProvider
   trigger?: CheckpointTrigger
   description?: string
-  silent?: boolean
 }): Promise<Checkpoint | undefined> => {
   try {
     const trigger = params.trigger ?? 'manual'
@@ -220,37 +219,14 @@ export const create_checkpoint = async (params: {
         CHECKPOINTS_STATE_KEY,
         checkpoints
       )
-      await params.panel_provider.send_checkpoints()
       await params.context.workspaceState.update(
         CHECKPOINT_OPERATION_IN_PROGRESS_STATE_KEY,
         undefined
       )
       new_checkpoint = checkpoint_object
     }
 
-    let timer: ReturnType<typeof setTimeout> | undefined
-    let did_show_modal = false
-
-    try {
-      if (!params.silent) {
-        timer = setTimeout(() => {
-          did_show_modal = true
-          params.panel_provider.send_message({
-            command: 'SHOW_PROGRESS',
-            title: 'Creating checkpoint...'
-          })
-        }, 1000)
-      }
-
-      await create_checkpoint_task()
-    } finally {
-      if (timer) clearTimeout(timer)
-      if (did_show_modal) {
-        params.panel_provider.send_message({
-          command: 'HIDE_PROGRESS'
-        })
-      }
-    }
+    await create_checkpoint_task()
     return new_checkpoint
   } catch (err: any) {
     vscode.window.showErrorMessage(

apps/editor/src/features/checkpoints/actions/delete-checkpoint.ts

@@ -23,7 +23,6 @@ export const delete_checkpoint = async (params: {
     CHECKPOINTS_STATE_KEY,
     updated_checkpoints
   )
-  await params.panel_provider.send_checkpoints()
 
   try {
     const checkpoint_path = get_checkpoint_path(
@@ -74,7 +73,6 @@ export const delete_checkpoint_with_undo = async (params: {
     CHECKPOINTS_STATE_KEY,
     updated_checkpoints
   )
-  await params.panel_provider.send_checkpoints()
   params.on_did_update_checkpoints?.(updated_checkpoints)
 
   const finalize = async () => {
@@ -122,7 +120,6 @@ export const delete_checkpoint_with_undo = async (params: {
         CHECKPOINTS_STATE_KEY,
         current_checkpoints
       )
-      await params.panel_provider.send_checkpoints()
       params.on_did_update_checkpoints?.(current_checkpoints)
 
       params.on_before_show_message?.()

apps/editor/src/features/checkpoints/actions/restore-checkpoint.ts

@@ -22,17 +22,13 @@ import { ongoing_preview_cleanup_promise } from '@/commands/apply-chat-response-
 import { dictionary } from '@shared/constants/dictionary'
 import { get_git_info } from '../utils/git-utils'
 import { PromptsForCommitMessagesUtils } from '@/utils/prompts-for-commit-messages-utils'
-import { checkpoints_emitter } from '../events'
-
 export const restore_checkpoint = async (params: {
   checkpoint: Checkpoint
   workspace_provider: WorkspaceProvider
   context: vscode.ExtensionContext
   panel_provider: PanelProvider
   options?: {
     skip_confirmation?: boolean
-    use_native_progress?: boolean
-    show_auto_closing_modal_on_success?: boolean
   }
 }) => {
   const operation_in_progress = params.context.workspaceState.get<number>(
@@ -49,37 +45,9 @@ export const restore_checkpoint = async (params: {
     ? 'Reverting...'
     : 'Restoring checkpoint...'
 
-  let current_progress = 0
-  let progress_reporter:
-    | vscode.Progress<{ message?: string; increment?: number }>
-    | undefined
-
-  if (!params.options?.use_native_progress) {
-    params.panel_provider.send_message({
-      command: 'SHOW_PROGRESS',
-      title,
-      delay_visibility: false
-    })
-    progress_reporter = {
-      report: (value) => {
-        if (value.increment) {
-          current_progress += value.increment
-          params.panel_provider.send_message({
-            command: 'SHOW_PROGRESS',
-            title,
-            progress: Math.min(current_progress, 100),
-            delay_visibility: false
-          })
-        }
-      }
-    }
-  }
-
   const main_task = async (
-    native_progress?: vscode.Progress<{ message?: string; increment?: number }>
+    progress: vscode.Progress<{ message?: string; increment?: number }>
   ) => {
-    const progress = native_progress || progress_reporter
-
     if (
       params.checkpoint.trigger == 'response-accepted' &&
       params.checkpoint.description
@@ -136,8 +104,7 @@ export const restore_checkpoint = async (params: {
             workspace_provider: params.workspace_provider,
             context: params.context,
             panel_provider: params.panel_provider,
-            trigger: 'before-checkpoint-restored',
-            silent: true
+            trigger: 'before-checkpoint-restored'
           })
         }
 
@@ -543,55 +510,31 @@ export const restore_checkpoint = async (params: {
 
   let temp_check: Checkpoint | undefined
   try {
-    if (params.options?.use_native_progress) {
-      temp_check = await vscode.window.withProgress(
-        {
-          location: vscode.ProgressLocation.Notification,
-          title,
-          cancellable: false
-        },
-        main_task
-      )
-    } else {
-      temp_check = await main_task()
-    }
+    temp_check = await vscode.window.withProgress(
+      {
+        location: vscode.ProgressLocation.Notification,
+        title,
+        cancellable: false
+      },
+      main_task
+    )
   } catch (e) {
     return
-  } finally {
-    if (!params.options?.use_native_progress) {
-      params.panel_provider.send_message({
-        command: 'HIDE_PROGRESS'
-      })
-    }
-    checkpoints_emitter.emit('checkpoints-updated')
   }
 
   const message = params.options?.skip_confirmation
     ? 'Successfully reverted changes.'
     : 'Checkpoint has been restored.'
 
-  if (params.checkpoint.trigger == 'response-accepted') {
-    params.panel_provider.send_message({
-      command: 'SHOW_PROGRESS',
-      title: message.endsWith('.') ? message.slice(0, -1) : message,
-      cancellable: false,
-      delay_visibility: true
-    })
-  } else if (params.options?.show_auto_closing_modal_on_success) {
-    params.panel_provider.send_message({
-      command: 'SHOW_AUTO_CLOSING_MODAL',
-      title: message.slice(0, -1),
-      type: 'success'
-    })
-  } else if (temp_check) {
+  if (temp_check) {
     const action = await vscode.window.showInformationMessage(message, 'Revert')
     if (action == 'Revert') {
       await restore_checkpoint({
         checkpoint: temp_check,
         workspace_provider: params.workspace_provider,
         context: params.context,
         panel_provider: params.panel_provider,
-        options: { skip_confirmation: true, use_native_progress: true }
+        options: { skip_confirmation: true }
       })
       await params.context.workspaceState.update(
         TEMPORARY_CHECKPOINT_STATE_KEY,
@@ -602,7 +545,6 @@ export const restore_checkpoint = async (params: {
         checkpoint_to_delete: temp_check,
         panel_provider: params.panel_provider
       })
-      checkpoints_emitter.emit('checkpoints-updated')
     }
   } else {
     vscode.window.showInformationMessage(message)

apps/editor/src/features/checkpoints/actions/toggle-checkpoint-star.ts

@@ -22,6 +22,5 @@ export const toggle_checkpoint_star = async (params: {
       CHECKPOINTS_STATE_KEY,
       checkpoints
     )
-    await params.panel_provider.send_checkpoints()
   }
 }

apps/editor/src/features/checkpoints/actions/update-checkpoint-description.ts

@@ -23,6 +23,5 @@ export const update_checkpoint_description = async (params: {
       CHECKPOINTS_STATE_KEY,
       checkpoints
     )
-    await params.panel_provider.send_checkpoints()
   }
 }