thinkpool-pair 0.3.5 → 0.4.0

package/bridge.mjs

@@ -7,6 +7,8 @@
      npx thinkpool-pair <ROOM> -- <cmd…>   # share a specific CLI (skip the picker)
      npx thinkpool-pair <ROOM> --headless  # pure relay — terminals open from the web
 
+   v0.4 — room file drops: files dropped/pasted in the web room download
+   to a temp dir here (file-put → file-done) so the agent can read them.
    v0.3 — multi-terminal. The bridge is your machine's presence in the
    room; terminals are things you open inside it. Your first agent runs
    ATTACHED (your own terminal, exactly as before). The web's
@@ -118,6 +120,17 @@ try {
   branch = head.startsWith('ref:') ? head.split('/').pop() : head.slice(0, 7)
 } catch { /* not a git repo — fine */ }
 
+// ── room file drops ────────────────────────────────────────────────
+// Files dropped/pasted in the web room arrive as `file-put { id, name, url }`
+// (a 1h signed Storage URL) — download to a per-room temp dir and answer
+// `file-done { id, path }`. The filename rule <id8>-<safe> is DETERMINISTIC
+// and shared with the web client, which computes host paths from the
+// announced `updir` without waiting on this round-trip. Spec:
+// docs/specs/2026-06-11-code-file-drop.md
+const UPDIR = path.join(os.tmpdir(), 'thinkpool-pair', room)
+const FILE_MAX_BYTES = 10 * 1024 * 1024
+const safeName = (n) => String(n || 'file').replace(/[^a-zA-Z0-9._-]/g, '_').slice(-80)
+
 const supabase = createClient(SUPABASE_URL, SUPABASE_ANON, {
   realtime: { params: { eventsPerSecond: 60 } },
 })
@@ -131,14 +144,33 @@ const terms = new Map()
 let attachedId = null
 let shuttingDown = false
 
+// One send door for every broadcast. supabase-js ≥2.10x deprecation-warns
+// (console.warn) whenever send() silently falls back to REST because the
+// socket can't push — at the flush timer's 28×/s that's a wall of warnings
+// straight into the host's ATTACHED terminal, shredding the TUI
+// (2026-06-11). Same delivery, explicit path, no warning: httpSend when
+// the socket is down. Loss on total offline is fine — scrollback replays.
+const bcast = (event, payload) => {
+  try {
+    if (channel.channelAdapter?.canPush?.() ?? true) {
+      channel.send({ type: 'broadcast', event, payload })
+    } else {
+      channel.httpSend(event, payload).catch(() => { /* offline — replay covers it */ })
+    }
+  } catch { /* noop */ }
+}
+
 const announce = () =>
-  channel.send({ type: 'broadcast', event: 'bridge', payload: {
+  bcast('bridge', {
     v: 2, name, repo: repoLabel, branch,
+    // updir: where room file-drops land (forward-slash normalised — the web
+    // client string-joins host paths onto it; Node accepts `/` on Windows).
+    updir: UPDIR.split(path.sep).join('/'),
     agents: installedAgents,
     // 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).
 const flushAll = () => {
@@ -146,28 +178,31 @@ const flushAll = () => {
     if (!t.buf) continue
     const b64 = Buffer.from(t.buf, 'utf8').toString('base64')
     t.buf = ''
-    channel.send({ type: 'broadcast', event: 'pty-out', payload: { term: id, b64 } })
+    bcast('pty-out', { term: id, b64 })
   }
 }
 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 }) {
@@ -191,7 +226,7 @@ function openTerm({ id, cmd, args = [], attached = false, cols, rows }) {
     terms.delete(id)
     if (id === attachedId) { attachedId = null; detachLocal() }
     if (shuttingDown) return
-    try { channel.send({ type: 'broadcast', event: 'term-exit', payload: { id } }) } catch { /* noop */ }
+    bcast('term-exit', { id })
     announce()
     if (terms.size === 0 && !headless) shutdown()
     else process.stderr.write(`\n  ◇ terminal "${cmd}" ended — still relaying ${terms.size} terminal(s). Ctrl-C to stop.\n`)
@@ -256,13 +291,33 @@ channel
     flushAll()
     for (const [id, t] of terms) {
       if (!t.scrollback) continue
-      channel.send({ type: 'broadcast', event: 'pty-replay', payload: {
+      bcast('pty-replay', {
         to: payload?.to ?? null, term: id,
         b64: Buffer.from(t.scrollback, 'utf8').toString('base64'),
-      } })
+      })
     }
     announce()
   })
+  .on('broadcast', { event: 'file-put' }, ({ payload }) => {
+    if (!payload?.id || !payload?.url) return
+    // Multi-bridge rooms: targeted like term-open; untargeted = solo case.
+    if (payload.host && payload.host !== name) return
+    ;(async () => {
+      try {
+        const r = await fetch(payload.url)
+        if (!r.ok) throw new Error(`HTTP ${r.status}`)
+        const buf = Buffer.from(await r.arrayBuffer())
+        if (buf.length > FILE_MAX_BYTES) throw new Error('file too large')
+        fs.mkdirSync(UPDIR, { recursive: true })
+        const fp = path.join(UPDIR, `${String(payload.id).slice(0, 8)}-${safeName(payload.name)}`)
+        fs.writeFileSync(fp, buf)
+        process.stderr.write(`\n  ◆ file from the room: ${fp}\n`)
+        bcast('file-done', { id: payload.id, path: fp.split(path.sep).join('/'), host: name })
+      } catch (e) {
+        process.stderr.write(`\n  ◇ room file download failed (${safeName(payload.name) || payload.id}): ${e.message}\n`)
+      }
+    })()
+  })
   .on('broadcast', { event: 'who' }, announce)
   .subscribe(status => {
     if (status === 'SUBSCRIBED') {
@@ -281,7 +336,7 @@ function shutdown() {
   clearInterval(flushTimer)
   try {
     const bye = Buffer.from('\r\n[ shared session ended ]\r\n', 'utf8').toString('base64')
-    for (const id of terms.keys()) channel.send({ type: 'broadcast', event: 'pty-out', payload: { term: id, b64: bye } })
+    for (const id of terms.keys()) bcast('pty-out', { term: id, b64: bye })
   } catch { /* noop */ }
   try { supabase.removeChannel(channel) } catch { /* noop */ }
   for (const t of terms.values()) { try { t.term.kill() } catch { /* noop */ } }

package/package.json

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