fix(release): scope package runner normalization to Windows

steipete · steipete · commit d95611f538fb · 2026-07-25T01:50:13.000-07:00
diff --git a/scripts/package-openclaw-for-docker.mjs b/scripts/package-openclaw-for-docker.mjs
@@ -222,10 +222,11 @@ function run(command, args, cwd, options = {}) {
     );
     const useProcessGroup = process.platform !== "win32";
     const env = options.env ?? process.env;
+    // Keep POSIX command selection stable; only Windows needs explicit npm/pnpm shim handling.
     const invocation =
-      command === "pnpm"
+      process.platform === "win32" && command === "pnpm"
         ? resolvePnpmRunner({ cwd, env, npmExecPath: env.npm_execpath, pnpmArgs: args })
-        : command === "npm"
+        : process.platform === "win32" && command === "npm"
           ? resolveNpmRunner({ env, npmArgs: args })
           : { args, command, shell: false };
     const child = spawn(invocation.command, invocation.args, {
diff --git a/test/e2e/qa-lab/runtime/package-openclaw-for-docker.e2e.test.ts b/test/e2e/qa-lab/runtime/package-openclaw-for-docker.e2e.test.ts
@@ -124,20 +124,6 @@ describe("package-openclaw-for-docker", () => {
     },
   );
 
-  it.runIf(process.platform !== "win32")(
-    "uses the npm resolver PATH when npm is absent from the caller PATH",
-    async () => {
-      const env = { ...process.env, PATH: "" };
-      const output = await runCommandForTest("npm", ["--version"], process.cwd(), {
-        captureStdout: true,
-        env,
-        timeoutMs: 30_000,
-      });
-
-      expect(output.trim()).toMatch(/^\d+\.\d+\.\d+$/u);
-    },
-  );
-
   it.runIf(process.platform === "win32")(
     "kills pnpm.cmd descendants when the package command times out",
     async () => {
