Skip to content

Add CLI support for order_attribution_config extension type#7322

Merged
Catherine-Fu merged 2 commits intomainfrom
add-order-attribution-config-spec
Apr 16, 2026
Merged

Add CLI support for order_attribution_config extension type#7322
Catherine-Fu merged 2 commits intomainfrom
add-order-attribution-config-spec

Conversation

@Catherine-Fu
Copy link
Copy Markdown
Contributor

@Catherine-Fu Catherine-Fu commented Apr 15, 2026

Summary

  • Add order_attribution_config specification in packages/app/src/cli/models/extensions/specifications/order_attribution_config.ts
  • Register it in load-specifications.ts
  • Uses createContractBasedModuleSpecification with copy_files build config for SVG icons (same pattern as channel_config)

Changes

  • New file: order_attribution_config.ts — specification with uidStrategy: 'single', experience: 'extension', copies icons/**/*.svg on deploy
  • Modified: load-specifications.ts — import and register the new spec in moduleSpecs

Test plan

  • shopify app generate extension lists order_attribution_config as an option
  • Scaffolded extension has correct TOML structure
  • shopify app deploy succeeds with the new extension type

Closes https://github.com/shop/issues-multi-channel-management/issues/3127

🤖 Generated with Claude Code

Register the order_attribution_config extension type in the CLI so
developers can generate and deploy this extension type. Follows the
same pattern as channel_config with copy_files build config for SVG
icons in the icons/ subdirectory.

Closes shop/issues-multi-channel-management#3127

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@Catherine-Fu Catherine-Fu marked this pull request as ready for review April 15, 2026 22:48
@Catherine-Fu Catherine-Fu requested a review from a team as a code owner April 15, 2026 22:48
Copilot AI review requested due to automatic review settings April 15, 2026 22:48
Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new local extension specification for order_attribution_config and registers it so the CLI can load it like other contract-based module specs.

Changes:

  • Added order_attribution_config contract-based module specification with uidStrategy: 'single' and experience: 'extension'.
  • Configured deploy/build to include icons/**/*.svg via copy_files + include_assets client step.
  • Registered the new specification in load-specifications.ts.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
packages/app/src/cli/models/extensions/specifications/order_attribution_config.ts Introduces the new contract-based module spec and asset-copy configuration for SVG icons.
packages/app/src/cli/models/extensions/load-specifications.ts Imports and adds the new spec to the locally loaded moduleSpecs list.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Catherine-Fu Catherine-Fu changed the title Add order_attribution_config extension specification Add CLI support for order_attribution_config extension type Apr 16, 2026
Comment thread packages/app/src/cli/models/extensions/specifications/order_attribution_config.ts Outdated
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@Catherine-Fu Catherine-Fu added this pull request to the merge queue Apr 16, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 16, 2026
@Catherine-Fu Catherine-Fu added this pull request to the merge queue Apr 16, 2026
Merged via the queue into main with commit f0d496f Apr 16, 2026
25 checks passed
@Catherine-Fu Catherine-Fu deleted the add-order-attribution-config-spec branch April 16, 2026 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants