@testmuai/rook 0.1.0 → 0.1.1

package/bin/rook.cjs

@@ -257,9 +257,9 @@ function wireSignalForwarding(
 
 function runViaSystemNode(entry) {
   const major = Number(process.versions.node.split(".")[0]);
-  if (major < 20) {
+  if (major < 22) {
     process.stderr.write(
-      `rook requires Node 20 or newer (found ${process.versions.node}).\n`,
+      `rook requires Node 22 or newer (found ${process.versions.node}).\n`,
     );
     process.exit(1);
   }

package/bin/rook.test.mjs

@@ -1,18 +1,49 @@
-import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
+import { describe, it, expect, vi, beforeEach, afterEach, afterAll } from "vitest";
 import {
   mkdirSync,
+  mkdtempSync,
+  realpathSync,
+  copyFileSync,
   rmSync,
   writeFileSync,
   chmodSync,
   existsSync,
   readFileSync,
 } from "node:fs";
+import { tmpdir } from "node:os";
 import { join, dirname } from "node:path";
-import { fileURLToPath } from "node:url";
+import { fileURLToPath, pathToFileURL } from "node:url";
 import { spawnSync, spawn } from "node:child_process";
 import { EventEmitter } from "node:events";
 
-const BIN_DIR = dirname(fileURLToPath(import.meta.url));
+/**
+ * A COPY of the trampoline, in a directory belonging to this file alone.
+ *
+ * The fixtures below are fake bundled-node runtimes, and they are written at
+ * the HOST's real platform tag on purpose — that is the path the trampoline
+ * resolves, which is the whole point of the test. Written into the real `bin/`,
+ * they are also the path the trampoline resolves for **everyone else**: while
+ * this file ran, any concurrent `rook.cjs` — `src/e2e/smoke.test.ts` spawns one
+ * — picked up `#!/bin/sh\nkill -TERM $$` instead of Node and killed itself.
+ * Two test files sharing one directory, run in parallel by vitest, failing
+ * intermittently with no output and a signal.
+ *
+ * rook.cjs requires nothing but node builtins, so a copy resolves exactly as
+ * the original does from wherever it sits — which makes isolation free.
+ */
+const REAL_BIN_DIR = dirname(fileURLToPath(import.meta.url));
+// realpath, because require.resolve() returns one: on macOS the temp dir is
+// /var/folders/… and resolves as /private/var/folders/…, so the assertions
+// would compare two spellings of the same path.
+const BIN_DIR = realpathSync(mkdtempSync(join(tmpdir(), "rook-trampoline-")));
+copyFileSync(join(REAL_BIN_DIR, "rook.cjs"), join(BIN_DIR, "rook.cjs"));
+
+/** The copy, as a specifier `import()` accepts on every platform. */
+const TRAMPOLINE = pathToFileURL(join(BIN_DIR, "rook.cjs")).href;
+
+afterAll(() => {
+  rmSync(BIN_DIR, { recursive: true, force: true });
+});
 
 function fakePkgRoot(tag) {
   return join(BIN_DIR, "node_modules", "@testmuai", `rook-node-${tag}`);
@@ -60,7 +91,7 @@ describe("rook.cjs bundled runtime resolution", () => {
     Object.defineProperty(process, "platform", { value: "darwin" });
     Object.defineProperty(process, "arch", { value: "arm64" });
 
-    const { findBundledNode } = await import("./rook.cjs?t=" + Date.now());
+    const { findBundledNode } = await import(TRAMPOLINE + "?t=" + Date.now());
     expect(findBundledNode()).toBe(expected);
   });
 
@@ -69,7 +100,7 @@ describe("rook.cjs bundled runtime resolution", () => {
     Object.defineProperty(process, "platform", { value: "darwin" });
     Object.defineProperty(process, "arch", { value: "x64" });
 
-    const { findBundledNode } = await import("./rook.cjs?t=" + Date.now());
+    const { findBundledNode } = await import(TRAMPOLINE + "?t=" + Date.now());
     expect(findBundledNode()).toBe(expected);
   });
 
@@ -78,7 +109,7 @@ describe("rook.cjs bundled runtime resolution", () => {
     Object.defineProperty(process, "platform", { value: "linux" });
     Object.defineProperty(process, "arch", { value: "arm64" });
 
-    const { findBundledNode } = await import("./rook.cjs?t=" + Date.now());
+    const { findBundledNode } = await import(TRAMPOLINE + "?t=" + Date.now());
     expect(findBundledNode()).toBe(expected);
   });
 
@@ -87,7 +118,7 @@ describe("rook.cjs bundled runtime resolution", () => {
     Object.defineProperty(process, "platform", { value: "win32" });
     Object.defineProperty(process, "arch", { value: "x64" });
 
-    const { findBundledNode } = await import("./rook.cjs?t=" + Date.now());
+    const { findBundledNode } = await import(TRAMPOLINE + "?t=" + Date.now());
     expect(findBundledNode()).toBe(expected);
   });
 
@@ -101,7 +132,7 @@ describe("rook.cjs bundled runtime resolution", () => {
     Object.defineProperty(process, "platform", { value: "linux" });
     Object.defineProperty(process, "arch", { value: "x64" });
 
-    const { findBundledNode } = await import("./rook.cjs?t=" + Date.now());
+    const { findBundledNode } = await import(TRAMPOLINE + "?t=" + Date.now());
     expect(findBundledNode()).toBeNull();
   });
 
@@ -111,7 +142,7 @@ describe("rook.cjs bundled runtime resolution", () => {
     Object.defineProperty(process, "platform", { value: "darwin" });
     Object.defineProperty(process, "arch", { value: "arm64" });
 
-    const { findBundledNode } = await import("./rook.cjs?t=" + Date.now());
+    const { findBundledNode } = await import(TRAMPOLINE + "?t=" + Date.now());
     expect(findBundledNode()).toBeNull();
   });
 
@@ -119,7 +150,7 @@ describe("rook.cjs bundled runtime resolution", () => {
     writeFakePackage("darwin-arm64", "node");
     Object.defineProperty(process, "platform", { value: "aix" });
 
-    const { findBundledNode } = await import("./rook.cjs?t=" + Date.now());
+    const { findBundledNode } = await import(TRAMPOLINE + "?t=" + Date.now());
     expect(findBundledNode()).toBeNull();
   });
 });