shipscreens 0.1.0 → 0.1.2

package/README.md

@@ -7,14 +7,6 @@ Build store screenshots as a build step.
 npx --yes shipscreens@latest demo
 ```
 
-## Rendered mode (optional)
-Rendered mode runs the UI renderer headlessly via Playwright Chromium.
-Install Chromium once per machine/CI runner:
-
-```bash
-npx playwright install --with-deps chromium
-```
-
 ## CI pipeline (copy/paste)
 ```bash
 npx --yes shipscreens@latest validate

package/bin/shipscreens.js

@@ -2,7 +2,9 @@
 import { parseArgs } from 'node:util';
 import fs from 'node:fs/promises';
 import path from 'node:path';
+import { fileURLToPath } from 'node:url';
 import { resolveDefaultConfigPath } from '../cli/loadConfig.js';
+import { DEVICE_GROUPS, DEVICE_LIBRARY } from '../src/constants/devices.js';
 import { runValidate } from '../cli/commands/validate.js';
 import { runLock } from '../cli/commands/lock.js';
 import { runBuild } from '../cli/commands/build.js';
@@ -25,6 +27,9 @@ import {
   PLACEHOLDER_WIDTH,
 } from '../cli/commands/init.js';
 
+const PACKAGE_ROOT = path.resolve(path.dirname(fileURLToPath(import.meta.url)), '..');
+const DEMO_SCREENS_DIR = path.join(PACKAGE_ROOT, 'cli', 'assets', 'demo-screens');
+
 function printHelp() {
   console.log(`ShipScreens CLI
 
@@ -33,6 +38,8 @@ Usage:
   shipscreens wizard [--dir <path>] [--force] [--style <value>] [--template <id>] [--defaults]
   shipscreens demo [--out <dir>] [--slides <n>] [--locales <id,id>] [--devices <id,id>] [--template <id>] [--mode <raw|rendered>] [--zip-out <path>] [--profile <appstore|playstore|any>] [--force]
   shipscreens agent-run [--dir <path>] [--slides <n>] [--locales <id,id>] [--devices <id,id>] [--style <value>] [--template <id>] [--mode <raw|rendered>] [--out <dir>] [--zip] [--profile <appstore|playstore|any>] [--require-pro] [--force]
+  shipscreens devices [--json]
+  shipscreens templates [--json]
   shipscreens validate [-c shipscreens.config.yml]
   shipscreens lock [-c shipscreens.config.yml] [--out shipscreens.lock.json] [--check]
   shipscreens build [-c shipscreens.config.yml] [--mode shipscreens|rendered|raw|frameit] [--format dir|zip] [--out <dir>] [--zip] [--zip-out <path>] [--clean]
@@ -48,6 +55,8 @@ Commands:
   wizard     Step-by-step setup (locales/devices/template)
   demo       Create a demo project + export (no assets required)
   agent-run  Non-interactive end-to-end flow (init -> validate -> build -> verify)
+  devices    List device ids (for shipscreens.config.yml)
+  templates  List style template ids (for init/wizard/agent-run)
   validate   Validate shipscreens.config.yml (schema + asset paths)
   lock       Generate shipscreens.lock.json with sha256 hashes
   build      Generate Fastlane-ready outputs (headless Chromium, zero-padded filenames)
@@ -76,7 +85,7 @@ Options:
   --profile <appstore|playstore|any>  Strict verification/staging rules per store (default: any)
   --from <dir>          Source output root for staging (default: config dir)
   --check               Verify outputs are valid/up-to-date (no write)
-  --json                Emit machine-readable JSON (plan)
+  --json                Emit machine-readable JSON (most commands; plan always emits JSON)
   --mode <shipscreens|rendered|raw|frameit> Build mode (default: rendered). shipscreens=rendered alias, frameit runs raw + export framefile --check
   --format <dir|zip>    Output format (default: dir)
   --zip                 Also write a ZIP (same as --format zip)
@@ -165,6 +174,26 @@ async function pathExists(targetPath) {
   }
 }
 
+async function loadDemoScreenPngs() {
+  try {
+    const entries = await fs.readdir(DEMO_SCREENS_DIR);
+    const pngNames = entries
+      .filter((name) => String(name).toLowerCase().endsWith('.png'))
+      .sort((a, b) => (a < b ? -1 : a > b ? 1 : 0));
+    if (!pngNames.length) return [];
+
+    const buffers = [];
+    for (const name of pngNames) {
+      // eslint-disable-next-line no-await-in-loop
+      buffers.push(await fs.readFile(path.join(DEMO_SCREENS_DIR, name)));
+    }
+    return buffers;
+  } catch (err) {
+    void err;
+    return [];
+  }
+}
+
 async function findPlaceholderAssets({ configPath }) {
   const { config, configDir } = await loadShipScreensConfig(configPath);
   const placeholder = createSolidPng({
@@ -235,12 +264,75 @@ async function main() {
     },
   });
 
+  const emitJson = Boolean(values.json);
+  const jsonEnvelope = (command, payload) => ({ version: 1, ok: true, command, ...payload });
+  const emitPayload = (command, payload, textLines) => {
+    if (emitJson) {
+      process.stdout.write(JSON.stringify(jsonEnvelope(command, payload), null, 2) + '\n');
+      return;
+    }
+    if (Array.isArray(textLines)) {
+      console.log(textLines.join('\n'));
+      return;
+    }
+    console.log(String(textLines || ''));
+  };
+
   const command = positionals[0];
   if (values.help || !command || command === 'help') {
     printHelp();
     return;
   }
 
+  if (command === 'devices') {
+    const devices = Object.entries(DEVICE_LIBRARY)
+      .map(([id, def]) => ({ id, ...def }))
+      .sort((a, b) => {
+        const groupA = String(a.group || '');
+        const groupB = String(b.group || '');
+        if (groupA !== groupB) return groupA < groupB ? -1 : 1;
+        const nameA = String(a.name || a.id);
+        const nameB = String(b.name || b.id);
+        return nameA < nameB ? -1 : nameA > nameB ? 1 : 0;
+      });
+
+    emitPayload(
+      'devices',
+      { devices, groups: DEVICE_GROUPS },
+      [
+        `Device ids (${devices.length}):`,
+        ...devices.map((d) => `- ${d.id}  ${d.width}x${d.height}  ${d.group || 'Unknown'}  ${d.name || ''}`.trim()),
+        '',
+        `Tip: devices[].platform is inferred for known device ids. For multiple Android devices, set devices[].androidBucket explicitly.`,
+      ]
+    );
+    return;
+  }
+
+  if (command === 'templates') {
+    const templates = listTemplateIds()
+      .map((id) => {
+        const resolved = resolveTemplateConfig(id);
+        if (!resolved) return null;
+        const { template, style, assets } = resolved;
+        return {
+          id: template.id,
+          name: template.name,
+          tags: template.tags || {},
+          style: style || null,
+          assets: assets || null,
+        };
+      })
+      .filter(Boolean);
+
+    emitPayload(
+      'templates',
+      { templates },
+      [`Templates (${templates.length}):`, ...templates.map((t) => `- ${t.id}  ${t.name}`)]
+    );
+    return;
+  }
+
   if (command === 'auth') {
     await runAuth({ subcommand: positionals[1], values });
     return;
@@ -292,6 +384,8 @@ async function main() {
     });
 
     // Overwrite placeholders with deterministic, non-placeholder demo PNGs.
+    // Prefer realistic UI samples (bundled with the npm package). Fall back to solid colors.
+    const demoScreens = await loadDemoScreenPngs();
     const demoColors = [
       [59, 130, 246, 255], // blue
       [16, 185, 129, 255], // emerald
@@ -304,19 +398,61 @@ async function main() {
     for (let localeIndex = 0; localeIndex < initResult.locales.length; localeIndex += 1) {
       const localeId = initResult.locales[localeIndex];
       for (let i = 1; i <= initResult.slides; i += 1) {
-        const color = demoColors[(i - 1 + localeIndex) % demoColors.length];
-        const png = createSolidPng({
-          width: PLACEHOLDER_WIDTH,
-          height: PLACEHOLDER_HEIGHT,
-          color,
-        });
+        const png =
+          demoScreens.length
+            ? demoScreens[(i - 1) % demoScreens.length]
+            : (() => {
+                const color = demoColors[(i - 1 + localeIndex) % demoColors.length];
+                return createSolidPng({
+                  width: PLACEHOLDER_WIDTH,
+                  height: PLACEHOLDER_HEIGHT,
+                  color,
+                });
+              })();
         const index = formatDemoIndex(i, indexWidth);
         const assetPath = path.join(projectDir, 'assets', localeId, `${index}.png`);
         await fs.writeFile(assetPath, png);
       }
     }
 
+    // Replace placeholder copy with a more meaningful demo (still safe: no translation/upload/speed claims).
+    const demoCopyByLocale = {
+      // Keep demo copy ASCII-only so it renders consistently across environments/fonts.
+      'en-US': [
+        ['Screenshots as a build step', 'wizard -> build -> verify -> tree'],
+        ['Fastlane-ready outputs', 'deliver (iOS) + supply (Android)'],
+        ['Reproducible exports', 'same inputs -> same outputs'],
+        ['Verify before upload', 'catch mixed folders early'],
+        ['Try it free', 'sample -> edit -> export'],
+      ],
+      // If the demo includes non-English locales, reuse the same copy to avoid implying translation is built-in.
+      'ja-JP': [
+        ['Screenshots as a build step', 'wizard -> build -> verify -> tree'],
+        ['Fastlane-ready outputs', 'deliver (iOS) + supply (Android)'],
+        ['Reproducible exports', 'same inputs -> same outputs'],
+        ['Verify before upload', 'catch mixed folders early'],
+        ['Try it free', 'sample -> edit -> export'],
+      ],
+    };
+    const demoCsvHeader = ['Locale'];
+    for (let i = 1; i <= initResult.slides; i += 1) {
+      demoCsvHeader.push(`Header_${i}`, `Subtitle_${i}`);
+    }
+    const demoCsvRows = [demoCsvHeader.join(',')];
+    for (const localeId of initResult.locales) {
+      const fallback = demoCopyByLocale['en-US'] || [];
+      const pairs = demoCopyByLocale[localeId] || fallback;
+      const row = [localeId];
+      for (let i = 1; i <= initResult.slides; i += 1) {
+        const pair = pairs[i - 1] || fallback[i - 1] || [`Slide ${i}`, 'Edit copy.csv -> rerun export'];
+        row.push(pair[0], pair[1]);
+      }
+      demoCsvRows.push(row.join(','));
+    }
+    await fs.writeFile(initResult.csvPath, `${demoCsvRows.join('\n')}\n`, 'utf8');
+
     const configPath = path.join(projectDir, 'shipscreens.config.yml');
+    await runCopyImport({ configPath, csvPath: initResult.csvPath, mode: 'replace' });
     await runValidate({ configPath });
     await runLock({ configPath });
 
@@ -335,7 +471,7 @@ async function main() {
 
     const rawZipOut = typeof values['zip-out'] === 'string' ? values['zip-out'].trim() : '';
     const zipOut = rawZipOut || undefined;
-    const { token: cliAuthToken } = await resolveCliAuth({ requirePro: false });
+    const { token: cliAuthToken, profile: cliAuthProfile } = await resolveCliAuth({ requirePro: false });
 
     const result = await runBuild({
       configPath,
@@ -345,6 +481,7 @@ async function main() {
       zipOut,
       clean: true,
       cliAuthToken,
+      cliAuthProfile,
     });
 
     const verifyResult = await runVerify({
@@ -372,7 +509,22 @@ async function main() {
       `stage -> ${stageResult.out}`,
       `project -> ${projectDir}`,
     ];
-    console.log(summary.join(', '));
+    emitPayload(
+      'demo',
+      {
+        demoRoot,
+        projectDir,
+        buildOutDir,
+        stageOutDir,
+        mode,
+        profile,
+        init: initResult,
+        build: result,
+        verify: verifyResult,
+        stage: stageResult,
+      },
+      summary.join(', ')
+    );
     return;
   }
 
@@ -398,7 +550,9 @@ async function main() {
           style: stylePreset.presetStyle,
           force: values.force,
         });
-    console.log(
+    emitPayload(
+      'init',
+      result,
       `OK: created starter files in ${result.dir}\n` +
         `- ${result.configPath}\n` +
         `- ${result.csvPath}\n` +
@@ -459,7 +613,7 @@ async function main() {
     const defaultProfile = hasIos && !hasAndroid ? 'appstore' : !hasIos && hasAndroid ? 'playstore' : 'any';
     const profile = values.profile || defaultProfile;
     const requirePro = Boolean(values['require-pro']);
-    const { token: cliAuthToken } = await resolveCliAuth({ requirePro });
+    const { token: cliAuthToken, profile: cliAuthProfile } = await resolveCliAuth({ requirePro });
 
     const result = await runBuild({
       configPath,
@@ -473,6 +627,7 @@ async function main() {
       framefileStrategy: values['framefile-strategy'],
       clean: values.clean,
       cliAuthToken,
+      cliAuthProfile,
     });
 
     const verifyResult = await runVerify({
@@ -493,7 +648,21 @@ async function main() {
       `${verifyResult.androidDevices} Android device(s) -> ${verifyResult.androidDir}`,
     ];
     if (result.zipPath || verifyResult.zipPath) summary.push(`zip -> ${result.zipPath || verifyResult.zipPath}`);
-    console.log(summary.join(', '));
+    emitPayload(
+      'agent-run',
+      {
+        configPath,
+        outDir,
+        zipOut: zipOut || null,
+        format,
+        mode,
+        profile,
+        requirePro,
+        build: result,
+        verify: verifyResult,
+      },
+      summary.join(', ')
+    );
     return;
   }
 
@@ -513,7 +682,9 @@ async function main() {
         csvPath: values.csv,
         mode: values['import-mode'],
       });
-      console.log(
+      emitPayload(
+        'copy import',
+        result,
         `OK: imported copy.csv (${result.mode}) -> ${result.configPath}\n` +
           `- locales: ${result.locales}\n` +
           `- slides: ${result.slides}`
@@ -525,17 +696,16 @@ async function main() {
 
   if (command === 'validate') {
     const result = await runValidate({ configPath });
-    console.log(`OK: validated (${result.assetCount} asset(s))`);
+    emitPayload('validate', { configPath, ...result }, `OK: validated (${result.assetCount} asset(s))`);
     return;
   }
 
   if (command === 'lock') {
     const result = await runLock({ configPath, outPath: values.out, check: values.check });
-    if (result.checked) {
-      console.log(`OK: lockfile up to date (${result.assetCount} asset(s)) -> ${result.outPath}`);
-    } else {
-      console.log(`OK: wrote lockfile (${result.assetCount} asset(s)) -> ${result.outPath}`);
-    }
+    const text = result.checked
+      ? `OK: lockfile up to date (${result.assetCount} asset(s)) -> ${result.outPath}`
+      : `OK: wrote lockfile (${result.assetCount} asset(s)) -> ${result.outPath}`;
+    emitPayload('lock', { configPath, ...result }, text);
     return;
   }
 
@@ -549,15 +719,15 @@ async function main() {
     if ((values.zip || format === 'zip') && outDir && outDir.toLowerCase().endsWith('.zip') && !zipOut) {
       zipOut = outDir;
       outDir = undefined;
-      console.warn(
+      const warning =
         `DEPRECATED: shipscreens build --out <file.zip> will change behavior in a future release.\n` +
-          `Use: shipscreens build --zip-out <file.zip>\n` +
-          `Note: --out now specifies an output directory root.`
-      );
+        `Use: shipscreens build --zip-out <file.zip>\n` +
+        `Note: --out now specifies an output directory root.\n`;
+      process.stderr.write(warning);
     }
 
     const requirePro = Boolean(values['require-pro']);
-    const { token: cliAuthToken } = await resolveCliAuth({ requirePro });
+    const { token: cliAuthToken, profile: cliAuthProfile } = await resolveCliAuth({ requirePro });
     const result = await runBuild({
       configPath,
       format,
@@ -570,6 +740,7 @@ async function main() {
       framefileStrategy: values['framefile-strategy'],
       clean: values.clean,
       cliAuthToken,
+      cliAuthProfile,
     });
     const summary = [
       `OK: built`,
@@ -579,7 +750,11 @@ async function main() {
       `${result.androidDevices} Android device(s) -> ${result.androidDir}`,
     ];
     if (result.zipPath) summary.push(`zip -> ${result.zipPath}`);
-    console.log(summary.join(', '));
+    emitPayload(
+      'build',
+      { configPath, format, mode: values.mode || 'rendered', outDir: outDir || null, zipOut: zipOut || null, ...result },
+      summary.join(', ')
+    );
     return;
   }
 
@@ -603,7 +778,17 @@ async function main() {
       `${result.androidDevices} Android device(s) -> ${result.androidDir}`,
     ];
     if (result.zipPath) summary.push(`zip -> ${result.zipPath}`);
-    console.log(summary.join(', '));
+    emitPayload(
+      'verify',
+      {
+        configPath,
+        outDir: rawOut || null,
+        zipOut: rawZipOut || null,
+        profile: values.profile || null,
+        ...result,
+      },
+      summary.join(', ')
+    );
     return;
   }
 
@@ -617,7 +802,11 @@ async function main() {
       profile: values.profile,
     });
     const details = result.iosVariant ? `, ios=${result.iosVariant}` : '';
-    console.log(`OK: staged (${result.profile}${details}), ${result.files} file(s) -> ${result.out}`);
+    emitPayload(
+      'stage',
+      { configPath, ...result },
+      `OK: staged (${result.profile}${details}), ${result.files} file(s) -> ${result.out}`
+    );
     return;
   }
 
@@ -658,7 +847,13 @@ async function main() {
   throw new Error(`Unknown command: ${command}`);
 }
 
+const wantsJson = process.argv.includes('--json');
 main().catch((err) => {
-  console.error(err?.message || String(err));
+  const message = err?.message || String(err);
+  if (wantsJson) {
+    process.stderr.write(JSON.stringify({ version: 1, ok: false, error: message }, null, 2) + '\n');
+  } else {
+    console.error(message);
+  }
   process.exitCode = 1;
 });

package/cli/commands/build.js

@@ -69,6 +69,7 @@ export async function runBuild({
   framefileStrategy,
   clean = false,
   cliAuthToken,
+  cliAuthProfile,
 } = {}) {
   await runValidate({ configPath });
   const { config, configDir } = await loadShipScreensConfig(configPath);
@@ -223,13 +224,12 @@ export async function runBuild({
 
   let chromium;
   try {
-    ({ chromium } = await import('playwright-core'));
+    ({ chromium } = await import('playwright-chromium'));
   } catch (err) {
     await closeServer();
     throw new Error(
-      `Playwright is required for rendered builds.\n` +
-        `Install Chromium once (or use --mode raw):\n` +
-        `  npx playwright install --with-deps chromium\n` +
+      `Playwright (Chromium) is required for shipscreens build.\n` +
+        `Install dependencies and try again.\n` +
         `${err?.message || String(err)}`
     );
   }
@@ -240,21 +240,7 @@ export async function runBuild({
 
   let browser = null;
   try {
-    try {
-      browser = await chromium.launch({ headless: true });
-    } catch (err) {
-      const msg = err?.message || String(err);
-      if (msg.includes("Executable doesn't exist") || msg.includes('playwright install')) {
-        throw new Error(
-          `Playwright Chromium is not installed on this machine.\n` +
-            `Run:\n` +
-            `  npx playwright install --with-deps chromium\n` +
-            `Then rerun shipscreens build.\n` +
-            msg
-        );
-      }
-      throw err;
-    }
+    browser = await chromium.launch({ headless: true });
     const context = await browser.newContext({
       acceptDownloads: true,
       viewport: { width: 1400, height: 900 },
@@ -267,17 +253,31 @@ export async function runBuild({
     };
 
     await context.addInitScript(
-      (payload, exportStructureMode, authToken) => {
+      ({ payload, exportStructureMode, authToken, authProfile }) => {
         window.__SHIPSCREENS_CLI = true;
-        if (authToken) {
-          window.__SHIPSCREENS_CLI_AUTH = { token: authToken };
+        // Prefer an injected in-memory payload so large screenshots won't fail on localStorage quota.
+        window.__SHIPSCREENS_CLI_PROJECT = payload;
+        window.__SHIPSCREENS_CLI_EXPORT_STRUCTURE_MODE = exportStructureMode;
+        if (authToken || authProfile) {
+          window.__SHIPSCREENS_CLI_AUTH = { token: authToken || null, profile: authProfile || null };
+        }
+        try {
+          localStorage.setItem('shipscreens_project', JSON.stringify(payload));
+        } catch (err) {
+          void err;
+        }
+        try {
+          localStorage.setItem('shipScreensExportStructureMode', exportStructureMode);
+        } catch (err) {
+          void err;
         }
-        localStorage.setItem('shipscreens_project', JSON.stringify(payload));
-        localStorage.setItem('shipScreensExportStructureMode', exportStructureMode);
       },
-      projectPayload,
-      outputMode,
-      cliAuthToken || null
+      {
+        payload: projectPayload,
+        exportStructureMode: outputMode,
+        authToken: cliAuthToken || null,
+        authProfile: cliAuthProfile || null,
+      }
     );
 
     const page = await context.newPage();

package/cli/schema.js

@@ -1,5 +1,6 @@
 import { z } from 'zod';
 import { isDisallowedPath, normalizeRepoRelativePath } from './pathUtils.js';
+import { DEVICE_LIBRARY } from '../src/constants/devices.js';
 
 export const SHIPSCREENS_CONFIG_SCHEMA_VERSION = 1;
 
@@ -19,6 +20,20 @@ const DeviceId = z
   .max(64)
   .regex(/^[a-z0-9_-]+$/i, 'Expected a device id like iphone69 or pixel8');
 
+const inferDevicePlatform = (deviceId) => {
+  const def = DEVICE_LIBRARY[deviceId];
+  const group = String(def?.group || '').toLowerCase();
+  if (!group) return null;
+  return group.includes('android') ? 'android' : 'ios';
+};
+
+const inferAndroidBucket = (deviceId) => {
+  const def = DEVICE_LIBRARY[deviceId];
+  const group = String(def?.group || '').toLowerCase();
+  if (group.includes('tablet')) return 'tenInchScreenshots';
+  return 'phoneScreenshots';
+};
+
 const LayoutMode = z.enum(['top', 'bottom', 'split']);
 const BadgeType = z.enum(['none', 'stars', 'award', 'download', 'googleplay']);
 const FrameColor = z.enum(['black', 'silver', 'titanium', 'blue', 'gold']);
@@ -143,6 +158,23 @@ const Slide = z
   })
   .strict();
 
+const DeviceConfig = z
+  .object({
+    id: DeviceId,
+    platform: z.enum(['ios', 'android']).optional(),
+    androidBucket: z
+      .enum(['phoneScreenshots', 'sevenInchScreenshots', 'tenInchScreenshots', 'tvScreenshots', 'wearScreenshots'])
+      .optional(),
+    label: z.string().min(1).optional(),
+  })
+  .strict()
+  .transform((device) => {
+    const platform = device.platform || inferDevicePlatform(device.id) || undefined;
+    const androidBucket =
+      platform === 'android' ? device.androidBucket || inferAndroidBucket(device.id) : device.androidBucket;
+    return { ...device, platform, androidBucket };
+  });
+
 const OutputCompat = z
   .object({
     directory: z.string().min(1).optional(),
@@ -186,26 +218,7 @@ export const ShipScreensConfigSchema = z
           .strict()
       )
       .min(1),
-    devices: z
-      .array(
-        z
-          .object({
-            id: DeviceId,
-            platform: z.enum(['ios', 'android']).optional(),
-            androidBucket: z
-              .enum([
-                'phoneScreenshots',
-                'sevenInchScreenshots',
-                'tenInchScreenshots',
-                'tvScreenshots',
-                'wearScreenshots',
-              ])
-              .optional(),
-            label: z.string().min(1).optional(),
-          })
-          .strict()
-      )
-      .min(1),
+    devices: z.array(DeviceConfig).min(1),
     assets: Assets,
     style: Style,
     slides: z.array(Slide).min(1),
@@ -247,6 +260,32 @@ export const ShipScreensConfigSchema = z
     if (config.assets?.badges) unique(config.assets.badges, 'assets.badges', (b) => b.id);
     unique(config.slides, 'slides', (s) => String(s.id));
 
+    const androidBuckets = new Map();
+    config.devices.forEach((device, deviceIdx) => {
+      if (device.platform === 'android') {
+        const bucket = device.androidBucket || 'phoneScreenshots';
+        const existingIdx = androidBuckets.get(bucket);
+        if (existingIdx != null) {
+          ctx.addIssue({
+            code: z.ZodIssueCode.custom,
+            message:
+              `Multiple Android devices map to the same Google Play bucket (${bucket}): ` +
+              `${config.devices[existingIdx].id}, ${device.id}\n` +
+              `Each bucket represents a single screenshot set on Google Play. Set devices[].androidBucket explicitly.`,
+            path: ['devices', deviceIdx, 'androidBucket'],
+          });
+        } else {
+          androidBuckets.set(bucket, deviceIdx);
+        }
+      } else if (device.androidBucket) {
+        ctx.addIssue({
+          code: z.ZodIssueCode.custom,
+          message: `androidBucket is only valid when platform=android`,
+          path: ['devices', deviceIdx, 'androidBucket'],
+        });
+      }
+    });
+
     const localeIds = new Set(config.locales.map((l) => l.id));
 
     config.slides.forEach((slide, slideIdx) => {