Description

PR #22400 ("fix: refer to rolldownOptions instead of deprecated rollupOptions in messages") fixed most user-facing references to the deprecated rollupOptions name, but three occurrences of rollupOptions.input in packages/vite/src/node/build.ts were not updated.

Affected version

[email protected] and current main branch

Affected file

packages/vite/src/node/build.ts

1. JSDoc comment (line 260) — shown as IDE hover tooltip for build.ssr:

/**
 * Produce SSR oriented build. Note this requires specifying SSR entry via
 * `rollupOptions.input`.   // ← should be rolldownOptions.input
 */
ssr?: boolean | string

2. Runtime error message (line 604) — thrown when an .html file is passed as SSR entry:

`rollupOptions.input should not be an html file when building for SSR. ` +
  `Please specify a dedicated SSR entry.`,

3. Runtime error message (line 617) — thrown when build.cssCodeSplit: false and CSS files are in input:

`When "build.cssCodeSplit: false" is set, "rollupOptions.input" should not include CSS files.`,

Inconsistency with PR #22400

PR #22400 updated the following in the same file:

Open question

Were these three occurrences intentionally left as-is (e.g. because users hitting these errors are likely using rollupOptions directly), or were they simply not covered by #22400?

If updating them is appropriate, the fix would be to replace rollupOptions.input with rolldownOptions.input in the three locations above. Happy to submit a PR once confirmed.