@mablhq/mabl-cli 2.105.4 → 2.105.14

package/index.d.ts

@@ -2,6 +2,12 @@ import {BrowserContext, Locator, Page, Frame} from 'playwright-core';
 import {TestType, Download} from '@playwright/test';
 
 declare module '@mablhq/mabl-cli' {
+  export type SnippetLogLevel = 'info' | 'warn' | 'error';
+  export interface SnippetLogLine {
+    message: string;
+    level: SnippetLogLevel;
+  }
+
   export type AuthServerPolicy = 'off' | 'none' | 'all';
   export enum TestResultStatus {
     passed = 'passed',
@@ -299,6 +305,12 @@ export type AgenticGenerationOptions =
 export interface AgenticGenerationResult {
   success: boolean;
   error?: string;
+  /**
+   * The original error when an unexpected infrastructure failure occurred
+   * (e.g., browser failed to launch, API returned 5xx during setup).
+   * Callers can rethrow this to signal a fatal job failure for monitoring.
+   */
+  fatalError?: Error;
   steps: string[];
   trainingSessionId: string;
   rawSteps: any[];

package/mablscript/MablStep.js

@@ -119,7 +119,9 @@ class MablStep extends MablAction_1.MablAction {
     }
     toLineDiffFormat(_omitStepId = false) {
         const lineDiffProperties = {
-            annotation: this.annotation,
+            annotation: this.annotation && Object.keys(this.annotation).length > 0
+                ? this.annotation
+                : undefined,
             disabled: this.isDisabled(),
             mablscript: this.toMablscript(),
         };

package/mablscript/MablStepV2.js

@@ -85,7 +85,7 @@ class MablStepV2 extends MablStep_1.MablStep {
         return false;
     }
     getResolvedDescription() {
-        return this.annotation?.description ?? this.getDescription();
+        return this.annotation?.description || this.getDescription();
     }
     toLineDiffFormat(omitStepId = false) {
         const descriptor = { ...this.toStepDescriptor() };
@@ -93,7 +93,9 @@ class MablStepV2 extends MablStep_1.MablStep {
             delete descriptor.id;
         }
         const lineDiffProperties = {
-            annotation: this.annotation,
+            annotation: this.annotation && Object.keys(this.annotation).length > 0
+                ? this.annotation
+                : undefined,
             disabled: this.isDisabled(),
             descriptor,
         };

package/mablscript/actions/FindAction.js

@@ -130,8 +130,11 @@ class FindAction extends MablAction_1.MablAction {
             if (findType === 'find_cookie') {
                 return new FindAction(findType, [findTarget]);
             }
+            const primarySelector = findTarget && typeof findTarget === 'object' && 'selector' in findTarget
+                ? findTarget.selector
+                : findTarget;
             return new FindAction(findType, [
-                findTarget,
+                primarySelector,
                 stepArgs.find.foundElementSelectors,
                 findOptions,
             ]);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mablhq/mabl-cli",
-  "version": "2.105.4",
+  "version": "2.105.14",
   "license": "SEE LICENSE IN LICENSE.txt",
   "description": "The official mabl command line interface tool",
   "main": "index.js",

package/trainer/cli/runAgenticGeneration.js

@@ -193,6 +193,7 @@ async function runAgenticGeneration(options) {
             steps: result.steps,
             trainingSessionId: result.trainingSessionId,
             error: result.error,
+            fatalError: result.fatalError,
             rawSteps: result.rawSteps,
             saveResult: result.saveResult,
         };

package/trainer/generation/AgenticGenerationRunner.js

@@ -110,7 +110,7 @@ class AgenticGenerationRunner {
                         }
                         : undefined;
             if (!pendingToolCall) {
-                this.logger.info('No pending tool call to finalize — session may have ended cleanly');
+                await this.resolveOrphanedToolCalls(sessionId, loopResult, saveResult);
                 return;
             }
             const savedSuccessfully = saveResult?.success && saveResult.testId;
@@ -148,6 +148,48 @@ class AgenticGenerationRunner {
             this.logger.error('Failed to finalize agent session', error instanceof Error ? error : undefined);
         }
     }
+    async resolveOrphanedToolCalls(sessionId, loopResult, saveResult) {
+        const session = await this.config.apiClient.getAgentSession(sessionId);
+        const allToolCalls = (0, messageConversion_1.extractToolCallsFromMessages)(session.messages, this.logger);
+        const clientToolCalls = (0, messageConversion_1.filterClientSideToolCalls)(allToolCalls);
+        const unansweredToolCalls = (0, messageConversion_1.filterToolCallsWithoutOutputs)(clientToolCalls, session.messages);
+        const savedSuccessfully = saveResult?.success && saveResult.testId;
+        const artifacts = [];
+        if (savedSuccessfully) {
+            artifacts.push({
+                artifact_type: 'remote',
+                entity_kind: mablApi_1.ArtifactEntityKind.Test,
+                entity_id: saveResult.testId,
+                entity_name: saveResult.savedTest?.name,
+            });
+        }
+        if (unansweredToolCalls.length > 0) {
+            this.logger.info(`Found ${unansweredToolCalls.length} unanswered tool call(s) in session — responding with error`);
+            const errorMessage = savedSuccessfully
+                ? 'Test saved'
+                : loopResult.failureReason ?? 'Generation failed';
+            const toolOutputParts = unansweredToolCalls.map((tc) => (0, messageConversion_1.createToolOutputMessagePart)({
+                toolCallId: tc.id,
+                toolName: tc.name,
+                result: {
+                    success: !!savedSuccessfully,
+                    testId: saveResult?.testId,
+                    testUrl: saveResult?.testUrl,
+                    message: errorMessage,
+                },
+            }));
+            const toolOutputMessage = (0, messageConversion_1.createToolOutputMessage)(toolOutputParts);
+            await this.config.apiClient.appendAgentSessionMessage(sessionId, toolOutputMessage, artifacts.length > 0 ? artifacts : undefined);
+        }
+        else if (artifacts.length > 0) {
+            this.logger.info(`No pending tool calls — associating saved test ${saveResult.testId} with session ${sessionId}`);
+            const message = (0, messageConversion_1.createErrorTextMessage)(loopResult.failureReason ?? 'Generation ended');
+            await this.config.apiClient.appendAgentSessionMessage(sessionId, message, artifacts);
+        }
+        else {
+            this.logger.info('No pending tool calls to finalize — session may have ended cleanly');
+        }
+    }
     async createBrowserAndNavigate() {
         this.logger.info('Creating browser and navigating to URL');
         const { testContext, accountFeatures } = await (0, createTrainingTestContext_1.createTrainingTestContext)({
@@ -339,6 +381,7 @@ class AgenticGenerationRunner {
         return {
             success: false,
             error: errorMessage,
+            fatalError: error instanceof Error ? error : new Error(errorMessage),
             steps: [],
             trainingSessionId: this.state?.getTrainingSessionId() ?? '',
             status: generation_types_1.TestGenerationStatus.Failed,

package/trainer/sharedUtils/mablscript/MablscriptImporter.js

@@ -425,7 +425,11 @@ class MablscriptImporter {
         }
     }
     convertStepGroupStep(mablStepGroup, isMobileFlow) {
-        const convertedSteps = mablStepGroup.steps.map((nestedMablStep) => this.convertMablStepToTrainerStep(nestedMablStep, isMobileFlow));
+        const convertedSteps = mablStepGroup.steps.map((nestedMablStep) => {
+            const step = this.convertMablStepToTrainerStep(nestedMablStep, isMobileFlow);
+            step.setDisabled(nestedMablStep.isDisabled());
+            return step;
+        });
         const intentDescription = mablStepGroup.intent?.description;
         const stepGroup = new ExecuteStepGroupStep_1.default(convertedSteps, intentDescription);
         const existingStepId = mablStepGroup.stepId();

package/trainer/sharedUtils/mablscript/stepSerialization.js

@@ -26,6 +26,7 @@ function parseStepToV2Format(step, isMablscriptFlow, variables) {
                 },
                 actionCode: StepGroupActionCode,
                 id: stepId,
+                ...(step.disabled ? { disabled: true } : {}),
             },
         };
     }