@mablhq/mabl-cli 2.99.12 → 2.100.0

package/commands/datatables/datatables_cmds/update.js

@@ -1,12 +1,13 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.buildUpdateRequest = buildUpdateRequest;
+exports.validateScenarios = validateScenarios;
 const mablApiClientFactory_1 = require("../../../api/mablApiClientFactory");
 const util_1 = require("../../commandUtil/util");
 const constants_1 = require("../../constants");
-const pureUtil_1 = require("../../../util/pureUtil");
 const utils_1 = require("../utils");
 const domUtil_1 = require("../../../domUtil");
+const logUtils_1 = require("../../../util/logUtils");
+const pureUtil_1 = require("../../../util/pureUtil");
 exports.command = `update <${constants_1.CommandArgId}> <${constants_1.CommandArgInput}>`;
 exports.describe = 'Update a mabl DataTable';
 exports.builder = (yargs) => {
@@ -32,27 +33,8 @@ exports.builder = (yargs) => {
     });
 };
 exports.handler = (0, util_1.failWrapper)(updateDataTable);
-async function updateDataTable(parsed) {
-    const { id: dataTableId, input } = parsed;
-    const format = parsed.format ?? (0, utils_1.inferFileFormat)(input);
-    const apiClient = await mablApiClientFactory_1.MablApiClientFactory.createApiClient();
-    const dataTable = await apiClient.getDataTable(dataTableId);
-    let scenarios;
-    switch (format) {
-        case constants_1.OutputFormats.Json:
-            scenarios = (0, utils_1.parseJsonScenarios)(input);
-            break;
-        case constants_1.OutputFormats.Csv:
-            scenarios = await (0, utils_1.parseCsvScenarios)(input);
-            break;
-        default:
-            throw Error(`${format} is not a valid option.`);
-    }
-    const request = buildUpdateRequest(scenarios, dataTable);
-    await apiClient.updateDataTable(dataTableId, request);
-}
-function buildUpdateRequest(scenarios, dataTable) {
-    if (scenarios.length === 0 || !scenarios) {
+function validateScenarios(scenarios, dataTable) {
+    if (!scenarios || scenarios.length === 0) {
         throw Error('Must have at least 1 scenario in DataTable.');
     }
     const validationVariables = scenarios[0].variables;
@@ -60,7 +42,7 @@ function buildUpdateRequest(scenarios, dataTable) {
         if (scenario.id && !dataTable.scenario_ids?.includes(scenario.id)) {
             throw Error(`Scenario in input with ID ${scenario.id} does not exist in DataTable scenario_ids.`);
         }
-        if (scenario.variables?.length === 0 || !scenario.variables) {
+        if (!scenario.variables || scenario.variables.length === 0) {
             throw Error('Must have at least 1 variable in a scenario.');
         }
         if (scenario.variables.length !== validationVariables?.length) {
@@ -75,49 +57,37 @@ function buildUpdateRequest(scenarios, dataTable) {
             }
         });
     });
-    const parsedScenarioMap = scenarios
-        .filter((scenario) => scenario.id !== undefined)
-        .reduce((acc, scenario) => {
-        acc[scenario.id] = scenario;
-        return acc;
-    }, {});
-    const updatedRows = dataTable.scenario_ids
-        ?.filter((id) => parsedScenarioMap[id] !== undefined)
-        .map((id) => {
-        const variableRow = (0, utils_1.scenarioAsVariableRow)(parsedScenarioMap[id]);
-        return {
-            id: variableRow.id || '',
-            scenario_id: variableRow.scenario_id || '',
-            table_id: dataTable.id,
-            organization_id: dataTable.workspace_id,
-            row: variableRow.row || [],
-        };
-    })
-        .filter(pureUtil_1.isDefined) || [];
-    const updatedRowIds = updatedRows
-        .map((row) => row.id)
-        .filter(pureUtil_1.isDefined);
-    const addedRows = scenarios
-        .filter((row) => row.id === undefined)
-        .map((scenario) => {
-        const variableRow = (0, utils_1.scenarioAsVariableRow)(scenario);
-        return {
-            id: variableRow.id,
-            scenario_id: variableRow.scenario_id,
-            table_id: dataTable.id,
-            organization_id: dataTable.workspace_id,
-            row: variableRow.row,
-        };
-    });
-    const deletedRowIds = dataTable.scenario_ids?.filter((id) => !updatedRowIds.includes(id)) ?? [];
-    const prototype = {
-        ...(0, utils_1.dataTableAsVariableTable)(dataTable),
-        row_ids: updatedRows.map((row) => row.id).filter(pureUtil_1.isDefined),
-    };
-    return {
-        prototype,
-        updatedRows,
-        deletedRowIds,
-        addedRows,
-    };
+}
+async function updateDataTable(parsed) {
+    const { id: dataTableId, input } = parsed;
+    const format = parsed.format ?? (0, utils_1.inferFileFormat)(input);
+    const apiClient = await mablApiClientFactory_1.MablApiClientFactory.createApiClient();
+    const dataTable = await apiClient.getDataTable(dataTableId);
+    let scenarios;
+    switch (format) {
+        case constants_1.OutputFormats.Json:
+            scenarios = (0, utils_1.parseJsonScenarios)(input);
+            break;
+        case constants_1.OutputFormats.Csv:
+            scenarios = await (0, utils_1.parseCsvScenarios)(input);
+            break;
+        default:
+            throw Error(`${format} is not a valid option.`);
+    }
+    validateScenarios(scenarios, dataTable);
+    const scenariosToUpdate = scenarios.filter((scenario) => scenario.id !== undefined);
+    const scenariosToCreate = scenarios.filter((scenario) => scenario.id === undefined);
+    const inputScenarioIds = new Set(scenariosToUpdate.map((scenario) => scenario.id));
+    const scenarioIdsToRemove = (dataTable.scenario_ids ?? []).filter((id) => !inputScenarioIds.has(id));
+    const updateResults = await Promise.allSettled(scenariosToUpdate.map((scenario) => apiClient.putScenario(scenario.id, scenario)));
+    (0, logUtils_1.logPromiseSettledRejections)(updateResults);
+    const createResults = await Promise.allSettled(scenariosToCreate.map((scenario) => apiClient.createScenario(dataTableId, scenario)));
+    (0, logUtils_1.logPromiseSettledRejections)(createResults);
+    const removeResults = await Promise.allSettled(scenarioIdsToRemove.map((scenarioId) => apiClient.removeScenario(scenarioId)));
+    (0, logUtils_1.logPromiseSettledRejections)(removeResults);
+    if (updateResults.some(pureUtil_1.isRejectedPromise) ||
+        createResults.some(pureUtil_1.isRejectedPromise) ||
+        removeResults.some(pureUtil_1.isRejectedPromise)) {
+        throw new Error('Some DataTable operations failed. See above for details.');
+    }
 }

package/commands/datatables/utils.js

@@ -11,8 +11,6 @@ exports.parseJsonScenarios = parseJsonScenarios;
 exports.jsonToScenarios = jsonToScenarios;
 exports.inferFileFormat = inferFileFormat;
 exports.isScenarioArray = isScenarioArray;
-exports.dataTableAsVariableTable = dataTableAsVariableTable;
-exports.scenarioAsVariableRow = scenarioAsVariableRow;
 exports.variableRowAsScenario = variableRowAsScenario;
 const constants_1 = require("../constants");
 const path_1 = require("path");
@@ -101,31 +99,6 @@ function isScenario(arg) {
 function isScenarioArray(arg) {
     return Array.isArray(arg) && arg.every((scenario) => isScenario(scenario));
 }
-function dataTableAsVariableTable(dataTable) {
-    return {
-        id: dataTable.id,
-        organization_id: dataTable.workspace_id,
-        name: dataTable.name,
-        created_time: dataTable.created_time,
-        created_by_id: dataTable.created_by_id,
-        last_updated_time: dataTable.last_updated_time,
-        last_updated_by_id: dataTable.last_updated_by_id,
-        row_ids: dataTable.scenario_ids,
-    };
-}
-function scenarioAsVariableRow(scenario) {
-    return {
-        id: scenario?.id,
-        table_id: scenario?.table_id,
-        created_time: scenario?.created_time,
-        created_by_id: scenario?.created_by_id,
-        last_updated_time: scenario?.last_updated_time,
-        last_updated_by_id: scenario?.last_updated_by_id,
-        organization_id: scenario?.workspace_id,
-        scenario_id: scenario?.name,
-        row: scenario?.variables,
-    };
-}
 function variableRowAsScenario(variableRow) {
     if (isScenario(variableRow)) {
         return variableRow;

package/commands/tests/tests_cmds/create.js

@@ -41,9 +41,13 @@ const runAgenticGeneration_1 = require("../../../trainer/cli/runAgenticGeneratio
 const mablApiClientFactory_1 = require("../../../api/mablApiClientFactory");
 const pureUtil_1 = require("../../../util/pureUtil");
 const saveGeneratedTest_1 = require("../../../trainer/generation/saveGeneratedTest");
+const ViewportStep_1 = __importStar(require("../../../trainer/sharedUtils/mablscript/canonical/ViewportStep"));
+const VisitUrlStep_1 = __importStar(require("../../../trainer/sharedUtils/mablscript/canonical/VisitUrlStep"));
 const inquirer = __importStar(require("inquirer"));
 const chalk = require('chalk');
 const CommandArgAutoSave = 'save';
+const CommandArgDisabled = 'disabled';
+const CommandArgEmpty = 'empty';
 var PostGenerationActions;
 (function (PostGenerationActions) {
     PostGenerationActions["Save"] = "Save the test";
@@ -136,6 +140,16 @@ exports.builder = (yargs) => {
         describe: 'Intent for the test, e.g. "smoke", "regression"',
         nargs: 1,
         type: 'string',
+    })
+        .option(CommandArgDisabled, {
+        describe: 'Create the test in a disabled state',
+        type: 'boolean',
+        default: false,
+    })
+        .option(CommandArgEmpty, {
+        describe: 'Create a minimal test with only viewport and visit-URL steps',
+        type: 'boolean',
+        default: false,
     })
         .option(constants_1.CommandArgTestIntent, {
         hidden: true,
@@ -186,7 +200,48 @@ async function launchTrainingSession(parsed) {
     const labels = parsed[constants_1.CommandArgLabels];
     const dataTableIds = parsed[constants_1.CommandArgDataTables];
     const intent = parsed[constants_1.CommandArgIntent];
+    const disabled = parsed[CommandArgDisabled];
+    const enabled = disabled ? false : undefined;
     const testIntent = parsed[constants_1.CommandArgTestIntent];
+    if (parsed[CommandArgEmpty]) {
+        const apiClient = await mablApiClientFactory_1.MablApiClientFactory.createApiClient();
+        const viewportStep = new ViewportStep_1.default({ width, height });
+        const visitUrlStep = new VisitUrlStep_1.default(url);
+        const rawSteps = [viewportStep, visitUrlStep];
+        const stepDescriptions = [
+            (0, ViewportStep_1.humanizeViewportStep)(viewportStep),
+            (0, VisitUrlStep_1.humanizeVisitUrlStep)(visitUrlStep),
+        ];
+        console.log(chalk.cyan('Creating empty test...'));
+        const result = await (0, saveGeneratedTest_1.saveGeneratedTest)({
+            apiClient,
+            workspaceId,
+            applicationId,
+            url,
+            testName,
+            testDescription,
+            labels,
+            rawSteps,
+            stepDescriptions,
+            credentialsId,
+            enabled,
+            testIntent,
+        });
+        if (result.success) {
+            console.log(chalk.green('\nTest saved successfully!'));
+            if (result.testId) {
+                console.log(chalk.gray('  Test ID:'), chalk.white(result.testId));
+            }
+            if (result.testUrl) {
+                console.log(chalk.gray('  View:'), chalk.cyan(result.testUrl));
+            }
+        }
+        else {
+            console.error(chalk.red('Failed to save test:'), result.error);
+            throw new Error(result.error);
+        }
+        return 'done';
+    }
     const headless = parsed[constants_1.CommandArgHeadless];
     const verbose = parsed[constants_1.CommandArgVerbose];
     const outputScreenshots = parsed[constants_1.CommandArgOutputScreenshots];
@@ -229,6 +284,7 @@ async function launchTrainingSession(parsed) {
                     apiClient,
                     result,
                     autoSave: false,
+                    enabled,
                 });
             }
             else {
@@ -241,6 +297,7 @@ async function launchTrainingSession(parsed) {
                     apiClient,
                     result,
                     autoSave: false,
+                    enabled,
                 });
             }
             return 'done';
@@ -305,6 +362,7 @@ async function saveTestFromContext(context) {
         testDescription: context.testIntent,
         rawSteps: context.result.rawSteps,
         stepDescriptions: context.result.steps,
+        enabled: context.enabled,
     });
     if (result.success) {
         console.log(chalk.green('✓ Test saved successfully!'));

package/core/execution/ApiTestUtils.js

@@ -3,7 +3,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.uuid = exports.createEmptyRequest = exports.PRE_REQUEST_LISTEN = exports.deduplicateApiTestExecutionResults = exports.ASSERT_TYPES = exports.ASSERT_TARGETS = exports.COLLECTION_WITH_FOLDERS_ERROR = exports.DEFAULT_OAUTH2_GRANT_TYPE = exports.DEFAULT_OAUTH2_CLIENT_CREDENTIALS_LOCATION = exports.HMAC_SHA1_SIGNATURE = exports.DEFAULT_ADD_OAUTH1_TO_HEADER_VALUE = exports.DEFAULT_OAUTH1_VERSION = exports.OAUTH2_AUTH = exports.OAUTH1_AUTH = exports.NO_AUTH = exports.BEARER_TOKEN_AUTH = exports.BASIC_AUTH = exports.API_KEY_AUTH = exports.INHERIT_AUTH_FROM_PARENT_VALUE = exports.MABL_GENERATED_COMMENT = exports.MABL_GENERATED_ASSERTION_TOKEN = exports.TEST_LISTEN = void 0;
+exports.uuid = exports.createEmptyRequest = exports.PRE_REQUEST_LISTEN = exports.ASSERT_TYPES = exports.ASSERT_TARGETS = exports.COLLECTION_WITH_FOLDERS_ERROR = exports.DEFAULT_OAUTH2_GRANT_TYPE = exports.DEFAULT_OAUTH2_CLIENT_CREDENTIALS_LOCATION = exports.HMAC_SHA1_SIGNATURE = exports.DEFAULT_ADD_OAUTH1_TO_HEADER_VALUE = exports.DEFAULT_OAUTH1_VERSION = exports.OAUTH2_AUTH = exports.OAUTH1_AUTH = exports.NO_AUTH = exports.BEARER_TOKEN_AUTH = exports.BASIC_AUTH = exports.API_KEY_AUTH = exports.INHERIT_AUTH_FROM_PARENT_VALUE = exports.MABL_GENERATED_COMMENT = exports.MABL_GENERATED_ASSERTION_TOKEN = exports.TEST_LISTEN = void 0;
 exports.compareStringsCaseInsensitive = compareStringsCaseInsensitive;
 exports.caseInsensitiveEquals = caseInsensitiveEquals;
 exports.getAssertionTypesForTarget = getAssertionTypesForTarget;
@@ -244,8 +244,6 @@ function createEmptyVariable(contentType) {
         description: '',
     };
 }
-var api_test_dedup_1 = require("./api-test-dedup");
-Object.defineProperty(exports, "deduplicateApiTestExecutionResults", { enumerable: true, get: function () { return api_test_dedup_1.deduplicateApiTestExecutionResults; } });
 function restoreCustomRequestFields(postmanResult, collection) {
     const requestItemsById = collection.item.reduce((collectionSet, itemDefinition) => {
         collectionSet[itemDefinition.id] = itemDefinition;

package/core/execution/PostmanUtils.js

@@ -4,6 +4,7 @@ exports.convertVariablesSummaryToPostmanFormat = convertVariablesSummaryToPostma
 exports.exportVariables = exportVariables;
 exports.sanitizeExportedVariables = sanitizeExportedVariables;
 const VariableUtils_1 = require("./VariableUtils");
+const variable_constants_1 = require("./variable-constants");
 const domUtil_1 = require("../../domUtil");
 function convertVariablesSummaryToPostmanFormat(summary) {
     const effectiveVariables = { ...summary.effective };
@@ -24,7 +25,7 @@ function exportVariables(postmanResult) {
     const exportedVariables = {};
     variablePrecedence
         .flatMap((variableList) => variableList.all())
-        .filter((variable) => Boolean(variable.key) && !variable.key?.startsWith(VariableUtils_1.FLOW_NAMESPACE))
+        .filter((variable) => Boolean(variable.key) && !variable.key?.startsWith(variable_constants_1.FLOW_NAMESPACE))
         .forEach((variable) => (exportedVariables[variable.key] = variable.value));
     return exportedVariables;
 }

package/core/execution/VariableUtils.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.API_CREDENTIALS_PASSWORD_VARIABLE_NAME = exports.API_CREDENTIALS_USERNAME_VARIABLE_NAME = exports.API_CREDENTIALS_NAMESPACE = exports.API_URL_VARIABLE_NAME = exports.API_NAMESPACE = exports.FLOW_NAMESPACE = exports.USER_NAMESPACE = void 0;
+exports.API_CREDENTIALS_PASSWORD_VARIABLE_NAME = exports.API_CREDENTIALS_USERNAME_VARIABLE_NAME = exports.API_CREDENTIALS_NAMESPACE = exports.API_URL_VARIABLE_NAME = void 0;
 exports.convertVariablesToPostmanFormat = convertVariablesToPostmanFormat;
 exports.generateVariablesSummaryForImport = generateVariablesSummaryForImport;
 exports.generateVariablesSummaryForExport = generateVariablesSummaryForExport;
@@ -8,13 +8,9 @@ exports.addFlowNamespace = addFlowNamespace;
 exports.addUserNamespace = addUserNamespace;
 exports.findUsedVariables = findUsedVariables;
 const domUtil_1 = require("../../domUtil");
-var variable_constants_1 = require("./variable-constants");
-Object.defineProperty(exports, "USER_NAMESPACE", { enumerable: true, get: function () { return variable_constants_1.USER_NAMESPACE; } });
-Object.defineProperty(exports, "FLOW_NAMESPACE", { enumerable: true, get: function () { return variable_constants_1.FLOW_NAMESPACE; } });
-Object.defineProperty(exports, "API_NAMESPACE", { enumerable: true, get: function () { return variable_constants_1.API_NAMESPACE; } });
-const variable_constants_2 = require("./variable-constants");
-exports.API_URL_VARIABLE_NAME = `${variable_constants_2.API_NAMESPACE}url`;
-exports.API_CREDENTIALS_NAMESPACE = `${variable_constants_2.API_NAMESPACE}credentials.`;
+const variable_constants_1 = require("./variable-constants");
+exports.API_URL_VARIABLE_NAME = `${variable_constants_1.API_NAMESPACE}url`;
+exports.API_CREDENTIALS_NAMESPACE = `${variable_constants_1.API_NAMESPACE}credentials.`;
 exports.API_CREDENTIALS_USERNAME_VARIABLE_NAME = `${exports.API_CREDENTIALS_NAMESPACE}username`;
 exports.API_CREDENTIALS_PASSWORD_VARIABLE_NAME = `${exports.API_CREDENTIALS_NAMESPACE}password`;
 function convertVariablesToPostmanFormat(value) {
@@ -128,10 +124,10 @@ function sanitizeVariables(variables) {
     return sanitizedVariables;
 }
 function addFlowNamespaceToVariableNames(variables) {
-    return addPrefixToVariableNames(variables, variable_constants_2.FLOW_NAMESPACE);
+    return addPrefixToVariableNames(variables, variable_constants_1.FLOW_NAMESPACE);
 }
 function addUserNamespaceToVariableNames(variables) {
-    return addPrefixToVariableNames(variables, variable_constants_2.USER_NAMESPACE);
+    return addPrefixToVariableNames(variables, variable_constants_1.USER_NAMESPACE);
 }
 function addPrefixToVariableNames(variables, prefix) {
     return mapVariables(variables, (variable) => ({
@@ -153,10 +149,10 @@ function extractValueParameters(parameters) {
     return parameters.filter((parameter) => parameter.type?.toString() === 'value');
 }
 function removeFlowNamespace(variableName) {
-    return removePrefix(variableName, variable_constants_2.FLOW_NAMESPACE);
+    return removePrefix(variableName, variable_constants_1.FLOW_NAMESPACE);
 }
 function removeUserNamespace(variableName) {
-    return removePrefix(variableName, variable_constants_2.USER_NAMESPACE);
+    return removePrefix(variableName, variable_constants_1.USER_NAMESPACE);
 }
 function removePrefix(variableName, prefix) {
     if (variableName.startsWith(prefix)) {
@@ -165,10 +161,10 @@ function removePrefix(variableName, prefix) {
     return variableName;
 }
 function addFlowNamespace(variableName) {
-    return addPrefix(variableName, variable_constants_2.FLOW_NAMESPACE);
+    return addPrefix(variableName, variable_constants_1.FLOW_NAMESPACE);
 }
 function addUserNamespace(variableName) {
-    return addPrefix(variableName, variable_constants_2.USER_NAMESPACE);
+    return addPrefix(variableName, variable_constants_1.USER_NAMESPACE);
 }
 function addPrefix(variableName, prefix) {
     if (variableName.startsWith(prefix)) {