donobu 5.51.0 → 5.52.1

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

@@ -602,33 +602,13 @@ Use this information to return an appropriate JSON object.`,
                 }
             }
         }
+        const originalGoto = (0, originalGotoRegistry_1.getOriginalGoto)(page);
+        // The navigation itself is the only outcome the caller cares about. A
+        // throw here is a genuine navigation error (bad URL, timeout, net::ERR_*),
+        // so we record a best-effort failed tool call and re-throw to the caller.
+        let resp;
         try {
-            const originalGoto = (0, originalGotoRegistry_1.getOriginalGoto)(page);
-            const resp = await originalGoto.call(page, url, options);
-            const pageTitle = await page.title();
-            const postCallImage = await PlaywrightUtils_1.PlaywrightUtils.takeViewportScreenshot(page);
-            const postCallImageId = await sharedState.persistence.saveScreenShot(flowId, postCallImage);
-            const completedAt = new Date().getTime();
-            await sharedState.persistence.setToolCall(flowId, {
-                id: MiscUtils_1.MiscUtils.createAdHocToolCallId(),
-                toolName: GoToWebpageTool_1.GoToWebpageTool.NAME,
-                parameters: {
-                    url: effectiveUrl,
-                },
-                outcome: {
-                    isSuccessful: true,
-                    forLlm: `Successfully navigated to ${effectiveUrl}`,
-                    metadata: {
-                        pageTitle: pageTitle,
-                        resolvedUrl: page.url(),
-                    },
-                },
-                postCallImageId: postCallImageId,
-                page: effectiveUrl,
-                startedAt: startedAt,
-                completedAt: completedAt,
-            });
-            return resp;
+            resp = await originalGoto.call(page, url, options);
         }
         catch (error) {
             // Best-effort screenshot and tool-call recording - if the page is gone
@@ -659,6 +639,55 @@ Use this information to return an appropriate JSON object.`,
             }
             throw error;
         }
