worker: add support for platformData

[jasnell]

The worker.platformData and worker.setPlatformData() APIs allow an arbitrary, cloneable JavaScript value to be set and passed to all new Worker instances spawned from the current context. It is similar to workerData except that platformData is set independently of the new Worker() constructor, and the value is passed automatically to all new Workers.

This is a partial fix of #30992
but does not implement a complete fix.

Signed-off-by: James M Snell [email protected]

/cc @addaleax @coreyfarrell

[devsnek]

lgtm with the global issue solved. Corey's proposed solution seems workable but still kind of unfortunate.

[Trott]

Would it make sense to implement this as experimental so we have some flexibility to tinker for the first few patch/minor releases?

[jasnell]

We could get by with making platformData a Map to achieve the result. And yes, experimental is fine too start.

[addaleax]

Since this works a lot like process.env except for non-string-data, maybe the name could also reflect that, because environment variables are something that developers probably tend to be more familiar with than the details of Workers?

[jasnell]

Since this works a lot like process.env except for non-string-data, maybe the name could also reflect that, because environment variables are something that developers probably tend to be more familiar with than the details of Workers?

My only concern there would be developers mistaking it for environment variables. Hmm... perhaps setEnvironmentData()?

[nodejs-github-bot]

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

[jasnell]

@coreyfarrell @addaleax @Trott @devsnek .. updated! Please take another look

[coreyfarrell]

This looks good, I think it will work for nyc once combined with the ability to inject code to run before the workers main script.

[jasnell]

@addaleax @coreyfarrell @gireeshpunathil ... alrighty! Updated! Please take a look.

[nodejs-github-bot]

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

[jasnell]

Landed in 802b3e7