@volter/supercode-teams 0.3.139 → 0.3.141

package/machine.mjs

@@ -194,17 +194,20 @@ async function claudeTranscriptHolds(projectsDirs, sessionId, landed) {
   return false;
 }
 /**
- * The doors whose work waits on a session, with the bound each answers within (the node door's own is 30 s): a send or
+ * The doors whose work waits on a session or mailbox, with the bound each answers within (the node door's own is 30 s): a send or
  * a hand-over waits on its receiver's door (a relay's receipt, up to 90 s; another machine's door, MAIL_SEND_MS), a
  * delegation or a declaration that starts a session waits for it to appear (up to a minute). Past its bound a call is
- * answered 504, which its client reads as unanswered: the work may still finish.
+ * answered 504, which its client reads as unanswered: the work may still finish. A board's mailbox long poll gets
+ * its maximum requested wait plus time to return the answer, rather than the ordinary door's shorter bound.
  */
+const BOARD_WAIT_MAX_SECONDS = 120;
 const DOOR_BOUNDS_MS = {
   'harness.v1.mail.send': 170_000,
   'harness.v1.mail.message': 170_000,
   'harness.v1.mail.handover': 140_000,
   'harness.v1.mail.delegate': 110_000,
   'harness.v1.agent.declare': 110_000,
+  'harness.v1.mail.board_wait': (BOARD_WAIT_MAX_SECONDS + 10) * 1000,
 };
 /** How many typed lines' keys a machine remembers; a server retries a line for minutes, not thousands of lines. */
 const TURN_KEYS_KEPT = 1000;
