support `?inline` worker in constructors worker

Clear and concise description of the problem

we can use inline worker with import worker from "./worker?worker&inline"

and need to support inline worker in constructors worker.

Suggested solution

1. use query syntax

const worker = new Worker(new URL('./worker.js?inline', import.meta.url))

but the end goal is to remove query suffixes everywhere for Vite.

1. put inline in worker options

const worker = new Worker(new URL('./worker.js', import.meta.url), {
  inline: true
})

that’s non-standard

1. use the comment

const worker = new Worker(new URL(/* #__INLINE__ */'./worker.js', import.meta.url))

and Rollup has a history of avoiding that too (e.g. webpackChunkName) and only supported pure comments thus far.

above solutions are not good.

this issue for it to discussed some ideas ❤️

Alternative

No response

Additional context

No response

Validations

• Follow our Code of Conduct
• Read the Contributing Guidelines.
• Read the docs.
• Check that there isn’t already an issue that request the same feature to avoid creating a duplicate.

[mathe42]

Will have a deeper look into it later.

[mathe42]

I’m not sure what’s the usecase for inlining workers yet, but I like the option with config.build.assetsInlineLimit since it reuses an existing config option. The naming could be a bit misleading though since worker isn’t really an asset, but I don’t think it’s a big issue 🤔

I don’t know either (limit requests?) but then just switch to http2. The only use case I can think of if you want to inline ALL workers. (When you want a single file so evrything is potentaly inlined into the html file)

(Only if you need to support IE11 (or older) in windows < 10 you have problems see https://caniuse.com/http2,webworkers )