feat: add --target flag

Author: egoist

src/cli.ts

@@ -37,6 +37,7 @@ cli
   .option('--external <id>', 'Mark a module id as external', {
     type: []
   })
+  .option('-t, --target <target>', 'Output target', { default: 'node' })
   .option('-c, --config <file>', 'Use a custom config file')
   .option('--minimal', 'Generate minimal output whenever possible')
   .option('--banner', 'Add banner with pkg info to the bundle')
@@ -64,7 +65,8 @@ cli
           fileName: options.fileName,
           minify: options.minify,
           extractCSS: options.extractCss,
-          sourceMap: options.map
+          sourceMap: options.map,
+          target: options.target
         },
         bundleNodeModules: options.bundleNodeModules,
         env: options.env,

src/index.ts

@@ -209,7 +209,8 @@ export class Bundler {
       nodeResolvePlugin({
         rootDir: this.rootDir,
         bundleNodeModules,
-        externals: config.externals
+        externals: config.externals,
+        browser: config.output.target === 'browser'
       })
     )
 

src/plugins/node-resolve.ts

@@ -7,14 +7,16 @@ interface Options {
   rootDir: string
   bundleNodeModules?: boolean | string[]
   externals: NormalizedConfig['externals']
+  browser: boolean
 }
 
 export default (options: Options) => {
   const plugin = require('rollup-plugin-node-resolve')({
     extensions: ['.js', '.json', '.jsx', '.ts', '.tsx'],
     preferBuiltins: true,
     jsnext: true,
-    module: true
+    module: true,
+    browser: options.browser
   })
 
   return {

src/types.ts

@@ -76,6 +76,8 @@ export interface BabelPresetOptions {
   minimal?: boolean
 }
 
+export type OutputTarget = 'node' | 'browser'
+
 export interface ConfigOutput {
   /**
    * Output format(s). You can append `min` to the format to generate minified bundle.
@@ -127,6 +129,12 @@ export interface ConfigOutput {
    * @default `true` for minified bundle, `false` otherwise
    */
   sourceMap?: boolean
+  /**
+   * Output target
+   * @default `node`
+   * @cli `--target <target>`
+   */
+  target?: OutputTarget
 }
 
 export interface Config {

test/__snapshots__/index.test.ts.snap

@@ -256,6 +256,15 @@ module.exports = index;
 "
 `;
 
+exports[`target:browser: target:browser dist/index.js 1`] = `
+"'use strict';
+
+var foo = 'browser';
+
+module.exports = foo;
+"
+`;
+
 exports[`uglify: uglify dist/index.min.js 1`] = `
 "\\"use strict\\";Object.defineProperty(exports,\\"__esModule\\",{value:!0});const a=1;exports.a=1;
 "

test/fixtures/target/browser/foo-browser.js

@@ -0,0 +1 @@
+export default 'browser'

test/fixtures/target/browser/index.js

@@ -0,0 +1,3 @@
+import foo from './foo'
+
+export default foo

test/fixtures/target/browser/package.json

@@ -0,0 +1,5 @@
+{
+  "browser": {
+    "./foo.js": "./foo-browser.js"
+  }
+}

test/index.test.ts

@@ -230,3 +230,16 @@ snapshot(
     }
   }
 )
+
+snapshot(
+  {
+    title: 'target:browser',
+    input: 'index.js',
+    cwd: fixture('target/browser')
+  },
+  {
+    output: {
+      target: 'browser'
+    }
+  }
+)