Please allow an option to minify js but not css

[shirotech]

Description

vite/packages/vite/src/node/plugins/css.ts

Line 424 in 5c6cf5a

content = await minifyCSS(content, config)

Looks like part of this code will run in both serve and build mode.

Suggested solution

• when in serve mode, minifying the CSS can be slow, and hence would be good if there is an option to opt-out of using the config.build.minify flag here, as it doesn't seem to belong in serve mode.
• if the above is not possible, please allow an option to configure the transform call here:

  vite/packages/vite/src/node/plugins/css.ts

  Line 1227 in 5c6cf5a

  const { code, warnings } = await transform(css, {

  it seems most options are hardcoded apart from target.
• possible introduce a new option called config.build.cssMinify as we have config.build.cssTarget already to keep it consistent, then we can spread it like so in this line:

    const { code, warnings } = await transform(css, {
      loader: 'css',
      minify: true,
      target: config.build.cssTarget || undefined, // probably need to keep this for backwards compatitibility
      ...config.build.cssMinify
    })

Alternative

No response

Additional context

No response

Validations

• Follow our Code of Conduct
• Read the Contributing Guidelines.
• Read the docs.
• Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

[sapphi-red]

Looks like part of this code will run in both serve and build mode.

No, it does not. There is always a return inside this if. So it won't reach that line when serve mode.

vite/packages/vite/src/node/plugins/css.ts

Line 359 in 5c6cf5a

if (config.command === 'serve') {

vite/packages/vite/src/node/plugins/css.ts

Line 382 in 5c6cf5a

return [

[shirotech]

I don't believe it was right to close the issue like that, the feature request still stands, as there are still use cases when someone wants to minify js, but not CSS. Right now it is not possible..

[sapphi-red]

I thought your reason of this request was performance.
If you want to disable css minification, it is a duplicate of #5619. I would be great if you write the reason why you would like to do that there.