bug: --typecheck produces one task per browser instance

[nwidynski]

Describe the bug

Type checking should be independent from browser instances, since they don't affect each other.

While i think tsc isn't invoked more than once per module, the test suite is reported once per file<->project combination, while it should only be reported once. This distorts the reported suite runtime and convolutes the console report in general.

Here is what it currently looks like:

Here is how i would expect it to look like:

Reproduction

Add multiple test.browser.instances and enable typecheck via vitest. You will observe typecheck being reported once per browser instance, although it is independent. The typecheck result duration will also be the same for each module, while still being summed for summary reporting.

System Info

OSX, [email protected], vitest@latest, vite@latest

Used Package Manager

npm

Validations

• Follow our Code of Conduct
• Read the Contributing Guidelines.
• Read the docs.
• Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
• Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
• The provided reproduction is a minimal reproducible example of the bug.

[github-actions]

Hello @nwidynski. Please provide a minimal reproduction using a GitHub repository or StackBlitz (you can also use examples). Issues marked with needs reproduction will be closed if they have no activity within 3 days.

[nwidynski]

@AriPerkkio https://stackblitz.com/edit/vitest-tests-browser-examples-5prfthrv?file=vitest.config.ts

Execute npm run test, which runs typechecking with vitest. You can observe a typecheck being reported for each browser instance and the runtime being wrongly reported.