Skip to content

fix(core): prioritize nx installation path in getNxRequirePaths#34158

Merged
FrozenPandaz merged 1 commit intomasterfrom
fix/nx-require-paths-order
Jan 20, 2026
Merged

fix(core): prioritize nx installation path in getNxRequirePaths#34158
FrozenPandaz merged 1 commit intomasterfrom
fix/nx-require-paths-order

Conversation

@FrozenPandaz
Copy link
Copy Markdown
Contributor

Current Behavior

getNxRequirePaths returns paths in the order [root, getNxInstallationPath(root)], which means the workspace root is checked first when resolving modules.

Expected Behavior

The nx installation path (.nx/installation) should be prioritized and checked first before falling back to the workspace root. This ensures that modules from the nx installation directory take precedence.

Related Issue(s)

N/A

@FrozenPandaz FrozenPandaz requested a review from a team as a code owner January 20, 2026 16:55
@FrozenPandaz FrozenPandaz requested a review from MaxKless January 20, 2026 16:55
@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 20, 2026

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

Project Deployment Review Updated (UTC)
nx-dev Ready Ready Preview Jan 20, 2026 5:02pm

Request Review

@netlify
Copy link
Copy Markdown

netlify Bot commented Jan 20, 2026

Deploy Preview for nx-docs ready!

Name Link
🔨 Latest commit 628c77b
🔍 Latest deploy log https://app.netlify.com/projects/nx-docs/deploys/696fb39e34fb000008a0b96c
😎 Deploy Preview https://deploy-preview-34158--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
Copy Markdown
Contributor

nx-cloud Bot commented Jan 20, 2026

View your CI Pipeline Execution ↗ for commit 628c77b

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

☁️ Nx Cloud last updated this comment at 2026-01-20 18:54:03 UTC

@FrozenPandaz FrozenPandaz enabled auto-merge (squash) January 20, 2026 18:30
@FrozenPandaz FrozenPandaz merged commit e586896 into master Jan 20, 2026
21 of 22 checks passed
@FrozenPandaz FrozenPandaz deleted the fix/nx-require-paths-order branch January 20, 2026 18:54
@github-actions
Copy link
Copy Markdown
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 26, 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.

2 participants