moshcode 0.74.0 → 0.76.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "moshcode",
-  "version": "0.74.0",
+  "version": "0.76.0",
   "type": "module",
   "description": "moshcode \u2014 a metal wrapper for coding engines and native UGig/CoinPay workflow CLIs, with OpenPRD and moshscript",
   "repository": {

package/src/billing.mjs

@@ -19,6 +19,7 @@ import { spawnSync } from "node:child_process";
 
 import { loadBusiness, loadTimers, newId, updateBusiness, updateTimers } from "./business-store.mjs";
 import { clientLabel, parseFields, resolveClient } from "./clients.mjs";
+import { captureSpec } from "./pty.mjs";
 import { GATEWAYS, defaultGateway, gatewayState } from "./payments.mjs";
 import { chargeFor, describeRate, formatMoney, isDollarPegged, isFiat, rateFor } from "./rates.mjs";
 import { humanDuration, selectEntries, windowFrom } from "./timer.mjs";
@@ -289,7 +290,12 @@ function handOff(record, invoice, business, fields, write, run) {
     return 0;
   }
 
-  const result = run("coinpay", args, { stdio: "inherit" });
+  // Mirrored like every other hand-off: sending an invoice is exactly the kind
+  // of thing you want to read back from the session page afterwards.
+  const launch = captureSpec({ cmd: "coinpay", args });
+  let result;
+  try { result = run(launch.cmd, launch.args, { stdio: "inherit" }); }
+  finally { launch.stop(); }
   if (result?.error) { write(err(String(result.error.message || result.error))); return 1; }
   if (result?.status) { write(err(`coinpay exited ${result.status} — invoice ${record.id} is still a local draft`)); return result.status; }
   updateBusiness((data) => {

package/src/commands.mjs

@@ -22,6 +22,7 @@ import { capture, killSession, remoteStatus, sendPrompt } from "./herd.mjs";
 import { herdStart, isRemoteMember, roster, waitForMany, waitMember } from "./herd-cli.mjs";
 import { endTask, findTask, readTasks, startTask } from "./herd-tasks.mjs";
 import { shellInvocation } from "./shell.mjs";
+import { captureSpec } from "./pty.mjs";
 import { identity, loginAuto, logout as forgetCreds } from "./auth.mjs";
 import { expandAlias, getAlias, loadAliases, removeAlias, setAlias } from "./aliases.mjs";
 import { CORE_CLI_COMMAND_NAMES, PIT_COMMANDS } from "./cli-schema.mjs";
@@ -116,7 +117,15 @@ const SHELL = {
     // has the reasoning, including why a headless run stays non-interactive.
     const { shell: sh, args: shArgs } = shellInvocation(cmd);
     ctx.out(`  ▶ shell: ${cmd}`);
-    const res = spawnSync(sh, shArgs, { stdio: "inherit" });
+    // Captured for the session mirror like the pit's own `!cmd`. A blocking
+    // spawn holds the event loop, so the follower's poll never runs and the
+    // whole command arrives in the drain stop() does — batched rather than
+    // live, which is still the difference between reading it from a phone and
+    // not.
+    const launch = captureSpec({ cmd: sh, args: shArgs });
+    let res;
+    try { res = spawnSync(launch.cmd, launch.args, { stdio: "inherit" }); }
+    finally { launch.stop(); }
     if (res.error) throw res.error;
     const code = res.status ?? 1;
     if (code !== 0) {

package/src/engines.mjs

@@ -32,11 +32,12 @@
 // a session that starts fresh is a small disappointment, and one that starts
 // with a flag the engine does not have is a crash.
 import { spawn } from "node:child_process";
-import { existsSync, mkdtempSync, readFileSync, rmSync, statSync, writeFileSync } from "node:fs";
-import { homedir, tmpdir } from "node:os";
+import { existsSync, readFileSync, statSync } from "node:fs";
+import { homedir } from "node:os";
 import path from "node:path";
 
-import { followFile, ptyEnabled, ptySpec, scriptFlavor, stripScriptBanner } from "./pty.mjs";
+import { setActiveChildInput } from "./mirror.mjs";
+import { captureSpec } from "./pty.mjs";
 
 export const ENGINES = {
   opencode: {
@@ -439,21 +440,29 @@ export function runCmd(cmd, args = [], { capture = false } = {}) {
     let child;
     const spec = spawnSpec(cmd, args);
     const stdio = capture ? ["inherit", "pipe", "pipe"] : "inherit";
-    try { child = spawn(spec.cmd, spec.args, { stdio }); }
-    catch (e) { resolve({ ok: false, error: e }); return; }
+    // The `capture` branch already reaches a watching browser: it re-writes
+    // every byte through this process's own stdout/stderr, which the mirror
+    // tees. The inherited branch does not — those bytes go to the tty and
+    // nowhere else — so it goes under a pty when a mirror is live. This is what
+    // an upgrade, a plugin install and an `mcp add` all run through, and all
+    // three used to be a rule, a blank stretch, and a result line.
+    const launch = capture ? { ...spec, stop: () => {} } : captureSpec(spec);
+    const finish = (result) => { try { launch.stop(); } catch { /* already drained */ } resolve(result); };
+    try { child = spawn(launch.cmd, launch.args, { stdio }); }
+    catch (e) { finish({ ok: false, error: e }); return; }
     let output = "";
     if (capture) {
       for (const [stream, sink] of [[child.stdout, process.stdout], [child.stderr, process.stderr]]) {
         stream?.on("data", (chunk) => { output += chunk.toString(); sink.write(chunk); });
       }
     }
-    child.on("error", (e) => resolve({ ok: false, error: e, output }));
+    child.on("error", (e) => finish({ ok: false, error: e, output }));
     // "exit" fires as soon as the process is gone, which with pipes can leave
     // the last chunk still queued — the one line we are trying to read. "close"
     // waits for the streams too. With stdio inherited there are no streams, so
     // the two are the same moment and existing callers are unaffected; the
     // distinction is kept explicit so neither branch changes by accident.
-    child.on(capture ? "close" : "exit", (code, signal) => resolve({ ok: true, code, signal, output }));
+    child.on(capture ? "close" : "exit", (code, signal) => finish({ ok: true, code, signal, output }));
   });
 }
 
@@ -501,38 +510,27 @@ export function openPassthrough(target, args = [], { onOutput } = {}) {
     // the child the tty's own file descriptors, so none of its bytes ever pass
     // through this process. See src/pty.mjs for why this is script(1) and not
     // a pipe or node-pty.
-    let transcript = null;
-    let workDir = null;
-    let stopFollow = null;
-    let launch = { ...spec, stdio: "inherit" };
-    if (ptyEnabled(onOutput)) {
-      try {
-        workDir = mkdtempSync(path.join(tmpdir(), "moshcode-pty-"));
-        transcript = path.join(workDir, "transcript");
-        writeFileSync(transcript, "");
-        const wrapped = ptySpec(spec.cmd, spec.args, transcript, scriptFlavor());
-        if (wrapped) {
-          launch = { ...wrapped, stdio: "inherit" };
-          let first = true;
-          stopFollow = followFile(transcript, (chunk) => {
-            const clean = stripScriptBanner(chunk, first);
-            first = false;
-            if (clean) onOutput(clean);
-          });
-        }
-      } catch {
-        // Capture is a nicety; never let it stop the session from opening.
-        transcript = null;
-      }
-    }
-
+    //
+    // `input` asks for the same pty to be one we can type into. An engine is
+    // the whole reason it exists: it puts up menus and trust prompts that only
+    // move for a keypress, and until we owned its stdin a session page could
+    // watch one of those appear and had no way to answer it.
+    const launch = captureSpec(spec, onOutput, { input: true });
+    // Route web keys here for as long as this child is up, and only when there
+    // is really somewhere for them to go — an unmirrored pit, or a box with no
+    // `script(1)`, still runs the plain inherited launch, and saying otherwise
+    // would have pressKey silently swallow keys the pit could have handled.
+    // Registered before the spawn on purpose: the fifo buffers, so a key that
+    // arrives while the engine is still starting is delivered, not dropped.
+    const typeable = launch.stdio !== "inherit";
+    if (typeable) setActiveChildInput(launch.write);
     const cleanup = () => {
-      try { stopFollow?.(); } catch { /* nothing left to drain */ }
-      if (workDir) { try { rmSync(workDir, { recursive: true, force: true }); } catch { /* temp dir */ } }
+      if (typeable) setActiveChildInput(null);
+      launch.stop();
     };
 
     let child;
-    try { child = spawn(launch.cmd, launch.args, { stdio: "inherit", env }); }
+    try { child = spawn(launch.cmd, launch.args, { stdio: launch.stdio, env }); }
     catch (e) { cleanup(); resolve({ ok: false, error: e }); return; }
     child.on("error", (e) => { cleanup(); resolve({ ok: false, error: e }); });
     child.on("exit", (code, signal) => { cleanup(); resolve({ ok: true, code, signal }); });

package/src/mirror.mjs

@@ -6,9 +6,12 @@
 // every network call is swallowed, because a flaky link must never take down
 // the terminal you're actually working in.
 //
-// What it can't see: once an engine takes the terminal (`/agents claude`), the
-// child writes straight to the tty on its own fd — those bytes never pass
-// through this process. The mirror shows the hand-off, not the engine's screen.
+// A child that takes the terminal (`/agents claude`) writes straight to the tty
+// on its own fd, so none of its bytes pass through this process on their own.
+// Both directions are handled in src/pty.mjs instead: its output is copied out
+// of a pty transcript, and its stdin is a fifo we hold, which is what lets a key
+// pressed on the session page answer an engine's prompt rather than land in the
+// pit behind it.
 import os from "node:os";
 import { loadCreds } from "./auth.mjs";
 
@@ -30,7 +33,7 @@ export function decodeKey(body) {
 
 // What each key looks like to a program reading the tty in raw mode, and the
 // keypress readline wants when it is the one holding the line.
-const KEY_BYTES = { up: "\u001b[A", down: "\u001b[B", right: "\u001b[C", left: "\u001b[D", enter: "\r" };
+export const KEY_BYTES = { up: "\u001b[A", down: "\u001b[B", right: "\u001b[C", left: "\u001b[D", enter: "\r" };
 const KEY_PRESS = {
   up: { name: "up" }, down: { name: "down" }, right: { name: "right" },
   left: { name: "left" }, enter: { name: "return" },
@@ -43,6 +46,14 @@ const KEY_PRESS = {
 export function pressKey(name, rl = null, stdin = process.stdin) {
   const bytes = KEY_BYTES[name];
   if (!bytes) return false;
+  // A child engine takes precedence over everything below, because when one is
+  // running it is the thing the person on the session page can see. It reads a
+  // real file descriptor rather than this process's stdin object, so the bytes
+  // have to be *written* — the synthesised event further down reaches readline
+  // and the pit's own raw-mode readers, and nothing that was spawned. This is
+  // the line that decides whether an arrow key lands on Claude's trust prompt.
+  const toChild = activeChildInput();
+  if (toChild && toChild(bytes)) return true;
   // At the prompt readline owns the line editor, so hand it a keypress rather
   // than bytes: ↑/↓ walk the history, ←/→ move within the line, enter runs it.
   if (rl) {
@@ -56,6 +67,48 @@ export function pressKey(name, rl = null, stdin = process.stdin) {
 const FLUSH_MS = 150;      // batch writes so a busy render is one request, not fifty
 const MAX_BUFFER = 16000;  // flush early once a batch gets big
 
+// Where a child process's output should be copied while a mirror is watching.
+//
+// Module-level rather than threaded through every call, because "is anyone
+// watching this pit" is one fact about the process and the launchers that need
+// it are scattered: the shell, the installers, the upgrader, the plugin/skill/
+// MCP hand-offs. Passing it down by hand is what left most of them writing
+// straight to the tty with the session page showing nothing — each new launcher
+// had to remember, and none of them did. src/pty.mjs reads this as its default,
+// so capture is what a launcher gets for free and opting out is the deliberate
+// act.
+let activeSink = null;
+
+/** Point child capture at this mirror (or null when the pit stops mirroring). */
+export function setActiveSink(sink) {
+  activeSink = typeof sink === "function" ? sink : null;
+}
+
+/** The sink a child's output should be copied to, or null when unmirrored. */
+export function activeChildSink() {
+  return activeSink;
+}
+
+// The other direction: where to put bytes so the program currently holding the
+// terminal reads them.
+//
+// Null almost always, and set only while a child owns the tty under a pty we
+// opened (src/pty.mjs captureWithInput). It has to be module-level for the same
+// reason the sink does — pressKey is called from the mirror's poll loop, which
+// has no idea which launcher is mid-flight — and it is what makes a key pressed
+// on the session page land in an engine rather than in the pit behind it.
+let activeInput = null;
+
+/** Point web keystrokes at a running child (or null when it exits). */
+export function setActiveChildInput(write) {
+  activeInput = typeof write === "function" ? write : null;
+}
+
+/** How to type into whatever child owns the terminal, or null for the pit. */
+export function activeChildInput() {
+  return activeInput;
+}
+
 export function createMirror({
   version = "",
   cwd = process.cwd(),

package/src/payments.mjs

@@ -22,6 +22,7 @@
 // `/payments connect stripe` records a *reference* — vault and key name — and
 // says out loud where the secret should go.
 import { spawnSync } from "node:child_process";
+import { captureSpec } from "./pty.mjs";
 
 import { loadBusiness, updateBusiness } from "./business-store.mjs";
 import { parseFields } from "./clients.mjs";
@@ -178,7 +179,10 @@ function connectGateway(args, write, run) {
     return 1;
   }
   write(info(`handing you to ${bone(gateway.bin)} — it owns its own session`));
-  const result = run(gateway.bin, gateway.connect, { stdio: "inherit" });
+  const launch = captureSpec({ cmd: gateway.bin, args: gateway.connect });
+  let result;
+  try { result = run(launch.cmd, launch.args, { stdio: "inherit" }); }
+  finally { launch.stop(); }
   if (result?.error) { write(err(String(result.error.message || result.error))); return 1; }
   if (result?.status) {
     write(err(`${gateway.bin} ${gateway.connect.join(" ")} exited ${result.status} — nothing recorded`));

package/src/pty.mjs

@@ -19,8 +19,14 @@
 // `script` disagree on both flag names and argument order, and anything we
 // cannot positively identify falls back to today's plain `inherit`.
 import { spawnSync } from "node:child_process";
-import { closeSync, existsSync, openSync, readSync, statSync } from "node:fs";
+import {
+  closeSync, constants, existsSync, mkdtempSync, openSync,
+  readFileSync, readSync, rmSync, statSync, writeFileSync, writeSync,
+} from "node:fs";
+import { tmpdir } from "node:os";
+import path from "node:path";
 import { StringDecoder } from "node:string_decoder";
+import { activeChildSink } from "./mirror.mjs";
 
 /**
  * POSIX single-quote escaping, for argv that has to survive being flattened
@@ -75,6 +81,18 @@ export function ptySpec(cmd, args = [], transcript, flavor) {
   return null;
 }
 
+/**
+ * The same thing for a shell *line* rather than an argv, which the input path
+ * needs: it prefixes the child with `stty` and `tty` so the session sizes
+ * itself and says where it landed, and those only exist as shell.
+ */
+export function ptyShellSpec(command, transcript, flavor) {
+  if (!command || !transcript) return null;
+  if (flavor === "util-linux") return { cmd: "script", args: ["-q", "-e", "-f", "-c", command, transcript] };
+  if (flavor === "bsd") return { cmd: "script", args: ["-q", "-F", transcript, "sh", "-c", command] };
+  return null;
+}
+
 /**
  * Follow a transcript as it is written, handing each new slice to `onChunk`.
  *
@@ -178,3 +196,272 @@ export function ptyEnabled(sink, flavor = scriptFlavor()) {
   if (process.env.MOSHCODE_MIRROR_PTY === "0") return false;
   return Boolean(flavor);
 }
+
+/**
+ * Wrap a spawn spec so a copy of everything the child prints reaches `onOutput`
+ * while the child still owns the real terminal.
+ *
+ * The whole capture dance in one place — temp transcript, the flavour-specific
+ * `script` argv, the follower, the banner strip, the cleanup — because every
+ * launcher in the pit needs it, and each one growing its own copy is how a
+ * shell command ended up invisible in the mirror while `/agents claude` was
+ * captured: both spawn `inherit`, and only one of them had been taught this.
+ *
+ * `onOutput` defaults to whatever the live mirror is (src/mirror.mjs), so a
+ * launcher gets capture without having to know the mirror exists — the reverse
+ * of how this started, where each launcher had to be taught separately and only
+ * two ever were. Pass `null` to opt a launch out.
+ *
+ * Returns `{ cmd, args, stdio, write, stop }`. With nothing watching, or on a
+ * box with no `script(1)` we can drive, `cmd`/`args` come back exactly as
+ * passed in, `stdio` is "inherit" and `write` returns false — the caller spawns
+ * what it always spawned. `stop()` must be called once the child exits: it
+ * drains the tail of the transcript (the last lines of a command are usually
+ * the ones you were waiting for) and removes the temp dir.
+ *
+ * `input: true` additionally makes the child's stdin something we can type
+ * into, so the session page can drive it — see captureWithInput.
+ */
+export function captureSpec(
+  { cmd, args = [] },
+  onOutput = activeChildSink(),
+  { flavor = scriptFlavor(), input = false, stdin = process.stdin, stdout = process.stdout } = {},
+) {
+  const plain = { cmd, args, stdio: "inherit", write: () => false, stop: () => {} };
+  if (!ptyEnabled(onOutput, flavor)) return plain;
+  if (input) {
+    const withInput = captureWithInput({ cmd, args }, onOutput, { flavor, stdin, stdout });
+    if (withInput) return withInput;
+    // No fifo, no local tty, nothing we could drive — fall through to the
+    // output-only capture rather than dropping capture altogether.
+  }
+  let workDir = null;
+  try {
+    workDir = mkdtempSync(path.join(tmpdir(), "moshcode-pty-"));
+    const transcript = path.join(workDir, "transcript");
+    writeFileSync(transcript, "");
+    const wrapped = ptySpec(cmd, args, transcript, flavor);
+    if (!wrapped) throw new Error("no script(1) spec for this flavour");
+    let first = true;
+    const stopFollow = followFile(transcript, (chunk) => {
+      const clean = stripScriptBanner(chunk, first);
+      first = false;
+      if (clean) onOutput(clean);
+    });
+    const dir = workDir;
+    return {
+      cmd: wrapped.cmd,
+      args: wrapped.args,
+      stdio: "inherit",
+      write: () => false,
+      stop() {
+        try { stopFollow(); } catch { /* nothing left to drain */ }
+        try { rmSync(dir, { recursive: true, force: true }); } catch { /* temp dir */ }
+      },
+    };
+  } catch {
+    // Capture is a nicety; never let it stop a command from running.
+    if (workDir) { try { rmSync(workDir, { recursive: true, force: true }); } catch { /* temp dir */ } }
+    return plain;
+  }
+}
+
+// ---------------------------------------------------------------------------
+// Typing into the child
+// ---------------------------------------------------------------------------
+
+/** A terminal geometry we can hand a child, with a sane floor. */
+function geometry(stdout) {
+  return { cols: Number(stdout?.columns) || 80, rows: Number(stdout?.rows) || 24 };
+}
+
+// Application cursor keys (DECCKM). A program that turns this on is saying "send
+// me ESC O B for down, not ESC [ B", and a terminal obliges — which is why the
+// distinction never comes up for the person at the keyboard, and why it bites
+// the moment we start synthesising keys ourselves. `less` is the plain
+// demonstration: fed the CSI form while it has DECCKM set, it does not scroll,
+// it prints "ESC[B" on its own prompt line as if you had typed the characters.
+//
+// The mode is not something we can ask about, but it is announced: the child
+// writes the escape on its way into full-screen mode, and every byte it writes
+// is already passing under our nose on the way to the mirror.
+const DECCKM = /\u001b\[\?1([hl])/g;
+
+/** Track a DECCKM change announced in `text`; returns the mode after it. */
+export function cursorKeyMode(text, current = false) {
+  const seen = [...String(text).matchAll(DECCKM)].pop();
+  return seen ? seen[1] === "h" : current;
+}
+
+/**
+ * Rewrite CSI cursor keys as SS3, for a child that asked for application mode.
+ *
+ * Only the four cursor keys move: everything else, including a literal ESC and
+ * anything the person at the keyboard typed, is left exactly as it arrived.
+ */
+export function toApplicationCursor(buf) {
+  const out = Buffer.from(buf);
+  for (let i = 0; i + 2 < out.length; i += 1) {
+    // ESC [ A|B|C|D  ->  ESC O A|B|C|D
+    if (out[i] === 0x1b && out[i + 1] === 0x5b && out[i + 2] >= 0x41 && out[i + 2] <= 0x44) {
+      out[i + 1] = 0x4f;
+    }
+  }
+  return out;
+}
+
+/**
+ * The same capture, but with a stdin the mirror can write to.
+ *
+ * `inherit` hands the child the tty's own file descriptors, which is why a key
+ * pressed on the session page could never reach it: there is no fd in this
+ * process between the browser and the program, so the best the mirror could do
+ * was synthesise a `data` event on its own `process.stdin` — which the pit's
+ * readline hears and a child does not (see pressKey in src/mirror.mjs). To type
+ * into an engine we have to own its stdin, and a fifo is the one way to do that
+ * with nothing but the base system: `script(1)` reads it and copies it to the
+ * pty master, exactly as it would a terminal.
+ *
+ * Owning stdin costs two things back, and both are paid here rather than
+ * written off as limitations:
+ *
+ *  - Size. `script` takes the pty's geometry from its own stdin, and a fifo has
+ *    none, so the child would start on a 0x0 terminal — which full-screen
+ *    engines do not survive. Nothing outside a pty can ioctl its master, but
+ *    `stty` inside it can, so the session sizes itself on the way in.
+ *  - Resize. For the same reason `script` can no longer forward SIGWINCH. The
+ *    child records its pty path on the way in, which is enough to resize it
+ *    from out here with `stty -F` when the real window changes, so dragging a
+ *    window edge still reaches the engine.
+ *
+ * The person at the keyboard has to keep working throughout, so local stdin is
+ * relayed byte-for-byte into the same fifo. That means raw mode: this tty has
+ * to stop echoing and stop buffering lines, because the pty on the other end is
+ * now the one doing both.
+ *
+ * Returns null when this box can't do it (no `mkfifo`, no local tty), which
+ * leaves the caller on the output-only path it had before.
+ */
+export function captureWithInput({ cmd, args = [] }, onOutput, { flavor, stdin, stdout } = {}) {
+  // Without a local terminal there is nothing to relay and raw mode is
+  // meaningless, so capture alone is the honest thing to offer.
+  if (!stdin?.isTTY || typeof stdin.setRawMode !== "function") return null;
+
+  let workDir = null;
+  let fd = null;
+  try {
+    workDir = mkdtempSync(path.join(tmpdir(), "moshcode-pty-"));
+    const transcript = path.join(workDir, "transcript");
+    const fifo = path.join(workDir, "input");
+    const ptsFile = path.join(workDir, "pts");
+    writeFileSync(transcript, "");
+
+    // node has no mkfifo, so this is the one call out to the system — and a box
+    // without it simply does not get the input path.
+    const made = spawnSync("mkfifo", [fifo]);
+    if (made.error || made.status !== 0) throw new Error("no mkfifo on this box");
+
+    const { cols, rows } = geometry(stdout);
+    const command = [
+      `tty > ${shQuote(ptsFile)} 2>/dev/null`,
+      `stty rows ${rows} cols ${cols} 2>/dev/null`,
+      // exec, so the engine *is* the process script is waiting on: its signals
+      // and its exit status pass straight through rather than via a shell.
+      `exec ${[cmd, ...args].map(shQuote).join(" ")}`,
+    ].join("; ");
+    const wrapped = ptyShellSpec(command, transcript, flavor);
+    if (!wrapped) throw new Error("no script(1) spec for this flavour");
+
+    // O_RDWR, not O_WRONLY: opening a fifo write-only blocks until a reader
+    // arrives, and the reader here is a child we have not spawned yet. Holding
+    // both ends also keeps the child from seeing EOF between writes.
+    fd = openSync(fifo, constants.O_RDWR);
+
+    let first = true;
+    // Which form of cursor key this child is asking for, learned from the same
+    // stream that goes to the mirror. Tracked on the raw chunk rather than the
+    // banner-stripped one: the mode switch is a control sequence, and nothing
+    // about the banner is in its way.
+    let appCursor = false;
+    const stopFollow = followFile(transcript, (chunk) => {
+      appCursor = cursorKeyMode(chunk, appCursor);
+      const clean = stripScriptBanner(chunk, first);
+      first = false;
+      if (clean) onOutput(clean);
+    });
+
+    let stopped = false;
+    /** Put bytes in front of the child, from the web or from the keyboard. */
+    const write = (data) => {
+      if (stopped || fd === null) return false;
+      const raw = typeof data === "string" ? Buffer.from(data, "latin1") : Buffer.from(data);
+      try { writeSync(fd, appCursor ? toApplicationCursor(raw) : raw); return true; }
+      catch { return false; }
+    };
+
+    // Raw, because the pty on the far end is now the one echoing and the one
+    // splitting lines. Leaving this tty cooked would double every character and
+    // hold Enter back until the child had already redrawn without it.
+    const wasRaw = Boolean(stdin.isRaw);
+    stdin.setRawMode(true);
+    stdin.resume();
+    const onData = (buf) => { write(buf); };
+    stdin.on("data", onData);
+
+    // The child's own tty, once its prelude has written it down. Read lazily:
+    // at the moment we spawn, that file does not exist yet.
+    let pts = null;
+    const childTty = () => {
+      if (pts) return pts;
+      try { pts = readFileSync(ptsFile, "utf8").trim() || null; } catch { pts = null; }
+      return pts;
+    };
+    let resizeTimer = null;
+    const onResize = () => {
+      clearTimeout(resizeTimer);
+      // Dragging an edge fires this continuously; settle for one ioctl per drag.
+      resizeTimer = setTimeout(() => {
+        resizeTimer = null;
+        const tty = childTty();
+        if (!tty || stopped) return;
+        const size = geometry(stdout);
+        // -F on util-linux, -f on BSD/macOS — the same disagreement as the
+        // script(1) flags above, and getting it wrong here is a usage error on
+        // every resize rather than anything visible.
+        const on = flavor === "bsd" ? "-f" : "-F";
+        try { spawnSync("stty", [on, tty, "rows", String(size.rows), "cols", String(size.cols)]); }
+        catch { /* the child owns it; a resize we lose is cosmetic */ }
+      }, 120);
+      resizeTimer.unref?.();
+    };
+    stdout?.on?.("resize", onResize);
+
+    const dir = workDir;
+    return {
+      cmd: wrapped.cmd,
+      args: wrapped.args,
+      // The fifo is the child's stdin; its output still goes straight to the
+      // real terminal, so the engine draws at full speed exactly as before.
+      stdio: [fd, "inherit", "inherit"],
+      write,
+      stop() {
+        if (stopped) return;
+        stopped = true;
+        clearTimeout(resizeTimer);
+        stdout?.off?.("resize", onResize);
+        stdin.off("data", onData);
+        // Hand the terminal back the way we found it. Getting this wrong leaves
+        // the pit with no echo, which reads as a hung shell.
+        try { stdin.setRawMode(wasRaw); } catch { /* not a tty any more */ }
+        stdin.pause();
+        try { stopFollow(); } catch { /* nothing left to drain */ }
+        if (fd !== null) { try { closeSync(fd); } catch { /* already gone */ } fd = null; }
+        try { rmSync(dir, { recursive: true, force: true }); } catch { /* temp dir */ }
+      },
+    };
+  } catch {
+    if (fd !== null) { try { closeSync(fd); } catch { /* already gone */ } }
+    if (workDir) { try { rmSync(workDir, { recursive: true, force: true }); } catch { /* temp dir */ } }
+    return null;
+  }
+}

package/src/tui.mjs

@@ -18,7 +18,7 @@ import { createPrd, listPrds, authoringPrompt } from "./prd.mjs";
 import { loginAuto, whoami, logout } from "./auth.mjs";
 import { startAutoSync } from "./autosync.mjs";
 import { loadCommand, saveCommand } from "./settings-sync.mjs";
-import { createMirror, pressKey, teeOutput } from "./mirror.mjs";
+import { activeChildInput, createMirror, pressKey, setActiveSink, teeOutput } from "./mirror.mjs";
 import { fetchMotdAd } from "./ads.mjs";
 import { runScript } from "./runtime.mjs";
 import { moshVocabulary } from "./commands.mjs";
@@ -28,6 +28,7 @@ import { cryptoCommand } from "./crypto.mjs";
 import { gamesCommand } from "./games.mjs";
 import { canOpenBrowser, openBrowser } from "./open-url.mjs";
 import { shellInvocation } from "./shell.mjs";
+import { captureSpec } from "./pty.mjs";
 import { needsRootHere, primeEscalation } from "./escalate.mjs";
 import { banner, hr, acid, ash, bone, dim, ok, err, warn, info, moshcodeVersion } from "./ui.mjs";
 import { CORE_CLI_COMMAND_NAMES } from "./cli-schema.mjs";
@@ -678,14 +679,23 @@ async function openWorkflowTool(key, tool, args) {
 // command string → `$SHELL +m -ic "<cmd>"` (one-off). Interactive so the command
 // can see the aliases and functions in ~/.zshrc — see src/shell.mjs for why
 // that is not optional. Resolves { ok, code, signal }.
-function runShell(rawCmd) {
+//
+// Captured through a pty when the mirror is watching, for the same reason an
+// engine is: a shell command is where most of what a pit does actually happens
+// — `!cmd`, /shell, and every shell-valued /alias land here — and with plain
+// `inherit` none of its bytes, on stdout or stderr, ever pass through this
+// process. The session page was showing the echoed command line and the exit
+// note with nothing in between.
+export function runShell(rawCmd, { onOutput } = {}) {
   return new Promise((resolve) => {
     const { shell, args } = shellInvocation(rawCmd);
+    const launch = captureSpec({ cmd: shell, args }, onOutput);
+    const done = (result) => { try { launch.stop(); } catch { /* already drained */ } resolve(result); };
     let child;
-    try { child = spawn(shell, args, { stdio: "inherit" }); }
-    catch (e) { resolve({ ok: false, error: e }); return; }
-    child.on("error", (e) => resolve({ ok: false, error: e }));
-    child.on("exit", (code, signal) => resolve({ ok: true, code, signal }));
+    try { child = spawn(launch.cmd, launch.args, { stdio: "inherit" }); }
+    catch (e) { done({ ok: false, error: e }); return; }
+    child.on("error", (e) => done({ ok: false, error: e }));
+    child.on("exit", (code, signal) => done({ ok: true, code, signal }));
   });
 }
 
@@ -700,7 +710,7 @@ async function openShell(rawCmd) {
     ? `${bone(shellName)} ${ash(flags)} ${ash(rawCmd)}`
     : `dropping to ${bone(shellName)} — ${ash("`exit` or Ctrl-D brings you back to the pit")}`));
   console.log(hr());
-  const r = await runShell(rawCmd);
+  const r = await runShell(rawCmd, { onOutput: childSink() });
   console.log(hr());
   if (!r.ok) {
     console.log(err(`couldn't start shell: ${r.error?.message || r.error}`));
@@ -720,14 +730,22 @@ function installTarget(key) {
     // something the installer's output scrolled into view.
     if (needsRootHere(target)) primeEscalation({ what: key, out: (s) => console.log(info(s.replace(/^· /, ""))) });
     console.log(hr());
-    const child = spawn(target.install.cmd, target.install.args, { stdio: "inherit" });
+    // Installers are long, chatty, and the thing you most want to read from a
+    // phone — so they go through the mirror's pty like everything else.
+    const launch = captureSpec(
+      { cmd: target.install.cmd, args: target.install.args },
+      childSink(),
+    );
+    const child = spawn(launch.cmd, launch.args, { stdio: "inherit" });
     child.on("error", (e) => {
+      launch.stop();
       console.log(hr());
       console.log(err(`install failed: ${e.message}`));
       if (e.code === "ENOENT" && target.installHelp) console.log(info(target.installHelp));
       resolve();
     });
     child.on("exit", (code) => {
+      launch.stop();
       console.log(hr());
       if (code !== 0) { console.log(err(`install exited ${code}`)); return resolve(); }
       console.log(ok(`${key} installed. 🤘`));
@@ -1292,6 +1310,10 @@ async function startMirror() {
   if (!started) return noop;
 
   activeMirror = mirror;
+  // Every launcher that spawns a child reads this rather than being handed a
+  // sink, so a command run from the pit is captured whether or not whoever
+  // wrote that launcher knew the mirror existed.
+  setActiveSink((chunk) => activeMirror?.write(chunk));
   const restoreTee = teeOutput((chunk) => mirror.write(chunk));
 
   // Commands arrive whenever; the prompt is only ready between engine
@@ -1310,7 +1332,18 @@ async function startMirror() {
       promptRl.write(`${body}\n`);
     }
   };
-  mirror.onCommand((body) => { queue.push(body); drainRemote(); });
+  mirror.onCommand((body) => {
+    // An engine has the terminal: send the line to it rather than parking it
+    // for a prompt that will not come back until the engine exits. Without this
+    // the arrow keys could answer a menu but nothing could answer a question,
+    // which is half a session page. Typed straight in, so it arrives the way
+    // the keyboard would deliver it — no `▸ (web)` note, because the engine
+    // echoes it itself and printing over an engine's screen shifts it.
+    const toChild = activeChildInput();
+    if (toChild && toChild(`${body}\r`)) return;
+    queue.push(body);
+    drainRemote();
+  });
 
   // Keys skip the queue: they are pressed the instant they arrive, whether the
   // prompt is armed or something else has the tty (a herd bar, the reader, a
@@ -1326,6 +1359,7 @@ async function startMirror() {
 async function stopMirror(restoreTee) {
   const mirror = activeMirror;
   activeMirror = null;
+  setActiveSink(null);
   try { restoreTee?.(); } catch { /* noop */ }
   try { await mirror?.stop(); } catch { /* best effort */ }
 }