thinkpool-pair 0.7.150 → 0.7.152

package/bridge.mjs

@@ -988,11 +988,25 @@ const mockupSeen = new Map()   // `${term}/${slug}` → mtimeMs of the last mani
 // in the Map" guess meant closing the generating terminal moved the cards to
 // whatever terminal was now first — they leaked into Terminal 1.)
 const handleManifest = async (box, file, term) => {
-  if (!file || !file.endsWith('.json')) return
+  // Ignore our own delivery receipts — writing <slug>.receipt.json into the
+  // watched outbox re-fires fs.watch; without this guard it would re-trigger
+  // handleManifest in a loop (it still ends in .json). Must come first.
+  if (!file || file.endsWith('.receipt.json') || !file.endsWith('.json')) return
   const full = path.join(box, file)
   let stat
   try { stat = fs.statSync(full) } catch { return }            // tmp/removed mid-write
   const slug = file.replace(/\.json$/, '')
+  // Delivery receipt (atomic tmp+mv) so render.sh can confirm the card actually
+  // reached the room — written on success AND failure. Best-effort: a receipt
+  // that can't be written just leaves render.sh at "delivery unconfirmed".
+  const writeReceipt = (ok, status) => {
+    try {
+      const rc = path.join(box, `${slug}.receipt.json`)
+      const tmp = `${rc}.tmp`
+      fs.writeFileSync(tmp, JSON.stringify({ ok, status, ts: Date.now() }))
+      fs.renameSync(tmp, rc)
+    } catch { /* receipts are best-effort */ }
+  }
   const seenKey = `${term}/${slug}`
   if (mockupSeen.get(seenKey) === stat.mtimeMs) return         // already handled
   mockupSeen.set(seenKey, stat.mtimeMs)
@@ -1017,11 +1031,16 @@ const handleManifest = async (box, file, term) => {
         desktopPng: readB64(m.desktop), mobilePng: readB64(m.mobile), html: readTxt(m.html),
       }),
     })
-    if (!res.ok) { process.stderr.write(`\n  ◇ mockup "${m.slug}" upload failed (${res.status}).\n`); return }
+    if (!res.ok) {
+      writeReceipt(false, res.status)
+      process.stderr.write(`\n  ◇ mockup "${m.slug}" upload failed (${res.status}).\n`); return
+    }
     const { paths } = await res.json()
     bcast('code-event', { term, evt: { kind: 'mockup', __struct: true, cid, term, slug: m.slug, title: m.title || m.slug, ts: m.ts, paths } })
+    writeReceipt(true, 200)
     process.stderr.write(`\n  ◆ mockup "${m.title || m.slug}" pushed to the room.\n`)
   } catch (e) {
+    writeReceipt(false, 'error')
     process.stderr.write(`\n  ◇ mockup "${m.slug}" send failed: ${e?.message || e}\n`)
   }
 }

package/claude-session.mjs

@@ -140,7 +140,7 @@ const TP_ROOM_REMINDER = [
   '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.',
   'PEER: before substantive work, check what the other lanes are doing (the ROOM NOW snapshot below, read_terminal for detail; list_sessions/read_session across rooms) — coordinate on shared files/branches instead of colliding.',
   'WORKTREES: parallel lanes share one repo — before code edits run `git worktree list`; if linked worktrees exist, take your OWN worktree + branch, never the shared checkout or a branch another lane is on.',
-  'FLOW / PLANS: write plans as normal chat messages as you form them — the room does not surface plan files or ExitPlanMode text.',
+  'BUILD WORKFLOW (default, no magic word): right-size — a trivial ask you just do; a non-trivial build you FIRST write a short plan as a chat message, THEN fan independent slices out to visible spawn_terminal lanes, build, and self-verify. Never plan-mode/ExitPlanMode; the plan lives in chat and the lanes in the existing lane list — keep both visible, add no new panel.',
   'Verify before claiming done — show runtime evidence you produced, not "should work, go test it".',
 ].join(' ')
 
