@copado/agentia-cli 1.1.0 → 1.2.0

package/dist/lib/mcp/testing-tools.js

@@ -1,13 +1,15 @@
 /* eslint-disable unicorn/text-encoding-identifier-case -- CRT file APIs require the literal utf-8 enum. */
 import { z } from 'zod';
-import { prepareTestingData, TESTING_DATA_MAX_CELL_BYTES, TESTING_DATA_MAX_COLUMNS, TESTING_DATA_MAX_ROWS, } from '../testing-data-core.js';
-import { createTestingDataTable, deleteTestingDataTable, getTestingDataTable, listTestingDataTables, updateTestingDataTable, } from '../testing-data-operations.js';
-import { abortTestingBuild, buildTestingGitStorageUpdate, countTestingBuilds, createTestingJob, createTestingProject, createTestingRobot, createTestingUserParameter, createTestingVariable, deleteTestingJob, deleteTestingProject, deleteTestingRobot, deleteTestingUserParameter, deleteTestingVariable, downloadTestingJobFiles, getTestingBuild, getTestingBuildLogs, getTestingJob, getTestingJobRefs, getTestingProject, getTestingProjectPermission, getTestingRobot, getTestingUserParameter, getTestingVariable, latestTestingBuilds, listTestingBuilds, listTestingJobFiles, listTestingJobs, listTestingProjectPermissions, listTestingProjects, listTestingRobots, listTestingUserParameters, listTestingVariables, normalizeTestingRemotePath, runTestingBuild, searchTestingBuilds, setTestingProjectPermission, updateTestingJob, updateTestingProject, updateTestingRobot, updateTestingUserParameter, updateTestingVariable, uploadTestingJobFiles, waitForTestingBuildResult, } from '../testing-operations.js';
+import { createTestingDataTable, deleteTestingDataTable, getTestingDataTable, listTestingDataTables, TESTING_DATA_MAX_CELL_BYTES, TESTING_DATA_MAX_COLUMNS, TESTING_DATA_MAX_ROWS, updateTestingDataTable, } from '../testing-data-table.js';
+import { buildTestingRunOptions } from '../testing-flags.js';
+import { abortTestingBuild, countTestingBuilds, createTestingJob, createTestingProject, createTestingRobot, createTestingUserParameter, createTestingVariable, deleteTestingJob, deleteTestingProject, deleteTestingRobot, deleteTestingUserParameter, deleteTestingVariable, downloadTestingJobFiles, executeTestingBuildRun, getTestingBuild, getTestingBuildLogs, getTestingJob, getTestingJobRefs, getTestingJobScheduleStatus, getTestingProject, getTestingProjectPermission, getTestingRobot, getTestingUserParameter, getTestingVariable, listTestingBuilds, listTestingJobFiles, listTestingJobs, listTestingProjectPermissions, listTestingProjects, listTestingRobots, listTestingUserParameters, listTestingVariables, removeTestingJobSchedule, searchTestingBuilds, setTestingJobSchedule, setTestingProjectPermission, updateTestingJob, updateTestingProject, updateTestingRobot, updateTestingUserParameter, updateTestingVariable, uploadTestingJobFiles, } from '../testing-operations.js';
 import { sanitizeTestingOutput } from '../testing-sanitization.js';
-import { getTestingJobSchedule, removeTestingJobSchedule, setTestingJobSchedule, } from '../testing-schedule.js';
-import { normalizeTestingId, TESTING_BUILD_STATUSES, TESTING_CONSOLE_COLORS, TESTING_JOB_PARALLEL_MODES, TESTING_JOB_SUITE_TYPES, TESTING_PROJECT_EXECUTORS, TESTING_PROJECT_TYPES, TESTING_RECORD_BUILD_OPTIONS, TESTING_ROBOT_REPORTING, TESTING_ROBOT_RUN_ENVIRONMENTS, TESTING_ROBOT_TYPES, TESTING_RUN_TYPES, TESTING_VARIABLE_TYPES, } from '../testing-types.js';
+import { TESTING_BUILD_STATUSES, TESTING_CONSOLE_COLORS, TESTING_JOB_PARALLEL_MODES, TESTING_JOB_SUITE_TYPES, TESTING_RECORD_BUILD_OPTIONS, TESTING_ROBOT_ACCEPTED_RUN_ENVIRONMENTS, TESTING_ROBOT_REPORTING, TESTING_ROBOT_REPORTING_DESCRIPTION, TESTING_ROBOT_RUN_ENVIRONMENT_DESCRIPTION, TESTING_ROBOT_TYPE_DESCRIPTION, TESTING_ROBOT_TYPES, TESTING_RUN_TYPES, TESTING_VARIABLE_TYPES, } from '../testing-types.js';
 import { safelyHandleTool } from './result.js';
 import { registerTestingLiveMcpTools, testingLiveMcpHandlers } from './testing-live-tools.js';
