@devicecloud.dev/dcd 5.2.0-beta.1 → 5.2.0-beta.3

package/dist/commands/cloud.d.ts

@@ -185,13 +185,13 @@ export declare const cloudCommand: import("citty").CommandDef<{
         readonly type: "string";
         readonly description: `[iOS only] iOS version to run your flow against (options: ${string})`;
     };
-    readonly 'ios-config': {
+    readonly 'ios-device-matrix': {
         readonly type: "string";
-        readonly description: "[iOS only] Device-matrix cell as <device>:<version>, e.g. iphone-16:18. Repeatable — every flow runs once per cell (no cross-product). Cannot be combined with --android-config.";
+        readonly description: "[iOS only] Device-matrix cell as <device>:<version>, e.g. iphone-16:18. Repeatable — every flow runs once per cell (no cross-product). Cannot be combined with --android-device-matrix.";
     };
-    readonly 'android-config': {
+    readonly 'android-device-matrix': {
         readonly type: "string";
-        readonly description: "[Android only] Device-matrix cell as <device>:<apiLevel> (append :play for Google Play), e.g. pixel-7:34 or pixel-7:34:play. Repeatable — every flow runs once per cell (no cross-product). Cannot be combined with --ios-config.";
+        readonly description: "[Android only] Device-matrix cell as <device>:<apiLevel> (append :play for Google Play), e.g. pixel-7:34 or pixel-7:34:play. Repeatable — every flow runs once per cell (no cross-product). Cannot be combined with --ios-device-matrix.";
     };
     readonly orientation: {
         readonly type: "string";

package/dist/commands/cloud.js

@@ -16,7 +16,7 @@ import { TestSubmissionService } from '../services/test-submission.service.js';
 import { VersionService } from '../services/version.service.js';
 import { EAndroidApiLevels, EAndroidDevices, EiOSDevices, EiOSVersions, isIosMatrixConfig, } from '../types/domain/device.types.js';
 import { resolveAuth } from '../utils/auth.js';
-import { matrixIsIos, parseDeviceMatrix } from '../utils/device-matrix.js';
+import { assertMatrixSupported, matrixIsIos, parseDeviceMatrix, } from '../utils/device-matrix.js';
 import { detectCiContext, isCI } from '../utils/ci.js';
 import { CliError, coerceArray, collectRepeatedFlag, getCliVersion, getUpgradeCommand, logger, parseIntFlag, validateEnum, } from '../utils/cli.js';
 import { fetchCompatibilityData, } from '../utils/compatibility.js';
@@ -139,9 +139,9 @@ export const cloudCommand = defineCommand({
             const androidDevice = validateEnum(args['android-device'], Object.values(EAndroidDevices), 'android-device');
             const androidNoSnapshot = Boolean(args['android-no-snapshot']);
             // Repeatable device-matrix flags: one validated cell each, no cross-product.
-            const iosConfigFlags = collectRepeatedFlag(rawArgs, ['--ios-config']);
-            const androidConfigFlags = collectRepeatedFlag(rawArgs, ['--android-config']);
-            const deviceMatrix = parseDeviceMatrix(iosConfigFlags, androidConfigFlags);
+            const iosMatrixFlags = collectRepeatedFlag(rawArgs, ['--ios-device-matrix']);
+            const androidMatrixFlags = collectRepeatedFlag(rawArgs, ['--android-device-matrix']);
+            const deviceMatrix = parseDeviceMatrix(iosMatrixFlags, androidMatrixFlags);
             const json = Boolean(args.json);
             const jsonFileFlag = Boolean(args['json-file']);
             const jsonFileName = args['json-file-name'];
@@ -433,8 +433,8 @@ export const cloudCommand = defineCommand({
                 'include-tags': includeTags,
                 'exclude-tags': excludeTags,
                 'exclude-flows': excludeFlows,
-                'ios-config': iosConfigFlags,
-                'android-config': androidConfigFlags,
+                'ios-device-matrix': iosMatrixFlags,
+                'android-device-matrix': androidMatrixFlags,
             };
             for (const [k, v] of Object.entries(args)) {
                 if (!canonicalFlagKeys.has(k))
@@ -560,6 +560,10 @@ export const cloudCommand = defineCommand({
             // there is no matrix, and tolerant of older APIs that lack the endpoint.
             if (deviceMatrix.length > 0) {
                 const estimate = await ApiGateway.estimateMatrix(apiUrl, auth, fields);
+                // A null estimate means the API predates the matrix (404/405). It would
+                // silently strip deviceMatrix and run one device — refuse rather than
+                // hand back a green single-device run the user reads as a matrix.
+                assertMatrixSupported(deviceMatrix, estimate);
                 if (estimate) {
                     const osPrefix = matrixIsIos(deviceMatrix) ? 'iOS' : 'API';
                     const rows = ui.fields([

package/dist/config/flags/device.flags.d.ts

@@ -27,13 +27,13 @@ export declare const deviceFlags: {
         readonly type: "string";
         readonly description: `[iOS only] iOS version to run your flow against (options: ${string})`;
     };
-    readonly 'ios-config': {
+    readonly 'ios-device-matrix': {
         readonly type: "string";
-        readonly description: "[iOS only] Device-matrix cell as <device>:<version>, e.g. iphone-16:18. Repeatable — every flow runs once per cell (no cross-product). Cannot be combined with --android-config.";
+        readonly description: "[iOS only] Device-matrix cell as <device>:<version>, e.g. iphone-16:18. Repeatable — every flow runs once per cell (no cross-product). Cannot be combined with --android-device-matrix.";
     };
-    readonly 'android-config': {
+    readonly 'android-device-matrix': {
         readonly type: "string";
-        readonly description: "[Android only] Device-matrix cell as <device>:<apiLevel> (append :play for Google Play), e.g. pixel-7:34 or pixel-7:34:play. Repeatable — every flow runs once per cell (no cross-product). Cannot be combined with --ios-config.";
+        readonly description: "[Android only] Device-matrix cell as <device>:<apiLevel> (append :play for Google Play), e.g. pixel-7:34 or pixel-7:34:play. Repeatable — every flow runs once per cell (no cross-product). Cannot be combined with --ios-device-matrix.";
     };
     readonly orientation: {
         readonly type: "string";

package/dist/config/flags/device.flags.js

@@ -32,13 +32,13 @@ export const deviceFlags = {
         type: 'string',
         description: `[iOS only] iOS version to run your flow against (options: ${iosVersions})`,
     },
-    'ios-config': {
+    'ios-device-matrix': {
         type: 'string',
-        description: `[iOS only] Device-matrix cell as <device>:<version>, e.g. iphone-16:18. Repeatable — every flow runs once per cell (no cross-product). Cannot be combined with --android-config.`,
+        description: `[iOS only] Device-matrix cell as <device>:<version>, e.g. iphone-16:18. Repeatable — every flow runs once per cell (no cross-product). Cannot be combined with --android-device-matrix.`,
     },
-    'android-config': {
+    'android-device-matrix': {
         type: 'string',
-        description: `[Android only] Device-matrix cell as <device>:<apiLevel> (append :play for Google Play), e.g. pixel-7:34 or pixel-7:34:play. Repeatable — every flow runs once per cell (no cross-product). Cannot be combined with --ios-config.`,
+        description: `[Android only] Device-matrix cell as <device>:<apiLevel> (append :play for Google Play), e.g. pixel-7:34 or pixel-7:34:play. Repeatable — every flow runs once per cell (no cross-product). Cannot be combined with --ios-device-matrix.`,
     },
     orientation: {
         type: 'string',

package/dist/constants.d.ts

@@ -171,13 +171,13 @@ export declare const flags: {
         readonly type: "string";
         readonly description: `[iOS only] iOS version to run your flow against (options: ${string})`;
     };
-    readonly 'ios-config': {
+    readonly 'ios-device-matrix': {
         readonly type: "string";
-        readonly description: "[iOS only] Device-matrix cell as <device>:<version>, e.g. iphone-16:18. Repeatable — every flow runs once per cell (no cross-product). Cannot be combined with --android-config.";
+        readonly description: "[iOS only] Device-matrix cell as <device>:<version>, e.g. iphone-16:18. Repeatable — every flow runs once per cell (no cross-product). Cannot be combined with --android-device-matrix.";
     };
-    readonly 'android-config': {
+    readonly 'android-device-matrix': {
         readonly type: "string";
-        readonly description: "[Android only] Device-matrix cell as <device>:<apiLevel> (append :play for Google Play), e.g. pixel-7:34 or pixel-7:34:play. Repeatable — every flow runs once per cell (no cross-product). Cannot be combined with --ios-config.";
+        readonly description: "[Android only] Device-matrix cell as <device>:<apiLevel> (append :play for Google Play), e.g. pixel-7:34 or pixel-7:34:play. Repeatable — every flow runs once per cell (no cross-product). Cannot be combined with --ios-device-matrix.";
     };
     readonly orientation: {
         readonly type: "string";

package/dist/utils/device-matrix.d.ts

@@ -1,7 +1,21 @@
 import { DeviceMatrixConfig } from '../types/domain/device.types.js';
 /**
- * Parse repeated `--ios-config <device>:<version>` and
- * `--android-config <device>:<apiLevel>[:play]` flags into an explicit device
+ * Refuse to submit a device matrix to an API that cannot honour it.
+ *
+ * The estimate endpoint is only called when a matrix was actually requested, so
+ * a null estimate (the gateway maps 404/405 to null) means the API predates the
+ * feature. That API would **silently strip** the unknown `deviceMatrix` field —
+ * its ValidationPipe runs `whitelist: true, forbidNonWhitelisted: false` — and
+ * run every flow on a single default device, exiting 0. The user would believe
+ * they had tested N devices when they tested one: the exact silent
+ * under-testing the device matrix exists to prevent. Fail loudly instead.
+ *
+ * @throws CliError when a matrix was requested but the API does not support it.
+ */
+export declare function assertMatrixSupported(deviceMatrix: DeviceMatrixConfig[], estimate: unknown | null): void;
+/**
+ * Parse repeated `--ios-device-matrix <device>:<version>` and
+ * `--android-device-matrix <device>:<apiLevel>[:play]` flags into an explicit device
  * matrix. Each entry is one validated cell — there is NO cross-product, because
  * the compatibility matrix is ragged and a cross-product would invent cells the
  * user never asked for.

package/dist/utils/device-matrix.js

@@ -1,8 +1,30 @@
 import { isIosMatrixConfig, } from '../types/domain/device.types.js';
 import { CliError } from './cli.js';
 /**
- * Parse repeated `--ios-config <device>:<version>` and
- * `--android-config <device>:<apiLevel>[:play]` flags into an explicit device
+ * Refuse to submit a device matrix to an API that cannot honour it.
+ *
+ * The estimate endpoint is only called when a matrix was actually requested, so
+ * a null estimate (the gateway maps 404/405 to null) means the API predates the
+ * feature. That API would **silently strip** the unknown `deviceMatrix` field —
+ * its ValidationPipe runs `whitelist: true, forbidNonWhitelisted: false` — and
+ * run every flow on a single default device, exiting 0. The user would believe
+ * they had tested N devices when they tested one: the exact silent
+ * under-testing the device matrix exists to prevent. Fail loudly instead.
+ *
+ * @throws CliError when a matrix was requested but the API does not support it.
+ */
+export function assertMatrixSupported(deviceMatrix, estimate) {
+    if (deviceMatrix.length === 0 || estimate)
+        return;
+    throw new CliError('This DeviceCloud API does not support device matrices, so ' +
+        '--ios-device-matrix / --android-device-matrix cannot be honoured. ' +
+        'Submitting anyway would silently run every flow on a single default ' +
+        'device and report success. Upgrade the API, or drop the matrix flags ' +
+        'and use --ios-device / --android-device for a single-device run.');
+}
+/**
+ * Parse repeated `--ios-device-matrix <device>:<version>` and
+ * `--android-device-matrix <device>:<apiLevel>[:play]` flags into an explicit device
  * matrix. Each entry is one validated cell — there is NO cross-product, because
  * the compatibility matrix is ragged and a cross-product would invent cells the
  * user never asked for.
@@ -14,13 +36,13 @@ import { CliError } from './cli.js';
  */
 export function parseDeviceMatrix(iosConfigs, androidConfigs) {
     if (iosConfigs.length > 0 && androidConfigs.length > 0) {
-        throw new CliError('A device matrix cannot mix platforms: use either --ios-config or --android-config, not both. One upload runs one binary.');
+        throw new CliError('A device matrix cannot mix platforms: use either --ios-device-matrix or --android-device-matrix, not both. One upload runs one binary.');
     }
     const configs = [];
     for (const raw of iosConfigs) {
         const parts = raw.split(':');
         if (parts.length !== 2 || !parts[0] || !parts[1]) {
-            throw new CliError(`Invalid --ios-config "${raw}". Expected <device>:<version>, e.g. iphone-16:18.`);
+            throw new CliError(`Invalid --ios-device-matrix "${raw}". Expected <device>:<version>, e.g. iphone-16:18.`);
         }
         configs.push({ iOSDevice: parts[0], iOSVersion: parts[1] });
     }
@@ -32,7 +54,7 @@ export function parseDeviceMatrix(iosConfigs, androidConfigs) {
             !parts[0] ||
             !parts[1] ||
             (parts.length === 3 && parts[2] !== 'play')) {
-            throw new CliError(`Invalid --android-config "${raw}". Expected <device>:<apiLevel> or <device>:<apiLevel>:play, e.g. pixel-7:34 or pixel-7:34:play.`);
+            throw new CliError(`Invalid --android-device-matrix "${raw}". Expected <device>:<apiLevel> or <device>:<apiLevel>:play, e.g. pixel-7:34 or pixel-7:34:play.`);
         }
         configs.push({
             androidDevice: parts[0],

package/package.json

@@ -60,7 +60,7 @@
     "type": "git",
     "url": "https://devicecloud.dev"
   },
-  "version": "5.2.0-beta.1",
+  "version": "5.2.0-beta.3",
   "bugs": {
     "url": "https://discord.gg/gm3mJwcNw8"
   },