@agfpd/iapeer 0.2.83 → 0.2.84

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agfpd/iapeer",
-  "version": "0.2.83",
+  "version": "0.2.84",
   "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,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.
+  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
@@ -253,6 +259,16 @@ export function runSupervisorDaemon(opts: SupervisorDaemonOptions): 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 => {
@@ -287,7 +303,12 @@ 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 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)
+      }
     }
   }
 
@@ -303,7 +324,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)