fix(build): ensure a non-zero exit code on failure (#1038)

Previously, when the following actions failed, the process would exit with an exit code of zero, which would break CI.
- transform with Esbuild
- transform CSS
- minify CSS
- load PostCSS config

Author: aleclarson

src/node/build/buildPluginCss.ts

@@ -85,6 +85,7 @@ export const createBuildCssPlugin = ({
           if (result.errors.length) {
             console.error(`[vite] error applying css transforms: `)
             result.errors.forEach(console.error)
+            process.exit(1)
           }
           css = result.code
           modules = result.modules
@@ -228,6 +229,7 @@ function minifyCSS(css: string) {
   if (res.errors && res.errors.length) {
     console.error(chalk.red(`[vite] error when minifying css:`))
     console.error(res.errors)
+    process.exit(1)
   }
 
   if (res.warnings && res.warnings.length) {

src/node/build/buildPluginEsbuild.ts

@@ -39,7 +39,8 @@ export const createEsbuildPlugin = async (
             ...jsxConfig,
             ...(isVueTs ? { loader: 'ts' } : null)
           },
-          jsx
+          jsx,
+          true // exitOnFailure
         )
       }
     }

src/node/esbuildService.ts

@@ -67,7 +67,8 @@ export const transform = async (
   src: string,
   request: string,
   options: TransformOptions = {},
-  jsxOption?: SharedConfig['jsx']
+  jsxOption?: SharedConfig['jsx'],
+  exitOnFailure?: boolean
 ) => {
   const service = await ensureService()
   const file = cleanUrl(request)
@@ -114,6 +115,9 @@ export const transform = async (
       console.error(e)
     }
     debug(`options used: `, options)
+    if (exitOnFailure) {
+      process.exit(1)
+    }
     return {
       code: '',
       map: undefined

src/node/utils/cssUtils.ts

@@ -68,7 +68,7 @@ export async function compileCss(
   isBuild: boolean = false
 ): Promise<SFCStyleCompileResults | string> {
   const id = hash_sum(publicPath)
-  const postcssConfig = await loadPostcssConfig(root)
+  const postcssConfig = await loadPostcssConfig(root, isBuild)
   const { compileStyleAsync } = resolveCompiler(root)
 
   if (
@@ -138,7 +138,8 @@ type PostCSSConfigResult = ReturnType<typeof postcssrc> extends Promise<infer T>
 let cachedPostcssConfig: PostCSSConfigResult | null | undefined
 
 async function loadPostcssConfig(
-  root: string
+  root: string,
+  exitOnFailure?: boolean
 ): Promise<PostCSSConfigResult | null> {
   if (cachedPostcssConfig !== undefined) {
     return cachedPostcssConfig
@@ -150,13 +151,16 @@ async function loadPostcssConfig(
     if (!/No PostCSS Config found/.test(e.message)) {
       console.error(chalk.red(`[vite] Error loading postcss config:`))
       console.error(e)
+      if (exitOnFailure) {
+        process.exit(1)
+      }
     }
     return (cachedPostcssConfig = null)
   }
 }
 
 export async function resolvePostcssOptions(root: string, isBuild: boolean) {
-  const config = await loadPostcssConfig(root)
+  const config = await loadPostcssConfig(root, isBuild)
   const options = config && config.options
   const plugins = config && config.plugins ? config.plugins.slice() : []
   plugins.unshift(require('postcss-import')())