donobu 5.27.2 → 5.27.4

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

@@ -21,6 +21,7 @@ const ChangeWebBrowserTabTool_1 = require("../../tools/ChangeWebBrowserTabTool")
 const CreateBrowserCookieReportTool_1 = require("../../tools/CreateBrowserCookieReportTool");
 const GoToWebpageTool_1 = require("../../tools/GoToWebpageTool");
 const RunAccessibilityTestTool_1 = require("../../tools/RunAccessibilityTestTool");
+const ansi_1 = require("../../utils/ansi");
 const BrowserUtils_1 = require("../../utils/BrowserUtils");
 const Logger_1 = require("../../utils/Logger");
 const MiscUtils_1 = require("../../utils/MiscUtils");
@@ -241,7 +242,11 @@ Valid options:
                     // All retry attempts exhausted
                     throw new ToolCallFailedException_1.ToolCallFailedException(AssertTool_1.AssertTool.NAME, {
                         isSuccessful: false,
-                        forLlm: `Assertion FAILED (cached) for: ${assertion}\nPlaywright Error: ${lastError?.message}`,
+                        // Strip ANSI: Playwright matchers style their messages for the
+                        // terminal, but this string flows into JSON-stringified exception
+                        // messages, the LLM, and HTML/markdown reports — places where the
+                        // codes never render and just become visible junk.
+                        forLlm: `Assertion FAILED (cached) for: ${assertion}\nPlaywright Error: ${(0, ansi_1.stripAnsi)(lastError?.message ?? '')}`,
                         metadata: {
                             cached: true,
                             steps: cached.steps,

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

@@ -108,24 +108,30 @@ export declare class DonobuFlow {
     /**
      * 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.
+     * Browser session state and the terminal-state metadata write are
+     * committed by whichever code path produced the terminal state
+     * (transitionState for tool-driven completion; onTargetClosed /
+     * onPersistentGptFailure / onInsufficientQuota / onUnexpectedException
+     * for failure paths) — by the time we reach onComplete those have
+     * already happened. This method just runs the post-completion side
+     * effects.
      */
     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.
+     * `persistState` enabled. Must be called BEFORE the in-memory `state`
+     * is mutated to a terminal value at every site that produces a
+     * terminal state — otherwise FlowCatalog.getFlowById can read the
+     * live FlowMetadata object (LOCAL deployments) and a frontend that
+     * observes the terminal state will race the (potentially network-
+     * bound) upload here, getting a 404 from a subsequent browser-state
+     * fetch.
      *
      * 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.
+     * goes against the context, not a specific page), so this is safe to
+     * call from failure handlers like onTargetClosed. If the read does
+     * fail, persistSessionState catches and logs internally — it doesn't
+     * propagate.
      */
     private persistTerminalSessionStateIfNeeded;
     /**

package/dist/esm/managers/DonobuFlow.js

@@ -215,10 +215,18 @@ class DonobuFlow {
     async onTargetClosed() {
         const result = await this.targetInspector.handleTargetClosed();
         if (!result.recovered) {
-            this.metadata.state = 'FAILED';
+            // Persist browser state BEFORE flipping the in-memory `state` to
+            // a terminal value. FlowCatalog.getFlowById serves the *live*
+            // FlowMetadata object for LOCAL deployments, so the next frontend
+            // poll observes terminal state the moment we mutate `state`.
+            // If we do that before the (potentially network-bound) session
+            // upload, an eager browser-state fetch from the frontend (e.g.
+            // FlowDeveloperTools auto-loads on terminal) races the upload and
+            // 404s. Same rationale as the ordering in transitionState.
             Logger_1.appLogger.error(result.reason);
-            this.metadata.result = { failed: result.reason };
             await this.persistTerminalSessionStateIfNeeded();
+            this.metadata.result = { failed: result.reason };
+            this.metadata.state = 'FAILED';
             await this.persistence.setFlowMetadata(this.metadata);
         }
     }
@@ -227,13 +235,13 @@ class DonobuFlow {
      * are internal retries). This method will mark the flow as a failure.
      */
     async onPersistentGptFailure(error) {
-        this.metadata.state = 'FAILED';
         Logger_1.appLogger.error(`Stopped flow due to the ${this.gptClient?.config.type} GPT platform throwing an internal error!`, error);
+        await this.persistTerminalSessionStateIfNeeded();
         this.metadata.result = {
             failed: `Stopped flow due to the ${this.gptClient?.config.type} GPT platform throwing an internal error!`,
             context: error.message,
         };
-        await this.persistTerminalSessionStateIfNeeded();
+        this.metadata.state = 'FAILED';
         await this.persistence.setFlowMetadata(this.metadata);
     }
     /**
@@ -241,15 +249,15 @@ class DonobuFlow {
      * usage quota or credits have been exhausted (HTTP 402).
      */
     async onInsufficientQuota(error) {
-        this.metadata.state = 'FAILED';
         const platform = error.gptPlatform;
         const isDonobu = platform === 'DONOBU';
         const failedMessage = isDonobu
             ? 'Your Donobu AI credits have been exhausted. Please add more credits to your account to continue running flows.'
             : `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();
+        this.metadata.result = { failed: failedMessage };
+        this.metadata.state = 'FAILED';
         await this.persistence.setFlowMetadata(this.metadata);
     }
     /**
@@ -339,21 +347,24 @@ class DonobuFlow {
      * method will mark the flow as a failure.
      */
     async onUnexpectedException(error) {
-        this.metadata.state = 'FAILED';
         Logger_1.appLogger.error('Stopped flow due to exception!', error);
+        await this.persistTerminalSessionStateIfNeeded();
         this.metadata.result = {
             failed: 'Internal error 🙈',
         };
-        await this.persistTerminalSessionStateIfNeeded();
+        this.metadata.state = 'FAILED';
         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.
+     * Browser session state and the terminal-state metadata write are
+     * committed by whichever code path produced the terminal state
+     * (transitionState for tool-driven completion; onTargetClosed /
+     * onPersistentGptFailure / onInsufficientQuota / onUnexpectedException
+     * for failure paths) — by the time we reach onComplete those have
+     * already happened. This method just runs the post-completion side
+     * effects.
      */
     async onComplete() {
         DonobuFlow.invokeFlowFinishedCallback(this.metadata.callbackUrl, this.metadata.id);
@@ -361,16 +372,19 @@ class DonobuFlow {
     }
     /**
      * 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.
+     * `persistState` enabled. Must be called BEFORE the in-memory `state`
+     * is mutated to a terminal value at every site that produces a
+     * terminal state — otherwise FlowCatalog.getFlowById can read the
+     * live FlowMetadata object (LOCAL deployments) and a frontend that
+     * observes the terminal state will race the (potentially network-
+     * bound) upload here, getting a 404 from a subsequent browser-state
+     * fetch.
      *
      * 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.
+     * goes against the context, not a specific page), so this is safe to
+     * call from failure handlers like onTargetClosed. If the read does
+     * fail, persistSessionState catches and logs internally — it doesn't
+     * propagate.
      */
     async persistTerminalSessionStateIfNeeded() {
         if (this.metadata.web?.browser?.persistState) {
@@ -647,10 +661,8 @@ Message: ${dialog.message()}`;
         // 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.
+        // The same rationale applies to the browser session state — see
+        // persistTerminalSessionStateIfNeeded for the full ordering note.
         if ((0, FlowMetadata_1.isComplete)(nextState)) {
             this.metadata.result = await this.createResultJson(nextState);
             await this.persistTerminalSessionStateIfNeeded();

package/dist/esm/reporter/render.d.ts

@@ -7,7 +7,6 @@
  * reporter and the auto-heal orchestrator) own I/O.
  */
 import type { DonobuReport } from './model';
-export declare function stripAnsi(str: string): string;
 interface TreatmentPlanRecord {
     generatedAtIso?: string;
     plan: {

package/dist/esm/reporter/render.js

@@ -8,18 +8,15 @@
  * reporter and the auto-heal orchestrator) own I/O.
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.stripAnsi = stripAnsi;
 exports.loadTriageData = loadTriageData;
 exports.renderHtml = renderHtml;
 const fs_1 = require("fs");
 const path_1 = require("path");
+const ansi_1 = require("../utils/ansi");
 const reportWalk_1 = require("./reportWalk");
 // ---------------------------------------------------------------------------
 // Helpers
 // ---------------------------------------------------------------------------
-function stripAnsi(str) {
-    return str.replace(/[\u001b\u009b][[()#;?]*(?:[0-9]{1,4}(?:;[0-9]{0,4})*)?[0-9A-ORZcf-nqry=><]/g, '');
-}
 /**
  * Convert ANSI SGR color codes to HTML spans. Handles the subset Playwright's
  * expect formatter emits: red (31), green (32), dim (2), and resets (0/22/39).
@@ -559,7 +556,7 @@ function renderErrors(errors) {
             html += `<pre class="snippet-block">${ansiToHtml(err.snippet)}</pre>`;
         }
         if (err.stack && err.stack !== err.message) {
-            html += `<details class="stack-details"><summary>Stack trace</summary><pre class="stack-block">${esc(stripAnsi(err.stack))}</pre></details>`;
+            html += `<details class="stack-details"><summary>Stack trace</summary><pre class="stack-block">${esc((0, ansi_1.stripAnsi)(err.stack))}</pre></details>`;
         }
     }
     return html;

package/dist/esm/reporter/renderMarkdown.js

@@ -9,7 +9,7 @@
  */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.renderMarkdown = renderMarkdown;
-const render_1 = require("./render");
+const ansi_1 = require("../utils/ansi");
 const reportWalk_1 = require("./reportWalk");
 function formatDuration(ms) {
     if (ms < 1000) {
@@ -154,7 +154,7 @@ function renderMarkdown(report) {
                     markdown += `\n<details>\n<summary>⚠️ Error Details</summary>\n\n`;
                     markdown += `\`\`\`\n${result.error.message || 'No error message available'}\n\`\`\`\n\n`;
                     if (result.error.snippet) {
-                        markdown += `**Code Snippet**:\n\`\`\`\n${(0, render_1.stripAnsi)(result.error.snippet)}\n\`\`\`\n\n`;
+                        markdown += `**Code Snippet**:\n\`\`\`\n${(0, ansi_1.stripAnsi)(result.error.snippet)}\n\`\`\`\n\n`;
                     }
                     markdown += `</details>\n\n`;
                 }

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

@@ -0,0 +1,11 @@
+/**
+ * Remove ANSI escape sequences (color/style codes) from a string.
+ *
+ * Playwright matchers emit terminal-styled errors. Once those messages cross
+ * into Donobu data flows — `forLlm` strings sent to the model, JSON-stringified
+ * exception messages, persisted reports — the codes never render as colors and
+ * just become visible junk (worse: `JSON.stringify` escapes the ESC byte into
+ * the 6-char literal `\u001b[..]m`). Strip at the boundary.
+ */
+export declare function stripAnsi(str: string): string;
+//# sourceMappingURL=ansi.d.ts.map

package/dist/esm/utils/ansi.js

@@ -0,0 +1,16 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.stripAnsi = stripAnsi;
+/**
+ * Remove ANSI escape sequences (color/style codes) from a string.
+ *
+ * Playwright matchers emit terminal-styled errors. Once those messages cross
+ * into Donobu data flows — `forLlm` strings sent to the model, JSON-stringified
+ * exception messages, persisted reports — the codes never render as colors and
+ * just become visible junk (worse: `JSON.stringify` escapes the ESC byte into
+ * the 6-char literal `\u001b[..]m`). Strip at the boundary.
+ */
+function stripAnsi(str) {
+    return str.replace(/[\u001b\u009b][[()#;?]*(?:[0-9]{1,4}(?:;[0-9]{0,4})*)?[0-9A-ORZcf-nqry=><]/g, '');
+}
+//# sourceMappingURL=ansi.js.map

package/dist/lib/page/extendPage.js

@@ -21,6 +21,7 @@ const ChangeWebBrowserTabTool_1 = require("../../tools/ChangeWebBrowserTabTool")
 const CreateBrowserCookieReportTool_1 = require("../../tools/CreateBrowserCookieReportTool");
 const GoToWebpageTool_1 = require("../../tools/GoToWebpageTool");
 const RunAccessibilityTestTool_1 = require("../../tools/RunAccessibilityTestTool");
+const ansi_1 = require("../../utils/ansi");
 const BrowserUtils_1 = require("../../utils/BrowserUtils");
 const Logger_1 = require("../../utils/Logger");
 const MiscUtils_1 = require("../../utils/MiscUtils");
@@ -241,7 +242,11 @@ Valid options:
                     // All retry attempts exhausted
                     throw new ToolCallFailedException_1.ToolCallFailedException(AssertTool_1.AssertTool.NAME, {
                         isSuccessful: false,
-                        forLlm: `Assertion FAILED (cached) for: ${assertion}\nPlaywright Error: ${lastError?.message}`,
+                        // Strip ANSI: Playwright matchers style their messages for the
+                        // terminal, but this string flows into JSON-stringified exception
+                        // messages, the LLM, and HTML/markdown reports — places where the
+                        // codes never render and just become visible junk.
+                        forLlm: `Assertion FAILED (cached) for: ${assertion}\nPlaywright Error: ${(0, ansi_1.stripAnsi)(lastError?.message ?? '')}`,
                         metadata: {
                             cached: true,
                             steps: cached.steps,

package/dist/managers/DonobuFlow.d.ts

@@ -108,24 +108,30 @@ export declare class DonobuFlow {
     /**
      * 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.
+     * Browser session state and the terminal-state metadata write are
+     * committed by whichever code path produced the terminal state
+     * (transitionState for tool-driven completion; onTargetClosed /
+     * onPersistentGptFailure / onInsufficientQuota / onUnexpectedException
+     * for failure paths) — by the time we reach onComplete those have
+     * already happened. This method just runs the post-completion side
+     * effects.
      */
     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.
+     * `persistState` enabled. Must be called BEFORE the in-memory `state`
+     * is mutated to a terminal value at every site that produces a
+     * terminal state — otherwise FlowCatalog.getFlowById can read the
+     * live FlowMetadata object (LOCAL deployments) and a frontend that
+     * observes the terminal state will race the (potentially network-
+     * bound) upload here, getting a 404 from a subsequent browser-state
+     * fetch.
      *
      * 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.
+     * goes against the context, not a specific page), so this is safe to
+     * call from failure handlers like onTargetClosed. If the read does
+     * fail, persistSessionState catches and logs internally — it doesn't
+     * propagate.
      */
     private persistTerminalSessionStateIfNeeded;
     /**

package/dist/managers/DonobuFlow.js

@@ -215,10 +215,18 @@ class DonobuFlow {
     async onTargetClosed() {
         const result = await this.targetInspector.handleTargetClosed();
         if (!result.recovered) {
-            this.metadata.state = 'FAILED';
+            // Persist browser state BEFORE flipping the in-memory `state` to
+            // a terminal value. FlowCatalog.getFlowById serves the *live*
+            // FlowMetadata object for LOCAL deployments, so the next frontend
+            // poll observes terminal state the moment we mutate `state`.
+            // If we do that before the (potentially network-bound) session
+            // upload, an eager browser-state fetch from the frontend (e.g.
+            // FlowDeveloperTools auto-loads on terminal) races the upload and
+            // 404s. Same rationale as the ordering in transitionState.
             Logger_1.appLogger.error(result.reason);
-            this.metadata.result = { failed: result.reason };
             await this.persistTerminalSessionStateIfNeeded();
+            this.metadata.result = { failed: result.reason };
+            this.metadata.state = 'FAILED';
             await this.persistence.setFlowMetadata(this.metadata);
         }
     }
@@ -227,13 +235,13 @@ class DonobuFlow {
      * are internal retries). This method will mark the flow as a failure.
      */
     async onPersistentGptFailure(error) {
-        this.metadata.state = 'FAILED';
         Logger_1.appLogger.error(`Stopped flow due to the ${this.gptClient?.config.type} GPT platform throwing an internal error!`, error);
+        await this.persistTerminalSessionStateIfNeeded();
         this.metadata.result = {
             failed: `Stopped flow due to the ${this.gptClient?.config.type} GPT platform throwing an internal error!`,
             context: error.message,
         };
-        await this.persistTerminalSessionStateIfNeeded();
+        this.metadata.state = 'FAILED';
         await this.persistence.setFlowMetadata(this.metadata);
     }
     /**
@@ -241,15 +249,15 @@ class DonobuFlow {
      * usage quota or credits have been exhausted (HTTP 402).
      */
     async onInsufficientQuota(error) {
-        this.metadata.state = 'FAILED';
         const platform = error.gptPlatform;
         const isDonobu = platform === 'DONOBU';
         const failedMessage = isDonobu
             ? 'Your Donobu AI credits have been exhausted. Please add more credits to your account to continue running flows.'
             : `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();
+        this.metadata.result = { failed: failedMessage };
+        this.metadata.state = 'FAILED';
         await this.persistence.setFlowMetadata(this.metadata);
     }
     /**
@@ -339,21 +347,24 @@ class DonobuFlow {
      * method will mark the flow as a failure.
      */
     async onUnexpectedException(error) {
-        this.metadata.state = 'FAILED';
         Logger_1.appLogger.error('Stopped flow due to exception!', error);
+        await this.persistTerminalSessionStateIfNeeded();
         this.metadata.result = {
             failed: 'Internal error 🙈',
         };
-        await this.persistTerminalSessionStateIfNeeded();
+        this.metadata.state = 'FAILED';
         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.
+     * Browser session state and the terminal-state metadata write are
+     * committed by whichever code path produced the terminal state
+     * (transitionState for tool-driven completion; onTargetClosed /
+     * onPersistentGptFailure / onInsufficientQuota / onUnexpectedException
+     * for failure paths) — by the time we reach onComplete those have
+     * already happened. This method just runs the post-completion side
+     * effects.
      */
     async onComplete() {
         DonobuFlow.invokeFlowFinishedCallback(this.metadata.callbackUrl, this.metadata.id);
@@ -361,16 +372,19 @@ class DonobuFlow {
     }
     /**
      * 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.
+     * `persistState` enabled. Must be called BEFORE the in-memory `state`
+     * is mutated to a terminal value at every site that produces a
+     * terminal state — otherwise FlowCatalog.getFlowById can read the
+     * live FlowMetadata object (LOCAL deployments) and a frontend that
+     * observes the terminal state will race the (potentially network-
+     * bound) upload here, getting a 404 from a subsequent browser-state
+     * fetch.
      *
      * 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.
+     * goes against the context, not a specific page), so this is safe to
+     * call from failure handlers like onTargetClosed. If the read does
+     * fail, persistSessionState catches and logs internally — it doesn't
+     * propagate.
      */
     async persistTerminalSessionStateIfNeeded() {
         if (this.metadata.web?.browser?.persistState) {
@@ -647,10 +661,8 @@ Message: ${dialog.message()}`;
         // 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.
+        // The same rationale applies to the browser session state — see
+        // persistTerminalSessionStateIfNeeded for the full ordering note.
         if ((0, FlowMetadata_1.isComplete)(nextState)) {
             this.metadata.result = await this.createResultJson(nextState);
             await this.persistTerminalSessionStateIfNeeded();

package/dist/reporter/render.d.ts

@@ -7,7 +7,6 @@
  * reporter and the auto-heal orchestrator) own I/O.
  */
 import type { DonobuReport } from './model';
-export declare function stripAnsi(str: string): string;
 interface TreatmentPlanRecord {
     generatedAtIso?: string;
     plan: {

package/dist/reporter/render.js

@@ -8,18 +8,15 @@
  * reporter and the auto-heal orchestrator) own I/O.
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.stripAnsi = stripAnsi;
 exports.loadTriageData = loadTriageData;
 exports.renderHtml = renderHtml;
 const fs_1 = require("fs");
 const path_1 = require("path");
+const ansi_1 = require("../utils/ansi");
 const reportWalk_1 = require("./reportWalk");
 // ---------------------------------------------------------------------------
 // Helpers
 // ---------------------------------------------------------------------------
-function stripAnsi(str) {
-    return str.replace(/[\u001b\u009b][[()#;?]*(?:[0-9]{1,4}(?:;[0-9]{0,4})*)?[0-9A-ORZcf-nqry=><]/g, '');
-}
 /**
  * Convert ANSI SGR color codes to HTML spans. Handles the subset Playwright's
  * expect formatter emits: red (31), green (32), dim (2), and resets (0/22/39).
@@ -559,7 +556,7 @@ function renderErrors(errors) {
             html += `<pre class="snippet-block">${ansiToHtml(err.snippet)}</pre>`;
         }
         if (err.stack && err.stack !== err.message) {
-            html += `<details class="stack-details"><summary>Stack trace</summary><pre class="stack-block">${esc(stripAnsi(err.stack))}</pre></details>`;
+            html += `<details class="stack-details"><summary>Stack trace</summary><pre class="stack-block">${esc((0, ansi_1.stripAnsi)(err.stack))}</pre></details>`;
         }
     }
     return html;

package/dist/reporter/renderMarkdown.js

@@ -9,7 +9,7 @@
  */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.renderMarkdown = renderMarkdown;
-const render_1 = require("./render");
+const ansi_1 = require("../utils/ansi");
 const reportWalk_1 = require("./reportWalk");
 function formatDuration(ms) {
     if (ms < 1000) {
@@ -154,7 +154,7 @@ function renderMarkdown(report) {
                     markdown += `\n<details>\n<summary>⚠️ Error Details</summary>\n\n`;
                     markdown += `\`\`\`\n${result.error.message || 'No error message available'}\n\`\`\`\n\n`;
                     if (result.error.snippet) {
-                        markdown += `**Code Snippet**:\n\`\`\`\n${(0, render_1.stripAnsi)(result.error.snippet)}\n\`\`\`\n\n`;
+                        markdown += `**Code Snippet**:\n\`\`\`\n${(0, ansi_1.stripAnsi)(result.error.snippet)}\n\`\`\`\n\n`;
                     }
                     markdown += `</details>\n\n`;
                 }

package/dist/utils/ansi.d.ts

@@ -0,0 +1,11 @@
+/**
+ * Remove ANSI escape sequences (color/style codes) from a string.
+ *
+ * Playwright matchers emit terminal-styled errors. Once those messages cross
+ * into Donobu data flows — `forLlm` strings sent to the model, JSON-stringified
+ * exception messages, persisted reports — the codes never render as colors and
+ * just become visible junk (worse: `JSON.stringify` escapes the ESC byte into
+ * the 6-char literal `\u001b[..]m`). Strip at the boundary.
+ */
+export declare function stripAnsi(str: string): string;
+//# sourceMappingURL=ansi.d.ts.map

package/dist/utils/ansi.js

@@ -0,0 +1,16 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.stripAnsi = stripAnsi;
+/**
+ * Remove ANSI escape sequences (color/style codes) from a string.
+ *
+ * Playwright matchers emit terminal-styled errors. Once those messages cross
+ * into Donobu data flows — `forLlm` strings sent to the model, JSON-stringified
+ * exception messages, persisted reports — the codes never render as colors and
+ * just become visible junk (worse: `JSON.stringify` escapes the ESC byte into
+ * the 6-char literal `\u001b[..]m`). Strip at the boundary.
+ */
+function stripAnsi(str) {
+    return str.replace(/[\u001b\u009b][[()#;?]*(?:[0-9]{1,4}(?:;[0-9]{0,4})*)?[0-9A-ORZcf-nqry=><]/g, '');
+}
+//# sourceMappingURL=ansi.js.map

package/package.json

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