Consolidate preview ongoing modal logic into PanelProvider to prevent duplicate modal displays

- instead of [Fragment], the panel provider should have dedicated method that checks if there is a response history item that has 'files' undefined. if so, should not show 'preview ongoing' modal
- the logic in panel_provider.show_preview_ongoing_modal doesn't work because the response without 'files' is already there

Author: robertpiosik

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

@@ -109,9 +109,7 @@ export const apply_chat_response_command = (params: {
 
         const choice = await new Promise<'Switch' | undefined>((resolve) => {
           params.panel_provider.preview_switch_choice_resolver = resolve
-          params.panel_provider.send_message({
-            command: 'SHOW_PREVIEW_ONGOING_MODAL'
-          })
+          params.panel_provider.show_preview_ongoing_modal()
         })
         params.panel_provider.preview_switch_choice_resolver = undefined
 

apps/editor/src/views/panel/backend/panel-provider.ts

@@ -667,6 +667,22 @@ export class PanelProvider implements vscode.WebviewViewProvider {
     }
   }
 
+  public show_preview_ongoing_modal() {
+    const items_without_files_count = this.response_history.filter(
+      (item) => item.files === undefined
+    ).length
+
+    if (items_without_files_count > 1) {
+      if (this.preview_switch_choice_resolver) {
+        this.preview_switch_choice_resolver(undefined)
+      }
+    } else {
+      this.send_message({
+        command: 'SHOW_PREVIEW_ONGOING_MODAL'
+      })
+    }
+  }
+
   async resolveWebviewView(
     webview_view: vscode.WebviewView,
     _: vscode.WebviewViewResolveContext,