@agfpd/iapeer 0.2.83 → 0.2.85

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agfpd/iapeer",
-  "version": "0.2.83",
+  "version": "0.2.85",
   "description": "Foundation core for the iapeer multi-agent ecosystem: identity, registry, storage, codec.",
   "license": "Apache-2.0",
   "author": "agfpd",

package/src/supervisor/daemon.ts

@@ -163,6 +163,13 @@ export function runSupervisorDaemon(opts: SupervisorDaemonOptions): void {
   const clients = new Set<unknown>()
   const writers = new WeakMap<object, BackpressureWriter>()
   const framers = new WeakMap<object, (chunk: Uint8Array) => void>()
+  // FIRST-PAINT TRACKING (Bug B fix): a client gets the full scrollback catch-up EXACTLY ONCE — on its
+  // first FRAME_RESIZE (which runClient always sends on connect, at the real terminal size). Re-dumping
+  // on a later resize STACKS duplicate histories in a terminal that ignores \x1b[3J (Apple Terminal has
+  // NO E3 capability) → smeared/overlapping scroll-up history. After the catch-up the supervisor does
+  // NOTHING on resize — the child redraws its viewport (SIGWINCH) and the terminal reflows its own
+  // scrollback, like a directly-launched claude.
+  const painted = new WeakSet<object>()
   // ATTACHED-CLIENT MARKER (spawn-flip Ф0b-3 slice 3c): keep `<runDir>/<session>.attached` in sync with
   // clients.size — present iff ≥1 operator is attached. The warm-deliver path (a DIFFERENT process)
   // reads it to gate the hosted busy-composer hold: hold ONLY when a human is attached (their unfinished
@@ -247,8 +254,14 @@ export function runSupervisorDaemon(opts: SupervisorDaemonOptions): void {
   } as unknown as Parameters<typeof Bun.spawn>[1]) as unknown as PtyChild
   xterm.onData(resp => child.terminal.write(resp)) // @xterm auto-answers DA1/DSR → back to child
 
-  /** Repaint a (re)attaching client from the headless snapshot: clear+home, then the full grid.
-   *  Backpressure-safe — a ~90 KB heavy snapshot MUST NOT drop past the socket buffer (port-dep #2). */
+  /** Catch up a (re)attaching client to the current screen + history ONCE: clear+home, then the full
+   *  grid from the headless snapshot. Backpressure-safe — a ~90 KB heavy snapshot MUST NOT drop past the
+   *  socket buffer (port-dep #2). Called EXACTLY ONCE per client (its first FRAME_RESIZE). It is NEVER
+   *  re-issued on a later resize: re-dumping stacks a second history copy in a terminal that ignores
+   *  \x1b[3J (Apple Terminal has no E3) → smeared scroll-up history (Bug B). After the first catch-up the
+   *  supervisor stops intervening: a resize just re-sizes the child (applyGeometry → SIGWINCH), and the
+   *  child redraws its own viewport while the terminal reflows its own scrollback — exactly like a
+   *  directly-launched claude. */
   const repaint = (s: { write(b: Buffer): number }): void => {
     const snap = '\x1b[2J\x1b[3J\x1b[H' + stripSpinnerScrollback(ser.serialize(), xterm.rows)
     sendClient(s, frame(FRAME_DATA, Buffer.from(snap, 'utf8')))
@@ -287,7 +300,13 @@ export function runSupervisorDaemon(opts: SupervisorDaemonOptions): void {
     } else if (type === FRAME_RESIZE) {
       const sz = parseSize(payload)
       applyGeometry(sz.cols, sz.rows)
-      repaint(s) // instant paint from the snapshot (delivered COMPLETE via backpressure)
+      if (!painted.has(s as object)) {
+        painted.add(s as object)
+        repaint(s) // FIRST resize for this client = the one-time catch-up (full snapshot at its real size)
+      }
+      // LATER resizes: nothing from the supervisor. applyGeometry already SIGWINCH'd the child → it
+      // redraws its own viewport (forwarded live) and the terminal reflows its own scrollback, exactly
+      // like a directly-launched claude. Re-dumping here would stack history in a no-\x1b[3J terminal.
     }
   }
 
@@ -303,7 +322,11 @@ export function runSupervisorDaemon(opts: SupervisorDaemonOptions): void {
         framers.set(s as object, makeFramer((t, p) => onClientFrame(s, t, p)))
         clients.add(s)
         syncAttached() // ≥1 client now attached → marker present
-        repaint(s) // UNCONDITIONAL repaint on attach (tmux-like)
+        // NO repaint here. A viewer's runClient sends its real terminal size as the FIRST FRAME_RESIZE on
+        // connect (client.ts open → "repaint at our size"); we paint THEN, ONCE, at the correct geometry.
+        // Painting here at the serve geometry (e.g. 220×50) would send a second, wrong-width dump that
+        // STACKS in a terminal ignoring \x1b[3J (Apple Terminal) → smeared scroll-up history (Bug B). A
+        // deliver client (deliverToHost) never resizes and ignores output → it gets no wasted paint.
       },
       data(s: object, d: Uint8Array) {
         framers.get(s)?.(d)