thinkpool-pair 0.6.22 → 0.6.24

package/bridge.mjs

@@ -184,9 +184,10 @@ const forceContinue = ownArgs.includes('--continue')
 const forceFresh    = ownArgs.includes('--fresh')
 
 // One yes/no on the bridge's stderr (same channel as the agent picker).
-const askYesNo = (q) => new Promise((resolve) => {
+// defaultYes=true → bare Enter means yes; false → Enter means no (opt-in).
+const askYesNo = (q, defaultYes = true) => new Promise((resolve) => {
   const rl = readline.createInterface({ input: process.stdin, output: process.stderr })
-  rl.question(q, (ans) => { rl.close(); resolve(!/^n/i.test(String(ans).trim())) })
+  rl.question(q, (ans) => { rl.close(); const a = String(ans).trim(); resolve(defaultYes ? !/^n/i.test(a) : /^y/i.test(a)) })
 })
 
 let attachedCmd = null, attachedArgs = [], continuing = false
@@ -227,10 +228,12 @@ if (process.stdin.isTTY && !headless && process.env.THINKPOOL_PAIR_AUTOUPDATE !=
   let declined = false
   try { declined = fs.existsSync(declineFile) } catch { /* noop */ }
   if (!declined) {
-    const yes = await askYesNo(`\n  Keep this bridge running with auto-updates (survives reboot, no terminal to babysit)?\n  Install it as a background service? [Y/n] `)
+    // Default NO — a bare Enter keeps it foreground (the expected "I ran it in
+    // my terminal" behaviour); installing a background daemon is deliberate opt-in.
+    const yes = await askYesNo(`\n  Run as a background service instead? It survives reboot + auto-updates,\n  but runs detached — you'd stop it with 'uninstall-service', not Ctrl-C.\n  Install background service? [y/N] `, false)
     if (yes) {
       svc.installService(room, attachedCmd ? [attachedCmd, ...attachedArgs] : [])
-      process.stderr.write(`  ◆ the background service is handling room ${room} now — you can close this terminal.\n\n`)
+      process.stderr.write(`\n  ◆ the background service is handling room ${room} now — you can close this terminal.\n  ◆ STOP IT with:  npx thinkpool-pair@latest uninstall-service ${room}\n     (closing the terminal does NOT stop it — it's detached under launchd/systemd.)\n\n`)
       process.exit(0)   // hand off to the supervised service (no double bridge)
     }
     try { fs.mkdirSync(path.dirname(declineFile), { recursive: true }); fs.writeFileSync(declineFile, new Date().toISOString()) } catch { /* noop */ }
@@ -536,6 +539,16 @@ channel
     const t = terms.get(payload.term)
     if (t && !t.attached) { try { t.term.resize(Math.max(payload.cols, 80), Math.max(payload.rows, 20)); announce() } catch { /* noop */ } }
   })
+  .on('broadcast', { event: 'session-deleted' }, async () => {
+    // The room was deleted from the dashboard. Close the bridge — and if we're a
+    // background service, uninstall ourselves first so the supervisor doesn't
+    // restart us into a room that no longer exists.
+    process.stderr.write('\n  ◆ this room was deleted — closing the bridge.\n')
+    if (process.env.THINKPOOL_PAIR_AUTOUPDATE === '1') {
+      try { const svc = await import('./service.mjs'); svc.uninstallService(room) } catch { /* noop */ }
+    }
+    shutdown()
+  })
   .on('broadcast', { event: 'term-open' }, ({ payload }) => {
     if (!payload?.id || !payload?.cmd) return
     // Multi-bridge rooms: a targeted open is for ONE machine. Untargeted
@@ -781,7 +794,7 @@ if (process.env.THINKPOOL_PAIR_AUTOUPDATE === '1' && VERSION) {
   }, 60000).unref()
 }
 
-function shutdown() {
+async function shutdown() {
   if (shuttingDown) return
   shuttingDown = true
   clearInterval(flushTimer)
@@ -789,11 +802,16 @@ function shutdown() {
     const bye = Buffer.from('\r\n[ shared session ended ]\r\n', 'utf8').toString('base64')
     for (const id of terms.keys()) bcast('pty-out', { term: id, b64: bye })
   } catch { /* noop */ }
-  try { supabase.removeChannel(channel) } catch { /* noop */ }
+  // Leave presence EXPLICITLY before exiting so the web room flips to dormant
+  // immediately. Previously we removeChannel()'d and process.exit()'d on the next
+  // line — the leave frame never flushed, so the web only noticed via the realtime
+  // heartbeat timeout (tens of seconds later) and looked stuck "live".
+  try { await channel.untrack() } catch { /* noop */ }
+  try { await supabase.removeChannel(channel) } catch { /* noop */ }
   for (const t of terms.values()) { try { t.term.kill() } catch { /* noop */ } }
   for (const s of sessions.values()) { try { s.session.end() } catch { /* noop */ } }
   detachLocal()
-  process.exit(0)
+  setTimeout(() => process.exit(0), 250)   // small grace for the leave/close frames to flush over the socket
 }
 process.on('SIGINT', shutdown)
 process.on('SIGTERM', shutdown)

package/package.json

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