Skip to content

Commit 84bac5a

Browse files
author
Alberto Iannaccone
committed
connect serial plotter app with websocket
1 parent 953d33f commit 84bac5a

14 files changed

+295
-124
lines changed

arduino-ide-extension/src/browser/arduino-ide-frontend-module.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -258,7 +258,6 @@ import {
258258
PlotterPath,
259259
PlotterService,
260260
} from '../common/protocol/plotter-service';
261-
import { PlotterConnection } from './plotter/plotter-connection';
262261
import { nls } from '@theia/core/lib/browser/nls';
263262

264263
const ElementQueries = require('css-element-queries/src/ElementQueries');
@@ -411,7 +410,7 @@ export default new ContainerModule((bind, unbind, isBound, rebind) => {
411410
})
412411
.inSingletonScope();
413412
bind(MonitorConnection).toSelf().inSingletonScope();
414-
bind(PlotterConnection).toSelf().inSingletonScope();
413+
415414
// Serial monitor service client to receive and delegate notifications from the backend.
416415
bind(MonitorServiceClient).to(MonitorServiceClientImpl).inSingletonScope();
417416

arduino-ide-extension/src/browser/contributions/burn-bootloader.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,8 +49,10 @@ export class BurnBootloader extends SketchContribution {
4949

5050
async burnBootloader(): Promise<void> {
5151
const monitorConfig = this.monitorConnection.monitorConfig;
52+
const serialConnection = this.monitorConnection.connectionType;
53+
5254
if (monitorConfig) {
53-
await this.monitorConnection.disconnect();
55+
await this.monitorConnection.disconnect(serialConnection);
5456
}
5557
try {
5658
const { boardsConfig } = this.boardsServiceClientImpl;
@@ -85,7 +87,7 @@ export class BurnBootloader extends SketchContribution {
8587
this.messageService.error(e.toString());
8688
} finally {
8789
if (monitorConfig) {
88-
await this.monitorConnection.connect(monitorConfig);
90+
await this.monitorConnection.connect(serialConnection, monitorConfig);
8991
}
9092
}
9193
}

arduino-ide-extension/src/browser/contributions/upload-sketch.ts

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -110,8 +110,9 @@ export class UploadSketch extends SketchContribution {
110110
}
111111
let shouldAutoConnect = false;
112112
const monitorConfig = this.monitorConnection.monitorConfig;
113+
const serialConnection = this.monitorConnection.connectionType;
113114
if (monitorConfig) {
114-
await this.monitorConnection.disconnect();
115+
await this.monitorConnection.disconnect(serialConnection);
115116
if (this.monitorConnection.autoConnect) {
116117
shouldAutoConnect = true;
117118
}
@@ -182,12 +183,11 @@ export class UploadSketch extends SketchContribution {
182183
Object.assign(board, { port }),
183184
10_000
184185
);
185-
if (shouldAutoConnect) {
186-
// Enabling auto-connect will trigger a connect.
187-
this.monitorConnection.autoConnect = true;
188-
} else {
189-
await this.monitorConnection.connect(monitorConfig);
190-
}
186+
this.monitorConnection.connect(
187+
serialConnection,
188+
monitorConfig,
189+
shouldAutoConnect
190+
);
191191
} catch (waitError) {
192192
this.messageService.error(
193193
nls.localize(

0 commit comments

Comments
 (0)