thinkpool-pair 0.6.9 → 0.6.11

package/README.md

@@ -35,6 +35,9 @@ The bridge self-heals at three levels — all cross-platform (macOS / Linux / Wi
   npx thinkpool-pair install-service <ROOM> -- claude   # set and forget
   npx thinkpool-pair uninstall-service <ROOM>           # remove it
   ```
+  The service runs `npx thinkpool-pair@latest`, so it **auto-updates** — new
+  versions apply on the next restart, no re-install. (Linux: run
+  `loginctl enable-linger $USER` once to keep it running after logout.)
 - **Watchdog** — if the realtime channel wedges for >60s, the bridge exits so the
   supervisor/service restarts a clean process. Brief network blips reconnect on
   their own.

package/bridge.mjs

@@ -49,11 +49,13 @@ const SCROLLBACK_MAX = 120_000
 
 // node-pty is a native module — loaded lazily so a friendly message shows
 // if it isn't built yet.
-let pty
+// Structured Claude (the default) runs through the Agent SDK and needs NO PTY,
+// so a failed node-pty build must not block the bridge — it only disables the
+// raw-PTY path (other CLIs / TP_PTY=1), which we flag clearly if it's used.
+let pty = null
 try { pty = (await import('node-pty')).default ?? (await import('node-pty')) }
 catch {
-  console.error('\n  node-pty isn\'t installed. From the bridge folder run:\n    npm i\n  then re-run.  (it needs a C toolchain — Xcode CLT on mac, build-essential on linux)\n')
-  process.exit(1)
+  console.error('\n  ⚠ node-pty not built — raw terminal sharing is off (structured Claude still works).\n    For PTY mode (bash / aider / codex / TP_PTY=1): install a C toolchain, then `npm i` in the bridge.\n    (Xcode CLT on mac · build-essential on linux · VS Build Tools on Windows.)\n')
 }
 
 // ── Known coding-agent CLIs we feature in the picker ──────────────
@@ -325,6 +327,11 @@ const attachedDims = () => ({
 
 function openTerm({ id, cmd, args = [], attached = false, cols, rows }) {
   if (terms.has(id)) return
+  if (!pty) {   // raw-PTY path needs node-pty; structured Claude never lands here
+    process.stderr.write(`\n  ⚠ can't open "${cmd}" — node-pty isn't built (PTY mode disabled).\n`)
+    bcast('term-exit', { id })
+    return
+  }
   const ad = attached ? attachedDims() : null
   const term = pty.spawn(cmd, args, {
     name: 'xterm-256color',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "thinkpool-pair",
-  "version": "0.6.9",
+  "version": "0.6.11",
   "description": "Share a local coding-agent CLI (Claude Code, Codex, Gemini, Aider, …) into a ThinkPool Code room, live.",
   "type": "module",
   "bin": {
@@ -21,7 +21,9 @@
   },
   "dependencies": {
     "@anthropic-ai/claude-agent-sdk": "^0.3.173",
-    "@supabase/supabase-js": "^2.45.0",
+    "@supabase/supabase-js": "^2.45.0"
+  },
+  "optionalDependencies": {
     "node-pty": "^1.2.0-beta.13"
   }
 }

package/service.mjs

@@ -12,24 +12,34 @@
 import os from 'node:os'
 import fs from 'node:fs'
 import path from 'node:path'
-import { fileURLToPath } from 'node:url'
 import { execSync } from 'node:child_process'
 
-const BRIDGE = fileURLToPath(new URL('./bridge.mjs', import.meta.url))
 const label = (room) => `io.thinkpool.pair.${room.toLowerCase()}`
 const xml = (s) => String(s).replace(/&/g, '&amp;').replace(/</g, '&lt;').replace(/>/g, '&gt;')
 
+// Resolve the real npx next to the current node, so the service stays seamless:
+// it runs `npx -y thinkpool-pair@latest …`, always picking up the newest publish
+// with no re-install. Absolute path + injected PATH so it works under launchd /
+// systemd's bare environment (which don't search PATH for argv[0]).
+function npxPath(platform) {
+  const bin = path.dirname(process.execPath)
+  const cand = path.join(bin, platform === 'win32' ? 'npx.cmd' : 'npx')
+  try { fs.accessSync(cand); return cand } catch { return platform === 'win32' ? 'npx.cmd' : 'npx' }
+}
+
 // Pure artifact builder — returned shape is testable without side effects.
-// `mode:'service'` lets the OS supervise (no --supervise); 'supervise' wraps it.
 export function buildArtifact(platform, { room, cmdArgs = [] }) {
-  const node = process.execPath
+  const npx = npxPath(platform)
   const cwd = process.cwd()
   const logDir = path.join(os.homedir(), '.thinkpool-pair')
   const log = path.join(logDir, `${room}.log`)
   const tail = cmdArgs.length ? ['--', ...cmdArgs] : []
+  // OS-supervised tiers (launchd KeepAlive / systemd Restart) don't need
+  // --supervise; the latest published bridge is fetched by npx each (re)start.
+  const pkgArgs = ['-y', 'thinkpool-pair@latest', room]
 
   if (platform === 'darwin') {
-    const args = [node, BRIDGE, room, ...tail]   // launchd KeepAlive supervises
+    const args = [npx, ...pkgArgs, ...tail]   // launchd KeepAlive supervises
     const file = path.join(os.homedir(), 'Library', 'LaunchAgents', `${label(room)}.plist`)
     const content = `<?xml version="1.0" encoding="UTF-8"?>
 <!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
@@ -47,7 +57,7 @@ export function buildArtifact(platform, { room, cmdArgs = [] }) {
   }
 
   if (platform === 'linux') {
-    const args = [node, BRIDGE, room, ...tail]   // systemd Restart=always supervises
+    const args = [npx, ...pkgArgs, ...tail]   // systemd Restart=always supervises
     const file = path.join(os.homedir(), '.config', 'systemd', 'user', `${label(room)}.service`)
     const content = `[Unit]
 Description=ThinkPool Code bridge (${room})
@@ -73,7 +83,8 @@ WantedBy=default.target
     // running --supervise gives login-persistence + crash-restart, dependency-free.
     const startup = path.join(process.env.APPDATA || path.join(os.homedir(), 'AppData', 'Roaming'), 'Microsoft', 'Windows', 'Start Menu', 'Programs', 'Startup')
     const file = path.join(startup, `thinkpool-pair-${room}.cmd`)
-    const inner = [`"${node}"`, `"${BRIDGE}"`, room, '--supervise', ...tail].join(' ')
+    // Startup folder isn't a supervisor → keep --supervise for crash-restart.
+    const inner = ['npx', '-y', 'thinkpool-pair@latest', room, '--supervise', ...tail].join(' ')
     const content = `@echo off\r\ntitle thinkpool-pair ${room}\r\n${inner}\r\n`
     return { file, content, logDir, post: [], note: 'Installed to the Startup folder — runs at login with --supervise (auto-restart on crash). Start it now without rebooting by double-clicking the .cmd, or run it from a terminal.' }
   }
@@ -90,7 +101,7 @@ export function installService(room, cmdArgs = []) {
   for (const cmd of a.post) {
     try { execSync(cmd, { stdio: 'inherit' }) } catch (e) { process.stderr.write(`  ⚠ "${cmd}" failed: ${e.message}\n`) }
   }
-  process.stderr.write(`  ◆ ${a.note}\n  ◆ logs: ${path.join(a.logDir, `${room}.log`)}\n  ◆ remove with: thinkpool-pair uninstall-service ${room}\n\n`)
+  process.stderr.write(`  ◆ ${a.note}\n  ◆ tracks thinkpool-pair@latest — new versions apply on next restart, no re-install.\n  ◆ logs: ${path.join(a.logDir, `${room}.log`)}\n  ◆ remove with: thinkpool-pair uninstall-service ${room}\n\n`)
 }
 
 export function uninstallService(room) {