feat!: exit with code (1) on build warnings

src/build.ts

@@ -42,6 +42,7 @@ export async function build (rootDir: string, stub: boolean, inputConfig: BuildC
     peerDependencies: [],
     alias: {},
     replace: {},
+    failOnWarn: true,
     rollup: {
       emitCJS: false,
       cjsBridge: false,

src/builder/rollup.ts

@@ -140,6 +140,9 @@ export function getRollupOptions (ctx: BuildContext): RollupOptions {
         return false
       }
       if (!isExplicitExternal) {
+        if (ctx.options.failOnWarn) {
+          throw new Error(`Implicit external ${id} detected. Either add to your project dependencies or add to \`externals\`. You can remove this protection by disabling \`failOnWarn\`.`)
+        }
         consola.warn(`Inlining implicit external ${id}`)
       }
       return isExplicitExternal

src/types.ts

@@ -66,6 +66,7 @@ export interface BuildOptions {
   devDependencies: string[]
   alias: { [find: string]: string },
   replace: { [find: string]: string },
+  failOnWarn?: boolean
   rollup: RollupBuildOptions
 }
 

src/validate.ts

@@ -9,7 +9,7 @@ import { BuildContext } from './types'
 export function validateDependencies (ctx: BuildContext) {
   const usedDependencies = new Set<string>()
   const unusedDependencies = new Set<string>(Object.keys(ctx.pkg.dependencies || {}))
-  const implicitDependnecies = new Set<string>()
+  const implicitDependencies = new Set<string>()
   for (const id of ctx.usedImports) {
     unusedDependencies.delete(id)
     usedDependencies.add(id)
@@ -26,14 +26,19 @@ export function validateDependencies (ctx: BuildContext) {
       !ctx.options.dependencies.includes(getpkg(id)) &&
       !ctx.options.peerDependencies.includes(getpkg(id))
     ) {
-      implicitDependnecies.add(id)
+      implicitDependencies.add(id)
     }
   }
   if (unusedDependencies.size) {
     consola.warn('Potential unused dependencies found:', Array.from(unusedDependencies).map(id => chalk.cyan(id)).join(', '))
   }
-  if (implicitDependnecies.size && !ctx.options.rollup.inlineDependencies) {
-    consola.warn('Potential implicit dependencies found:', Array.from(implicitDependnecies).map(id => chalk.cyan(id)).join(', '))
+  if (implicitDependencies.size && !ctx.options.rollup.inlineDependencies) {
+    consola.warn('Potential implicit dependencies found:', Array.from(implicitDependencies).map(id => chalk.cyan(id)).join(', '))
+  }
+  if (unusedDependencies.size || (implicitDependencies.size && !ctx.options.rollup.inlineDependencies)) {
+    if (ctx.options.failOnWarn) {
+      throw new Error('Failing build due to warnings. You can remove this protection by disabling `failOnWarn`.')
+    }
   }
 }