+        // Navigation succeeded. Everything below is best-effort Donobu telemetry
+        // (page title, screenshot, persisted tool call) and must never fail the
+        // caller's page.goto(). A SPA that self-reloads right after the `load`
+        // event - e.g. a feature-flag bootstrap - tears down the execution
+        // context, so post-navigation calls like page.title() can throw
+        // "Execution context was destroyed, most likely because of a navigation".
+        // That is normal app behavior, not a navigation failure.
+        try {
+            // page.title() evaluates in the page context, so it is the most likely
+            // casualty of a mid-flight reload. Capture it defensively (the CDP-based
+            // screenshot can still succeed via the compositor) so we can record a
+            // useful tool call even when the title is momentarily unavailable.
+            let pageTitle = '';
+            try {
+                pageTitle = await page.title();
+            }
+            catch (titleError) {
+                if (!PlaywrightUtils_1.PlaywrightUtils.isPageClosedError(titleError)) {
+                    throw titleError;
+                }
+                Logger_1.appLogger.debug('page.title() failed after navigation (execution context destroyed by an in-flight reload); recording goto with an empty title.');
+            }
+            const postCallImage = await PlaywrightUtils_1.PlaywrightUtils.takeViewportScreenshot(page);
+            const postCallImageId = await sharedState.persistence.saveScreenShot(flowId, postCallImage);
+            const completedAt = new Date().getTime();
+            await sharedState.persistence.setToolCall(flowId, {
+                id: MiscUtils_1.MiscUtils.createAdHocToolCallId(),
+                toolName: GoToWebpageTool_1.GoToWebpageTool.NAME,
+                parameters: {
+                    url: effectiveUrl,
+                },
+                outcome: {
+                    isSuccessful: true,
+                    forLlm: `Successfully navigated to ${effectiveUrl}`,
+                    metadata: {
+                        pageTitle: pageTitle,
+                        resolvedUrl: page.url(),
+                    },
+                },
+                postCallImageId: postCallImageId,
+                page: effectiveUrl,
+                startedAt: startedAt,
+                completedAt: completedAt,
+            });
+        }
+        catch (telemetryError) {
+            Logger_1.appLogger.warn('Navigation succeeded but Donobu post-goto telemetry failed; this does not affect the navigation:', telemetryError);
+        }
+        return resp;
     };
     page.run = async (toolName, toolParams, options) => {
         // Thin wrapper to run a Donobu tool and throw on failure so tests can await it directly.

package/dist/esm/managers/DonobuFlow.js

@@ -145,7 +145,10 @@ class DonobuFlow {
     async run() {
         while (true) {
             try {
-                this.controlPanel.update({ state: this.metadata.state });
+                this.controlPanel.update({
+                    state: this.metadata.state,
+                    availableToolNames: this.toolManager.tools.map((t) => t.name),
+                });
                 switch (this.metadata.state) {
                     case 'UNSTARTED':
                         await this.onUnstarted();
@@ -339,6 +342,14 @@ class DonobuFlow {
                 });
                 this.metadata.state = 'RUNNING_ACTION';
                 break;
+            case 'RUN_TOOL':
+                this.proposedToolCalls.length = 0;
+                this.proposedToolCalls.push({
+                    name: userAction.toolName,
+                    parameters: userAction.parameters,
+                });
+                this.metadata.state = 'RUNNING_ACTION';
+                break;
         }
         await this.persistence.setFlowMetadata(this.metadata);
     }

package/dist/esm/models/ControlPanel.d.ts

@@ -6,10 +6,17 @@ export type UserAction = {
     userInstruction?: string;
 } | {
     type: 'END';
+} | {
+    type: 'RUN_TOOL';
+    toolName: string;
+    parameters: Record<string, unknown>;
 };
 export type ControlPanelDataUpdate = {
     state: State;
     headline?: string;
+    /** Names of tools loaded in the flow's ToolManager. Surfaced to the UI so
+     * the control panel can offer only tools the flow can actually run. */
+    availableToolNames?: string[];
 };
 export interface ControlPanel {
     /** Cheap, idempotent render update. */

package/dist/lib/page/extendPage.js

@@ -602,33 +602,13 @@ Use this information to return an appropriate JSON object.`,
                 }
             }
         }
+        const originalGoto = (0, originalGotoRegistry_1.getOriginalGoto)(page);
+        // The navigation itself is the only outcome the caller cares about. A
+        // throw here is a genuine navigation error (bad URL, timeout, net::ERR_*),
+        // so we record a best-effort failed tool call and re-throw to the caller.
+        let resp;
         try {
-            const originalGoto = (0, originalGotoRegistry_1.getOriginalGoto)(page);
-            const resp = await originalGoto.call(page, url, options);
-            const pageTitle = await page.title();
-            const postCallImage = await PlaywrightUtils_1.PlaywrightUtils.takeViewportScreenshot(page);
-            const postCallImageId = await sharedState.persistence.saveScreenShot(flowId, postCallImage);
-            const completedAt = new Date().getTime();
-            await sharedState.persistence.setToolCall(flowId, {
-                id: MiscUtils_1.MiscUtils.createAdHocToolCallId(),
-                toolName: GoToWebpageTool_1.GoToWebpageTool.NAME,
-                parameters: {
-                    url: effectiveUrl,
-                },
-                outcome: {
-                    isSuccessful: true,
-                    forLlm: `Successfully navigated to ${effectiveUrl}`,
-                    metadata: {
-                        pageTitle: pageTitle,
-                        resolvedUrl: page.url(),
-                    },
-                },
-                postCallImageId: postCallImageId,
-                page: effectiveUrl,
-                startedAt: startedAt,
-                completedAt: completedAt,
-            });
-            return resp;
+            resp = await originalGoto.call(page, url, options);
         }
         catch (error) {
             // Best-effort screenshot and tool-call recording - if the page is gone
@@ -659,6 +639,55 @@ Use this information to return an appropriate JSON object.`,
             }
             throw error;
         }
+        // Navigation succeeded. Everything below is best-effort Donobu telemetry
+        // (page title, screenshot, persisted tool call) and must never fail the
+        // caller's page.goto(). A SPA that self-reloads right after the `load`
+        // event - e.g. a feature-flag bootstrap - tears down the execution
+        // context, so post-navigation calls like page.title() can throw
+        // "Execution context was destroyed, most likely because of a navigation".
+        // That is normal app behavior, not a navigation failure.
+        try {
+            // page.title() evaluates in the page context, so it is the most likely
+            // casualty of a mid-flight reload. Capture it defensively (the CDP-based
+            // screenshot can still succeed via the compositor) so we can record a
+            // useful tool call even when the title is momentarily unavailable.
+            let pageTitle = '';
+            try {
+                pageTitle = await page.title();
+            }
+            catch (titleError) {
+                if (!PlaywrightUtils_1.PlaywrightUtils.isPageClosedError(titleError)) {
+                    throw titleError;
+                }
+                Logger_1.appLogger.debug('page.title() failed after navigation (execution context destroyed by an in-flight reload); recording goto with an empty title.');
+            }
+            const postCallImage = await PlaywrightUtils_1.PlaywrightUtils.takeViewportScreenshot(page);
+            const postCallImageId = await sharedState.persistence.saveScreenShot(flowId, postCallImage);
+            const completedAt = new Date().getTime();
+            await sharedState.persistence.setToolCall(flowId, {
+                id: MiscUtils_1.MiscUtils.createAdHocToolCallId(),
+                toolName: GoToWebpageTool_1.GoToWebpageTool.NAME,
+                parameters: {
+                    url: effectiveUrl,
+                },
+                outcome: {
+                    isSuccessful: true,
+                    forLlm: `Successfully navigated to ${effectiveUrl}`,
+                    metadata: {
+                        pageTitle: pageTitle,
+                        resolvedUrl: page.url(),
+                    },
+                },
+                postCallImageId: postCallImageId,
+                page: effectiveUrl,
+                startedAt: startedAt,
+                completedAt: completedAt,
+            });
+        }
+        catch (telemetryError) {
+            Logger_1.appLogger.warn('Navigation succeeded but Donobu post-goto telemetry failed; this does not affect the navigation:', telemetryError);
+        }
+        return resp;
     };
     page.run = async (toolName, toolParams, options) => {
         // Thin wrapper to run a Donobu tool and throw on failure so tests can await it directly.

package/dist/managers/DonobuFlow.js

@@ -145,7 +145,10 @@ class DonobuFlow {
     async run() {
         while (true) {
             try {
-                this.controlPanel.update({ state: this.metadata.state });
+                this.controlPanel.update({
+                    state: this.metadata.state,
+                    availableToolNames: this.toolManager.tools.map((t) => t.name),
+                });
                 switch (this.metadata.state) {
                     case 'UNSTARTED':
                         await this.onUnstarted();
@@ -339,6 +342,14 @@ class DonobuFlow {
                 });
                 this.metadata.state = 'RUNNING_ACTION';
                 break;
+            case 'RUN_TOOL':
+                this.proposedToolCalls.length = 0;
+                this.proposedToolCalls.push({
+                    name: userAction.toolName,
+                    parameters: userAction.parameters,
+                });
+                this.metadata.state = 'RUNNING_ACTION';
+                break;
         }
         await this.persistence.setFlowMetadata(this.metadata);
     }

package/dist/models/ControlPanel.d.ts

@@ -6,10 +6,17 @@ export type UserAction = {
     userInstruction?: string;
 } | {
     type: 'END';
+} | {
+    type: 'RUN_TOOL';
+    toolName: string;
+    parameters: Record<string, unknown>;
 };
 export type ControlPanelDataUpdate = {
     state: State;
     headline?: string;
+    /** Names of tools loaded in the flow's ToolManager. Surfaced to the UI so
+     * the control panel can offer only tools the flow can actually run. */
+    availableToolNames?: string[];
 };
 export interface ControlPanel {
     /** Cheap, idempotent render update. */

package/package.json

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