fix: propagate connection status on disconnect

the new connection status was missing from the ws message

Signed-off-by: Akos Kitta <a.kitta@arduino.cc>

Author: Akos Kitta

arduino-ide-extension/src/node/monitor-service.ts

@@ -184,7 +184,7 @@ export class MonitorService extends CoreClientAware implements Disposable {
       this.updateClientsSettings({
         monitorUISettings: {
           connectionStatus: 'connected',
-          connected: true,
+          connected: true, // TODO: should be removed when plotter app understand the `connectionStatus` message
           serialPort: this.port.address,
         },
       });
@@ -196,7 +196,7 @@ export class MonitorService extends CoreClientAware implements Disposable {
       this.updateClientsSettings({
         monitorUISettings: {
           connectionStatus: 'not-connected',
-          connected: false,
+          connected: false, // TODO: should be removed when plotter app understand the `connectionStatus` message
         },
       });
 
@@ -268,7 +268,7 @@ export class MonitorService extends CoreClientAware implements Disposable {
       this.updateClientsSettings({
         monitorUISettings: {
           connectionStatus: 'connected',
-          connected: true,
+          connected: true, // TODO: should be removed when plotter app understand the `connectionStatus` message
           serialPort: this.port.address,
         },
       });
@@ -311,7 +311,10 @@ export class MonitorService extends CoreClientAware implements Disposable {
         if (duplex === this.duplex) {
           this.duplex = null;
           this.updateClientsSettings({
-            monitorUISettings: { connected: false },
+            monitorUISettings: {
+              connected: false, // TODO: should be removed when plotter app understand the `connectionStatus` message
+              connectionStatus: 'not-connected',
+            },
           });
         }
         this.logger.info(
@@ -322,7 +325,10 @@ export class MonitorService extends CoreClientAware implements Disposable {
         if (duplex === this.duplex) {
           this.duplex = null;
           this.updateClientsSettings({
-            monitorUISettings: { connected: false },
+            monitorUISettings: {
+              connected: false, // TODO: should be removed when plotter app understand the `connectionStatus` message
+              connectionStatus: 'not-connected',
+            },
           });
         }
         this.logger.info(
@@ -545,7 +551,7 @@ export class MonitorService extends CoreClientAware implements Disposable {
         ...settings.monitorUISettings,
         connectionStatus,
         serialPort: this.port.address,
-        connected: isMonitorConnected(connectionStatus),
+        connected: isMonitorConnected(connectionStatus), // TODO: should be removed when plotter app understand the `connectionStatus` message
       },
       pluggableMonitorSettings: reconciledSettings,
     });