thinkpool-pair 0.7.128 → 0.7.130

package/bridge.mjs

@@ -83,7 +83,7 @@ const flowRedispatch = new Map()
 const flowBudgets = new Map()
 import { formatPeek, PEEK, siblingsOf, resolveSibling, crossPostDecision, CROSSPOST, spawnDecision, CROSSROOM, formatPairRoster, crossRoomPostDecision } from './cross-terminal.mjs'
 import { turnInFlight } from './update-gate.mjs'
-import { saveSession, flushSession, deleteSession, loadAll, canResume, loadPtyId, savePtyId, loadNames, saveNames, appendDurableEvents, seedDurableEvents, readDurablePage, readDurableOldestSeq } from './session-store.mjs'
+import { saveSession, flushSession, deleteSession, loadAll, canResume, loadPtyId, savePtyId, loadNames, saveNames, appendDurableEvents, seedDurableEvents, readDurablePage, readDurableOldestSeq, loadSummary, saveSummary } from './session-store.mjs'
 import { stampEvent, makeSeqCounter, maxSeq, seqable, capReplayEvents, chunkReplayEvents, boundEventForBroadcast, inlineImageBlocks } from './event-id.mjs'
 import { makeThrottledTrack } from './presence.mjs'
 
@@ -1222,12 +1222,16 @@ function restoredTurnOpen(log) {
 // relay STRUCTURED events. onEvent → broadcast `code-event` + print locally +
 // persist to the host file; tool calls round-trip through the perm card; the
 // rolling log replays to joiners and survives bridge restarts (session-store).
-function openStructured({ id, model, resume, log, commands, mode, spawnedBy, rolePrompt, flowSessionId, flowTaskKey, cwd, reviewSliceRoots }) {
+function openStructured({ id, model, resume, log, commands, mode, spawnedBy, rolePrompt, flowSessionId, flowTaskKey, cwd, reviewSliceRoots, openedAt }) {
   if (sessions.has(id)) return
   // spawnedBy: set when this lane was Dispatched (spawn_terminal). Restored from the
   // session store so the Ensemble flag survives a bridge restart (else a respin
   // stripped it and the lane reverted to a plain tab — the t6 "no chip" bug).
-  const entry = { cmd: 'claude', kind: 'structured', log: Array.isArray(log) ? log.slice(-STRUCTURED_LOG_MAX) : [], pending: new Map(), session: null, recovered: false, commands: Array.isArray(commands) ? commands : undefined, mode, model: model || null, spawnedBy: spawnedBy || undefined, flowSessionId: flowSessionId || null, flowTaskKey: flowTaskKey || null, cwd: cwd || null }
+  const entry = { cmd: 'claude', kind: 'structured', log: Array.isArray(log) ? log.slice(-STRUCTURED_LOG_MAX) : [], pending: new Map(), session: null, recovered: false, commands: Array.isArray(commands) ? commands : undefined, mode, model: model || null, spawnedBy: spawnedBy || undefined, flowSessionId: flowSessionId || null, flowTaskKey: flowTaskKey || null, cwd: cwd || null,
+    // Stable creation order — persisted so a bridge restart restores tabs in the SAME
+    // order (not readdir/filesystem order). Legacy recs (no openedAt) derive it from the
+    // first transcript event ts, so even the first post-fix restart is ordered right.
+    openedAt: openedAt || (Array.isArray(log) ? (log.find((e) => e?.ts)?.ts || 0) : 0) || Date.now() }
   // C1 (RT-2): per-term contiguous seq counter. Seeded from the restored log's max
   // so a bridge restart resumes ABOVE every persisted seq — fresh events never reuse
   // an old seq, which is what keeps the between-turns restart dup-free (see
@@ -1340,7 +1344,7 @@ function openStructured({ id, model, resume, log, commands, mode, spawnedBy, rol
   // subagent-block; without cwd it runs outside its worktree; without rolePrompt it loses its
   // Flow role. blockSubagents / onLaneDone / onReviewVerdict / onSubmitPlan all derive from
   // these, so restoring them restores the whole behavior.
-  const sessionData = () => ({ sessionId: entry.session?.sessionId || resume || null, log: entry.log, commands: entry.commands, mode: entry.mode, spawnedBy: entry.spawnedBy, flowSessionId: entry.flowSessionId, flowTaskKey: entry.flowTaskKey, cwd: entry.cwd, rolePrompt: entry.rolePrompt, flowReviewTarget: entry.flowReviewTarget || null, revertTarget: entry.revertTarget || null, reviewSliceRoots: entry.reviewSliceRoots || [] })
+  const sessionData = () => ({ sessionId: entry.session?.sessionId || resume || null, log: entry.log, commands: entry.commands, mode: entry.mode, spawnedBy: entry.spawnedBy, flowSessionId: entry.flowSessionId, flowTaskKey: entry.flowTaskKey, cwd: entry.cwd, rolePrompt: entry.rolePrompt, flowReviewTarget: entry.flowReviewTarget || null, revertTarget: entry.revertTarget || null, reviewSliceRoots: entry.reviewSliceRoots || [], openedAt: entry.openedAt || null })
   const persist = () => saveSession(room, id, sessionData())
   // Synchronous flush of this session's record. Used on open (so a brand-new session
   // has a file under its id BEFORE its first event — surviving a restart inside the
@@ -2213,6 +2217,13 @@ channel
   // re-open within a page load).
   .on('broadcast', { event: 'code-summarize' }, async ({ payload }) => {
     const by = payload?.by || null
+    // Peek — any device opening the panel fetches the cached recap (cross-device +
+    // cross-restart persistence) WITHOUT spending a model call. No cache → silent.
+    if (payload?.peek) {
+      const cached = loadSummary(room)
+      if (cached && cached.text) bcast('code-summary', { status: 'done', ...cached })
+      return
+    }
     bcast('code-summary', { status: 'generating', by })
     try {
       // Pull assistant/user text out of a transcript-event array, keep the tail.
@@ -2253,18 +2264,23 @@ channel
         ? `\n\nThere are also ${closedIds.length} CLOSED terminal(s). Recent excerpts (do NOT name them individually):\n${(closedDigests.join('\n---\n') || '(no retained detail)').slice(0, 1400)}`
         : ''
 
-      const prompt = `Write a concise recap of this multi-terminal coding session for someone glancing at it. Rules:
-- NAME each LIVE terminal and say whether it is working or done, and briefly what it is doing/did.
-- Do NOT name the CLOSED terminals individually — fold them into ONE aggregate clause ("plus N closed lanes that handled …").
-- 2 to 4 sentences total, plain prose, no markdown, no headers, no bullet lists.
+      const prompt = `Summarize this multi-terminal coding session as a STRUCTURED LIST. Output ONLY these lines, nothing else:
+- ONE line per LIVE terminal, EXACTLY: <terminal name> | <working or done> | <one clause, max 12 words, no trailing period>
+- If there are CLOSED terminals, ONE final line, EXACTLY: CLOSED | <count> | <one aggregate clause about what they handled — do NOT name them>
+Use the pipe "|" as the separator. No headers, no markdown, no bullets, no extra prose. Example:
+Bridge Whip | working | SDK 0.7.128, detection gates live
+Reddit | done | GTM strategy mapped
+CLOSED | 52 | flow-trigger churn and @pool integration
 
 ${liveBlock}${closedBlock}
 
-Recap:`
+Output:`
 
       const res = await oneShotSummary({ prompt, env: env || process.env, cwd: cwd || process.cwd() })
       if (!res || !res.text) { bcast('code-summary', { status: 'error', error: 'Summary model unavailable on this provider.', by }); return }
-      bcast('code-summary', { status: 'done', text: res.text, model: res.model, generatedAt: Date.now(), by })
+      const summary = { text: res.text, model: res.model, generatedAt: Date.now() }
+      try { saveSummary(room, summary) } catch { /* cache is best-effort */ }
+      bcast('code-summary', { status: 'done', ...summary, by })
       process.stderr.write(`\n  ◆ session summary generated (${res.model}) for ${by || 'someone'}.\n`)
     } catch (e) {
       bcast('code-summary', { status: 'error', error: String(e?.message || e).slice(0, 140), by })
@@ -2290,11 +2306,16 @@ Recap:`
           // Closed sessions are unlinked by deleteSession, so a file existing on disk ==
           // a session that was live at shutdown (never an explicitly closed one).
           const all = loadAll(room)
+          // Restore in stable creation order (openedAt) so the tab strip comes back in the
+          // SAME order across a bridge restart — NOT readdir/filesystem order, which reshuffled
+          // the tabs (Max, 2026-07-02). Legacy recs fall back to their first event ts / savedAt.
+          const okey = (r) => r.openedAt || (Array.isArray(r.log) ? (r.log.find((e) => e?.ts)?.ts || 0) : 0) || r.savedAt || 0
+          all.sort((a, b) => okey(a) - okey(b))
           if (all.length) for (const rec of all) {
             // FL-M6 — restore the flow context (id/role/cwd) so an in-flight flow survives a
             // bridge restart: the conductor keeps its subagent-block + plan interception, and
             // lanes keep their worktree cwd + the ability to mark done.
-            openStructured({ id: rec.id, resume: canResume(rec) ? rec.sessionId : undefined, log: rec.log, commands: rec.commands, mode: rec.mode, spawnedBy: rec.spawnedBy, flowSessionId: rec.flowSessionId, flowTaskKey: rec.flowTaskKey, cwd: rec.cwd, rolePrompt: rec.rolePrompt, reviewSliceRoots: rec.reviewSliceRoots })
+            openStructured({ id: rec.id, resume: canResume(rec) ? rec.sessionId : undefined, log: rec.log, commands: rec.commands, mode: rec.mode, spawnedBy: rec.spawnedBy, flowSessionId: rec.flowSessionId, flowTaskKey: rec.flowTaskKey, cwd: rec.cwd, rolePrompt: rec.rolePrompt, reviewSliceRoots: rec.reviewSliceRoots, openedAt: rec.openedAt })
             const re = sessions.get(rec.id)
             if (re && rec.flowReviewTarget) re.flowReviewTarget = rec.flowReviewTarget
             // S4 — a resumed re-dispatch lane keeps its surviving revert target across a bridge restart.

package/package.json

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

package/session-store.mjs

@@ -191,3 +191,14 @@ export function loadNames(room) {
 export function saveNames(room, names) {
   try { ensureDir(room); fs.writeFileSync(namesFile(room), JSON.stringify(names || {})) } catch { /* noop */ }
 }
+
+// Session recap cache (2026-07-02, Session Info panel). The last on-demand summary,
+// stored as a dotfile (NOT *.json → never in loadAll) so it survives a bridge restart
+// and any device that opens the panel can fetch it (persists across devices). One per room.
+const summaryFile = (room) => path.join(dir(room), '.summary')
+export function loadSummary(room) {
+  try { return JSON.parse(fs.readFileSync(summaryFile(room), 'utf8')) || null } catch { return null }
+}
+export function saveSummary(room, summary) {
+  try { ensureDir(room); fs.writeFileSync(summaryFile(room), JSON.stringify(summary || null)) } catch { /* noop */ }
+}