donobu 5.18.1 → 5.18.2

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