fix(core): make forked-process-task-runner robust to closed processes

[scottbessler]

Ensure that processes that have ended are no longer in the set of processes. Further, ensure the processes are connected before sending them messages.

Current Behavior

forked-process-task-runner never removes processes from this.processes. As such, if you send the task-runner process messages after one of the processes has exited, it throws errors like these:

Error: Channel closed
    at new NodeError (node:internal/errors:393:5)
    at ChildProcess.target.send (node:internal/child_process:740:16)
    at nx/src/tasks-runner/forked-process-task-runner.ts:414:5
    at Set.forEach (<anonymous>)
    at process.<anonymous> (nx/src/tasks-runner/forked-process-task-runner.ts:412:8)
    at process.emit (node:events:525:35)

Expected Behavior

forked-process-task-runner should be resilient to this

Related Issue(s)

Fixes #

[vercel]

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

Name	Status	Preview	Updated
nx-dev	✅ Ready (Inspect)	Visit Preview	Jan 11, 2023 at 10:45PM (UTC)

[nx-cloud]

☁️ Nx Cloud Report

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

📂 See all runs for this branch

---

⌛ The following targets are in progress

• nx affected --target=e2e --base=5970246b51c17428b8d9f04123b06f780ac824bc --head=5c7572a865839d425ed9df1f611a70ada579fa3d --exclude=e2e-storybook,e2e-storybook-angular,e2e-react-native,e2e-detox --parallel=1
• nx affected -t test lint --base=5970246b51c17428b8d9f04123b06f780ac824bc --head=5c7572a865839d425ed9df1f611a70ada579fa3d --parallel=1

✅ Successfully ran 8 targets

• nx affected --target=e2e --base=5970246b51c17428b8d9f04123b06f780ac824bc --head=8e6cb50b40659a6d602f2e4099365c30a346a2be --exclude=e2e-storybook,e2e-storybook-angular,e2e-react-native,e2e-detox --parallel=1
• nx affected -t test lint --base=5970246b51c17428b8d9f04123b06f780ac824bc --head=8e6cb50b40659a6d602f2e4099365c30a346a2be --parallel=1
• nx affected --target=build --base=5970246b51c17428b8d9f04123b06f780ac824bc --head=8e6cb50b40659a6d602f2e4099365c30a346a2be --parallel=3
• nx run-many -t check-imports check-commit check-lock-files root-lint depcheck documentation --parallel=3 --no-dte
• nx-cloud record -- nx format:check --base=5970246b51c17428b8d9f04123b06f780ac824bc --head=8e6cb50b40659a6d602f2e4099365c30a346a2be
• nx affected --target=build --base=5970246b51c17428b8d9f04123b06f780ac824bc --head=5c7572a865839d425ed9df1f611a70ada579fa3d --parallel=3
• nx run-many -t check-imports check-commit check-lock-files root-lint depcheck documentation --parallel=3 --no-dte
• nx-cloud record -- nx format:check --base=5970246b51c17428b8d9f04123b06f780ac824bc --head=5c7572a865839d425ed9df1f611a70ada579fa3d

---

Sent with 💌 from NxCloud.

[scottbessler]

thanks!

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