Support naming worker with query suffix

[tiamed]

Description

The workers loaded using ?worker is Untitled in chrome://inspect/#workers.
Need to provide an option for configuring the name.

Suggested solution

?worker&name=xxx

Alternative

new MyWorker({
  name: 'xxx',
});

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.

[sapphi-red]

It seems name param works with the Constructor one.

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

I think it makes sense to add to the Query Suffixes one too.

import MyWorker from './worker?worker'

const worker = new MyWorker({ name: 'foo' })

[ihch]

I didn't do the implementation with query suffix, but I implemented the naming method using a constructor.

Can I try to pull request?

(The query suffix implementation is having trouble with type definitions and I'm looking at this issue microsoft/TypeScript#38638 )

[sapphi-red]

@ihch

There's two ways to create a worker in Vite. One is by the Constructors and the other is by the query suffixes.
The name param already works with the former one. But doesn't work with the latter one.

In my previous comment, I suggested to add the name param support for the query suffix one by passing a name param to the exported constructor. I guess you implemented in this way. I don't think we should support ?worker&name=foo so it's fine.

Anyway, pull requests are welcome and are recommended if you have a concrete code as it's easier to discuss.

[ihch]

Sorry for the delay!
I created a pull request. #14032