donobu 5.18.2 → 5.18.3

package/dist/esm/lib/test/testExtension.js

@@ -25,6 +25,7 @@ const BrowserUtils_1 = require("../../utils/BrowserUtils");
 const FlowLogBuffer_1 = require("../../utils/FlowLogBuffer");
 const Logger_1 = require("../../utils/Logger");
 const MiscUtils_1 = require("../../utils/MiscUtils");
+const PageLogListeners_1 = require("../../utils/PageLogListeners");
 const cacheLocator_1 = require("../ai/cache/cacheLocator");
 const extendPage_1 = require("../page/extendPage");
 const tbd_1 = require("../page/tbd");
@@ -101,6 +102,11 @@ exports.test = test_1.test.extend({
         });
         extendedPage._dnb.donobuFlowMetadata.name = getSanitizedTestName(testInfo);
         extendedPage._dnb.donobuFlowMetadata.overallObjective = overallObjective;
+        // Register browser console and network listeners so that logs from these
+        // sources are captured into the flow's logBuffer. In Studio-launched flows
+        // this is done by WebTargetInspector.initialize(), but that method is not
+        // called during test runs, so we wire the listeners up here directly.
+        (0, PageLogListeners_1.registerPageLogListeners)(page);
         // Bind the Playwright-provided `use` callback to an async resource so that
         // any microtasks scheduled inside the test body keep the flow logging
         // context. Without this, Playwright may re-use earlier async resources that

package/dist/esm/managers/WebTargetInspector.js

@@ -6,6 +6,7 @@ const SetDonobuAnnotations_1 = require("../bindings/SetDonobuAnnotations");
 const PageClosedException_1 = require("../exceptions/PageClosedException");
 const BrowserUtils_1 = require("../utils/BrowserUtils");
 const Logger_1 = require("../utils/Logger");
