thinkpool-pair 0.7.219 → 0.7.221

package/bridge.mjs

@@ -65,7 +65,7 @@ import { writeLaneArtifact, digestSlice, appendDigest, resumeLane } from './flow
 import { createFlowWorktree, worktreeSpec } from './flow-worktree.mjs'
 import { startPreview, stopAllPreviews, previews } from './flow-preview.mjs'
 import { ViewportManager, createViewportTools, sharedViewportBrowser } from './viewport.mjs'
-import { designPrompt, resolveDesignSource, restoreDesignSources, syncRealtimeAuth, validateDesignRequest } from './design-edit.mjs'
+import { designPrompt, resolveDesignSource, resolveManifestDesignSource, restoreDesignSources, syncRealtimeAuth, validateDesignRequest } from './design-edit.mjs'
 // FL-M9 — per-lane preview servers leak (one per done lane, never stopped until shutdown).
 // Lane previews are keyed `lane:<flowId>:<laneId>`; stop a whole flow's set when it assembles
 // (the assembled preview supersedes them) or when a lane is reverted.
@@ -1321,8 +1321,8 @@ const handleManifest = async (box, file, term, trustedDesignSource = false) => {
   try { m = JSON.parse(fs.readFileSync(full, 'utf8')) } catch { return }
   if (!m?.slug) return
   const producer = trustedDesignSource ? sessions.get(term) : null
-  const designRecord = producer && m.source
-    ? resolveDesignSource(m.source, producer.cwd || process.cwd())
+  const designRecord = producer
+    ? resolveManifestDesignSource(m, producer.cwd || process.cwd())
     : null
   // 2026-07-07: these used to swallow read errors silently — a transient
   // unreadable file (race with the render script, permissions, mid-write)

package/claude-session.mjs

@@ -21,7 +21,7 @@ import { sanitizeSession } from './transcript-sanitize.mjs'
 import { reviewGatePreToolDecision } from './flow-review-gate.mjs'
 import { crossPostNeedsCard } from './cross-terminal.mjs'
 import { correctContext } from './context-windows.mjs'
-import { THINKPOOL_REMOTE_DELIVERY_RULES } from './thinkpool-room-prompt.mjs'
+import { THINKPOOL_DESIGN_DELIVERY_RULE, THINKPOOL_REMOTE_DELIVERY_RULES } from './thinkpool-room-prompt.mjs'
 import { stallDecision, stallEvent, isCompactTurn } from './turn-stall.mjs'
 
 // The caret-pulled SDK's real version (^0.3.x auto-upgrades on restart). Resolved
@@ -196,7 +196,8 @@ const MODES = new Set(['default', 'acceptEdits', 'plan', 'bypassPermissions'])
 // the bridge's roomContext callback — see roomReminder() inside startClaudeSession.
 const TP_ROOM_REMINDER = [
   'You are Claude in a ThinkPool Code room, driven live from a phone or browser — NOT a local terminal. Keep using the room\'s features.',
-  'ARTIFACTS: never hand the room a local path, a file:// URL, a localhost address, an `open …`/Preview reference, or "it\'s in ~/claude-shots/" — the room cannot see the host. Surface every image by reading the PNG into your turn so it renders inline; for HTML — mockups and mockup-iterate output included — use the $TP_MOCKUP_OUTBOX render helper (inline desktop+mobile card) and/or publish to a GitHub Pages URL and post that link.',
+  'ARTIFACTS: never hand the room a local path, a file:// URL, a localhost address, an `open …`/Preview reference, or "it\'s in ~/claude-shots/" — the room cannot see the host. Surface every image by reading the PNG into your turn so it renders inline.',
+  THINKPOOL_DESIGN_DELIVERY_RULE,
   'REMOTE USER: the people here may have no terminal and no access to this host — never ask them to run a local command, open a local file, or check something on the machine. Anything host-side, YOU run, and show the result.',
   'SHOW, don\'t narrate: a picture of the result beats a wall of text on a phone — err toward more screenshots.',
   'ENSEMBLE: delegated work goes through the room — fan parallel work out with spawn_terminal (your own fresh lanes, visible and steerable by the people here), NOT the built-in invisible Task/Agent subagents; never hijack a busy sibling.',

package/design-edit.mjs

@@ -48,6 +48,15 @@ export function resolveDesignSource(file, workspaceRoot) {
   return { previewId, revision, sourcePath, workspaceRoot: root, source }
 }
 
+// Older lane worktrees predate the explicit `source` manifest field and emit the
+// authored mockup path as `html` only. Preserve the explicit field when present;
+// otherwise let the same containment/generated-output checks decide whether the
+// HTML path is canonical editable source or view-only render evidence.
+export function resolveManifestDesignSource(manifest, workspaceRoot) {
+  if (!manifest || typeof manifest !== 'object') return null
+  return resolveDesignSource(manifest.source || manifest.html, workspaceRoot)
+}
+
 // Design provenance is intentionally host-only, but it must survive a bridge
 // restart. Rebuild the in-memory registry from manifests already written by the
 // trusted lane render workflow; do not re-upload or re-broadcast old cards.
@@ -68,8 +77,7 @@ export function restoreDesignSources(box, workspaceRoot, term, limit = 100) {
   for (const { file } of files) {
     try {
       const manifest = JSON.parse(fs.readFileSync(file, 'utf8'))
-      if (!manifest?.source) continue
-      const record = resolveDesignSource(manifest.source, workspaceRoot)
+      const record = resolveManifestDesignSource(manifest, workspaceRoot)
       if (!record) continue
       Object.assign(record, {
         term,

package/package.json

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

package/thinkpool-room-prompt.mjs

@@ -3,13 +3,15 @@
 // the first human turn in each fresh native thread. Keep the runtime-specific
 // orchestration rules in their own drivers — these are the cross-runtime truths
 // about where the people are and how finished work must reach them.
+export const THINKPOOL_DESIGN_DELIVERY_RULE = 'THINKPOOL DESIGN (mandatory for HTML): save the editable HTML source in the workspace, then use the $TP_MOCKUP_OUTBOX render helper so the room receives a Thinkpool Design card with both desktop (1440×900) and mobile (390×844) previews. Never paste raw HTML into chat, send an .html file as the room deliverable, or substitute a bare URL or single screenshot for this card. A shareable browser URL may accompany the card, but never replaces it.'
+
 export const THINKPOOL_REMOTE_DELIVERY_RULES = Object.freeze([
   'REMOTE USER (authoritative default for ThinkPool Code rooms): unless a person explicitly says they are at the host machine, assume everyone driving this room is remote — possibly on a phone — with no terminal and no access to the host filesystem. Never ask them to run a local command, open a local file, or "go check" something on the machine. Anything host-side, you run yourself and show the result in the room. Only suggest actions they can actually do from the room UI or a browser.',
   'LINKS & ARTIFACTS: a local filesystem path, file:// URL, localhost/127.0.0.1 address, or host-only preview is useless to a remote room. Every link you surface must be reachable by the people in the room.',
-  'Whenever you produce HTML or shareable markup — a demo, mockup, preview, report, or page — publish it to a browser-renderable GitHub-shareable URL, normally GitHub Pages (or an equivalent URL that actually renders; raw.githubusercontent.com serves HTML as plain text). Give the room that shareable URL, never only a local path. If you cannot publish it, say so and ask how to proceed instead of falling back to a host-only path.',
+  'Whenever you produce HTML or shareable markup — a demo, mockup, preview, report, or page — publish it to a browser-renderable GitHub-shareable URL, normally GitHub Pages (or an equivalent URL that actually renders; raw.githubusercontent.com serves HTML as plain text). Give the room that shareable URL in addition to the Thinkpool Design card, never instead of it and never only as a local path. If you cannot publish it, say so instead of falling back to a host-only path.',
   'SHOW VISUAL WORK: whenever you build, change, or fix anything visual — UI, page, HTML artifact, chart, diagram, or rendered result — capture a PNG and surface the image in your turn so it appears inline in the room; the room auto-uploads surfaced images. Prefer showing the result over describing it.',
   'BRIDGE PREVIEWS: for a built web UI, use preview_start (default root: dist), then preview_capture for exact desktop 1440x900 and mobile 390x844 PNGs, and preview_inspect when DOM text or selector geometry helps. Run the project build first and stop the preview server with preview_stop when done.',
-  'HTML PREVIEWS: when a mockup render helper is available through $TP_MOCKUP_OUTBOX, use it to surface the inline desktop-and-mobile preview card. Also publish the artifact and post its shareable browser URL. Never leave HTML viewable only on the host.',
+  THINKPOOL_DESIGN_DELIVERY_RULE,
   'VERIFY BEFORE CLAIMING: run or serve what you changed, observe it, and show the evidence in the room — the PNG, passing test output, or real response. If something could not be verified, say exactly what remains unverified.',
 ])