donobu 5.18.0 → 5.18.2

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

@@ -31,6 +31,24 @@ const donobuTestStack_1 = require("../test/utils/donobuTestStack");
 const originalGotoRegistry_1 = require("./originalGotoRegistry");
 const SmartSelector_1 = require("./SmartSelector");
 const tbd_1 = require("./tbd");
+/**
+ * Resolve a URL against the browser context's baseURL (if set), mirroring
+ * what Playwright does internally before issuing the navigation request.
+ * This gives us the *intended* URL (before any server-side redirects).
+ */
+function resolveBaseUrl(page, url) {
+    const baseURL = page.context()._options?.baseURL;
+    // Only resolve against baseURL if the URL is relative (no scheme like https:).
+    if (baseURL && !/^[a-zA-Z][a-zA-Z\d+\-.]*:/.test(url)) {
+        try {
+            return new URL(url, baseURL).href;
+        }
+        catch {
+            return url;
+        }
+    }
+    return url;
+}
 // Donobu page extension helpers: decorate Playwright pages with Donobu behaviors and keep one
 // coherent flow (and persistence record) per browser context so new tabs share state safely.
 const PLACEHOLDER_FLOW_URL = 'https://example.com';
@@ -340,13 +358,14 @@ Use this information to return an appropriate JSON object.`,
     page.goto = async (url, options) => {
         const startedAt = new Date().getTime();
         const flowId = sharedState.donobuFlowMetadata.id;
+        const effectiveUrl = resolveBaseUrl(page, url);
         // First navigation sets the target website and records the tool call with screenshots.
         if (sharedState.donobuFlowMetadata.web?.targetWebsite === PLACEHOLDER_FLOW_URL) {
             sharedState.donobuFlowMetadata = {
                 ...sharedState.donobuFlowMetadata,
                 web: {
                     ...sharedState.donobuFlowMetadata.web,
-                    targetWebsite: url,
+                    targetWebsite: effectiveUrl,
                 },
             };
             await sharedState.persistence.setFlowMetadata(sharedState.donobuFlowMetadata);
@@ -362,18 +381,18 @@ Use this information to return an appropriate JSON object.`,
                 id: MiscUtils_1.MiscUtils.createAdHocToolCallId(),
                 toolName: GoToWebpageTool_1.GoToWebpageTool.NAME,
                 parameters: {
-                    url: url,
+                    url: effectiveUrl,
                 },
                 outcome: {
                     isSuccessful: true,
-                    forLlm: `Successfully navigated to ${url}`,
+                    forLlm: `Successfully navigated to ${effectiveUrl}`,
                     metadata: {
                         pageTitle: pageTitle,
                         resolvedUrl: page.url(),
                     },
                 },
                 postCallImageId: postCallImageId,
-                page: url,
+                page: effectiveUrl,
                 startedAt: startedAt,
                 completedAt: completedAt,
             });
@@ -390,7 +409,7 @@ Use this information to return an appropriate JSON object.`,
                     id: MiscUtils_1.MiscUtils.createAdHocToolCallId(),
                     toolName: GoToWebpageTool_1.GoToWebpageTool.NAME,
                     parameters: {
-                        url: url,
+                        url: effectiveUrl,
                     },
                     outcome: {
                         isSuccessful: false,
@@ -398,7 +417,7 @@ Use this information to return an appropriate JSON object.`,
                         metadata: null,
                     },
                     postCallImageId: postCallImageId,
-                    page: url,
+                    page: effectiveUrl,
                     startedAt: startedAt,
                     completedAt: completedAt,
                 });

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,6 +22,7 @@ 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 cacheLocator_1 = require("../ai/cache/cacheLocator");
@@ -55,16 +56,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 +88,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,
@@ -113,7 +117,7 @@ exports.test = test_1.test.extend({
             throw error;
         }
         finally {
-            await finalizeTest(extendedPage, testInfo);
+            await finalizeTest(extendedPage, testInfo, logBuffer);
         }
     },
 });
@@ -224,7 +228,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 +241,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/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/lib/page/extendPage.js

