feat: spare detecting invalid sketch name error

The invalid sketch name detection requires at least one extra FS access.
Do not try to detect invalid sketch name error,
but use the original `NotFound` from the CLI.

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

Author: Akos Kitta

Diff for: arduino-ide-extension/src/node/sketches-service-impl.ts

@@ -188,6 +188,13 @@ export class SketchesServiceImpl
   }
 
   async loadSketch(uri: string): Promise<SketchWithDetails> {
+    return this.doLoadSketch(uri);
+  }
+
+  private async doLoadSketch(
+    uri: string,
+    detectInvalidSketchNameError = true
+  ): Promise<SketchWithDetails> {
     const { client, instance } = await this.coreClient;
     const req = new LoadSketchRequest();
     const requestSketchPath = FileUri.fsPath(uri);
@@ -202,17 +209,19 @@ export class SketchesServiceImpl
         if (err) {
           let rejectWith: unknown = err;
           if (isNotFoundError(err)) {
-            const invalidMainSketchFilePath = await isInvalidSketchNameError(
-              err,
-              requestSketchPath
-            );
-            if (invalidMainSketchFilePath) {
-              rejectWith = SketchesError.InvalidName(
-                err.details,
-                FileUri.create(invalidMainSketchFilePath).toString()
+            rejectWith = SketchesError.NotFound(err.details, uri);
+            // detecting the invalid sketch name error is not for free as it requires multiple filesystem access.
+            if (detectInvalidSketchNameError) {
+              const invalidMainSketchFilePath = await isInvalidSketchNameError(
+                err,
+                requestSketchPath
               );
-            } else {
-              rejectWith = SketchesError.NotFound(err.details, uri);
+              if (invalidMainSketchFilePath) {
+                rejectWith = SketchesError.InvalidName(
+                  err.details,
+                  FileUri.create(invalidMainSketchFilePath).toString()
+                );
+              }
             }
           }
           reject(rejectWith);
@@ -318,7 +327,7 @@ export class SketchesServiceImpl
 
       let sketch: Sketch | undefined = undefined;
       try {
-        sketch = await this.loadSketch(uri);
+        sketch = await this.doLoadSketch(uri, false);
         this.logger.debug(
           `Loaded sketch ${JSON.stringify(
             sketch
@@ -391,7 +400,7 @@ export class SketchesServiceImpl
       )) {
         let sketch: SketchWithDetails | undefined = undefined;
         try {
-          sketch = await this.loadSketch(uri);
+          sketch = await this.doLoadSketch(uri, false);
         } catch {}
         if (!sketch) {
           needsUpdate = true;
@@ -414,14 +423,14 @@ export class SketchesServiceImpl
 
   async cloneExample(uri: string): Promise<Sketch> {
     const [sketch, parentPath] = await Promise.all([
-      this.loadSketch(uri),
+      this.doLoadSketch(uri, false),
       this.createTempFolder(),
     ]);
     const destinationUri = FileUri.create(
       path.join(parentPath, sketch.name)
     ).toString();
     const copiedSketchUri = await this.copy(sketch, { destinationUri });
-    return this.loadSketch(copiedSketchUri);
+    return this.doLoadSketch(copiedSketchUri, false);
   }
 
   async createNewSketch(): Promise<Sketch> {
@@ -478,7 +487,7 @@ export class SketchesServiceImpl
       fs.mkdir(sketchDir, { recursive: true }),
     ]);
     await fs.writeFile(sketchFile, inoContent, { encoding: 'utf8' });
-    return this.loadSketch(FileUri.create(sketchDir).toString());
+    return this.doLoadSketch(FileUri.create(sketchDir).toString(), false);
   }
 
   /**
@@ -529,7 +538,7 @@ export class SketchesServiceImpl
     uri: string
   ): Promise<SketchWithDetails | undefined> {
     try {
-      const sketch = await this.loadSketch(uri);
+      const sketch = await this.doLoadSketch(uri, false);
       return sketch;
     } catch (err) {
       if (SketchesError.NotFound.is(err) || SketchesError.InvalidName.is(err)) {
@@ -554,7 +563,7 @@ export class SketchesServiceImpl
     }
     // Nothing to do when source and destination are the same.
     if (sketch.uri === destinationUri) {
-      await this.loadSketch(sketch.uri); // Sanity check.
+      await this.doLoadSketch(sketch.uri, false); // Sanity check.
       return sketch.uri;
     }
 
@@ -575,7 +584,10 @@ export class SketchesServiceImpl
             if (oldPath !== newPath) {
               await fs.rename(oldPath, newPath);
             }
-            await this.loadSketch(FileUri.create(destinationPath).toString()); // Sanity check.
+            await this.doLoadSketch(
+              FileUri.create(destinationPath).toString(),
+              false
+            ); // Sanity check.
             resolve();
           } catch (e) {
             reject(e);
@@ -597,7 +609,7 @@ export class SketchesServiceImpl
   }
 
   async archive(sketch: Sketch, destinationUri: string): Promise<string> {
-    await this.loadSketch(sketch.uri); // sanity check
+    await this.doLoadSketch(sketch.uri, false); // sanity check
     const { client } = await this.coreClient;
     const archivePath = FileUri.fsPath(destinationUri);
     // The CLI cannot override existing archives, so we have to wipe it manually: https://github.com/arduino/arduino-cli/issues/1160