Skip to content

Conversation

Coly010
Copy link
Contributor

@Coly010 Coly010 commented Aug 14, 2025

Current Behavior

The @nx/js:node executor had several process management issues:

  • Multiple concurrent builds could cause overlapping/corrupted console output,
    especially in NestJS projects
  • Process stdout streams weren't properly coordinated between different tasks
  • Rapid file changes could trigger multiple simultaneous rebuilds without
    proper debouncing
  • Console output formatting could break with mixed output from overlapping
    processes

Expected Behavior

The node executor should:

  • Handle process output streams cleanly with no overlapping/corrupted output
  • Properly debounce rebuild triggers to prevent resource conflicts
  • Maintain clean console formatting even with rapid file changes
  • Coordinate output between concurrent processes using a global stream manager

Related Issue(s)

Fixes #30247
Fixes #22999
Fixes #22945

@Coly010 Coly010 self-assigned this Aug 14, 2025
Copy link

vercel bot commented Aug 14, 2025

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

Project Deployment Preview Updated (UTC)
nx-dev Ready Ready Preview Aug 15, 2025 5:25pm
Copy link
Contributor

nx-cloud bot commented Aug 14, 2025

View your CI Pipeline Execution ↗ for commit 42b416d

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 39m 36s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 1m 46s 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 51s View ↗

☁️ Nx Cloud last updated this comment at 2025-08-15 17:58:11 UTC

@Coly010 Coly010 marked this pull request as ready for review August 14, 2025 16:08
@Coly010 Coly010 requested a review from a team as a code owner August 14, 2025 16:08
@Coly010 Coly010 requested a review from leosvelperez August 14, 2025 16:08
@Coly010 Coly010 force-pushed the node-nest-rebuilds branch from c8de146 to 41fd2af Compare August 15, 2025 10:16
@Coly010 Coly010 force-pushed the node-nest-rebuilds branch from 13ea88a to f3c3632 Compare August 15, 2025 12:54
@Coly010 Coly010 force-pushed the node-nest-rebuilds branch from 84d9cea to 42b416d Compare August 15, 2025 17:12
@Coly010 Coly010 merged commit d48ff63 into master Aug 15, 2025
7 checks passed
@Coly010 Coly010 deleted the node-nest-rebuilds branch August 15, 2025 18:01
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 Aug 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

2 participants