donobu 5.27.0 → 5.27.2

package/dist/esm/main.d.ts

@@ -21,6 +21,7 @@ export * from './lib/test/fixtures/browserState';
 export * from './lib/test/fixtures/gptClients';
 export * from './lib/test/testExtension';
 export * from './lib/test/utils/donobuTestStack';
+export { AdminApiController } from './managers/AdminApiController';
 export { DonobuFlow } from './managers/DonobuFlow';
 export { distillAllowedEnvVariableNames, DonobuFlowsManager, prepareToolCallsForRerun, } from './managers/DonobuFlowsManager';
 export { type DonobuStack, setupDonobuStack } from './managers/DonobuStack';

package/dist/esm/main.js

@@ -14,7 +14,7 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.TargetRuntimePluginRegistry = exports.PersistencePluginRegistry = exports.createDefaultToolRegistry = exports.ToolManager = exports.PluginLoader = exports.InteractionVisualizer = exports.setupDonobuStack = exports.prepareToolCallsForRerun = exports.DonobuFlowsManager = exports.distillAllowedEnvVariableNames = exports.DonobuFlow = exports.env = exports.runGenerateSiteTests = exports.VercelAiGptClient = exports.OpenAiGptClient = exports.GptClientPluginRegistry = exports.GptClient = exports.GoogleGenerativeAiGptClient = exports.fixAssertFields = exports.DonobuGptClient = exports.AnthropicGptClient = void 0;
+exports.TargetRuntimePluginRegistry = exports.PersistencePluginRegistry = exports.createDefaultToolRegistry = exports.ToolManager = exports.PluginLoader = exports.InteractionVisualizer = exports.setupDonobuStack = exports.prepareToolCallsForRerun = exports.DonobuFlowsManager = exports.distillAllowedEnvVariableNames = exports.DonobuFlow = exports.AdminApiController = exports.env = exports.runGenerateSiteTests = exports.VercelAiGptClient = exports.OpenAiGptClient = exports.GptClientPluginRegistry = exports.GptClient = exports.GoogleGenerativeAiGptClient = exports.fixAssertFields = exports.DonobuGptClient = exports.AnthropicGptClient = void 0;
 exports.startDonobuServer = startDonobuServer;
 const commander_1 = require("commander");
 const v4_1 = require("zod/v4");
@@ -56,6 +56,8 @@ __exportStar(require("./lib/test/fixtures/browserState"), exports);
 __exportStar(require("./lib/test/fixtures/gptClients"), exports);
 __exportStar(require("./lib/test/testExtension"), exports);
 __exportStar(require("./lib/test/utils/donobuTestStack"), exports);
+var AdminApiController_2 = require("./managers/AdminApiController");
+Object.defineProperty(exports, "AdminApiController", { enumerable: true, get: function () { return AdminApiController_2.AdminApiController; } });
 var DonobuFlow_1 = require("./managers/DonobuFlow");
 Object.defineProperty(exports, "DonobuFlow", { enumerable: true, get: function () { return DonobuFlow_1.DonobuFlow; } });
 var DonobuFlowsManager_1 = require("./managers/DonobuFlowsManager");

package/dist/esm/managers/DonobuFlow.d.ts

