rollup 0.36.0 → 0.36.4

package/CHANGELOG.md

@@ -1,5 +1,27 @@
 # rollup changelog
 
+## 0.36.4
+
+* Only depend on program-level call expressions ([#977](https://github.com/rollup/rollup/issues/977))
+
+## 0.36.3
+
+* Add `legacy` option for IE8 support ([#989](https://github.com/rollup/rollup/pull/989))
+
+## 0.36.2
+
+* Insert semicolons where necessary to fix broken code ([#1004](https://github.com/rollup/rollup/issues/1004))
+* Include module ID and location when warning about top-level `this`  ([#1012](https://github.com/rollup/rollup/pull/1012))
+* More informative error for missing exports ([#1033](https://github.com/rollup/rollup/issues/1033))
+* `options.moduleContext` for per-module context overrides ([#1023](https://github.com/rollup/rollup/pull/1023))
+
+## 0.36.1
+
+* Include naked block statements ([#981](https://github.com/rollup/rollup/issues/981))
+* Correctly include falsy alternate statements in optimised if blocks ([#973](https://github.com/rollup/rollup/issues/973))
+* Prevent omission of default exports that are only used by the exporting module ([#967](https://github.com/rollup/rollup/pull/967))
+* Prevent warning on `auto` exports with ES output ([#966](https://github.com/rollup/rollup/pull/966))
+
 ## 0.36.0
 
 * `export { foo as default }` no longer creates a live binding ([#860](https://github.com/rollup/rollup/issues/860))

package/README.md

@@ -14,7 +14,7 @@
          alt="license">
   </a>
   <a href="https://david-dm.org/rollup/rollup">
-    <img src="https://david-dm.org/rollup/rollup.svg"
+    <img src="https://david-dm.org/rollup/rollup/status.svg"
          alt="dependency status">
   </a>
   <a href="https://codecov.io/github/rollup/rollup?branch=master">
@@ -36,6 +36,8 @@
 
 Rollup can be used via a [JavaScript API](https://github.com/rollup/rollup/wiki/JavaScript-API) or a [Command Line Interface](https://github.com/rollup/rollup/wiki/Command-Line-Interface). Install with `npm install -g rollup` and run `rollup --help` to get started.
 
+If the command line's not your jam, there's also a [step-by-step tutorial video series](https://code.lengstorf.com/learn-rollup-js/) (with accompanying written walkthrough).
+
 [Dive into the wiki](https://github.com/rollup/rollup/wiki) when you're ready to learn more about Rollup and ES6 modules.
 
 

package/bin/rollup

@@ -257,7 +257,7 @@ var minimist = interopDefault(index);
 
 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 version = "0.36.0";
+var version = "0.36.4";
 
 var index$1 = createCommonjsModule(function (module) {
 /*

package/dist/rollup.browser.js

@@ -1,6 +1,6 @@
 /*
-	Rollup.js v0.36.0
-	Sun Sep 18 2016 17:58:06 GMT-0400 (EDT) - commit 152afb9732975188c7f4bf716c143da549afd432
+	Rollup.js v0.36.4
+	Wed Nov 23 2016 09:01:44 GMT-0500 (EST) - commit fca14dbab102a9fa8efc7249287e2145583712ea
 
 
 	https://github.com/rollup/rollup
@@ -224,9 +224,7 @@ function mapSequence ( array, fn ) {
 	return promise.then( function () { return results; } );
 }
 
-function validateKeys ( object, allowedKeys ) {
-	var actualKeys = keys( object );
-
+function validateKeys ( actualKeys, allowedKeys ) {
 	var i = actualKeys.length;
 
 	while ( i-- ) {
@@ -5049,6 +5047,11 @@ function makeLegalIdentifier ( str ) {
 	return str;
 }
 
+function relativeId ( id ) {
+	if ( typeof process === 'undefined' ) return id;
+	return id.replace( process.cwd(), '' ).replace( /^[\/\\]/, '' );
+}
+
 // properties are for debugging purposes only
 var ARRAY    = { ARRAY:    true, toString: function () { return '[[ARRAY]]'; } };
 var NUMBER   = { NUMBER:   true, toString: function () { return '[[NUMBER]]'; } };
@@ -5093,7 +5096,7 @@ SyntheticNamespaceDeclaration.prototype.getName = function getName () {
 	return this.name;
 };
 
-SyntheticNamespaceDeclaration.prototype.renderBlock = function renderBlock ( es, indentString ) {
+SyntheticNamespaceDeclaration.prototype.renderBlock = function renderBlock ( es, legacy, indentString ) {
 		var this$1 = this;
 
 	var members = keys( this.originals ).map( function (name) {
@@ -5106,7 +5109,8 @@ SyntheticNamespaceDeclaration.prototype.renderBlock = function renderBlock ( es,
 		return ("" + indentString + name + ": " + (original.getName( es )));
 	});
 
-	return ((this.module.bundle.varOrConst) + " " + (this.getName( es )) + " = Object.freeze({\n" + (members.join( ',\n' )) + "\n});\n\n");
+	var callee = legacy ? "(Object.freeze || Object)" : "Object.freeze";
+	return ((this.module.bundle.varOrConst) + " " + (this.getName( es )) + " = " + callee + "({\n" + (members.join( ',\n' )) + "\n});\n\n");
 };
 
 var ExternalDeclaration = function ExternalDeclaration ( module, name ) {
@@ -5256,6 +5260,12 @@ Node$1.prototype.initialise = function initialise ( scope ) {
 	this.eachChild( function (child) { return child.initialise( this$1.scope || scope ); } );
 };
 
+Node$1.prototype.insertSemicolon = function insertSemicolon ( code ) {
+	if ( code.original[ this.end - 1 ] !== ';' ) {
+		code.insertLeft( this.end, ';' );
+	}
+};
+
 Node$1.prototype.locate = function locate () {
 	// useful for debugging
 	var location = getLocation( this.module.code, this.start );
@@ -5324,7 +5334,7 @@ Parameter.prototype.getName = function getName () {
 };
 
 var Scope = function Scope ( options ) {
-	options = options || {};
+	if ( options === void 0 ) options = {};
 
 	this.parent = options.parent;
 	this.isBlockScope = !!options.isBlockScope;
@@ -5699,6 +5709,16 @@ var BlockStatement = (function (Statement) {
 		}
 	};
 
+	BlockStatement.prototype.render = function render ( code, es ) {
+		if (this.body.length) {
+			for ( var node of this.body ) {
+				node.render( code, es );
+			}
+		} else {
+			Statement.prototype.render.call(this, code, es);
+		}
+	};
+
 	return BlockStatement;
 }(Statement));
 
@@ -5886,7 +5906,9 @@ var CallExpression = (function (Node) {
 	};
 
 	CallExpression.prototype.initialise = function initialise ( scope ) {
-		this.module.bundle.dependentExpressions.push( this );
+		if ( isProgramLevel( this ) ) {
+			this.module.bundle.dependentExpressions.push( this );
+		}
 		Node.prototype.initialise.call( this, scope );
 	};
 
@@ -5897,6 +5919,17 @@ var CallExpression = (function (Node) {
 	return CallExpression;
 }(Node$1));
 
+function isProgramLevel ( node ) {
+	do {
+		if ( node.type === 'Program' ) {
+			return true;
+		}
+		node = node.parent;
+	} while ( node && !/Function/.test( node.type ) );
+
+	return false;
+}
+
 // TODO is this basically identical to FunctionDeclaration?
 var ClassDeclaration = (function (Node) {
 	function ClassDeclaration () {
@@ -6153,7 +6186,7 @@ var ExportDefaultDeclaration = (function (Node) {
 		var treeshake = this.module.bundle.treeshake;
 		var name = this.getName( es );
 
-		if ( this.shouldInclude ) {
+		if ( this.shouldInclude || this.declaration.activated ) {
 			if ( this.activated ) {
 				if ( functionOrClassDeclaration.test( this.declaration.type ) ) {
 					if ( this.declaration.id ) {
@@ -6171,6 +6204,8 @@ var ExportDefaultDeclaration = (function (Node) {
 					} else {
 						code.overwrite( this.start, this.declaration.start, ((this.module.bundle.varOrConst) + " " + name + " = ") );
 					}
+
+					this.insertSemicolon( code );
 				}
 			} else {
 				// remove `var foo` from `var foo = bar()`, if `foo` is unused
@@ -6180,7 +6215,7 @@ var ExportDefaultDeclaration = (function (Node) {
 			Node.prototype.render.call( this, code, es );
 		} else {
 			if ( treeshake ) {
-				if ( functionOrClassDeclaration.test( this.declaration.type ) && !this.declaration.activated ) {
+				if ( functionOrClassDeclaration.test( this.declaration.type ) ) {
 					code.remove( this.leadingCommentStart || this.start, this.next || this.end );
 				} else {
 					var hasEffects = this.declaration.hasEffects( this.module.scope );
@@ -6290,11 +6325,16 @@ var ExportNamedDeclaration = (function (Node) {
 var ExpressionStatement = (function (Statement) {
 	function ExpressionStatement () {
 		Statement.apply(this, arguments);
-	}if ( Statement ) ExpressionStatement.__proto__ = Statement;
+	}
+
+	if ( Statement ) ExpressionStatement.__proto__ = Statement;
 	ExpressionStatement.prototype = Object.create( Statement && Statement.prototype );
 	ExpressionStatement.prototype.constructor = ExpressionStatement;
 
-	
+	ExpressionStatement.prototype.render = function render ( code, es ) {
+		Statement.prototype.render.call( this, code, es );
+		if ( this.shouldInclude ) this.insertSemicolon( code );
+	};
 
 	return ExpressionStatement;
 }(Statement));
@@ -6557,6 +6597,16 @@ var Identifier = (function (Node) {
 	return Identifier;
 }(Node$1));
 
+// Statement types which may contain if-statements as direct children.
+var statementsWithIfStatements = new Set([
+	'DoWhileStatement',
+	'ForInStatement',
+	'ForOfStatement',
+	'ForStatement',
+	'IfStatement',
+	'WhileStatement'
+]);
+
 // TODO DRY this out
 var IfStatement = (function (Statement) {
 	function IfStatement () {
@@ -6578,7 +6628,9 @@ var IfStatement = (function (Statement) {
 			else if ( this.testValue ) {
 				this.consequent.initialise( scope );
 				this.alternate = null;
-			} else {
+			}
+
+			else {
 				if ( this.alternate ) this.alternate.initialise( scope );
 				this.consequent = null;
 			}
@@ -6605,9 +6657,18 @@ var IfStatement = (function (Statement) {
 					code.remove( this.start, this.consequent.start );
 					code.remove( this.consequent.end, this.end );
 					this.consequent.render( code, es );
-				} else {
+				}
+
+				else {
 					code.remove( this.start, this.alternate ? this.alternate.start : this.next || this.end );
-					if ( this.alternate ) this.alternate.render( code, es );
+
+					if ( this.alternate ) {
+						this.alternate.render( code, es );
+					}
+
+					else if ( statementsWithIfStatements.has( this.parent.type ) ) {
+						code.insertRight( this.start, '{}' );
+					}
 				}
 			}
 		}
@@ -6835,6 +6896,8 @@ var TemplateLiteral = (function (Node) {
 	return TemplateLiteral;
 }(Node$1));
 
+var warning = "The 'this' keyword is equivalent to 'undefined' at the top level of an ES module, and has been rewritten. See https://github.com/rollup/rollup/wiki/Troubleshooting#this-is-undefined for more information";
+
 var ThisExpression = (function (Node) {
 	function ThisExpression () {
 		Node.apply(this, arguments);
@@ -6848,9 +6911,13 @@ var ThisExpression = (function (Node) {
 		var lexicalBoundary = scope.findLexicalBoundary();
 
 		if ( lexicalBoundary.isModuleScope ) {
-			this.alias = this.module.bundle.context;
+			this.alias = this.module.context;
 			if ( this.alias === 'undefined' ) {
-				this.module.bundle.onwarn( 'The `this` keyword is equivalent to `undefined` at the top level of an ES module, and has been rewritten' );
+				var ref = getLocation( this.module.code, this.start );
+				var line = ref.line;
+				var column = ref.column;
+				var detail = (relativeId( this.module.id )) + " (" + line + ":" + (column + 1) + ")"; // use one-based column number convention
+				this.module.bundle.onwarn( (detail + " " + warning) );
 			}
 		}
 	};
@@ -7081,7 +7148,7 @@ function getSeparator ( code, start ) {
 	return (";\n" + lineStart);
 }
 
-var forStatement = /^For(?:Of|In)Statement/;
+var forStatement = /^For(?:Of|In)?Statement/;
 
 var VariableDeclaration = (function (Node) {
 	function VariableDeclaration () {
@@ -7171,9 +7238,16 @@ var VariableDeclaration = (function (Node) {
 
 		if ( treeshake && empty ) {
 			code.remove( this.leadingCommentStart || this.start, this.next || this.end );
-		} else if ( this.end > c ) {
-			var hasSemicolon = code.original[ this.end - 1 ] === ';';
-			code.overwrite( c, this.end, hasSemicolon ? ';' : '' );
+		} else {
+			// always include a semi-colon (https://github.com/rollup/rollup/pull/1013),
+			// unless it's a var declaration in a loop head
+			var needsSemicolon = !forStatement.test( this.parent.type );
+
+			if ( this.end > c ) {
+				code.overwrite( c, this.end, needsSemicolon ? ';' : '' );
+			} else if ( needsSemicolon  ) {
+				this.insertSemicolon( code );
+			}
 		}
 	};
 
@@ -7390,6 +7464,7 @@ var Module = function Module (ref) {
 	this.bundle = bundle;
 	this.id = id;
 	this.excludeFromSourcemap = /\0/.test( id );
+	this.context = bundle.getModuleContext( id );
 
 	// all dependencies
 	this.sources = [];
@@ -7538,11 +7613,11 @@ Module.prototype.addImport = function addImport ( node ) {
 	if ( !~this.sources.indexOf( source ) ) this.sources.push( source );
 
 	node.specifiers.forEach( function (specifier) {
-			var localName = specifier.local.name;
+		var localName = specifier.local.name;
 
 		if ( this$1.imports[ localName ] ) {
 			var err = new Error( ("Duplicated import '" + localName + "'") );
-			err.file = this$1.id;
+				err.file = this$1.id;
 			err.loc = getLocation( this$1.code, specifier.start );
 			throw err;
 		}
@@ -7629,18 +7704,18 @@ Module.prototype.findParent = function findParent () {
 
 Module.prototype.findScope = function findScope () {
 	return this.scope;
-	};
+};
 
-	Module.prototype.getExports = function getExports () {
-		var exports = blank();
+Module.prototype.getExports = function getExports () {
+	var exports = blank();
 
-		keys( this.exports ).forEach( function (name) {
+	keys( this.exports ).forEach( function (name) {
 		exports[ name ] = true;
 	});
 
 	keys( this.reexports ).forEach( function (name) {
-		exports[ name ] = true;
-	});
+			exports[ name ] = true;
+		});
 
 	this.exportAllModules.forEach( function (module) {
 		if ( module.isExternal ) return; // TODO
@@ -7661,7 +7736,7 @@ Module.prototype.namespace = function namespace () {
 	return this.declarations['*'];
 };
 
-Module.prototype.render = function render ( es ) {
+Module.prototype.render = function render ( es, legacy ) {
 	var magicString = this.magicString.clone();
 
 	for ( var node of this.ast.body ) {
@@ -7669,7 +7744,7 @@ Module.prototype.render = function render ( es ) {
 	}
 
 	if ( this.namespace().needsNamespaceBlock ) {
-		magicString.append( '\n\n' + this.namespace().renderBlock( es, '\t' ) ); // TODO use correct indentation
+		magicString.append( '\n\n' + this.namespace().renderBlock( es, legacy, '\t' ) ); // TODO use correct indentation
 	}
 
 	return magicString.trim();
@@ -7713,7 +7788,7 @@ Module.prototype.trace = function trace ( name ) {
 
 		var declaration = otherModule.traceExport( importDeclaration.name );
 
-		if ( !declaration ) throw new Error( ("Module " + (otherModule.id) + " does not export " + (importDeclaration.name) + " (imported by " + (this.id) + ")") );
+		if ( !declaration ) throw new Error( ("'" + (importDeclaration.name) + "' is not exported by " + (relativeId( otherModule.id )) + " (imported by " + (relativeId( this.id )) + "). For help fixing this error see https://github.com/rollup/rollup/wiki/Troubleshooting#name-is-not-exported-by-module") );
 		return declaration;
 	}
 
@@ -8216,7 +8291,11 @@ function badExports ( option, keys ) {
 	throw new Error( ("'" + option + "' was specified for options.exports, but entry module has following exports: " + (keys.join(', '))) );
 }
 
-function getExportMode ( bundle, exportMode, moduleName ) {
+function getExportMode ( bundle, ref ) {
+	var exportMode = ref.exports;
+	var moduleName = ref.moduleName;
+	var format = ref.format;
+
 	var exportKeys = keys( bundle.entryModule.exports )
 		.concat( keys( bundle.entryModule.reexports ) )
 		.concat( bundle.entryModule.exportAllSources ); // not keys, but makes our job easier this way
@@ -8235,7 +8314,7 @@ function getExportMode ( bundle, exportMode, moduleName ) {
 		} else if ( exportKeys.length === 1 && exportKeys[0] === 'default' ) {
 			exportMode = 'default';
 		} else {
-			if ( bundle.entryModule.exports.default ) {
+			if ( bundle.entryModule.exports.default && format !== 'es') {
 				bundle.onwarn( ("Using named and default exports together. Consumers of your bundle will have to use " + (moduleName || 'bundle') + "['default'] to access the default export, which may not be what you want. Use `exports: 'named'` to disable this warning. See https://github.com/rollup/rollup/wiki/JavaScript-API#exports for more information") );
 			}
 			exportMode = 'named';
@@ -8573,11 +8652,10 @@ var Bundle = function Bundle ( options ) {
 
 	this.plugins = ensureArray( options.plugins );
 
-	this.plugins.forEach( function (plugin) {
-		if ( plugin.options ) {
-			options = plugin.options( options ) || options;
-		}
-	});
+	options = this.plugins.reduce( function ( acc, plugin ) {
+		if ( plugin.options ) return plugin.options( acc ) || acc;
+		return acc;
+	}, options);
 
 	this.entry = options.entry;
 	this.entryId = null;
@@ -8615,6 +8693,18 @@ var Bundle = function Bundle ( options ) {
 
 	this.context = String( options.context );
 
+	if ( typeof options.moduleContext === 'function' ) {
+		this.getModuleContext = function (id) { return options.moduleContext( id ) || this$1.context; };
+	} else if ( typeof options.moduleContext === 'object' ) {
+		var moduleContext = new Map();
+		Object.keys( options.moduleContext ).forEach( function (key) {
+			moduleContext.set( resolve( key ), options.moduleContext[ key ] );
+		});
+		this.getModuleContext = function (id) { return moduleContext.get( id ) || this$1.context; };
+	} else {
+		this.getModuleContext = function () { return this$1.context; };
+	}
+
 	if ( typeof options.external === 'function' ) {
 		this.isExternal = options.external;
 	} else {
@@ -8625,6 +8715,7 @@ var Bundle = function Bundle ( options ) {
 	this.onwarn = options.onwarn || makeOnwarn();
 
 	this.varOrConst = options.preferConst ? 'const' : 'var';
+	this.legacy = options.legacy;
 	this.acornOptions = options.acorn || {};
 
 	this.dependentExpressions = [];
@@ -8820,7 +8911,7 @@ Bundle.prototype.fetchModule = function fetchModule ( id, importer ) {
 			});
 
 			this$1.modules.push( module );
-				this$1.moduleById.set( id, module );
+			this$1.moduleById.set( id, module );
 
 			return this$1.fetchAllDependencies( module ).then( function () {
 				keys( module.exports ).forEach( function (name) {
@@ -8895,6 +8986,7 @@ Bundle.prototype.getPathRelativeToEntryDirname = function getPathRelativeToEntry
 };
 
 Bundle.prototype.render = function render ( options ) {
+		var this$1 = this;
 		if ( options === void 0 ) options = {};
 
 	if ( options.format === 'es6' ) {
@@ -8905,7 +8997,7 @@ Bundle.prototype.render = function render ( options ) {
 	var format = options.format || 'es';
 
 	// Determine export mode - 'default', 'named', 'none'
-	var exportMode = getExportMode( this, options.exports, options.moduleName );
+	var exportMode = getExportMode( this, options );
 
 	var magicString = new Bundle$1({ separator: '\n\n' });
 	var usedModules = [];
@@ -8913,7 +9005,7 @@ Bundle.prototype.render = function render ( options ) {
 	timeStart( 'render modules' );
 
 	this.orderedModules.forEach( function (module) {
-		var source = module.render( format === 'es' );
+		var source = module.render( format === 'es', this$1.legacy );
 
 		if ( source.toString().length ) {
 			magicString.addSource( source );
@@ -8990,7 +9082,7 @@ Bundle.prototype.render = function render ( options ) {
 	return { code: code, map: map };
 };
 
-Bundle.prototype.sort = function sort () {
+	Bundle.prototype.sort = function sort () {
 		var this$1 = this;
 
 	var hasCycles;
@@ -9058,7 +9150,7 @@ Bundle.prototype.sort = function sort () {
 						visited[ module.id ] = true;
 						for ( var i = 0; i < module.dependencies.length; i += 1 ) {
 							var dependency = module.dependencies[i];
-								if ( !visited[ dependency.id ] && findParent( dependency ) ) return true;
+							if ( !visited[ dependency.id ] && findParent( dependency ) ) return true;
 						}
 					};
 
@@ -9077,7 +9169,7 @@ Bundle.prototype.sort = function sort () {
 	return ordered;
 };
 
-var VERSION = '0.36.0';
+var VERSION = '0.36.4';
 
 var ALLOWED_KEYS = [
 	'acorn',
@@ -9094,6 +9186,8 @@ var ALLOWED_KEYS = [
 	'indent',
 	'interop',
 	'intro',
+	'legacy',
+	'moduleContext',
 	'moduleId',
 	'moduleName',
 	'noConflict',
@@ -9109,20 +9203,24 @@ var ALLOWED_KEYS = [
 	'useStrict'
 ];
 
-function rollup ( options ) {
+function checkOptions ( options ) {
 	if ( !options || !options.entry ) {
-		return Promise.reject( new Error( 'You must supply options.entry to rollup' ) );
+		return new Error( 'You must supply options.entry to rollup' );
 	}
 
 	if ( options.transform || options.load || options.resolveId || options.resolveExternal ) {
-		return Promise.reject( new Error( 'The `transform`, `load`, `resolveId` and `resolveExternal` options are deprecated in favour of a unified plugin API. See https://github.com/rollup/rollup/wiki/Plugins for details' ) );
+		return new Error( 'The `transform`, `load`, `resolveId` and `resolveExternal` options are deprecated in favour of a unified plugin API. See https://github.com/rollup/rollup/wiki/Plugins for details' );
 	}
 
-	var error = validateKeys( options, ALLOWED_KEYS );
+	var error = validateKeys( keys(options), ALLOWED_KEYS );
+	if ( error ) return error;
 
-	if ( error ) {
-		return Promise.reject( error );
-	}
+	return null;
+}
+
+function rollup ( options ) {
+	var error = checkOptions ( options );
+	if ( error ) return Promise.reject( error );
 
 	var bundle = new Bundle( options );