@wenrwa/marketplace-cli 0.8.3 → 0.8.4

package/dist/__tests__/bridge-session-supersede-exit.test.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/__tests__/bridge-session-supersede-exit.test.js

@@ -0,0 +1,74 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+/**
+ * A re-open on a live id (SessionManager#open supersedes) kills the OLD PTY.
+ * That process exits asynchronously, AFTER the new session is registered
+ * under the same id. Its exit must not drop the NEW session: before this
+ * guard, the old onExit ran `sessions.delete(id)` unconditionally, so the
+ * fresh PTY kept running but untracked (input dropped, attach answered
+ * "Unknown session", nothing could close it) and onSessionExit announced the
+ * agent as ended while it was alive.
+ *
+ * Hit by an explicit restart on either lane: the hosted oc-21 relaunch
+ * (relay `open` on a live id) and the browser's local restart after an
+ * engine/model change (close → open).
+ */
+const sessionManager_1 = require("../bridge/sessionManager");
+class ExitablePty {
+    killed = false;
+    exitCbs = [];
+    write() { }
+    resize() { }
+    kill() { this.killed = true; }
+    onData() { }
+    onExit(cb) { this.exitCbs.push(cb); }
+    exit(code) { for (const cb of this.exitCbs)
+        cb(code); }
+}
+const make = () => {
+    const ptys = [];
+    const factory = { spawn: () => { const p = new ExitablePty(); ptys.push(p); return p; } };
+    const sm = new sessionManager_1.SessionManager(factory);
+    const exits = [];
+    sm.setOnSessionExit((id) => exits.push(id));
+    return { sm, ptys, exits };
+};
+describe('supersede: the old process exiting late never drops the new session', () => {
+    it('re-open, then the OLD pty exits → the new session is still tracked and attachable', async () => {
+        const { sm, ptys, exits } = make();
+        sm.open({ cols: 80, rows: 24, id: 'agent-1' });
+        sm.open({ cols: 80, rows: 24, id: 'agent-1' }); // supersede
+        expect(ptys[0].killed).toBe(true);
+        ptys[0].exit(129); // SIGHUP lands ~600ms later in real life
+        expect(sm.list()).toEqual(['agent-1']);
+        expect(exits).toEqual([]); // the agent did NOT end
+        await expect(sm.attach('agent-1', () => { })).resolves.toBeDefined();
+    });
+    it('close, re-open, then the OLD pty exits → same', () => {
+        const { sm, ptys, exits } = make();
+        sm.open({ cols: 80, rows: 24, id: 'agent-1' });
+        sm.close('agent-1');
+        sm.open({ cols: 80, rows: 24, id: 'agent-1' });
+        ptys[0].exit(129);
+        expect(sm.list()).toEqual(['agent-1']);
+        expect(exits).toEqual([]);
+    });
+    it('the CURRENT pty exiting still ends the session and notifies (unchanged)', () => {
+        const { sm, ptys, exits } = make();
+        sm.open({ cols: 80, rows: 24, id: 'agent-1' });
+        sm.open({ cols: 80, rows: 24, id: 'agent-1' });
+        ptys[1].exit(0);
+        expect(sm.list()).toEqual([]);
+        expect(exits).toEqual(['agent-1']);
+    });
+});
+describe('a plain close still reports the exit (unchanged)', () => {
+    it('close with nothing reopened → onSessionExit fires', () => {
+        const { sm, ptys, exits } = make();
+        sm.open({ cols: 80, rows: 24, id: 'agent-1' });
+        sm.close('agent-1');
+        ptys[0].exit(129);
+        expect(sm.list()).toEqual([]);
+        expect(exits).toEqual(['agent-1']);
+    });
+});

package/dist/bridge/sessionManager.js

@@ -188,6 +188,14 @@ class SessionManager {
             session.exited = true;
             session.sink = null;
             session.snapshot.dispose();
+            // A superseded (or closed-then-reopened) PTY exits AFTER its successor
+            // registered under the same id. That late exit is not this id ending:
+            // dropping the map entry here orphaned the live successor (untracked,
+            // input dropped, unclosable) and announced a running agent as ended.
+            // A plain close (nothing reopened) still notifies, as before.
+            const current = this.sessions.get(id);
+            if (current && current !== session)
+                return;
             this.sessions.delete(id);
             // fcct-26 — tell anyone listening that this session ENDED. Without an
             // outward signal the only way to know a crew member finished its turn is

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wenrwa/marketplace-cli",
-  "version": "0.8.3",
+  "version": "0.8.4",
   "description": "CLI tool for the Wenrwa Agent Marketplace — monitor bounties, agents, workspaces, and treasury from your terminal",
   "author": "Wenrwa <dev@wenrwa.com>",
   "license": "MIT",