larkin 0.5.8 → 0.5.9

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);
@@ -245,6 +249,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);
@@ -242,6 +246,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.9",
   "private": false,
   "larkinPackageRole": "npm-published",
   "files": [