Skip to content

fix(core): prevent alias from overwriting root deps in pnpm parser#34064

Merged
leosvelperez merged 1 commit intomasterfrom
nxc-3655
Jan 13, 2026
Merged

fix(core): prevent alias from overwriting root deps in pnpm parser#34064
leosvelperez merged 1 commit intomasterfrom
nxc-3655

Conversation

@leosvelperez
Copy link
Member

Current Behavior

When using pnpm aliases, the project graph may miss the actual dependency node (the aliased package overwrites it), resulting in incomplete or confusing dependency graphs.

Expected Behavior

The project graph should include both the alias and the actual dependency node, matching the behavior of other package managers.

@leosvelperez leosvelperez self-assigned this Jan 9, 2026
@leosvelperez leosvelperez requested review from a team as code owners January 9, 2026 14:26
@vercel
Copy link

vercel bot commented Jan 9, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
nx-dev Ready Ready Preview Jan 9, 2026 2:41pm

@netlify
Copy link

netlify bot commented Jan 9, 2026

Deploy Preview for nx-docs ready!

Name Link
🔨 Latest commit f82d37e
🔍 Latest deploy log https://app.netlify.com/projects/nx-docs/deploys/69610ffdd9ccbd00081a649e
😎 Deploy Preview https://deploy-preview-34064--nx-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@nx-cloud
Copy link
Contributor

nx-cloud bot commented Jan 9, 2026

View your CI Pipeline Execution ↗ for commit f82d37e

Command Status Duration Result
nx affected --targets=lint,test,test-kt,build,e... ✅ Succeeded 5m 47s View ↗
nx run-many -t check-imports check-lock-files c... ✅ Succeeded 2m 49s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 12s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 3s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2026-01-09 17:11:55 UTC

@leosvelperez leosvelperez merged commit 0bb9d67 into master Jan 13, 2026
21 of 22 checks passed
@leosvelperez leosvelperez deleted the nxc-3655 branch January 13, 2026 16:25
FrozenPandaz pushed a commit that referenced this pull request Jan 14, 2026
…34064)

## Current Behavior

When using pnpm aliases, the project graph may miss the actual
dependency node (the aliased package overwrites it), resulting in
incomplete or confusing dependency graphs.

## Expected Behavior

The project graph should include both the alias and the actual
dependency node, matching the behavior of other package managers.

(cherry picked from commit 0bb9d67)
@github-actions
Copy link
Contributor

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 19, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants