thinkpool-pair 0.7.151 → 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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "thinkpool-pair",
-  "version": "0.7.151",
+  "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']