@volter/supercode-teams 0.2.64 → 0.2.66

package/bin/teams.mjs

@@ -80,7 +80,8 @@ if (workspaceVerb) {
     process.stderr.write(JSON.stringify({ version: 1, error: { code: error.code ?? 'invalid_request', message: error.message } }) + '\n');
     process.exitCode = (error.exitCode ?? ({ invalid_request: 2, ambiguous_reference: 2, authentication_required: 3, not_found: 4, forbidden: 4, unsupported: 5, machine_offline: 5, controller_busy: 6, conflict: 6, resync_required: 6, revision_mismatch: 6, idempotency_conflict: 6, batch_gap: 6, capture_conflict: 6, source_paused: 6 }[error.code] ?? 1));
   });
-  if (process.platform !== 'win32') process.exit();
+  // Exit only once stdout has drained: a pipe takes 64 KB at a time, and exiting sooner cut a large --json answer off.
+  if (process.platform !== 'win32') { await new Promise((resolve) => process.stdout.write('', resolve)); process.exit(); }
 }
 
 const home = flags.home ? String(flags.home) : teamsHome();

package/machine.mjs

@@ -116,6 +116,11 @@ function launchCommand(program, args) {
  * user's ~/.config/supercode when the server's environment has none): { failed, how } or null when nothing is
  * recorded. A pane that failed has a `died` line with its status or signal; one that ended cleanly only `closed`.
  */
+/** How a pane the host keeps after its program ended (`dead`) ended, as `paneExit` says it. */
+function deadPaneExit(session) {
+  return Number.isInteger(session.exitCode) ? { failed: session.exitCode !== 0, how: `status ${session.exitCode}` } : { failed: true, how: 'failed' };
+}
+
 /** How often a pane whose launcher asked to be told of its end is looked for. */
 const LAUNCH_END_POLL_MS = 5000;
 
@@ -408,9 +413,12 @@ export class TeamsMachine extends MachineCore {
     const pending = this.#pendingNotices();
     if (!pending.length) { clearInterval(this.endWatch); this.endWatch = null; return; }
     for (const ask of pending) {
-      if (await this.#paneByContext(ask.pane)) continue;
+      // A pane has ended when it is gone, or when its program has exited and the host keeps it for its last screen
+      // (a ConPTY pane whose program failed).
+      const session = await this.#paneByContext(ask.pane);
+      if (session && !session.dead) continue;
       appendFileSync(this.#noticesPath(), JSON.stringify({ t: Date.now(), pane: ask.pane, told: true }) + '\n', { mode: 0o600 });
-      const exit = paneExit(ask.pane);
+      const exit = session?.dead ? deadPaneExit(session) : paneExit(ask.pane);
       const how = !exit ? 'ended (this machine keeps no exit status for it)' : exit.failed ? `failed (${exit.how})` : 'finished (status 0)';
       this.#sendMail(ask.notify.to, {
         op: 'send', from: `sc:${ask.notify.machine}:pane:${ask.pane}`, from_name: `pane ${ask.pane}@${ask.notify.machine}`, to: ask.notify.to, queue: false,
@@ -1129,7 +1137,9 @@ export class TeamsMachine extends MachineCore {
             contextKey,
             cwd: params.cwd,
             env: this.#paneEnv(contextKey, params.env),
-            name: params.name ?? contextKey,
+            // The pane's own tmux session is always its key: the exit hooks record and clean up p_* sessions only, and
+            // a pane named otherwise lingered dead with its exit status lost. A launch's name is its title.
+            name: contextKey,
           });
         } catch (error) {
           this.launches.record({ key, pane: contextKey, kind: params.kind ?? null, status: 'failed', error: error.message });
@@ -1138,6 +1148,12 @@ export class TeamsMachine extends MachineCore {
         }
         this.launches.record({ key, pane: contextKey, kind: params.kind ?? null, status: 'ready' });
         act('launch', `pane:${contextKey}`, 'ok', params.kind ?? params.program);
+        if (typeof params.name === 'string' && params.name.trim()) {
+          const titles = readJson(this.p.titles, { v: 1, titles: {} });
+          titles.titles[`pane:${contextKey}`] = params.name.trim();
+          writeJson(this.p.titles, titles);
+          await this.#renameWindow(contextKey, params.name.trim()).catch(() => null);
+        }
         // The first input is delivered when the pane can TAKE it, not when the
         // pane exists: a program that is still painting its banner has no
         // input box yet, and text sent into one lands half in the scrollback and
@@ -1150,9 +1166,10 @@ export class TeamsMachine extends MachineCore {
         // A program that ends at once (not found, not runnable, failing on its first line) takes its pane with it:
         // the caller is told, with its exit status, instead of being handed a pane that is already gone.
         await new Promise((resolve) => setTimeout(resolve, LAUNCH_SETTLE_MS));
-        if (!await this.#paneByContext(contextKey)) {
+        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 = paneExit(contextKey);
+          const exit = settled?.dead ? deadPaneExit(settled) : paneExit(contextKey);
           const status = exit && !exit.failed ? 'finished' : 'exited';
           this.launches.record({ key, pane: contextKey, kind: params.kind ?? null, status, error: exit?.failed ? exit.how : undefined });
           return { key, pane: contextKey, reused: false, status, exit: exit?.how ?? null };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@volter/supercode-teams",
-  "version": "0.2.64",
+  "version": "0.2.66",
   "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",