larkin 0.5.8 → 0.5.10

package/dist/runtime/claude-model-catalog.cjs

@@ -30,7 +30,7 @@ async function runClaudeControl(call) {
         };
         const timer = setTimeout(() => finish(new Error("Claude model catalog timed out")), call.timeout);
         timer.unref?.();
-        child.on("error", () => finish(new Error("Claude model catalog command failed")));
+        child.on("error", (error) => finish(new Error(`Claude model catalog command failed: ${error.message}`)));
         child.on("exit", () => finish(new Error("Claude model catalog control channel exited before response")));
         child.stderr.resume();
         child.stdout.on("data", (chunk) => {

package/dist/runtime/claude-model-catalog.mjs

@@ -27,7 +27,7 @@ async function runClaudeControl(call) {
         };
         const timer = setTimeout(() => finish(new Error("Claude model catalog timed out")), call.timeout);
         timer.unref?.();
-        child.on("error", () => finish(new Error("Claude model catalog command failed")));
+        child.on("error", (error) => finish(new Error(`Claude model catalog command failed: ${error.message}`)));
         child.on("exit", () => finish(new Error("Claude model catalog control channel exited before response")));
         child.stderr.resume();
         child.stdout.on("data", (chunk) => {

package/dist/runtime/codex-model-catalog.cjs

@@ -31,7 +31,7 @@ async function runCodexAppServer(call) {
         };
         const timer = setTimeout(() => finish(new Error("Codex model catalog timed out")), call.timeout);
         timer.unref?.();
-        child.on("error", () => finish(new Error("Codex model catalog command failed")));
+        child.on("error", (error) => finish(new Error(`Codex model catalog command failed: ${error.message}`)));
         child.on("exit", () => finish(new Error("Codex model catalog app-server exited before response")));
         child.stderr.resume();
         child.stdout.on("data", (chunk) => {

package/dist/runtime/codex-model-catalog.mjs

@@ -28,7 +28,7 @@ async function runCodexAppServer(call) {
         };
         const timer = setTimeout(() => finish(new Error("Codex model catalog timed out")), call.timeout);
         timer.unref?.();
-        child.on("error", () => finish(new Error("Codex model catalog command failed")));
+        child.on("error", (error) => finish(new Error(`Codex model catalog command failed: ${error.message}`)));
         child.on("exit", () => finish(new Error("Codex model catalog app-server exited before response")));
         child.stderr.resume();
         child.stdout.on("data", (chunk) => {

package/dist/runtime/pi-rpc-client.cjs

@@ -47,11 +47,15 @@ class PiRpcClient {
             this.stderrBuffer = Buffer.concat([this.stderrBuffer, value.subarray(0, this.maxStderrBytes - this.stderrBuffer.length)]);
         });
         process.once("error", (error) => this.fail(new Error(`Pi RPC process failed: ${error.message}`), true));
-        process.once("exit", (code, signal) => {
+        const settle = (code, signal) => {
             this.exited = true;
             const stderr = this.stderrBuffer.toString("utf8").trim().slice(0, 2_000);
             this.fail(new Error(`Pi RPC process exited (${code ?? signal ?? "unknown"})${stderr ? `: ${stderr}` : ""}`), false);
-        });
+        };
+        process.once("exit", settle);
+        // A spawn that never started emits "error" + "close" and never "exit", so
+        // "close" is the only terminal event Node guarantees for failed spawns.
+        process.once("close", settle);
     }
     subscribe(listener) {
         this.eventListeners.add(listener);
@@ -89,11 +93,11 @@ class PiRpcClient {
                     throw new Error("stdin is unavailable");
                 this.process.stdin.write(`${JSON.stringify({ id, type: command, ...fields })}\n`, (error) => {
                     if (error)
-                        this.fail(new Error(`Pi RPC ${command} write failed: ${error.message}`), true);
+                        this.failWrite(command, error);
                 });
             }
             catch (error) {
-                this.fail(new Error(`Pi RPC ${command} write failed: ${error instanceof Error ? error.message : String(error)}`), true);
+                this.failWrite(command, error);
             }
         });
     }
@@ -210,6 +214,24 @@ class PiRpcClient {
     protocolFailure(detail) {
         this.fail(new Error(`Pi RPC protocol error: ${detail}`), true);
     }
+    /**
+     * A failed write matters less than why the process is gone: a spawn failure or
+     * an early exit explains it, and that verdict is queued on the next tick. Defer
+     * the write error by a single turn (next tick runs before the check phase, so a
+     * real spawn/exit verdict always wins) and report it only if no verdict arrives.
+     * No arbitration timer; concurrent write failures collapse into the client's
+     * single first-error terminal state via `fail`'s idempotence.
+     */
+    failWrite(command, error) {
+        if (this.failed)
+            return;
+        const message = `Pi RPC ${command} write failed: ${error instanceof Error ? error.message : String(error)}`;
+        setImmediate(() => {
+            if (this.failed)
+                return;
+            this.fail(new Error(message), true);
+        });
+    }
     fail(error, terminate) {
         if (this.failed)
             return;
@@ -245,6 +267,9 @@ class PiRpcClient {
                 resolve();
             };
             this.process.once("exit", finish);
+            // A failed spawn never emits "exit": settle on "close" instead of waiting
+            // out the grace timers (or never, when nothing else keeps the loop alive).
+            this.process.once("close", finish);
             this.process.kill("SIGTERM");
             forceTimer = setTimeout(() => {
                 if (this.exited) {

package/dist/runtime/pi-rpc-client.mjs

@@ -44,11 +44,15 @@ export class PiRpcClient {
             this.stderrBuffer = Buffer.concat([this.stderrBuffer, value.subarray(0, this.maxStderrBytes - this.stderrBuffer.length)]);
         });
         process.once("error", (error) => this.fail(new Error(`Pi RPC process failed: ${error.message}`), true));
-        process.once("exit", (code, signal) => {
+        const settle = (code, signal) => {
             this.exited = true;
             const stderr = this.stderrBuffer.toString("utf8").trim().slice(0, 2_000);
             this.fail(new Error(`Pi RPC process exited (${code ?? signal ?? "unknown"})${stderr ? `: ${stderr}` : ""}`), false);
-        });
+        };
+        process.once("exit", settle);
+        // A spawn that never started emits "error" + "close" and never "exit", so
+        // "close" is the only terminal event Node guarantees for failed spawns.
+        process.once("close", settle);
     }
     subscribe(listener) {
         this.eventListeners.add(listener);
@@ -86,11 +90,11 @@ export class PiRpcClient {
                     throw new Error("stdin is unavailable");
                 this.process.stdin.write(`${JSON.stringify({ id, type: command, ...fields })}\n`, (error) => {
                     if (error)
-                        this.fail(new Error(`Pi RPC ${command} write failed: ${error.message}`), true);
+                        this.failWrite(command, error);
                 });
             }
             catch (error) {
-                this.fail(new Error(`Pi RPC ${command} write failed: ${error instanceof Error ? error.message : String(error)}`), true);
+                this.failWrite(command, error);
             }
         });
     }
@@ -207,6 +211,24 @@ export class PiRpcClient {
     protocolFailure(detail) {
         this.fail(new Error(`Pi RPC protocol error: ${detail}`), true);
     }
+    /**
+     * A failed write matters less than why the process is gone: a spawn failure or
+     * an early exit explains it, and that verdict is queued on the next tick. Defer
+     * the write error by a single turn (next tick runs before the check phase, so a
+     * real spawn/exit verdict always wins) and report it only if no verdict arrives.
+     * No arbitration timer; concurrent write failures collapse into the client's
+     * single first-error terminal state via `fail`'s idempotence.
+     */
+    failWrite(command, error) {
+        if (this.failed)
+            return;
+        const message = `Pi RPC ${command} write failed: ${error instanceof Error ? error.message : String(error)}`;
+        setImmediate(() => {
+            if (this.failed)
+                return;
+            this.fail(new Error(message), true);
+        });
+    }
     fail(error, terminate) {
         if (this.failed)
             return;
@@ -242,6 +264,9 @@ export class PiRpcClient {
                 resolve();
             };
             this.process.once("exit", finish);
+            // A failed spawn never emits "exit": settle on "close" instead of waiting
+            // out the grace timers (or never, when nothing else keeps the loop alive).
+            this.process.once("close", finish);
             this.process.kill("SIGTERM");
             forceTimer = setTimeout(() => {
                 if (this.exited) {

package/dist/runtime/runtime-readiness.cjs

@@ -181,12 +181,22 @@ function resolveRuntimeExecutable(command, env = process.env) {
 }
 function classifyRuntimePrerequisite(runtime, error, executable) {
     const reason = (error instanceof Error ? error.message : String(error)).replace(/[\r\n]+/g, " ").slice(0, 500);
-    if (/ENOENT|not found|no such file|spawn .* failed/i.test(reason))
+    if (/ENOENT|not found|no such file|spawn .* failed/i.test(reason)) {
+        // A resolved executable that still exists on disk is not a missing install:
+        // the spawn failure comes from the launch environment (working directory,
+        // interpreter, permissions). Keep the raw reason instead of sending the
+        // user to reinstall the runtime.
+        if (executable && node_fs_1.default.existsSync(executable))
+            return {
+                runtime, state: "unavailable", executable, reason,
+                nextAction: `Verify the ${runtime} launch environment (working directory, interpreter, permissions), then retry.`,
+            };
         return {
             runtime, state: "missing", ...(executable ? { executable } : {}),
             reason: `${runtime} is not installed`,
             nextAction: runtimeInstallNextAction(runtime),
         };
+    }
     if (/no authenticated available models|login|credential|unauthenticated|unauthorized|oauth/i.test(reason))
         return {
             runtime, state: "unauthenticated", ...(executable ? { executable } : {}), reason,

package/dist/runtime/runtime-readiness.mjs

@@ -128,12 +128,22 @@ export function resolveRuntimeExecutable(command, env = process.env) {
 }
 export function classifyRuntimePrerequisite(runtime, error, executable) {
     const reason = (error instanceof Error ? error.message : String(error)).replace(/[\r\n]+/g, " ").slice(0, 500);
-    if (/ENOENT|not found|no such file|spawn .* failed/i.test(reason))
+    if (/ENOENT|not found|no such file|spawn .* failed/i.test(reason)) {
+        // A resolved executable that still exists on disk is not a missing install:
+        // the spawn failure comes from the launch environment (working directory,
+        // interpreter, permissions). Keep the raw reason instead of sending the
+        // user to reinstall the runtime.
+        if (executable && fs.existsSync(executable))
+            return {
+                runtime, state: "unavailable", executable, reason,
+                nextAction: `Verify the ${runtime} launch environment (working directory, interpreter, permissions), then retry.`,
+            };
         return {
             runtime, state: "missing", ...(executable ? { executable } : {}),
             reason: `${runtime} is not installed`,
             nextAction: runtimeInstallNextAction(runtime),
         };
+    }
     if (/no authenticated available models|login|credential|unauthenticated|unauthorized|oauth/i.test(reason))
         return {
             runtime, state: "unauthenticated", ...(executable ? { executable } : {}), reason,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "larkin",
-  "version": "0.5.8",
+  "version": "0.5.10",
   "private": false,
   "larkinPackageRole": "npm-published",
   "files": [