thinkpool-pair 0.7.359 → 0.7.360

package/bridge.mjs

@@ -2850,7 +2850,7 @@ function openStructured({ id, runtime = 'claude', model, models, effort, resume,
   // Visual QA runs in the bridge process, outside either agent runtime's sandbox.
   // It remains scoped to this lane's cwd and private mockup outbox.
   entry.viewport = new ViewportManager({
-    workspaceRoot: cwd || process.cwd(), ownerId: id, outbox: mockupOutbox,
+    workspaceRoot: cwd || process.cwd(), ownerId: id, roomCode: room, outbox: mockupOutbox,
     authContext: () => ({ accessToken: codeAuthToken, supabaseUrl: SUPABASE_URL }),
     designContext: () => {
       const active = designActive.get(id)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "thinkpool-pair",
-  "version": "0.7.359",
+  "version": "0.7.360",
   "description": "Connect Claude Code, Codex, or Hermes on your computer to a Thinkpool Code room.",
   "type": "module",
   "bin": {

package/thinkpool-capabilities.json

@@ -1,6 +1,6 @@
 {
   "schemaVersion": 1,
-  "bundleVersion": 19,
+  "bundleVersion": 20,
   "contracts": [
     {
       "id": "room-coordination",
@@ -88,13 +88,13 @@
     },
     {
       "id": "visual-proof",
-      "version": 5,
+      "version": 6,
       "routes": [
         {
           "id": "visual-proof",
           "tools": ["preview_start", "preview_capture", "preview_inspect", "preview_stop"],
           "trigger": "\\b(ui|ux|visual|design|frontend|html|css|page|route|mockup|screenshot|responsive|desktop|mobile|preview)\\b",
-          "prompt": "For built visual work, run the build, use preview_start, preview_capture at desktop and mobile, preview_inspect when rendered state matters, and preview_stop when finished. For an intentional application-preview Design card, use the project's source-aware Design build command when one exists (for example npm run build:design); otherwise build normally and keep selector-based source matching labeled as fallback. preview_capture uses a fresh isolated browser and is inline verification evidence by default; it must not create a transcript card. Set card=true only when the person asked for a mockup/Design artifact or the visual itself is an intentional deliverable that should remain editable in the transcript. A card requires settled meaningful content at both desktop and mobile, rejects loading/public-auth fallback shells, and is delivered after the final agent response. Room and protected routes automatically use the bridge account as a read-only authenticated visual harness; external REST writes, presence, ordinary room broadcasts, and refresh-token access are blocked, while non-mutating roster/transcript snapshot requests are allowed. Surface PNG evidence only when no interactive source-backed Design card is displayed."
+          "prompt": "For built visual work, run the build, use preview_start, preview_capture at desktop and mobile, preview_inspect when rendered state matters, and preview_stop when finished. Omit the preview path to render the current authenticated room; pass / explicitly only when the public landing page is intended. For an intentional application-preview Design card, use the project's source-aware Design build command when one exists (for example npm run build:design); otherwise build normally and keep selector-based source matching labeled as fallback. preview_capture uses a fresh isolated browser and is inline verification evidence by default; it must not create a transcript card. Set card=true only when the person asked for a mockup/Design artifact or the visual itself is an intentional deliverable that should remain editable in the transcript. A card requires settled meaningful content at both desktop and mobile, rejects loading/public-auth fallback shells, and is delivered after the final agent response. Room and protected routes automatically use the bridge account as a read-only authenticated visual harness; external REST writes, presence, ordinary room broadcasts, and refresh-token access are blocked, while non-mutating roster/transcript snapshot requests are allowed. Surface PNG evidence only when no interactive source-backed Design card is displayed."
         }
       ],
       "impact": [

package/viewport.mjs

@@ -383,6 +383,8 @@ export class CdpBrowser {
             expression: `(() => {
               if (document.querySelector('#invitation-room-title')) return true;
               if (document.querySelector('#boot, .boot-loader')) return false;
+              const room = document.querySelector('[data-tp-preview-room-status]');
+              if (room?.dataset.tpPreviewRoomStatus === 'pending') return false;
               const loading = [...document.querySelectorAll('span')].some((node) => /^(loading session|restoring terminals…|connecting(?:…|\.\.\.))$/i.test((node.textContent || '').trim()));
               return !loading && (document.body?.innerText || '').trim().length > 1;
             })()`,
@@ -405,6 +407,7 @@ export class CdpBrowser {
                 loadingSession: [...document.querySelectorAll('span')].some((node) => (node.textContent || '').trim() === 'Loading session'),
                 connecting: [...document.querySelectorAll('span')].some((node) => (node.textContent || '').trim().toLowerCase() === 'connecting…'),
                 invitation: !!document.querySelector('#invitation-room-title'),
+                roomStatus: document.querySelector('[data-tp-preview-room-status]')?.dataset.tpPreviewRoomStatus || null,
                 heldLocks: locks.held?.length || 0,
                 pendingLocks: locks.pending?.length || 0,
               };
@@ -413,7 +416,7 @@ export class CdpBrowser {
             returnByValue: true,
           }, sessionId).catch(() => ({ result: { value: {} } }))
           const d = diagnostic.result?.value || {}
-          throw new Error(`Authenticated preview did not leave its loading state within 15 seconds (session=${d.sessionPresent ? 'present' : 'missing'}, inlineBoot=${d.inlineBoot ? 'yes' : 'no'}, loadingSession=${d.loadingSession ? 'yes' : 'no'}, connecting=${d.connecting ? 'yes' : 'no'}, authLocks=${d.heldLocks || 0}/${d.pendingLocks || 0}, invitation=${d.invitation ? 'yes' : 'no'}, reads=${authResponses.join(',') || 'none'}, local=${localResponses.join(',') || 'none'}, runtime=${runtimeExceptions.join(' | ') || 'none'}); no screenshot was created.`)
+          throw new Error(`Authenticated preview did not leave its loading state within 15 seconds (session=${d.sessionPresent ? 'present' : 'missing'}, inlineBoot=${d.inlineBoot ? 'yes' : 'no'}, loadingSession=${d.loadingSession ? 'yes' : 'no'}, connecting=${d.connecting ? 'yes' : 'no'}, roomStatus=${d.roomStatus || 'none'}, authLocks=${d.heldLocks || 0}/${d.pendingLocks || 0}, invitation=${d.invitation ? 'yes' : 'no'}, reads=${authResponses.join(',') || 'none'}, local=${localResponses.join(',') || 'none'}, runtime=${runtimeExceptions.join(' | ') || 'none'}); no screenshot was created.`)
         }
       }
       const settle = auth
@@ -585,9 +588,10 @@ export class CdpBrowser {
 export const sharedViewportBrowser = new CdpBrowser()
 
 export class ViewportManager {
-  constructor({ workspaceRoot, ownerId, outbox, browser = sharedViewportBrowser, startPreviewImpl = startPreview, designContext = null, authContext = null } = {}) {
+  constructor({ workspaceRoot, ownerId, roomCode, outbox, browser = sharedViewportBrowser, startPreviewImpl = startPreview, designContext = null, authContext = null } = {}) {
     this.workspaceRoot = path.resolve(workspaceRoot || process.cwd())
     this.ownerId = ownerId || randomUUID()
+    this.roomCode = typeof roomCode === 'string' && roomCode.trim() ? roomCode.trim() : null
     this.outbox = outbox || path.join(os.tmpdir(), 'thinkpool-viewport-captures', this.ownerId)
     this.browser = browser
     this.startPreviewImpl = startPreviewImpl
@@ -623,6 +627,14 @@ export class ViewportManager {
     return this.preview
   }
 
+  defaultRoute() {
+    return this.roomCode ? `/?r=${encodeURIComponent(this.roomCode)}` : '/'
+  }
+
+  resolveRoute(route) {
+    return normalizeRoute(route ?? this.defaultRoute())
+  }
+
   pageUrl(route) {
     const preview = this.requirePreview()
     return new URL(normalizeRoute(route), preview.url).href
@@ -642,8 +654,8 @@ export class ViewportManager {
     }
   }
 
-  async capture({ route = '/', viewports = 'both', title = 'Viewport capture', fullPage = true, waitMs = 300, card = false } = {}) {
-    const normalizedRoute = normalizeRoute(route)
+  async capture({ route, viewports = 'both', title = 'Viewport capture', fullPage = true, waitMs = 300, card = false } = {}) {
+    const normalizedRoute = this.resolveRoute(route)
     const url = this.pageUrl(normalizedRoute)
     const names = viewports === 'both' ? ['desktop', 'mobile'] : [viewports]
     if (names.some((name) => !DEFAULT_VIEWPORTS[name])) throw new Error('viewports must be "both", "desktop", or "mobile".')
@@ -704,12 +716,12 @@ export class ViewportManager {
       await fsp.writeFile(tmp, JSON.stringify(manifest))
       await fsp.rename(tmp, manifestPath)
     }
-    return { url, slug, manifestPath, captures, snapshotPath, cardReady }
+    return { url, route: normalizedRoute, slug, manifestPath, captures, snapshotPath, cardReady }
   }
 
-  async inspect({ route = '/', viewport = 'mobile', selector, waitMs = 300 } = {}) {
+  async inspect({ route, viewport = 'mobile', selector, waitMs = 300 } = {}) {
     if (!DEFAULT_VIEWPORTS[viewport]) throw new Error('viewport must be "desktop" or "mobile".')
-    const normalizedRoute = normalizeRoute(route)
+    const normalizedRoute = this.resolveRoute(route)
     return this.browser.inspect({
       url: this.pageUrl(normalizedRoute), viewport: DEFAULT_VIEWPORTS[viewport], selector, waitMs,
       auth: await this.previewAuth(normalizedRoute),
@@ -742,9 +754,9 @@ export function createViewportTools({ tool, z, manager }) {
     ),
     tool(
       'preview_capture',
-      'Capture this lane preview in a fresh isolated browser at exact desktop (1440×900) and mobile (390×844) CSS viewports. Defaults to both and full-page. Captures are inline verification evidence by default. Room and protected routes automatically use the bridge account as a read-only authenticated visual harness; external REST writes, presence, and room broadcasts are blocked, apart from non-mutating roster/transcript snapshot requests. Set card=true only for a user-requested mockup or visual deliverable that should remain editable in the transcript; complete desktop+mobile settled content is then required and the card arrives after the final response.',
+      'Capture this lane preview in a fresh isolated browser at exact desktop (1440×900) and mobile (390×844) CSS viewports. Omit path to preview the current authenticated room; pass / explicitly for the public landing page. Defaults to both and full-page. Captures are inline verification evidence by default. Room and protected routes automatically use the bridge account as a read-only authenticated visual harness; external REST writes, presence, and room broadcasts are blocked, apart from non-mutating roster/transcript snapshot requests. Set card=true only for a user-requested mockup or visual deliverable that should remain editable in the transcript; complete desktop+mobile settled content is then required and the card arrives after the final response.',
       {
-        path: z.string().max(500).optional().describe('route within the preview, e.g. / or /settings; never a full URL'),
+        path: z.string().max(500).optional().describe('route within the preview; omit for the current authenticated room, or pass / for the public landing page; never a full URL'),
         viewports: z.enum(['both', 'desktop', 'mobile']).optional(),
         title: z.string().max(120).optional(),
         fullPage: z.boolean().optional(),
@@ -754,13 +766,13 @@ export function createViewportTools({ tool, z, manager }) {
       async (args) => {
         try {
           const result = await manager.capture({
-            route: args?.path || '/', viewports: args?.viewports || 'both', title: args?.title || 'Viewport capture',
+            route: args?.path, viewports: args?.viewports || 'both', title: args?.title || 'Viewport capture',
             fullPage: args?.fullPage !== false, waitMs: args?.waitMs ?? 300, card: args?.card === true,
           })
           const delivery = result.cardReady
             ? 'The complete room preview card is queued after your final response.'
             : 'This is verification evidence only; no room card was created.'
-          const content = [{ type: 'text', text: `Captured ${Object.keys(result.captures).join(' + ')} for ${args?.path || '/'}. ${delivery}` }]
+          const content = [{ type: 'text', text: `Captured ${Object.keys(result.captures).join(' + ')} for ${result.route}. ${delivery}` }]
           for (const [name, capture] of Object.entries(result.captures)) {
             content.push({ type: 'text', text: `${name}: ${capture.width}×${capture.height}${capture.capped ? ' (height capped)' : ''}` })
             content.push({ type: 'image', data: capture.png.toString('base64'), mimeType: 'image/png' })
@@ -771,16 +783,16 @@ export function createViewportTools({ tool, z, manager }) {
     ),
     tool(
       'preview_inspect',
-      'Inspect rendered DOM state in this lane preview at an exact desktop or mobile viewport. Returns page text, document dimensions, and optional selector text/geometry without changing the page.',
+      'Inspect rendered DOM state in this lane preview at an exact desktop or mobile viewport. Omit path for the current authenticated room; pass / explicitly for the public landing page. Returns page text, document dimensions, and optional selector text/geometry without changing the page.',
       {
-        path: z.string().max(500).optional().describe('route within the preview; never a full URL'),
+        path: z.string().max(500).optional().describe('route within the preview; omit for the current authenticated room, or pass / for the public landing page; never a full URL'),
         viewport: z.enum(['desktop', 'mobile']).optional(),
         selector: z.string().max(500).optional(),
         waitMs: z.number().int().min(0).max(MAX_SETTLE_MS).optional(),
       },
       async (args) => {
         try {
-          const result = await manager.inspect({ route: args?.path || '/', viewport: args?.viewport || 'mobile', selector: args?.selector, waitMs: args?.waitMs ?? 300 })
+          const result = await manager.inspect({ route: args?.path, viewport: args?.viewport || 'mobile', selector: args?.selector, waitMs: args?.waitMs ?? 300 })
           return textResult(JSON.stringify(result, null, 2))
         } catch (error) { return errorResult(error) }
       },