Chore: Build node workers as ES modules

[ibgreen]

@belom88 Is it possible to test with this branch?

[belom88]

This is exactly what I've reverted in #2790 . We can probably revert *.cjs back to *.js extensions but --format=esm is exactly what is breaking the tile converter right now.
cjs/js is about what is working in NodeJS by default. #2775 seem for me a breaking commit.

[belom88]

The error is "Worker exception: Error: Dynamic require of "stream" is not supported"

[ibgreen]

The error is "Worker exception: Error: Dynamic require of "stream" is not supported"

Can you pintpoint the line where this is happening? Since I can't find it in the source code I am assuming it is happening in the built code. But if we are building with --format=esm, seems that require shouldn't be emitted, so it is a mystery error?

[belom88]

The error is "Worker exception: Error: Dynamic require of "stream" is not supported"

Can you pintpoint the line where this is happening? Since I can't find it in the source code I am assuming it is happening in the built code. But if we are building with --format=esm, seems that require shouldn't be emitted, so it is a mystery error?

Probably, the root cause is in decoder/encoder libs (modules/draco/src/libs/draco_encoder.js, modules/draco/src/libs/draco_wasm_wrapper.js)

[belom88]

The tile converter fails with "Worker exception: Error: Dynamic require of "stream" is not supported". Probably draco encoder/decoder libs a not compatible with esm.

[ibgreen]

Probably draco encoder/decoder libs a not compatible with esm.

Yes that could be. I suppose if they were renamed to .cjs it might work. I guess in that case we'd need to copy them from the Google CDN.

[ibgreen]

@belom88 Can you take another look at this now that it is rebased on the new build system. Do your concerns still apply?