@agfpd/iapeer 0.2.84 → 0.2.85

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agfpd/iapeer",
-  "version": "0.2.84",
+  "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,11 +163,12 @@ 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 dump EXACTLY ONCE — on its first
-  // FRAME_RESIZE (which runClient always sends on connect, at the real terminal size). Re-dumping
-  // scrollback on every attach/resize STACKS duplicate histories in a terminal that ignores \x1b[3J
-  // (Apple Terminal has NO E3 capability) → smeared/overlapping scroll-up history. Subsequent resizes
-  // refresh the VIEWPORT only (repaintViewport), never the scrollback.
+  // 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)
@@ -253,22 +254,18 @@ 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')))
   }
-  /** Refresh ONLY the visible viewport (current screen), never the scrollback — for a resize AFTER the
-   *  first full paint. `\x1b[2J\x1b[H` clears the screen and homes WITHOUT touching scrollback (no
-   *  \x1b[3J), then we write exactly the viewport rows (no scrollback segments), so nothing scrolls into
-   *  the client's native scrollback. This is what keeps a mid-session resize from stacking another
-   *  history copy in a terminal that ignores \x1b[3J (Bug B). */
-  const repaintViewport = (s: { write(b: Buffer): number }): void => {
-    const lines = ser.serialize().split('\r\n')
-    const viewport = lines.slice(Math.max(0, lines.length - xterm.rows)).join('\r\n')
-    sendClient(s, frame(FRAME_DATA, Buffer.from('\x1b[2J\x1b[H' + viewport, 'utf8')))
-  }
 
   // Resize the model + child pty to (c, r). Shared by client FRAME_RESIZE and the last-detach restore.
   const applyGeometry = (c: number, r: number): void => {
@@ -305,10 +302,11 @@ export function runSupervisorDaemon(opts: SupervisorDaemonOptions): void {
       applyGeometry(sz.cols, sz.rows)
       if (!painted.has(s as object)) {
         painted.add(s as object)
-        repaint(s) // FIRST paint for this client: full snapshot (scrollback + viewport) at its real size
-      } else {
-        repaintViewport(s) // later resize: refresh the screen only — never re-dump scrollback (anti-stack)
+        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.
     }
   }