mixdog 1.3.5 → 1.4.0

package/src/runtime/visualize-bridge/client.mjs

@@ -6,6 +6,7 @@
  * Response: { ok:true, text, tabs?, path? } | { ok:false, error }.
  */
 import { bridgeDiscoveryChanged, readBridgeDiscovery, readBridgeDiscoveryDetail } from '../bridge-discovery.mjs';
+import { visualizeOwnerToolMeta } from './tool-meta.mjs';
 
 const DISCOVERY_FILE = 'visualize-bridge.json';
 const REQUEST_TIMEOUT_MS = 30_000;
@@ -38,12 +39,14 @@ export function validateVisualizeArgs(args) {
   if (!args || typeof args !== 'object' || Array.isArray(args)) {
     return { ok: false, error: 'visualize arguments must be an object' };
   }
-  if (!ACTIONS.has(args.action)) return { ok: false, error: 'action must be open, close, annotate, capture, export, check, or inspect' };
+  if (!ACTIONS.has(args.action))
+    return { ok: false, error: 'action must be open, close, annotate, capture, export, check, or inspect' };
   const input = args.input ?? {};
   if (typeof input !== 'object' || Array.isArray(input)) return { ok: false, error: 'input must be an object' };
   if (args.action === 'open' && input.content != null) {
     const content = input.content;
-    if (input.targets != null) return { ok: false, error: 'open takes either input.targets or input.content, not both' };
+    if (input.targets != null)
+      return { ok: false, error: 'open takes either input.targets or input.content, not both' };
     if (typeof content !== 'object' || Array.isArray(content)) {
       return { ok: false, error: 'open input.content must be { kind, source, title? }' };
     }
@@ -79,7 +82,10 @@ export function validateVisualizeArgs(args) {
       return { ok: false, error: 'open input.version must be a positive integer (a kept version number)' };
     }
   }
-  if (['annotate', 'capture', 'export', 'check', 'inspect'].includes(args.action) && (typeof input.tab !== 'string' || !input.tab)) {
+  if (
+    ['annotate', 'capture', 'export', 'check', 'inspect'].includes(args.action) &&
+    (typeof input.tab !== 'string' || !input.tab)
+  ) {
     return { ok: false, error: `${args.action} requires input.tab` };
   }
   if (args.action === 'inspect') {
@@ -92,7 +98,10 @@ export function validateVisualizeArgs(args) {
         !sized(viewport.width, 200, 3000) ||
         (viewport.height != null && !sized(viewport.height, 200, 4000))
       ) {
-        return { ok: false, error: 'inspect input.viewport must be { width: 200-3000, height?: 200-4000 } in whole pixels' };
+        return {
+          ok: false,
+          error: 'inspect input.viewport must be { width: 200-3000, height?: 200-4000 } in whole pixels',
+        };
       }
     }
     if (selector != null && (typeof selector !== 'string' || !selector.trim() || selector.length > 500)) {
@@ -103,7 +112,8 @@ export function validateVisualizeArgs(args) {
     return { ok: false, error: 'capture requires input.output ending in .png' };
   }
   if (args.action === 'export') {
-    if (input.format !== 'png' && input.format !== 'pdf') return { ok: false, error: 'export requires input.format png or pdf' };
+    if (input.format !== 'png' && input.format !== 'pdf')
+      return { ok: false, error: 'export requires input.format png or pdf' };
     if (typeof input.output !== 'string' || !new RegExp('\\.' + input.format + '$', 'i').test(input.output.trim())) {
       return { ok: false, error: `export requires input.output ending in .${input.format}` };
     }
@@ -160,7 +170,11 @@ export async function executeVisualizeTool(args, options = {}) {
   for (let attempt = 0; attempt < 2; attempt += 1) {
     const timeout = AbortSignal.timeout(validated.action === 'export' ? EXPORT_REQUEST_TIMEOUT_MS : REQUEST_TIMEOUT_MS);
     try {
-      result = await requestBridge(discovery, payload, options.signal ? AbortSignal.any([timeout, options.signal]) : timeout);
+      result = await requestBridge(
+        discovery,
+        payload,
+        options.signal ? AbortSignal.any([timeout, options.signal]) : timeout
+      );
       break;
     } catch (error) {
       if (error?.name === 'TimeoutError') return toolError('visualize bridge timed out');
@@ -175,7 +189,12 @@ export async function executeVisualizeTool(args, options = {}) {
   }
   const { body, status } = result;
   if (!body?.ok) {
-    return toolError(String(body?.error || `visualize bridge request failed (HTTP ${status})`).replace(/^Error:\s*/, ''));
+    return toolError(
+      String(body?.error || `visualize bridge request failed (HTTP ${status})`).replace(/^Error:\s*/, '')
+    );
   }
-  return { content: [{ type: 'text', text: formatSuccess(body) }] };
+  // The host ran it for another session's panel: the owner rides as a field
+  // (toolMeta, kept with the stored result), never only as prose.
+  const toolMeta = visualizeOwnerToolMeta(body.ownerSessionId);
+  return { content: [{ type: 'text', text: formatSuccess(body) }], ...(toolMeta ? { toolMeta } : {}) };
 }

package/src/runtime/visualize-bridge/tool-defs.mjs

@@ -10,8 +10,8 @@ export const TOOL_DEFS = [
     name: 'visualize',
     title: 'Mixdog Visualize',
     description:
-      "Show the user any artifact you produced or changed (mockups, docs, decks, sheets, PDFs, images, video, local dev pages) in the desktop side panel beside the chat. Panel tabs auto-reload when a file changes, so re-opening after edits is unnecessary. " +
-      'open: input.targets (1-3 file paths, 2-3 = side-by-side comparison; or a single http(s) URL alone, never mixed with files or other URLs), optional label, optional version (show that kept version of a file tab read-only; each open/reload that changes the files keeps a version). close: optional tab (omit = all). ' +
+      'Show the user any artifact you produced or changed (mockups, docs, decks, sheets, PDFs, images, video, local dev pages) in the desktop side panel beside the chat. Panel tabs auto-reload when a file changes, so re-opening after edits is unnecessary. ' +
+      'open: input.targets (1-3 file paths, relative to the working directory or absolute anywhere on this machine; 2-3 = side-by-side comparison; or a single http(s) URL alone, never mixed with files or other URLs), optional label, optional version (show that kept version of a file tab read-only; each open/reload that changes the files keeps a version). close: optional tab (omit = all). ' +
       'annotate: tab + add [{selector,label?}] | remove [selector] | clear; HTML pages only, never edits the file. capture: tab + output (.png path). ' +
       'export: tab + format (png|pdf) + output (project-relative path ending .png/.pdf) + optional version (a kept version of the tab) + optional overwrite (true replaces an existing file, only under design/; otherwise an existing file is refused). Writes a record of the approved page inside the project and returns its project-relative path, size/dimensions and the page problems seen while rendering. A single HTML page (file, or inline html/chart) is rendered whole in a hidden window, so the tab need not be on screen (PDF: HTML pages only; PNG has a pixel cap, over it export pdf or shorten the page); any other tab exports as capture does (the displayed tab, PNG only). Same project containment as capture: links and paths outside the working directory are refused. ' +
       'check: tab; HTML pages only; waits briefly for the page to settle and returns only its runtime problems (uncaught errors, unhandled rejections, CSP violations, console errors/warnings, missing local assets). ' +
@@ -36,7 +36,8 @@ export const TOOL_DEFS = [
               items: { type: 'string' },
               minItems: 1,
               maxItems: 3,
-              description: 'open: 1-3 project-relative/absolute file paths (side by side), or exactly one http(s) URL as the only target.',
+              description:
+                'open: 1-3 file paths (side by side), relative to the working directory or absolute in any folder, or exactly one http(s) URL as the only target.',
             },
             content: {
               type: 'object',
@@ -57,7 +58,8 @@ export const TOOL_DEFS = [
             version: {
               type: 'integer',
               minimum: 1,
-              description: 'open (file tabs): kept version number to show read-only instead of the live files. export: kept version number to render instead of the live page.',
+              description:
+                'open (file tabs): kept version number to show read-only instead of the live files. export: kept version number to render instead of the live page.',
             },
             tab: { type: 'string', description: 'Tab id from open (close: omit for all).' },
             add: {
@@ -72,9 +74,16 @@ export const TOOL_DEFS = [
             },
             remove: { type: 'array', items: { type: 'string' }, description: 'annotate: selectors to un-outline.' },
             clear: { type: 'boolean', description: 'annotate: remove all outlines.' },
-            output: { type: 'string', description: 'capture: project path ending .png. export: project path ending .png or .pdf (matching format).' },
+            output: {
+              type: 'string',
+              description:
+                'capture: project path ending .png. export: project path ending .png or .pdf (matching format).',
+            },
             format: { type: 'string', enum: ['png', 'pdf'], description: 'export: file format.' },
-            overwrite: { type: 'boolean', description: 'export: replace an existing output file; honored only for paths under design/.' },
+            overwrite: {
+              type: 'boolean',
+              description: 'export: replace an existing output file; honored only for paths under design/.',
+            },
             viewport: {
               type: 'object',
               properties: {
@@ -83,9 +92,14 @@ export const TOOL_DEFS = [
               },
               required: ['width'],
               additionalProperties: false,
-              description: 'inspect: size to lay the page out at (height defaults to 800); omit for the panel\'s current size.',
+              description:
+                "inspect: size to lay the page out at (height defaults to 800); omit for the panel's current size.",
+            },
+            selector: {
+              type: 'string',
+              maxLength: 500,
+              description: 'inspect: CSS selector scoping the read-out to one subtree.',
             },
-            selector: { type: 'string', maxLength: 500, description: 'inspect: CSS selector scoping the read-out to one subtree.' },
           },
           additionalProperties: false,
         },

package/src/runtime/visualize-bridge/tool-meta.d.mts

@@ -0,0 +1,5 @@
+export declare const VISUALIZE_OWNER_META_CODE: 'visualize_owner';
+export declare function visualizeOwnerMetaSessionId(value: unknown): string;
+export declare function visualizeOwnerToolMeta(
+  ownerSessionId: unknown
+): { code: typeof VISUALIZE_OWNER_META_CODE; missing: string[]; ownerSessionId: string } | undefined;

package/src/runtime/visualize-bridge/tool-meta.mjs

@@ -0,0 +1,22 @@
+/**
+ * The structured twin of a `visualize` result (the tool result's `toolMeta`,
+ * kept with the stored message and restored onto its transcript item): the
+ * desktop host ran the call for another session's work (a sub-agent, a room
+ * member) and keeps its tabs in that owner conversation's panel. Chat cards
+ * read the owner from here, never from the result text.
+ */
+export const VISUALIZE_OWNER_META_CODE = 'visualize_owner';
+
+const SESSION_ID = /^[A-Za-z0-9_.:-]{1,128}$/;
+
+/** A session id fit for `toolMeta.ownerSessionId`; '' for anything else. */
+export function visualizeOwnerMetaSessionId(value) {
+  return typeof value === 'string' && SESSION_ID.test(value) ? value : '';
+}
+
+/** The `toolMeta` of a result run for `ownerSessionId`'s panel; undefined when
+ *  the id is not a session id. */
+export function visualizeOwnerToolMeta(ownerSessionId) {
+  const id = visualizeOwnerMetaSessionId(ownerSessionId);
+  return id ? { code: VISUALIZE_OWNER_META_CODE, missing: [], ownerSessionId: id } : undefined;
+}

package/src/runtime/web-search/lib/fetch-pipeline.mjs

@@ -81,7 +81,7 @@ function fetchFailureKind(error) {
 
 export async function runFetchPipeline(
   url,
-  { http, browser, signal, timeoutMs = 30000, hostPacer = pacer, onAttempt = () => {} } = {}
+  { http, browser, signal, timeoutMs = 30000, hostPacer = pacer, onAttempt = () => {}, sleep: pause = sleep } = {}
 ) {
   const budget = Number.isFinite(timeoutMs) && timeoutMs > 0 ? timeoutMs : 30000;
   const deadline = Date.now() + budget;
@@ -150,7 +150,7 @@ export async function runFetchPipeline(
         if (!transient || retry === 1) break;
         const delay = Math.max(400, error.retryAfterMs || 0);
         if (Date.now() + delay >= deadline) throw error;
-        await sleep(delay, undefined, { signal: overall });
+        await pause(delay, undefined, { signal: overall });
       }
     }
     // Rendering does not fix a rate limit or a failing upstream server.