fix: add code location and codeframe for rollup errors (#406)

Author: danielroe

src/build.ts

@@ -1,11 +1,13 @@
 import { promises as fsp } from 'fs'
-import { relative, resolve, join, dirname } from 'pathe'
+import { relative, resolve, join, dirname, isAbsolute } from 'pathe'
 import * as rollup from 'rollup'
 import fse from 'fs-extra'
 import { defu } from 'defu'
 import { watch } from 'chokidar'
 import { debounce } from 'perfect-debounce'
 import type { TSConfig } from 'pkg-types'
+import type { RollupError } from 'rollup'
+import type { OnResolveResult, PartialMessage } from 'esbuild'
 import { printFSTree } from './utils/tree'
 import { getRollupConfig, RollupConfig } from './rollup/config'
 import { prettyPath, writeFile, isDirectory } from './utils'
@@ -138,7 +140,7 @@ async function _build (nitro: Nitro, rollupConfig: RollupConfig) {
 
   nitro.logger.start('Building server...')
   const build = await rollup.rollup(rollupConfig).catch((error) => {
-    nitro.logger.error('Rollup error: ' + error.message)
+    nitro.logger.error(formatRollupError(error))
     throw error
   })
 
@@ -204,7 +206,7 @@ function startRollupWatcher (nitro: Nitro, rollupConfig: RollupConfig) {
 
       // Encountered an error while bundling
       case 'ERROR':
-        nitro.logger.error('Rollup error: ', event.error)
+        nitro.logger.error(formatRollupError(event.error))
     }
   })
   return watcher
@@ -240,3 +242,21 @@ async function _watch (nitro: Nitro, rollupConfig: RollupConfig) {
 
   await reload()
 }
+
+function formatRollupError (_error: RollupError | OnResolveResult) {
+  try {
+    const logs: string[] = []
+    for (const error of ('errors' in _error ? _error.errors : [_error as RollupError])) {
+      const id = error.id || (_error as RollupError).id
+      let path = isAbsolute(id) ? relative(process.cwd(), id) : id
+      const location = (error as RollupError).loc || (error as PartialMessage).location
+      if (location) {
+        path += `:${location.line}:${location.column}`
+      }
+      logs.push(`Rollup error while processing \`${path}\`` + '\n\n' + ((error as PartialMessage).text || (error as RollupError).frame))
+    }
+    return logs.join('\n\n')
+  } catch {
+    return _error?.toString()
+  }
+}