ignore hints no longer ignored with v4.1.0

[ghiscoding]

Describe the bug

I have a few lines that I want to ignore and I had some v8 ignore if or v8 ignore next and it was working fine when used with rolldown-vite with Vitest < 4.1 and they no longer work unless I add @preserve but I don't want to since they end up in the minified code.

I'm not sure if has anything to do with PR #9204 but it was working fine with version ^4.0.18, it only regressed with 4.1

I currently have these overrides to use Rolldown-Vite

overrides:
  '@vitest/coverage-v8>vite': npm:rolldown-vite@latest
  '@vitest/ui>vite': npm:rolldown-vite@latest
  vitest>vite: npm:rolldown-vite@latest

I tried to use Vite 8 instead, but that didn't change anything

overrides:
  '@vitest/coverage-v8>vite': ^8.0.0
  '@vitest/ui>vite': ^8.0.0
  vitest>vite: ^8.0.0

Reproduction

For example this simple ignore, is no longer being ignored

function alert(msg) {
  // v8 ignore if
  if (!msg) {
    msg = 'Hello World';
  }
  alert(msg);
}

but if I add @preserve then it is ignored

function alert(msg) {
  // v8 ignore if -- @preserve
  if (!msg) {
    msg = 'Hello World';
  }
  alert(msg);
}

The @preserve wasn't necessary as long as we were adding Rolldown-Vite overrides.

For a full, but very large, repo here is the update to v4.1.0. I used to have 100% coverage with the v8 ignores, but I no longer have 100% with v4.1.0
ghiscoding/slickgrid-universal#2478

System Info

'vitest': 4.1.0
'@vitest/coverage-v8': 4.1.0
'@vitest/ui': 4.1.0
platform: Windows 10

Used Package Manager

pnpm

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.

[ghiscoding]

it seems to actually be caused by the fact that I still need to use an hybrid of Vite 7 and Vite 8. After changing my overrides to use Vite 8 and recreating my pnpm lock file, it does work now

catalog:
  vite: ^7.3.1

overrides:
  '@vitest/coverage-v8>vite': ^8.0.0
  '@vitest/ui>vite': ^8.0.0
  vitest>vite: ^8.0.0

it might be because the Vite 8 overrides weren't properly applied and it actually helped after bumping even vite: ^7.3.1 to Vite 8 and then re-downgrading to Vite 7 afterward. And if you're wondering why I'm having an hybrid of both versions, it's really because my project is a monorepo supporting multiple frameworks and they're not all upgrade to Vite 8 yet, hence the hybrid...

anyway it works now, so closing and if anyone else has problems then try steps above.
Thanks