chore: test Vite PR 18329

[hi-ogawa]

Description

Sourcemap viz of fixtures/src/Vue/Counter/Counter.vue

• before
• after

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

• It's really useful if your PR references an issue where it is discussed ahead of time. If the feature is substantial or introduces breaking changes without a discussion, PR might be closed.
• Ideally, include a test that fails without this PR but passes with it.
• Please, don't make changes to pnpm-lock.yaml unless you introduce a new test example.

Tests

• Run the tests with pnpm test:ci.

Documentation

• If you introduce new functionality, document it. You can run documentation with pnpm run docs command.

Changesets

• Changes in changelog are generated from PR name. Please, make sure that it explains your changes in an understandable manner. Please, prefix changeset messages with feat:, fix:, perf:, docs:, or chore:.

[netlify]

✅ Deploy Preview for vitest-dev ready!

Built without sensitive environment variables

Name	Link
🔨 Latest commit	aa77ad7
🔍 Latest deploy log	https://app.netlify.com/sites/vitest-dev/deploys/670a17ede5cdbf00089c9ae9
😎 Deploy Preview	https://deploy-preview-6696--vitest-dev.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[AriPerkkio]

@hi-ogawa could you try if commenting out this part fixes the coverage? The new helper method added by Vite is on first line so I wonder if that's breaking here.

vitest/packages/vite-node/src/source-map.ts

Lines 64 to 68 in c4076e6

	// If the first line is not present on source maps, add simple 1:1 mapping ([0,0,0,0], [1,0,0,0])
	// so that debuggers can be set to break on first line
	if (map.mappings.startsWith(';')) {
	map.mappings = `AAAA,CAAA${map.mappings}`
	}

[hi-ogawa]

@AriPerkkio Good find! Commenting out that mapping fixed the issue. There is a diff in lines and statements, but I can see the same diff on main when I comment out the same line, so I suppose it's been affecting the coverage already.

I guess we need to find a different way to make inspect-brk work.

[hi-ogawa]

For example, currently the first line <script setup lang="ts"> is considered covered:

• https://htmlpreview.github.io/?https://raw.githubusercontent.com/hi-ogawa/vitest/chore-vite-ssr-identity/test/coverage-test/coverage-now/Vue/Defined.vue.html

but it's removed after removing first line mapping:

• https://htmlpreview.github.io/?https://raw.githubusercontent.com/hi-ogawa/vitest/chore-vite-ssr-identity/test/coverage-test/coverage-18329-x/Vue/Defined.vue.html

[AriPerkkio]

I guess we need to find a different way to make inspect-brk work.

I think for now it would be enough to set this breakpoint when we detect that vitest --inspect-brk is used.

[hi-ogawa]

Closing in favor of #6732