Skip to content

fix(core): remove graph creation from postinstall hook #32027

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 27, 2025

Conversation

AgentEnder
Copy link
Member

Current Behavior

The post install hook creates the project graph... which contains quite a bit of work and appears to be hanging at times.

Expected Behavior

The post install hook is slimmer, and has a hard timeout of 30s to avoid hanging.

Related Issue(s)

Fixes #31694

@AgentEnder AgentEnder requested a review from a team as a code owner July 22, 2025 20:12
@AgentEnder AgentEnder requested a review from Cammisuli July 22, 2025 20:12
Copy link

vercel bot commented Jul 22, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
nx-dev ✅ Ready (Inspect) Visit Preview Jul 25, 2025 10:41pm

Copy link
Contributor

nx-cloud bot commented Jul 22, 2025

View your CI Pipeline Execution ↗ for commit ce52677

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 8m 55s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 2m 25s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 3s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded 3s View ↗
nx documentation ✅ Succeeded 1m 44s View ↗

☁️ Nx Cloud last updated this comment at 2025-07-25 22:45:28 UTC

Copy link

@Hardanish-Singh Hardanish-Singh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AgentEnder AgentEnder merged commit 06ffc84 into master Jul 27, 2025
5 checks passed
@AgentEnder AgentEnder deleted the trim-postinstall branch July 27, 2025 21:56
jaysoo pushed a commit that referenced this pull request Jul 28, 2025
## Current Behavior
The post install hook creates the project graph... which contains quite
a bit of work and appears to be hanging at times.

## Expected Behavior
The post install hook is slimmer, and has a hard timeout of 30s to avoid
hanging.

## Related Issue(s)
<!-- Please link the issue being fixed so it gets closed when this is
merged. -->

Fixes #31694
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.

nx.json should support parallel as a percentage to be consistent with cli arg
3 participants