feat: fetch experience field from API instead of hardcoding it

isaacroldan · Claude Code · isaacroldan · commit e123eb9bb928 · 2026-03-26T11:53:32.000+01:00
Co-authored-by: Claude Code &lt;claude-code@anthropic.com&gt;
diff --git a/packages/app/src/cli/api/graphql/app-management/generated/active-app-release-from-api-key.ts b/packages/app/src/cli/api/graphql/app-management/generated/active-app-release-from-api-key.ts
@@ -24,7 +24,13 @@ export type ActiveAppReleaseFromApiKeyQuery = {
           handle: string
           config: JsonMapType
           target?: string | null
-          specification: {identifier: string; externalIdentifier: string; name: string; managementExperience: string}
+          specification: {
+            identifier: string
+            externalIdentifier: string
+            name: string
+            experience: string
+            managementExperience: string
+          }
         }[]
       }
     }
@@ -91,6 +97,7 @@ export const ActiveAppReleaseFromApiKey = {
                 {kind: 'Field', name: {kind: 'Name', value: 'identifier'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'externalIdentifier'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'name'}},
+                {kind: 'Field', name: {kind: 'Name', value: 'experience'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'managementExperience'}},
               ],
             },
diff --git a/packages/app/src/cli/api/graphql/app-management/generated/active-app-release.ts b/packages/app/src/cli/api/graphql/app-management/generated/active-app-release.ts
@@ -24,7 +24,13 @@ export type ActiveAppReleaseQuery = {
           handle: string
           config: JsonMapType
           target?: string | null
-          specification: {identifier: string; externalIdentifier: string; name: string; managementExperience: string}
+          specification: {
+            identifier: string
+            externalIdentifier: string
+            name: string
+            experience: string
+            managementExperience: string
+          }
         }[]
       }
     }
@@ -46,7 +52,13 @@ export type AppVersionInfoFragment = {
         handle: string
         config: JsonMapType
         target?: string | null
-        specification: {identifier: string; externalIdentifier: string; name: string; managementExperience: string}
+        specification: {
+          identifier: string
+          externalIdentifier: string
+          name: string
+          experience: string
+          managementExperience: string
+        }
       }[]
     }
   }
@@ -58,7 +70,13 @@ export type ReleasedAppModuleFragment = {
   handle: string
   config: JsonMapType
   target?: string | null
-  specification: {identifier: string; externalIdentifier: string; name: string; managementExperience: string}
+  specification: {
+    identifier: string
+    externalIdentifier: string
+    name: string
+    experience: string
+    managementExperience: string
+  }
 }
 
 export const ReleasedAppModuleFragmentDoc = {
@@ -85,6 +103,7 @@ export const ReleasedAppModuleFragmentDoc = {
                 {kind: 'Field', name: {kind: 'Name', value: 'identifier'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'externalIdentifier'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'name'}},
+                {kind: 'Field', name: {kind: 'Name', value: 'experience'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'managementExperience'}},
               ],
             },
@@ -186,6 +205,7 @@ export const AppVersionInfoFragmentDoc = {
                 {kind: 'Field', name: {kind: 'Name', value: 'identifier'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'externalIdentifier'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'name'}},
+                {kind: 'Field', name: {kind: 'Name', value: 'experience'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'managementExperience'}},
               ],
             },
@@ -254,6 +274,7 @@ export const ActiveAppRelease = {
                 {kind: 'Field', name: {kind: 'Name', value: 'identifier'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'externalIdentifier'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'name'}},
+                {kind: 'Field', name: {kind: 'Name', value: 'experience'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'managementExperience'}},
               ],
             },
diff --git a/packages/app/src/cli/api/graphql/app-management/generated/app-version-by-id.ts b/packages/app/src/cli/api/graphql/app-management/generated/app-version-by-id.ts
@@ -18,7 +18,13 @@ export type AppVersionByIdQuery = {
       handle: string
       config: JsonMapType
       target?: string | null
-      specification: {identifier: string; externalIdentifier: string; name: string; managementExperience: string}
+      specification: {
+        identifier: string
+        externalIdentifier: string
+        name: string
+        experience: string
+        managementExperience: string
+      }
     }[]
   }
 }
@@ -32,7 +38,13 @@ export type VersionInfoFragment = {
     handle: string
     config: JsonMapType
     target?: string | null
-    specification: {identifier: string; externalIdentifier: string; name: string; managementExperience: string}
+    specification: {
+      identifier: string
+      externalIdentifier: string
+      name: string
+      experience: string
+      managementExperience: string
+    }
   }[]
 }
 
@@ -90,6 +102,7 @@ export const VersionInfoFragmentDoc = {
                 {kind: 'Field', name: {kind: 'Name', value: 'identifier'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'externalIdentifier'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'name'}},
+                {kind: 'Field', name: {kind: 'Name', value: 'experience'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'managementExperience'}},
               ],
             },
