@invarn/cibuild 2.0.0 → 2.0.1

package/dist/src/runner-node-step.test.d.ts

@@ -0,0 +1,12 @@
+/**
+ * Node-kind step execution must ride a temp script FILE, never `node -e`.
+ *
+ * The released `ci` binary is pkg-built. pkg's bootstrap intercepts child
+ * spawns of `node` and re-executes the pkg binary itself — and that
+ * bootstrap cannot handle `-e`: it resolves it as an entrypoint path and
+ * dies with `Cannot find module '<cwd>/-e'` before the script runs. A
+ * real file path works under both a system node and the pkg bootstrap,
+ * exactly like bash steps already do via their temp-file pattern.
+ */
+export {};
+//# sourceMappingURL=runner-node-step.test.d.ts.map

package/dist/src/runner-node-step.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"runner-node-step.test.d.ts","sourceRoot":"","sources":["../../src/runner-node-step.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG"}

package/dist/src/runner-node-step.test.js

@@ -0,0 +1,61 @@
+/**
+ * Node-kind step execution must ride a temp script FILE, never `node -e`.
+ *
+ * The released `ci` binary is pkg-built. pkg's bootstrap intercepts child
+ * spawns of `node` and re-executes the pkg binary itself — and that
+ * bootstrap cannot handle `-e`: it resolves it as an entrypoint path and
+ * dies with `Cannot find module '<cwd>/-e'` before the script runs. A
+ * real file path works under both a system node and the pkg bootstrap,
+ * exactly like bash steps already do via their temp-file pattern.
+ */
+import { describe, test, expect, afterEach } from '@jest/globals';
+import { mkdtempSync, readFileSync, rmSync } from 'node:fs';
+import { tmpdir } from 'node:os';
+import { join } from 'node:path';
+import { PipelineRunner, StepExecutionError } from './runner.js';
+import { loadConfig } from './config.js';
+function nodeStep(script) {
+    return {
+        id: 'step_test',
+        name: 'node-under-test',
+        kind: 'node',
+        script,
+    };
+}
+describe('PipelineRunner node-kind steps', () => {
+    let outDir = null;
+    afterEach(() => {
+        if (outDir)
+            rmSync(outDir, { recursive: true, force: true });
+        outDir = null;
+    });
+    test('executes the script from a file: argv[1] is a real path, never -e', async () => {
+        outDir = mkdtempSync(join(tmpdir(), 'cibuild-node-step-'));
+        const probePath = join(outDir, 'probe.json');
+        const runner = new PipelineRunner(loadConfig('/nonexistent/ci.config.json'));
+        await runner.runStep(nodeStep(`require('fs').writeFileSync(${JSON.stringify(probePath)}, JSON.stringify({` +
+            `argv1: process.argv[1] ?? null,` +
+            `filename: typeof __filename === 'string' ? __filename : null` +
+            `}))`));
+        const probe = JSON.parse(readFileSync(probePath, 'utf-8'));
+        // `node -e` leaves argv[1] undefined and has no __filename; a
+        // file-executed script has both, and neither may be the literal '-e'.
+        expect(probe.argv1).not.toBeNull();
+        expect(probe.argv1).not.toBe('-e');
+        expect(probe.filename).not.toBeNull();
+        expect(probe.filename).toMatch(/\.cjs$/);
+    });
+    test('propagates a non-zero exit as StepExecutionError', async () => {
+        const runner = new PipelineRunner(loadConfig('/nonexistent/ci.config.json'));
+        await expect(runner.runStep(nodeStep('process.exit(3)'))).rejects.toThrow(StepExecutionError);
+    });
+    test('cleans up the temp script file after the step exits', async () => {
+        outDir = mkdtempSync(join(tmpdir(), 'cibuild-node-step-'));
+        const probePath = join(outDir, 'probe.json');
+        const runner = new PipelineRunner(loadConfig('/nonexistent/ci.config.json'));
+        await runner.runStep(nodeStep(`require('fs').writeFileSync(${JSON.stringify(probePath)}, JSON.stringify({ filename: __filename }))`));
+        const { filename } = JSON.parse(readFileSync(probePath, 'utf-8'));
+        expect(() => readFileSync(filename)).toThrow();
+    });
+});
+//# sourceMappingURL=runner-node-step.test.js.map