@@ -2387,6 +2390,13 @@ export class TeamsMachine extends MachineCore {
    */
   #deliverInitialInput({ key, contextKey, session, kind, by, input, act, nativeInput, executable = {}, resumed = {} }) {
     if (this.pendingInitialInputs.has(key)) return;
+    // Input confirmation can finish after the child. Keep the launch's latest
+    // status and completion; a late input receipt must not make it ready again.
+    const recordInput = facts => {
+      const launch = this.launches.find(key);
+      if (launch?.pane !== contextKey) return;
+      this.launches.record({ ...launch, ...facts, status: launch.status });
+    };
     const abort = new AbortController();
     const task = (async () => {
       let nativeReadyBy = 'native-prompt';
@@ -2396,7 +2406,7 @@ export class TeamsMachine extends MachineCore {
         const result = nativeReadyBy === 'pane-gone' || nativeReadyBy === 'deadline'
           ? { delivered: false, refused: true, reason: nativeReadyBy }
           : await this.#deliverClaudeInitialInput(session, contextKey, nativeInput, abort.signal);
-        this.launches.record({ key, pane: contextKey, kind, by, ...resumed, ...executable, status: 'ready', inputDelivered: result.delivered,
+        recordInput({ key, pane: contextKey, kind, by, ...resumed, ...executable, inputDelivered: result.delivered,
           inputReadyBy: nativeReadyBy, ...(result.delivered ? { inputReceipt: result.receipt } : { inputError: result.reason }),
           inputRefused: result.refused === true, inputPending: result.queued === true,
           ...(result.native_session ? { native_session: result.native_session } : {}) });
@@ -2412,7 +2422,7 @@ export class TeamsMachine extends MachineCore {
         const typed = nativeReadyBy === 'pane-gone' || nativeReadyBy === 'deadline' ? { delivered: false, reason: nativeReadyBy }
           : await this.#typeCodexFirstInput(session, input, abort.signal);
         if (!typed.delivered) {
-          this.launches.record({ key, pane: contextKey, kind, by, ...resumed, ...executable, status: 'ready', inputDelivered: false,
+          recordInput({ key, pane: contextKey, kind, by, ...resumed, ...executable, inputDelivered: false,
             inputReadyBy: nativeReadyBy, inputError: typed.reason });
           act('launch', `pane:${contextKey}`, 'failed', `first input undelivered: ${typed.reason}`);
           return;
@@ -2421,7 +2431,7 @@ export class TeamsMachine extends MachineCore {
       if (nativeInput) {
         const result = await confirmNativeInitialInput(nativeInput, { signal: abort.signal,
           deadlineMs: Date.now() + INITIAL_INPUT_READY_DEADLINE_MS, live: async () => Boolean(await this.#paneByContext(contextKey)) });
-        this.launches.record({ key, pane: contextKey, kind, by, ...resumed, ...executable, status: 'ready', inputDelivered: result.delivered,
+        recordInput({ key, pane: contextKey, kind, by, ...resumed, ...executable, inputDelivered: result.delivered,
           inputReadyBy: nativeReadyBy, ...(result.delivered ? { inputReceipt: result.receipt } : { inputError: result.reason }),
           ...(result.native_session ? { native_session: result.native_session } : {}) });
         act('launch', `pane:${contextKey}`, result.delivered ? 'ok' : result.queued ? 'input-pending' : 'failed',
@@ -2431,19 +2441,19 @@ export class TeamsMachine extends MachineCore {
       const readyBy = await this.#awaitInputReady(session, contextKey, kind, abort.signal);
       if (readyBy === 'aborted') return;
       if (readyBy === 'pane-gone' || readyBy === 'deadline') {
-        this.launches.record({ key, pane: contextKey, kind, by, ...resumed, ...executable, status: 'ready', inputDelivered: false, inputReadyBy: readyBy });
+        recordInput({ key, pane: contextKey, kind, by, ...resumed, ...executable, inputDelivered: false, inputReadyBy: readyBy });
         act('launch', `pane:${contextKey}`, 'failed', `first input undelivered: ${readyBy}`);
         return;
       }
       const result = await this.#deliverPaneText(session, contextKey, input, { signal: abort.signal });
-      this.launches.record({ key, pane: contextKey, kind, by, ...resumed, ...executable, status: 'ready', inputDelivered: result.delivered,
+      recordInput({ key, pane: contextKey, kind, by, ...resumed, ...executable, inputDelivered: result.delivered,
         inputReadyBy: readyBy, ...(!result.delivered ? { inputError: result.reason } : {}) });
       act('launch', `pane:${contextKey}`, result.delivered ? 'ok' : result.queued ? 'input-pending' : 'failed',
         result.delivered ? `first input delivered (${readyBy})` : `first input unconfirmed: ${result.reason}`);
     })().catch((error) => {
       // Nothing above is expected to throw; if something does, the launch record
       // is where it is answerable, not a crashed daemon.
-      this.launches.record({ key, pane: contextKey, kind, by, ...resumed, ...executable, status: 'ready', inputDelivered: false, inputError: error?.message ?? String(error) });
+      recordInput({ key, pane: contextKey, kind, by, ...resumed, ...executable, inputDelivered: false, inputError: error?.message ?? String(error) });
     }).finally(() => {
       if (this.pendingInitialInputs.get(key)?.task === task) this.pendingInitialInputs.delete(key);
     });
@@ -3199,7 +3209,8 @@ export class TeamsMachine extends MachineCore {
       cancelled_at: found.cancelled_at ?? null,
       live: Boolean(session && !session.dead), label: session?.label ?? null, activeCommand: session?.activeCommand ?? null,
       exit_output: found.pane && (!session || session.dead) ? paneExitOutput(found.pane) : null,
-      exit: session?.dead ? deadPaneExit(session) : found.pane && !session ? paneExit(found.pane) : null,
+      exit: found.completion ?? (found.status === 'finished' ? { failed: false, how: 'status 0' }
+        : session?.dead ? deadPaneExit(session) : found.pane && !session ? paneExit(found.pane) : null),
       ...initialInputFacts(found),
     };
   }
@@ -3499,6 +3510,11 @@ export class TeamsMachine extends MachineCore {
         let existing = this.launches.find(key);
         while (existing) {
           if (existing.status === 'cancelled') throw Object.assign(new Error(`launch key ${key} was cancelled before issuance; use a new key`), { code: 409, reason: 'launch_key_cancelled' });
+          // Finished itself records an observed clean exit, including older
+          // journals that did not keep a separate completion field.
+          if (existing.status === 'finished') {
+            return { key, pane: existing.pane, reused: true, status: 'finished', exit: existing.completion?.how ?? 'status 0' };
+          }
           if (existing.status === 'issued') return { key, pane: existing.pane, reused: true, status: existing.status };
           if (existing.status === 'ready') {
             const session = await this.#paneByContext(existing.pane);
@@ -3508,6 +3524,14 @@ export class TeamsMachine extends MachineCore {
               this.#recoverClaudeInitialInputs([session], key);
               return { key, pane: existing.pane, reused: true, status: existing.status };
             }
+            // The child can finish after creation but before the original
+            // caller's settlement read. Retain its observed exit before a
+            // concurrent or restarted caller can issue this key again.
+            const exit = session?.dead ? deadPaneExit(session) : paneExit(existing.pane);
+            if (exit?.failed === false) {
+              this.launches.record({ ...existing, status: 'finished', completion: exit });
+              return { key, pane: existing.pane, reused: true, status: 'finished', exit: exit.how };
+            }
           }
           break;
         }
@@ -3558,7 +3582,13 @@ export class TeamsMachine extends MachineCore {
         // The agent the launch's environment names is kept with the launch: the pane's session is bound to it once the
         // session is seen in the pane (sessionBindings).
         const agent = launchAgent(params.env);
-        this.launches.record({ key, pane: contextKey, kind: params.kind ?? null, by, ...resumed, ...executable, status: 'ready', ...(agent ? { agent } : {}) });
+        // createSession can return a retained exit descriptor with no pane.
+        // Its first journal write must already contain that completion, before
+        // any title/input/settlement await lets another caller see the key.
+        const createdExit = session.dead ? deadPaneExit(session) : null;
+        this.launches.record({ key, pane: contextKey, kind: params.kind ?? null, by, ...resumed, ...executable,
+          status: createdExit ? createdExit.failed ? 'exited' : 'finished' : 'ready',
+          ...(createdExit?.failed === false ? { completion: createdExit } : {}), ...(agent ? { agent } : {}) });
         if (params.kind === 'codex' && (params.arguments ?? []).includes('--dangerously-bypass-hook-trust')) this.#settleCodexStartup(key, contextKey, session, act);
         act('launch', `pane:${contextKey}`, 'ok', params.kind ?? params.program);
         if (typeof params.name === 'string' && params.name.trim()) {
@@ -3582,9 +3612,10 @@ export class TeamsMachine extends MachineCore {
         const settled = await this.#paneByContext(contextKey);
         if (!settled || settled.dead) {
           // Finished (it ran to a clean end) is an answer; exited (it failed at once) is the caller's error.
-          const exit = settled?.dead ? deadPaneExit(settled) : paneExit(contextKey);
+          const exit = settled?.dead ? deadPaneExit(settled) : session.dead ? deadPaneExit(session) : paneExit(contextKey);
           const status = exit && !exit.failed ? 'finished' : 'exited';
-          this.launches.record({ key, pane: contextKey, kind: params.kind ?? null, by, ...resumed, ...executable, status, error: exit?.failed ? exit.how : undefined });
+          this.launches.record({ ...this.launches.find(key), key, pane: contextKey, kind: params.kind ?? null, by, ...resumed, ...executable, status,
+            ...(exit && !exit.failed ? { completion: exit } : {}), error: exit?.failed ? exit.how : undefined });
           this.#panesChanged();
           return { key, pane: contextKey, reused: false, status, exit: exit?.how ?? null };
         }
@@ -4133,7 +4164,7 @@ export class TeamsMachine extends MachineCore {
         const board = new RegExp(`^sc:${machineName(this.env).replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}:board:[^:\\s]+$`);
         if (method === 'harness.v1.mail.board_wait') {
           const boards = (Array.isArray(params.boards) ? params.boards : []).filter((address) => typeof address === 'string' && board.test(address)).slice(0, 256);
-          const seconds = Math.min(120, Math.max(0, Number.isFinite(Number(params.timeout)) ? Math.floor(Number(params.timeout)) : 60));
+          const seconds = Math.min(BOARD_WAIT_MAX_SECONDS, Math.max(0, Number.isFinite(Number(params.timeout)) ? Math.floor(Number(params.timeout)) : 60));
           const waiting = () => boards.filter((address) => this.mailStore.hasClaimable(address));
           let woken = waiting();
           if (!woken.length && boards.length) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@volter/supercode-teams",
-  "version": "0.3.139",
+  "version": "0.3.141",
   "type": "module",
   "description": "A machine daemon serving this machine's doors, and a local or remote Teams server for shared session discovery, OpenTelemetry ingestion and access to enrolled machines.",
   "license": "MIT",