supercov 4.24.3 → 4.24.5

package/docs/supported-suites.md

@@ -65,7 +65,8 @@ one run and preserves runner identity wherever the runner exposes it.
 ### Bun
 
 Run `supercov -- bun test` or the package script that launches it. Supercov
-loads a native Bun adapter through `BUN_OPTIONS`, preserving existing Bun
+loads a native Bun adapter before configured preloads in the isolated copy,
+and through `BUN_OPTIONS` for commands without them, preserving existing Bun
 options, configured preloads, and the project's files. Named ESM imports,
 namespace imports, and CommonJS `require('bun:test')` are supported. Bun ignores
 Node's preload settings, so the Node adapter alone cannot attribute its tests.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "supercov",
-  "version": "4.24.3",
+  "version": "4.24.5",
   "description": "Coverage, security and code quality for coding agents",
   "keywords": [
     "coverage",
@@ -123,14 +123,14 @@
     "test:windows-tls": "node scripts/windows-tls-test.mjs"
   },
   "optionalDependencies": {
-    "@supercov/cli-darwin-arm64": "4.24.3",
-    "@supercov/cli-darwin-x64": "4.24.3",
-    "@supercov/cli-linux-arm64-gnu": "4.24.3",
-    "@supercov/cli-linux-arm64-musl": "4.24.3",
-    "@supercov/cli-linux-x64-gnu": "4.24.3",
-    "@supercov/cli-linux-x64-musl": "4.24.3",
-    "@supercov/cli-win32-arm64": "4.24.3",
-    "@supercov/cli-win32-x64": "4.24.3"
+    "@supercov/cli-darwin-arm64": "4.24.5",
+    "@supercov/cli-darwin-x64": "4.24.5",
+    "@supercov/cli-linux-arm64-gnu": "4.24.5",
+    "@supercov/cli-linux-arm64-musl": "4.24.5",
+    "@supercov/cli-linux-x64-gnu": "4.24.5",
+    "@supercov/cli-linux-x64-musl": "4.24.5",
+    "@supercov/cli-win32-arm64": "4.24.5",
+    "@supercov/cli-win32-x64": "4.24.5"
   },
   "peerDependencies": {
     "@playwright/test": ">=1.55.0",

package/runtime/javascript/bunTest.mjs

@@ -1,5 +1,6 @@
-// Bun does not load the Node --import preload. BUN_OPTIONS loads this before
-// its test files and their configured preloads. Older Bun versions create a
+// Bun does not load the Node --import preload. The isolated configuration
+// loads this before configured preloads; BUN_OPTIONS covers other commands.
+// Older Bun versions create a
 // native API per file; newer versions share one. Adapt the factory and the
 // existing API so both keep native test registration and execution semantics.
 import * as runtime from "./runtime.mjs";

package/runtime/javascript/launchSupervisor.mjs

@@ -314,7 +314,7 @@ function wrapResult(value, mapping) {
 // non-writable own data property (and `undefined` for a non-configurable
 // accessor without a getter). Constructors commonly expose `prototype` this
 // way. Returning another capability proxy is a TypeError before user code can
-// run, as seen with PrismaSessionStorage during an Essential Apps VM bake.
+// run, as seen with PrismaSessionStorage during a VM bake.
 function fixedProxyValue(target, property) {
     const descriptor = Reflect.getOwnPropertyDescriptor(target, property);
     if (!descriptor || descriptor.configurable)
@@ -734,14 +734,24 @@ function injectChildEnvironment(method, args) {
 // right after it listened read as one that never became ready, and a
 // Playwright worker as one that "exited unexpectedly".
 const CRASH_SIGNALS = new Set(["SIGSEGV", "SIGBUS", "SIGILL", "SIGFPE", "SIGABRT", "SIGTRAP", "SIGSYS"]);
-function noteCrash(signal, child, command, started) {
-    if (!CRASH_SIGNALS.has(signal))
+// The command a child ran, its arguments too: `spawn("npm", ["run",
+// "build"])` names its build only with them.
+function commandLine(args) {
+    const words = Array.isArray(args[1]) ? args[1] : [];
+    return [args[0], ...words].map((word) => String(word ?? "")).join(" ");
+}
+// A child that a crash signal ended, or that exited non-zero: a build the
+// command started from its own code, which the command's words do not
+// name, is told apart from one it never ran by this.
+function noteExit(signal, code, child, args, started) {
+    const crashed = CRASH_SIGNALS.has(signal);
+    if (!crashed && !(signal == null && typeof code === "number" && code !== 0))
         return;
     record({
         event: "child-exit",
         ...(typeof child === "number" ? { child } : {}),
-        signal,
-        command: safeArgument(String(command ?? "")),
+        ...(crashed ? { signal } : { code }),
+        command: safeArgument(commandLine(args)),
         ranMs: Date.now() - started,
     });
 }
@@ -752,14 +762,14 @@ function watchChild(method, args, run) {
         launched = run();
     }
     catch (error) {
-        // execSync and execFileSync throw for a child a signal ended.
-        noteCrash(error?.signal, error?.pid, args[0], started);
+        // execSync and execFileSync throw for a child that failed.
+        noteExit(error?.signal, error?.status, error?.pid, args, started);
         throw error;
     }
     if (method === "spawnSync")
-        noteCrash(launched?.signal, launched?.pid, args[0], started);
+        noteExit(launched?.signal, launched?.status, launched?.pid, args, started);
     else if (typeof launched?.once === "function")
-        launched.once("exit", (_code, signal) => noteCrash(signal, launched.pid, args[0], started));
+        launched.once("exit", (code, signal) => noteExit(signal, code, launched.pid, args, started));
     return launched;
 }
 function patchChildProcesses() {