@@ -107,8 +107,27 @@ export declare class DonobuFlow {
     private onUnexpectedException;
     /**
      * This method is called when a flow is complete (i.e. when {@link DonobuFlow.run} should return).
+     *
+     * Browser session state and the final metadata write are committed by
+     * whichever code path produced the terminal state (transitionState for
+     * tool-driven completion; the on*Failure handlers for failure paths) —
+     * by the time we reach onComplete those have already happened.
      */
     private onComplete;
+    /**
+     * Persists the current browser session state if the flow's config has
+     * `persistState` enabled. Must be called BEFORE every site that commits
+     * a terminal `state` to persistence — otherwise a frontend that polls
+     * for the flow's state can observe "complete" between the metadata
+     * write and the session-state upload, fire an eager browser-state
+     * fetch (e.g. FlowDeveloperTools auto-loads on terminal), and 404 until
+     * the upload finishes.
+     *
+     * The browser context typically survives all-pages-closed (the read
+     * goes against the context, not a specific page), so this is also
+     * safe to call from the failure handlers like onTargetClosed.
+     */
+    private persistTerminalSessionStateIfNeeded;
     /**
      * Attempt to POST a JSON body containing given flow ID to the given
      * ${@link callbackUrl} if the URL is non-null. Note that there is no retying

package/dist/esm/managers/DonobuFlow.js

@@ -218,6 +218,7 @@ class DonobuFlow {
             this.metadata.state = 'FAILED';
             Logger_1.appLogger.error(result.reason);
             this.metadata.result = { failed: result.reason };
+            await this.persistTerminalSessionStateIfNeeded();
             await this.persistence.setFlowMetadata(this.metadata);
         }
     }
@@ -232,6 +233,7 @@ class DonobuFlow {
             failed: `Stopped flow due to the ${this.gptClient?.config.type} GPT platform throwing an internal error!`,
             context: error.message,
         };
+        await this.persistTerminalSessionStateIfNeeded();
         await this.persistence.setFlowMetadata(this.metadata);
     }
     /**
@@ -247,6 +249,7 @@ class DonobuFlow {
             : `Your ${platform} API quota has been exhausted. Please check your account's billing and usage limits; this may happen if there is a lack of funds in the account`;
         Logger_1.appLogger.error(failedMessage, error);
         this.metadata.result = { failed: failedMessage };
+        await this.persistTerminalSessionStateIfNeeded();
         await this.persistence.setFlowMetadata(this.metadata);
     }
     /**
@@ -341,18 +344,38 @@ class DonobuFlow {
         this.metadata.result = {
             failed: 'Internal error 🙈',
         };
+        await this.persistTerminalSessionStateIfNeeded();
         await this.persistence.setFlowMetadata(this.metadata);
     }
     /**
      * This method is called when a flow is complete (i.e. when {@link DonobuFlow.run} should return).
+     *
+     * Browser session state and the final metadata write are committed by
+     * whichever code path produced the terminal state (transitionState for
+     * tool-driven completion; the on*Failure handlers for failure paths) —
+     * by the time we reach onComplete those have already happened.
      */
     async onComplete() {
-        await this.persistence.setFlowMetadata(this.metadata);
+        DonobuFlow.invokeFlowFinishedCallback(this.metadata.callbackUrl, this.metadata.id);
+        this.controlPanel.close();
+    }
+    /**
+     * Persists the current browser session state if the flow's config has
+     * `persistState` enabled. Must be called BEFORE every site that commits
+     * a terminal `state` to persistence — otherwise a frontend that polls
+     * for the flow's state can observe "complete" between the metadata
+     * write and the session-state upload, fire an eager browser-state
+     * fetch (e.g. FlowDeveloperTools auto-loads on terminal), and 404 until
+     * the upload finishes.
+     *
+     * The browser context typically survives all-pages-closed (the read
+     * goes against the context, not a specific page), so this is also
+     * safe to call from the failure handlers like onTargetClosed.
+     */
+    async persistTerminalSessionStateIfNeeded() {
         if (this.metadata.web?.browser?.persistState) {
             await this.targetInspector.persistSessionState(this.persistence, this.metadata.id);
         }
-        DonobuFlow.invokeFlowFinishedCallback(this.metadata.callbackUrl, this.metadata.id);
-        this.controlPanel.close();
     }
     /**
      * Attempt to POST a JSON body containing given flow ID to the given
@@ -623,8 +646,14 @@ Message: ${dialog.message()}`;
         // the result object before the final state is set. If we did not do
         // this, then someone polling for a flow's state may see the flow
         // finished but still see a null result.
+        //
+        // The same rationale applies to the browser session state: an eager
+        // fetch from the frontend (e.g. FlowDeveloperTools auto-loads on
+        // terminal state) would otherwise race the upload that onComplete
+        // performs and 404 until the user manually refreshed.
         if ((0, FlowMetadata_1.isComplete)(nextState)) {
             this.metadata.result = await this.createResultJson(nextState);
+            await this.persistTerminalSessionStateIfNeeded();
         }
         const lastState = this.metadata.state;
         this.metadata.state = nextState;

package/dist/main.d.ts

@@ -21,6 +21,7 @@ export * from './lib/test/fixtures/browserState';
 export * from './lib/test/fixtures/gptClients';
 export * from './lib/test/testExtension';
 export * from './lib/test/utils/donobuTestStack';
+export { AdminApiController } from './managers/AdminApiController';
 export { DonobuFlow } from './managers/DonobuFlow';
 export { distillAllowedEnvVariableNames, DonobuFlowsManager, prepareToolCallsForRerun, } from './managers/DonobuFlowsManager';
 export { type DonobuStack, setupDonobuStack } from './managers/DonobuStack';

package/dist/main.js

@@ -14,7 +14,7 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.TargetRuntimePluginRegistry = exports.PersistencePluginRegistry = exports.createDefaultToolRegistry = exports.ToolManager = exports.PluginLoader = exports.InteractionVisualizer = exports.setupDonobuStack = exports.prepareToolCallsForRerun = exports.DonobuFlowsManager = exports.distillAllowedEnvVariableNames = exports.DonobuFlow = exports.env = exports.runGenerateSiteTests = exports.VercelAiGptClient = exports.OpenAiGptClient = exports.GptClientPluginRegistry = exports.GptClient = exports.GoogleGenerativeAiGptClient = exports.fixAssertFields = exports.DonobuGptClient = exports.AnthropicGptClient = void 0;
+exports.TargetRuntimePluginRegistry = exports.PersistencePluginRegistry = exports.createDefaultToolRegistry = exports.ToolManager = exports.PluginLoader = exports.InteractionVisualizer = exports.setupDonobuStack = exports.prepareToolCallsForRerun = exports.DonobuFlowsManager = exports.distillAllowedEnvVariableNames = exports.DonobuFlow = exports.AdminApiController = exports.env = exports.runGenerateSiteTests = exports.VercelAiGptClient = exports.OpenAiGptClient = exports.GptClientPluginRegistry = exports.GptClient = exports.GoogleGenerativeAiGptClient = exports.fixAssertFields = exports.DonobuGptClient = exports.AnthropicGptClient = void 0;
 exports.startDonobuServer = startDonobuServer;
 const commander_1 = require("commander");
 const v4_1 = require("zod/v4");
@@ -56,6 +56,8 @@ __exportStar(require("./lib/test/fixtures/browserState"), exports);
 __exportStar(require("./lib/test/fixtures/gptClients"), exports);
 __exportStar(require("./lib/test/testExtension"), exports);
 __exportStar(require("./lib/test/utils/donobuTestStack"), exports);
+var AdminApiController_2 = require("./managers/AdminApiController");
+Object.defineProperty(exports, "AdminApiController", { enumerable: true, get: function () { return AdminApiController_2.AdminApiController; } });
 var DonobuFlow_1 = require("./managers/DonobuFlow");
 Object.defineProperty(exports, "DonobuFlow", { enumerable: true, get: function () { return DonobuFlow_1.DonobuFlow; } });
 var DonobuFlowsManager_1 = require("./managers/DonobuFlowsManager");

package/dist/managers/DonobuFlow.d.ts

@@ -107,8 +107,27 @@ export declare class DonobuFlow {
     private onUnexpectedException;
     /**
      * This method is called when a flow is complete (i.e. when {@link DonobuFlow.run} should return).
+     *
+     * Browser session state and the final metadata write are committed by
+     * whichever code path produced the terminal state (transitionState for
+     * tool-driven completion; the on*Failure handlers for failure paths) —
+     * by the time we reach onComplete those have already happened.
      */
     private onComplete;
+    /**
+     * Persists the current browser session state if the flow's config has
+     * `persistState` enabled. Must be called BEFORE every site that commits
+     * a terminal `state` to persistence — otherwise a frontend that polls
+     * for the flow's state can observe "complete" between the metadata
+     * write and the session-state upload, fire an eager browser-state
+     * fetch (e.g. FlowDeveloperTools auto-loads on terminal), and 404 until
+     * the upload finishes.
+     *
+     * The browser context typically survives all-pages-closed (the read
+     * goes against the context, not a specific page), so this is also
+     * safe to call from the failure handlers like onTargetClosed.
+     */
+    private persistTerminalSessionStateIfNeeded;
     /**
      * Attempt to POST a JSON body containing given flow ID to the given
      * ${@link callbackUrl} if the URL is non-null. Note that there is no retying

package/dist/managers/DonobuFlow.js

@@ -218,6 +218,7 @@ class DonobuFlow {
             this.metadata.state = 'FAILED';
             Logger_1.appLogger.error(result.reason);
             this.metadata.result = { failed: result.reason };
+            await this.persistTerminalSessionStateIfNeeded();
             await this.persistence.setFlowMetadata(this.metadata);
         }
     }
@@ -232,6 +233,7 @@ class DonobuFlow {
             failed: `Stopped flow due to the ${this.gptClient?.config.type} GPT platform throwing an internal error!`,
             context: error.message,
         };
+        await this.persistTerminalSessionStateIfNeeded();
         await this.persistence.setFlowMetadata(this.metadata);
     }
     /**
@@ -247,6 +249,7 @@ class DonobuFlow {
             : `Your ${platform} API quota has been exhausted. Please check your account's billing and usage limits; this may happen if there is a lack of funds in the account`;
         Logger_1.appLogger.error(failedMessage, error);
         this.metadata.result = { failed: failedMessage };
+        await this.persistTerminalSessionStateIfNeeded();
         await this.persistence.setFlowMetadata(this.metadata);
     }
     /**
@@ -341,18 +344,38 @@ class DonobuFlow {
         this.metadata.result = {
             failed: 'Internal error 🙈',
         };
+        await this.persistTerminalSessionStateIfNeeded();
         await this.persistence.setFlowMetadata(this.metadata);
     }
     /**
      * This method is called when a flow is complete (i.e. when {@link DonobuFlow.run} should return).
+     *
+     * Browser session state and the final metadata write are committed by
+     * whichever code path produced the terminal state (transitionState for
+     * tool-driven completion; the on*Failure handlers for failure paths) —
+     * by the time we reach onComplete those have already happened.
      */
     async onComplete() {
-        await this.persistence.setFlowMetadata(this.metadata);
+        DonobuFlow.invokeFlowFinishedCallback(this.metadata.callbackUrl, this.metadata.id);
+        this.controlPanel.close();
+    }
+    /**
+     * Persists the current browser session state if the flow's config has
+     * `persistState` enabled. Must be called BEFORE every site that commits
+     * a terminal `state` to persistence — otherwise a frontend that polls
+     * for the flow's state can observe "complete" between the metadata
+     * write and the session-state upload, fire an eager browser-state
+     * fetch (e.g. FlowDeveloperTools auto-loads on terminal), and 404 until
+     * the upload finishes.
+     *
+     * The browser context typically survives all-pages-closed (the read
+     * goes against the context, not a specific page), so this is also
+     * safe to call from the failure handlers like onTargetClosed.
+     */
+    async persistTerminalSessionStateIfNeeded() {
         if (this.metadata.web?.browser?.persistState) {
             await this.targetInspector.persistSessionState(this.persistence, this.metadata.id);
         }
-        DonobuFlow.invokeFlowFinishedCallback(this.metadata.callbackUrl, this.metadata.id);
-        this.controlPanel.close();
     }
     /**
      * Attempt to POST a JSON body containing given flow ID to the given
@@ -623,8 +646,14 @@ Message: ${dialog.message()}`;
         // the result object before the final state is set. If we did not do
         // this, then someone polling for a flow's state may see the flow
         // finished but still see a null result.
+        //
+        // The same rationale applies to the browser session state: an eager
+        // fetch from the frontend (e.g. FlowDeveloperTools auto-loads on
+        // terminal state) would otherwise race the upload that onComplete
+        // performs and 404 until the user manually refreshed.
         if ((0, FlowMetadata_1.isComplete)(nextState)) {
             this.metadata.result = await this.createResultJson(nextState);
+            await this.persistTerminalSessionStateIfNeeded();
         }
         const lastState = this.metadata.state;
         this.metadata.state = nextState;

package/package.json

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