Made the port optional for upload using programmer

Signed-off-by: Akos Kitta <kittaakos@typefox.io>

Author: Akos Kitta

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

@@ -98,20 +98,27 @@ export class UploadSketch extends SketchContribution {
             let options: CoreService.Upload.Options | undefined = undefined;
             const sketchUri = uri;
             const optimizeForDebug = this.editorMode.compileForDebug;
+            const { selectedPort } = boardsConfig;
 
             if (usingProgrammer) {
                 const programmer = selectedProgrammer;
                 if (!programmer) {
                     throw new Error('Programmer is not selected. Please select a programmer.');
                 }
+                let port: undefined | string = undefined;
+                // If the port is set by the user, we pass it to the CLI as it might be required.
+                // If it is not set but the CLI requires it, we let the CLI to complain.
+                if (selectedPort) {
+                    port = selectedPort.address;
+                }
                 options = {
                     sketchUri,
                     fqbn,
                     optimizeForDebug,
-                    programmer
+                    programmer,
+                    port
                 };
             } else {
-                const { selectedPort } = boardsConfig;
                 if (!selectedPort) {
                     throw new Error('No ports selected. Please select a port.');
                 }

arduino-ide-extension/src/common/protocol/core-service.ts

@@ -26,7 +26,7 @@ export namespace CoreService {
     export namespace Upload {
         export type Options =
             Compile.Options & Readonly<{ port: string }> |
-            Compile.Options & Readonly<{ programmer: Programmer }>;
+            Compile.Options & Readonly<{ programmer: Programmer, port?: string }>;
     }
 
 }

arduino-ide-extension/src/node/core-service-impl.ts

@@ -103,11 +103,12 @@ export class CoreServiceImpl implements CoreService {
         uploadReq.setInstance(instance);
         uploadReq.setSketchPath(sketchpath);
         uploadReq.setFqbn(fqbn);
-        if ('port' in options) {
-            uploadReq.setPort(options.port);
-        } else {
+        if ('programmer' in options) {
             uploadReq.setProgrammer(options.programmer.id);
         }
+        if (options.port) {
+            uploadReq.setPort(options.port);
+        }
         const result = client.upload(uploadReq);
 
         try {