Guy Bedford

> There is a text overlapping issue when running in run_in_shared_worker and run_in_dedicated_worker. Can you clarify what you mean by this? Is that a change required in this PR further...

Please re-request a review when this is ready for another review.

Under this specification, it throws a `SyntaxError` when it sees no `[[ModuleSource]]` defined as you describe in https://tc39.es/proposal-source-phase-imports/#sec-source-text-module-record-initialize-environment (and also separately for dynamic imports). With the new ESM Phase Imports...

I added an editorial update in https://github.com/tc39/ecma262/pull/3492/commits/a91247d9c5fc8a5ba65c47f41d7b92f25a171ff4 to change this to "must be initialized" over should.

Thanks for clearing the clear reproduction on this one. This edge case gets to the heart of the projects assumptions, and fixing it is a heuristic exercise unfortunately. The question...

If your browser does not support dynamic import, you should use an older version of es-module-shims that polyfills dynamic import. Perhaps try version 1.10.1.

Very few browsers do not support dynamic import, so the minimum versions here are actually very low. See https://caniuse.com/es6-module-dynamic-import. The main thing is Firefox before 67 and Safari before 11.1,...

You might try adding something like the following to the top of the page: ```html (function () { var dynamicImportSupported = false; try { eval("import(m)") } catch (e) { if...

Perhaps try using `wasm-tools shrink` to reduce the test case here, to share further.

`wasm-tools shrink --help` gives a full end to end example demostrating how to create a `predicate.sh` script that can pass or fail on variations of the original wasm file. Alternatively...