thinkpool-pair 0.3.5 → 0.3.6

package/bridge.mjs

@@ -152,22 +152,25 @@ const flushAll = () => {
 const flushTimer = setInterval(flushAll, 35)
 
 // ── shared-PTY geometry ───────────────────────────────────────────
-// The room's view is the product. DEFAULT: the shared PTY is PINNED to a
-// standard 120×32 — one source resolution chosen for web legibility
-// (scale ≈1 on a laptop pane), regardless of the host's window. A huge
-// host window made viewers' text microscopic; a tiny one broke their
-// layout — both ends of the same bug (2026-06-10). The host's local
-// terminal just doesn't use its extra space; below 120×32 it wraps
-// oddly locally, the room stays clean.
-// TP_COLS/TP_ROWS pin a different size. TP_FOLLOW=1 restores
-// follow-the-host-window (floored), for solo screen-mirror use.
-const FLOOR_COLS = 100, FLOOR_ROWS = 28
+// The attached PTY is CAPPED at 120×32, never pinned (2026-06-11).
+// Mangling physics: a PTY *wider/taller* than the host's window forces
+// mid-line wraps → the host's own view shreds (the 2026-06-10 pin did
+// exactly this to any window under 120 cols). A PTY *smaller* than the
+// window is harmless — it just leaves an empty margin. So per axis:
+//   attached = min(host TTY, 120×32)
+// Host view is always clean (PTY ≤ window); web viewers never get
+// microscopic text (grid ≤ 120 cols — the original reason for the pin;
+// they render the exact grid and scale, per d78c978).
+// TP_COLS/TP_ROWS force an exact size (you own the consequences);
+// TP_FOLLOW=1 follows the host window uncapped (solo screen-mirror).
+const CAP_COLS = 120, CAP_ROWS = 32
+const FALLBACK_COLS = 100, FALLBACK_ROWS = 28   // stdout size unknown (rare)
 const FOLLOW = process.env.TP_FOLLOW === '1'
-const PIN_COLS = parseInt(process.env.TP_COLS, 10) || (FOLLOW ? null : 120)
-const PIN_ROWS = parseInt(process.env.TP_ROWS, 10) || (FOLLOW ? null : 32)
+const PIN_COLS = parseInt(process.env.TP_COLS, 10) || null
+const PIN_ROWS = parseInt(process.env.TP_ROWS, 10) || null
 const attachedDims = () => ({
-  cols: PIN_COLS || Math.max(process.stdout.columns || FLOOR_COLS, FLOOR_COLS),
-  rows: PIN_ROWS || Math.max(process.stdout.rows || FLOOR_ROWS, FLOOR_ROWS),
+  cols: PIN_COLS || (FOLLOW ? (process.stdout.columns || FALLBACK_COLS) : Math.min(process.stdout.columns || FALLBACK_COLS, CAP_COLS)),
+  rows: PIN_ROWS || (FOLLOW ? (process.stdout.rows || FALLBACK_ROWS) : Math.min(process.stdout.rows || FALLBACK_ROWS, CAP_ROWS)),
 })
 
 function openTerm({ id, cmd, args = [], attached = false, cols, rows }) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "thinkpool-pair",
-  "version": "0.3.5",
+  "version": "0.3.6",
   "description": "Share a local coding-agent CLI (Claude Code, Codex, Gemini, Aider, …) into a ThinkPool Code room, live.",
   "type": "module",
   "bin": { "thinkpool-pair": "bridge.mjs" },