rollup 0.41.0 → 0.41.4

package/CHANGELOG.md

@@ -1,8 +1,26 @@
 # rollup changelog
 
+## 0.41.4
+
+* Fix cases of multiple `export * from 'external'` declarations ([#1252](https://github.com/rollup/rollup/issues/1252))
+* Fix 'TODO' error message ([#1257](https://github.com/rollup/rollup/issues/1257))
+
+## 0.41.3
+
+* Don't treat `this.foo` as possible namespace ([#1258](https://github.com/rollup/rollup/issues/1258))
+
+## 0.41.2
+
+* Optimize `namespace['foo']` references ([#1240](https://github.com/rollup/rollup/pull/1240))
+
+## 0.41.1
+
+* Include `new` expressions where callee is a class with side-effects ([#980](https://github.com/rollup/rollup/issues/980) [#1233](https://github.com/rollup/rollup/issues/1233))
+
 ## 0.41.0
 
 * Add `this.warn(...)` and `this.error(...)` methods to plugin `transform` hook contexts ([#1140](https://github.com/rollup/rollup/issues/1140))
+* `throw` always considered to be a side effect ([#1227](https://github.com/rollup/rollup/pull/1227))
 
 ## 0.40.2
 

package/bin/rollup

@@ -245,9 +245,9 @@ function isNumber (x) {
     return /^[-+]?(?:\d+(?:\.\d*)?|\.\d+)(e[-+]?\d+)?$/.test(x);
 }
 
-var help = "rollup version __VERSION__\n=====================================\n\nUsage: rollup [options] <entry file>\n\nBasic options:\n\n-v, --version            Show version number\n-h, --help               Show this help message\n-c, --config             Use this config file (if argument is used but value\n                           is unspecified, defaults to rollup.config.js)\n-w, --watch              Watch files in bundle and rebuild on changes\n-i, --input              Input (alternative to <entry file>)\n-o, --output <output>    Output (if absent, prints to stdout)\n-f, --format [es]       Type of output (amd, cjs, es, iife, umd)\n-e, --external           Comma-separate list of module IDs to exclude\n-g, --globals            Comma-separate list of `module ID:Global` pairs\n                            Any module IDs defined here are added to external\n-n, --name               Name for UMD export\n-u, --id                 ID for AMD module (default is anonymous)\n-m, --sourcemap          Generate sourcemap (`-m inline` for inline map)\n--no-strict              Don't emit a `\"use strict\";` in the generated modules.\n--no-indent              Don't indent result\n--environment <values>   Settings passed to config file (see example)\n--no-conflict            Generate a noConflict method for UMD globals\n--intro                  Content to insert at top of bundle (inside wrapper)\n--outro                  Content to insert at end of bundle (inside wrapper)\n--banner                 Content to insert at top of bundle (outside wrapper)\n--footer                 Content to insert at end of bundle (outside wrapper)\n\nExamples:\n\n# use settings in config file\nrollup -c\n\n# in config file, process.env.INCLUDE_DEPS === 'true'\n# and process.env.BUILD === 'production'\nrollup -c --environment INCLUDE_DEPS,BUILD:production\n\n# create CommonJS bundle.js from src/main.js\nrollup --format=cjs --output=bundle.js -- src/main.js\n\n# create self-executing IIFE using `window.jQuery`\n# and `window._` as external globals\nrollup -f iife --globals jquery:jQuery,lodash:_ \\\n  -i src/app.js -o build/app.js -m build/app.js.map\n\nNotes:\n\n* When piping to stdout, only inline sourcemaps are permitted\n\nFor more information visit https://github.com/rollup/rollup/wiki\n";
+var help = "rollup version __VERSION__\n=====================================\n\nUsage: rollup [options] <entry file>\n\nBasic options:\n\n-v, --version            Show version number\n-h, --help               Show this help message\n-c, --config             Use this config file (if argument is used but value\n                           is unspecified, defaults to rollup.config.js)\n-w, --watch              Watch files in bundle and rebuild on changes\n-i, --input              Input (alternative to <entry file>)\n-o, --output <output>    Output (if absent, prints to stdout)\n-f, --format [es]       Type of output (amd, cjs, es, iife, umd)\n-e, --external           Comma-separate list of module IDs to exclude\n-g, --globals            Comma-separate list of `module ID:Global` pairs\n                            Any module IDs defined here are added to external\n-n, --name               Name for UMD export\n-u, --id                 ID for AMD module (default is anonymous)\n-m, --sourcemap          Generate sourcemap (`-m inline` for inline map)\n--no-strict              Don't emit a `\"use strict\";` in the generated modules.\n--no-indent              Don't indent result\n--environment <values>   Settings passed to config file (see example)\n--no-conflict            Generate a noConflict method for UMD globals\n--silent                 Don't print warnings\n--intro                  Content to insert at top of bundle (inside wrapper)\n--outro                  Content to insert at end of bundle (inside wrapper)\n--banner                 Content to insert at top of bundle (outside wrapper)\n--footer                 Content to insert at end of bundle (outside wrapper)\n\nExamples:\n\n# use settings in config file\nrollup -c\n\n# in config file, process.env.INCLUDE_DEPS === 'true'\n# and process.env.BUILD === 'production'\nrollup -c --environment INCLUDE_DEPS,BUILD:production\n\n# create CommonJS bundle.js from src/main.js\nrollup --format=cjs --output=bundle.js -- src/main.js\n\n# create self-executing IIFE using `window.jQuery`\n# and `window._` as external globals\nrollup -f iife --globals jquery:jQuery,lodash:_ \\\n  -i src/app.js -o build/app.js -m build/app.js.map\n\nNotes:\n\n* When piping to stdout, only inline sourcemaps are permitted\n\nFor more information visit https://github.com/rollup/rollup/wiki\n";
 
-var version = "0.41.0";
+var version = "0.41.4";
 
 var path$1 = path__default;
 var Module = module$1;
@@ -747,6 +747,10 @@ function execute ( options, command ) {
 		external = ( optionsExternal || [] ).concat( commandExternal );
 	}
 
+	if ( command.silent ) {
+		options.onwarn = function () {};
+	}
+
 	if ( !options.onwarn ) {
 		var seen = new Set();
 

package/dist/rollup.browser.js

@@ -1,6 +1,6 @@
 /*
-	Rollup.js v0.41.0
-	Fri Jan 06 2017 17:01:19 GMT-0500 (EST) - commit e4682a3a476452597c7c69e19359f7e10b2275d0
+	Rollup.js v0.41.4
+	Sat Jan 14 2017 15:42:17 GMT-0500 (EST) - commit 80da8ecf3e377820a5fc7ca6fcff8a273339614c
 
 
 	https://github.com/rollup/rollup
@@ -6021,6 +6021,11 @@ function callHasEffects ( scope, callee, isNew ) {
 			if ( fnHasEffects( node, isNew && isES5Function( node ) ) ) return true;
 		}
 
+		else if ( /Class/.test( node.type ) ) {
+			// TODO find constructor (may belong to a superclass)
+			return true;
+		}
+
 		else if ( isReference( node ) ) {
 			var flattened = flatten( node );
 			var declaration = scope.findDeclaration( flattened.name );
@@ -6042,12 +6047,14 @@ function callHasEffects ( scope, callee, isNew ) {
 			}
 		}
 
-		else {
-			if ( !node.gatherPossibleValues ) {
-				throw new Error( 'TODO' );
-			}
+		else if ( node.gatherPossibleValues ) {
 			node.gatherPossibleValues( values );
 		}
+
+		else {
+			// probably an error in the user's code — err on side of caution
+			return true;
+		}
 	}
 
 	return false;
@@ -7014,13 +7021,24 @@ var LogicalExpression = (function (Node) {
 	return LogicalExpression;
 }(Node$1));
 
+var validProp = /^[a-zA-Z_$][a-zA-Z_$0-9]*$/;
+
 var Keypath = function Keypath ( node ) {
 	var this$1 = this;
 
 	this.parts = [];
 
 	while ( node.type === 'MemberExpression' ) {
-		this$1.parts.unshift( node.property );
+		var prop = node.property;
+
+		if ( node.computed  ) {
+			if ( prop.type !== 'Literal' || typeof prop.value !== 'string' || !validProp.test( prop.value ) ) {
+				this$1.computed = true;
+				return;
+			}
+		}
+
+		this$1.parts.unshift( prop );
 		node = node.object;
 	}
 
@@ -7042,21 +7060,21 @@ var MemberExpression = (function (Node) {
 		// if this resolves to a namespaced declaration, prepare
 		// to replace it
 		// TODO this code is a bit inefficient
-		if ( isReference( this ) ) { // TODO optimise namespace access like `foo['bar']` as well
-			var keypath = new Keypath( this );
+		var keypath = new Keypath( this );
 
+		if ( !keypath.computed && keypath.root.type === 'Identifier' ) {
 			var declaration = scope.findDeclaration( keypath.root.name );
 
 			while ( declaration.isNamespace && keypath.parts.length ) {
 				var exporterId = declaration.module.id;
 
 				var part = keypath.parts[0];
-				declaration = declaration.module.traceExport( part.name );
+				declaration = declaration.module.traceExport( part.name || part.value );
 
 				if ( !declaration ) {
 					this$1.module.warn({
 						code: 'MISSING_EXPORT',
-						message: ("'" + (part.name) + "' is not exported by '" + (relativeId( exporterId )) + "'"),
+						message: ("'" + (part.name || part.value) + "' is not exported by '" + (relativeId( exporterId )) + "'"),
 						url: "https://github.com/rollup/rollup/wiki/Troubleshooting#name-is-not-exported-by-module"
 					}, part.start );
 					this$1.replacement = 'undefined';
@@ -8142,6 +8160,12 @@ Module.prototype.trace = function trace ( name ) {
 Module.prototype.traceExport = function traceExport ( name ) {
 		var this$1 = this;
 
+	// export * from 'external'
+	if ( name[0] === '*' ) {
+		var module = this.bundle.moduleById.get( name.slice( 1 ) );
+		return module.traceExport( '*' );
+	}
+
 	// export { foo } from './other.js'
 	var reexportDeclaration = this.reexports[ name ];
 	if ( reexportDeclaration ) {
@@ -8169,8 +8193,8 @@ Module.prototype.traceExport = function traceExport ( name ) {
 	if ( name === 'default' ) return;
 
 	for ( var i = 0; i < this.exportAllModules.length; i += 1 ) {
-		var module = this$1.exportAllModules[i];
-		var declaration$2 = module.traceExport( name );
+		var module$1 = this$1.exportAllModules[i];
+		var declaration$2 = module$1.traceExport( name );
 
 		if ( declaration$2 ) return declaration$2;
 	}
@@ -8716,7 +8740,7 @@ function umd ( bundle, magicString, ref, options ) {
 		("(function() {\n\t\t\t\tvar current = " + (globalProp(options.moduleName)) + ";\n\t\t\t\tvar exports = factory(" + globalDeps + ");\n\t\t\t\t" + (globalProp(options.moduleName)) + " = exports;\n\t\t\t\texports.noConflict = function() { " + (globalProp(options.moduleName)) + " = current; return exports; };\n\t\t\t})()") : ("(" + defaultExport + "factory(" + globalDeps + "))");
 
 	var wrapperIntro =
-		("(function (global, factory) {\n\t\t\ttypeof exports === 'object' && typeof module !== 'undefined' ? " + cjsExport + "factory(" + (cjsDeps.join( ', ' )) + ") :\n\t\t\ttypeof define === 'function' && define.amd ? define(" + amdParams + "factory) :\n\t\t\t" + globalExport + ";\n\t\t}(this, (function (" + args + ") {" + useStrict + "\n\n\t\t").replace( /^\t\t/gm, '' ).replace( /^\t/gm, magicString.getIndentString() );
+		("(function (global, factory) {\n\t\t\ttypeof exports === 'object' && typeof module !== 'undefined' ? " + cjsExport + "factory(" + (cjsDeps.join( ', ' )) + ") :\n\t\t\ttypeof define === 'function' && define.amd ? define(" + amdParams + "factory) :\n\t\t\t" + globalExport + ";\n\t\t}(this, (function (" + args + ") {" + useStrict + "\n\n\t\t").replace( /^\t\t/gm, '' ).replace( /^\t/gm, indentString || '\t' );
 
 	// var foo__default = 'default' in foo ? foo['default'] : foo;
 	var interopBlock = getInteropBlock( bundle, options );
@@ -9848,7 +9872,7 @@ Bundle$$1.prototype.warn = function warn ( warning ) {
 	this.onwarn( warning );
 };
 
-var VERSION = '0.41.0';
+var VERSION = '0.41.4';
 
 var ALLOWED_KEYS = [
 	'acorn',