fix(vite): include same-ID resolutions in module timings

[SaKaNa-Y]

Important

Please take a moment to read this. Thank you!

I should include a brief explanation of the problem in my own words in every PR. If that explanation is missing, please @mention me and do not merge this PR until I have added it. You may also leave this PR unaddressed (because this means I have not fulfilled my responsibilities as the author).

If my explanation is unclear or difficult to follow, please ask me to clarify or provide reproduction steps or supporting evidence.

I welcome suggestions and counterarguments, especially questions about anything I may have overlooked. (Your feedback helps me learn and improve. 🙏)

I hold myself to this standard for every PR, regardless of its size.

When investigating slow module processing in the Vite module graph, a successful plugin resolveId call can be missing from the module's resolve timings. This happens when the hook returns its input ID, either as a string or as an object with an id field. Plugin details retain the call, but the module summary excludes it because the ID did not change.

Record successful same-ID calls for module aggregation. The existing resolution traversal handles the self-reference, and invalidation removes the recorded timing. Calls that return undefined or null remain in plugin details because their records do not identify the final module reliably.

Related fixes: #599 includes timings from transform hooks that return no code. #606 includes timings from load hooks that return no code. Each PR targets main and can be reviewed and merged independently.

Validation on this branch:

• Real Vite tests fail with the old condition and pass with this change for both string and object results. They also cover module lookup and invalidation.
• A control with two importers keeps declined resolutions out of both target modules.
• pnpm test --run: 52 files passed, 354 tests passed, 2 skipped.
• pnpm lint, pnpm typecheck, and pnpm build passed.

[pkg-pr-new]

Open in StackBlitz

@vitejs/devtools

npm i https://pkg.pr.new/@vitejs/devtools@607

@vitejs/devtools-kit

npm i https://pkg.pr.new/@vitejs/devtools-kit@607

@vitejs/devtools-oxc

npm i https://pkg.pr.new/@vitejs/devtools-oxc@607

@vitejs/devtools-rolldown

npm i https://pkg.pr.new/@vitejs/devtools-rolldown@607

@vitejs/devtools-vite

npm i https://pkg.pr.new/@vitejs/devtools-vite@607

@vitejs/devtools-vitest

npm i https://pkg.pr.new/@vitejs/devtools-vitest@607

commit: 68a5fde