donobu 5.60.5 → 5.60.7

package/dist/esm/lib/ai/locate/locateElement.js

@@ -13,6 +13,14 @@ const locateSchema_1 = require("./locateSchema");
 const DISAMBIGUATE_THRESHOLD = 5;
 /** Maximum outer HTML length per snippet shown during disambiguation. */
 const SNIPPET_MAX_CHARS = 200;
+/**
+ * Per-candidate read budget during disambiguation. Locator `evaluate`/
+ * `boundingBox` default to *no* timeout, so a candidate that detaches mid-loop
+ * would wait forever (the surrounding try/catch only traps errors, not hangs,
+ * and the AbortSignal isn't plumbed into these ops). Cap it and let a miss fall
+ * through to the placeholder snippet below.
+ */
+const CANDIDATE_READ_TIMEOUT_MILLIS = 1000;
 /**
  * Resolve a natural-language element description to a Playwright {@link Locator}.
  *
@@ -87,8 +95,10 @@ async function disambiguate(page, description, gptClient, locator, locateResult,
             const html = await nth.evaluate((el, max) => {
                 const raw = el.outerHTML;
                 return raw.length > max ? raw.slice(0, max) + '…' : raw;
-            }, SNIPPET_MAX_CHARS);
-            const box = await nth.boundingBox();
+            }, SNIPPET_MAX_CHARS, { timeout: CANDIDATE_READ_TIMEOUT_MILLIS });
+            const box = await nth.boundingBox({
+                timeout: CANDIDATE_READ_TIMEOUT_MILLIS,
+            });
             const pos = box
                 ? `(${Math.round(box.x)}, ${Math.round(box.y)})`
                 : '(unknown)';

package/dist/esm/lib/page/extendPage.js

@@ -166,7 +166,7 @@ async function extendPage(page, options) {
             aiInvocations: [],
         };
         const showMouse = async (p) => {
-            if (interactionVisualizer.defaultMessageDurationMillis > 0) {
+            if (interactionVisualizer.defaultCursorDurationMillis > 0) {
                 await interactionVisualizer.showMouse(p);
             }
         };

package/dist/esm/managers/DonobuFlowsManager.js

@@ -174,7 +174,7 @@ class DonobuFlowsManager {
                     gptConfigName: gptClientData.gptConfigName,
                     hasGptConfigNameOverride: !gptClientData.agentName,
                     customTools: flowParams.customTools ?? null,
-                    defaultMessageDuration: interactionVisualizer.defaultMessageDurationMillis,
+                    defaultMessageDuration: interactionVisualizer.defaultCursorDurationMillis,
                     callbackUrl: flowParams.callbackUrl || null,
                     overallObjective: flowParams.overallObjective ?? null,
                     allowedTools: allowedTools.map((tool) => tool.name),

package/dist/esm/managers/InteractionVisualizer.d.ts

@@ -1,37 +1,38 @@
 import type { Locator, Page } from 'playwright';
 export declare class InteractionVisualizer {
-    readonly defaultMessageDurationMillis: number;
+    readonly defaultCursorDurationMillis: number;
     private static readonly RAW_MOUSE_D;
     private static readonly TIP_X;
     private static readonly TIP_Y;
     private static readonly SVG_MOUSE;
-    /** Message tooltip box styling — mirrors the legacy `.donobu-message` rule. */
-    private static readonly MESSAGE_MAX_WIDTH;
-    private static readonly MESSAGE_MARGIN;
+    /**
+     * Upper bound on resolving a target's bounding box. `Locator.boundingBox`
+     * defaults to *no* timeout, so a locator that resolves to zero elements
+     * (e.g. detached mid-flow) would otherwise wait forever and hang the flow.
+     * The cursor is cosmetic, so we cap the wait and treat a miss as a no-op.
+     */
+    private static readonly BOUNDING_BOX_TIMEOUT_MILLIS;
     /**
      * Per-page cursor state. Keyed weakly so closed/GC'd pages drop out without
      * us tracking page lifecycles.
      */
     private readonly states;