package/dist/src/runner.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"runner.d.ts","sourceRoot":"","sources":["../../src/runner.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,OAAO,EAAE,WAAW,EAAE,QAAQ,EAAE,MAAM,YAAY,CAAC;AAMjE,qBAAa,kBAAmB,SAAQ,KAAK;IAElC,MAAM,EAAE,MAAM;IACd,QAAQ,EAAE,MAAM;IAChB,QAAQ,EAAE,MAAM;gBAFhB,MAAM,EAAE,MAAM,EACd,QAAQ,EAAE,MAAM,EAChB,QAAQ,EAAE,MAAM;CAK1B;AAED,qBAAa,cAAc;IACb,OAAO,CAAC,MAAM;gBAAN,MAAM,EAAE,QAAQ;IAE9B,OAAO,CAAC,IAAI,EAAE,OAAO,EAAE,UAAU,CAAC,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAoJhE,OAAO,CAAC,cAAc;IAYtB,gBAAgB,CAAC,QAAQ,EAAE,WAAW,GAAG,IAAI;IAwB7C,eAAe,CAAC,QAAQ,EAAE,MAAM,EAAE,EAAE,YAAY,EAAE,MAAM,GAAG,IAAI;IAczD,WAAW,CAAC,QAAQ,EAAE,WAAW,EAAE,QAAQ,CAAC,EAAE,MAAM,EAAE,EAAE,YAAY,CAAC,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;CA2CpG"}
+{"version":3,"file":"runner.d.ts","sourceRoot":"","sources":["../../src/runner.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,OAAO,EAAE,WAAW,EAAE,QAAQ,EAAE,MAAM,YAAY,CAAC;AAMjE,qBAAa,kBAAmB,SAAQ,KAAK;IAElC,MAAM,EAAE,MAAM;IACd,QAAQ,EAAE,MAAM;IAChB,QAAQ,EAAE,MAAM;gBAFhB,MAAM,EAAE,MAAM,EACd,QAAQ,EAAE,MAAM,EAChB,QAAQ,EAAE,MAAM;CAK1B;AAED,qBAAa,cAAc;IACb,OAAO,CAAC,MAAM;gBAAN,MAAM,EAAE,QAAQ;IAE9B,OAAO,CAAC,IAAI,EAAE,OAAO,EAAE,UAAU,CAAC,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAkKhE,OAAO,CAAC,cAAc;IAYtB,gBAAgB,CAAC,QAAQ,EAAE,WAAW,GAAG,IAAI;IAwB7C,eAAe,CAAC,QAAQ,EAAE,MAAM,EAAE,EAAE,YAAY,EAAE,MAAM,GAAG,IAAI;IAczD,WAAW,CAAC,QAAQ,EAAE,WAAW,EAAE,QAAQ,CAAC,EAAE,MAAM,EAAE,EAAE,YAAY,CAAC,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;CA2CpG"}

package/dist/src/runner.js

@@ -121,8 +121,23 @@ export -f envman
                     args = ["-e", step.script];
                     break;
                 case "node":
+                    // Write the script to a file instead of passing it via -e: the
+                    // released ci binary is pkg-built, and pkg's bootstrap intercepts
+                    // child spawns of `node`, re-executing the pkg binary itself —
+                    // which resolves `-e` as an entrypoint path and dies with
+                    // `Cannot find module '<cwd>/-e'`. A real file path executes
+                    // under both a system node and the pkg bootstrap.
                     cmd = this.config.interpreters.node;
-                    args = ["-e", step.script];
+                    try {
+                        const tmpDir = mkdtempSync(join(tmpdir(), 'cibuild-script-'));
+                        tmpScriptPath = join(tmpDir, `step-${step.id || 'script'}.cjs`);
+                        writeFileSync(tmpScriptPath, step.script, 'utf-8');
+                    }
+                    catch (err) {
+                        reject(new Error(`Failed to create temporary script file: ${err}`));
+                        return;
+                    }
+                    args = [tmpScriptPath];
                     break;
                 default:
                     reject(new Error(`Unknown step kind: ${step.kind}`));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@invarn/cibuild",
-  "version": "2.0.0",
+  "version": "2.0.1",
   "description": "CI Build CLI — local pipeline orchestration and validation",
   "type": "module",
   "main": "dist/cli.cjs",