fix(worker): import.meta.*

[poyoho]

Description

fix: #7699

Additional context

1. import.meta.glob will transform to dynamic import, so only support format: es and module worker.
2. import.meta.globEager will transform to static import, so only support format: iife|es and module worker.
3. supplement unit-test for worker constructor in nested worker and simple worker.
4. clean worker sourcemap warning.
5. inline worker sourcemap 404

---

What is the purpose of this pull request?

• Bug fix
• New Feature
• Documentation update
• Other

Before submitting the PR, please make sure you do the following

• Read the Contributing Guidelines.
• Read the Pull Request Guidelines and follow the Commit Convention.
• Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
• Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
• Ideally, include relevant tests that fail without this PR but pass with it.

[sapphi-red]

Also The emitted file "assets/url-worker.f4391e1e.js.map" overwrites a previously emitted file of the same name. warning happens in my environment.

[poyoho]

Also The emitted file "assets/url-worker.f4391e1e.js.map" overwrites a previously emitted file of the same name. warning happens in my environment.

resolved. 😊

[poyoho]

sourcemap had a micro error. wait for a mins.

[poyoho]

done!

[poyoho]

import.meta.url build with es it will replace with esbuild minify will got an unexpected data.

This can be solved only by upgrading the target of esbuild

import.meta.url build with iife it will polyfill with rollup will be used document in the worker

Rollup should set format: "es" to avoid it. But I think format: "iife" can still be replaced.

[patak-cat]

I wonder what is your current assessment of this PR @poyoho. If we would like to avoid possible regressions, I think the best would be to queue it for Vite 3.0 (We may start the beta in around two weeks so I'm currently leaning towards this option). If there is some straightforward fix that you would like to extract from this PR to be applied in 2.9, maybe that is also an option. What do you think? 2.9 looks quite solid to me... maybe we should already start focusing our efforts in 3.0

[poyoho]

In this PR, I reverted the import.meta.url all content (contained renderChunk hook replace), So I think this PR is like a patch.

Fix the following:

1. import.meta.glob will transform to dynamic import, so only support format: es and module worker.
2. import.meta.globEager will transform to static import, so only support format: iife|es and module worker.
3. supplement unit-test for worker constructor in nested worker and simple worker.
4. fix clean worker source map warning. (emit same file many times)
5. fix inline worker source map 404 (inline worker self.location.href base on site base. make it use absolute paths.)

And about import.meta.url in worker.

import.meta.url build with es it will replace with esbuild minify will got an unexpected data.

This can be solved only by upgrading the target of esbuild.

import.meta.url build with iife it will polyfill with rollup will be used document in the worker

Rollup should set format: "es" to avoid it.

Discussing this problem, we can ignore the classic worker because it can't work in dev. I think it can be preset that the user has already set a higher version of the target.

format: es should need to setting higher version of the target. So I think it is ok.

format: iife after setting higher version of the target it also output compatible code I think format: "iife" can still be replaced.

[poyoho]

@bluwy Thank for your review.

[bluwy]

LGTM. Perhaps we can tackle the import.meta.url replacement in a separate PR, or in Vite 3 when we preserve import.meta.url.

[poyoho]

I've done something similar, but it's really a little difficult

[poyoho]

the import.meta.url can resolve in #7837 (comment).