shell true

Author: nojaf

lib_dev/process.js

@@ -1,8 +1,19 @@
 import * as child_process from "node:child_process";
 import * as fs from "node:fs/promises";
 import * as path from "node:path";
+import { execPath } from "node:process";
 import { bsc_exe, rescript_legacy_exe } from "#cli/bins";
 
+// On GitHub Actions runners, `npm` is not a native binary but a shell script
+// installed alongside the `node` executable in the hostedtoolcache directory.
+// Spawning it directly by name (`spawn("npm")`) can fail with ENOENT because
+// the PATH in CI sometimes gets polluted with bogus entries (e.g. `/tmp/xfs-*`
+// prepended), causing `execvp` to stop searching before reaching the real npm.
+// To avoid relying on PATH resolution, we resolve `npm` relative to
+// `process.execPath` (the current Node binary) and run it with `shell: true`
+// so the shebang (`#!/usr/bin/env node`) inside the npm script is honoured.
+const npmBin = path.join(path.dirname(execPath), "npm");
+
 /**
  * @typedef {{
  *   throwOnFail?: boolean,
@@ -52,6 +63,7 @@ export function setup(cwd = process.cwd()) {
 
     if (process.env.CI) {
       console.log("[exec] PATH =", options.env?.PATH ?? process.env.PATH);
+      console.log("[npm] using", npmBin);
     }
 
     const stdoutChunks = [];
@@ -256,7 +268,10 @@ export function setup(cwd = process.cwd()) {
      * @return {Promise<ExecResult>}
      */
     npm(args = [], options = {}) {
-      return exec("npm", args, options);
+      return exec(npmBin, args, {
+        ...options,
+        shell: process.platform !== "win32" || options.shell,
+      });
     },
   };
 }