@proagentstore/cli 0.4.36 → 0.4.37

package/dist/browser-runner/coding/runtime.js

@@ -225,12 +225,35 @@ export class CodingRuntime {
     isUnderTakeover(sessionId) {
         return this.takeovers.has(sessionId);
     }
-    /** Stop every session (runner shutdown). */
+    /**
+     * Stop every session (runner shutdown).
+     *
+     * Each `stop()` is isolated (#274 lineage). `stop()` kills a child process, and killing an
+     * already-dead or wedged one throws — so a single bad session used to abort the loop, leaving
+     * every LATER session's engine running and skipping `sessions.clear()` entirely. The caller
+     * (`LocalRunner.close`) wraps this in a `catch {}`, so that escape was silent: the runner
+     * reported a clean shutdown while orphaned CLI processes kept editing the user's repo with
+     * nothing left holding their ids. Same shape as the `browserContext.close()` leak — a throw
+     * on the teardown path is what MAKES the leak, so every session is stopped independently and
+     * the maps always clear.
+     */
     closeAll() {
-        for (const s of this.sessions.values())
-            s.stop();
+        const failures = [];
+        for (const s of this.sessions.values()) {
+            try {
+                s.stop();
+            }
+            catch (e) {
+                failures.push(e);
+            }
+        }
         this.sessions.clear();
         this.takeovers.clear();
+        if (failures.length) {
+            // Cleared the state, then report: teardown completed as far as it could, but the
+            // operator needs to know a child process may have survived it.
+            throw new AggregateError(failures, `${failures.length} coding session(s) failed to stop`);
+        }
     }
     /** True if any session this runtime owns still has a live agent process. */
     hasLiveSessions() {

package/dist/browser-runner/runner.js

@@ -193,8 +193,12 @@ export class LocalRunner {
         try {
             this.coding.closeAll();
         }
-        catch {
-            // a stuck coding session must not block the browser teardown below
+        catch (e) {
+            // A stuck coding session must not block the browser teardown below — but it must not
+            // vanish either. `closeAll` now stops every session independently and only throws to
+            // report which ones refused, so reaching here means a child process may have outlived
+            // the runner. Silence here is what let that read as a clean shutdown.
+            console.warn(`[runner] ${e instanceof Error ? e.message : String(e)} — a CLI process may still be running; check \`ps\` if a repo keeps changing`);
         }
         await this.mcp?.stop().catch(() => undefined);
         this.mcp = null;

package/dist/index.js

@@ -533,7 +533,14 @@ var publishCommand = new Command5("publish").description("Publish an agent to Pr
     writeError("No agent.json found. Run `pags init` first.");
     process.exit(1);
   }
-  const manifest = JSON.parse(readFileSync3(manifestPath, "utf-8"));
+  let manifest;
+  try {
+    manifest = JSON.parse(readFileSync3(manifestPath, "utf-8"));
+  } catch (e) {
+    writeError(`agent.json is not valid JSON: ${e instanceof Error ? e.message : String(e)}`);
+    process.exit(1);
+    return;
+  }
   const slug = manifest.id;
   if (!slug) {
     writeError("agent.json missing id");
@@ -592,8 +599,12 @@ var publishCommand = new Command5("publish").description("Publish an agent to Pr
         cwd: dir,
         stdio: "inherit"
       });
-    } catch {
-      writeLine("  Push skipped (up to date or no commits)");
+    } catch (e) {
+      writeError(`
+  Push failed: ${e instanceof Error ? e.message : String(e)}`);
+      writeError("  Nothing was published \u2014 fix the push (pull/rebase, or check your GitHub auth) and retry.\n");
+      process.exit(1);
+      return;
     }
   }
   writeLine("\n  Registering agent in store...");
@@ -874,10 +885,23 @@ async function connectViaRelay(instanceIds, localUrl, runnerToken, opts, force =
   writeLine(`  Agents:   ${instanceIds.length} instance${instanceIds.length === 1 ? "" : "s"}`);
   writeLine("  No cloudflared needed. Ctrl+C to disconnect.");
   writeLine("\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550");
+  let heartbeatFailing = false;
   const heartbeat = () => {
     const timer = setTimeout(async () => {
+      let failure = null;
       for (const id of [...attached.keys()]) {
-        await requestPags("POST", `/v1/instances/${apiPathSegment(id)}/runtime/heartbeat`, opts, { runnerNode }).catch(() => void 0);
+        try {
+          await requestPags("POST", `/v1/instances/${apiPathSegment(id)}/runtime/heartbeat`, opts, { runnerNode });
+        } catch (e) {
+          failure = e instanceof Error ? e.message : String(e);
+        }
+      }
+      if (failure && !heartbeatFailing) {
+        heartbeatFailing = true;
+        writeError(`Heartbeat failed: ${failure} \u2014 the console will show this machine as OFFLINE until it recovers. The relay itself is still connected; don't run \`pags up --force\` elsewhere.`);
+      } else if (!failure && heartbeatFailing) {
+        heartbeatFailing = false;
+        writeLine("Heartbeat recovered \u2014 this machine reads as online again.");
       }
       heartbeat();
     }, 3e4);
@@ -1499,7 +1523,10 @@ var upCommand = new Command7("up").description("Start the browser runner for all
             stdio: "inherit",
             env: process.env
           });
-        } catch {
+        } catch (e) {
+          const status = e.status;
+          writeLine(`  Restart failed${typeof status === "number" ? ` (exit ${status})` : ""} \u2014 the runner is NOT running. Run 'pags up' again.`);
+          process.exit(typeof status === "number" && status !== 0 ? status : 1);
         }
         process.exit(0);
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@proagentstore/cli",
-  "version": "0.4.36",
+  "version": "0.4.37",
   "description": "CLI for creating, publishing, and running ProAgentStore agents",
   "license": "MIT",
   "type": "module",