can start stop/daemon.

Signed-off-by: Akos Kitta <[email protected]>

Author: Akos Kitta

arduino-ide-extension/src/browser/contributions/daemon.ts

@@ -25,14 +25,17 @@ export namespace Daemon {
     export const START_DAEMON: Command = {
       id: 'arduino-start-daemon',
       label: nls.localize('arduino/daemon/start', 'Start Daemon'),
+      category: 'Arduino',
     };
     export const STOP_DAEMON: Command = {
       id: 'arduino-stop-daemon',
       label: nls.localize('arduino/daemon/stop', 'Stop Daemon'),
+      category: 'Arduino',
     };
     export const RESTART_DAEMON: Command = {
       id: 'arduino-restart-daemon',
       label: nls.localize('arduino/daemon/restart', 'Restart Daemon'),
+      category: 'Arduino',
     };
   }
 }

arduino-ide-extension/src/node/core-client-provider.ts

@@ -51,13 +51,16 @@ export class CoreClientProvider {
 
   @postConstruct()
   protected init(): void {
-    this.daemon.getPort().then((port) => this.create(port));
-    this.daemon.onDaemonStopped(() => {
-      // TODO
+    this.daemon.tryGetPort().then((port) => {
+      if (port) {
+        this.create(port);
+      }
     });
+    this.daemon.onDaemonStarted((port) => this.create(port));
+    this.daemon.onDaemonStopped(() => this.closeClient());
     this.configService.onConfigChange(async () => {
       const port = await this.daemon.getPort();
-      this.create(port, 10); // Due to a config change, run the indexes update immediately.
+      this.create(port, 10); // Due to a config change, run the indexes update "immediately".
     });
   }
 
@@ -90,7 +93,7 @@ export class CoreClientProvider {
     port: string,
     indexesUpdateDelay = 10_000
   ): Promise<CoreClientProvider.Client> {
-    this.toDisposeBeforeCreate.dispose();
+    this.closeClient();
     const address = this.address(port);
     const client = await this.createClient(address);
     this.toDisposeBeforeCreate.pushAll([
@@ -129,15 +132,18 @@ export class CoreClientProvider {
     }
   }
 
-  private async useClient(
+  private useClient(
     client: CoreClientProvider.Client
-  ): Promise<CoreClientProvider.Client> {
-    // TODO: why async? -> for local testing and `wait`
+  ): CoreClientProvider.Client {
     this._client = client;
     this.onClientReadyEmitter.fire(this._client);
     return this._client;
   }
 
+  private closeClient(): void {
+    return this.toDisposeBeforeCreate.dispose();
+  }
+
   private async createClient(
     address: string
   ): Promise<CoreClientProvider.Client> {