feat: worker support query url

[poyoho]

Description

fix: #7569

Additional context

feat support below syntax to import worker url.

import workerUrl from '../simple-worker?worker&url'

function text(el, text) {
  document.querySelector(el).textContent = text
}

const worker = new Worker(workerUrl, { type: 'classic' })

worker.addEventListener('message', (ev) => {
  text('.simple-worker-url', JSON.stringify(ev.data))
})

about new URL

new URL('./hello.js?worker', import.meta.url)

I think new URL is better not to add too much non-conforming syntax, so I do not support it now. And if the href is not added to break the workerImportMetaUrl match, it may make another issues

For example:

new Worker(new URL('./hello.js?worker', import.meta.url))

And doing a little refactor, I think worker related logic should be in worker.ts.

---

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]

It seems this PR supersedes #5845.

[poyoho]

oh, If the Author goto rebase and complementing his missing parts of this I will close this.

[bluwy]

I'd still be in favour of this PR though as it has more tests and refactors 🙂

[patak-dev]

For reference, we talked with the team about the PR and we are good to merge it. We tried to find a way to avoid introducing new URL queries but this feature justifies the addition, and there doesn't seem to be any other way until the standard evolves to support modifiers. Let's merge the PR once it is rebased.

[poyoho]

For reference, we talked with the team about the PR and we are good to merge it. We tried to find a way to avoid introducing new URL queries but this feature justifies the addition, and there doesn't seem to be any other way until the standard evolves to support modifiers. Let's merge the PR once it is rebased.

May be we can clean all the url query in vite@4 😁