Skip to content

test: add regression tests for clientSteps passthrough in specification factories#7144

Open
craigmichaelmartin wants to merge 1 commit intomainfrom
craig/clientsteps-regression-tests
Open

test: add regression tests for clientSteps passthrough in specification factories#7144
craigmichaelmartin wants to merge 1 commit intomainfrom
craig/clientsteps-regression-tests

Conversation

@craigmichaelmartin
Copy link
Copy Markdown
Contributor

Ensures clientSteps is properly forwarded in createContractBasedModuleSpecification, createExtensionSpecification, and createConfigExtensionSpecification.

Prevents regression of the bug fixed in #7141 where clientSteps was accidentally dropped from createContractBasedModuleSpecification during a conflict resolution, breaking channel_config extension builds.

@craigmichaelmartin craigmichaelmartin requested a review from a team as a code owner April 1, 2026 16:52
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

We detected some changes at packages/*/src and there are no updates in the .changeset.
If the changes are user-facing, run pnpm changeset add to track your changes and include them in the next release CHANGELOG.

Caution

DO NOT create changesets for features which you do not wish to be included in the public changelog of the next CLI release.

…on factories

Ensures clientSteps is properly forwarded in createContractBasedModuleSpecification,
createExtensionSpecification, and createConfigExtensionSpecification.

Prevents regression of the bug fixed in #7141 where clientSteps was accidentally
dropped from createContractBasedModuleSpecification during a conflict resolution,
breaking channel_config extension builds.
@shopify-river shopify-river bot force-pushed the craig/clientsteps-regression-tests branch from de39c10 to fe650cf Compare April 1, 2026 20:47
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.

1 participant