Skip to content

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

Merged
craigmichaelmartin merged 1 commit into
mainfrom
craig/clientsteps-regression-tests
Apr 6, 2026
Merged

test: add regression tests for clientSteps passthrough in specification factories#7144
craigmichaelmartin merged 1 commit into
mainfrom
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

github-actions Bot commented Apr 1, 2026

Copy link
Copy Markdown
Contributor

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.
@craigmichaelmartin craigmichaelmartin added this pull request to the merge queue Apr 6, 2026
Merged via the queue into main with commit 3cc7aff Apr 6, 2026
66 of 68 checks passed
@craigmichaelmartin craigmichaelmartin deleted the craig/clientsteps-regression-tests branch April 6, 2026 09:20
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.

2 participants