-    constructor(defaultMessageDurationMillis: number);
-    private static escapeHtml;
+    constructor(defaultCursorDurationMillis: number);
     /**
-     * Moves the virtual cursor to the center of the specified element and optionally displays a message.
+     * Moves the virtual cursor to the center of the specified element.
      *
      * @param page - The Playwright page instance where the cursor will be displayed.
      * @param locator - Optional target element to point at. If omitted, cursor remains at current position.
-     * @param message - Optional message to display near the cursor during the interaction.
-     * @param duration - Duration in milliseconds for the cursor animation and message display.
+     * @param duration - Duration in milliseconds for the cursor animation.
      *                   Defaults to the instance's configured duration. If ≤ 0, no action is taken
      *
-     * @returns Promise that resolves when the cursor movement and message display are complete
+     * @returns Promise that resolves once the cursor has reached the target
      *
      * @remarks
      * - The cursor animates smoothly to the element's center point
-     * - Messages are positioned automatically to avoid viewport edges
-     * - Overlays are `pointer-events: none`, so they never intercept real interactions
+     * - The overlay is `pointer-events: none`, so it never intercepts real interactions
      */
-    pointAt(page: Page, locator?: Pick<Locator, 'boundingBox'>, message?: string, duration?: number): Promise<void>;
+    pointAt(page: Page, locator?: Pick<Locator, 'boundingBox'>, duration?: number): Promise<void>;
     /**
      * Shows the virtual mouse cursor on the page at its current position.
      *
@@ -54,18 +55,10 @@ export declare class InteractionVisualizer {
      * (e.g. cleared by a navigation) by swallowing its dispose error.
      */
     private renderCursor;
-    private showMessage;
     /**
      * Builds the cursor overlay markup: the arrow SVG plus a `@keyframes` glide
      * from the previous position and a one-shot ripple at the arrow tip.
      */
     private buildCursorHtml;
-    /**
-     * Builds the message tooltip markup, positioned near `target` and clamped
-     * inside the viewport. Without being able to measure the rendered box, we
-     * keep it on-screen with a conservative max-width clamp and anchor it above
-     * the cursor (via `translateY(-100%)`) when the cursor sits low in the page.
-     */
-    private buildMessageHtml;
 }
 //# sourceMappingURL=InteractionVisualizer.d.ts.map

package/dist/esm/managers/InteractionVisualizer.js

