fix: errors in worker handling

[poyoho]

Description

classic worker options analyse error and pop importAnalysis plugin in worker bundle

1. case: ') ... ,' mean no worker options parmas will got a error catch.because commaIndex > endIndex find a error index.
2. ignore importAnalysis in worker, preLoad is not support in worker(preload method use document). There is no CSS in the worker, so there is no need to load dependencies first and then modules so we can ignore importAnalyseBulld
3. refactor worker test

• move the worker in truth dir
• jestPerTestSetup add <testdir>/config.js to config vite test server
• add format: es test for worker.

4. @vite-ignore error message add postion

Additional context

---

What is the purpose of this pull request?

• Bug fix
• New Feature
• Documentation update
• Other

[patak-dev]

I think we should pass isWorker as a param to resolveBuildPlugins instead. Adding it to the config will make it visible to the final user

[bluwy]

I'm not 100% sure if removing import analysis for workers is safe, but I have some small suggestions below.

[patak-dev]

@poyoho, @bluwy has a point, we are going to lose import.meta.glob support in workers (we should add a test case for it).

[poyoho]

I update desc

1. case: ') ... ,' mean no worker options parmas will got a error catch.because commaIndex > endIndex find a error index.
2. ignore importAnalysis in worker, preLoad is not support in worker(preload method use document). There is no CSS in the worker, so there is no need to load dependencies first and then modules so we can ignore importAnalyseBulld

please cc

@patak-dev @bluwy

[patak-dev]

Looks good to me. I've added it for discussion in the next team meeting due to the new isWorker field in ResolvedConfig. This PR should be merged before releasing 2.9

[bluwy]

This LGTM too, but will have this discussed in the meeting as mentioned by patak.

Also some minor nitpicks below 😬

[poyoho]

@patak-dev @bluwy

There are major changes to the script of the unit test, I don't know if this is good or not, please help to rereview 😘

[patak-dev]

We talked today about the PR with the team, it is ok to have isWorker in ResolvedConfig 👍🏼
Thanks again for your work here @poyoho!