@@ -158,6 +171,7 @@ export const AppVersionById = {
                 {kind: 'Field', name: {kind: 'Name', value: 'identifier'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'externalIdentifier'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'name'}},
+                {kind: 'Field', name: {kind: 'Name', value: 'experience'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'managementExperience'}},
               ],
             },
diff --git a/packages/app/src/cli/api/graphql/app-management/generated/app-version-by-tag.ts b/packages/app/src/cli/api/graphql/app-management/generated/app-version-by-tag.ts
@@ -18,7 +18,13 @@ export type AppVersionByTagQuery = {
       handle: string
       config: JsonMapType
       target?: string | null
-      specification: {identifier: string; externalIdentifier: string; name: string; managementExperience: string}
+      specification: {
+        identifier: string
+        externalIdentifier: string
+        name: string
+        experience: string
+        managementExperience: string
+      }
     }[]
   }
 }
@@ -82,6 +88,7 @@ export const AppVersionByTag = {
                 {kind: 'Field', name: {kind: 'Name', value: 'identifier'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'externalIdentifier'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'name'}},
+                {kind: 'Field', name: {kind: 'Name', value: 'experience'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'managementExperience'}},
               ],
             },
diff --git a/packages/app/src/cli/api/graphql/app-management/generated/create-app-version.ts b/packages/app/src/cli/api/graphql/app-management/generated/create-app-version.ts
@@ -20,7 +20,13 @@ export type CreateAppVersionMutation = {
         handle: string
         config: JsonMapType
         target?: string | null
-        specification: {identifier: string; externalIdentifier: string; name: string; managementExperience: string}
+        specification: {
+          identifier: string
+          externalIdentifier: string
+          name: string
+          experience: string
+          managementExperience: string
+        }
       }[]
       metadata: {versionTag?: string | null; message?: string | null}
     } | null
@@ -161,6 +167,7 @@ export const CreateAppVersion = {
                 {kind: 'Field', name: {kind: 'Name', value: 'identifier'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'externalIdentifier'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'name'}},
+                {kind: 'Field', name: {kind: 'Name', value: 'experience'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'managementExperience'}},
               ],
             },
