donobu 5.18.1 → 5.18.3

package/dist/esm/lib/test/testExtension.d.ts

@@ -1,10 +1,12 @@
 import type { GptClient } from '../../clients/GptClient';
 import type { BrowserStorageState } from '../../models/BrowserStorageState';
+import { FlowLogBuffer } from '../../utils/FlowLogBuffer';
 import type { DonobuExtendedPage } from '../page/DonobuExtendedPage';
 export * from 'playwright/test';
 export declare const test: import("@playwright/test").TestType<import("@playwright/test").PlaywrightTestArgs & import("@playwright/test").PlaywrightTestOptions & {
     flowLoggingContext: {
         flowId: string;
+        logBuffer: FlowLogBuffer;
     };
     storageState?: BrowserStorageState | Promise<BrowserStorageState>;
     gptClient?: GptClient;

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

@@ -22,8 +22,10 @@ const v4_1 = require("zod/v4");
 const envVars_1 = require("../../envVars");
 const DonobuFlowsManager_1 = require("../../managers/DonobuFlowsManager");
 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");
@@ -55,16 +57,19 @@ exports.test = test_1.test.extend({
             // Donobu flow ID, so metadata would overwrite and concurrent executions could
             // clobber each other.
             const flowId = (0, crypto_1.randomUUID)();
-            const flowContext = { flowId };
+            const logBuffer = new FlowLogBuffer_1.FlowLogBuffer();
+            const flowContext = { flowId, logBuffer };
             const asyncScope = new async_hooks_1.AsyncResource('DonobuFlowContext');
             await Logger_1.loggingContext.run(flowContext, async () => {
                 Logger_1.loggingContext.enterWith(flowContext);
                 (0, Logger_1.setProcessLocalFlowId)(flowId);
+                (0, Logger_1.setProcessLocalLogBuffer)(logBuffer);
                 try {
-                    await asyncScope.runInAsyncScope(() => use({ flowId }));
+                    await asyncScope.runInAsyncScope(() => use({ flowId, logBuffer }));
                 }
                 finally {
                     (0, Logger_1.setProcessLocalFlowId)(null);
+                    (0, Logger_1.setProcessLocalLogBuffer)(null);
                 }
             });
         },
@@ -84,7 +89,7 @@ exports.test = test_1.test.extend({
             .optional()
             .default(250)
             .parse(testInfo.project.metadata['visualCueDurationMs']);
-        const { flowId } = flowLoggingContext;
+        const { flowId, logBuffer } = flowLoggingContext;
         const extendedPage = await (0, extendPage_1.extendPage)(page, {
             flowId: flowId,
             visualCueDurationMs: visualCueDurationMs,
@@ -97,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
@@ -113,7 +123,7 @@ exports.test = test_1.test.extend({
             throw error;
         }
         finally {
-            await finalizeTest(extendedPage, testInfo);
+            await finalizeTest(extendedPage, testInfo, logBuffer);
         }
     },
 });
@@ -224,7 +234,7 @@ async function attachStepScreenshots(sharedState, testInfo) {
         contentType: 'application/json',
     });
 }
-async function finalizeTest(page, testInfo) {
+async function finalizeTest(page, testInfo, logBuffer) {
     const sharedState = page._dnb;
     try {
         sharedState.donobuFlowMetadata.state =
@@ -237,6 +247,17 @@ async function finalizeTest(page, testInfo) {
             body: JSON.stringify(sharedState.donobuFlowMetadata, null, 2),
             contentType: 'application/json',
         });
+        // Persist captured flow logs so they are available in the Donobu UI,
+        // mirroring what DonobuFlowsManager does for Studio-launched flows.
+        if (logBuffer) {
+            try {
+                const snapshot = logBuffer.snapshot();
+                await sharedState.persistence.setFlowFile(sharedState.donobuFlowMetadata.id, 'logs.json', Buffer.from(JSON.stringify(snapshot)));
+            }
+            catch (error) {
+                Logger_1.appLogger.error('Failed to persist flow logs:', error);
+            }
+        }
         // Attach step-level screenshots from the flow's tool call history.
         // These enable the HTML report to show a visual timeline of what the
         // AI agent saw at each step.

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/Logger.d.ts

@@ -1,5 +1,6 @@
 import { AsyncLocalStorage } from 'async_hooks';
 import winston from 'winston';
+import type { FlowLogBuffer } from '../utils/FlowLogBuffer';
 export declare const loggingContext: AsyncLocalStorage<unknown>;
 /**
  * ###################################################################################
@@ -12,11 +13,12 @@ export declare const loggingContext: AsyncLocalStorage<unknown>;
  * outside the ALS store even though we seed it later in the fixture.
  *
  * We therefore keep a *process-local* fallback that stores the most recent flow
- * ID for the worker. Because Playwright guarantees only one test runs at a time
- * per process, this is safe: there is no race between concurrent tests in the
- * same worker, yet we still avoid leaking IDs across workers.
+ * ID and log buffer for the worker. Because Playwright guarantees only one test
+ * runs at a time per process, this is safe: there is no race between concurrent
+ * tests in the same worker, yet we still avoid leaking IDs across workers.
  */
 export declare function setProcessLocalFlowId(flowId: string | null): void;
+export declare function setProcessLocalLogBuffer(buffer: FlowLogBuffer | null): void;
 export declare const appLogger: winston.Logger;
 export declare const accessLogger: winston.Logger;
 export declare const browserLogger: winston.Logger;

package/dist/esm/utils/Logger.js

@@ -5,6 +5,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.networkLogger = exports.browserLogger = exports.accessLogger = exports.appLogger = exports.loggingContext = void 0;
 exports.setProcessLocalFlowId = setProcessLocalFlowId;
+exports.setProcessLocalLogBuffer = setProcessLocalLogBuffer;
 exports.logErrorWithoutStack = logErrorWithoutStack;
 exports.formatLogInfoForTest = formatLogInfoForTest;
 const async_hooks_1 = require("async_hooks");
@@ -142,13 +143,15 @@ class FlowLogBufferTransport extends winston_transport_1.default {
     }
     log(info, callback) {
         const store = exports.loggingContext.getStore();
-        if (store?.logBuffer) {
-            store.logBuffer.push({ ...info, source: this.source });
+        const buffer = store?.logBuffer || processLocalLogBuffer;
+        if (buffer) {
+            buffer.push({ ...info, source: this.source });
         }
         callback();
     }
 }
 let processLocalFlowId = null;
+let processLocalLogBuffer = null;
 /**
  * ###################################################################################
  * # WARNING! Only use this function within the context of Playwright test fixtures! #
@@ -160,13 +163,16 @@ let processLocalFlowId = null;
  * outside the ALS store even though we seed it later in the fixture.
  *
  * We therefore keep a *process-local* fallback that stores the most recent flow
- * ID for the worker. Because Playwright guarantees only one test runs at a time
- * per process, this is safe: there is no race between concurrent tests in the
- * same worker, yet we still avoid leaking IDs across workers.
+ * ID and log buffer for the worker. Because Playwright guarantees only one test
+ * runs at a time per process, this is safe: there is no race between concurrent
+ * tests in the same worker, yet we still avoid leaking IDs across workers.
  */
 function setProcessLocalFlowId(flowId) {
     processLocalFlowId = flowId;
 }
+function setProcessLocalLogBuffer(buffer) {
+    processLocalLogBuffer = buffer;
+}
 // Format to add the currently running flow's ID (if any) from AsyncLocalStorage
 const flowIdFormat = winston_1.default.format((info) => {
     const store = exports.loggingContext.getStore();

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.d.ts

@@ -1,10 +1,12 @@
 import type { GptClient } from '../../clients/GptClient';
 import type { BrowserStorageState } from '../../models/BrowserStorageState';
+import { FlowLogBuffer } from '../../utils/FlowLogBuffer';
 import type { DonobuExtendedPage } from '../page/DonobuExtendedPage';
 export * from 'playwright/test';
 export declare const test: import("@playwright/test").TestType<import("@playwright/test").PlaywrightTestArgs & import("@playwright/test").PlaywrightTestOptions & {
     flowLoggingContext: {
         flowId: string;
+        logBuffer: FlowLogBuffer;
     };
     storageState?: BrowserStorageState | Promise<BrowserStorageState>;
     gptClient?: GptClient;

package/dist/lib/test/testExtension.js

@@ -22,8 +22,10 @@ const v4_1 = require("zod/v4");
 const envVars_1 = require("../../envVars");
 const DonobuFlowsManager_1 = require("../../managers/DonobuFlowsManager");
 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");
@@ -55,16 +57,19 @@ exports.test = test_1.test.extend({
             // Donobu flow ID, so metadata would overwrite and concurrent executions could
             // clobber each other.
             const flowId = (0, crypto_1.randomUUID)();
-            const flowContext = { flowId };
+            const logBuffer = new FlowLogBuffer_1.FlowLogBuffer();
+            const flowContext = { flowId, logBuffer };
             const asyncScope = new async_hooks_1.AsyncResource('DonobuFlowContext');
             await Logger_1.loggingContext.run(flowContext, async () => {
                 Logger_1.loggingContext.enterWith(flowContext);
                 (0, Logger_1.setProcessLocalFlowId)(flowId);
+                (0, Logger_1.setProcessLocalLogBuffer)(logBuffer);
                 try {
-                    await asyncScope.runInAsyncScope(() => use({ flowId }));
+                    await asyncScope.runInAsyncScope(() => use({ flowId, logBuffer }));
                 }
                 finally {
                     (0, Logger_1.setProcessLocalFlowId)(null);
+                    (0, Logger_1.setProcessLocalLogBuffer)(null);
                 }
             });
         },
@@ -84,7 +89,7 @@ exports.test = test_1.test.extend({
             .optional()
             .default(250)
             .parse(testInfo.project.metadata['visualCueDurationMs']);
-        const { flowId } = flowLoggingContext;
+        const { flowId, logBuffer } = flowLoggingContext;
         const extendedPage = await (0, extendPage_1.extendPage)(page, {
             flowId: flowId,
             visualCueDurationMs: visualCueDurationMs,
@@ -97,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
@@ -113,7 +123,7 @@ exports.test = test_1.test.extend({
             throw error;
         }
         finally {
-            await finalizeTest(extendedPage, testInfo);
+            await finalizeTest(extendedPage, testInfo, logBuffer);
         }
     },
 });
@@ -224,7 +234,7 @@ async function attachStepScreenshots(sharedState, testInfo) {
         contentType: 'application/json',
     });
 }
-async function finalizeTest(page, testInfo) {
+async function finalizeTest(page, testInfo, logBuffer) {
     const sharedState = page._dnb;
     try {
         sharedState.donobuFlowMetadata.state =
@@ -237,6 +247,17 @@ async function finalizeTest(page, testInfo) {
             body: JSON.stringify(sharedState.donobuFlowMetadata, null, 2),
             contentType: 'application/json',
         });
+        // Persist captured flow logs so they are available in the Donobu UI,
+        // mirroring what DonobuFlowsManager does for Studio-launched flows.
+        if (logBuffer) {
+            try {
+                const snapshot = logBuffer.snapshot();
+                await sharedState.persistence.setFlowFile(sharedState.donobuFlowMetadata.id, 'logs.json', Buffer.from(JSON.stringify(snapshot)));
+            }
+            catch (error) {
+                Logger_1.appLogger.error('Failed to persist flow logs:', error);
+            }
+        }
         // Attach step-level screenshots from the flow's tool call history.
         // These enable the HTML report to show a visual timeline of what the
         // AI agent saw at each step.

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/Logger.d.ts

@@ -1,5 +1,6 @@
 import { AsyncLocalStorage } from 'async_hooks';
 import winston from 'winston';
+import type { FlowLogBuffer } from '../utils/FlowLogBuffer';
 export declare const loggingContext: AsyncLocalStorage<unknown>;
 /**
  * ###################################################################################
@@ -12,11 +13,12 @@ export declare const loggingContext: AsyncLocalStorage<unknown>;
  * outside the ALS store even though we seed it later in the fixture.
  *
  * We therefore keep a *process-local* fallback that stores the most recent flow
- * ID for the worker. Because Playwright guarantees only one test runs at a time
- * per process, this is safe: there is no race between concurrent tests in the
- * same worker, yet we still avoid leaking IDs across workers.
+ * ID and log buffer for the worker. Because Playwright guarantees only one test
+ * runs at a time per process, this is safe: there is no race between concurrent
+ * tests in the same worker, yet we still avoid leaking IDs across workers.
  */
 export declare function setProcessLocalFlowId(flowId: string | null): void;
+export declare function setProcessLocalLogBuffer(buffer: FlowLogBuffer | null): void;
 export declare const appLogger: winston.Logger;
 export declare const accessLogger: winston.Logger;
 export declare const browserLogger: winston.Logger;

package/dist/utils/Logger.js

@@ -5,6 +5,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.networkLogger = exports.browserLogger = exports.accessLogger = exports.appLogger = exports.loggingContext = void 0;
 exports.setProcessLocalFlowId = setProcessLocalFlowId;
+exports.setProcessLocalLogBuffer = setProcessLocalLogBuffer;
 exports.logErrorWithoutStack = logErrorWithoutStack;
 exports.formatLogInfoForTest = formatLogInfoForTest;
 const async_hooks_1 = require("async_hooks");
@@ -142,13 +143,15 @@ class FlowLogBufferTransport extends winston_transport_1.default {
     }
     log(info, callback) {
         const store = exports.loggingContext.getStore();
-        if (store?.logBuffer) {
-            store.logBuffer.push({ ...info, source: this.source });
+        const buffer = store?.logBuffer || processLocalLogBuffer;
+        if (buffer) {
+            buffer.push({ ...info, source: this.source });
         }
         callback();
     }
 }
 let processLocalFlowId = null;
+let processLocalLogBuffer = null;
 /**
  * ###################################################################################
  * # WARNING! Only use this function within the context of Playwright test fixtures! #
@@ -160,13 +163,16 @@ let processLocalFlowId = null;
  * outside the ALS store even though we seed it later in the fixture.
  *
  * We therefore keep a *process-local* fallback that stores the most recent flow
- * ID for the worker. Because Playwright guarantees only one test runs at a time
- * per process, this is safe: there is no race between concurrent tests in the
- * same worker, yet we still avoid leaking IDs across workers.
+ * ID and log buffer for the worker. Because Playwright guarantees only one test
+ * runs at a time per process, this is safe: there is no race between concurrent
+ * tests in the same worker, yet we still avoid leaking IDs across workers.
  */
 function setProcessLocalFlowId(flowId) {
     processLocalFlowId = flowId;
 }
+function setProcessLocalLogBuffer(buffer) {
+    processLocalLogBuffer = buffer;
+}
 // Format to add the currently running flow's ID (if any) from AsyncLocalStorage
 const flowIdFormat = winston_1.default.format((info) => {
     const store = exports.loggingContext.getStore();

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.1",
+  "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",