(The lines below were relying on this variable being set inruntime_common.js, which I had moved into an if-block and made not global. I discovered this after tests failed. I figured the simplest solution was to make this code use its own local variable instead of re-introducing the global variable.)

To save on code size and keep this patch as small as possible can you revert this and hoistconst parentPort = worker_threads['parentPort']; outside theif?

Also, we still usevar in most places in our codebase sadly (there are a couple of reasons, that are probably not worth explaining here... i really should create an FAQ entry or some such for it).

(I'm not sure if some increasingly specific helper functions like this were the best route to go. One alternative would have been to make theget/require_node helpers to allow Deno/Bun too because they're mostly interchangeable, but it's probably better to be explicit. Another alternative would've been to make some generic helpers that take named parameters declaring which of Node/Node-Canary/Deno/Bun are allowed.)

I'm hoping we can just avoid this completely, but allowing NODE_JS_TEST to be set to /path/to/bun or /path/to/deno. No need for new config vars I think/hope.

I embraced that idea and it's much simpler now.

Ideally this change would live only in test/ code and only effectNODE_JS_TEST, not the main version of node js that we use.

i.e. can you make local function intest/common.py that does this override? Then use the local version instead ofshared.get_node_version?