fix: remove command flags from API

Author: Akos Kitta

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

@@ -16,15 +16,12 @@ import {
   arduinoCert,
   certificateList,
 } from '../dialogs/certificate-uploader/utils';
-import { ArduinoFirmwareUploader } from '../../common/protocol/arduino-firmware-uploader';
+import {
+  ArduinoFirmwareUploader,
+  UploadCertificateParams,
+} from '../../common/protocol/arduino-firmware-uploader';
 import { nls } from '@theia/core/lib/common';
 
-interface UploadCertificateParams {
-  readonly fqbn: string;
-  readonly address: string;
-  readonly urls: readonly string[];
-}
-
 @injectable()
 export class UploadCertificate extends Contribution {
   @inject(UploadCertificateDialog)
@@ -81,16 +78,7 @@ export class UploadCertificate extends Contribution {
 
     registry.registerCommand(UploadCertificate.Commands.UPLOAD_CERT, {
       execute: async (params: UploadCertificateParams) => {
-        const { fqbn, address, urls } = params;
-        return this.arduinoFirmwareUploader.uploadCertificates({
-          flags: [
-            '-b',
-            fqbn,
-            '-a',
-            address,
-            ...urls.flatMap((url) => ['-u', url]),
-          ],
-        });
+        return this.arduinoFirmwareUploader.uploadCertificates(params);
       },
     });
 

arduino-ide-extension/src/common/protocol/arduino-firmware-uploader.ts

@@ -3,17 +3,22 @@ import type { Port } from './boards-service';
 export const ArduinoFirmwareUploaderPath =
   '/services/arduino-firmware-uploader';
 export const ArduinoFirmwareUploader = Symbol('ArduinoFirmwareUploader');
-export type FirmwareInfo = {
+export interface FirmwareInfo {
   board_name: string;
   board_fqbn: string;
   module: string;
   firmware_version: string;
   Latest: boolean;
-};
+}
+export interface UploadCertificateParams {
+  readonly fqbn: string;
+  readonly address: string;
+  readonly urls: readonly string[];
+}
 export interface ArduinoFirmwareUploader {
   list(fqbn?: string): Promise<FirmwareInfo[]>;
   flash(firmware: FirmwareInfo, port: Port): Promise<string>;
-  uploadCertificates(params: { flags: string[] }): Promise<any>;
+  uploadCertificates(params: UploadCertificateParams): Promise<unknown>;
   updatableBoards(): Promise<string[]>;
   availableFirmwares(fqbn: string): Promise<FirmwareInfo[]>;
 }

arduino-ide-extension/src/node/arduino-firmware-uploader-impl.ts

@@ -4,6 +4,7 @@ import type { Port } from '../common/protocol';
 import {
   ArduinoFirmwareUploader,
   FirmwareInfo,
+  UploadCertificateParams,
 } from '../common/protocol/arduino-firmware-uploader';
 import { spawnCommand } from './exec-util';
 import { MonitorManager } from './monitor-manager';
@@ -17,8 +18,17 @@ export class ArduinoFirmwareUploaderImpl implements ArduinoFirmwareUploader {
   @inject(MonitorManager)
   private readonly monitorManager: MonitorManager;
 
-  async uploadCertificates(params: { flags: string[] }): Promise<string> {
-    return await this.runCommand(['certificates', 'flash', ...params.flags]);
+  async uploadCertificates(params: UploadCertificateParams): Promise<string> {
+    const { fqbn, address, urls } = params;
+    return await this.runCommand([
+      'certificates',
+      'flash',
+      '-b',
+      fqbn,
+      '-a',
+      address,
+      ...urls.flatMap((url) => ['-u', url]),
+    ]);
   }
 
   async list(fqbn?: string): Promise<FirmwareInfo[]> {