fix(js): node executor now correctly kills tasks when exiting

[ndcunningham]

Previously, the node executor od @nx/js did not kill spawned processes due to not awaiting finishing of killing and infinite loop.
This fix allows for seemless running nx in docker containers with support for signals

Taken from: #19219

[vercel]

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

1 Ignored Deployment

Name	Status	Preview	Updated (UTC)
nx-dev	⬜️ Ignored (Inspect)	Visit Preview	Jun 4, 2024 4:42pm

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit aeca8f2. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution

---

🟥 Failed Commands
nx affected --targets=e2e,e2e-ci --base=a8ae302ae49f76aa068c90b59e935dcbc4deb46b --head=aeca8f2f1afedcce985800f978805c8e17535935 --parallel=1

✅ Successfully ran 5 targets

• nx affected --targets=lint,test,build --base=a8ae302ae49f76aa068c90b59e935dcbc4deb46b --head=aeca8f2f1afedcce985800f978805c8e17535935 --parallel=3
• nx run-many -t check-imports check-commit check-lock-files check-codeowners documentation --parallel=1 --no-dte
• nx-cloud record -- nx format:check --base=a8ae302ae49f76aa068c90b59e935dcbc4deb46b --head=aeca8f2f1afedcce985800f978805c8e17535935
• nx documentation --no-dte
• nx affected -t e2e-macos-ci --parallel=1 --base=a8ae302ae49f76aa068c90b59e935dcbc4deb46b --head=aeca8f2f1afedcce985800f978805c8e17535935

---

Sent with 💌 from NxCloud.

[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.