Skip to content

Commit 2887d33

Browse files
ryancbahanclaude
andcommitted
Migrate webhook_subscription spec to direct transform parameters
Replace transformConfig wrapper with deployConfig, transformLocalToRemote, and transformRemoteToLocal set directly. No behavior change — forward transform (URI resolution) and reverse transform preserved as-is. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent c0af9b7 commit 2887d33

File tree

1 file changed

+6
-10
lines changed

1 file changed

+6
-10
lines changed

packages/app/src/cli/models/extensions/specifications/app_config_webhook_subscription.ts

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import {WebhookSubscriptionUriValidation, removeTrailingSlash} from './validation/common.js'
22
import {prependApplicationUrl} from './validation/url_prepender.js'
33
import {WebhookSubscription} from './types/app_config_webhook.js'
4-
import {CustomTransformationConfig, createConfigExtensionSpecification} from '../specification.js'
4+
import {createConfigExtensionSpecification} from '../specification.js'
55
import {CurrentAppConfiguration} from '../../app/app.js'
66
import {zod} from '@shopify/cli-kit/node/schema'
77

@@ -54,8 +54,10 @@ function transformToWebhookSubscriptionConfig(content: object) {
5454
}
5555
}
5656

57-
const WebhookSubscriptionTransformConfig: CustomTransformationConfig = {
58-
forward: (content, appConfiguration) => {
57+
const appWebhookSubscriptionSpec = createConfigExtensionSpecification({
58+
identifier: WebhookSubscriptionSpecIdentifier,
59+
schema: SingleWebhookSubscriptionSchema,
60+
transformLocalToRemote: (content, appConfiguration) => {
5961
const webhookConfig = content as WebhookSubscription
6062
let appUrl: string | undefined
6163
if ('application_url' in appConfiguration) {
@@ -66,13 +68,7 @@ const WebhookSubscriptionTransformConfig: CustomTransformationConfig = {
6668
uri: prependApplicationUrl(webhookConfig.uri, appUrl),
6769
}
6870
},
69-
reverse: transformToWebhookSubscriptionConfig,
70-
}
71-
72-
const appWebhookSubscriptionSpec = createConfigExtensionSpecification({
73-
identifier: WebhookSubscriptionSpecIdentifier,
74-
schema: SingleWebhookSubscriptionSchema,
75-
transformConfig: WebhookSubscriptionTransformConfig,
71+
transformRemoteToLocal: transformToWebhookSubscriptionConfig,
7672
uidStrategy: 'dynamic',
7773
})
7874

0 commit comments

Comments
 (0)