thinkpool-pair 0.7.33 → 0.7.35

package/bridge.mjs

@@ -650,7 +650,13 @@ function openStructured({ id, model, resume, log, commands, mode }) {
   if (entry.log.length) process.stderr.write(`\n  ◆ restored ${entry.log.length} prior events (${id.slice(0, 8)})${resume ? ' + resuming live context' : ''}.\n`)
   // Persist the permission mode alongside the transcript so a bridge restart
   // restores the session in the SAME mode (a bypass room stays bypass on resume).
-  const persist = () => saveSession(room, id, { sessionId: entry.session?.sessionId || resume || null, log: entry.log, commands: entry.commands, mode: entry.mode })
+  const sessionData = () => ({ sessionId: entry.session?.sessionId || resume || null, log: entry.log, commands: entry.commands, mode: entry.mode })
+  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
+  // 1.5s saveSession debounce window) and on shutdown (so events since the last
+  // debounced write aren't lost). Contract #2: restart resumes, no lost messages.
+  entry.flush = () => flushSession(room, id, sessionData())
   entry.session = startClaudeSession({
     cwd: process.cwd(), model, resume, mode,
     env: { ...process.env, TP_MOCKUP_OUTBOX: mockupOutbox },
@@ -708,6 +714,10 @@ function openStructured({ id, model, resume, log, commands, mode }) {
           : `\n  ${A.yel}● permission: ${req.toolName}${argStr(req.input)} — approve in the room.${A.rst}\n`)
     }),
   })
+  // Brand-new session (no restored log): write its record now so a restart that lands
+  // before the first debounced save still restores this id instead of dropping it (the
+  // room would otherwise show a fresh empty terminal + strand the transcript).
+  if (!entry.log.length) entry.flush()
   announce()
   if (!entry.log.length) process.stderr.write(`\n  ◆ structured Claude session (${id.slice(0, 8)}) — driven from the room.\n`)
   return entry
@@ -1002,7 +1012,11 @@ channel
         // only runs on a fresh start (guard above), so a network re-subscribe
         // with live sessions won't re-spawn them.
         if (wantStructured(startCmd)) {
-          const all = loadAll(room).filter((r) => r.log?.length || r.sessionId)
+          // Restore EVERY saved rec — a freshly opened session (persisted on open) may
+          // have an empty log + no sessionId yet and must still come back under its id.
+          // 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)
           if (all.length) for (const rec of all) openStructured({ id: rec.id, resume: canResume(rec) ? rec.sessionId : undefined, log: rec.log, commands: rec.commands, mode: rec.mode })
           // Fresh open: ONLY for an explicit `-- <claude>` share. In account mode
           // (autoAgent set, no attachedCmd) the web's `?new=1` flow opens the first
@@ -1171,6 +1185,10 @@ async function shutdown(code = 0, farewell = true) {
   // correct and must not hang on the (possibly dead) realtime socket. This is the
   // bug behind the watchdog/auto-update raw process.exit() paths, which skipped
   // teardown entirely and orphaned agent processes + left stale presence.
+  // Flush structured session state synchronously BEFORE ending sessions. saveSession
+  // debounces ~1.5s, so without this any events since the last write are lost on exit
+  // (Contract #2). writeFileSync, well inside the 1500ms hard-exit backstop above.
+  for (const s of sessions.values()) { try { s.flush?.() } 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()

package/package.json

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