From f441e3c4db45eb92c485600b9cc37650bb1dee16 Mon Sep 17 00:00:00 2001 From: gonzaloriestra <14979109+gonzaloriestra@users.noreply.github.com> Date: Fri, 12 Jun 2026 00:10:24 +0000 Subject: [PATCH] [Refactor] Use uniq and flatMap in app-management-client.ts Refactor the `allowedTemplates` function in `packages/app/src/cli/utilities/developer-platform-client/app-management-client.ts` to use `uniq` from `@shopify/cli-kit/common/array` and `.flatMap()` instead of manual `Set` deduplication. --- .../developer-platform-client/app-management-client.ts | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/packages/app/src/cli/utilities/developer-platform-client/app-management-client.ts b/packages/app/src/cli/utilities/developer-platform-client/app-management-client.ts index 94badf04945..ebd4227cd6f 100644 --- a/packages/app/src/cli/utilities/developer-platform-client/app-management-client.ts +++ b/packages/app/src/cli/utilities/developer-platform-client/app-management-client.ts @@ -160,6 +160,7 @@ import { } from '@shopify/cli-kit/node/api/business-platform' import {CLI_KIT_VERSION} from '@shopify/cli-kit/common/version' import {encodeGid, numericIdFromEncodedGid, numericIdFromGid} from '@shopify/cli-kit/common/gid' +import {uniq} from '@shopify/cli-kit/common/array' import {versionSatisfies} from '@shopify/cli-kit/node/node-package-manager' import {outputDebug} from '@shopify/cli-kit/node/output' import {developerDashboardFqdn, normalizeStoreFqdn} from '@shopify/cli-kit/node/context/fqdn' @@ -1343,8 +1344,8 @@ export async function allowedTemplates( version: string = CLI_KIT_VERSION, ): Promise { // Extract both types of flags from templates - const allBetaFlags = Array.from(new Set(templates.map((ext) => ext.organizationBetaFlags ?? []).flat())) - const allExpFlags = Array.from(new Set(templates.map((ext) => ext.organizationExpFlags ?? []).flat())) + const allBetaFlags = uniq(templates.flatMap((ext) => ext.organizationBetaFlags ?? [])) + const allExpFlags = uniq(templates.flatMap((ext) => ext.organizationExpFlags ?? [])) // Fetch both flag types in parallel const [enabledBetaFlags, enabledExpFlags] = await Promise.all([