worker: add postMessageToThread

[ShogunPanda]

This PR is a total rethinking of #53488.

This PR adds a new API to worker_threads called postMessageToThread that allow to send arbitrary messages to any worker thread just identifying it by its threadId.

The recipient thread must have a listener on the new workerMessage event emitted on process otherwise the postMessageToThread will throw an exception.

In order not to have any thread block, the call is async to use Atomics.waitAsync.

[mcollina]

lgtm

I prefer this design

[benjamingr]

This is better

[Flarna]

Not sure if new errors should be also marked as experimental like the API using it.

And I'm also not sure if workerMessage should be documented in process events as it is a reserved event now.

If we see a too high risk of clashes by reserving this event name we could move to a symbol.

[ShogunPanda]

Not sure if new errors should be also marked as experimental like the API using it.

You are right: I marked them as well.

And I'm also not sure if workerMessage should be documented in process events as it is a reserved event now.

I documented so that users know they might have to listen to it.

If we see a too high risk of clashes by reserving this event name we could move to a symbol.

👍

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/60182/

[nodejs-github-bot]

Commit Queue failed

- Loading data for nodejs/node/pull/53682
✔  Done loading data for nodejs/node/pull/53682
----------------------------------- PR info ------------------------------------
Title      worker: add postMessageToThread (#53682)
Author     Paolo Insogna <paolo@cowtech.it> (@ShogunPanda)
Branch     ShogunPanda:send-to-worker -> nodejs:main
Labels     semver-minor, process, lib / src, author ready, worker, needs-ci
Commits    1
 - worker: add postMessageToThread
Committers 1
 - Paolo Insogna <paolo@cowtech.it>
PR-URL: https://github.com/nodejs/node/pull/53682
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/53682
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
--------------------------------------------------------------------------------
   ⚠  Commits were pushed since the last approving review:
   ⚠  - worker: add postMessageToThread
   ℹ  This PR was created on Tue, 02 Jul 2024 08:38:46 GMT
   ✔  Approvals: 4
   ✔  - Matteo Collina (@mcollina) (TSC): https://github.com/nodejs/node/pull/53682#pullrequestreview-2153146746
   ✔  - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/53682#pullrequestreview-2157902186
   ✔  - Benjamin Gruenbaum (@benjamingr) (TSC): https://github.com/nodejs/node/pull/53682#pullrequestreview-2161833054
   ✔  - Gerhard Stöbich (@Flarna): https://github.com/nodejs/node/pull/53682#pullrequestreview-2162710662
   ✔  Last GitHub CI successful
   ℹ  Last Full PR CI on 2024-07-08T19:33:46Z: https://ci.nodejs.org/job/node-test-pull-request/60182/
- Querying data for job/node-test-pull-request/60182/
   ✔  Last Jenkins CI successful
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu

https://github.com/nodejs/node/actions/runs/9852162608

[nodejs-github-bot]

Landed in 66a635c