enhance: improve error logging (#642)

.prettierignore

@@ -1,2 +1,3 @@
 pnpm-lock.yaml
 README.md
+test/integration/errors

src/bin/index.ts

@@ -11,6 +11,7 @@ import { bundle } from '../../src/index'
 import { prepare } from '../prepare'
 import { RollupWatcher } from 'rollup'
 import { logOutputState } from '../plugins/output-state-plugin'
+import { normalizeError } from '../lib/normalize-error'
 
 const helpMessage = `
 Usage: bunchee [options]
@@ -310,10 +311,15 @@ async function run(args: CliArgs) {
     }
   }
 
+  function onBuildError(error: Error) {
+    logError(error)
+  }
+
   let buildError: any
   bundleConfig._callbacks = {
     onBuildStart,
     onBuildEnd,
+    onBuildError,
   }
 
   if (watch) {
@@ -406,12 +412,9 @@ function logWatcherBuildTime(result: RollupWatcher[], spinner: Spinner) {
   })
 }
 
-function logError(error: any) {
-  if (!error) return
-  // logging source code in format
-  if (error.frame) {
-    process.stderr.write(error.frame + '\n')
-  }
+function logError(err: unknown) {
+  const error = normalizeError(err)
+  logger.error(error)
 }
 
 main().catch(exit)

src/bundle.ts

@@ -173,13 +173,18 @@ async function bundle(
   const generateTypes = hasTsConfig && options.dts !== false
   const rollupJobsOptions: BundleJobOptions = { isFromCli, generateTypes }
 
-  const assetJobs = await createAssetRollupJobs(
-    options,
-    buildContext,
-    rollupJobsOptions,
-  )
-
-  options._callbacks?.onBuildEnd?.(assetJobs)
+  try {
+    const assetJobs = await createAssetRollupJobs(
+      options,
+      buildContext,
+      rollupJobsOptions,
+    )
+
+    options._callbacks?.onBuildEnd?.(assetJobs)
+  } catch (error) {
+    options._callbacks?.onBuildError?.(error)
+    return Promise.reject(error)
+  }
 }
 
 export default bundle

src/lib/normalize-error.ts

@@ -0,0 +1,9 @@
+export function normalizeError(error: any) {
+  // Remove the noise from rollup plugin error
+  if (error.code === 'PLUGIN_ERROR') {
+    const normalizedError = new Error(error.message)
+    normalizedError.stack = error.stack
+    error = normalizedError
+  }
+  return error
+}

src/rollup-job.ts

@@ -15,6 +15,7 @@ import {
   BundleJobOptions,
 } from './types'
 import { removeOutputDir } from './utils'
+import { normalizeError } from './lib/normalize-error'
 
 export async function createAssetRollupJobs(
   options: BundleConfig,
@@ -44,7 +45,12 @@ export async function createAssetRollupJobs(
     bundleOrWatch(options, rollupConfig),
   )
 
-  return await Promise.all(rollupJobs)
+  try {
+    return await Promise.all(rollupJobs)
+  } catch (err: unknown) {
+    const error = normalizeError(err)
+    throw error
+  }
 }
 
 async function bundleOrWatch(

src/types.ts

@@ -58,6 +58,12 @@ type BundleConfig = {
      * @experimental
      */
     onBuildEnd?: (assetJobs: any) => void
+
+    /*
+     * This hook is called when the build errors
+     * @experimental
+     */
+    onBuildError?: (assetJob: any) => void
   }
 }
 

test/integration/errors/compile-error/package.json

@@ -0,0 +1,4 @@
+{
+  "name": "compile-error",
+  "main": "./dist/index.js"
+}

test/integration/errors/compile-error/src/index.tsx

@@ -0,0 +1,3 @@
+expor t functi on functi on App() {
+  return <div>App</di v>;
+}