Use new URL(..., import.meta.url) pattern in minimal runtime if EXPORT_ES6 is enabled

[valadaptive]

This should make it possible to use MINIMAL_RUNTIME_STREAMING_WASM_INSTANTIATION with bundlers like Vite that recognize and transform new URL('...', import.meta.url).

[valadaptive]

Ignore Copilot; GitHub auto-enabled that setting for my account 🙃

[Copilot]

Pull request overview

This PR adds support for bundler-friendly new URL(..., import.meta.url) pattern in minimal runtime when EXPORT_ES6 is enabled, specifically for the MINIMAL_RUNTIME_STREAMING_WASM_INSTANTIATION feature. This enables better integration with modern bundlers like Vite that recognize and transform this pattern.

• Introduces conditional logic to use new URL() pattern when both EXPORT_ES6 and MINIMAL_RUNTIME_STREAMING_WASM_INSTANTIATION are enabled
• Adds test to verify the generated code contains the expected URL pattern
• Handles special case for Audio Worklet environments where new URL() cannot be used

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File	Description
src/postamble_minimal.js	Adds preprocessor directives to conditionally generate new URL() pattern for wasm module loading based on ES6 and Audio Worklet settings
test/test_other.py	Adds test to verify that ES6 modules with streaming instantiation generate correct URL resolution using import.meta.url

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[kripken]

cc @RReverser

[RReverser]

We already do this for the non-minimal runtime, so makes sense to me.

Technically it does add a few extra bytes to the minimal runtime's output, but I suppose that's fine? @sbc100

[sbc100]

We have test_vite in test_browser.py what should be able to not only test that this string exists but that it works in the vite bundler.

IIUC we could add a variant of that test that uses -sMINIMAL_RUNTIME -sMINIMAL_RUNTIME_STREAMING_WASM_INSTANTIATION ?

[valadaptive]

I've added a test for Vite. Should I keep this one as well?

[sbc100]

Probably not worth keeping this one too

[valadaptive]

Removed it.

[sbc100]

Rather than adding a new test here you should be able to using @parameterize.

Something like:

  @parameterized({                                                                                        
    '': ([],),                                                                                                                          
    'minimal': (['-sMINIMAL_RUNTIME', '-sMINIMAL_RUNTIME_STREAMING_WASM_INSTANTIATION'],),                                        
  })  
  def test_vite_minimal_runtime(self, args):

(then add args to the compile_btest arguments).

[valadaptive]

Updated.

[sbc100]

Can you confirm this test fails before this PR, and passes after?

[valadaptive]

Manually confirmed the (newly @parameterized) test fails without the changes made in this PR.

[valadaptive]

CI seems to be green now; I think it was an issue with the autogenerated copyright date. Anything holding this up?