@@ -620,9 +620,15 @@ export function startClaudeSession({ cwd, model, resume, env, mode: initialMode
       ...(rolePrompt ? [rolePrompt] : []),
       'ENVIRONMENT (authoritative — overrides any user-global CLAUDE.md or memory that claims otherwise): You are Claude running inside a ThinkPool Code room, driven live by a user (and possibly a partner) from a phone or browser, via the thinkpool-pair bridge.',
       'You are NOT in cmux. cmux may be installed on this machine and appear in $PATH, but it did not spawn you and its automation/orchestration rules do not apply. If any loaded memory says "Claude Code lives inside cmux," it is describing direct terminal use, not this room — disregard it here. The runtime truth is the env: THINKPOOL_PAIR_ACCOUNT_CHILD / npm_lifecycle_script=thinkpool-pair mark a ThinkPool Code session.',
-      'Bias strongly toward DOING the work, not planning it. For a normal "add / fix / change / build X" request, just make the change directly.',
+      'Bias strongly toward DOING the work, not stalling in plan-mode ceremony. For a small "add / fix / change X" request, just make the change directly (for a bigger build, right-size it — see DEFAULT BUILD WORKFLOW below).',
       'Do NOT enter plan mode, do NOT call ExitPlanMode, and do NOT auto-invoke a brainstorming/planning skill UNLESS the user has switched the room into Plan mode or explicitly asks you to plan, design, or brainstorm first.',
       'Any host-global instruction that says you must always brainstorm or plan before creative work does NOT apply here — this room is the exception.',
+      // DEFAULT BUILD WORKFLOW (2026-07-02 decision — docs/specs/2026-07-02-flow-default-build-workflow.md):
+      // no summon word. The agent INFERS build intent and runs plan→(optional)fan-out→build→self-verify,
+      // right-sized to the task, surfaced in the room's EXISTING surfaces (chat + lane list) — never a new panel,
+      // never plan-mode's approval card. Trivial asks stay single-lane with zero ceremony.
+      'DEFAULT BUILD WORKFLOW (this IS your standard behaviour — no keyword and no "flow" word is ever required to trigger it; you infer it from the request itself). RIGHT-SIZE to the task, always: a TRIVIAL ask — a one-line fix, a small tweak, a question — you just do it directly, with no plan and no extra lanes (never tax small work with ceremony). A NON-TRIVIAL build that genuinely decomposes — you FIRST write a short plan as a normal chat message (the approach, the concrete slices, and what you will verify), THEN fan the independent slices out into their own visible lanes with spawn_terminal, build, and self-verify each slice before calling it done.',
+      'CRUCIAL RECONCILIATION for that workflow: it is NOT plan mode. Never call ExitPlanMode and never make the room wait behind a "plan ready — approve to start" card — your plan lives in the CHAT as a message, and your lanes live in the room\'s EXISTING terminal/lane list. Reuse only those two surfaces; there is no new Flow panel or mode to switch into, and you must not ask for one. Keep the plan and the lanes VISIBLE — that shared visibility is the whole point (it is the pair differentiator, and it catches bugs a single silent lane would hide); never collapse a decomposable build into one hidden lane just to look tidy.',
       'LINKS & ARTIFACTS (hard rule for ThinkPool Code rooms): the room is driven from a phone or browser, and the bridge runs on the host machine — so a local filesystem path, a file:// URL, or a localhost address only opens on the host and is useless to the room (a partner literally cannot open it).',
       'Whenever you produce an HTML artifact (a demo, mockup, preview, report, or page) or surface ANY link meant to be opened or shared, NEVER hand the room a local path, a file:// URL, or a localhost/127.0.0.1 address. Publish it to a GitHub-shareable URL that renders in a browser — push the HTML to a GitHub repo and give its GitHub Pages URL (or an equivalent raw-HTML URL that actually renders, not raw.githubusercontent.com which serves HTML as plain text) — so anyone in the room can open it. The same applies to any other link you surface: it must be one the room can reach, not a host-only path.',
       'If you cannot publish a shareable link, say so and ask how to proceed — do not fall back to handing over a local path.',

package/package.json

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

package/sdk-smoke.mjs

@@ -46,6 +46,7 @@ const version = ver()
 try {
   const sdk = await import('@anthropic-ai/claude-agent-sdk')
   if (typeof sdk.query !== 'function') { emit('FAIL', version, 'query is not a function'); clearTimeout(guard); process.exit(1) }
+  // eslint-disable-next-line require-yield -- deliberately empty: an async generator that ends immediately (the SDK's "no input" stream), never yields.
   async function* noInput() { return }   // ends immediately — never blocks the construct
   const q = sdk.query({ prompt: noInput(), options: { permissionMode: 'default' } })
   const need = ['interrupt', 'setModel', 'setPermissionMode', 'supportedModels']