donobu 5.21.0 → 5.21.2

package/dist/esm/tools/AssertTool.js

@@ -10,7 +10,7 @@ const MiscUtils_1 = require("../utils/MiscUtils");
 const PlaywrightUtils_1 = require("../utils/PlaywrightUtils");
 const TargetUtils_1 = require("../utils/TargetUtils");
 const Tool_1 = require("./Tool");
-const DEFAULT_RETRIES = 0;
+const DEFAULT_RETRIES = 1;
 const DEFAULT_RETRY_WAIT_SECONDS = 3;
 exports.AssertCoreSchema = v4_1.z.object({
     assertionToTestFor: v4_1.z.string()
@@ -107,8 +107,19 @@ It will use a screenshot of the current viewport of the webpage, the webpage's t
                 await page.waitForTimeout(retryWaitSeconds * 1000);
             }
             const screenshot = await PlaywrightUtils_1.PlaywrightUtils.takeViewportScreenshot(page);
-            const webpageRawText = await page.locator('body').innerText({
-                timeout: 5000,
+            const webpageTextTimeoutMs = 5000;
+            const webpageRawText = await page
+                .locator('body')
+                .innerText({ timeout: webpageTextTimeoutMs })
+                .catch((error) => {
+                const msg = `[${MiscUtils_1.MiscUtils.errName(error)}: Failed to extract page text within ${webpageTextTimeoutMs}ms. The page may still be loading or the main thread may be blocked.]`;
+                Logger_1.appLogger.warn(msg);
+                return msg;
+            });
+            const webpageTitle = await page.title().catch((error) => {
+                const msg = `[${MiscUtils_1.MiscUtils.errName(error)}: Failed to retrieve page title]`;
+                Logger_1.appLogger.warn(msg);
+                return msg;
             });
             const promptMessages = [
                 {
@@ -140,7 +151,7 @@ CRITICAL RULES for generating structured steps — follow these precisely:
 
 The current date and time in ISO 8601 format is ${new Date().toISOString()}.
 The current webpage URL is: ${page.url()}
-The current webpage title is: ${await page.title()}
+The current webpage title is: ${webpageTitle}
 The raw text of the current webpage is:
 \`\`\`
 ${webpageRawText}

package/dist/esm/utils/Logger.js

@@ -21,7 +21,7 @@ const MAX_FILES = 2;
  * Formats an error object into a detailed string representation
  */
 function formatError(error) {
-    return `${error.constructor.name}: ${error.message}\n${error.stack || ''}`;
+    return `${MiscUtils_1.MiscUtils.errName(error)}: ${error.message}\n${error.stack || ''}`;
 }
 /**
  * Winston stores extra positional args (e.g., logger.error('msg', err, meta))

package/dist/esm/utils/MiscUtils.d.ts

@@ -3,6 +3,7 @@ import type { GptMessage } from '../models/GptMessage';
 export declare class MiscUtils {
     private constructor();
     static readonly DONOBU_VERSION: string;
+    static errName(error: unknown): string;
     static capitalize(s: string): string;
     /**
      * Force a true/false/undefined value for a given value. This is a stripped down

package/dist/esm/utils/MiscUtils.js

@@ -12,6 +12,16 @@ const v4_1 = __importDefault(require("zod/v4"));
 const envVars_1 = require("../envVars");
 class MiscUtils {
     constructor() { }
+    static errName(error) {
+        if (error instanceof Error) {
+            return error.constructor.name !== 'Error'
+                ? error.constructor.name
+                : error.name;
+        }
+        else {
+            return typeof error;
+        }
+    }
     static capitalize(s) {
         return s.charAt(0).toUpperCase() + s.slice(1);
     }

package/dist/tools/AssertTool.js

@@ -10,7 +10,7 @@ const MiscUtils_1 = require("../utils/MiscUtils");
 const PlaywrightUtils_1 = require("../utils/PlaywrightUtils");
 const TargetUtils_1 = require("../utils/TargetUtils");
 const Tool_1 = require("./Tool");
-const DEFAULT_RETRIES = 0;
+const DEFAULT_RETRIES = 1;
 const DEFAULT_RETRY_WAIT_SECONDS = 3;
 exports.AssertCoreSchema = v4_1.z.object({
     assertionToTestFor: v4_1.z.string()
@@ -107,8 +107,19 @@ It will use a screenshot of the current viewport of the webpage, the webpage's t
                 await page.waitForTimeout(retryWaitSeconds * 1000);
             }
             const screenshot = await PlaywrightUtils_1.PlaywrightUtils.takeViewportScreenshot(page);
-            const webpageRawText = await page.locator('body').innerText({
-                timeout: 5000,
+            const webpageTextTimeoutMs = 5000;
+            const webpageRawText = await page
+                .locator('body')
+                .innerText({ timeout: webpageTextTimeoutMs })
+                .catch((error) => {
+                const msg = `[${MiscUtils_1.MiscUtils.errName(error)}: Failed to extract page text within ${webpageTextTimeoutMs}ms. The page may still be loading or the main thread may be blocked.]`;
+                Logger_1.appLogger.warn(msg);
+                return msg;
+            });
+            const webpageTitle = await page.title().catch((error) => {
+                const msg = `[${MiscUtils_1.MiscUtils.errName(error)}: Failed to retrieve page title]`;
+                Logger_1.appLogger.warn(msg);
+                return msg;
             });
             const promptMessages = [
                 {
@@ -140,7 +151,7 @@ CRITICAL RULES for generating structured steps — follow these precisely:
 
 The current date and time in ISO 8601 format is ${new Date().toISOString()}.
 The current webpage URL is: ${page.url()}
-The current webpage title is: ${await page.title()}
+The current webpage title is: ${webpageTitle}
 The raw text of the current webpage is:
 \`\`\`
 ${webpageRawText}

package/dist/utils/Logger.js

@@ -21,7 +21,7 @@ const MAX_FILES = 2;
  * Formats an error object into a detailed string representation
  */
 function formatError(error) {
-    return `${error.constructor.name}: ${error.message}\n${error.stack || ''}`;
+    return `${MiscUtils_1.MiscUtils.errName(error)}: ${error.message}\n${error.stack || ''}`;
 }
 /**
  * Winston stores extra positional args (e.g., logger.error('msg', err, meta))

package/dist/utils/MiscUtils.d.ts

@@ -3,6 +3,7 @@ import type { GptMessage } from '../models/GptMessage';
 export declare class MiscUtils {
     private constructor();
     static readonly DONOBU_VERSION: string;
+    static errName(error: unknown): string;
     static capitalize(s: string): string;
     /**
      * Force a true/false/undefined value for a given value. This is a stripped down

package/dist/utils/MiscUtils.js

@@ -12,6 +12,16 @@ const v4_1 = __importDefault(require("zod/v4"));
 const envVars_1 = require("../envVars");
 class MiscUtils {
     constructor() { }
+    static errName(error) {
+        if (error instanceof Error) {
+            return error.constructor.name !== 'Error'
+                ? error.constructor.name
+                : error.name;
+        }
+        else {
+            return typeof error;
+        }
+    }
     static capitalize(s) {
         return s.charAt(0).toUpperCase() + s.slice(1);
     }

package/package.json

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