thinkpool-pair 0.3.1 → 0.3.5

package/bridge.mjs

@@ -135,7 +135,9 @@ const announce = () =>
   channel.send({ type: 'broadcast', event: 'bridge', payload: {
     v: 2, name, repo: repoLabel, branch,
     agents: installedAgents,
-    terms: [...terms.entries()].map(([id, t]) => ({ id, cmd: t.cmd, alive: true })),
+    // cols/rows: the PTY's one true size — web viewers render this grid and
+    // scale it to their own page instead of voting to reflow it.
+    terms: [...terms.entries()].map(([id, t]) => ({ id, cmd: t.cmd, alive: true, cols: t.term.cols, rows: t.term.rows })),
   } })
 
 // One flush timer batches every terminal's pending bytes (~35ms cadence).
@@ -149,12 +151,32 @@ 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
+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 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),
+})
+
 function openTerm({ id, cmd, args = [], attached = false, cols, rows }) {
   if (terms.has(id)) return
+  const ad = attached ? attachedDims() : null
   const term = pty.spawn(cmd, args, {
     name: 'xterm-256color',
-    cols: cols || (attached ? (process.stdout.columns || 100) : 100),
-    rows: rows || (attached ? (process.stdout.rows || 30) : 30),
+    cols: cols || (attached ? ad.cols : (PIN_COLS || 120)),
+    rows: rows || (attached ? ad.rows : (PIN_ROWS || 32)),
     cwd: process.cwd(), env: process.env,
   })
   const entry = { term, cmd, attached, scrollback: '', buf: '' }
@@ -192,10 +214,14 @@ process.stdin.on('data', d => {
   if (t) t.term.write(d.toString('utf8'))
   else if (d.includes(3)) shutdown()   // Ctrl-C once detached/headless
 })
-// attached terminal follows the host's TTY size (web resize never touches it)
+// attached terminal follows the host's TTY size — but never below the
+// floor, and never at all when pinned. Re-announce so viewers rescale.
 process.stdout.on('resize', () => {
   const t = attachedId && terms.get(attachedId)
-  if (t) { try { t.term.resize(process.stdout.columns, process.stdout.rows) } catch { /* noop */ } }
+  if (!t) return
+  const { cols, rows } = attachedDims()
+  if (t.term.cols === cols && t.term.rows === rows) return
+  try { t.term.resize(cols, rows); announce() } catch { /* noop */ }
 })
 
 channel
@@ -205,10 +231,12 @@ channel
     if (t) t.term.write(payload.data)
   })
   .on('broadcast', { event: 'resize' }, ({ payload }) => {
-    // headless terms are web-sized (last writer wins); the attached term is host-sized.
+    // headless terms are web-sized (last writer wins); the attached term is
+    // host-sized. Floor at 80 cols — one narrow phone viewer must not reflow
+    // the shared PTY into a 40-col column for everyone (2026-06-10 bug).
     if (!payload?.cols || !payload?.rows) return
     const t = terms.get(payload.term)
-    if (t && !t.attached) { try { t.term.resize(payload.cols, payload.rows) } catch { /* noop */ } }
+    if (t && !t.attached) { try { t.term.resize(Math.max(payload.cols, 80), Math.max(payload.rows, 20)); announce() } catch { /* noop */ } }
   })
   .on('broadcast', { event: 'term-open' }, ({ payload }) => {
     if (!payload?.id || !payload?.cmd) return

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "thinkpool-pair",
-  "version": "0.3.1",
+  "version": "0.3.5",
   "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" },