diff --git a/packages/app/src/cli/api/graphql/app-management/generated/specifications.ts b/packages/app/src/cli/api/graphql/app-management/generated/specifications.ts
@@ -12,6 +12,7 @@ export type FetchSpecificationsQuery = {
     name: string
     identifier: string
     externalIdentifier: string
+    experience: string
     features: string[]
     uidStrategy:
       | {__typename: 'UidStrategiesClientProvided'; appModuleLimit: number; isClientProvided: boolean}
@@ -54,6 +55,7 @@ export const FetchSpecifications = {
                 {kind: 'Field', name: {kind: 'Name', value: 'name'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'identifier'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'externalIdentifier'}},
+                {kind: 'Field', name: {kind: 'Name', value: 'experience'}},
                 {kind: 'Field', name: {kind: 'Name', value: 'features'}},
                 {
                   kind: 'Field',
diff --git a/packages/app/src/cli/api/graphql/app-management/queries/active-app-release.graphql b/packages/app/src/cli/api/graphql/app-management/queries/active-app-release.graphql
@@ -37,6 +37,7 @@ fragment ReleasedAppModule on AppModule {
     identifier
     externalIdentifier
     name
+    experience
     managementExperience
   }
 }
diff --git a/packages/app/src/cli/api/graphql/app-management/queries/specifications.graphql b/packages/app/src/cli/api/graphql/app-management/queries/specifications.graphql
@@ -3,6 +3,7 @@ query fetchSpecifications($organizationId: ID!) {
     name
     identifier
     externalIdentifier
+    experience
     features
     uidStrategy {
       __typename
diff --git a/packages/app/src/cli/models/extensions/extension-instance.ts b/packages/app/src/cli/models/extensions/extension-instance.ts
@@ -2,15 +2,6 @@ import {BaseConfigType, MAX_EXTENSION_HANDLE_LENGTH, MAX_UID_LENGTH} from './sch
 import {FunctionConfigType} from './specifications/function.js'
 import {ExtensionFeature, ExtensionSpecification} from './specification.js'
 import {SingleWebhookSubscriptionType} from './specifications/app_config_webhook_schemas/webhooks_schema.js'
-import {AppHomeSpecIdentifier} from './specifications/app_config_app_home.js'
-import {AppAccessSpecIdentifier} from './specifications/app_config_app_access.js'
-import {AppProxySpecIdentifier} from './specifications/app_config_app_proxy.js'
-import {BrandingSpecIdentifier} from './specifications/app_config_branding.js'
-import {PosSpecIdentifier} from './specifications/app_config_point_of_sale.js'
-import {PrivacyComplianceWebhooksSpecIdentifier} from './specifications/app_config_privacy_compliance_webhooks.js'
-import {WebhooksSpecIdentifier} from './specifications/app_config_webhook.js'
-import {WebhookSubscriptionSpecIdentifier} from './specifications/app_config_webhook_subscription.js'
-import {EventsSpecIdentifier} from './specifications/app_config_events.js'
 import {
   ExtensionBuildOptions,
   buildFunctionExtension,
@@ -34,23 +25,6 @@ import {outputDebug} from '@shopify/cli-kit/node/output'
 import {extractJSImports, extractImportPathsRecursively} from '@shopify/cli-kit/node/import-extractor'
 import {uniq} from '@shopify/cli-kit/common/array'
 
-// DEPRECATED. We should get the experience from the specification instead of hardcoding it based on the identifier.
-// This is a temporary solution to avoid breaking changes while we update the API and the specifications query.
-export const CONFIG_EXTENSION_IDS: string[] = [
-  AppAccessSpecIdentifier,
-  AppHomeSpecIdentifier,
-  AppProxySpecIdentifier,
-  BrandingSpecIdentifier,
-  PosSpecIdentifier,
-  PrivacyComplianceWebhooksSpecIdentifier,
-  WebhookSubscriptionSpecIdentifier,
-  WebhooksSpecIdentifier,
-  EventsSpecIdentifier,
-  // Hardcoded identifiers that don't exist locally.
-  'data',
-  'admin',
-]
-
 /**
  * Class that represents an instance of a local extension
  * Before creating this class we've validated that:
diff --git a/packages/app/src/cli/services/generate/fetch-extension-specifications.ts b/packages/app/src/cli/services/generate/fetch-extension-specifications.ts
@@ -43,12 +43,16 @@ export async function fetchSpecifications({
       // has been using so far. This is a workaround to keep the CLI working until the API is updated.
       if (spec.identifier === 'theme_app_extension') spec.identifier = 'theme'
       if (spec.identifier === 'subscription_management') spec.identifier = 'product_subscription'
+
       newSpec.registrationLimit = spec.options.registrationLimit
       newSpec.surface = spec.features?.argo?.surface
 
       // Hardcoded value for the post purchase extension because the value is wrong in the API
       if (spec.identifier === 'checkout_post_purchase') newSpec.surface = 'post_purchase'
 
+      // Hardcoded value for the webhook_subscription extension because the value is wrong in the API
+      if (spec.identifier === 'webhook_subscription') spec.experience = 'configuration'
+
       return newSpec
     })
 
diff --git a/packages/app/src/cli/utilities/developer-platform-client/app-management-client.test.ts b/packages/app/src/cli/utilities/developer-platform-client/app-management-client.test.ts
@@ -95,6 +95,7 @@ function moduleFromExtension(extension: ExtensionInstance) {
       identifier: extension.specification.identifier,
       externalIdentifier: extension.specification.externalIdentifier,
       name: extension.specification.externalName,
+      experience: extension.specification.experience,
       managementExperience: 'cli',
     },
   }
@@ -1352,6 +1353,14 @@ describe('appExtensionRegistrations', () => {
       registrationId: BrandingSpecIdentifier,
       registrationTitle: 'Config Extension',
       type: 'config_extension',
+      specification: {
+        identifier: BrandingSpecIdentifier,
+        name: 'Config Extension',
+        experience: 'configuration' as const,
+        options: {
+          managementExperience: 'cli' as const,
+        },
+      },
     })
 
     const extensionModule = createMockAppModuleVersion({
@@ -1451,6 +1460,14 @@ describe('appExtensionRegistrations', () => {
         registrationId: id,
         registrationTitle: `Config Extension ${index}`,
         type: 'config_extension',
+        specification: {
+          identifier: id,
+          name: `Config Extension ${index}`,
+          experience: 'configuration' as const,
+          options: {
+            managementExperience: 'cli' as const,
+          },
+        },
       }),
     )
 
@@ -1515,6 +1532,14 @@ describe('appExtensionRegistrations', () => {
           registrationId: id,
           registrationTitle: `Config ${id}`,
           type: 'config',
+          specification: {
+            identifier: id,
+            name: `Config ${id}`,
+            experience: 'configuration' as const,
+            options: {
+              managementExperience: 'cli' as const,
+            },
+          },
         }),
       ),
       // Regular CLI-managed extensions
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

Original file line number	Diff line number	Diff line change
`@@ -37,6 +37,7 @@ fragment ReleasedAppModule on AppModule {`
`37`	`37`	`identifier`
`38`	`38`	`externalIdentifier`
`39`	`39`	`name`
	`40`	`+ experience`
`40`	`41`	`managementExperience`
`41`	`42`	`}`
`42`	`43`	`}`