fix(vite): include no-result transform hooks 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 plugin can appear cheaper than its recorded calls. A transform hook can spend time on a module and return undefined or null to keep the input code. Plugin details record that time, but the module summary filters out transforms without a code result.

Include all recorded transforms in the module summary. This preserves the existing code results and module sizes while counting the time spent in hooks that return no code.

Related fixes: #606 includes timings from load hooks that return no code. #607 includes timings from successful resolveId calls that return the input ID. Each PR targets main and can be reviewed and merged independently.

Validation on this branch:

• The regression cases fail for undefined and null with the old aggregation. They pass with this change. Unchanged code, changed code, and an empty string also pass.
• pnpm test --run: 52 files passed, 356 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@599

@vitejs/devtools-kit

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

@vitejs/devtools-oxc

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

@vitejs/devtools-rolldown

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

@vitejs/devtools-vite

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

@vitejs/devtools-vitest

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

commit: 406dc44

[SaKaNa-Y]

Actually, all three PRs fix missing hook timings in specific cases.

#599 fixes the issue where time spent in transform is not included in the module summary when it returns null or undefined.

#606 fixes the issue where time spent in load is not counted when it returns null or undefined.

#607 fixes the issue where time spent in resolveId is not included in the module summary when it successfully returns the same module ID as its input.