fix(devkit): restore peer dep range to 2 majors

[AgentEnder]

Current Behavior

Nx release now updates peer dependency versions, which is great... but it broke the peer dep range in the published version of @nx/devkit.

Expected Behavior

@nx/devkit maintains a version range of +/- 1 major version of nx. We will fix release soon.

Related Issue(s)

Fixes #31801 (already fixed, but comments on this highlighted the problem)

[vercel]

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

Project	Deployment	Preview	Updated (UTC)
nx-dev	Ready	Preview	Aug 19, 2025 9:45pm

[nx-cloud]

View your CI Pipeline Execution ↗ for commit a21b3ab

Command	Status	Duration	Result
nx affected --targets=lint,test,build,e2e,e2e-c...	✅ Succeeded	15m 13s	View ↗
nx run-many -t check-imports check-commit check...	✅ Succeeded	2m 1s	View ↗
nx-cloud record -- nx-cloud conformance:check	✅ Succeeded	2s	View ↗
nx-cloud record -- nx format:check	✅ Succeeded	6s	View ↗
nx-cloud record -- nx sync:check	✅ Succeeded	5s	View ↗
nx documentation	✅ Succeeded	1m 31s	View ↗

---

☁️ Nx Cloud last updated this comment at 2025-08-20 02:29:26 UTC

[FrozenPandaz]

Maybe we can remove quick and dirty from the function name lol

[AgentEnder]

Maybe we can remove quick and dirty from the function name lol

Nah, its encouragement to get a real fix in

[Coly010]

Maybe we can remove quick and dirty from the function name lol

Nah, its encouragement to get a real fix in

We could prefix it with hack and add a todo for me

[AgentEnder]

@claude change the new function name from quickAndDirty.... to hack... and add a todo for Colum to come back and remove it after peer dep handling is fixed

[claude]

Claude finished @AgentEnder's task —— View job

---

Task: Rename function and add TODO ✅

Todo List:

• Read scripts/nx-release.ts to locate the function
• Rename function from quickAndDirty... to hack...
• Add TODO comment for Colum
• Commit and push changes

Changes Made:

• Renamed quickAndDirtyFixForDevkitPeerDependencies to hackFixForDevkitPeerDependencies
• Added TODO comment: // TODO(Coly010): Remove this function once peer dependency handling is properly fixed in nx release
• Updated function call on line 203

The function name now reflects that it's a temporary hack, and there's a clear TODO for Colum to remove it once the proper peer dependency handling is implemented.

---

[graphite-app]

Consider adding a null check to prevent potential runtime errors. If devkitPackageJson.peerDependencies or devkitPackageJson.peerDependencies['nx'] is undefined, the current code will throw a TypeError.

Suggested change:

if (devkitPackageJson.peerDependencies?.nx && !devkitPackageJson.peerDependencies.nx.includes('<')) {

This ensures the code gracefully handles cases where the peer dependency structure might not exist as expected.

Suggested change

	if (!beforeVersion.includes('<')) {
	if (devkitPackageJson.peerDependencies?.nx && !devkitPackageJson.peerDependencies.nx.includes('<')) {

Spotted by Diamond



Is this helpful? React 👍 or 👎 to let us know.

[github-actions]

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.