[mkantautas]

Vite 5 renamed the SSR flag it passes to a config function: Vite 4 calls it with { command, mode, ssrBuild }, Vite 5+ with { command, mode, isSsrBuild, isPreview }. Knip only passed ssrBuild, so with build: { rollupOptions: { input: isSsrBuild ? './server/app.ts' : undefined } } the SSR entry and everything it imports are reported as unused files.

Knip now passes isSsrBuild along with ssrBuild, so configs written for either version see the SSR build.

Extended the vite plugin fixture with an SSR-only input. On main the test fails with server/app.ts in unused files; it passes now. The vitest-regression fixture still covers ssrBuild. The full test suite and pnpm run ci pass.

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/knip@2191

npm i https://pkg.pr.new/@knip/language-server@2191

npm i https://pkg.pr.new/@knip/mcp@2191

commit: c9e852f

[webpro]

Thanks, Mantas! Good catch on the Vite 5 rename, and keeping ssrBuild keeps Vite 4 configs working.

[webpro]

🚀 This pull request is included in v6.43.0. See Release 6.43.0 for release notes.

Using Knip in a commercial project? Please consider becoming a sponsor.