fix: properly handle bundleNodeModules when it's an array

Author: egoist

bili.config.ts

@@ -8,7 +8,6 @@ const config: Config = {
   },
   bundleNodeModules: true,
   externals: [
-    ...Object.keys(require('./package').dependencies),
     'spawn-sync' // from cross-spawn which is from execa which is from term-size which is from boxen
   ],
   babel: {

src/plugins/node-resolve.ts

@@ -51,13 +51,13 @@ export default (options: Options) => {
         if (!options.bundleNodeModules) {
           return false
         }
-        const bundleNodeModules =
-          Array.isArray(options.bundleNodeModules) &&
-          options.bundleNodeModules.some(name =>
+        if (Array.isArray(options.bundleNodeModules)) {
+          const shouldBundle = options.bundleNodeModules.some(name =>
             id.includes(`/node_modules/${name}/`)
           )
-        if (!bundleNodeModules) {
-          return false
+          if (!shouldBundle) {
+            return false
+          }
         }
       }