dont export __esModule

Author: Rich-Harris

src/index.js

@@ -9,6 +9,11 @@ import { flatten, isReference } from './ast-utils.js';
 var firstpass = /\b(?:require|module|exports|global)\b/;
 var exportsPattern = /^(?:module\.)?exports(?:\.([a-zA-Z_$][a-zA-Z_$0-9]*))?$/;
 
+var blacklistedExports = {
+	__esModule: true,
+	default: true
+};
+
 function getName ( id ) {
 	const base = basename( id );
 	const ext = extname( base );
@@ -154,7 +159,10 @@ return module.exports;
 
 export default (${name} && typeof ${name} === 'object' && 'default' in ${name} ? ${name}['default'] : ${name});\n`;
 
-			outro += Object.keys( namedExports ).map( x => `export var ${x} = ${name}.${x};` ).join( '\n' );
+			outro += Object.keys( namedExports )
+				.filter( key => !blacklistedExports[ key ] )
+				.map( x => `export var ${x} = ${name}.${x};` )
+				.join( '\n' );
 
 			magicString.trim()
 				.prepend( importBlock + intro )

test/samples/__esModule/answer.js

@@ -0,0 +1,2 @@
+exports.__esModule = true;
+exports.answer = 42;

test/samples/__esModule/main.js

@@ -0,0 +1,2 @@
+exports.__esModule = true;
+exports.answer = 42;

test/test.js

@@ -172,53 +172,57 @@ describe( 'rollup-plugin-commonjs', function () {
 		});
 	});
 
-	it( 'identifies named exports from object literals', function () {
+	it( 'handles references to `global`', function () {
 		return rollup.rollup({
-			entry: 'samples/named-exports-from-object-literal/main.js',
+			entry: 'samples/global/main.js',
 			plugins: [ commonjs() ]
 		}).then( function ( bundle ) {
 			var generated = bundle.generate({
 				format: 'cjs'
 			});
 
-			var fn = new Function ( 'module', 'assert', generated.code );
-			fn( {}, assert );
+			var window = {};
+
+			var fn = new Function ( 'window', 'module', generated.code );
+			fn( window, {} );
+
+			assert.equal( window.foo, 'bar', generated.code );
 		});
 	});
 
-	it( 'handles references to `global`', function () {
+	it( 'handles transpiled CommonJS modules', function () {
 		return rollup.rollup({
-			entry: 'samples/global/main.js',
+			entry: 'samples/corejs/literal-with-default.js',
 			plugins: [ commonjs() ]
 		}).then( function ( bundle ) {
 			var generated = bundle.generate({
 				format: 'cjs'
 			});
 
-			var window = {};
+			var module = { exports: {} };
 
-			var fn = new Function ( 'window', 'module', generated.code );
-			fn( window, {} );
+			var fn = new Function ( 'module', 'exports', generated.code );
+			fn( module, module.exports );
 
-			assert.equal( window.foo, 'bar', generated.code );
+			assert.equal( module.exports, 'foobar', generated.code );
 		});
 	});
 
-	it( 'handles transpiled CommonJS modules', function () {
+	it( 'does not export __esModule', function () {
 		return rollup.rollup({
-			entry: 'samples/corejs/literal-with-default.js',
+			entry: 'samples/__esModule/main.js',
 			plugins: [ commonjs() ]
 		}).then( function ( bundle ) {
 			var generated = bundle.generate({
 				format: 'cjs'
 			});
 
-			var module = {};
+			var fn = new Function ( 'module', 'exports', generated.code );
+			var module = { exports: {} };
 
-			var fn = new Function ( 'module', generated.code );
-			fn( module );
+			fn( module, module.exports );
 
-			assert.equal( module.exports, 'foobar', generated.code );
+			assert.ok( !module.exports.__esModule );
 		});
 	});
 });