chore(release): update execa and fix incompatibility with Listr

Author: christian-bromann

package-lock.json

@@ -154,7 +154,7 @@
     "eslint-plugin-jsdoc": "^48.0.0",
     "eslint-plugin-simple-import-sort": "^12.0.0",
     "eslint-plugin-wdio": "^8.24.12",
-    "execa": "8.0.1",
+    "execa": "9.3.0",
     "exit": "^0.1.2",
     "fs-extra": "^11.0.0",
     "glob": "10.4.1",

package.json

@@ -5,6 +5,29 @@ import { buildAll } from './build';
 import { BuildOptions } from './utils/options';
 import { isPrereleaseVersion, isValidVersionInput, SEMVER_INCREMENTS, updateChangeLog } from './utils/release-utils';
 
+/**
+ * We have to wrap execa in a promise to ensure it works with Listr. Listr uses rxjs under the hood which
+ * seems to have issues with execa's `ResultPromise` as it never resolves a task.
+ * @param command command to run
+ * @param args    arguments to pass to the command
+ * @param options execa options
+ * @returns a promise that resolves with the stdout and stderr of the command
+ */
+async function execa(command: string, args: string[], options?: any) {
+  /**
+   * consecutive imports are cached and don't have an impact on the execution speed
+   */
+  const { execa: execaOrig } = await import('execa');
+
+  return new Promise<{ stdout: string, stderr: string }>((resolve, reject) => {
+    const run = execaOrig(command, args, options);
+    run.then(({ stdout, stderr }) => resolve({
+      stdout: stdout as unknown as string,
+      stderr: stderr as unknown as string
+    }), (err) => reject(err))
+  })
+}
+
 /**
  * Runs a litany of tasks used to ensure a safe release of a new version of Stencil
  * @param opts build options containing the metadata needed to release a new version of Stencil
@@ -18,8 +41,6 @@ export async function runReleaseTasks(opts: BuildOptions, args: ReadonlyArray<st
   const isDryRun = args.includes('--dry-run') || opts.version.includes('dryrun');
   let tagPrefix: string;
 
-  const { execa } = await import('execa');
-
   if (isDryRun) {
     console.log(color.bold.yellow(`\n  🏃‍ Dry Run!\n`));
   }