Deploy bundle before running files using remote run modes (#1019)

## Changes
* Deploy the current bundle before running files using any of our remote
run modes.
  * Upload and run file
  * Run as Workflow
* Remove sync destination validation. We depend on bundle completely
now.
* Remove sync status and button to start syncing.

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

Author: kartikgupta-db

packages/databricks-vscode/package.json

@@ -150,7 +150,10 @@
                 "title": "Run File as Workflow",
                 "category": "Databricks",
                 "enablement": "!inDebugMode",
-                "icon": "$(play)"
+                "icon": {
+                    "dark": "resources/dark/logo.svg",
+                    "light": "resources/light/logo.svg"
+                }
             },
             {
                 "command": "databricks.run.runEditorContents",
@@ -252,12 +255,6 @@
                 "enablement": "databricks.context.activated && databricks.context.bundle.isTargetSet",
                 "category": "Databricks"
             },
-            {
-                "command": "databricks.bundle.cancelAllRuns",
-                "title": "Cancel all runs",
-                "enablement": "databricks.context.activated && databricks.context.bundle.isTargetSet",
-                "category": "Databricks"
-            },
             {
                 "command": "databricks.bundle.cancelRun",
                 "title": "Cancel run",
@@ -405,16 +402,6 @@
                     "when": "view == configurationView && viewItem =~ /^databricks.configuration.cluster.*$/",
                     "group": "inline@2"
                 },
-                {
-                    "command": "databricks.sync.start",
-                    "when": "view == configurationView && viewItem == databricks.configuration.sync.stopped",
-                    "group": "inline@0"
-                },
-                {
-                    "command": "databricks.sync.stop",
-                    "when": "view == configurationView && viewItem =~ /^databricks.configuration.sync.(?!error|stopped|selected).*$/",
-                    "group": "inline@0"
-                },
                 {
                     "command": "databricks.cluster.stop",
                     "when": "view == configurationView && viewItem =~ /^databricks.configuration.cluster.(running|pending)$/",

packages/databricks-vscode/src/bundle/BundleCommands.ts

@@ -1,6 +1,7 @@
-import {Disposable, window} from "vscode";
+import {Disposable, ProgressLocation, window} from "vscode";
 import {BundleRemoteStateModel} from "./models/BundleRemoteStateModel";
 import {onError} from "../utils/onErrorDecorator";
+import {BundleWatcher} from "./BundleWatcher";
 import {
     TreeNode as BundleResourceExplorerTreeNode,
     ResourceTreeNode as BundleResourceExplorerResourceTreeNode,
@@ -24,6 +25,28 @@ export class BundleCommands implements Disposable {
         "Databricks Asset Bundles"
     );
 
+    constructor(
+        private readonly bundleRemoteStateModel: BundleRemoteStateModel,
+        private readonly bundleRunStatusManager: BundleRunStatusManager,
+        private readonly bundleWatcher: BundleWatcher
+    ) {
+        this.disposables.push(
+            this.outputChannel,
+            this.bundleWatcher.onDidChange(async () => {
+                await this.bundleRemoteStateModel.refresh();
+            })
+        );
+    }
+
+    async refreshRemoteState() {
+        await window.withProgress(
+            {location: {viewId: "dabsResourceExplorerView"}},
+            async () => {
+                await this.bundleRemoteStateModel.refresh();
+            }
+        );
+    }
+
     private writeToChannel = (data: string) => {
         this.outputChannel.append(data);
     };
@@ -33,29 +56,29 @@ export class BundleCommands implements Disposable {
         this.outputChannel.appendLine("");
     }
 
-    constructor(
-        private readonly bundleRemoteStateModel: BundleRemoteStateModel,
-        private readonly bundleRunStatusManager: BundleRunStatusManager
-    ) {
-        this.disposables.push(this.outputChannel);
-    }
-
     @onError({popup: {prefix: "Error refreshing remote state."}})
-    async refreshRemoteState() {
-        await this.bundleRemoteStateModel.refresh();
+    async refreshRemoteStateCommand() {
+        await this.refreshRemoteState();
     }
 
-    @onError({popup: {prefix: "Error deploying the bundle."}})
     async deploy() {
+        this.prepareOutputChannel();
         await window.withProgress(
-            {location: {viewId: "dabsResourceExplorerView"}},
+            {location: ProgressLocation.Notification, cancellable: false},
             async () => {
                 await this.bundleRemoteStateModel.deploy(
                     this.writeToChannel,
                     this.writeToChannel
                 );
             }
         );
+
+        await this.refreshRemoteState();
+    }
+
+    @onError({popup: {prefix: "Error deploying the bundle."}})
+    async deployCommand() {
+        await this.deploy();
     }
 
     @onError({popup: {prefix: "Error running resource."}})
@@ -64,16 +87,7 @@ export class BundleCommands implements Disposable {
             throw new Error(`Cannot run resource of type ${treeNode.type}`);
         }
         //TODO: Don't deploy if there is no diff between local and remote state
-        this.prepareOutputChannel();
-        await window.withProgress(
-            {location: {viewId: "dabsResourceExplorerView"}},
-            async () => {
-                await this.bundleRemoteStateModel.deploy(
-                    this.writeToChannel,
-                    this.writeToChannel
-                );
-            }
-        );
+        await this.deploy();
 
         await this.bundleRunStatusManager.run(
             treeNode.resourceKey,