@@ -4,8 +4,8 @@ exports.InteractionVisualizer = void 0;
 const Logger_1 = require("../utils/Logger");
 const PlaywrightUtils_1 = require("../utils/PlaywrightUtils");
 class InteractionVisualizer {
-    constructor(defaultMessageDurationMillis) {
-        this.defaultMessageDurationMillis = defaultMessageDurationMillis;
+    constructor(defaultCursorDurationMillis) {
+        this.defaultCursorDurationMillis = defaultCursorDurationMillis;
         /**
          * Per-page cursor state. Keyed weakly so closed/GC'd pages drop out without
          * us tracking page lifecycles.
@@ -15,30 +15,21 @@ class InteractionVisualizer {
     /* ------------------------------------------------------------------ */
     /*  Public API                                                        */
     /* ------------------------------------------------------------------ */
-    static escapeHtml(text) {
-        return text
-            .replace(/&/g, '&')
-            .replace(/</g, '&lt;')
-            .replace(/>/g, '&gt;')
-            .replace(/"/g, '&quot;');
-    }
     /**
-     * Moves the virtual cursor to the center of the specified element and optionally displays a message.
+     * Moves the virtual cursor to the center of the specified element.
      *
      * @param page - The Playwright page instance where the cursor will be displayed.
      * @param locator - Optional target element to point at. If omitted, cursor remains at current position.
-     * @param message - Optional message to display near the cursor during the interaction.
-     * @param duration - Duration in milliseconds for the cursor animation and message display.
+     * @param duration - Duration in milliseconds for the cursor animation.
      *                   Defaults to the instance's configured duration. If ≤ 0, no action is taken
      *
-     * @returns Promise that resolves when the cursor movement and message display are complete
+     * @returns Promise that resolves once the cursor has reached the target
      *
      * @remarks
      * - The cursor animates smoothly to the element's center point
-     * - Messages are positioned automatically to avoid viewport edges
-     * - Overlays are `pointer-events: none`, so they never intercept real interactions
+     * - The overlay is `pointer-events: none`, so it never intercepts real interactions
      */
-    async pointAt(page, locator, message, duration = this.defaultMessageDurationMillis) {
+    async pointAt(page, locator, duration = this.defaultCursorDurationMillis) {
         if (!duration || duration <= 0) {
             return;
         }
@@ -47,20 +38,21 @@ class InteractionVisualizer {
             const from = { ...state.pos };
             let target = from;
             if (locator) {
-                const box = await locator.boundingBox();
+                // Cap the wait and swallow a timeout: an unresolvable locator must
+                // leave the (cosmetic) cursor in place, never block the flow.
+                const box = await locator
+                    .boundingBox({
+                    timeout: InteractionVisualizer.BOUNDING_BOX_TIMEOUT_MILLIS,
+                })
+                    .catch(() => null);
                 if (box) {
                     target = { x: box.x + box.width / 2, y: box.y + box.height / 2 };
                 }
             }
             state.pos = target;
-            await Promise.all([
-                state.visible
-                    ? this.renderCursor(page, state, from, target, duration / 2)
-                    : Promise.resolve(),
-                message?.trim()
-                    ? this.showMessage(page, target, message.trim(), duration)
-                    : Promise.resolve(),
-            ]);
+            if (state.visible) {
+                await this.renderCursor(page, state, from, target, duration / 2);
+            }
         }
         catch (error) {
             if (!PlaywrightUtils_1.PlaywrightUtils.isPageClosedError(error)) {
@@ -137,26 +129,15 @@ class InteractionVisualizer {
         // showOverlay resolves the moment the overlay is added, before the CSS
         // glide plays. Callers (e.g. ClickTool) await pointAt expecting the cursor
         // to reach the element *before* they act, so block for the animation here.
-        if (animMs > 0) {
+        // Only the moving case animates (see buildCursorHtml), so a no-op move
+        // (e.g. an unresolved target) shouldn't incur the wait.
+        const animating = animMs > 0 && (from.x !== to.x || from.y !== to.y);
+        if (animating) {
             await new Promise((resolve) => {
                 setTimeout(resolve, animMs);
             });
         }
     }
-    async showMessage(page, target, text, duration) {
-        // Sticky overlay + self-managed removal. The `duration` option on
-        // showOverlay only composites into the recorded screencast stream, not the
-        // live page render layer (so it would be invisible in headed/screenshots);
-        // a no-duration overlay renders everywhere, matching the cursor.
-        const handle = await page.screencast.showOverlay(this.buildMessageHtml(page, target, text));
-        const timer = setTimeout(() => {
-            handle.dispose().catch(() => {
-                // Page navigated/closed before the message expired — nothing to clean up.
-            });
-        }, duration);
-        // Don't let a pending message removal keep the process alive on shutdown.
-        timer.unref?.();
-    }
     /**
      * Builds the cursor overlay markup: the arrow SVG plus a `@keyframes` glide
      * from the previous position and a one-shot ripple at the arrow tip.
@@ -179,41 +160,6 @@ class InteractionVisualizer {
 </style>
 <div class="donobu-cursor">${InteractionVisualizer.SVG_MOUSE}</div>`;
     }
-    /**
-     * Builds the message tooltip markup, positioned near `target` and clamped
-     * inside the viewport. Without being able to measure the rendered box, we
-     * keep it on-screen with a conservative max-width clamp and anchor it above
-     * the cursor (via `translateY(-100%)`) when the cursor sits low in the page.
-     */
-    buildMessageHtml(page, target, text) {
-        const maxW = InteractionVisualizer.MESSAGE_MAX_WIDTH;
-        const margin = InteractionVisualizer.MESSAGE_MARGIN;
-        const vp = page.viewportSize() ?? { width: 1280, height: 720 };
-        // Clamp the (centered) anchor so a max-width box can't overflow either edge.
-        const centerX = Math.max(maxW / 2 + margin, Math.min(vp.width - maxW / 2 - margin, target.x));
-        const placeAbove = target.y > vp.height * 0.6;
-        const top = placeAbove ? Math.max(margin, target.y - 24) : target.y + 24;
-        const translate = placeAbove
-            ? 'translate(-50%, -100%)'
-            : 'translate(-50%, 0)';
-        const style = [
-            'position:fixed',
-            `left:${centerX}px`,
-            `top:${top}px`,
-            `transform:${translate}`,
-            `max-width:${maxW}px`,
-            'z-index:2147483647',
-            'pointer-events:none',
-            'background:#000',
-            'color:#fff',
-            'padding:8px 10px',
-            'border-radius:5px',
-            "font:12px/1 'Source Sans Pro','Helvetica Neue',Helvetica,Arial,sans-serif",
-            'white-space:pre-wrap',
-            'box-shadow:2px 2px 10px rgba(0,0,0,.2)',
-        ].join(';');
-        return `<div style="${style}">${InteractionVisualizer.escapeHtml(text)}</div>`;
-    }
 }
 exports.InteractionVisualizer = InteractionVisualizer;
 /* ------------------------------------------------------------------ */
@@ -235,7 +181,11 @@ InteractionVisualizer.SVG_MOUSE = `
      stroke-linejoin="round">
   <path d="${InteractionVisualizer.RAW_MOUSE_D}" />
 </svg>`;
-/** Message tooltip box styling — mirrors the legacy `.donobu-message` rule. */
-InteractionVisualizer.MESSAGE_MAX_WIDTH = 300;
-InteractionVisualizer.MESSAGE_MARGIN = 8;
+/**
+ * Upper bound on resolving a target's bounding box. `Locator.boundingBox`
+ * defaults to *no* timeout, so a locator that resolves to zero elements
+ * (e.g. detached mid-flow) would otherwise wait forever and hang the flow.
+ * The cursor is cosmetic, so we cap the wait and treat a miss as a no-op.
+ */
+InteractionVisualizer.BOUNDING_BOX_TIMEOUT_MILLIS = 1000;
 //# sourceMappingURL=InteractionVisualizer.js.map

package/dist/esm/tools/ReplayableInteraction.js

@@ -432,7 +432,7 @@ class ReplayableInteraction extends Tool_1.Tool {
         // Only reveal the cursor now that we have a real target to point at, so a
         // non-interactive proposal never pops a stationary cursor.
         await (0, TargetUtils_1.webInspector)(context).showInteractionCursor();
-        await context.interactionVisualizer.pointAt(page, pointTarget.first(), undefined, ReplayableInteraction.PREVIEW_CURSOR_DURATION_MILLIS);
+        await context.interactionVisualizer.pointAt(page, pointTarget.first(), ReplayableInteraction.PREVIEW_CURSOR_DURATION_MILLIS);
     }
     async callFromGpt(context, parameters) {
         const page = (0, TargetUtils_1.webPage)(context);

package/dist/esm/utils/MiscUtils.js

@@ -204,25 +204,21 @@ class MiscUtils {
      * Merges adjacent user messages (some LLMs like Gemini's or Anthropic's require this).
      */
     static mergeAdjacentUserMessages(messages) {
-        let updatedMessages = [];
-        for (let i = 0; i < messages.length; ++i) {
-            if (i === messages.length - 1) {
-                updatedMessages.push(messages[i]);
-                break;
-            }
-            const message = messages[i];
-            const adjacentMessage = messages[i + 1];
-            if (message.type === 'user' && adjacentMessage.type === 'user') {
-                const mergedMessage = {
+        const updatedMessages = [];
+        // Fold each user message into the previously emitted one when that is also
+        // a user message. Folding into the accumulated result (rather than
+        // consuming pairs) collapses runs of any length — pair-consuming would
+        // leave the 3rd, 5th, ... message in a run un-merged.
+        for (const message of messages) {
+            const last = updatedMessages[updatedMessages.length - 1];
+            if (message.type === 'user' && last?.type === 'user') {
+                updatedMessages[updatedMessages.length - 1] = {
                     type: 'user',
-                    items: [...message.items, ...adjacentMessage.items],
+                    items: [...last.items, ...message.items],
                 };
-                updatedMessages.push(mergedMessage);
-                // Skip the next message.
-                ++i;
             }
             else {
-                updatedMessages.push(messages[i]);
+                updatedMessages.push(message);
             }
         }
         return updatedMessages;

package/dist/lib/ai/locate/locateElement.js

@@ -13,6 +13,14 @@ const locateSchema_1 = require("./locateSchema");
 const DISAMBIGUATE_THRESHOLD = 5;
 /** Maximum outer HTML length per snippet shown during disambiguation. */
 const SNIPPET_MAX_CHARS = 200;
+/**
+ * Per-candidate read budget during disambiguation. Locator `evaluate`/
+ * `boundingBox` default to *no* timeout, so a candidate that detaches mid-loop
+ * would wait forever (the surrounding try/catch only traps errors, not hangs,
+ * and the AbortSignal isn't plumbed into these ops). Cap it and let a miss fall
+ * through to the placeholder snippet below.
+ */
+const CANDIDATE_READ_TIMEOUT_MILLIS = 1000;
 /**
  * Resolve a natural-language element description to a Playwright {@link Locator}.
  *
@@ -87,8 +95,10 @@ async function disambiguate(page, description, gptClient, locator, locateResult,
             const html = await nth.evaluate((el, max) => {
                 const raw = el.outerHTML;
                 return raw.length > max ? raw.slice(0, max) + '…' : raw;
-            }, SNIPPET_MAX_CHARS);
-            const box = await nth.boundingBox();
+            }, SNIPPET_MAX_CHARS, { timeout: CANDIDATE_READ_TIMEOUT_MILLIS });
+            const box = await nth.boundingBox({
+                timeout: CANDIDATE_READ_TIMEOUT_MILLIS,
+            });
             const pos = box
                 ? `(${Math.round(box.x)}, ${Math.round(box.y)})`
                 : '(unknown)';

package/dist/lib/page/extendPage.js

@@ -166,7 +166,7 @@ async function extendPage(page, options) {
             aiInvocations: [],
         };
         const showMouse = async (p) => {
-            if (interactionVisualizer.defaultMessageDurationMillis > 0) {
+            if (interactionVisualizer.defaultCursorDurationMillis > 0) {
                 await interactionVisualizer.showMouse(p);
             }
         };

package/dist/managers/DonobuFlowsManager.js

@@ -174,7 +174,7 @@ class DonobuFlowsManager {
                     gptConfigName: gptClientData.gptConfigName,
                     hasGptConfigNameOverride: !gptClientData.agentName,
                     customTools: flowParams.customTools ?? null,
-                    defaultMessageDuration: interactionVisualizer.defaultMessageDurationMillis,
+                    defaultMessageDuration: interactionVisualizer.defaultCursorDurationMillis,
                     callbackUrl: flowParams.callbackUrl || null,
                     overallObjective: flowParams.overallObjective ?? null,
                     allowedTools: allowedTools.map((tool) => tool.name),

package/dist/managers/InteractionVisualizer.d.ts

@@ -1,37 +1,38 @@
 import type { Locator, Page } from 'playwright';
 export declare class InteractionVisualizer {
-    readonly defaultMessageDurationMillis: number;
+    readonly defaultCursorDurationMillis: number;
     private static readonly RAW_MOUSE_D;
     private static readonly TIP_X;
     private static readonly TIP_Y;
     private static readonly SVG_MOUSE;
-    /** Message tooltip box styling — mirrors the legacy `.donobu-message` rule. */
-    private static readonly MESSAGE_MAX_WIDTH;
-    private static readonly MESSAGE_MARGIN;
+    /**
+     * Upper bound on resolving a target's bounding box. `Locator.boundingBox`
+     * defaults to *no* timeout, so a locator that resolves to zero elements
+     * (e.g. detached mid-flow) would otherwise wait forever and hang the flow.
+     * The cursor is cosmetic, so we cap the wait and treat a miss as a no-op.
+     */
+    private static readonly BOUNDING_BOX_TIMEOUT_MILLIS;
     /**
      * Per-page cursor state. Keyed weakly so closed/GC'd pages drop out without
      * us tracking page lifecycles.
      */
     private readonly states;
-    constructor(defaultMessageDurationMillis: number);
-    private static escapeHtml;
+    constructor(defaultCursorDurationMillis: number);
     /**
-     * Moves the virtual cursor to the center of the specified element and optionally displays a message.
+     * Moves the virtual cursor to the center of the specified element.
      *
      * @param page - The Playwright page instance where the cursor will be displayed.
      * @param locator - Optional target element to point at. If omitted, cursor remains at current position.
-     * @param message - Optional message to display near the cursor during the interaction.
-     * @param duration - Duration in milliseconds for the cursor animation and message display.
+     * @param duration - Duration in milliseconds for the cursor animation.
      *                   Defaults to the instance's configured duration. If ≤ 0, no action is taken
      *
-     * @returns Promise that resolves when the cursor movement and message display are complete
+     * @returns Promise that resolves once the cursor has reached the target
      *
      * @remarks
      * - The cursor animates smoothly to the element's center point
-     * - Messages are positioned automatically to avoid viewport edges
-     * - Overlays are `pointer-events: none`, so they never intercept real interactions
+     * - The overlay is `pointer-events: none`, so it never intercepts real interactions
      */
-    pointAt(page: Page, locator?: Pick<Locator, 'boundingBox'>, message?: string, duration?: number): Promise<void>;
+    pointAt(page: Page, locator?: Pick<Locator, 'boundingBox'>, duration?: number): Promise<void>;
     /**
      * Shows the virtual mouse cursor on the page at its current position.
      *
@@ -54,18 +55,10 @@ export declare class InteractionVisualizer {
      * (e.g. cleared by a navigation) by swallowing its dispose error.
      */
     private renderCursor;
-    private showMessage;
     /**
      * Builds the cursor overlay markup: the arrow SVG plus a `@keyframes` glide
      * from the previous position and a one-shot ripple at the arrow tip.
      */
     private buildCursorHtml;
-    /**
-     * Builds the message tooltip markup, positioned near `target` and clamped
-     * inside the viewport. Without being able to measure the rendered box, we
-     * keep it on-screen with a conservative max-width clamp and anchor it above
-     * the cursor (via `translateY(-100%)`) when the cursor sits low in the page.
-     */
-    private buildMessageHtml;
 }
 //# sourceMappingURL=InteractionVisualizer.d.ts.map

package/dist/managers/InteractionVisualizer.js

@@ -4,8 +4,8 @@ exports.InteractionVisualizer = void 0;
 const Logger_1 = require("../utils/Logger");
 const PlaywrightUtils_1 = require("../utils/PlaywrightUtils");
 class InteractionVisualizer {
-    constructor(defaultMessageDurationMillis) {
-        this.defaultMessageDurationMillis = defaultMessageDurationMillis;
+    constructor(defaultCursorDurationMillis) {
+        this.defaultCursorDurationMillis = defaultCursorDurationMillis;
         /**
          * Per-page cursor state. Keyed weakly so closed/GC'd pages drop out without
          * us tracking page lifecycles.
@@ -15,30 +15,21 @@ class InteractionVisualizer {
     /* ------------------------------------------------------------------ */
     /*  Public API                                                        */
     /* ------------------------------------------------------------------ */
-    static escapeHtml(text) {
-        return text
-            .replace(/&/g, '&')
-            .replace(/</g, '&lt;')
-            .replace(/>/g, '&gt;')
-            .replace(/"/g, '&quot;');
-    }
     /**
-     * Moves the virtual cursor to the center of the specified element and optionally displays a message.
+     * Moves the virtual cursor to the center of the specified element.
      *
      * @param page - The Playwright page instance where the cursor will be displayed.
      * @param locator - Optional target element to point at. If omitted, cursor remains at current position.
-     * @param message - Optional message to display near the cursor during the interaction.
-     * @param duration - Duration in milliseconds for the cursor animation and message display.
+     * @param duration - Duration in milliseconds for the cursor animation.
      *                   Defaults to the instance's configured duration. If ≤ 0, no action is taken
      *
-     * @returns Promise that resolves when the cursor movement and message display are complete
+     * @returns Promise that resolves once the cursor has reached the target
      *
      * @remarks
      * - The cursor animates smoothly to the element's center point
-     * - Messages are positioned automatically to avoid viewport edges
-     * - Overlays are `pointer-events: none`, so they never intercept real interactions
+     * - The overlay is `pointer-events: none`, so it never intercepts real interactions
      */
-    async pointAt(page, locator, message, duration = this.defaultMessageDurationMillis) {
+    async pointAt(page, locator, duration = this.defaultCursorDurationMillis) {
         if (!duration || duration <= 0) {
             return;
         }
@@ -47,20 +38,21 @@ class InteractionVisualizer {
             const from = { ...state.pos };
             let target = from;
             if (locator) {
-                const box = await locator.boundingBox();
+                // Cap the wait and swallow a timeout: an unresolvable locator must
+                // leave the (cosmetic) cursor in place, never block the flow.
+                const box = await locator
+                    .boundingBox({
+                    timeout: InteractionVisualizer.BOUNDING_BOX_TIMEOUT_MILLIS,
+                })
+                    .catch(() => null);
                 if (box) {
                     target = { x: box.x + box.width / 2, y: box.y + box.height / 2 };
                 }
             }
             state.pos = target;
-            await Promise.all([
-                state.visible
-                    ? this.renderCursor(page, state, from, target, duration / 2)
-                    : Promise.resolve(),
-                message?.trim()
-                    ? this.showMessage(page, target, message.trim(), duration)
-                    : Promise.resolve(),
-            ]);
+            if (state.visible) {
+                await this.renderCursor(page, state, from, target, duration / 2);
+            }
         }
         catch (error) {
             if (!PlaywrightUtils_1.PlaywrightUtils.isPageClosedError(error)) {
@@ -137,26 +129,15 @@ class InteractionVisualizer {
         // showOverlay resolves the moment the overlay is added, before the CSS
         // glide plays. Callers (e.g. ClickTool) await pointAt expecting the cursor
         // to reach the element *before* they act, so block for the animation here.
-        if (animMs > 0) {
+        // Only the moving case animates (see buildCursorHtml), so a no-op move
+        // (e.g. an unresolved target) shouldn't incur the wait.
+        const animating = animMs > 0 && (from.x !== to.x || from.y !== to.y);
+        if (animating) {
             await new Promise((resolve) => {
                 setTimeout(resolve, animMs);
             });
         }
     }
-    async showMessage(page, target, text, duration) {
-        // Sticky overlay + self-managed removal. The `duration` option on
-        // showOverlay only composites into the recorded screencast stream, not the
-        // live page render layer (so it would be invisible in headed/screenshots);
-        // a no-duration overlay renders everywhere, matching the cursor.
-        const handle = await page.screencast.showOverlay(this.buildMessageHtml(page, target, text));
-        const timer = setTimeout(() => {
-            handle.dispose().catch(() => {
-                // Page navigated/closed before the message expired — nothing to clean up.
-            });
-        }, duration);
-        // Don't let a pending message removal keep the process alive on shutdown.
-        timer.unref?.();
-    }
     /**
      * Builds the cursor overlay markup: the arrow SVG plus a `@keyframes` glide
      * from the previous position and a one-shot ripple at the arrow tip.
@@ -179,41 +160,6 @@ class InteractionVisualizer {
 </style>
 <div class="donobu-cursor">${InteractionVisualizer.SVG_MOUSE}</div>`;
     }
-    /**
-     * Builds the message tooltip markup, positioned near `target` and clamped
-     * inside the viewport. Without being able to measure the rendered box, we
-     * keep it on-screen with a conservative max-width clamp and anchor it above
-     * the cursor (via `translateY(-100%)`) when the cursor sits low in the page.
-     */
-    buildMessageHtml(page, target, text) {
-        const maxW = InteractionVisualizer.MESSAGE_MAX_WIDTH;
-        const margin = InteractionVisualizer.MESSAGE_MARGIN;
-        const vp = page.viewportSize() ?? { width: 1280, height: 720 };
-        // Clamp the (centered) anchor so a max-width box can't overflow either edge.
-        const centerX = Math.max(maxW / 2 + margin, Math.min(vp.width - maxW / 2 - margin, target.x));
-        const placeAbove = target.y > vp.height * 0.6;
-        const top = placeAbove ? Math.max(margin, target.y - 24) : target.y + 24;
-        const translate = placeAbove
-            ? 'translate(-50%, -100%)'
-            : 'translate(-50%, 0)';
-        const style = [
-            'position:fixed',
-            `left:${centerX}px`,
-            `top:${top}px`,
-            `transform:${translate}`,
-            `max-width:${maxW}px`,
-            'z-index:2147483647',
-            'pointer-events:none',
-            'background:#000',
-            'color:#fff',
-            'padding:8px 10px',
-            'border-radius:5px',
-            "font:12px/1 'Source Sans Pro','Helvetica Neue',Helvetica,Arial,sans-serif",
-            'white-space:pre-wrap',
-            'box-shadow:2px 2px 10px rgba(0,0,0,.2)',
-        ].join(';');
-        return `<div style="${style}">${InteractionVisualizer.escapeHtml(text)}</div>`;
-    }
 }
 exports.InteractionVisualizer = InteractionVisualizer;
 /* ------------------------------------------------------------------ */
@@ -235,7 +181,11 @@ InteractionVisualizer.SVG_MOUSE = `
      stroke-linejoin="round">
   <path d="${InteractionVisualizer.RAW_MOUSE_D}" />
 </svg>`;
-/** Message tooltip box styling — mirrors the legacy `.donobu-message` rule. */
-InteractionVisualizer.MESSAGE_MAX_WIDTH = 300;
-InteractionVisualizer.MESSAGE_MARGIN = 8;
+/**
+ * Upper bound on resolving a target's bounding box. `Locator.boundingBox`
+ * defaults to *no* timeout, so a locator that resolves to zero elements
+ * (e.g. detached mid-flow) would otherwise wait forever and hang the flow.
+ * The cursor is cosmetic, so we cap the wait and treat a miss as a no-op.
+ */
+InteractionVisualizer.BOUNDING_BOX_TIMEOUT_MILLIS = 1000;
 //# sourceMappingURL=InteractionVisualizer.js.map

package/dist/tools/ReplayableInteraction.js

@@ -432,7 +432,7 @@ class ReplayableInteraction extends Tool_1.Tool {
         // Only reveal the cursor now that we have a real target to point at, so a
         // non-interactive proposal never pops a stationary cursor.
         await (0, TargetUtils_1.webInspector)(context).showInteractionCursor();
-        await context.interactionVisualizer.pointAt(page, pointTarget.first(), undefined, ReplayableInteraction.PREVIEW_CURSOR_DURATION_MILLIS);
+        await context.interactionVisualizer.pointAt(page, pointTarget.first(), ReplayableInteraction.PREVIEW_CURSOR_DURATION_MILLIS);
     }
     async callFromGpt(context, parameters) {
         const page = (0, TargetUtils_1.webPage)(context);

package/dist/utils/MiscUtils.js

@@ -204,25 +204,21 @@ class MiscUtils {
      * Merges adjacent user messages (some LLMs like Gemini's or Anthropic's require this).
      */
     static mergeAdjacentUserMessages(messages) {
-        let updatedMessages = [];
-        for (let i = 0; i < messages.length; ++i) {
-            if (i === messages.length - 1) {
-                updatedMessages.push(messages[i]);
-                break;
-            }
-            const message = messages[i];
-            const adjacentMessage = messages[i + 1];
-            if (message.type === 'user' && adjacentMessage.type === 'user') {
-                const mergedMessage = {
+        const updatedMessages = [];
+        // Fold each user message into the previously emitted one when that is also
+        // a user message. Folding into the accumulated result (rather than
+        // consuming pairs) collapses runs of any length — pair-consuming would
+        // leave the 3rd, 5th, ... message in a run un-merged.
+        for (const message of messages) {
+            const last = updatedMessages[updatedMessages.length - 1];
+            if (message.type === 'user' && last?.type === 'user') {
+                updatedMessages[updatedMessages.length - 1] = {
                     type: 'user',
-                    items: [...message.items, ...adjacentMessage.items],
+                    items: [...last.items, ...message.items],
                 };
-                updatedMessages.push(mergedMessage);
-                // Skip the next message.
-                ++i;
             }
             else {
-                updatedMessages.push(messages[i]);
+                updatedMessages.push(message);
             }
         }
         return updatedMessages;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "donobu",
-  "version": "5.60.5",
+  "version": "5.60.7",
   "description": "Create browser automations with an LLM agent and replay them as Playwright scripts.",
   "main": "dist/main.js",
   "module": "dist/esm/main.js",