Allow "one of" multi env flag requirements

Previously, the multi environment required flags
array was flat and any missing flag would cause
validation to fail. However, some commands provide
a few different flags that will meet the same req.
This commit updates validation logic to take in a
nested array to account for this.

Example:

`['name', ['theme', 'live', 'development']]`

To run with multiple environments, `name` flag and
one of `theme`, `live`, or `development` are required

Author: dejmedus

packages/app/src/cli/services/context.test.ts

@@ -575,6 +575,98 @@ describe('ensureDeployContext', () => {
     })
     unsetAppConfigValueSpy.mockRestore()
   })
+
+  test('sets didMigrateExtensionsToDevDash to true when app modules are missing registration UUIDs', async () => {
+    // Given
+    const app = testAppWithConfig({config: {client_id: APP2.apiKey}})
+    const identifiers = {
+      app: APP2.apiKey,
+      extensions: {},
+      extensionIds: {},
+      extensionsNonUuidManaged: {},
+    }
+    vi.mocked(ensureDeploymentIdsPresence).mockResolvedValue(identifiers)
+    vi.mocked(getAppConfigurationFileName).mockReturnValue('shopify.app.toml')
+
+    const activeAppVersion = {
+      appModuleVersions: [
+        {registrationId: 'id-1', registrationUuid: 'uuid-1', type: 'app_access', registrationTitle: 'module-1'},
+        {registrationId: '', registrationUuid: 'uuid-2', type: 'pos_ui_extension', registrationTitle: 'module-2'},
+        {
+          registrationId: 'id-3',
+          registrationUuid: 'uuid-3',
+          type: 'checkout_ui_extension',
+          registrationTitle: 'module-3',
+        },
+      ],
+    }
+
+    const developerPlatformClient = buildDeveloperPlatformClient({
+      supportsAtomicDeployments: true,
+      activeAppVersion: () => Promise.resolve(activeAppVersion),
+    })
+
+    // When
+    const result = await ensureDeployContext({
+      app,
+      remoteApp: APP2,
+      organization: ORG1,
+      reset: false,
+      force: false,
+      noRelease: false,
+      developerPlatformClient,
+      skipBuild: false,
+    })
+
+    // Then
+    expect(result.didMigrateExtensionsToDevDash).toBe(true)
+  })
+
+  test('sets didMigrateExtensionsToDevDash to false when all app modules have registration UUIDs', async () => {
+    // Given
+    const app = testAppWithConfig({config: {client_id: APP2.apiKey}})
+    const identifiers = {
+      app: APP2.apiKey,
+      extensions: {},
+      extensionIds: {},
+      extensionsNonUuidManaged: {},
+    }
+    vi.mocked(ensureDeploymentIdsPresence).mockResolvedValue(identifiers)
+    vi.mocked(getAppConfigurationFileName).mockReturnValue('shopify.app.toml')
+
+    const activeAppVersion = {
+      appModuleVersions: [
+        {registrationId: 'id-1', registrationUuid: 'uuid-1', type: 'app_access', registrationTitle: 'module-1'},
+        {registrationId: 'id-2', registrationUuid: 'uuid-2', type: 'pos_ui_extension', registrationTitle: 'module-2'},
+        {
+          registrationId: 'id-3',
+          registrationUuid: 'uuid-3',
+          type: 'checkout_ui_extension',
+          registrationTitle: 'module-3',
+        },
+      ],
+    }
+
+    const developerPlatformClient = buildDeveloperPlatformClient({
+      supportsAtomicDeployments: true,
+      activeAppVersion: () => Promise.resolve(activeAppVersion),
+    })
+
+    // When
+    const result = await ensureDeployContext({
+      app,
+      remoteApp: APP2,
+      organization: ORG1,
+      reset: false,
+      force: false,
+      noRelease: false,
+      developerPlatformClient,
+      skipBuild: false,
+    })
+
+    // Then
+    expect(result.didMigrateExtensionsToDevDash).toBe(false)
+  })
 })
 
 describe('ensureThemeExtensionDevContext', () => {

packages/app/src/cli/services/context.ts

@@ -168,7 +168,7 @@ export async function ensureDeployContext(options: DeployOptions): Promise<Ensur
   // if the current active app version is missing user_identifiers in some app module, then we are migrating to dev dash
   let didMigrateExtensionsToDevDash = false
   if (developerPlatformClient.supportsAtomicDeployments && activeAppVersion) {
-    const missingUids = activeAppVersion.appModuleVersions.some((version) => !version.registrationUuid)
+    const missingUids = activeAppVersion.appModuleVersions.some((version) => !version.registrationId)
     didMigrateExtensionsToDevDash = missingUids
   }
 

packages/theme/src/cli/utilities/theme-command.ts

@@ -73,7 +73,7 @@ export default abstract class ThemeCommand extends Command {
   >(_opts?: Input<TFlags, TGlobalFlags, TArgs>): Promise<void> {
     // Parse command flags using the current command class definitions
     const klass = this.constructor as unknown as Input<TFlags, TGlobalFlags, TArgs> & {
-      multiEnvironmentsFlags: string[] | null
+      multiEnvironmentsFlags: RequiredFlags
       flags: FlagOutput
     }
     const requiredFlags = klass.multiEnvironmentsFlags