Skip to content

fix(cdk): add @angular/forms to peer dependencies - #33666

Merged
crisbeto merged 1 commit into
angular:mainfrom
caleniuc:fix/cdk-forms-peer-dependency
Aug 27, 2026
Merged

fix(cdk): add @angular/forms to peer dependencies#33666
crisbeto merged 1 commit into
angular:mainfrom
caleniuc:fix/cdk-forms-peer-dependency

Conversation

@caleniuc

@caleniuc caleniuc commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Entry points such as @angular/cdk/stepper and @angular/cdk/listbox import runtime symbols from @angular/forms, but the package is not listed in the CDK peer dependencies.

This can cause module resolution failures with package managers that enforce dependency isolation, such as pnpm with global virtual store.

You can reproduce it from this repo: https://github.com/caleniuc/angular-cdk-forms-repro

Will throw this error:

image

@pullapprove
pullapprove Bot requested review from andrewseguin and ok7sai August 12, 2026 16:56
@google-cla

google-cla Bot commented Aug 12, 2026

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@caleniuc

caleniuc commented Aug 26, 2026

Copy link
Copy Markdown
Contributor Author

Hi, @andrewseguin @ok7sai! Fixed the cla check, can you please take a look at the PR? Thank you!

LE: added info on how to reproduce in the description.

@crisbeto crisbeto added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Aug 27, 2026
@crisbeto
crisbeto merged commit 9ed16f1 into angular:main Aug 27, 2026
29 checks passed
@crisbeto

Copy link
Copy Markdown
Member

This PR was merged into the repository. The changes were merged into the following branches:

crisbeto pushed a commit that referenced this pull request Aug 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants