thinkpool-pair 0.7.226 → 0.7.228

package/codex-event-mapper.mjs

@@ -37,6 +37,14 @@ const resultText = (value) => {
   if (value == null) return ''
   try { return JSON.stringify(value) } catch { return String(value) }
 }
+// Codex uses both `message: string` and `message: { message: string }` across
+// its exec/app-server error paths. Room events must keep `message` scalar: an
+// object persists successfully but crashes React when that transcript replays.
+const errorText = (value, fallback) => {
+  if (typeof value === 'string' && value.trim()) return value
+  if (value && typeof value === 'object' && typeof value.message === 'string' && value.message.trim()) return value.message
+  return resultText(value) || fallback
+}
 
 export class CodexEventMapper {
   /**
@@ -187,13 +195,13 @@ export class CodexEventMapper {
       }
 
       case 'turn.failed': {
-        this._emit({ kind: 'error', message: ev.message || ev.error || 'codex turn failed', recoverable: true })
+        this._emit({ kind: 'error', message: errorText(ev.message ?? ev.error, 'codex turn failed'), recoverable: true })
         this._emit({ kind: 'result', subtype: 'error_during_execution', sessionId: this._sessionId, model: this._model, costUsd: null, usage: null, numTurns: 1, durationMs: undefined, denials: 0, resultText: null })
         return
       }
 
       case 'error':
-        this._emit({ kind: 'error', message: ev.message || ev.error || 'codex error', recoverable: true })
+        this._emit({ kind: 'error', message: errorText(ev.message ?? ev.error, 'codex error'), recoverable: true })
         return
 
       default:

package/package.json

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

package/viewport.mjs

@@ -350,9 +350,13 @@ export class ViewportManager {
       await fsp.writeFile(file, result.png)
       captures[name] = { ...result, file }
     }
-    const html = this.root && fs.existsSync(path.join(this.root, 'index.html')) ? path.join(this.root, 'index.html') : null
+    // A preview root may be a multi-file build. Its index.html is not a portable
+    // artifact: root-relative chunks cannot reliably boot when that text is later
+    // injected into a sandboxed srcDoc (Windows Edge exposed the permanent loader).
+    // preview_capture is therefore screenshot-only evidence. Editable mockups use
+    // render.sh's explicit trusted `source` contract instead.
     const manifest = {
-      slug, title: String(title || 'Viewport capture').slice(0, 120), html,
+      slug, title: String(title || 'Viewport capture').slice(0, 120),
       desktop: captures.desktop?.file || '', mobile: captures.mobile?.file || '', ts: Date.now(),
     }
     const manifestPath = path.join(this.outbox, `${slug}.json`)