@volter/supercode-terminal 0.2.52 → 0.2.54

package/index.d.ts

@@ -252,6 +252,7 @@ export type {
 export type { TerminalInputDelivery, TmuxCommand, TmuxWindowSize };
 
 /** The bounded live composer text, or null for an unrecognized screen. */
-export function composerText(kind: string, screen: string | undefined): string | null;
-/** False means unconfirmed and may leave a draft; never blindly resend. */
-export function deliverComposerInput(host: TmuxTerminalHost | PtyTerminalHost | PaneServerHost, sessionId: string, kind: string, text: string, options?: { deadlineMs?: number; signal?: AbortSignal }): Promise<{ delivered: boolean; reason?: string; delivery?: TerminalInputDelivery }>;
+export function composerText(kind: string, screen: string | undefined, options?: { allowBusy?: boolean }): string | null;
+export function restoreComposerInput(host: TmuxTerminalHost | PtyTerminalHost | PaneServerHost, sessionId: string, kind: string, text: string): Promise<{ restored: boolean; reason?: string; remaining?: string }>;
+/** Failed input is restored or returns an explicit cleanup obligation; never blindly resend. */
+export function deliverComposerInput(host: TmuxTerminalHost | PtyTerminalHost | PaneServerHost, sessionId: string, kind: string, text: string, options?: { deadlineMs?: number; signal?: AbortSignal }): Promise<{ delivered: boolean; reason?: string; restored?: boolean; cleanup_required?: boolean; cleanup_reason?: string; cleanup_remaining?: string; delivery?: TerminalInputDelivery }>;

package/index.mjs

@@ -1,6 +1,6 @@
 import { execFile, spawn, spawnSync } from 'node:child_process';
 import { randomBytes, randomUUID } from 'node:crypto';
-import { appendFileSync, cpSync, existsSync, readFileSync, renameSync, rmSync, writeFileSync } from 'node:fs';
+import { appendFileSync, cpSync, existsSync, mkdirSync, readFileSync, renameSync, rmSync, writeFileSync } from 'node:fs';
 import { homedir } from 'node:os';
 import { dirname, join } from 'node:path';
 import { connect as netConnect } from 'node:net';
@@ -514,11 +514,15 @@ for (const event of ['end', 'close', 'error']) process.stdin.on(event, sweep);
  * signal=…` when it failed), under SUPERCODE_HOME or ~/.config/supercode: the machine daemon tells a clean finish from
  * a failure by it, and a ConPTY pane that ended cleanly is otherwise gone without a trace.
  */
-function recordPaneExit(contextKey, exitCode, signal) {
+function recordPaneExit(contextKey, exitCode, signal, screen = null) {
   if (!contextKey) return;
   const at = new Date().toISOString().slice(0, 19);
   const line = exitCode === 0 && !signal ? `${at} ${contextKey} closed\n` : `${at} ${contextKey} died status=${exitCode ?? ''} signal=${signal || ''}\n`;
   try { appendFileSync(join(process.env.SUPERCODE_HOME || join(homedir(), '.config', 'supercode'), 'pane-exits.log'), line); } catch { /* no log is no worse than before */ }
+  if ((exitCode !== 0 || signal) && typeof screen === 'string' && /^p_[A-Za-z0-9_-]+$/.test(contextKey)) {
+    try { const dir=join(process.env.SUPERCODE_HOME || join(homedir(), '.config', 'supercode'),'pane-exits');mkdirSync(dir,{recursive:true,mode:0o700});writeFileSync(join(dir,`${contextKey}.screen`),Buffer.from(screen).subarray(-32768),{mode:0o600}); }
+    catch { /* The receipt explicitly reports unavailable output. */ }
+  }
 }
 
 export class PtyTerminalHost {
@@ -597,7 +601,7 @@ export class PtyTerminalHost {
         // server) outlives it, as it outlives a tmux pane. What stayed in the pane's console ends with the console.
         // Only closing a pane, or its daemon dying, ends a whole tree.
         releasePane(child);
-        recordPaneExit(entry.contextKey, exitCode, signal);
+        recordPaneExit(entry.contextKey, exitCode, signal, entry.serializer.serialize({scrollback:200}));
         this.#tellExit(entry);
         this.#writeWatchList();
         if (exitCode === 0) this.#forget(entry);
@@ -1734,12 +1738,13 @@ export function composerHoldsDraft(kind, screen) {
 /** Read only a bounded, live Claude/Codex composer, never a prompt in scrollback.
  * Unknown layouts and collapsed pastes are deliberately unconfirmed.
  */
-export function composerText(kind, screen) {
+export function composerText(kind, screen, { allowBusy = false } = {}) {
   if (kind !== 'claude-code' && kind !== 'codex') return null;
   const lines = String(screen ?? '').split('\n').map(styledCells);
   const plain = lines.map((cells) => cells.map((cell) => cell.char).join(''));
+  if (!allowBusy && composerBusy(screen)) return null;
   const rule = /^\s*[─━]{4,}\s*$/u;
-  const footer = /^\s*(?:\? for shortcuts\b|⚠\s+\d+\s+warnings?\s+·\s+f2 to view\s*$|[^\n]+ · (?:~(?:[\\/]|(?=\s|$))|[\\/]|[A-Z]:[\\/])[^\n]*$|.*← for agents\s*$|.*\bcontext(?:\s+\d+%)?\s+left\b)/iu;
+  const footer = /^\s*(?:.*\btab to queue (?:a )?message\b.*$|\? for shortcuts\b|⚠\s+\d+\s+warnings?\s+·\s+f2 to view\s*$|[^\n]+ · (?:~(?:[\\/]|(?=\s|$))|[\\/]|[A-Z]:[\\/])[^\n]*$|.*← for agents\s*$|.*\bcontext(?:\s+\d+%)?\s+left\b)/iu;
   const caret = kind === 'codex' ? /^\s*›(?:\s|$)/u : /^\s*[❯>](?:\s|$)/u;
   for (let row = plain.length - 1; row >= 0; row -= 1) {
     if (!caret.test(plain[row])) continue;
@@ -1786,44 +1791,87 @@ function composerMatches(value, wanted) {
   return offsets.has(wanted.length);
 }
 
-/** Paste once, then submit only text read back from the live composer.
- * False means unconfirmed, and can leave a draft: callers must not blindly resend.
- */
-export async function deliverComposerInput(host, sessionId, kind, text, { deadlineMs = Date.now() + 10_000, signal } = {}) {
-  const read = async () => composerText(kind, await host.capture(sessionId, { preserveEscapes: true }));
-  const expired = () => signal?.aborted || Date.now() >= deadlineMs;
-  const failed = (reason) => ({ delivered: false, reason });
-  if (expired()) return failed('deadline');
-  const before = await read();
-  if (before !== '') return failed(before === null ? 'no-composer' : 'draft');
-  const wanted = String(text).trim();
-  if (!wanted || /[\x00-\x08\x0b-\x1f\x7f]/u.test(text)) return failed('unsupported-text');
-  if (expired()) return failed('deadline');
-  const delivery = await host.sendInput(sessionId, `${text}\n`, { submitMode: 'none', deadlineMs });
-  // A dropped paste must never be followed by Enter. Give the editor time to render it.
-  let pasted = false;
-  for (let attempt = 0; attempt < 10 && !expired(); attempt += 1) {
+/** Busy evidence must come from the live UI bottom, not a fleet label or a
+ * quoted phrase in the transcript. */
+function composerBusy(screen) {
+  const bottom=String(screen??'').split('\n').map(line=>styledCells(line).map(cell=>cell.char).join('')).filter(line=>line.trim()).slice(-8).join('\n');
+  return /\b(?:esc to (?:interrupt|cancel)|tab to queue (?:a )?message)\b/iu.test(bottom);
+}
+
+/** Undo only text this input wrote, including when a turn started meanwhile.
+ * Never Enter, Escape, or Ctrl-C: restoration must not submit or stop work. */
+export async function restoreComposerInput(host, sessionId, kind, text) {
+  const read=async()=>composerText(kind,await host.capture(sessionId,{preserveEscapes:true}),{allowBusy:true});
+  let remaining=await read();
+  if(remaining==='')return {restored:true};
+  if(!composerMatches(remaining,String(text).trim()))return {restored:false,reason:'composer-changed-or-unreadable'};
+  const lines=remaining.split('\n').length+2;
+  for(let line=0;line<lines;line++) {
+    if(await read()!==remaining)return {restored:false,reason:'composer-changed',remaining};
+    await host.sendInput(sessionId,'\u0005\u0015',{submitMode:'none'});
     await sleep(100);
-    const current = await read();
-    if (composerMatches(current, wanted)) { pasted = true; break; }
-    if (current !== '') return failed('paste-unconfirmed');
-  }
-  if (!pasted) return failed('paste-unconfirmed');
-  for (let attempt = 0; attempt < 3; attempt += 1) {
-    // Re-read immediately before each Enter, including the first one.
-    if (expired()) return failed('deadline');
-    if (!composerMatches(await read(), wanted)) return failed('composer-changed');
-    if (expired()) return failed('deadline');
-    // A keystream Enter has no hidden delay or retry between the read and the write.
-    await host.sendInput(sessionId, '\r', { submitMode: 'single', deadlineMs });
-    for (let poll = 0; poll < 10 && !expired(); poll += 1) {
+    let after=await read();
+    if(after==='')return {restored:true};
+    if(after===null||!remaining.includes(after))return {restored:false,reason:'composer-changed',remaining};
+    remaining=after;
+    if(await read()!==remaining)return {restored:false,reason:'composer-changed',remaining};
+    await host.sendInput(sessionId,'\u007f',{submitMode:'none'});
+    await sleep(100);after=await read();
+    if(after==='')return {restored:true};
+    if(after===null||!remaining.replace(/\n/g,'').includes(after.replace(/\n/g,'')))return {restored:false,reason:'composer-changed',remaining};
+    remaining=after;
+  }
+  return {restored:false,reason:'restore-unconfirmed',remaining};
+}
+
+/** Paste only into a stable idle composer. A failed paste is restored; an
+ * unreadable/concurrently changed composer returns an explicit repair obligation. */
+export async function deliverComposerInput(host, sessionId, kind, text, { deadlineMs = Date.now() + 10_000, signal } = {}) {
+  const capture=()=>host.capture(sessionId,{preserveEscapes:true});
+  const read=async()=>composerText(kind,await capture(),{allowBusy:true});
+  const expired=()=>signal?.aborted||Date.now()>=deadlineMs;
+  let touched=false;
+  const failed=async reason=>{
+    if(!touched)return {delivered:false,reason};
+    let restored;
+    try{restored=await restoreComposerInput(host,sessionId,kind,text);}catch(error){restored={restored:false,reason:error.message};}
+    return {delivered:false,reason,restored:restored.restored,cleanup_required:!restored.restored,cleanup_reason:restored.reason,cleanup_remaining:restored.remaining};
+  };
+  if(expired())return failed('deadline');
+  const wanted=String(text).trim();
+  if(!wanted||/[\x00-\x08\x0b-\x1f\x7f]/u.test(text))return failed('unsupported-text');
+  for(let check=0;check<2;check++) {
+    const screen=await capture(),before=composerText(kind,screen);
+    if(composerBusy(screen))return failed('busy-composer');
+    if(before!=='')return failed(before===null?'no-composer':'draft');
+    if(check===0)await sleep(200);
+  }
+  if(expired())return failed('deadline');
+  try {
+    touched=true;
+    const delivery=await host.sendInput(sessionId,`${text}\n`,{submitMode:'none',deadlineMs});
+    let pasted=false;
+    for(let attempt=0;attempt<10&&!expired();attempt++) {
       await sleep(100);
-      const current = await read();
-      if (current === '') return { delivered: true, delivery };
-      if (!composerMatches(current, wanted)) return failed('submission-unconfirmed');
+      if(composerMatches(await read(),wanted)){pasted=true;break;}
     }
-  }
-  return failed('submit-unconfirmed');
+    if(!pasted)return failed('paste-unconfirmed');
+    for(let attempt=0;attempt<3;attempt++) {
+      if(expired())return failed('deadline');
+      const screen=await capture();
+      if(composerBusy(screen))return failed('busy-composer');
+      if(!composerMatches(composerText(kind,screen),wanted))return failed('composer-changed');
+      if(expired())return failed('deadline');
+      await host.sendInput(sessionId,'\r',{submitMode:'single',deadlineMs});
+      for(let poll=0;poll<10&&!expired();poll++) {
+        await sleep(100);
+        const current=await read();
+        if(current==='')return {delivered:true,delivery};
+        if(!composerMatches(current,wanted))return failed('submission-unconfirmed');
+      }
+    }
+    return failed('submit-unconfirmed');
+  }catch(error){return failed(error?.message??String(error));}
 }
 
 /** The carets each kind draws at the start of its composer line (Claude on Windows draws `>`). */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@volter/supercode-terminal",
-  "version": "0.2.52",
+  "version": "0.2.54",
   "type": "module",
   "description": "Optional leak-safe terminal and tmux host adapter for Volter Harness frontends",
   "exports": {

package/substrate/tmux.js

@@ -343,7 +343,8 @@ async function declareSessionColorEnvironmentAsync(socket) {
 // Targets are not expanded as formats, so the shell that run-shell starts resolves
 // the pane and does nothing when it no longer belongs to a p_* session.
 const PANE_EXIT_LOG = '"${SUPERCODE_HOME:-$HOME/.config/supercode}/pane-exits.log"';
-const PANE_DIED_HOOK = "run-shell 'p=\"#{hook_pane}\"; t() { tmux -S \"#{socket_path}\" \"$@\"; }; s=$(t display -p -t \"$p\" \"##{session_name}\" 2>/dev/null) || exit 0; case \"$s\" in p_*) ;; *) exit 0 ;; esac; printf \"%s %s died status=%s signal=%s\\n\" \"$(date +%FT%T)\" \"$s\" \"$(t display -p -t \"$p\" \"##{pane_dead_status}\")\" \"$(t display -p -t \"$p\" \"##{pane_dead_signal}\")\" >> " + PANE_EXIT_LOG + "; t kill-pane -t \"$p\"'";
+const PANE_EXIT_CAPTURE = 'case "$s" in *[!a-zA-Z0-9_-]*) exit 0 ;; esac; d="${SUPERCODE_HOME:-$HOME/.config/supercode}/pane-exits"; umask 077; mkdir -p "$d"; t capture-pane -p -e -S -200 -t "$p" 2>/dev/null | tail -c 32768 > "$d/$s.screen"';
+const PANE_DIED_HOOK = "run-shell 'p=\"#{hook_pane}\"; t() { tmux -S \"#{socket_path}\" \"$@\"; }; s=$(t display -p -t \"$p\" \"##{session_name}\" 2>/dev/null) || exit 0; case \"$s\" in p_*) ;; *) exit 0 ;; esac; printf \"%s %s died status=%s signal=%s\\n\" \"$(date +%FT%T)\" \"$s\" \"$(t display -p -t \"$p\" \"##{pane_dead_status}\")\" \"$(t display -p -t \"$p\" \"##{pane_dead_signal}\")\" >> " + PANE_EXIT_LOG + "; " + PANE_EXIT_CAPTURE + "; t kill-pane -t \"$p\"'";
 const SESSION_CLOSED_HOOK = "if-shell -F '#{m:p_*,#{hook_session_name}}' { run-shell 'printf \"%s %s closed\\n\" \"$(date +%FT%T)\" \"#{hook_session_name}\" >> " + PANE_EXIT_LOG + "' }";
 function sessionServerOptionArgs() {
     return [