Add support for bundle destroy command (#1205)

## Changes
<!-- Summary of your changes that are easy to understand -->

## Tests
<!-- How is this tested? -->

Author: kartikgupta-db

packages/databricks-vscode/package.json

@@ -227,28 +227,28 @@
                 "command": "databricks.bundle.refreshRemoteState",
                 "icon": "$(refresh)",
                 "title": "Refresh remote state",
-                "enablement": "databricks.context.activated && databricks.context.bundle.isTargetSet",
+                "enablement": "databricks.context.activated && databricks.context.bundle.isTargetSet && databricks.context.bundle.deploymentState == idle",
                 "category": "Databricks"
             },
             {
                 "command": "databricks.bundle.deploy",
                 "icon": "$(cloud-upload)",
                 "title": "Deploy bundle",
-                "enablement": "databricks.context.activated && databricks.context.bundle.isTargetSet",
+                "enablement": "databricks.context.activated && databricks.context.bundle.isTargetSet && databricks.context.bundle.deploymentState == idle",
                 "category": "Databricks"
             },
             {
                 "command": "databricks.bundle.deployAndRun",
                 "icon": "$(debug-start)",
                 "title": "Deploy the bundle and run the resource",
-                "enablement": "databricks.context.activated && databricks.context.bundle.isTargetSet",
+                "enablement": "databricks.context.activated && databricks.context.bundle.isTargetSet && databricks.context.bundle.deploymentState == idle",
                 "category": "Databricks"
             },
             {
                 "command": "databricks.bundle.cancelRun",
                 "title": "Cancel run",
                 "icon": "$(debug-stop)",
-                "enablement": "databricks.context.activated && databricks.context.bundle.isTargetSet",
+                "enablement": "databricks.context.activated && databricks.context.bundle.isTargetSet && databricks.context.bundle.deploymentState == idle",
                 "category": "Databricks"
             },
             {
@@ -314,16 +314,23 @@
             {
                 "command": "databricks.bundle.variable.openFile",
                 "title": "Override bundle variables",
-                "enablement": "databricks.context.activated && databricks.context.bundle.isTargetSet",
+                "enablement": "databricks.context.activated && databricks.context.bundle.isTargetSet && databricks.context.bundle.deploymentState == idle",
                 "category": "Databricks",
                 "icon": "$(gear)"
             },
             {
                 "command": "databricks.bundle.variable.reset",
                 "title": "Reset to default values",
-                "enablement": "databricks.context.activated && databricks.context.bundle.isTargetSet",
+                "enablement": "databricks.context.activated && databricks.context.bundle.isTargetSet && databricks.context.bundle.deploymentState == idle",
                 "category": "Databricks",
                 "icon": "$(discard)"
+            },
+            {
+                "command": "databricks.bundle.destroy",
+                "title": "Destroy bundle",
+                "enablement": "databricks.context.activated && databricks.context.bundle.isTargetSet && databricks.context.bundle.deploymentState == idle",
+                "category": "Databricks",
+                "icon": "$(trash)"
             }
         ],
         "viewsContainers": {
@@ -438,6 +445,11 @@
                     "when": "view == dabsResourceExplorerView && databricks.context.bundle.deploymentState == idle",
                     "group": "navigation@1"
                 },
+                {
+                    "command": "databricks.bundle.destroy",
+                    "when": "view == dabsResourceExplorerView && databricks.context.bundle.deploymentState == idle",
+                    "group": "navigation@1"
+                },
                 {
                     "command": "databricks.bundle.variable.openFile",
                     "when": "view == dabsVariableView && databricks.context.bundle.deploymentState == idle",

packages/databricks-vscode/src/bundle/models/BundleRemoteStateModel.ts

@@ -9,7 +9,6 @@ import lodash from "lodash";
 import {WorkspaceConfigs} from "../../vscode-objs/WorkspaceConfigs";
 import {logging} from "@databricks/databricks-sdk";
 import {Loggers} from "../../logger";
-import {MsPythonExtensionWrapper} from "../../language/MsPythonExtensionWrapper";
 
 /* eslint-disable @typescript-eslint/naming-convention */
 export type BundleResourceModifiedStatus = "created" | "deleted" | "updated";
@@ -49,7 +48,6 @@ export class BundleRemoteStateModel extends BaseModelWithStateCache<BundleRemote
     constructor(
         private readonly cli: CliWrapper,
         private readonly workspaceFolder: Uri,
-        private readonly pythonExtension: MsPythonExtensionWrapper,
         private readonly workspaceConfigs: WorkspaceConfigs
     ) {
         super();
@@ -73,7 +71,24 @@ export class BundleRemoteStateModel extends BaseModelWithStateCache<BundleRemote
             this.target,
             this.authProvider,
             this.workspaceFolder,
-            this.pythonExtension,
+            this.workspaceConfigs.databrickscfgLocation,
+            this.logger
+        );
+    }
+
+    @Mutex.synchronise("mutex")
+    public async destroy() {
+        if (this.target === undefined) {
+            throw new Error("Target is undefined");
+        }
+        if (this.authProvider === undefined) {
+            throw new Error("No authentication method is set");
+        }
+
+        await this.cli.bundleDestroy(
+            this.target,
+            this.authProvider,
+            this.workspaceFolder,
             this.workspaceConfigs.databrickscfgLocation,
             this.logger
         );
@@ -118,18 +133,18 @@ export class BundleRemoteStateModel extends BaseModelWithStateCache<BundleRemote
             return {};
         }
 
-        const output = await this.cli.bundleSummarise(
+        const {stdout} = await this.cli.bundleSummarise(
             this.target,
             this.authProvider,
             this.workspaceFolder,
             this.workspaceConfigs.databrickscfgLocation,
             this.logger
         );
 
-        if (output === "" || output === undefined) {
+        if (stdout === "" || stdout === undefined) {
             return {};
         }
-        return JSON.parse(output);
+        return JSON.parse(stdout);
     }
 
     /**

packages/databricks-vscode/src/bundle/models/BundleValidateModel.ts

@@ -67,13 +67,15 @@ export class BundleValidateModel extends BaseModelWithStateCache<BundleValidateS
         }
 
         const validateOutput = JSON.parse(
-            await this.cli.bundleValidate(
-                this.target,
-                this.authProvider,
-                this.workspaceFolder,
-                workspaceConfigs.databrickscfgLocation,
-                this.logger
-            )
+            (
+                await this.cli.bundleValidate(
+                    this.target,
+                    this.authProvider,
+                    this.workspaceFolder,
+                    workspaceConfigs.databrickscfgLocation,
+                    this.logger
+                )
+            ).stdout
         ) as BundleTarget;
 
         return {