donobu 5.52.2 → 5.53.0

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

@@ -15,6 +15,7 @@ const InteractionVisualizer_1 = require("../../managers/InteractionVisualizer");
 const ToolManager_1 = require("../../managers/ToolManager");
 const WebTargetInspector_1 = require("../../managers/WebTargetInspector");
 const ControlPanel_1 = require("../../models/ControlPanel");
+const MetadataVersion_1 = require("../../models/MetadataVersion");
 const AnalyzePageTextTool_1 = require("../../tools/AnalyzePageTextTool");
 const AssertTool_1 = require("../../tools/AssertTool");
 const AuditTool_1 = require("../../tools/AuditTool");
@@ -88,7 +89,7 @@ async function extendPage(page, options) {
         const sharedState = {
             donobuFlowMetadata: {
                 id: options?.flowId ?? (0, crypto_1.randomUUID)(),
-                metadataVersion: 1,
+                metadataVersion: MetadataVersion_1.CURRENT_METADATA_VERSION,
                 name: null,
                 createdWithDonobuVersion: MiscUtils_1.MiscUtils.DONOBU_VERSION,
                 target: 'web',
@@ -104,6 +105,7 @@ async function extendPage(page, options) {
                     targetWebsite: PLACEHOLDER_FLOW_URL,
                 },
                 envVars: options?.envVars ?? null,
+                tags: [],
                 gptConfigName: null,
                 hasGptConfigNameOverride: false,
                 customTools: null,

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

@@ -409,6 +409,7 @@ exports.test = test_1.test.extend({
         extendedPage._dnb.donobuFlowMetadata.name = getSanitizedTestName(testInfo);
         extendedPage._dnb.donobuFlowMetadata.testId = testInfo.testId;
         extendedPage._dnb.donobuFlowMetadata.overallObjective = overallObjective;
+        extendedPage._dnb.donobuFlowMetadata.tags = testInfo.tags;
         // 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
@@ -421,12 +422,27 @@ exports.test = test_1.test.extend({
         const asyncScope = new async_hooks_1.AsyncResource('DonobuTestContext');
         const boundUse = asyncScope.bind(use);
         try {
+            const testMetadata = flowMetadataToTestMetadata(testInfo.testId, extendedPage._dnb.donobuFlowMetadata);
             try {
-                const testMetadata = flowMetadataToTestMetadata(testInfo.testId, extendedPage._dnb.donobuFlowMetadata);
                 await extendedPage._dnb.testsPersistence.createTest(testMetadata);
             }
             catch {
-                // Test already exists from a prior run — nothing to do.
+                // Test already exists from a prior run. Refresh the fields the SDK
+                // is the source of truth for so changes in the test code (e.g. a
+                // new `@tag`, an updated overallObjective annotation, additional
+                // `ENV` annotations, a different target URL) flow through to the
+                // persisted record — while leaving Studio-owned fields like
+                // `suiteId` and `nextRunMode` untouched.
+                const existing = await extendedPage._dnb.testsPersistence.getTestById(testInfo.testId);
+                await extendedPage._dnb.testsPersistence.updateTest({
+                    ...existing,
+                    provenance: testMetadata.provenance,
+                    web: testMetadata.web,
+                    envVars: testMetadata.envVars,
+                    overallObjective: testMetadata.overallObjective,
+                    allowedTools: testMetadata.allowedTools,
+                    tags: testMetadata.tags,
+                });
             }
             await extendedPage._dnb.persistence.setFlowMetadata(extendedPage._dnb.donobuFlowMetadata);
             await boundUse(extendedPage);
@@ -846,6 +862,43 @@ async function attachStepScreenshots(sharedState, testInfo) {
         contentType: 'application/json',
     });
 }
+/**
+ * Build the per-flow `donobu-test-result.json` payload the website renders as
+ * the report's summary row + error/skip sections: the test-level scalars
+ * (status, objective, annotations, assertion errors).
+ */
+function buildDonobuTestResult(testInfo) {
+    const objective = testInfo.annotations.find((a) => a.type === 'objective')?.description ??
+        null;
+    return {
+        status: testInfo.status ?? 'unknown',
+        specTitle: testInfo.title,
+        file: path_1.default.relative(testInfo.config.rootDir, testInfo.file),
+        projectName: testInfo.project.name,
+        durationMs: testInfo.duration,
+        objective,
+        annotations: testInfo.annotations,
+        // TestInfoError exposes only message/stack at runtime; the assertion diff
+        // (expected/received) is embedded in the message for expect() failures, and
+        // the stack carries the code frame — enough for the website's error block.
+        errors: (testInfo.errors ?? []).map((e) => ({
+            message: e.message,
+            stack: e.stack,
+        })),
+    };
+}
+/**
+ * Upload a per-flow JSON artifact through the flow's persistence layer.
+ * Best-effort: a failed upload must never affect the test result.
+ */
+async function persistFlowJson(persistence, flowId, fileId, value) {
+    try {
+        await persistence.setFlowFile(flowId, fileId, Buffer.from(JSON.stringify(value), 'utf8'));
+    }
+    catch (error) {
+        Logger_1.appLogger.warn(`Failed to persist report artifact "${fileId}" for flow ${flowId}: ${error.message}`);
+    }
+}
 /**
  * Capture a live screenshot of the flow's final visual state at teardown (page
  * still open) and persist it as a per-flow file — the single source of truth
@@ -902,16 +955,15 @@ async function finalizeTest(page, testInfo, logBuffer, videoOption) {
             body: JSON.stringify(sharedState.donobuFlowMetadata, null, 2),
             contentType: 'application/json',
         });
+        // Persist the per-flow test result (status, objective, annotations, and
+        // assertion errors with snippets/diffs) so the website can render the
+        // report's summary row + error section. Playwright's TestInfoError already
+        // carries the code `snippet`, so no source-file access is needed.
+        await persistFlowJson(sharedState.persistence, sharedState.donobuFlowMetadata.id, 'donobu-test-result.json', buildDonobuTestResult(testInfo));
         // 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);
-            }
+            await persistFlowJson(sharedState.persistence, sharedState.donobuFlowMetadata.id, 'logs.json', logBuffer.snapshot());
         }
         // Attach step-level screenshots from the flow's tool call history.
         // These enable the HTML report to show a visual timeline of what the
@@ -927,6 +979,7 @@ async function finalizeTest(page, testInfo, logBuffer, videoOption) {
                     body: JSON.stringify(nativeSteps),
                     contentType: 'application/json',
                 });
+                await persistFlowJson(sharedState.persistence, sharedState.donobuFlowMetadata.id, 'donobu-native-steps.json', nativeSteps);
             }
         }
         catch {
@@ -945,6 +998,7 @@ async function finalizeTest(page, testInfo, logBuffer, videoOption) {
                     body: JSON.stringify(aiInvocations),
                     contentType: 'application/json',
                 });
+                await persistFlowJson(sharedState.persistence, sharedState.donobuFlowMetadata.id, 'donobu-ai-invocations.json', aiInvocations);
             }
         }
         catch {
@@ -1062,6 +1116,7 @@ function flowMetadataToTestMetadata(testId, flowMeta) {
         resultJsonSchema: flowMeta.resultJsonSchema,
         callbackUrl: flowMeta.callbackUrl,
         maxToolCalls: flowMeta.maxToolCalls,
+        tags: flowMeta.tags,
         suiteId: null,
         nextRunMode: 'DETERMINISTIC',
         provenance: (0, buildProvenance_1.buildProvenance)('CODE'),

package/dist/esm/managers/DonobuFlowsManager.js

@@ -53,6 +53,7 @@ const ToolRequiresGptException_1 = require("../exceptions/ToolRequiresGptExcepti
 const UnknownToolException_1 = require("../exceptions/UnknownToolException");
 const CreateDonobuFlow_1 = require("../models/CreateDonobuFlow");
 const GptConfig_1 = require("../models/GptConfig");
+const MetadataVersion_1 = require("../models/MetadataVersion");
 const resolveTargetRuntime_1 = require("../targets/resolveTargetRuntime");
 const CustomToolRunnerTool_1 = require("../tools/CustomToolRunnerTool");
 const buildProvenance_1 = require("../utils/buildProvenance");
@@ -162,7 +163,7 @@ class DonobuFlowsManager {
                 const flowMetadata = {
                     id: flowId,
                     target: flowParams.target,
-                    metadataVersion: 1,
+                    metadataVersion: MetadataVersion_1.CURRENT_METADATA_VERSION,
                     createdWithDonobuVersion: MiscUtils_1.MiscUtils.DONOBU_VERSION,
                     name: flowParams.name || null,
                     envVars: allowedEnvVarsByName,
@@ -185,6 +186,7 @@ class DonobuFlowsManager {
                     state: 'UNSTARTED',
                     nextState: null,
                     videoDisabled: flowParams.videoDisabled,
+                    tags: [],
                     testId: flowParams.testId ?? null,
                     // Target-specific fields (browser, targetWebsite, isControlPanelEnabled, etc.)
                     ...targetRuntime.getMetadataFields(),

package/dist/esm/managers/PageInspector.js

@@ -933,24 +933,22 @@ class PageInspector {
         if (document.scrollingElement) {
             maybeAddScrollable(document.scrollingElement);
         }
-        // 2) Iterate and assign numbers
-        uniqueElements.forEach((element) => {
-            if (element === document.scrollingElement) {
-                // Special-case: always keep the root scrolling element
-                element.setAttribute(interactableAttribute, offset.toString());
-                offset++;
-                return; // skip the usual checks
-            }
-            else if (element.hasAttribute(interactableAttribute)) {
-                // Skip if this element already carries a value (e.g. assigned via <label>)
-                return;
+        /**
+         * Run the visibility / enabled / top-most checks on a single element and,
+         * if they pass, assign it the next interactable number. Returns `true` if
+         * the element (or, via the <label htmlFor> mapping, its associated control)
+         * was attributed.
+         */
+        function tryAttributeElement(element) {
+            if (element.hasAttribute(interactableAttribute)) {
+                return false;
             }
             const rect = element.getBoundingClientRect();
             const style = window.getComputedStyle(element);
             const visible = isElementVisible(rect, style) && isElementMoreThanHalfInViewport(rect);
             const enabled = isElementEnabled(element, style);
             if (!visible || !enabled) {
-                return;
+                return false;
             }
             // Check a few probe points to make sure the element is top-most
             for (const pt of getPointsToCheck(rect)) {
@@ -959,9 +957,9 @@ class PageInspector {
                     if (elToCheck === element) {
                         element.setAttribute(interactableAttribute, offset.toString());
                         offset++;
-                        return; // this element done
+                        return true; // this element done
                     }
-                    // Handle <label> -> control mapping
+                    // Handle <label> -> control mapping (explicit `for`/`htmlFor`)
                     if (elToCheck.tagName.toLowerCase() === 'label' &&
                         elToCheck.htmlFor) {
                         const forId = elToCheck.htmlFor;
@@ -972,11 +970,41 @@ class PageInspector {
                             control.setAttribute(interactableAttribute, offset.toString());
                             offset++;
                         }
-                        return;
+                        return true;
                     }
                     elToCheck = elToCheck.parentElement;
                 }
             }
+            return false;
+        }
+        // 2) Iterate and assign numbers
+        uniqueElements.forEach((element) => {
+            if (element === document.scrollingElement) {
+                // Special-case: always keep the root scrolling element
+                element.setAttribute(interactableAttribute, offset.toString());
+                offset++;
+                return; // skip the usual checks
+            }
+            else if (element.hasAttribute(interactableAttribute)) {
+                // Skip if this element already carries a value (e.g. assigned via <label>)
+                return;
+            }
+            if (tryAttributeElement(element)) {
+                return;
+            }
+            // Fallback: the element is a visually-hidden native control (e.g. a 0x0,
+            // opacity-0, pointer-events:none <input>) wrapped in a styled <label>.
+            // This is the standard pattern for Ant Design Segmented/Radio/Checkbox/
+            // Switch and many other component libraries: the native input is hidden
+            // and the surrounding <label> is the real clickable surface. Since the
+            // hidden control fails the visibility/enabled checks above, attribute the
+            // wrapping <label> instead so the toggle is still annotated.
+            const wrappingLabel = element.closest('label');
+            if (wrappingLabel &&
+                wrappingLabel !== element &&
+                !wrappingLabel.hasAttribute(interactableAttribute)) {
+                tryAttributeElement(wrappingLabel);
+            }
         });
         return offset;
     }

package/dist/esm/managers/TestsManager.js

@@ -5,6 +5,7 @@ const crypto_1 = require("crypto");
 const CannotDeleteRunningFlowException_1 = require("../exceptions/CannotDeleteRunningFlowException");
 const SuiteNotFoundException_1 = require("../exceptions/SuiteNotFoundException");
 const TestNotFoundException_1 = require("../exceptions/TestNotFoundException");
+const MetadataVersion_1 = require("../models/MetadataVersion");
 const buildProvenance_1 = require("../utils/buildProvenance");
 const displayName_1 = require("../utils/displayName");
 const FederatedPagination_1 = require("./FederatedPagination");
@@ -24,7 +25,7 @@ class TestsManager {
             : undefined;
         const testMetadata = {
             id: testId,
-            metadataVersion: 1,
+            metadataVersion: MetadataVersion_1.CURRENT_METADATA_VERSION,
             name: (0, displayName_1.getDisplayName)({ name: params.name ?? null, web }),
             suiteId: params.suiteId ?? null,
             nextRunMode: params.nextRunMode ?? 'AUTONOMOUS',
@@ -38,6 +39,7 @@ class TestsManager {
             resultJsonSchema: params.resultJsonSchema ?? null,
             maxToolCalls: params.maxToolCalls ?? null,
             videoDisabled: params.videoDisabled,
+            tags: [],
             provenance: (0, buildProvenance_1.buildProvenance)('DONOBU_STUDIO'),
         };
         // If the test is part of a suite, write it to the same persistence layer

package/dist/esm/models/CreateSuite.d.ts

@@ -24,6 +24,7 @@ export declare const CreateSuiteSchema: z.ZodObject<{
     callbackUrl: z.ZodOptional<z.ZodNullable<z.ZodString>>;
     maxToolCalls: z.ZodOptional<z.ZodNullable<z.ZodNumber>>;
     videoDisabled: z.ZodOptional<z.ZodNullable<z.ZodOptional<z.ZodBoolean>>>;
+    tags: z.ZodOptional<z.ZodOptional<z.ZodArray<z.ZodString>>>;
     name: z.ZodString;
     description: z.ZodOptional<z.ZodNullable<z.ZodString>>;
     web: z.ZodOptional<z.ZodObject<{

package/dist/esm/models/CreateTest.d.ts

@@ -23,6 +23,7 @@ export declare const CreateTestSchema: z.ZodObject<{
     callbackUrl: z.ZodOptional<z.ZodNullable<z.ZodString>>;
     maxToolCalls: z.ZodOptional<z.ZodNullable<z.ZodNumber>>;
     videoDisabled: z.ZodOptional<z.ZodNullable<z.ZodOptional<z.ZodBoolean>>>;
+    tags: z.ZodOptional<z.ZodOptional<z.ZodArray<z.ZodString>>>;
     target: z.ZodString;
     web: z.ZodOptional<z.ZodObject<{
         browser: z.ZodOptional<z.ZodNullable<z.ZodObject<{

package/dist/esm/models/FlowMetadata.d.ts

@@ -140,6 +140,7 @@ export declare const FlowMetadataSchema: z.ZodObject<{
     callbackUrl: z.ZodNullable<z.ZodString>;
     maxToolCalls: z.ZodNullable<z.ZodNumber>;
     videoDisabled: z.ZodNullable<z.ZodOptional<z.ZodBoolean>>;
+    tags: z.ZodOptional<z.ZodArray<z.ZodString>>;
     id: z.ZodString;
     metadataVersion: z.ZodOptional<z.ZodNumber>;
     name: z.ZodNullable<z.ZodString>;

package/dist/esm/models/MetadataVersion.d.ts

@@ -0,0 +1,14 @@
+/**
+ * Current metadata schema version. Bump this when the metadata JSON structure
+ * changes in a way that requires read-time normalization.
+ *
+ * Shared by Flow and Test metadata — both schemas carry the field as a single
+ * monotonic counter so a single bump covers a schema change that touches
+ * either or both.
+ *
+ * History:
+ *   v1 — first version to carry `metadataVersion`. Replaced top-level
+ *        `browser` / `targetWebsite` with the `{ target, web }` wrapper.
+ */
+export declare const CURRENT_METADATA_VERSION = 1;
+//# sourceMappingURL=MetadataVersion.d.ts.map

package/dist/esm/models/MetadataVersion.js

@@ -0,0 +1,17 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.CURRENT_METADATA_VERSION = void 0;
+/**
+ * Current metadata schema version. Bump this when the metadata JSON structure
+ * changes in a way that requires read-time normalization.
+ *
+ * Shared by Flow and Test metadata — both schemas carry the field as a single
+ * monotonic counter so a single bump covers a schema change that touches
+ * either or both.
+ *
+ * History:
+ *   v1 — first version to carry `metadataVersion`. Replaced top-level
+ *        `browser` / `targetWebsite` with the `{ target, web }` wrapper.
+ */
+exports.CURRENT_METADATA_VERSION = 1;
+//# sourceMappingURL=MetadataVersion.js.map

package/dist/esm/models/RunConfig.d.ts

@@ -255,6 +255,7 @@ export declare const RunConfigSchema: z.ZodObject<{
     callbackUrl: z.ZodNullable<z.ZodString>;
     maxToolCalls: z.ZodNullable<z.ZodNumber>;
     videoDisabled: z.ZodNullable<z.ZodOptional<z.ZodBoolean>>;
+    tags: z.ZodOptional<z.ZodArray<z.ZodString>>;
 }, z.core.$loose>;
 export type RunConfig = z.infer<typeof RunConfigSchema>;
 export {};

package/dist/esm/models/RunConfig.js

@@ -64,5 +64,13 @@ exports.RunConfigSchema = v4_1.z.looseObject({
         .optional()
         .nullable()
         .describe('If true, do not record a video of the execution.'),
+    tags: v4_1.z
+        .array(v4_1.z.string())
+        .optional()
+        .describe('Tags applied to this run (e.g. "@smoke", "@regression"). For Playwright ' +
+        "tests, populated from Playwright's testInfo.tags — which itself includes " +
+        'both tags parsed from the test name and tags from the `tag` test details object. ' +
+        'An empty array is meaningful: it signals "this run has no tags," which ' +
+        'should override any name-parsed tags that consumers might otherwise infer.'),
 });
 //# sourceMappingURL=RunConfig.js.map

package/dist/esm/models/SuiteMetadata.d.ts

@@ -136,6 +136,7 @@ export declare const SuiteMetadataSchema: z.ZodObject<{
     callbackUrl: z.ZodOptional<z.ZodNullable<z.ZodString>>;
     maxToolCalls: z.ZodOptional<z.ZodNullable<z.ZodNumber>>;
     videoDisabled: z.ZodOptional<z.ZodNullable<z.ZodOptional<z.ZodBoolean>>>;
+    tags: z.ZodOptional<z.ZodOptional<z.ZodArray<z.ZodString>>>;
     id: z.ZodString;
     metadataVersion: z.ZodOptional<z.ZodNumber>;
     name: z.ZodString;

package/dist/esm/models/TestMetadata.d.ts

@@ -133,6 +133,7 @@ export declare const TestMetadataSchema: z.ZodObject<{
     callbackUrl: z.ZodNullable<z.ZodString>;
     maxToolCalls: z.ZodNullable<z.ZodNumber>;
     videoDisabled: z.ZodNullable<z.ZodOptional<z.ZodBoolean>>;
+    tags: z.ZodOptional<z.ZodArray<z.ZodString>>;
     id: z.ZodString;
     metadataVersion: z.ZodOptional<z.ZodNumber>;
     name: z.ZodNullable<z.ZodString>;
@@ -315,6 +316,7 @@ export declare const TestListItemSchema: z.ZodObject<{
     callbackUrl: z.ZodNullable<z.ZodString>;
     maxToolCalls: z.ZodNullable<z.ZodNumber>;
     videoDisabled: z.ZodNullable<z.ZodOptional<z.ZodBoolean>>;
+    tags: z.ZodOptional<z.ZodArray<z.ZodString>>;
     id: z.ZodString;
     metadataVersion: z.ZodOptional<z.ZodNumber>;
     name: z.ZodNullable<z.ZodString>;
@@ -473,6 +475,7 @@ export declare const TestListItemSchema: z.ZodObject<{
         callbackUrl: z.ZodNullable<z.ZodString>;
         maxToolCalls: z.ZodNullable<z.ZodNumber>;
         videoDisabled: z.ZodNullable<z.ZodOptional<z.ZodBoolean>>;
+        tags: z.ZodOptional<z.ZodArray<z.ZodString>>;
         id: z.ZodString;
         metadataVersion: z.ZodOptional<z.ZodNumber>;
         name: z.ZodNullable<z.ZodString>;
@@ -665,6 +668,7 @@ export declare const TestListItemPaginatedResultSchema: z.ZodObject<{
         callbackUrl: z.ZodNullable<z.ZodString>;
         maxToolCalls: z.ZodNullable<z.ZodNumber>;
         videoDisabled: z.ZodNullable<z.ZodOptional<z.ZodBoolean>>;
+        tags: z.ZodOptional<z.ZodArray<z.ZodString>>;
         id: z.ZodString;
         metadataVersion: z.ZodOptional<z.ZodNumber>;
         name: z.ZodNullable<z.ZodString>;
@@ -823,6 +827,7 @@ export declare const TestListItemPaginatedResultSchema: z.ZodObject<{
             callbackUrl: z.ZodNullable<z.ZodString>;
             maxToolCalls: z.ZodNullable<z.ZodNumber>;
             videoDisabled: z.ZodNullable<z.ZodOptional<z.ZodBoolean>>;
+            tags: z.ZodOptional<z.ZodArray<z.ZodString>>;
             id: z.ZodString;
             metadataVersion: z.ZodOptional<z.ZodNumber>;
             name: z.ZodNullable<z.ZodString>;

package/dist/esm/persistence/normalizeFlowMetadata.d.ts

@@ -1,9 +1,4 @@
 import type { FlowMetadata } from '../models/FlowMetadata';
-/**
- * Current metadata schema version. Bump this when the metadata JSON structure
- * changes in a way that requires read-time normalization.
- */
-export declare const CURRENT_METADATA_VERSION = 1;
 /**
  * Normalizes flow metadata written by older SDK versions to the current schema.
  *

package/dist/esm/persistence/normalizeFlowMetadata.js

@@ -1,12 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.CURRENT_METADATA_VERSION = void 0;
 exports.normalizeFlowMetadata = normalizeFlowMetadata;
-/**
- * Current metadata schema version. Bump this when the metadata JSON structure
- * changes in a way that requires read-time normalization.
- */
-exports.CURRENT_METADATA_VERSION = 1;
+const MetadataVersion_1 = require("../models/MetadataVersion");
 /**
  * Normalizes flow metadata written by older SDK versions to the current schema.
  *
@@ -16,7 +11,7 @@ exports.CURRENT_METADATA_VERSION = 1;
  * introduced in migration v9).
  */
 function normalizeFlowMetadata(raw) {
-    if (raw.metadataVersion === exports.CURRENT_METADATA_VERSION) {
+    if (raw.metadataVersion === MetadataVersion_1.CURRENT_METADATA_VERSION) {
         return raw;
     }
     // Pre-v5 format: `browser` and `targetWebsite` at the top level, no `target`.

package/dist/lib/page/extendPage.js

@@ -15,6 +15,7 @@ const InteractionVisualizer_1 = require("../../managers/InteractionVisualizer");
 const ToolManager_1 = require("../../managers/ToolManager");
 const WebTargetInspector_1 = require("../../managers/WebTargetInspector");
 const ControlPanel_1 = require("../../models/ControlPanel");
+const MetadataVersion_1 = require("../../models/MetadataVersion");
 const AnalyzePageTextTool_1 = require("../../tools/AnalyzePageTextTool");
 const AssertTool_1 = require("../../tools/AssertTool");
 const AuditTool_1 = require("../../tools/AuditTool");
@@ -88,7 +89,7 @@ async function extendPage(page, options) {
         const sharedState = {
             donobuFlowMetadata: {
                 id: options?.flowId ?? (0, crypto_1.randomUUID)(),
-                metadataVersion: 1,
+                metadataVersion: MetadataVersion_1.CURRENT_METADATA_VERSION,
                 name: null,
                 createdWithDonobuVersion: MiscUtils_1.MiscUtils.DONOBU_VERSION,
                 target: 'web',
@@ -104,6 +105,7 @@ async function extendPage(page, options) {
                     targetWebsite: PLACEHOLDER_FLOW_URL,
                 },
                 envVars: options?.envVars ?? null,
+                tags: [],
                 gptConfigName: null,
                 hasGptConfigNameOverride: false,
                 customTools: null,

package/dist/lib/test/testExtension.js

@@ -409,6 +409,7 @@ exports.test = test_1.test.extend({
         extendedPage._dnb.donobuFlowMetadata.name = getSanitizedTestName(testInfo);
         extendedPage._dnb.donobuFlowMetadata.testId = testInfo.testId;
         extendedPage._dnb.donobuFlowMetadata.overallObjective = overallObjective;
+        extendedPage._dnb.donobuFlowMetadata.tags = testInfo.tags;
         // 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
@@ -421,12 +422,27 @@ exports.test = test_1.test.extend({
         const asyncScope = new async_hooks_1.AsyncResource('DonobuTestContext');
         const boundUse = asyncScope.bind(use);
         try {
+            const testMetadata = flowMetadataToTestMetadata(testInfo.testId, extendedPage._dnb.donobuFlowMetadata);
             try {
-                const testMetadata = flowMetadataToTestMetadata(testInfo.testId, extendedPage._dnb.donobuFlowMetadata);
                 await extendedPage._dnb.testsPersistence.createTest(testMetadata);
             }
             catch {
-                // Test already exists from a prior run — nothing to do.
+                // Test already exists from a prior run. Refresh the fields the SDK
+                // is the source of truth for so changes in the test code (e.g. a
+                // new `@tag`, an updated overallObjective annotation, additional
+                // `ENV` annotations, a different target URL) flow through to the
+                // persisted record — while leaving Studio-owned fields like
+                // `suiteId` and `nextRunMode` untouched.
+                const existing = await extendedPage._dnb.testsPersistence.getTestById(testInfo.testId);
+                await extendedPage._dnb.testsPersistence.updateTest({
+                    ...existing,
+                    provenance: testMetadata.provenance,
+                    web: testMetadata.web,
+                    envVars: testMetadata.envVars,
+                    overallObjective: testMetadata.overallObjective,
+                    allowedTools: testMetadata.allowedTools,
+                    tags: testMetadata.tags,
+                });
             }
             await extendedPage._dnb.persistence.setFlowMetadata(extendedPage._dnb.donobuFlowMetadata);
             await boundUse(extendedPage);
@@ -846,6 +862,43 @@ async function attachStepScreenshots(sharedState, testInfo) {
         contentType: 'application/json',
     });
 }
+/**
+ * Build the per-flow `donobu-test-result.json` payload the website renders as
+ * the report's summary row + error/skip sections: the test-level scalars
+ * (status, objective, annotations, assertion errors).
+ */
+function buildDonobuTestResult(testInfo) {
+    const objective = testInfo.annotations.find((a) => a.type === 'objective')?.description ??
+        null;
+    return {
+        status: testInfo.status ?? 'unknown',
+        specTitle: testInfo.title,
+        file: path_1.default.relative(testInfo.config.rootDir, testInfo.file),
+        projectName: testInfo.project.name,
+        durationMs: testInfo.duration,
+        objective,
+        annotations: testInfo.annotations,
+        // TestInfoError exposes only message/stack at runtime; the assertion diff
+        // (expected/received) is embedded in the message for expect() failures, and
+        // the stack carries the code frame — enough for the website's error block.
+        errors: (testInfo.errors ?? []).map((e) => ({
+            message: e.message,
+            stack: e.stack,
+        })),
+    };
+}
+/**
+ * Upload a per-flow JSON artifact through the flow's persistence layer.
+ * Best-effort: a failed upload must never affect the test result.
+ */
+async function persistFlowJson(persistence, flowId, fileId, value) {
+    try {
+        await persistence.setFlowFile(flowId, fileId, Buffer.from(JSON.stringify(value), 'utf8'));
+    }
+    catch (error) {
+        Logger_1.appLogger.warn(`Failed to persist report artifact "${fileId}" for flow ${flowId}: ${error.message}`);
+    }
+}
 /**
  * Capture a live screenshot of the flow's final visual state at teardown (page
  * still open) and persist it as a per-flow file — the single source of truth
@@ -902,16 +955,15 @@ async function finalizeTest(page, testInfo, logBuffer, videoOption) {
             body: JSON.stringify(sharedState.donobuFlowMetadata, null, 2),
             contentType: 'application/json',
         });
+        // Persist the per-flow test result (status, objective, annotations, and
+        // assertion errors with snippets/diffs) so the website can render the
+        // report's summary row + error section. Playwright's TestInfoError already
+        // carries the code `snippet`, so no source-file access is needed.
+        await persistFlowJson(sharedState.persistence, sharedState.donobuFlowMetadata.id, 'donobu-test-result.json', buildDonobuTestResult(testInfo));
         // 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);
-            }
+            await persistFlowJson(sharedState.persistence, sharedState.donobuFlowMetadata.id, 'logs.json', logBuffer.snapshot());
         }
         // Attach step-level screenshots from the flow's tool call history.
         // These enable the HTML report to show a visual timeline of what the
@@ -927,6 +979,7 @@ async function finalizeTest(page, testInfo, logBuffer, videoOption) {
                     body: JSON.stringify(nativeSteps),
                     contentType: 'application/json',
                 });
+                await persistFlowJson(sharedState.persistence, sharedState.donobuFlowMetadata.id, 'donobu-native-steps.json', nativeSteps);
             }
         }
         catch {
@@ -945,6 +998,7 @@ async function finalizeTest(page, testInfo, logBuffer, videoOption) {
                     body: JSON.stringify(aiInvocations),
                     contentType: 'application/json',
                 });
+                await persistFlowJson(sharedState.persistence, sharedState.donobuFlowMetadata.id, 'donobu-ai-invocations.json', aiInvocations);
             }
         }
         catch {
@@ -1062,6 +1116,7 @@ function flowMetadataToTestMetadata(testId, flowMeta) {
         resultJsonSchema: flowMeta.resultJsonSchema,
         callbackUrl: flowMeta.callbackUrl,
         maxToolCalls: flowMeta.maxToolCalls,
+        tags: flowMeta.tags,
         suiteId: null,
         nextRunMode: 'DETERMINISTIC',
         provenance: (0, buildProvenance_1.buildProvenance)('CODE'),

package/dist/managers/DonobuFlowsManager.js

@@ -53,6 +53,7 @@ const ToolRequiresGptException_1 = require("../exceptions/ToolRequiresGptExcepti
 const UnknownToolException_1 = require("../exceptions/UnknownToolException");
 const CreateDonobuFlow_1 = require("../models/CreateDonobuFlow");
 const GptConfig_1 = require("../models/GptConfig");
+const MetadataVersion_1 = require("../models/MetadataVersion");
 const resolveTargetRuntime_1 = require("../targets/resolveTargetRuntime");
 const CustomToolRunnerTool_1 = require("../tools/CustomToolRunnerTool");
 const buildProvenance_1 = require("../utils/buildProvenance");
@@ -162,7 +163,7 @@ class DonobuFlowsManager {
                 const flowMetadata = {
                     id: flowId,
                     target: flowParams.target,
-                    metadataVersion: 1,
+                    metadataVersion: MetadataVersion_1.CURRENT_METADATA_VERSION,
                     createdWithDonobuVersion: MiscUtils_1.MiscUtils.DONOBU_VERSION,
                     name: flowParams.name || null,
                     envVars: allowedEnvVarsByName,
@@ -185,6 +186,7 @@ class DonobuFlowsManager {
                     state: 'UNSTARTED',
                     nextState: null,
                     videoDisabled: flowParams.videoDisabled,
+                    tags: [],
                     testId: flowParams.testId ?? null,
                     // Target-specific fields (browser, targetWebsite, isControlPanelEnabled, etc.)
                     ...targetRuntime.getMetadataFields(),

package/dist/managers/PageInspector.js

@@ -933,24 +933,22 @@ class PageInspector {
         if (document.scrollingElement) {
             maybeAddScrollable(document.scrollingElement);
         }
-        // 2) Iterate and assign numbers
-        uniqueElements.forEach((element) => {
-            if (element === document.scrollingElement) {
-                // Special-case: always keep the root scrolling element
-                element.setAttribute(interactableAttribute, offset.toString());
-                offset++;
-                return; // skip the usual checks
-            }
-            else if (element.hasAttribute(interactableAttribute)) {
-                // Skip if this element already carries a value (e.g. assigned via <label>)
-                return;
+        /**
+         * Run the visibility / enabled / top-most checks on a single element and,
+         * if they pass, assign it the next interactable number. Returns `true` if
+         * the element (or, via the <label htmlFor> mapping, its associated control)
+         * was attributed.
+         */
+        function tryAttributeElement(element) {
+            if (element.hasAttribute(interactableAttribute)) {
+                return false;
             }
             const rect = element.getBoundingClientRect();
             const style = window.getComputedStyle(element);
             const visible = isElementVisible(rect, style) && isElementMoreThanHalfInViewport(rect);
             const enabled = isElementEnabled(element, style);
             if (!visible || !enabled) {
-                return;
+                return false;
             }
             // Check a few probe points to make sure the element is top-most
             for (const pt of getPointsToCheck(rect)) {
@@ -959,9 +957,9 @@ class PageInspector {
                     if (elToCheck === element) {
                         element.setAttribute(interactableAttribute, offset.toString());
                         offset++;
-                        return; // this element done
+                        return true; // this element done
                     }
-                    // Handle <label> -> control mapping
+                    // Handle <label> -> control mapping (explicit `for`/`htmlFor`)
                     if (elToCheck.tagName.toLowerCase() === 'label' &&
                         elToCheck.htmlFor) {
                         const forId = elToCheck.htmlFor;
@@ -972,11 +970,41 @@ class PageInspector {
                             control.setAttribute(interactableAttribute, offset.toString());
                             offset++;
                         }
-                        return;
+                        return true;
                     }
                     elToCheck = elToCheck.parentElement;
                 }
             }
+            return false;
+        }
+        // 2) Iterate and assign numbers
+        uniqueElements.forEach((element) => {
+            if (element === document.scrollingElement) {
+                // Special-case: always keep the root scrolling element
+                element.setAttribute(interactableAttribute, offset.toString());
+                offset++;
+                return; // skip the usual checks
+            }
+            else if (element.hasAttribute(interactableAttribute)) {
+                // Skip if this element already carries a value (e.g. assigned via <label>)
+                return;
+            }
+            if (tryAttributeElement(element)) {
+                return;
+            }
+            // Fallback: the element is a visually-hidden native control (e.g. a 0x0,
+            // opacity-0, pointer-events:none <input>) wrapped in a styled <label>.
+            // This is the standard pattern for Ant Design Segmented/Radio/Checkbox/
+            // Switch and many other component libraries: the native input is hidden
+            // and the surrounding <label> is the real clickable surface. Since the
+            // hidden control fails the visibility/enabled checks above, attribute the
+            // wrapping <label> instead so the toggle is still annotated.
+            const wrappingLabel = element.closest('label');
+            if (wrappingLabel &&
+                wrappingLabel !== element &&
+                !wrappingLabel.hasAttribute(interactableAttribute)) {
+                tryAttributeElement(wrappingLabel);
+            }
         });
         return offset;
     }

package/dist/managers/TestsManager.js

@@ -5,6 +5,7 @@ const crypto_1 = require("crypto");
 const CannotDeleteRunningFlowException_1 = require("../exceptions/CannotDeleteRunningFlowException");
 const SuiteNotFoundException_1 = require("../exceptions/SuiteNotFoundException");
 const TestNotFoundException_1 = require("../exceptions/TestNotFoundException");
+const MetadataVersion_1 = require("../models/MetadataVersion");
 const buildProvenance_1 = require("../utils/buildProvenance");
 const displayName_1 = require("../utils/displayName");
 const FederatedPagination_1 = require("./FederatedPagination");
@@ -24,7 +25,7 @@ class TestsManager {
             : undefined;
         const testMetadata = {
             id: testId,
-            metadataVersion: 1,
+            metadataVersion: MetadataVersion_1.CURRENT_METADATA_VERSION,
             name: (0, displayName_1.getDisplayName)({ name: params.name ?? null, web }),
             suiteId: params.suiteId ?? null,
             nextRunMode: params.nextRunMode ?? 'AUTONOMOUS',
@@ -38,6 +39,7 @@ class TestsManager {
             resultJsonSchema: params.resultJsonSchema ?? null,
             maxToolCalls: params.maxToolCalls ?? null,
             videoDisabled: params.videoDisabled,
+            tags: [],
             provenance: (0, buildProvenance_1.buildProvenance)('DONOBU_STUDIO'),
         };
         // If the test is part of a suite, write it to the same persistence layer

package/dist/models/CreateSuite.d.ts

@@ -24,6 +24,7 @@ export declare const CreateSuiteSchema: z.ZodObject<{
     callbackUrl: z.ZodOptional<z.ZodNullable<z.ZodString>>;
     maxToolCalls: z.ZodOptional<z.ZodNullable<z.ZodNumber>>;
     videoDisabled: z.ZodOptional<z.ZodNullable<z.ZodOptional<z.ZodBoolean>>>;
+    tags: z.ZodOptional<z.ZodOptional<z.ZodArray<z.ZodString>>>;
     name: z.ZodString;
     description: z.ZodOptional<z.ZodNullable<z.ZodString>>;
     web: z.ZodOptional<z.ZodObject<{

package/dist/models/CreateTest.d.ts

@@ -23,6 +23,7 @@ export declare const CreateTestSchema: z.ZodObject<{
     callbackUrl: z.ZodOptional<z.ZodNullable<z.ZodString>>;
     maxToolCalls: z.ZodOptional<z.ZodNullable<z.ZodNumber>>;
     videoDisabled: z.ZodOptional<z.ZodNullable<z.ZodOptional<z.ZodBoolean>>>;
+    tags: z.ZodOptional<z.ZodOptional<z.ZodArray<z.ZodString>>>;
     target: z.ZodString;
     web: z.ZodOptional<z.ZodObject<{
         browser: z.ZodOptional<z.ZodNullable<z.ZodObject<{

package/dist/models/FlowMetadata.d.ts

@@ -140,6 +140,7 @@ export declare const FlowMetadataSchema: z.ZodObject<{
     callbackUrl: z.ZodNullable<z.ZodString>;
     maxToolCalls: z.ZodNullable<z.ZodNumber>;
     videoDisabled: z.ZodNullable<z.ZodOptional<z.ZodBoolean>>;
+    tags: z.ZodOptional<z.ZodArray<z.ZodString>>;
     id: z.ZodString;
     metadataVersion: z.ZodOptional<z.ZodNumber>;
     name: z.ZodNullable<z.ZodString>;

package/dist/models/MetadataVersion.d.ts

@@ -0,0 +1,14 @@
+/**
+ * Current metadata schema version. Bump this when the metadata JSON structure
+ * changes in a way that requires read-time normalization.
+ *
+ * Shared by Flow and Test metadata — both schemas carry the field as a single
+ * monotonic counter so a single bump covers a schema change that touches
+ * either or both.
+ *
+ * History:
+ *   v1 — first version to carry `metadataVersion`. Replaced top-level
+ *        `browser` / `targetWebsite` with the `{ target, web }` wrapper.
+ */
+export declare const CURRENT_METADATA_VERSION = 1;
+//# sourceMappingURL=MetadataVersion.d.ts.map

package/dist/models/MetadataVersion.js

@@ -0,0 +1,17 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.CURRENT_METADATA_VERSION = void 0;
+/**
+ * Current metadata schema version. Bump this when the metadata JSON structure
+ * changes in a way that requires read-time normalization.
+ *
+ * Shared by Flow and Test metadata — both schemas carry the field as a single
+ * monotonic counter so a single bump covers a schema change that touches
+ * either or both.
+ *
+ * History:
+ *   v1 — first version to carry `metadataVersion`. Replaced top-level
+ *        `browser` / `targetWebsite` with the `{ target, web }` wrapper.
+ */
+exports.CURRENT_METADATA_VERSION = 1;
+//# sourceMappingURL=MetadataVersion.js.map

package/dist/models/RunConfig.d.ts

@@ -255,6 +255,7 @@ export declare const RunConfigSchema: z.ZodObject<{
     callbackUrl: z.ZodNullable<z.ZodString>;
     maxToolCalls: z.ZodNullable<z.ZodNumber>;
     videoDisabled: z.ZodNullable<z.ZodOptional<z.ZodBoolean>>;
+    tags: z.ZodOptional<z.ZodArray<z.ZodString>>;
 }, z.core.$loose>;
 export type RunConfig = z.infer<typeof RunConfigSchema>;
 export {};

package/dist/models/RunConfig.js

@@ -64,5 +64,13 @@ exports.RunConfigSchema = v4_1.z.looseObject({
         .optional()
         .nullable()
         .describe('If true, do not record a video of the execution.'),
+    tags: v4_1.z
+        .array(v4_1.z.string())
+        .optional()
+        .describe('Tags applied to this run (e.g. "@smoke", "@regression"). For Playwright ' +
+        "tests, populated from Playwright's testInfo.tags — which itself includes " +
+        'both tags parsed from the test name and tags from the `tag` test details object. ' +
+        'An empty array is meaningful: it signals "this run has no tags," which ' +
+        'should override any name-parsed tags that consumers might otherwise infer.'),
 });
 //# sourceMappingURL=RunConfig.js.map

package/dist/models/SuiteMetadata.d.ts

@@ -136,6 +136,7 @@ export declare const SuiteMetadataSchema: z.ZodObject<{
     callbackUrl: z.ZodOptional<z.ZodNullable<z.ZodString>>;
     maxToolCalls: z.ZodOptional<z.ZodNullable<z.ZodNumber>>;
     videoDisabled: z.ZodOptional<z.ZodNullable<z.ZodOptional<z.ZodBoolean>>>;
+    tags: z.ZodOptional<z.ZodOptional<z.ZodArray<z.ZodString>>>;
     id: z.ZodString;
     metadataVersion: z.ZodOptional<z.ZodNumber>;
     name: z.ZodString;

package/dist/models/TestMetadata.d.ts

@@ -133,6 +133,7 @@ export declare const TestMetadataSchema: z.ZodObject<{
     callbackUrl: z.ZodNullable<z.ZodString>;
     maxToolCalls: z.ZodNullable<z.ZodNumber>;
     videoDisabled: z.ZodNullable<z.ZodOptional<z.ZodBoolean>>;
+    tags: z.ZodOptional<z.ZodArray<z.ZodString>>;
     id: z.ZodString;
     metadataVersion: z.ZodOptional<z.ZodNumber>;
     name: z.ZodNullable<z.ZodString>;
@@ -315,6 +316,7 @@ export declare const TestListItemSchema: z.ZodObject<{
     callbackUrl: z.ZodNullable<z.ZodString>;
     maxToolCalls: z.ZodNullable<z.ZodNumber>;
     videoDisabled: z.ZodNullable<z.ZodOptional<z.ZodBoolean>>;
+    tags: z.ZodOptional<z.ZodArray<z.ZodString>>;
     id: z.ZodString;
     metadataVersion: z.ZodOptional<z.ZodNumber>;
     name: z.ZodNullable<z.ZodString>;
@@ -473,6 +475,7 @@ export declare const TestListItemSchema: z.ZodObject<{
         callbackUrl: z.ZodNullable<z.ZodString>;
         maxToolCalls: z.ZodNullable<z.ZodNumber>;
         videoDisabled: z.ZodNullable<z.ZodOptional<z.ZodBoolean>>;
+        tags: z.ZodOptional<z.ZodArray<z.ZodString>>;
         id: z.ZodString;
         metadataVersion: z.ZodOptional<z.ZodNumber>;
         name: z.ZodNullable<z.ZodString>;
@@ -665,6 +668,7 @@ export declare const TestListItemPaginatedResultSchema: z.ZodObject<{
         callbackUrl: z.ZodNullable<z.ZodString>;
         maxToolCalls: z.ZodNullable<z.ZodNumber>;
         videoDisabled: z.ZodNullable<z.ZodOptional<z.ZodBoolean>>;
+        tags: z.ZodOptional<z.ZodArray<z.ZodString>>;
         id: z.ZodString;
         metadataVersion: z.ZodOptional<z.ZodNumber>;
         name: z.ZodNullable<z.ZodString>;
@@ -823,6 +827,7 @@ export declare const TestListItemPaginatedResultSchema: z.ZodObject<{
             callbackUrl: z.ZodNullable<z.ZodString>;
             maxToolCalls: z.ZodNullable<z.ZodNumber>;
             videoDisabled: z.ZodNullable<z.ZodOptional<z.ZodBoolean>>;
+            tags: z.ZodOptional<z.ZodArray<z.ZodString>>;
             id: z.ZodString;
             metadataVersion: z.ZodOptional<z.ZodNumber>;
             name: z.ZodNullable<z.ZodString>;

package/dist/persistence/normalizeFlowMetadata.d.ts

@@ -1,9 +1,4 @@
 import type { FlowMetadata } from '../models/FlowMetadata';
-/**
- * Current metadata schema version. Bump this when the metadata JSON structure
- * changes in a way that requires read-time normalization.
- */
-export declare const CURRENT_METADATA_VERSION = 1;
 /**
  * Normalizes flow metadata written by older SDK versions to the current schema.
  *

package/dist/persistence/normalizeFlowMetadata.js

@@ -1,12 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.CURRENT_METADATA_VERSION = void 0;
 exports.normalizeFlowMetadata = normalizeFlowMetadata;
-/**
- * Current metadata schema version. Bump this when the metadata JSON structure
- * changes in a way that requires read-time normalization.
- */
-exports.CURRENT_METADATA_VERSION = 1;
+const MetadataVersion_1 = require("../models/MetadataVersion");
 /**
  * Normalizes flow metadata written by older SDK versions to the current schema.
  *
@@ -16,7 +11,7 @@ exports.CURRENT_METADATA_VERSION = 1;
  * introduced in migration v9).
  */
 function normalizeFlowMetadata(raw) {
-    if (raw.metadataVersion === exports.CURRENT_METADATA_VERSION) {
+    if (raw.metadataVersion === MetadataVersion_1.CURRENT_METADATA_VERSION) {
         return raw;
     }
     // Pre-v5 format: `browser` and `targetWebsite` at the top level, no `target`.

package/package.json

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