@@ -31,6 +31,24 @@ const donobuTestStack_1 = require("../test/utils/donobuTestStack");
 const originalGotoRegistry_1 = require("./originalGotoRegistry");
 const SmartSelector_1 = require("./SmartSelector");
 const tbd_1 = require("./tbd");
+/**
+ * Resolve a URL against the browser context's baseURL (if set), mirroring
+ * what Playwright does internally before issuing the navigation request.
+ * This gives us the *intended* URL (before any server-side redirects).
+ */
+function resolveBaseUrl(page, url) {
+    const baseURL = page.context()._options?.baseURL;
+    // Only resolve against baseURL if the URL is relative (no scheme like https:).
+    if (baseURL && !/^[a-zA-Z][a-zA-Z\d+\-.]*:/.test(url)) {
+        try {
+            return new URL(url, baseURL).href;
+        }
+        catch {
+            return url;
+        }
+    }
+    return url;
+}
 // Donobu page extension helpers: decorate Playwright pages with Donobu behaviors and keep one
 // coherent flow (and persistence record) per browser context so new tabs share state safely.
 const PLACEHOLDER_FLOW_URL = 'https://example.com';
@@ -340,13 +358,14 @@ Use this information to return an appropriate JSON object.`,
     page.goto = async (url, options) => {
         const startedAt = new Date().getTime();
         const flowId = sharedState.donobuFlowMetadata.id;
+        const effectiveUrl = resolveBaseUrl(page, url);
         // First navigation sets the target website and records the tool call with screenshots.
         if (sharedState.donobuFlowMetadata.web?.targetWebsite === PLACEHOLDER_FLOW_URL) {
             sharedState.donobuFlowMetadata = {
                 ...sharedState.donobuFlowMetadata,
                 web: {
                     ...sharedState.donobuFlowMetadata.web,
-                    targetWebsite: url,
+                    targetWebsite: effectiveUrl,
                 },
             };
             await sharedState.persistence.setFlowMetadata(sharedState.donobuFlowMetadata);
@@ -362,18 +381,18 @@ Use this information to return an appropriate JSON object.`,
                 id: MiscUtils_1.MiscUtils.createAdHocToolCallId(),
                 toolName: GoToWebpageTool_1.GoToWebpageTool.NAME,
                 parameters: {
-                    url: url,
+                    url: effectiveUrl,
                 },
                 outcome: {
                     isSuccessful: true,
-                    forLlm: `Successfully navigated to ${url}`,
+                    forLlm: `Successfully navigated to ${effectiveUrl}`,
                     metadata: {
                         pageTitle: pageTitle,
                         resolvedUrl: page.url(),
                     },
                 },
                 postCallImageId: postCallImageId,
-                page: url,
+                page: effectiveUrl,
                 startedAt: startedAt,
                 completedAt: completedAt,
             });
@@ -390,7 +409,7 @@ Use this information to return an appropriate JSON object.`,
                     id: MiscUtils_1.MiscUtils.createAdHocToolCallId(),
                     toolName: GoToWebpageTool_1.GoToWebpageTool.NAME,
                     parameters: {
-                        url: url,
+                        url: effectiveUrl,
                     },
                     outcome: {
                         isSuccessful: false,
@@ -398,7 +417,7 @@ Use this information to return an appropriate JSON object.`,
                         metadata: null,
                     },
                     postCallImageId: postCallImageId,
-                    page: url,
+                    page: effectiveUrl,
                     startedAt: startedAt,
                     completedAt: completedAt,
                 });

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,6 +22,7 @@ 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 cacheLocator_1 = require("../ai/cache/cacheLocator");
@@ -55,16 +56,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 +88,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,
@@ -113,7 +117,7 @@ exports.test = test_1.test.extend({
             throw error;
         }
         finally {
-            await finalizeTest(extendedPage, testInfo);
+            await finalizeTest(extendedPage, testInfo, logBuffer);
         }
     },
 });
@@ -224,7 +228,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 +241,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/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/package.json

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