fix: redundant CLI downloads [HEAD-450][HEAD-504][HEAD-505]

snykTask/src/index.ts

@@ -154,6 +154,7 @@ async function runSnykTest(
     .argIf(taskArgs.organization, `--org=${taskArgs.organization}`)
     .argIf(taskArgs.projectName, `--project-name=${projectNameArg}`)
     .arg(`--json-file-output=${jsonReportOutputPath}`)
+    .argIf(isDebugMode(), '-d')
     .line(taskArgs.additionalArguments);
 
   const options = getOptionsToExecuteSnykCLICommand(

snykTask/src/install/index.ts

@@ -54,17 +54,51 @@ export async function downloadExecutable(
   targetDirectory: string,
   executable: Executable,
 ) {
-  const fileWriter = fs.createWriteStream(
-    path.join(targetDirectory, executable.filename),
-    {
-      mode: 0o766,
-    },
-  );
-  return new Promise<void>((resolve, reject) => {
-    https.get(executable.downloadUrl, (response) => {
-      response.on('end', () => resolve());
-      response.on('error', (err) => reject(err));
-      response.pipe(fileWriter);
-    });
+  const filePath = path.join(targetDirectory, executable.filename);
+
+  // Check if the file already exists
+  if (fs.existsSync(filePath)) {
+    console.log(
+      `File ${executable.filename} already exists, skipping download.`,
+    );
+    return;
+  }
+
+  const fileWriter = fs.createWriteStream(filePath, {
+    mode: 0o766,
   });
+
+  // Wrapping the download in a function for easy retrying
+  const doDownload = () =>
+    new Promise<void>((resolve, reject) => {
+      https.get(executable.downloadUrl, (response) => {
+        response.on('end', () => resolve());
+        response.on('error', (err) => {
+          console.error(
+            `Download of ${executable.filename} failed: ${err.message}`,
+          );
+          reject(err);
+        });
+        response.pipe(fileWriter);
+      });
+    });
+
+  // Try to download the file, retry once after 5 seconds if the first attempt fails
+  try {
+    await doDownload();
+  } catch (err) {
+    console.error(`Download of ${executable.filename} failed: ${err.message}`);
+    console.log(
+      `Retrying download of ${executable.filename} after 5 seconds...`,
+    );
+    await new Promise((resolve) => setTimeout(resolve, 5000));
+    try {
+      await doDownload();
+      console.log(`Retry successful for ${executable.filename}`);
+    } catch (retryErr) {
+      console.error(
+        `Retry failed for ${executable.filename}: ${retryErr.message}`,
+      );
+    }
+  }
 }