+const PageLogListeners_1 = require("../utils/PageLogListeners");
 const PlaywrightUtils_1 = require("../utils/PlaywrightUtils");
 const PageInspector_1 = require("./PageInspector");
 /**
@@ -185,81 +186,7 @@ The active (i.e. in focus) tab is ${this._target.current.url()}`;
     /* ------------------------------------------------------------------ */
     handleNewPage(page, callbacks) {
         this._target.current = page;
-        page.on('console', (msg) => {
-            const { url, lineNumber, columnNumber } = msg.location();
-            const hasSourceLocation = lineNumber !== 0 || columnNumber !== 0;
-            const meta = url
-                ? hasSourceLocation
-                    ? { url, lineNumber, columnNumber }
-                    : { url }
-                : undefined;
-            switch (msg.type()) {
-                case 'error':
-                    Logger_1.browserLogger.error(msg.text(), meta);
-                    break;
-                case 'warning':
-                    Logger_1.browserLogger.warn(msg.text(), meta);
-                    break;
-                case 'debug':
-                    Logger_1.browserLogger.debug(msg.text(), meta);
-                    break;
-                default:
-                    Logger_1.browserLogger.info(msg.text(), meta);
-                    break;
-            }
-        });
-        page.on('pageerror', (error) => {
-            Logger_1.browserLogger.error(error.message, {
-                stack: error.stack,
-                url: page.url(),
-            });
-        });
-        // Fallback timing: track request start times keyed by the Request object
-        // itself (via WeakMap) so concurrent requests to the same URL don't collide.
-        // Prefer Playwright's native timing API when it has data.
-        const requestStartTimes = new WeakMap();
-        page.on('request', (request) => {
-            requestStartTimes.set(request, Date.now());
-        });
-        const getDuration = (request) => {
-            const timing = request.timing();
-            if (timing.responseEnd >= 0) {
-                return Math.round(timing.responseEnd);
-            }
-            const startTime = requestStartTimes.get(request);
-            return startTime !== undefined ? Date.now() - startTime : undefined;
-        };
-        page.on('response', (response) => {
-            const request = response.request();
-            const status = response.status();
-            const duration = getDuration(request);
-            const meta = {
-                method: request.method(),
-                url: request.url(),
-                status,
-                duration,
-                resourceType: request.resourceType(),
-            };
-            if (status >= 500) {
-                Logger_1.networkLogger.error(request.url(), meta);
-            }
-            else if (status >= 400) {
-                Logger_1.networkLogger.warn(request.url(), meta);
-            }
-            else {
-                Logger_1.networkLogger.info(request.url(), meta);
-            }
-        });
-        page.on('requestfailed', (request) => {
-            const duration = getDuration(request);
-            Logger_1.networkLogger.error(`${request.url()} FAILED: ${request.failure()?.errorText}`, {
-                method: request.method(),
-                url: request.url(),
-                resourceType: request.resourceType(),
-                failureReason: request.failure()?.errorText,
-                duration,
-            });
-        });
+        (0, PageLogListeners_1.registerPageLogListeners)(page);
         if (callbacks.metadata.runMode !== 'INSTRUCT') {
             page.on('domcontentloaded', async () => {
                 await this._interactionVisualizer.showMouse(page);

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

@@ -0,0 +1,15 @@
+import type { Page } from 'playwright';
+/**
+ * Registers Playwright page event listeners that route browser console messages
+ * and network request/response data to the {@link browserLogger} and
+ * {@link networkLogger} Winston loggers. These loggers include a
+ * {@link FlowLogBufferTransport} that captures entries into the per-flow
+ * log buffer (when one is available via AsyncLocalStorage or the process-local
+ * fallback).
+ *
+ * This function is called from both {@link WebTargetInspector.handleNewPage}
+ * (Studio-launched flows) and the Playwright test extension fixture, ensuring
+ * identical logging behaviour across both runtime contexts.
+ */
+export declare function registerPageLogListeners(page: Page): void;
+//# sourceMappingURL=PageLogListeners.d.ts.map

package/dist/esm/utils/PageLogListeners.js

@@ -0,0 +1,94 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.registerPageLogListeners = registerPageLogListeners;
+const Logger_1 = require("./Logger");
+/**
+ * Registers Playwright page event listeners that route browser console messages
+ * and network request/response data to the {@link browserLogger} and
+ * {@link networkLogger} Winston loggers. These loggers include a
+ * {@link FlowLogBufferTransport} that captures entries into the per-flow
+ * log buffer (when one is available via AsyncLocalStorage or the process-local
+ * fallback).
+ *
+ * This function is called from both {@link WebTargetInspector.handleNewPage}
+ * (Studio-launched flows) and the Playwright test extension fixture, ensuring
+ * identical logging behaviour across both runtime contexts.
+ */
+function registerPageLogListeners(page) {
+    page.on('console', (msg) => {
+        const { url, lineNumber, columnNumber } = msg.location();
+        const hasSourceLocation = lineNumber !== 0 || columnNumber !== 0;
+        const meta = url
+            ? hasSourceLocation
+                ? { url, lineNumber, columnNumber }
+                : { url }
+            : undefined;
+        switch (msg.type()) {
+            case 'error':
+                Logger_1.browserLogger.error(msg.text(), meta);
+                break;
+            case 'warning':
+                Logger_1.browserLogger.warn(msg.text(), meta);
+                break;
+            case 'debug':
+                Logger_1.browserLogger.debug(msg.text(), meta);
+                break;
+            default:
+                Logger_1.browserLogger.info(msg.text(), meta);
+                break;
+        }
+    });
+    page.on('pageerror', (error) => {
+        Logger_1.browserLogger.error(error.message, {
+            stack: error.stack,
+            url: page.url(),
+        });
+    });
+    // Fallback timing: track request start times keyed by the Request object
+    // itself (via WeakMap) so concurrent requests to the same URL don't collide.
+    // Prefer Playwright's native timing API when it has data.
+    const requestStartTimes = new WeakMap();
+    page.on('request', (request) => {
+        requestStartTimes.set(request, Date.now());
+    });
+    const getDuration = (request) => {
+        const timing = request.timing();
+        if (timing.responseEnd >= 0) {
+            return Math.round(timing.responseEnd);
+        }
+        const startTime = requestStartTimes.get(request);
+        return startTime !== undefined ? Date.now() - startTime : undefined;
+    };
+    page.on('response', (response) => {
+        const request = response.request();
+        const status = response.status();
+        const duration = getDuration(request);
+        const meta = {
+            method: request.method(),
+            url: request.url(),
+            status,
+            duration,
+            resourceType: request.resourceType(),
+        };
+        if (status >= 500) {
+            Logger_1.networkLogger.error(request.url(), meta);
+        }
+        else if (status >= 400) {
+            Logger_1.networkLogger.warn(request.url(), meta);
+        }
+        else {
+            Logger_1.networkLogger.info(request.url(), meta);
+        }
+    });
+    page.on('requestfailed', (request) => {
+        const duration = getDuration(request);
+        Logger_1.networkLogger.error(`${request.url()} FAILED: ${request.failure()?.errorText}`, {
+            method: request.method(),
+            url: request.url(),
+            resourceType: request.resourceType(),
+            failureReason: request.failure()?.errorText,
+            duration,
+        });
+    });
+}
+//# sourceMappingURL=PageLogListeners.js.map

package/dist/lib/test/testExtension.js

@@ -25,6 +25,7 @@ const BrowserUtils_1 = require("../../utils/BrowserUtils");
 const FlowLogBuffer_1 = require("../../utils/FlowLogBuffer");
 const Logger_1 = require("../../utils/Logger");
 const MiscUtils_1 = require("../../utils/MiscUtils");
+const PageLogListeners_1 = require("../../utils/PageLogListeners");
 const cacheLocator_1 = require("../ai/cache/cacheLocator");
 const extendPage_1 = require("../page/extendPage");
 const tbd_1 = require("../page/tbd");
@@ -101,6 +102,11 @@ exports.test = test_1.test.extend({
         });
         extendedPage._dnb.donobuFlowMetadata.name = getSanitizedTestName(testInfo);
         extendedPage._dnb.donobuFlowMetadata.overallObjective = overallObjective;
+        // Register browser console and network listeners so that logs from these
+        // sources are captured into the flow's logBuffer. In Studio-launched flows
+        // this is done by WebTargetInspector.initialize(), but that method is not
+        // called during test runs, so we wire the listeners up here directly.
+        (0, PageLogListeners_1.registerPageLogListeners)(page);
         // Bind the Playwright-provided `use` callback to an async resource so that
         // any microtasks scheduled inside the test body keep the flow logging
         // context. Without this, Playwright may re-use earlier async resources that

package/dist/managers/WebTargetInspector.js

@@ -6,6 +6,7 @@ const SetDonobuAnnotations_1 = require("../bindings/SetDonobuAnnotations");
 const PageClosedException_1 = require("../exceptions/PageClosedException");
 const BrowserUtils_1 = require("../utils/BrowserUtils");
 const Logger_1 = require("../utils/Logger");
+const PageLogListeners_1 = require("../utils/PageLogListeners");
 const PlaywrightUtils_1 = require("../utils/PlaywrightUtils");
 const PageInspector_1 = require("./PageInspector");
 /**
@@ -185,81 +186,7 @@ The active (i.e. in focus) tab is ${this._target.current.url()}`;
     /* ------------------------------------------------------------------ */
     handleNewPage(page, callbacks) {
         this._target.current = page;
-        page.on('console', (msg) => {
-            const { url, lineNumber, columnNumber } = msg.location();
-            const hasSourceLocation = lineNumber !== 0 || columnNumber !== 0;
-            const meta = url
-                ? hasSourceLocation
-                    ? { url, lineNumber, columnNumber }
-                    : { url }
-                : undefined;
-            switch (msg.type()) {
-                case 'error':
-                    Logger_1.browserLogger.error(msg.text(), meta);
-                    break;
-                case 'warning':
-                    Logger_1.browserLogger.warn(msg.text(), meta);
-                    break;
-                case 'debug':
-                    Logger_1.browserLogger.debug(msg.text(), meta);
-                    break;
-                default:
-                    Logger_1.browserLogger.info(msg.text(), meta);
-                    break;
-            }
-        });
-        page.on('pageerror', (error) => {
-            Logger_1.browserLogger.error(error.message, {
-                stack: error.stack,
-                url: page.url(),
-            });
-        });
-        // Fallback timing: track request start times keyed by the Request object
-        // itself (via WeakMap) so concurrent requests to the same URL don't collide.
-        // Prefer Playwright's native timing API when it has data.
-        const requestStartTimes = new WeakMap();
-        page.on('request', (request) => {
-            requestStartTimes.set(request, Date.now());
-        });
-        const getDuration = (request) => {
-            const timing = request.timing();
-            if (timing.responseEnd >= 0) {
-                return Math.round(timing.responseEnd);
-            }
-            const startTime = requestStartTimes.get(request);
-            return startTime !== undefined ? Date.now() - startTime : undefined;
-        };
-        page.on('response', (response) => {
-            const request = response.request();
-            const status = response.status();
-            const duration = getDuration(request);
-            const meta = {
-                method: request.method(),
-                url: request.url(),
-                status,
-                duration,
-                resourceType: request.resourceType(),
-            };
-            if (status >= 500) {
-                Logger_1.networkLogger.error(request.url(), meta);
-            }
-            else if (status >= 400) {
-                Logger_1.networkLogger.warn(request.url(), meta);
-            }
-            else {
-                Logger_1.networkLogger.info(request.url(), meta);
-            }
-        });
-        page.on('requestfailed', (request) => {
-            const duration = getDuration(request);
-            Logger_1.networkLogger.error(`${request.url()} FAILED: ${request.failure()?.errorText}`, {
-                method: request.method(),
-                url: request.url(),
-                resourceType: request.resourceType(),
-                failureReason: request.failure()?.errorText,
-                duration,
-            });
-        });
+        (0, PageLogListeners_1.registerPageLogListeners)(page);
         if (callbacks.metadata.runMode !== 'INSTRUCT') {
             page.on('domcontentloaded', async () => {
                 await this._interactionVisualizer.showMouse(page);

package/dist/utils/PageLogListeners.d.ts

@@ -0,0 +1,15 @@
+import type { Page } from 'playwright';
+/**
+ * Registers Playwright page event listeners that route browser console messages
+ * and network request/response data to the {@link browserLogger} and
+ * {@link networkLogger} Winston loggers. These loggers include a
+ * {@link FlowLogBufferTransport} that captures entries into the per-flow
+ * log buffer (when one is available via AsyncLocalStorage or the process-local
+ * fallback).
+ *
+ * This function is called from both {@link WebTargetInspector.handleNewPage}
+ * (Studio-launched flows) and the Playwright test extension fixture, ensuring
+ * identical logging behaviour across both runtime contexts.
+ */
+export declare function registerPageLogListeners(page: Page): void;
+//# sourceMappingURL=PageLogListeners.d.ts.map

package/dist/utils/PageLogListeners.js

@@ -0,0 +1,94 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.registerPageLogListeners = registerPageLogListeners;
+const Logger_1 = require("./Logger");
+/**
+ * Registers Playwright page event listeners that route browser console messages
+ * and network request/response data to the {@link browserLogger} and
+ * {@link networkLogger} Winston loggers. These loggers include a
+ * {@link FlowLogBufferTransport} that captures entries into the per-flow
+ * log buffer (when one is available via AsyncLocalStorage or the process-local
+ * fallback).
+ *
+ * This function is called from both {@link WebTargetInspector.handleNewPage}
+ * (Studio-launched flows) and the Playwright test extension fixture, ensuring
+ * identical logging behaviour across both runtime contexts.
+ */
+function registerPageLogListeners(page) {
+    page.on('console', (msg) => {
+        const { url, lineNumber, columnNumber } = msg.location();
+        const hasSourceLocation = lineNumber !== 0 || columnNumber !== 0;
+        const meta = url
+            ? hasSourceLocation
+                ? { url, lineNumber, columnNumber }
+                : { url }
+            : undefined;
+        switch (msg.type()) {
+            case 'error':
+                Logger_1.browserLogger.error(msg.text(), meta);
+                break;
+            case 'warning':
+                Logger_1.browserLogger.warn(msg.text(), meta);
+                break;
+            case 'debug':
+                Logger_1.browserLogger.debug(msg.text(), meta);
+                break;
+            default:
+                Logger_1.browserLogger.info(msg.text(), meta);
+                break;
+        }
+    });
+    page.on('pageerror', (error) => {
+        Logger_1.browserLogger.error(error.message, {
+            stack: error.stack,
+            url: page.url(),
+        });
+    });
+    // Fallback timing: track request start times keyed by the Request object
+    // itself (via WeakMap) so concurrent requests to the same URL don't collide.
+    // Prefer Playwright's native timing API when it has data.
+    const requestStartTimes = new WeakMap();
+    page.on('request', (request) => {
+        requestStartTimes.set(request, Date.now());
+    });
+    const getDuration = (request) => {
+        const timing = request.timing();
+        if (timing.responseEnd >= 0) {
+            return Math.round(timing.responseEnd);
+        }
+        const startTime = requestStartTimes.get(request);
+        return startTime !== undefined ? Date.now() - startTime : undefined;
+    };
+    page.on('response', (response) => {
+        const request = response.request();
+        const status = response.status();
+        const duration = getDuration(request);
+        const meta = {
+            method: request.method(),
+            url: request.url(),
+            status,
+            duration,
+            resourceType: request.resourceType(),
+        };
+        if (status >= 500) {
+            Logger_1.networkLogger.error(request.url(), meta);
+        }
+        else if (status >= 400) {
+            Logger_1.networkLogger.warn(request.url(), meta);
+        }
+        else {
+            Logger_1.networkLogger.info(request.url(), meta);
+        }
+    });
+    page.on('requestfailed', (request) => {
+        const duration = getDuration(request);
+        Logger_1.networkLogger.error(`${request.url()} FAILED: ${request.failure()?.errorText}`, {
+            method: request.method(),
+            url: request.url(),
+            resourceType: request.resourceType(),
+            failureReason: request.failure()?.errorText,
+            duration,
+        });
+    });
+}
+//# sourceMappingURL=PageLogListeners.js.map

package/package.json

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