From df16ab3dc3d9f2cd7f11911c40c3b7e7b4e701ea Mon Sep 17 00:00:00 2001 From: gonzaloriestra <14979109+gonzaloriestra@users.noreply.github.com> Date: Sat, 23 May 2026 00:20:35 +0000 Subject: [PATCH] [Refactor] Use .find() in ensureThemeExtensionDevContext Refactored `ensureThemeExtensionDevContext` in `packages/app/src/cli/services/context.ts` to replace the `.filter().[0]` pattern with the more idiomatic `.find()` method. This change improves code readability and leverages short-circuiting for a minor performance gain. --- packages/app/src/cli/services/context.ts | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/packages/app/src/cli/services/context.ts b/packages/app/src/cli/services/context.ts index 2381683d7c..ec7257bca9 100644 --- a/packages/app/src/cli/services/context.ts +++ b/packages/app/src/cli/services/context.ts @@ -118,12 +118,12 @@ export async function ensureThemeExtensionDevContext( apiKey, organizationId: '1', }) - const remoteRegistrations = remoteSpecifications.app.extensionRegistrations.filter((extension) => { + const remoteRegistration = remoteSpecifications.app.extensionRegistrations.find((extension) => { return extension.type === 'THEME_APP_EXTENSION' }) - if (remoteRegistrations[0]) { - return remoteRegistrations[0] + if (remoteRegistration) { + return remoteRegistration } const registration = await createExtension(apiKey, extension.graphQLType, extension.handle, developerPlatformClient)