Fix sandbox stuck loading

Don't `await` message being sent to the webview when the webview is being built, since the webview is likely unable to receive the message yet. Fixes redhat-developer#3323 Signed-off-by: David Thompson <[email protected]>
datho7561 · Sep 25, 2023 · c2dca1a · c2dca1a
1 parent 06a8bfa
commit c2dca1a
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/webview/cluster/clusterViewLoader.ts b/src/webview/cluster/clusterViewLoader.ts
@@ -358,7 +358,6 @@ export default class ClusterViewLoader {
         await cfg.update('crcNameserver', event.nameserver);
     }
 
-    // eslint-disable-next-line @typescript-eslint/require-await
     static async loadView(title: string): Promise<vscode.WebviewPanel> {
         const localResourceRoot = vscode.Uri.file(path.join(ClusterViewLoader.extensionPath, 'out', 'clusterViewer'));
         if (panel) {
@@ -372,11 +371,13 @@ export default class ClusterViewLoader {
             });
             panel.iconPath = vscode.Uri.file(path.join(ClusterViewLoader.extensionPath, 'images/context/cluster-node.png'));
             panel.webview.html = await loadWebviewHtml('clusterViewer', panel);
-            await panel.webview.postMessage({action: 'cluster', data: ''});
+            const messageListenerDisposable = panel.webview.onDidReceiveMessage(clusterEditorMessageListener);
             panel.onDidDispose(()=> {
+                messageListenerDisposable.dispose();
                 panel = undefined;
             });
-            panel.webview.onDidReceiveMessage(clusterEditorMessageListener);
+            // don't await this message being sent, since the webview may not be set up yet
+            void panel.webview.postMessage({action: 'cluster', data: ''});
         }
         return panel;
     }