+const projectIdSchema = z.number().int().positive().describe('CRT project ID.');
+const jobIdSchema = z.number().int().positive().describe('CRT job ID.');
+const optionalJobIdSchema = z.number().int().positive().optional().describe('CRT job ID.');
 const gitStorageSchema = {
     gitBranch: z.string().optional().describe('Git branch to use for a Git-backed CRT job.'),
     gitDomainCredential: z.string().optional().describe('CRT domain credential ID for Git repository access.'),
@@ -17,50 +19,14 @@ const gitStorageSchema = {
     gitUseCallerCredentials: z.boolean().optional().describe('Use caller credentials for Git repository access.'),
     gitUsername: z.string().optional().describe('Git username for repository access. Prefer domain credentials.'),
 };
-function parseParameter(spec) {
-    const separator = spec.indexOf('=');
-    const key = separator === -1 ? spec.trim() : spec.slice(0, separator).trim();
-    if (separator === -1 || key.length === 0) {
-        throw new Error(`Invalid execution parameter: ${spec}. Use KEY=VALUE.`);
-    }
-    return { key, value: spec.slice(separator + 1) };
-}
-function runOptions(args) {
-    if (args.branch !== undefined && args.tag !== undefined) {
-        throw new Error('Use either branch or tag, not both.');
-    }
-    if (args.rerunFailed !== undefined && args.rerunFailedFromLatest === true) {
-        throw new Error('Use either rerunFailed or rerunFailedFromLatest, not both.');
-    }
-    return {
-        branch: args.branch,
-        consoleColors: args.consoleColors,
-        consoleWidth: args.consoleWidth,
-        dataTables: args.datatable,
-        doc: args.doc,
-        dryRun: args.dryRun,
-        exclude: args.exclude,
-        exitOnError: args.exitOnError,
-        exitOnFailure: args.exitOnFailure,
-        include: args.include,
-        jobId: args.job,
-        metadata: args.metadata,
-        name: args.name,
-        parameters: (args.param ?? []).map((param) => parseParameter(param)),
-        projectId: args.project,
-        record: args.record,
-        rerunFailedFromLatest: args.rerunFailedFromLatest === true ? true : undefined,
-        rerunFailedId: args.rerunFailed,
-        runEmptySuite: args.runEmptySuite,
-        runType: args.runType,
-        setTags: args.setTag,
-        skip: args.skip,
-        skipOnFailure: args.skipOnFailure,
-        suites: args.suite,
-        tag: args.tag,
-        tests: args.test,
-        variables: args.variable,
-    };
+const jobScreenSchema = {
+    screenDepth: z.number().int().positive().optional().describe('Screen color depth.'),
+    screenHeight: z.number().int().positive().optional().describe('Screen height.'),
+    screenWidth: z.number().int().positive().optional().describe('Screen width.'),
+};
+function requireConfirm(confirm, action) {
+    if (confirm !== true)
+        throw new Error(`${action} requires confirm: true.`);
 }
 async function testingProjectList(args = {}) {
     return { result: await listTestingProjects({ name: args.name }) };
@@ -68,11 +34,8 @@ async function testingProjectList(args = {}) {
 async function testingProjectCreate(args) {
     return {
         result: await createTestingProject({
-            dashboardId: args.dashboardId,
             description: args.description,
             name: args.name,
-            projectExecutor: args.executor,
-            projectType: args.type,
         }),
     };
 }
@@ -100,8 +63,7 @@ async function testingProjectPermissionSet(args) {
     };
 }
 async function testingProjectDelete(args) {
-    if (args.confirm !== true)
-        throw new Error('Project deletion requires confirm: true.');
+    requireConfirm(args.confirm, 'Project deletion');
     return { result: await deleteTestingProject({ projectId: args.id }) };
 }
 async function testingRobotList(args) {
@@ -111,41 +73,16 @@ async function testingRobotGet(args) {
     return { result: await getTestingRobot({ projectId: args.project, robotId: args.id }) };
 }
 async function testingRobotDelete(args) {
-    if (args.confirm !== true)
-        throw new Error('Robot deletion requires confirm: true.');
+    requireConfirm(args.confirm, 'Robot deletion');
     return { result: await deleteTestingRobot({ projectId: args.project, robotId: args.id }) };
 }
 async function testingRobotCreate(args) {
-    if (args.kind !== undefined && args.runEnvironment !== undefined) {
-        throw new Error('Use either kind or runEnvironment, not both.');
-    }
-    return {
-        result: await createTestingRobot({
-            description: args.description,
-            fwVersion: args.fwVersion,
-            kind: args.runEnvironment ?? args.kind,
-            label: args.label,
-            name: args.name,
-            os: args.os,
-            projectId: args.project,
-            reporting: args.reporting,
-            robotType: args.robotType,
-        }),
-    };
+    const { project: projectId, ...rest } = args;
+    return { result: await createTestingRobot({ ...rest, projectId }) };
 }
 async function testingRobotUpdate(args) {
-    return {
-        result: await updateTestingRobot({
-            description: args.description,
-            fwVersion: args.fwVersion,
-            label: args.label,
-            name: args.name,
-            projectId: args.project,
-            reporting: args.reporting,
-            robotId: args.id,
-            robotType: args.robotType,
-        }),
-    };
+    const { id: robotId, project: projectId, ...rest } = args;
+    return { result: await updateTestingRobot({ ...rest, projectId, robotId }) };
 }
 async function testingJobList(args) {
     return { result: await listTestingJobs({ name: args.name, projectId: args.project }) };
@@ -154,78 +91,37 @@ async function testingJobGet(args) {
     return { result: await getTestingJob({ jobId: args.id, projectId: args.project }) };
 }
 async function testingJobUpdate(args) {
-    if (args.showVideoParams === true && args.hideVideoParams === true) {
-        throw new Error('Use either showVideoParams or hideVideoParams, not both.');
-    }
-    const screenValues = [args.screenDepth, args.screenHeight, args.screenWidth];
-    const screenSupplied = screenValues.filter((value) => value !== undefined);
-    if (screenSupplied.length > 0 && screenSupplied.length !== screenValues.length) {
-        throw new Error('Provide screenWidth, screenHeight, and screenDepth together.');
-    }
+    const { gitDomainCredential: domainCredentialId, id: jobId, parallelMode: parallelExecution, project: projectId, robot: robotId, ...rest } = args;
     return {
         result: await updateTestingJob({
-            analytics: args.caseDurationChangeDetection === undefined
-                ? undefined
-                : { caseDurationChangeDetection: args.caseDurationChangeDetection },
-            description: args.description,
-            domainCredentialId: args.gitDomainCredential,
-            jobId: args.id,
-            name: args.name,
-            parallelExecution: args.parallelMode,
-            projectId: args.project,
-            robotId: args.robot,
-            screen: screenSupplied.length === 0
-                ? undefined
-                : { depth: args.screenDepth, height: args.screenHeight, width: args.screenWidth },
-            showVideoParams: args.showVideoParams === true ? true : args.hideVideoParams === true ? false : undefined,
-            storage: buildTestingGitStorageUpdate({
-                gitBranch: args.gitBranch,
-                gitPassword: args.gitPassword,
-                gitTag: args.gitTag,
-                gitUrl: args.gitUrl,
-                gitUseCallerCredentials: args.gitUseCallerCredentials,
-                gitUsername: args.gitUsername,
-            }),
-            suiteType: args.suiteType,
-            timeout: args.timeout,
+            ...rest,
+            domainCredentialId,
+            jobId,
+            parallelExecution,
+            projectId,
+            robotId,
+            screenMissingMessage: 'Provide screenWidth, screenHeight, and screenDepth together.',
         }),
     };
 }
 async function testingJobDelete(args) {
-    if (args.confirm !== true)
-        throw new Error('Job deletion requires confirm: true.');
+    requireConfirm(args.confirm, 'Job deletion');
     return { result: await deleteTestingJob({ jobId: args.id, projectId: args.project }) };
 }
 async function testingJobScheduleGet(args) {
-    const schedule = await getTestingJobSchedule({ jobId: args.job, projectId: args.project });
-    return { result: { configured: schedule !== undefined, jobId: args.job, projectId: args.project, schedule } };
+    return { result: await getTestingJobScheduleStatus({ jobId: args.job, projectId: args.project }) };
 }
 async function testingJobScheduleSet(args) {
-    return {
-        result: await setTestingJobSchedule({
-            cron: args.cron,
-            jobId: args.job,
-            projectId: args.project,
-            record: args.record,
-            timezone: args.timezone,
-        }),
-    };
+    const { job: jobId, project: projectId, ...rest } = args;
+    return { result: await setTestingJobSchedule({ ...rest, jobId, projectId }) };
 }
 async function testingJobScheduleRemove(args) {
-    if (args.confirm !== true)
-        throw new Error('Schedule removal requires confirm: true.');
+    requireConfirm(args.confirm, 'Schedule removal');
     return { result: await removeTestingJobSchedule({ jobId: args.job, projectId: args.project }) };
 }
 async function testingDataTableList(args) {
-    return {
-        result: await listTestingDataTables({
-            fields: args.fields,
-            jobId: args.job,
-            name: args.name,
-            projectId: args.project,
-            version: args.revision,
-        }),
-    };
+    const { job: jobId, project: projectId, revision: version, ...rest } = args;
+    return { result: await listTestingDataTables({ ...rest, jobId, projectId, version }) };
 }
 async function testingDataTableGet(args) {
     return {
@@ -238,15 +134,16 @@ async function testingDataTableGet(args) {
     };
 }
 async function testingDataTableCreate(args) {
-    if (args.confirm !== true)
-        throw new Error('Data-table creation requires confirm: true.');
+    requireConfirm(args.confirm, 'Data-table creation');
     return {
         result: await createTestingDataTable({
             body: {
                 ...(args.aiGenerated === undefined ? {} : { aiGenerated: args.aiGenerated }),
-                data: prepareTestingData(args.data, args.header ?? 'row'),
+                data: args.data,
                 ...(args.defaultRow === undefined ? {} : { defaultRow: args.defaultRow }),
+                header: args.header ?? 'row',
                 name: args.name,
+                prepare: true,
                 ...(args.topics === undefined ? {} : { topics: args.topics }),
             },
             jobId: args.job,
@@ -255,19 +152,15 @@ async function testingDataTableCreate(args) {
     };
 }
 async function testingDataTableUpdate(args) {
-    if (args.confirm !== true)
-        throw new Error('Data-table update requires confirm: true.');
-    const body = {
-        ...(args.aiGenerated === undefined ? {} : { aiGenerated: args.aiGenerated }),
-        ...(args.data === undefined ? {} : { data: prepareTestingData(args.data, args.header ?? 'row') }),
-        ...(args.defaultRow === undefined ? {} : { defaultRow: args.defaultRow }),
-        ...(args.topics === undefined ? {} : { topics: args.topics }),
-    };
-    if (Object.keys(body).length === 0)
-        throw new Error('Data-table update requires at least one field.');
+    requireConfirm(args.confirm, 'Data-table update');
     return {
         result: await updateTestingDataTable({
-            body,
+            body: {
+                ...(args.aiGenerated === undefined ? {} : { aiGenerated: args.aiGenerated }),
+                ...(args.data === undefined ? {} : { data: args.data, header: args.header ?? 'row', prepare: true }),
+                ...(args.defaultRow === undefined ? {} : { defaultRow: args.defaultRow }),
+                ...(args.topics === undefined ? {} : { topics: args.topics }),
+            },
             jobId: args.job,
             projectId: args.project,
             tableId: args.id,
@@ -275,51 +168,31 @@ async function testingDataTableUpdate(args) {
     };
 }
 async function testingDataTableDelete(args) {
-    if (args.confirm !== true)
-        throw new Error('Data-table deletion requires confirm: true.');
-    return {
-        result: await deleteTestingDataTable({
-            jobId: args.job,
-            projectId: args.project,
-            tableId: args.id,
-        }),
-    };
+    requireConfirm(args.confirm, 'Data-table deletion');
+    return { result: await deleteTestingDataTable({ jobId: args.job, projectId: args.project, tableId: args.id }) };
 }
 async function testingJobCreate(args) {
-    const screenValues = [args.screenDepth, args.screenHeight, args.screenWidth];
-    if (screenValues.some((value) => value !== undefined) && screenValues.includes(undefined)) {
-        throw new Error('Provide screenWidth, screenHeight, and screenDepth together.');
-    }
+    const { gitDomainCredential: gitDomainCredentialId, parallelMode: parallelExecution, project: projectId, robot: robotId, ...rest } = args;
     return {
         result: await createTestingJob({
-            analytics: args.caseDurationChangeDetection === undefined
-                ? undefined
-                : { caseDurationChangeDetection: args.caseDurationChangeDetection },
-            description: args.description,
-            dryRun: args.dryRun,
-            gitBranch: args.gitBranch,
-            gitDomainCredentialId: args.gitDomainCredential,
-            gitPassword: args.gitPassword,
-            gitTag: args.gitTag,
-            gitUrl: args.gitUrl,
-            gitUseCallerCredentials: args.gitUseCallerCredentials,
-            gitUsername: args.gitUsername,
-            name: args.name,
-            parallelExecution: args.parallelMode,
-            projectId: args.project,
-            robotId: args.robot,
-            screen: args.screenWidth === undefined
-                ? undefined
-                : { depth: args.screenDepth, height: args.screenHeight, width: args.screenWidth },
-            showVideoParams: args.showVideoParams,
-            suiteType: args.suiteType,
-            timeout: args.timeout,
-            validation: args.validation,
+            ...rest,
+            gitDomainCredentialId,
+            parallelExecution,
+            projectId,
+            robotId,
+            screenMissingMessage: 'Provide screenWidth, screenHeight, and screenDepth together.',
         }),
     };
 }
 async function testingJobFiles(args) {
-    return { result: await listTestingJobFiles({ jobId: args.job, projectId: args.project }) };
+    return {
+        result: await listTestingJobFiles({
+            branch: args.branch,
+            jobId: args.job,
+            projectId: args.project,
+            sharedCredentials: args.sharedCredentials,
+        }),
+    };
 }
 async function testingJobRefs(args) {
     return {
@@ -331,197 +204,89 @@ async function testingJobRefs(args) {
     };
 }
 async function testingJobUpload(args) {
-    const operations = args.operations.map((operation) => ({
-        encoding: operation.encoding,
-        op: operation.op,
-        path: normalizeTestingRemotePath(operation.path),
-        value: operation.value,
-    }));
-    for (const operation of operations) {
-        if (operation.op === 'remove' && operation.value !== undefined) {
-            throw new Error('Remove operations must not include value.');
-        }
-        if (operation.op !== 'remove' && operation.value === undefined) {
-            throw new Error(`${operation.op} operations require value.`);
-        }
-    }
+    const { authorEmail, authorName, job: jobId, project: projectId, ...rest } = args;
     return {
         result: await uploadTestingJobFiles({
-            author: args.authorName === undefined && args.authorEmail === undefined
+            ...rest,
+            author: authorName === undefined && authorEmail === undefined
                 ? undefined
-                : { email: args.authorEmail ?? 'agentia@copado.com', name: args.authorName ?? 'Agentia CLI' },
-            branch: args.branch,
-            commitMessage: args.commitMessage,
-            dryRun: args.dryRun,
-            jobId: args.job,
-            operations,
-            parentCommitHash: args.parentCommitHash,
-            projectId: args.project,
+                : { email: authorEmail ?? 'agentia@copado.com', name: authorName ?? 'Agentia CLI' },
+            jobId,
+            projectId,
         }),
     };
 }
 async function testingJobDownload(args) {
+    const { file: files, job: jobId, project: projectId, ...rest } = args;
+    return { result: await downloadTestingJobFiles({ ...rest, files, jobId, projectId }) };
+}
+async function testingBuildRun(args) {
+    const options = buildTestingRunOptions(args);
     return {
-        result: await downloadTestingJobFiles({
-            branch: args.branch,
-            files: args.file,
-            jobId: args.job,
-            outputDir: args.outputDir,
-            projectId: args.project,
+        result: await executeTestingBuildRun({
+            ...options,
+            streamLogs: args.streamLogs,
+            strictDataDrivenWait: true,
+            timeout: args.timeout,
+            wait: args.waitForResult === true || args.streamLogs === true,
+            waitForResult: args.waitForResult,
         }),
     };
 }
-async function testingBuildRun(args) {
-    if ((args.datatable?.length ?? 0) > 0 && (args.waitForResult === true || args.streamLogs === true)) {
-        throw new Error('Data-driven MCP runs start a batch and cannot wait or stream logs.');
-    }
-    const options = runOptions(args);
-    const run = await runTestingBuild(options);
-    if (args.waitForResult !== true && args.streamLogs !== true)
-        return { result: run };
-    const buildId = normalizeTestingId(run.build);
-    if (buildId === undefined)
-        throw new Error('CRT did not return a build ID for the started run.');
-    const wait = await waitForTestingBuildResult({
-        ...options,
-        buildId,
-        sensitiveValues: options.parameters?.filter((parameter) => parameter.sensitive === true).map((parameter) => parameter.value ?? ''),
-        streamLogs: args.streamLogs,
-        timeoutMs: (args.timeout ?? 60) * 60_000,
-    });
-    return { result: { ...run, finalBuild: wait.build, wait } };
-}
 async function testingBuildList(args) {
     if (args.count === true) {
         if (args.job === undefined)
             throw new Error('count requires job.');
         return { result: { count: await countTestingBuilds({ jobId: args.job, projectId: args.project }) } };
     }
-    return {
-        result: await listTestingBuilds({
-            jobId: args.job,
-            page: args.page,
-            pageSize: args.pageSize,
-            projectId: args.project,
-            resultSize: args.resultSize ?? 'compact',
-            status: args.status,
-        }),
-    };
+    const { job: jobId, project: projectId, resultSize = 'compact', ...rest } = args;
+    return { result: await listTestingBuilds({ ...rest, jobId, projectId, resultSize }) };
 }
 async function testingBuildGet(args) {
     return {
-        result: await getTestingBuild({
-            buildId: args.build,
-            full: args.full,
-            jobId: args.job,
-            projectId: args.project,
-        }),
-    };
-}
-async function testingBuildLatest(args) {
-    return {
-        result: await latestTestingBuilds({
-            projectId: args.project,
-            resultSize: args.resultSize ?? 'compact',
-            status: args.status,
-        }),
+        result: await getTestingBuild({ buildId: args.build, full: args.full, jobId: args.job, projectId: args.project }),
     };
 }
 async function testingBuildSearch(args) {
-    return {
-        result: await searchTestingBuilds({
-            count: args.count,
-            from: args.from,
-            jobId: args.job,
-            order: args.order,
-            page: args.page,
-            pageSize: args.pageSize,
-            projectId: args.project,
-            robots: args.robot,
-            runTypes: args.runType,
-            sort: args.sort,
-            statuses: args.status,
-            to: args.to,
-        }),
-    };
+    const { job: jobId, project: projectId, robot: robots, runType: runTypes, status: statuses, ...rest } = args;
+    return { result: await searchTestingBuilds({ ...rest, jobId, projectId, robots, runTypes, statuses }) };
 }
 async function testingBuildLogs(args) {
+    const { build: buildId, job: jobId, project: projectId, ...rest } = args;
     return {
-        result: sanitizeTestingOutput(await getTestingBuildLogs({
-            buildId: args.build,
-            container: args.container,
-            jobId: args.job,
-            limitBytes: args.limitBytes,
-            output: args.output,
-            pretty: args.pretty,
-            projectId: args.project,
-            sinceSeconds: args.sinceSeconds,
-            tailLines: args.tailLines,
-            timestamps: args.timestamps,
-        })),
+        result: sanitizeTestingOutput(await getTestingBuildLogs({ ...rest, buildId, jobId, projectId }), {
+            preserveLogKeys: true,
+        }),
     };
 }
 async function testingBuildAbort(args) {
-    if (args.confirm !== true)
-        throw new Error('Build abort requires confirm: true.');
-    return {
-        result: await abortTestingBuild({
-            buildId: args.build,
-            jobId: args.job,
-            projectId: args.project,
-        }),
-    };
+    requireConfirm(args.confirm, 'Build abort');
+    return { result: await abortTestingBuild({ buildId: args.build, jobId: args.job, projectId: args.project }) };
 }
 async function testingVariableList(args) {
-    return {
-        result: await listTestingVariables({
-            jobId: args.job,
-            key: args.key,
-            projectId: args.project,
-            robotId: args.robot,
-            sensitive: args.sensitive,
-            type: args.type,
-            userSpecific: args.userSpecific,
-        }),
-    };
+    const { job: jobId, project: projectId, robot: robotId, ...rest } = args;
+    return { result: await listTestingVariables({ ...rest, jobId, projectId, robotId }) };
 }
 async function testingVariableGet(args) {
     return { result: await getTestingVariable({ projectId: args.project, variableId: args.id }) };
 }
 async function testingVariableCreate(args) {
-    return {
-        result: await createTestingVariable({
-            description: args.description,
-            jobId: args.job,
-            key: args.key,
-            projectId: args.project,
-            robotId: args.robot,
-            sensitive: args.sensitive,
-            type: args.type,
-            userSpecific: args.userSpecific,
-            value: args.value,
-        }),
-    };
+    const { job: jobId, project: projectId, robot: robotId, ...rest } = args;
+    return { result: await createTestingVariable({ ...rest, jobId, projectId, robotId }) };
 }
 async function testingVariableUpdate(args) {
-    if (args.sensitive === true && args.notSensitive === true) {
-        throw new Error('Use either sensitive or notSensitive, not both.');
-    }
+    const { id: variableId, project: projectId, ...rest } = args;
     return {
         result: await updateTestingVariable({
-            description: args.description,
-            key: args.key,
-            projectId: args.project,
-            sensitive: args.sensitive === true ? true : args.notSensitive === true ? false : undefined,
-            type: args.type,
-            value: args.value,
-            variableId: args.id,
+            ...rest,
+            projectId,
+            sensitiveConflictMessage: 'Use either sensitive or notSensitive, not both.',
+            variableId,
         }),
     };
 }
 async function testingVariableDelete(args) {
-    if (args.confirm !== true)
-        throw new Error('Variable deletion requires confirm: true.');
+    requireConfirm(args.confirm, 'Variable deletion');
     return { result: await deleteTestingVariable({ projectId: args.project, variableId: args.id }) };
 }
 async function testingUserParameterList(args) {
@@ -531,38 +296,22 @@ async function testingUserParameterGet(args) {
     return { result: await getTestingUserParameter({ parameterId: args.id }) };
 }
 async function testingUserParameterCreate(args) {
-    return {
-        result: await createTestingUserParameter({
-            description: args.description,
-            key: args.key,
-            sensitive: args.sensitive,
-            unique: args.unique,
-            value: args.value,
-        }),
-    };
+    return { result: await createTestingUserParameter(args) };
 }
 async function testingUserParameterUpdate(args) {
-    return {
-        result: await updateTestingUserParameter({
-            description: args.description,
-            key: args.key,
-            parameterId: args.id,
-            unique: args.unique,
-            value: args.value,
-        }),
-    };
+    const { id: parameterId, ...rest } = args;
+    return { result: await updateTestingUserParameter({ ...rest, parameterId }) };
 }
 async function testingUserParameterDelete(args) {
-    if (args.confirm !== true)
-        throw new Error('User-parameter deletion requires confirm: true.');
+    requireConfirm(args.confirm, 'User-parameter deletion');
     return { result: await deleteTestingUserParameter({ parameterId: args.id }) };
 }
 const buildListSchema = {
     count: z.boolean().optional().describe('Return only the build count; requires job.'),
-    job: z.number().int().positive().optional().describe('CRT job/test ID.'),
+    job: optionalJobIdSchema,
     page: z.number().int().positive().optional().describe('Build page number.'),
     pageSize: z.number().int().positive().optional().describe('Build page size.'),
-    project: z.number().int().positive().describe('CRT project ID.'),
+    project: projectIdSchema,
     resultSize: z.enum(['compact']).optional().describe('CRT result size.'),
     status: z.enum(TESTING_BUILD_STATUSES).optional().describe('Build status filter.'),
 };
@@ -572,21 +321,28 @@ const uploadSchema = {
     branch: z.string().optional().describe('Optional CRT branch name.'),
     commitMessage: z.string().optional().describe('Commit message for CRT file history.'),
     dryRun: z.boolean().optional().describe('Return a sanitized plan without making a network mutation.'),
-    job: z.number().int().positive().describe('CRT job/test ID.'),
-    operations: z.array(z.object({
+    job: jobIdSchema,
+    operations: z
+        .array(z.object({
         encoding: z.enum(['utf-8', 'base64']).optional(),
         op: z.enum(['add', 'replace', 'remove']),
         path: z.string().min(1),
         value: z.string().max(1_048_576).optional(),
-    })).min(1).max(100).describe('Bounded structured file operations; local paths and stdin are not available in MCP.'),
+    }))
+        .min(1)
+        .max(100)
+        .describe('Bounded structured file operations; local paths and stdin are not available in MCP.'),
     parentCommitHash: z.string().optional().describe('Expected parent commit hash.'),
-    project: z.number().int().positive().describe('CRT project ID.'),
+    project: projectIdSchema,
 };
 const runOptionSchema = {
     branch: z.string().optional().describe('Git branch/ref to use for this run. Mutually exclusive with tag.'),
     consoleColors: z.enum(TESTING_CONSOLE_COLORS).optional().describe('Robot Framework console color mode.'),
     consoleWidth: z.number().int().positive().optional().describe('Robot Framework console width.'),
-    datatable: z.array(z.string()).optional().describe('Data-driven TABLE_ID:DATASETS selections.'),
+    datatable: z
+        .array(z.string())
+        .optional()
+        .describe('Data-driven TABLE_ID:DATASETS selections. Batch runs cannot wait or stream logs.'),
     doc: z.string().optional().describe('Top-level Robot Framework suite documentation.'),
     dryRun: z.boolean().optional().describe('Validate Robot Framework data without executing library keywords.'),
     exclude: z.array(z.string()).optional().describe('Robot Framework tags to exclude from this run.'),
@@ -604,30 +360,32 @@ const runOptionSchema = {
     setTag: z.array(z.string()).optional().describe('Tags to set on every executed test.'),
     skip: z.array(z.string()).optional().describe('Tags for tests to skip.'),
     skipOnFailure: z.array(z.string()).optional().describe('Tags whose failing tests should be skipped.'),
-    streamLogs: z.boolean().optional().describe('Poll logs while waiting and return them once; MCP never streams chunks.'),
+    streamLogs: z
+        .boolean()
+        .optional()
+        .describe('Wait for the run and return the finished log once. MCP never streams chunks. Implies a wait, and is rejected for data-driven runs.'),
     suite: z.array(z.string()).optional().describe('Robot Framework suites to select.'),
     tag: z.string().optional().describe('Git tag to use for this run. Mutually exclusive with branch.'),
     test: z.array(z.string()).optional().describe('Robot Framework tests/tasks to select.'),
     timeout: z.number().int().positive().optional().describe('Maximum wait time in minutes.'),
     variable: z.array(z.string()).optional().describe('Robot Framework variables.'),
-    waitForResult: z.boolean().optional().describe('Poll until the build reaches a terminal result.'),
-};
-const buildRunSchema = {
-    job: z.number().int().positive().describe('CRT job/test ID.'),
-    project: z.number().int().positive().describe('CRT project ID.'),
-    ...runOptionSchema,
+    waitForResult: z
+        .boolean()
+        .optional()
+        .describe('Poll until the build reaches a terminal result. Omitted means do not wait. The CLI waits by default; this tool does not. Rejected for data-driven runs.'),
 };
+const buildRunSchema = { job: jobIdSchema, project: projectIdSchema, ...runOptionSchema };
 const testingDataMatrixSchema = z
     .array(z.array(z.string().max(TESTING_DATA_MAX_CELL_BYTES)).max(TESTING_DATA_MAX_COLUMNS))
     .max(TESTING_DATA_MAX_ROWS);
 const buildLogSchema = {
     build: z.number().int().positive().describe('CRT build/run ID.'),
     container: z.string().optional().describe('Specific CRT log stream/container.'),
-    job: z.number().int().positive().describe('CRT job/test ID.'),
+    job: jobIdSchema,
     limitBytes: z.number().int().positive().optional().describe('Maximum log bytes to fetch.'),
     output: z.string().optional().describe('Output file for logs.'),
     pretty: z.boolean().optional().describe('Request readable log formatting.'),
-    project: z.number().int().positive().describe('CRT project ID.'),
+    project: projectIdSchema,
     sinceSeconds: z.number().int().nonnegative().optional().describe('Fetch logs from the last N seconds.'),
     tailLines: z.number().int().nonnegative().optional().describe('Return only the last N lines.'),
     timestamps: z.boolean().optional().describe('Add timestamps to log lines.'),
@@ -635,7 +393,7 @@ const buildLogSchema = {
 export const agentiaMcpTestingToolHandlers = {
     testingBuildAbort,
     testingBuildGet,
-    testingBuildLatest,
+    testingBuildLatest: testingBuildList,
     testingBuildList,
     testingBuildLogs,
     testingBuildRun,
@@ -697,11 +455,8 @@ export function registerAgentiaMcpTestingTools(server) {
     server.registerTool('agentia_testing_project_create', {
         description: 'Create a Copado Robotic Testing project.',
         inputSchema: {
-            dashboardId: z.number().int().positive().optional().describe('Optional CRT dashboard ID.'),
             description: z.string().optional().describe('Project description.'),
-            executor: z.enum(TESTING_PROJECT_EXECUTORS).optional().describe('Project execution backend.'),
             name: z.string().describe('Project name.'),
-            type: z.enum(TESTING_PROJECT_TYPES).optional().describe('CRT project type.'),
         },
         title: 'Agentia Testing Project Create',
     }, (args) => safelyHandleTool(() => testingProjectCreate(args)));
@@ -721,18 +476,18 @@ export function registerAgentiaMcpTestingTools(server) {
     }, (args) => safelyHandleTool(() => testingProjectUpdate(args)));
     server.registerTool('agentia_testing_project_permission_list', {
         description: 'List assignments for a Copado Robotic Testing project.',
-        inputSchema: { project: z.number().int().positive().describe('CRT project ID.') },
+        inputSchema: { project: projectIdSchema },
         title: 'Agentia Testing Project Permission List',
     }, (args) => safelyHandleTool(() => testingProjectPermissionList(args)));
     server.registerTool('agentia_testing_project_permission_get', {
         description: "Get the caller's permission on a Copado Robotic Testing project.",
-        inputSchema: { project: z.number().int().positive().describe('CRT project ID.') },
+        inputSchema: { project: projectIdSchema },
         title: 'Agentia Testing Project Permission Get',
     }, (args) => safelyHandleTool(() => testingProjectPermissionGet(args)));
     const permissionMutationSchema = {
         edit: z.boolean().optional().describe('Grant edit access.'),
         group: z.number().int().positive().optional().describe('Platform group ID; exclusive with user.'),
-        project: z.number().int().positive().describe('CRT project ID.'),
+        project: projectIdSchema,
         user: z.number().int().positive().optional().describe('Platform user ID; exclusive with group.'),
         view: z.boolean().optional().describe('Grant view access.'),
     };
@@ -748,23 +503,17 @@ export function registerAgentiaMcpTestingTools(server) {
     }, (args) => safelyHandleTool(() => testingProjectPermissionSet(args)));
     server.registerTool('agentia_testing_project_delete', {
         description: 'Delete a Copado Robotic Testing project.',
-        inputSchema: {
-            confirm: z.boolean().describe('Must be true to confirm project deletion.'),
-            id: z.number().int().positive().describe('CRT project ID.'),
-        },
+        inputSchema: { confirm: z.boolean().describe('Must be true to confirm project deletion.'), id: projectIdSchema },
         title: 'Agentia Testing Project Delete',
     }, (args) => safelyHandleTool(() => testingProjectDelete(args)));
     server.registerTool('agentia_testing_robot_list', {
         description: 'List Copado Robotic Testing robots.',
-        inputSchema: { project: z.number().int().positive().describe('CRT project ID.') },
+        inputSchema: { project: projectIdSchema },
         title: 'Agentia Testing Robot List',
     }, (args) => safelyHandleTool(() => testingRobotList(args)));
     server.registerTool('agentia_testing_robot_get', {
         description: 'Get a Copado Robotic Testing robot from the project robot list.',
-        inputSchema: {
-            id: z.string().describe('CRT robot ID.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
-        },
+        inputSchema: { id: z.string().describe('CRT robot ID.'), project: projectIdSchema },
         title: 'Agentia Testing Robot Get',
     }, (args) => safelyHandleTool(() => testingRobotGet(args)));
     server.registerTool('agentia_testing_robot_delete', {
@@ -772,7 +521,7 @@ export function registerAgentiaMcpTestingTools(server) {
         inputSchema: {
             confirm: z.boolean().describe('Must be true to confirm robot deletion.'),
             id: z.string().describe('CRT robot ID.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
+            project: projectIdSchema,
         },
         title: 'Agentia Testing Robot Delete',
     }, (args) => safelyHandleTool(() => testingRobotDelete(args)));
@@ -781,14 +530,15 @@ export function registerAgentiaMcpTestingTools(server) {
         inputSchema: {
             description: z.string().optional().describe('Robot description.'),
             fwVersion: z.string().optional().describe('Robot framework version.'),
-            kind: z.enum(TESTING_ROBOT_RUN_ENVIRONMENTS).optional().describe('Robot run environment.'),
             label: z.string().optional().describe('Robot label.'),
             name: z.string().describe('Robot name.'),
-            os: z.string().optional().describe('Robot operating system.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
-            reporting: z.enum(TESTING_ROBOT_REPORTING).optional().describe('Testing result retention.'),
-            robotType: z.enum(TESTING_ROBOT_TYPES).optional().describe('CRT robot type.'),
-            runEnvironment: z.enum(TESTING_ROBOT_RUN_ENVIRONMENTS).optional().describe('Robot execution environment.'),
+            project: projectIdSchema,
+            reporting: z.enum(TESTING_ROBOT_REPORTING).optional().describe(TESTING_ROBOT_REPORTING_DESCRIPTION),
+            robotType: z.enum(TESTING_ROBOT_TYPES).optional().describe(TESTING_ROBOT_TYPE_DESCRIPTION),
+            runEnvironment: z
+                .enum(TESTING_ROBOT_ACCEPTED_RUN_ENVIRONMENTS)
+                .optional()
+                .describe(TESTING_ROBOT_RUN_ENVIRONMENT_DESCRIPTION),
         },
         title: 'Agentia Testing Robot Create',
     }, (args) => safelyHandleTool(() => testingRobotCreate(args)));
@@ -800,72 +550,66 @@ export function registerAgentiaMcpTestingTools(server) {
             id: z.string().describe('CRT robot ID.'),
             label: z.string().optional().describe('New robot label.'),
             name: z.string().optional().describe('New robot name.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
-            reporting: z.enum(TESTING_ROBOT_REPORTING).optional().describe('Testing result retention.'),
-            robotType: z.enum(TESTING_ROBOT_TYPES).optional().describe('CRT robot type.'),
+            project: projectIdSchema,
+            reporting: z.enum(TESTING_ROBOT_REPORTING).optional().describe(TESTING_ROBOT_REPORTING_DESCRIPTION),
+            robotType: z.enum(TESTING_ROBOT_TYPES).optional().describe(TESTING_ROBOT_TYPE_DESCRIPTION),
         },
         title: 'Agentia Testing Robot Update',
     }, (args) => safelyHandleTool(() => testingRobotUpdate(args)));
     server.registerTool('agentia_testing_job_list', {
-        description: 'List Copado Robotic Testing jobs/tests.',
+        description: 'List Copado Robotic Testing jobs.',
         inputSchema: {
-            name: z.string().optional().describe('Case-insensitive exact job/test name filter.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
+            name: z.string().optional().describe('Case-insensitive exact job name filter.'),
+            project: projectIdSchema,
         },
         title: 'Agentia Testing Job List',
     }, (args) => safelyHandleTool(() => testingJobList(args)));
     server.registerTool('agentia_testing_job_get', {
-        description: 'Get a Copado Robotic Testing job/test from the project job list.',
-        inputSchema: {
-            id: z.string().describe('CRT job/test ID.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
-        },
+        description: 'Get a Copado Robotic Testing job from the project job list.',
+        inputSchema: { id: z.string().describe('CRT job ID.'), project: projectIdSchema },
         title: 'Agentia Testing Job Get',
     }, (args) => safelyHandleTool(() => testingJobGet(args)));
     server.registerTool('agentia_testing_job_update', {
-        description: 'Update Copado Robotic Testing job/test metadata or Git storage.',
+        description: 'Update Copado Robotic Testing job metadata or Git storage.',
         inputSchema: {
             caseDurationChangeDetection: z.boolean().optional().describe('Enable or disable duration analytics.'),
-            description: z.string().optional().describe('Job/test description.'),
+            description: z.string().optional().describe('Job description.'),
             ...gitStorageSchema,
-            hideVideoParams: z.boolean().optional().describe('Hide video parameters.'),
-            id: z.number().int().positive().describe('CRT job/test ID.'),
-            name: z.string().optional().describe('Job/test name.'),
+            id: jobIdSchema,
+            name: z.string().optional().describe('Job name.'),
             parallelMode: z.enum(TESTING_JOB_PARALLEL_MODES).optional().describe('Parallel execution mode.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
+            project: projectIdSchema,
             robot: z.number().int().positive().optional().describe('CRT robot ID.'),
-            screenDepth: z.number().int().positive().optional().describe('Screen color depth.'),
-            screenHeight: z.number().int().positive().optional().describe('Screen height.'),
-            screenWidth: z.number().int().positive().optional().describe('Screen width.'),
-            showVideoParams: z.boolean().optional().describe('Show video parameters.'),
+            ...jobScreenSchema,
+            showVideoParams: z
+                .boolean()
+                .optional()
+                .describe('Enable or disable video streaming and recording capabilities when running the test job.'),
             suiteType: z.enum(TESTING_JOB_SUITE_TYPES).optional().describe('CRT suite/job type.'),
             timeout: z.string().optional().describe('Build timeout, for example 10m.'),
         },
         title: 'Agentia Testing Job Update',
     }, (args) => safelyHandleTool(() => testingJobUpdate(args)));
     server.registerTool('agentia_testing_job_delete', {
-        description: 'Delete a Copado Robotic Testing job/test.',
+        description: 'Delete a Copado Robotic Testing job.',
         inputSchema: {
             confirm: z.boolean().describe('Must be true to confirm job deletion.'),
-            id: z.number().int().positive().describe('CRT job/test ID.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
+            id: jobIdSchema,
+            project: projectIdSchema,
         },
         title: 'Agentia Testing Job Delete',
     }, (args) => safelyHandleTool(() => testingJobDelete(args)));
     server.registerTool('agentia_testing_job_schedule_get', {
         description: 'Get a Copado Robotic Testing job schedule.',
-        inputSchema: {
-            job: z.number().int().positive().describe('CRT job/test ID.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
-        },
+        inputSchema: { job: jobIdSchema, project: projectIdSchema },
         title: 'Agentia Testing Job Schedule Get',
     }, (args) => safelyHandleTool(() => testingJobScheduleGet(args)));
     server.registerTool('agentia_testing_job_schedule_set', {
         description: 'Create or update a Copado Robotic Testing job schedule.',
         inputSchema: {
             cron: z.string().min(1).describe('Cron expression.'),
-            job: z.number().int().positive().describe('CRT job/test ID.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
+            job: jobIdSchema,
+            project: projectIdSchema,
             record: z.enum(TESTING_RECORD_BUILD_OPTIONS).optional().describe('Scheduled run recording mode.'),
             timezone: z.string().optional().describe('Schedule timezone.'),
         },
@@ -875,8 +619,8 @@ export function registerAgentiaMcpTestingTools(server) {
         description: 'Remove a Copado Robotic Testing job schedule.',
         inputSchema: {
             confirm: z.boolean().describe('Must be true to confirm schedule removal.'),
-            job: z.number().int().positive().describe('CRT job/test ID.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
+            job: jobIdSchema,
+            project: projectIdSchema,
         },
         title: 'Agentia Testing Job Schedule Remove',
     }, (args) => safelyHandleTool(() => testingJobScheduleRemove(args)));
@@ -884,9 +628,9 @@ export function registerAgentiaMcpTestingTools(server) {
         description: 'List Copado Robotic Testing data tables.',
         inputSchema: {
             fields: z.string().optional().describe('Comma-separated response fields.'),
-            job: z.number().int().positive().optional().describe('CRT job/test ID filter.'),
+            job: optionalJobIdSchema,
             name: z.string().optional().describe('Exact table name filter.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
+            project: projectIdSchema,
             revision: z.string().optional().describe('Table revision/version filter.'),
         },
         title: 'Agentia Testing Data Table List',
@@ -895,8 +639,8 @@ export function registerAgentiaMcpTestingTools(server) {
         description: 'Get a Copado Robotic Testing data table.',
         inputSchema: {
             id: z.number().int().positive().describe('CRT data-table ID.'),
-            job: z.number().int().positive().optional().describe('CRT job/test ID filter.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
+            job: optionalJobIdSchema,
+            project: projectIdSchema,
             revision: z.string().optional().describe('Table revision/version.'),
         },
         title: 'Agentia Testing Data Table Get',
@@ -909,9 +653,9 @@ export function registerAgentiaMcpTestingTools(server) {
             data: testingDataMatrixSchema.min(1).describe('Rows with a header row followed by datasets.'),
             defaultRow: z.number().int().positive().optional().describe('Default dataset row.'),
             header: z.enum(['row', 'column']).optional().describe('Input header orientation.'),
-            job: z.number().int().positive().optional().describe('Optional CRT job/test ID scope.'),
+            job: optionalJobIdSchema,
             name: z.string().min(1).describe('Data-table name.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
+            project: projectIdSchema,
             topics: z.array(z.string()).optional().describe('Topic tags.'),
         },
         title: 'Agentia Testing Data Table Create',
@@ -925,8 +669,8 @@ export function registerAgentiaMcpTestingTools(server) {
             defaultRow: z.number().int().positive().optional().describe('Default dataset row.'),
             header: z.enum(['row', 'column']).optional().describe('Input header orientation.'),
             id: z.number().int().positive().describe('CRT data-table ID.'),
-            job: z.number().int().positive().optional().describe('Optional CRT job/test ID scope.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
+            job: optionalJobIdSchema,
+            project: projectIdSchema,
             topics: z.array(z.string()).optional().describe('Replacement topic tags.'),
         },
         title: 'Agentia Testing Data Table Update',
@@ -936,25 +680,26 @@ export function registerAgentiaMcpTestingTools(server) {
         inputSchema: {
             confirm: z.boolean().describe('Must be true to confirm deletion.'),
             id: z.number().int().positive().describe('CRT data-table ID.'),
-            job: z.number().int().positive().optional().describe('Optional CRT job/test ID scope.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
+            job: optionalJobIdSchema,
+            project: projectIdSchema,
         },
         title: 'Agentia Testing Data Table Delete',
     }, (args) => safelyHandleTool(() => testingDataTableDelete(args)));
     server.registerTool('agentia_testing_job_create', {
-        description: 'Create a Copado Robotic Testing job/test from uploaded files or Git storage.',
+        description: 'Create a Copado Robotic Testing job from uploaded files or Git storage.',
         inputSchema: {
             caseDurationChangeDetection: z.boolean().optional().describe('Enable or disable duration analytics.'),
-            description: z.string().optional().describe('Job/test description.'),
+            description: z.string().optional().describe('Job description.'),
             dryRun: z.boolean().optional().describe('Return a sanitized creation plan without network mutation.'),
-            name: z.string().describe('Job/test name.'),
+            name: z.string().describe('Job name.'),
             parallelMode: z.enum(TESTING_JOB_PARALLEL_MODES).optional().describe('Parallel execution mode.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
+            project: projectIdSchema,
             robot: z.string().describe('CRT robot ID.'),
-            screenDepth: z.number().int().positive().optional().describe('Screen color depth.'),
-            screenHeight: z.number().int().positive().optional().describe('Screen height.'),
-            screenWidth: z.number().int().positive().optional().describe('Screen width.'),
-            showVideoParams: z.boolean().optional().describe('Show or hide video parameters.'),
+            ...jobScreenSchema,
+            showVideoParams: z
+                .boolean()
+                .optional()
+                .describe('Enable or disable video streaming and recording capabilities when running the test job.'),
             suiteType: z.enum(TESTING_JOB_SUITE_TYPES).optional().describe('CRT suite/job type.'),
             timeout: z.string().optional().describe('Build timeout, for example 10m.'),
             validation: z.boolean().optional().describe('Ask CRT to validate the created job.'),
@@ -963,111 +708,99 @@ export function registerAgentiaMcpTestingTools(server) {
         title: 'Agentia Testing Job Create',
     }, (args) => safelyHandleTool(() => testingJobCreate(args)));
     server.registerTool('agentia_testing_job_run', {
-        description: 'Run a Copado Robotic Testing job/test.',
+        description: 'Run a Copado Robotic Testing job. Does not wait unless waitForResult or streamLogs is set. Data-driven runs reject waiting.',
         inputSchema: buildRunSchema,
         title: 'Agentia Testing Job Run',
     }, (args) => safelyHandleTool(() => testingBuildRun(args)));
-    server.registerTool('agentia_testing_job_files', {
-        description: 'List files for a Copado Robotic Testing job/test.',
+    server.registerTool('agentia_testing_job_files_list', {
+        description: 'List files for a Copado Robotic Testing job.',
         inputSchema: {
-            job: z.number().int().positive().describe('CRT job/test ID.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
+            branch: z.string().optional().describe('Optional CRT branch name.'),
+            job: jobIdSchema,
+            project: projectIdSchema,
+            sharedCredentials: z.boolean().optional().describe('Use shared stored Git credentials.'),
         },
-        title: 'Agentia Testing Job Files',
+        title: 'Agentia Testing Job Files List',
     }, (args) => safelyHandleTool(() => testingJobFiles(args)));
     server.registerTool('agentia_testing_job_refs', {
-        description: 'List Git branches and tags for a Copado Robotic Testing job/test.',
+        description: 'List Git branches and tags for a Copado Robotic Testing job.',
         inputSchema: {
-            job: z.number().int().positive().describe('CRT job/test ID.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
+            job: jobIdSchema,
+            project: projectIdSchema,
             sharedCredentials: z.boolean().optional().describe('Use shared stored Git credentials.'),
         },
         title: 'Agentia Testing Job Refs',
     }, (args) => safelyHandleTool(() => testingJobRefs(args)));
-    server.registerTool('agentia_testing_job_upload', {
-        description: 'Upload file operations to a Copado Robotic Testing job/test.',
+    server.registerTool('agentia_testing_job_files_upload', {
+        description: 'Upload file operations to a Copado Robotic Testing job.',
         inputSchema: uploadSchema,
-        title: 'Agentia Testing Job Upload',
+        title: 'Agentia Testing Job Files Upload',
     }, (args) => safelyHandleTool(() => testingJobUpload(args)));
-    server.registerTool('agentia_testing_job_download', {
-        description: 'Download files from a Copado Robotic Testing job/test.',
+    server.registerTool('agentia_testing_job_files_download', {
+        description: 'Download files from a Copado Robotic Testing job.',
         inputSchema: {
             branch: z.string().optional().describe('Optional CRT branch name.'),
             file: z.array(z.string()).describe('Remote CRT file paths.'),
-            job: z.number().int().positive().describe('CRT job/test ID.'),
+            job: jobIdSchema,
             outputDir: z.string().optional().describe('Output directory for downloaded files.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
+            project: projectIdSchema,
         },
-        title: 'Agentia Testing Job Download',
+        title: 'Agentia Testing Job Files Download',
     }, (args) => safelyHandleTool(() => testingJobDownload(args)));
-    server.registerTool('agentia_testing_build_run', {
-        description: 'Run a Copado Robotic Testing job/test.',
-        inputSchema: buildRunSchema,
-        title: 'Agentia Testing Build Run',
-    }, (args) => safelyHandleTool(() => testingBuildRun(args)));
-    server.registerTool('agentia_testing_build_list', {
-        description: 'List Copado Robotic Testing builds/runs.',
+    server.registerTool('agentia_testing_job_runs_list', {
+        description: 'List Copado Robotic Testing job runs.',
         inputSchema: buildListSchema,
-        title: 'Agentia Testing Build List',
+        title: 'Agentia Testing Job Runs List',
     }, (args) => safelyHandleTool(() => testingBuildList(args)));
-    server.registerTool('agentia_testing_build_get', {
-        description: 'Get a Copado Robotic Testing build/run.',
+    server.registerTool('agentia_testing_job_runs_get', {
+        description: 'Get the result of a Copado Robotic Testing job run.',
         inputSchema: {
             build: z.number().int().positive().describe('CRT build/run ID.'),
             full: z.boolean().optional().describe('Fetch the full build result.'),
-            job: z.number().int().positive().describe('CRT job/test ID.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
+            job: jobIdSchema,
+            project: projectIdSchema,
         },
-        title: 'Agentia Testing Build Get',
+        title: 'Agentia Testing Job Runs Get',
     }, (args) => safelyHandleTool(() => testingBuildGet(args)));
-    server.registerTool('agentia_testing_build_latest', {
-        description: 'List the latest Copado Robotic Testing builds/runs.',
-        inputSchema: {
-            project: z.number().int().positive().describe('CRT project ID.'),
-            resultSize: z.enum(['compact']).optional().describe('CRT result size.'),
-            status: z.enum(TESTING_BUILD_STATUSES).optional().describe('Build status filter.'),
-        },
-        title: 'Agentia Testing Build Latest',
-    }, (args) => safelyHandleTool(() => testingBuildLatest(args)));
-    server.registerTool('agentia_testing_build_search', {
-        description: 'Search Copado Robotic Testing build metrics.',
+    server.registerTool('agentia_testing_job_runs_search', {
+        description: 'Search Copado Robotic Testing job run metrics.',
         inputSchema: {
             count: z.boolean().optional().describe('Return only the filtered count.'),
             from: z.string().optional().describe('Start date or epoch filter.'),
-            job: z.number().int().positive().describe('CRT job/test ID.'),
+            job: jobIdSchema,
             order: z.enum(['asc', 'desc']).optional().describe('Sort order.'),
             page: z.number().int().positive().optional().describe('Result page.'),
             pageSize: z.number().int().positive().optional().describe('Result page size.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
+            project: projectIdSchema,
             robot: z.array(z.string()).optional().describe('Robot ID filters.'),
             runType: z.array(z.enum(TESTING_RUN_TYPES)).optional().describe('Run type filters.'),
             sort: z.enum(['date', 'duration', 'id', 'status']).optional().describe('Sort field.'),
             status: z.array(z.string()).optional().describe('Status filters.'),
             to: z.string().optional().describe('End date or epoch filter.'),
         },
-        title: 'Agentia Testing Build Search',
+        title: 'Agentia Testing Job Runs Search',
     }, (args) => safelyHandleTool(() => testingBuildSearch(args)));
-    server.registerTool('agentia_testing_build_logs', {
-        description: 'Get logs for a Copado Robotic Testing build/run.',
+    server.registerTool('agentia_testing_job_runs_logs', {
+        description: 'Get logs for a Copado Robotic Testing job run.',
         inputSchema: buildLogSchema,
-        title: 'Agentia Testing Build Logs',
+        title: 'Agentia Testing Job Runs Logs',
     }, (args) => safelyHandleTool(() => testingBuildLogs(args)));
-    server.registerTool('agentia_testing_build_abort', {
-        description: 'Abort a Copado Robotic Testing build/run.',
+    server.registerTool('agentia_testing_job_runs_abort', {
+        description: 'Abort a Copado Robotic Testing job run.',
         inputSchema: {
             build: z.number().int().positive().describe('CRT build/run ID.'),
             confirm: z.boolean().describe('Must be true to confirm the abort.'),
-            job: z.number().int().positive().describe('CRT job/test ID.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
+            job: jobIdSchema,
+            project: projectIdSchema,
         },
-        title: 'Agentia Testing Build Abort',
+        title: 'Agentia Testing Job Runs Abort',
     }, (args) => safelyHandleTool(() => testingBuildAbort(args)));
     server.registerTool('agentia_testing_variable_list', {
         description: 'List Copado Robotic Testing variables/secrets.',
         inputSchema: {
-            job: z.number().int().positive().optional().describe('Job/test ID filter.'),
+            job: optionalJobIdSchema,
             key: z.string().optional().describe('Exact variable key filter.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
+            project: projectIdSchema,
             robot: z.number().int().positive().optional().describe('Robot ID filter.'),
             sensitive: z.boolean().optional().describe('Sensitive status filter.'),
             type: z.enum(TESTING_VARIABLE_TYPES).optional().describe('Variable type filter.'),
@@ -1077,19 +810,16 @@ export function registerAgentiaMcpTestingTools(server) {
     }, (args) => safelyHandleTool(() => testingVariableList(args)));
     server.registerTool('agentia_testing_variable_get', {
         description: 'Get a Copado Robotic Testing variable/secret with value redaction.',
-        inputSchema: {
-            id: z.string().describe('CRT variable/secret ID.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
-        },
+        inputSchema: { id: z.string().describe('CRT variable/secret ID.'), project: projectIdSchema },
         title: 'Agentia Testing Variable Get',
     }, (args) => safelyHandleTool(() => testingVariableGet(args)));
     server.registerTool('agentia_testing_variable_create', {
         description: 'Create a Copado Robotic Testing variable/secret.',
         inputSchema: {
             description: z.string().optional().describe('Variable description.'),
-            job: z.number().int().positive().optional().describe('CRT job/test ID scope.'),
+            job: optionalJobIdSchema,
             key: z.string().describe('Variable key.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
+            project: projectIdSchema,
             robot: z.number().int().positive().optional().describe('CRT robot ID scope.'),
             sensitive: z.boolean().optional().describe('Mark variable as sensitive.'),
             type: z.enum(TESTING_VARIABLE_TYPES).optional().describe('Variable type.'),
@@ -1105,7 +835,7 @@ export function registerAgentiaMcpTestingTools(server) {
             id: z.string().describe('CRT variable/secret ID.'),
             key: z.string().optional().describe('Variable key.'),
             notSensitive: z.boolean().optional().describe('Mark variable as not sensitive.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
+            project: projectIdSchema,
             sensitive: z.boolean().optional().describe('Mark variable as sensitive.'),
             type: z.enum(TESTING_VARIABLE_TYPES).optional().describe('Variable type.'),
             value: z.string().max(1_048_576).optional().describe('Structured variable value; redacted from output.'),
@@ -1117,7 +847,7 @@ export function registerAgentiaMcpTestingTools(server) {
         inputSchema: {
             confirm: z.boolean().describe('Must be true to confirm variable deletion.'),
             id: z.string().describe('CRT variable/secret ID.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
+            project: projectIdSchema,
         },
         title: 'Agentia Testing Variable Delete',
     }, (args) => safelyHandleTool(() => testingVariableDelete(args)));
@@ -1165,47 +895,6 @@ export function registerAgentiaMcpTestingTools(server) {
         },
         title: 'Agentia Testing User Parameter Delete',
     }, (args) => safelyHandleTool(() => testingUserParameterDelete(args)));
-    server.registerTool('agentia_testing_create', {
-        description: 'Create a Copado Robotic Testing test from uploaded files or Git storage.',
-        inputSchema: {
-            caseDurationChangeDetection: z.boolean().optional().describe('Enable or disable duration analytics.'),
-            description: z.string().optional().describe('Test description.'),
-            dryRun: z.boolean().optional().describe('Return a sanitized creation plan without network mutation.'),
-            name: z.string().describe('Test name.'),
-            parallelMode: z.enum(TESTING_JOB_PARALLEL_MODES).optional().describe('Parallel execution mode.'),
-            project: z.number().int().positive().describe('CRT project ID.'),
-            robot: z.string().describe('CRT robot ID.'),
-            screenDepth: z.number().int().positive().optional().describe('Screen color depth.'),
-            screenHeight: z.number().int().positive().optional().describe('Screen height.'),
-            screenWidth: z.number().int().positive().optional().describe('Screen width.'),
-            showVideoParams: z.boolean().optional().describe('Show or hide video parameters.'),
-            suiteType: z.enum(TESTING_JOB_SUITE_TYPES).optional().describe('CRT suite/job type.'),
-            timeout: z.string().optional().describe('Build timeout, for example 10m.'),
-            validation: z.boolean().optional().describe('Ask CRT to validate the created job.'),
-            ...gitStorageSchema,
-        },
-        title: 'Agentia Testing Create',
-    }, (args) => safelyHandleTool(() => testingJobCreate(args)));
-    server.registerTool('agentia_testing_update', {
-        description: 'Update a Copado Robotic Testing test by uploading file operations.',
-        inputSchema: uploadSchema,
-        title: 'Agentia Testing Update',
-    }, (args) => safelyHandleTool(() => testingJobUpload(args)));
-    server.registerTool('agentia_testing_run', {
-        description: 'Run a Copado Robotic Testing test.',
-        inputSchema: buildRunSchema,
-        title: 'Agentia Testing Run',
-    }, (args) => safelyHandleTool(() => testingBuildRun(args)));
-    server.registerTool('agentia_testing_status', {
-        description: 'List Copado Robotic Testing run status.',
-        inputSchema: buildListSchema,
-        title: 'Agentia Testing Status',
-    }, (args) => safelyHandleTool(() => testingBuildList(args)));
-    server.registerTool('agentia_testing_logs', {
-        description: 'Get logs for a Copado Robotic Testing run.',
-        inputSchema: buildLogSchema,
-        title: 'Agentia Testing Logs',
-    }, (args) => safelyHandleTool(() => testingBuildLogs(args)));
     registerTestingLiveMcpTools(server);
 }
 //# sourceMappingURL=testing-tools.js.map