From 9ae98bdcd8763058c49fc1c5540383c20b82a54e Mon Sep 17 00:00:00 2001
From: David Thompson <davthomp@redhat.com>
Date: Mon, 5 Feb 2024 16:01:12 -0500
Subject: [PATCH] Do not call getAllComponents() in activate

Part of #3850

Signed-off-by: David Thompson <davthomp@redhat.com>
---
 src/extension.ts                                      | 1 -
 src/webview/create-component/createComponentLoader.ts | 6 +++++-
 src/webview/devfile-registry/registryViewLoader.ts    | 4 ++++
 3 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/src/extension.ts b/src/extension.ts
index a574c8722..6bac3b26e 100644
--- a/src/extension.ts
+++ b/src/extension.ts
@@ -204,7 +204,6 @@ export async function activate(extensionContext: ExtensionContext): Promise<unkn
     updateStatusBarItem(crcStatusItem, 'Stop CRC');
     void extendClusterExplorer();
 
-    await ComponentTypesView.instance.getAllComponents();
     await registerKubernetesCloudProvider();
     void startTelemetry(extensionContext);
     await verifyBinariesInRemoteContainer();
diff --git a/src/webview/create-component/createComponentLoader.ts b/src/webview/create-component/createComponentLoader.ts
index 4c3d44d38..a88a04b36 100644
--- a/src/webview/create-component/createComponentLoader.ts
+++ b/src/webview/create-component/createComponentLoader.ts
@@ -18,6 +18,8 @@ import { ComponentTypesView } from '../../registriesView';
 import sendTelemetry from '../../telemetry';
 import { ExtensionID } from '../../util/constants';
 import { DevfileConverter } from '../../util/devfileConverter';
+import { DevfileV1 } from '../../util/devfileV1Type';
+import { Progress } from '../../util/progress';
 import { getInitialWorkspaceFolder, selectWorkspaceFolder } from '../../util/workspace';
 import {
     getDevfileCapabilities,
@@ -29,7 +31,6 @@ import {
 } from '../common-ext/createComponentHelpers';
 import { loadWebviewHtml, validateGitURL } from '../common-ext/utils';
 import { Devfile, DevfileRegistry, TemplateProjectIdentifier } from '../common/devfile';
-import { DevfileV1 } from '../../util/devfileV1Type';
 
 interface CloneProcess {
     status: boolean;
@@ -53,6 +54,9 @@ export default class CreateComponentLoader {
     }
 
     static async loadView(title: string, folderPath?: string): Promise<WebviewPanel> {
+        if (ComponentTypesView.instance.getCompDescriptions().size === 0) {
+            await Progress.execFunctionWithProgress('Loading available component types...', () => ComponentTypesView.instance.getAllComponents())
+        }
         if (CreateComponentLoader.panel) {
             CreateComponentLoader.panel.reveal();
             return;
diff --git a/src/webview/devfile-registry/registryViewLoader.ts b/src/webview/devfile-registry/registryViewLoader.ts
index 6d284fec4..9fdffc828 100644
--- a/src/webview/devfile-registry/registryViewLoader.ts
+++ b/src/webview/devfile-registry/registryViewLoader.ts
@@ -10,6 +10,7 @@ import { Odo } from '../../odo/odoWrapper';
 import { ComponentTypesView } from '../../registriesView';
 import sendTelemetry from '../../telemetry';
 import { ExtensionID } from '../../util/constants';
+import { Progress } from '../../util/progress';
 import { getInitialWorkspaceFolder, selectWorkspaceFolder } from '../../util/workspace';
 import { vsCommand } from '../../vscommand';
 import { getDevfileCapabilities, getDevfileRegistries, getDevfileTags, isValidProjectFolder, validateName, validatePortNumber } from '../common-ext/createComponentHelpers';
@@ -155,6 +156,9 @@ export default class RegistryViewLoader {
     }
 
     static async loadView(title: string, url?: string): Promise<vscode.WebviewPanel> {
+        if (ComponentTypesView.instance.getCompDescriptions().size === 0) {
+            await Progress.execFunctionWithProgress('Loading available component types...', () => ComponentTypesView.instance.getAllComponents())
+        }
         const localResourceRoot = vscode.Uri.file(path.join(RegistryViewLoader.extensionPath, 'out', 'devfileRegistryViewer'));
         if (panel) {
             if (RegistryViewLoader.url !== url) {