akm-cli 0.9.1-beta.1 → 0.9.1-beta.2

package/dist/runtime.js

@@ -78,8 +78,17 @@ function nodeSpawnAdapter(cmd, options) {
         detached: options.detached,
         stdio: [stdioFor(options.stdin), stdioFor(options.stdout), stdioFor(options.stderr)],
     });
+    // Node's 'exit' fires (null, signal) when the child dies from a signal.
+    // Resolving `code ?? 0` reported that as SUCCESS, so an OOM-killed or
+    // segfaulted child came back exit 0 with partial stdout. Bun resolves
+    // 128 + signum for the same case; match it so both runtimes agree and
+    // callers that only check `exitCode !== 0` classify signal deaths correctly.
+    let signalCode = null;
     const exited = new Promise((resolve, reject) => {
-        child.once("exit", (code) => resolve(code ?? 0));
+        child.once("exit", (code, signal) => {
+            signalCode = signal;
+            resolve(code ?? (signal ? 128 + signalNumber(signal) : 0));
+        });
         child.once("error", reject);
     });
     return {
@@ -96,12 +105,25 @@ function nodeSpawnAdapter(cmd, options) {
         get exitCode() {
             return child.exitCode;
         },
+        get signalCode() {
+            return signalCode ?? child.signalCode;
+        },
         pid: child.pid,
         kill(signal) {
             child.kill(signal);
         },
     };
 }
+/**
+ * Map a signal name to its number so a signal death can be reported as the
+ * conventional 128 + signum exit status. Falls back to SIGKILL's 9 for a name
+ * this platform does not define, which still yields a non-zero status — the
+ * property that matters for classifying the run as failed.
+ */
+function signalNumber(signal) {
+    const { constants } = nodeRequire("node:os");
+    return constants.signals[signal] ?? 9;
+}
 // `node:stream`'s Writable.toWeb is available on Node >=17; referenced via the
 // class to avoid a static import that Bun's typings may not expose.
 function Writable_toWeb(w) {