v5.6.16

- Don't hoist constant elements to the same function as their original paths function parent.
 - Push each violation paths ancestry to the breakOnScopePaths collection to avoid constant hoisting to nested paths.

.gitignore

@@ -15,3 +15,4 @@ test/core/tmp
 /packages/babel-runtime/helpers/*.js
 /packages/babel-runtime/regenerator/*.js
 /lib
+_babel.github.io

.npmignore

@@ -13,3 +13,4 @@
 /vendor
 /packages
 /src
+_babel.github.io

.travis.yml

@@ -1,14 +1,34 @@
-#sudo: false
 language: node_js
 cache:
   directories:
-    - node_modules
+  - node_modules
 node_js:
-  - "0.12"
-  - "iojs"
+- "0.12"
+- iojs
 
-before_script: "npm install -g codeclimate-test-reporter"
-script: "make test-travis"
+before_script: ./tools/setup-git.sh
+script: make test-travis
+after_success: ./tools/merge-development-with-master.sh
 
 notifications:
   slack: babeljs:5Wy4QX13KVkGy9CnU0rmvgeK
+
+before_deploy:
+- make prepublish
+
+deploy:
+  provider: npm
+  email: sebmck@gmail.com
+  skip_cleanup: true
+  api_key:
+    secure: Q/pZStwherdYPCqCa0aUuiEktLcx6ccBxieyH8j9IXdDgty0ydmnajZfUnlZuCjN13XI9esM44nSJFTWZvntTryDQAQm37c63VXhAEnw/qrAINI06yt0gLBTT69/fKvIAkH8l48nmW32ZS2dse3rHRPZF1CwyQLC/pdMip8I4sM=
+  on:
+    branch: master
+    tags: true
+    repo: babel/babel
+
+after_deploy:
+- make publish-cli
+- make publish-runtime
+- make build-website
+- make comment-issues

CHANGELOG.md

@@ -9,10 +9,20 @@
 > - [Internal]
 > - [Polish]
 
-_Note: Gaps between patch versions are faulty/broken releases._
+_Note: Gaps between patch versions are faulty, broken or test releases._
 
 See [CHANGELOG - 6to5](CHANGELOG-6to5.md) for the pre-4.0.0 version changelog.
 
+## 5.6.12
+
+ * **Bug Fix**
+  * Fix finding parent for top-level shadowed functions.
+
+## 5.6.11
+
+ ** **Internal**
+  * Merge `es6.parameters.rest` and `es6.parameters.default` transformers. See commit [c0fd4c1f9e0b18231f585c4fa793e4cb0e01aed1](https://github.com/babel/babel/commit/c0fd4c1f9e0b18231f585c4fa793e4cb0e01aed1) for more info.
+
 ## 5.6.10
 
  * **Bug Fix**

CONTRIBUTING.md

@@ -143,8 +143,11 @@ your [`$PATH`](http://unix.stackexchange.com/questions/26047/how-to-correctly-ad
 
 + [trim-right](http://ghub.io/trim-right) Trims the rightside whitespace.
 
-+ [user-home](http://ghub.io/user-home) Gets the users home directory. This is used to resolve the babel-node/babel/register cache.
++ [path-exists](https://www.npmjs.com/package/path-exists) Checks if a path exists. (replaces the deprecated `fs.exists` methods)
 
++ [home-or-tmp](https://github.com/sindresorhus/home-or-tmp) Gets the user home directory with fallback to the system temporary directory. This is used to resolve the babel-node/babel/register cache.
+
++ [resolve](https://www.npmjs.com/package/resolve) Implements the [`require.resolve()` algorithm](http://nodejs.org/docs/v0.12.0/api/all.html#all_require_resolve) such that we can `require.resolve()` on behalf of a file asynchronously and synchronously.
 
 #### Code Standards
 

Makefile

@@ -9,7 +9,7 @@ BROWSERIFY_IGNORE = -i esprima-fb
 
 export NODE_ENV = test
 
-.PHONY: clean test test-cov test-clean test-travis test-simple test-all test-browser test-parser publish build bootstrap publish-core publish-runtime build-core watch-core build-core-test clean-core
+.PHONY: clean test test-cov test-clean test-travis test-simple test-all test-browser test-parser publish build bootstrap publish-core publish-runtime build-website build-core watch-core build-core-test clean-core prepublish
 
 build-core: clean-core
 	node $(BABEL_CMD) src --out-dir lib --copy-files
@@ -81,34 +81,20 @@ test-browser:
 
 publish: lint
 	git pull --rebase
-
 	make test
-
 	read -p "Version: " version; \
 	npm version $$version --message "v%s"
-
-	make build
-
-	cp dist/browser.js browser.js
-	cp dist/browser.min.js browser.min.js
-
-	cp dist/polyfill.js browser-polyfill.js
-	cp dist/polyfill.min.js browser-polyfill.min.js
-
-	cp dist/external-helpers.js external-helpers.js
-	cp dist/external-helpers.min.js external-helpers.min.js
-
-	node tools/cache-templates
-	test -f templates.json
-
-	npm publish
-
 	git push --follow-tags
 
-	make publish-cli
-	make publish-runtime
-
-	rm -rf templates.json browser.js browser.min.js browser-polyfill.js browser-polyfill.min.js external-helpers.js external-helpers.min.js
+prepublish: build
+	cp dist/browser.js browser.js
+	cp dist/browser.min.js browser.min.js
+	cp dist/polyfill.js browser-polyfill.js
+	cp dist/polyfill.min.js browser-polyfill.min.js
+	cp dist/external-helpers.js external-helpers.js
+	cp dist/external-helpers.min.js external-helpers.min.js
+	node tools/cache-templates
+	test -f templates.json
 
 publish-runtime:
 	cd packages; \
@@ -117,10 +103,13 @@ publish-runtime:
 	npm publish
 
 publish-cli:
-	cd packages; \
-	node build-cli.js; \
-	cd babel-cli; \
-	npm publish
+	@./tools/publish-cli.sh
+
+build-website:
+	@./tools/build-website.sh
+
+comment-issues:
+	@./tools/comment-issues.sh
 
 bootstrap:
 	npm list --global --depth 1 babel >/dev/null 2>&1 && npm uninstall -g babel || true

README.md

@@ -5,15 +5,24 @@
 </p>
 
 <p align="center">
-  <strong>Babel</strong> is a compiler for writing next generation JavaScript.
+  The compiler for writing next generation JavaScript.
 </p>
 
 <p align="center">
-  For questions and support please visit the <a href="https://babel-slack.herokuapp.com">slack channel</a> or <a href="http://stackoverflow.com/questions/tagged/babeljs">StackOverflow</a>. The Babel issue tracker is <strong>exclusively</strong> for bug reports and feature requests.
+  <a href="https://travis-ci.org/babel/babel"><img alt="Build Status" src="https://img.shields.io/travis/babel/babel.svg?style=flat"></a>
+  <a href="http://badge.fury.io/js/babel-core"><img alt="npm version" src="https://badge.fury.io/js/babel-core.svg"></a>
+  <a href="https://npmjs.org/package/babel-core"><img alt="Downloads" src="http://img.shields.io/npm/dm/babel-core.svg"></a>
 </p>
 
 <p align="center">
-  Issues without instructions to reproduce <strong>will be immediately closed<strong>.
+  <a href="http://issuestats.com/github/babel/babel"><img alt="Issue Stats" src="http://issuestats.com/github/babel/babel/badge/pr?style=flat"></a>
+  <a href="http://issuestats.com/github/babel/babel"><img alt="Issue Stats" src="http://issuestats.com/github/babel/babel/badge/issue?style=flat"></a>
+</p>
+
+----
+
+<p align="center">
+  For questions and support please visit the <a href="https://babel-slack.herokuapp.com">Slack community</a> or <a href="http://stackoverflow.com/questions/tagged/babeljs">StackOverflow</a>. The Babel issue tracker is <strong>exclusively</strong> for bug reports and feature requests.
 </p>
 
 <p align="center">

doc/node-props.md

@@ -1,5 +1,5 @@
 # Properties of nodes
-These are properties babel stores in AST node objects for internal use, as opposed to properties that are part of the AST spec (ESTree at the time of this writing).
+These are properties babel stores in AST node objects for internal use, as opposed to properties that are part of the AST spec ([ESTree](https://github.com/estree/estree) at the time of this writing).
 
 ## `_blockHoist`
 `node._blockHoist != null` triggers the [block-hoist transformer](/src/babel/transformation/transformers/internal/block-hoist.js). Value should be `true` or an integer in the range `0..3`. `true` is equivalent to `2`. The value indicates whether the node should be hoisted and to what degree. See the source code for more detailed information.
@@ -8,4 +8,4 @@ These are properties babel stores in AST node objects for internal use, as oppos
 Stores a representation of a node's position in the tree and relationship to other nodes.
 
 ## `shadow`
-A truthy value on a function node triggers the [shadow-functions transformer](/src/babel/transformation/transformers/internal/shadow-functions.js), which transforms the node so that it references (or inherits) `arguments` and `this` from the parent scope. It is invoked for arrow functions, for example.
+A truthy value on a function node triggers the [shadow-functions transformer](/src/babel/transformation/transformers/internal/shadow-functions.js), which transforms the node so that it references (or inherits) `arguments` and the `this` context from the parent scope. It is invoked for arrow functions, for example.

package.json

@@ -1,7 +1,7 @@
 {
   "name": "babel-core",
   "description": "A compiler for writing next generation JavaScript",
-  "version": "5.6.10",
+  "version": "5.6.16",
   "author": "Sebastian McKenzie <sebmck@gmail.com>",
   "homepage": "https://babeljs.io/",
   "license": "MIT",
@@ -10,17 +10,17 @@
     "./lib/babel/api/register/node.js": "./lib/babel/api/register/browser.js"
   },
   "keywords": [
-    "harmony",
+    "6to5",
+    "babel",
     "classes",
-    "modules",
-    "let",
     "const",
-    "var",
     "es6",
+    "harmony",
+    "let",
+    "modules",
     "transpile",
     "transpiler",
-    "6to5",
-    "babel"
+    "var"
   ],
   "scripts": {
     "bench": "make bench",
@@ -63,7 +63,8 @@
     "path-exists": "^1.0.0",
     "path-is-absolute": "^1.0.0",
     "private": "^0.1.6",
-    "regenerator": "0.8.31",
+    "recast": "0.10.16",
+    "regenerator": "0.8.32",
     "regexpu": "^1.1.2",
     "repeating": "^1.1.2",
     "resolve": "^1.1.6",
@@ -76,11 +77,11 @@
     "trim-right": "^1.0.0"
   },
   "devDependencies": {
-    "babel": "5.6.7",
+    "babel": "5.6.10",
+    "babel-eslint": "^3.1.19",
     "browserify": "^9.0.8",
     "chai": "^2.2.0",
     "eslint": "^0.21.2",
-    "babel-eslint": "^3.1.9",
     "esvalid": "^1.1.0",
     "istanbul": "^0.3.5",
     "matcha": "^0.6.0",

packages/babel-cli/bin/_babel-node

@@ -41,6 +41,7 @@ babel.register({
 //
 
 var _eval = function (code, filename) {
+  code = code.trim();
   if (!code) return undefined;
 
   code = babel.transform(code, {

packages/babel-cli/bin/babel/dir.js

@@ -28,7 +28,7 @@ module.exports = function (commander, filenames) {
 
     outputFileSync(dest, data.code);
 
-    console.log(src + " -> " + dest);
+    util.log(src + " -> " + dest);
   };
 
   var handleFile = function (src, filename) {

packages/babel-cli/bin/babel/file.js

@@ -135,7 +135,7 @@ module.exports = function (commander, filenames, opts) {
         ignoreInitial: true
       }).on("all", function (type, filename) {
         if (type === "add" || type === "change") {
-          console.log(type, filename);
+          util.log(type + " " + filename);
           try {
             walk();
           } catch (err) {

packages/babel-cli/bin/babel/index.js

@@ -45,6 +45,7 @@ commander.option("-w, --watch", "Recompile files on changes");
 commander.option("-o, --out-file [out]", "Compile all input files into a single file");
 commander.option("-d, --out-dir [out]", "Compile an input directory of modules into an output directory");
 commander.option("-D, --copy-files", "When compiling a directory copy over non-compilable files");
+commander.option("-q, --quiet", "Don't log anything");
 
 commander.on("--help", function () {
   var outKeys = function (title, obj) {
@@ -67,7 +68,7 @@ commander.on("--help", function () {
 });
 
 var pkg = require("../../package.json");
-commander.version(pkg.version);
+commander.version(pkg.version + " (babel-core " + require("babel-core").version + ")");
 commander.usage("[options] <files ...>");
 commander.parse(process.argv);
 

packages/babel-cli/bin/babel/util.js

@@ -25,6 +25,10 @@ exports.addSourceMappingUrl = function (code, loc) {
   return code + "\n//# sourceMappingURL=" + path.basename(loc);
 };
 
+exports.log = function (msg) {
+  if (!commander.quiet) console.log(msg);
+};
+
 exports.transform = function (filename, code, opts) {
   opts = _.defaults(opts || {}, index.opts);
   opts.filename = filename;

packages/babel-cli/package.json

@@ -1,14 +1,12 @@
 {
   "name": "babel",
   "description": "Turn ES6 code into readable vanilla ES5 with source maps",
-  "version": "5.6.9",
   "author": "Sebastian McKenzie <sebmck@gmail.com>",
   "homepage": "https://babeljs.io/",
   "license": "MIT",
   "repository": "babel/babel",
   "preferGlobal": true,
   "dependencies": {
-    "babel-core": "^5.6.9",
     "chokidar": "^1.0.0",
     "commander": "^2.6.0",
     "convert-source-map": "^1.1.0",
@@ -27,4 +25,4 @@
     "babel-external-helpers": "./bin/babel-external-helpers",
     "babel-plugin": "./bin/babel-plugin/index.js"
   }
-}
+}

packages/babel-runtime/package.json

@@ -1,11 +1,10 @@
 {
   "name": "babel-runtime",
   "description": "babel selfContained runtime",
-  "version": "5.6.9",
   "license": "MIT",
   "repository": "babel/babel",
   "author": "Sebastian McKenzie <sebmck@gmail.com>",
   "dependencies": {
     "core-js": "^0.9.0"
   }
-}
+}

packages/build-runtime.js

@@ -73,7 +73,6 @@ each(File.helpers, function (helperName) {
 writeFile("regenerator/index.js", readFile("regenerator/runtime-module", true));
 writeFile("regenerator/runtime.js", selfContainify(readFile("regenerator/runtime")));
 
-//
 
 var coreDefinitions = require("babel-plugin-runtime/lib/definitions");
 
@@ -93,6 +92,5 @@ each(paths, function (path) {
   writeFile("core-js/" + path + ".js", defaultify('require("core-js/library/fn/' + path + '")'));
 });
 
-//
 
 updatePackage();

src/acorn/plugins/flow.js

@@ -1,4 +1,3 @@
-Error.stackTraceLimit = Infinity;
 var acorn = require("../src/index")
 
 var pp = acorn.Parser.prototype
@@ -16,6 +15,15 @@ pp.expectRelational = function (op) {
   }
 }
 
+pp.flow_parseTypeInitialiser = function (tok) {
+  var oldInType = this.inType
+  this.inType = true
+  this.expect(tok || tt.colon)
+  var type = this.flow_parseType()
+  this.inType = oldInType
+  return type;
+}
+
 pp.flow_parseDeclareClass = function (node) {
   this.next()
   this.flow_parseInterfaceish(node, true)
@@ -41,12 +49,7 @@ pp.flow_parseDeclareFunction = function (node) {
   typeNode.params = tmp.params
   typeNode.rest = tmp.rest
   this.expect(tt.parenR)
-
-  var oldInType = this.inType
-  this.inType = true
-  this.expect(tt.colon)
-  typeNode.returnType = this.flow_parseType()
-  this.inType = oldInType
+  typeNode.returnType = this.flow_parseTypeInitialiser()
 
   typeContainer.typeAnnotation = this.finishNode(typeNode, "FunctionTypeAnnotation")
   id.typeAnnotation = this.finishNode(typeContainer, "TypeAnnotation")
@@ -157,15 +160,7 @@ pp.flow_parseTypeAlias = function (node) {
     node.typeParameters = null
   }
 
-  var oldInType = this.inType;
-  this.inType = true;
-
-  this.expect(tt.eq)
-
-  node.right = this.flow_parseType()
-
-  this.inType = oldInType;
-
+  node.right = this.flow_parseTypeInitialiser(tt.eq)
   this.semicolon()
 
   return this.finishNode(node, "TypeAlias")
@@ -218,11 +213,9 @@ pp.flow_parseObjectTypeIndexer = function (node, isStatic) {
 
   this.expect(tt.bracketL)
   node.id = this.flow_parseObjectPropertyKey()
-  this.expect(tt.colon)
-  node.key = this.flow_parseType()
+  node.key = this.flow_parseTypeInitialiser()
   this.expect(tt.bracketR)
-  this.expect(tt.colon)
-  node.value = this.flow_parseType()
+  node.value = this.flow_parseTypeInitialiser()
 
   this.flow_objectTypeSemicolon()
   return this.finishNode(node, "ObjectTypeIndexer")
@@ -249,8 +242,7 @@ pp.flow_parseObjectTypeMethodish = function (node) {
     node.rest = this.flow_parseFunctionTypeParam()
   }
   this.expect(tt.parenR)
-  this.expect(tt.colon)
-  node.returnType = this.flow_parseType()
+  node.returnType = this.flow_parseTypeInitialiser()
 
   return this.finishNode(node, "FunctionTypeAnnotation")
 }
@@ -269,7 +261,7 @@ pp.flow_parseObjectTypeCallProperty = function (node, isStatic) {
   var valueNode = this.startNode()
   node.static = isStatic
   node.value = this.flow_parseObjectTypeMethodish(valueNode)
-    this.flow_objectTypeSemicolon()
+  this.flow_objectTypeSemicolon()
   return this.finishNode(node, "ObjectTypeCallProperty")
 }
 
@@ -314,9 +306,8 @@ pp.flow_parseObjectType = function (allowStatic) {
         if (this.eat(tt.question)) {
           optional = true
         }
-        this.expect(tt.colon)
         node.key = propertyKey
-        node.value = this.flow_parseType()
+        node.value = this.flow_parseTypeInitialiser()
         node.optional = optional
         node.static = isStatic
         this.flow_objectTypeSemicolon()
@@ -384,9 +375,8 @@ pp.flow_parseFunctionTypeParam = function () {
   if (this.eat(tt.question)) {
     optional = true
   }
-  this.expect(tt.colon)
   node.optional = optional
-  node.typeAnnotation = this.flow_parseType()
+  node.typeAnnotation = this.flow_parseTypeInitialiser()
   return this.finishNode(node, "FunctionTypeParam")
 }
 
@@ -583,13 +573,7 @@ pp.flow_parseType = function () {
 
 pp.flow_parseTypeAnnotation = function () {
   var node = this.startNode()
-
-  var oldInType = this.inType
-  this.inType = true
-  this.expect(tt.colon)
-  node.typeAnnotation = this.flow_parseType()
-  this.inType = oldInType
-
+  node.typeAnnotation = this.flow_parseTypeInitialiser()
   return this.finishNode(node, "TypeAnnotation")
 }
 

src/babel/generation/index.js

@@ -57,34 +57,16 @@ class CodeGenerator {
     return format;
   }
 
-  static findCommonStringDelimiter(code, tokens) {
-    var occurences = {
-      single: 0,
-      double: 0
-    };
-
-    var checked = 0;
-
-    for (var i = 0; i < tokens.length; i++) {
-      var token = tokens[i];
-      if (token.type.label !== "string") continue;
-
-      var raw = code.slice(token.start, token.end);
-      if (raw[0] === "'") {
-        occurences.single++;
-      } else {
-        occurences.double++;
-      }
-
-      checked++;
-      if (checked >= 3) break;
+  static findCommonStringDelimiter(code, tokens, occurences = {"'": 0, "\"": 0}) {
+    if (tokens.length === 0 || occurences["'"] + occurences["\""] >= 3) {
+      return occurences["'"] > occurences["\""] ? "single" : "double";
     }
 
-    if (occurences.single > occurences.double) {
-      return "single";
-    } else {
-      return "double";
+    if (tokens[0].type.label === "string") {
+      occurences[code[tokens[0].start]]++;
     }
+
+    return this.findCommonStringDelimiter(code, tokens.slice(1), occurences);
   }
 
   static generators = {

src/babel/messages.js

@@ -1,7 +1,7 @@
 import * as util from  "util";
 
 export const MESSAGES = {
-  tailCallReassignmentDeopt: "Function reference has been reassigned so it's probably be dereferenced so we can't optimise this with confidence",
+  tailCallReassignmentDeopt: "Function reference has been reassigned, so it will probably be dereferenced, therefore we can't optimise this with confidence",
   JSXNamespacedTags: "Namespace tags are not supported. ReactJSX is not XML.",
   classesIllegalBareSuper: "Illegal use of bare super",
   classesIllegalSuperCall: "Direct super call is illegal in non-constructor, use super.$1() instead",
@@ -17,7 +17,7 @@ export const MESSAGES = {
   missingTemplatesDirectory: "no templates directory - this is most likely the result of a broken `npm publish`. Please report to https://github.com/babel/babel/issues",
   unsupportedOutputType: "Unsupported output type $1",
   illegalMethodName: "Illegal method name $1",
-  lostTrackNodePath: "We lost track of this nodes position, likely because the AST was directly manipulated",
+  lostTrackNodePath: "We lost track of this node's position, likely because the AST was directly manipulated",
 
   modulesIllegalExportName: "Illegal export $1",
   modulesDuplicateDeclarations: "Duplicate module declarations with the same source but in different scopes",
@@ -26,7 +26,7 @@ export const MESSAGES = {
   undeclaredVariableType: "Referencing a type alias outside of a type annotation",
   undeclaredVariableSuggestion: "Reference to undeclared variable $1 - did you mean $2?",
 
-  traverseNeedsParent: "Must pass a scope and parentPath unless traversing a Program/File got a $1 node",
+  traverseNeedsParent: "You must pass a scope and parentPath unless traversing a Program/File got a $1 node",
   traverseVerifyRootFunction: "You passed `traverse()` a function when it expected a visitor object, are you sure you didn't mean `{ enter: Function }`?",
   traverseVerifyVisitorProperty: "You passed `traverse()` a visitor object with the property $1 that has the invalid property $2",
   traverseVerifyNodeType: "You gave us a visitor for the node type $1 but it's not a valid type",
@@ -37,7 +37,7 @@ export const MESSAGES = {
   pluginNotTransformer: "The plugin $1 didn't export a Plugin instance",
   pluginUnknown: "Unknown plugin $1",
 
-  pluginNotFile: "Plugin $1 is resolving to a different Babel version to what is doing the actual transformation..."
+  pluginNotFile: "Plugin $1 is resolving to a different Babel version than what is performing the transformation."
 };
 
 export function get(key: String, ...args) {

src/babel/transformation/modules/system.js

@@ -33,8 +33,7 @@ var hoistVariablesVisitor = {
     }
 
     // for (var i in test)
-    // for (var i = 0;;)
-    if (t.isFor(parent) && (parent.left === node || parent.init === node)) {
+    if (t.isFor(parent) && parent.left === node) {
       return node.declarations[0].id;
     }
 
@@ -78,6 +77,7 @@ export default class SystemFormatter extends AMDFormatter {
   constructor(file) {
     super(file);
 
+    this._setters = null;
     this.exportIdentifier = file.scope.generateUidIdentifier("export");
     this.noInteropRequireExport = true;
     this.noInteropRequireImport = true;
@@ -178,11 +178,14 @@ export default class SystemFormatter extends AMDFormatter {
 
     var block = t.blockStatement(program.body);
 
+    var setterListNode = this._buildRunnerSetters(block, hoistDeclarators);
+    this._setters = setterListNode;
+
     var runner = util.template("system", {
       MODULE_DEPENDENCIES: t.arrayExpression(this.buildDependencyLiterals()),
       EXPORT_IDENTIFIER:   this.exportIdentifier,
       MODULE_NAME:         moduleNameLiteral,
-      SETTERS:             this._buildRunnerSetters(block, hoistDeclarators),
+      SETTERS:             setterListNode,
       EXECUTE:             t.functionExpression(null, [], block)
     }, true);
 

src/babel/transformation/templates/default-parameter.js

@@ -1 +1 @@
-let VARIABLE_NAME = ARGUMENTS[ARGUMENT_KEY] === undefined ? DEFAULT_VALUE : ARGUMENTS[ARGUMENT_KEY];
+let VARIABLE_NAME = ARGUMENTS.length <= ARGUMENT_KEY || ARGUMENTS[ARGUMENT_KEY] === undefined ? DEFAULT_VALUE : ARGUMENTS[ARGUMENT_KEY];

src/babel/transformation/templates/helper-create-decorated-object.js

@@ -13,7 +13,7 @@
 
     descriptor.enumerable = true;
     descriptor.configurable = true;
-    descriptor.writable = true;
+    if ("value" in descriptor || descriptor.initializer) descriptor.writable = true;
 
     if (decorators) {
       for (var f = 0; f < decorators.length; f++) {

src/babel/transformation/transformers/deprecated.json

@@ -8,5 +8,8 @@
   "utility.inlineExpressions": "minification.constantFolding",
   "utility.deadCodeElimination": "minification.deadCodeElimination",
   "utility.removeConsoleCalls": "minification.removeConsole",
-  "utility.removeDebugger": "minification.removeDebugger"
+  "utility.removeDebugger": "minification.removeDebugger",
+
+  "es6.parameters.rest": "es6.parameters",
+  "es6.parameters.default": "es6.parameters"
 }

src/babel/transformation/transformers/es6/classes/index.js

@@ -0,0 +1,23 @@
+import LooseTransformer from "./loose";
+import VanillaTransformer from "./vanilla";
+import * as t from "../../../../types";
+import { bare } from "../../../helpers/name-method";
+
+export var visitor = {
+  ClassDeclaration(node) {
+    return t.variableDeclaration("let", [
+      t.variableDeclarator(node.id, t.toExpression(node))
+    ]);
+  },
+
+  ClassExpression(node, parent, scope, file) {
+    var inferred = bare(node, parent, scope);
+    if (inferred) return inferred;
+
+    if (file.isLoose("es6.classes")) {
+      return new LooseTransformer(this, file).run();
+    } else {
+      return new VanillaTransformer(this, file).run();
+    }
+  }
+};

src/babel/transformation/transformers/es6/classes/loose.js

@@ -0,0 +1,24 @@
+import VanillaTransformer from "./vanilla";
+import * as t from "../../../../types";
+
+export default class LooseClassTransformer extends VanillaTransformer {
+  constructor() {
+    super(...arguments);
+    this.isLoose = true;
+  }
+
+  _processMethod(node) {
+    if (!node.decorators) {
+      // use assignments instead of define properties for loose classes
+
+      var classRef = this.classRef;
+      if (!node.static) classRef = t.memberExpression(classRef, t.identifier("prototype"));
+      var methodName = t.memberExpression(classRef, node.key, node.computed || t.isLiteral(node.key));
+
+      var expr = t.expressionStatement(t.assignmentExpression("=", methodName, node.value));
+      t.inheritsComments(expr, node);
+      this.body.push(expr);
+      return true;
+    }
+  }
+}

src/babel/transformation/transformers/es6/classes/vanilla.js

@@ -1,27 +1,15 @@
-import type NodePath from "../../../traversal/path";
-import type File from "../../file";
-import memoiseDecorators from "../../helpers/memoise-decorators";
-import ReplaceSupers from "../../helpers/replace-supers";
-import * as nameMethod from "../../helpers/name-method";
-import * as defineMap from "../../helpers/define-map";
-import * as messages from "../../../messages";
-import * as util from  "../../../util";
-import * as t from "../../../types";
+import type NodePath from "../../../../traversal/path";
+import type File from "../../../file";
+import memoiseDecorators from "../../../helpers/memoise-decorators";
+import ReplaceSupers from "../../../helpers/replace-supers";
+import * as nameMethod from "../../../helpers/name-method";
+import * as defineMap from "../../../helpers/define-map";
+import * as messages from "../../../../messages";
+import * as util from  "../../../../util";
+import * as t from "../../../../types";
 
 const PROPERTY_COLLISION_METHOD_NAME = "__initializeProperties";
 
-export var visitor = {
-  ClassDeclaration(node) {
-    return t.variableDeclaration("let", [
-      t.variableDeclarator(node.id, t.toExpression(node))
-    ]);
-  },
-
-  ClassExpression(node, parent, scope, file) {
-    return new ClassTransformer(this, file).run();
-  }
-};
-
 var collectPropertyReferencesVisitor = {
   Identifier: {
     enter(node, parent, scope, state) {
@@ -37,16 +25,12 @@ var collectPropertyReferencesVisitor = {
 };
 
 var verifyConstructorVisitor = {
-  MethodDefinition: {
-    enter() {
-      this.skip();
-    }
+  MethodDefinition() {
+    this.skip();
   },
 
-  Property: {
-    enter(node) {
-      if (node.method) this.skip();
-    }
+  Property(node) {
+    if (node.method) this.skip();
   },
 
   CallExpression: {
@@ -55,35 +39,45 @@ var verifyConstructorVisitor = {
         state.hasBareSuper = true;
         state.bareSuper = this;
 
-        if (!state.hasSuper) {
+        if (!state.isDerived) {
           throw this.errorWithNode("super call is only allowed in derived constructor");
         }
       }
     }
   },
 
-  FunctionDeclaration: {
-    enter() {
-      this.skip();
-    }
+  "FunctionDeclaration|FunctionExpression"() {
+    this.skip();
   },
 
-  FunctionExpression: {
-    enter() {
-      this.skip();
-    }
-  },
+  ThisExpression(node, parent, scope, state) {
+    if (state.isDerived && !state.hasBareSuper) {
+      if (this.inShadow()) {
+        // https://github.com/babel/babel/issues/1920
+        var thisAlias = state.constructorPath.getData("this");
 
-  ThisExpression: {
-    enter(node, parent, scope, state) {
-      if (state.hasSuper && !state.hasBareSuper) {
+        if (!thisAlias) {
+          thisAlias = state.constructorPath.setData(
+            "this",
+            state.constructorPath.scope.generateUidIdentifier("this")
+          );
+        }
+
+        return thisAlias;
+      } else {
         throw this.errorWithNode("'this' is not allowed before super()");
       }
     }
+  },
+
+  Super(node, parent, scope, state) {
+    if (state.isDerived && !state.hasBareSuper && !this.parentPath.isCallExpression({ callee: node })) {
+      throw this.errorWithNode("'super.*' is not allowed before super()");
+    }
   }
 };
 
-class ClassTransformer {
+export default class ClassTransformer {
 
   /**
    * Description
@@ -96,26 +90,29 @@ class ClassTransformer {
     this.path   = path;
     this.file   = file;
 
-    this.hasInstanceDescriptors = false;
-    this.hasStaticDescriptors   = false;
-
-    this.instanceMutatorMap = {};
-    this.staticMutatorMap   = {};
+    this.clearDescriptors();
 
     this.instancePropBody = [];
     this.instancePropRefs = {};
     this.staticPropBody   = [];
     this.body             = [];
 
-    this.hasConstructor = false;
-    this.hasDecorators  = false;
-    this.className      = this.node.id;
-    this.classRef       = this.node.id || this.scope.generateUidIdentifier("class");
+    this.pushedConstructor = false;
+    this.pushedInherits    = false;
+    this.hasDecorators     = false;
+    this.isLoose           = false;
+
+    // class id
+    this.classId = this.node.id;
+
+    // this is the name of the binding that will **always** reference the class we've constructed
+    this.classRef = this.node.id || this.scope.generateUidIdentifier("class");
+
+    // this is a direct reference to the class we're building, class decorators can shadow the classRef
+    this.directRef = null;
 
     this.superName = this.node.superClass || t.identifier("Function");
-    this.hasSuper  = !!this.node.superClass;
-
-    this.isLoose = file.isLoose("es6.classes");
+    this.isDerived = !!this.node.superClass;
   }
 
   /**
@@ -126,7 +123,6 @@ class ClassTransformer {
 
   run() {
     var superName = this.superName;
-    var classRef  = this.classRef;
     var file      = this.file;
 
     //
@@ -136,16 +132,7 @@ class ClassTransformer {
     //
 
     var constructorBody = this.constructorBody = t.blockStatement([]);
-    var constructor;
-
-    if (this.className) {
-      constructor = t.functionDeclaration(this.className, [], constructorBody);
-      body.push(constructor);
-    } else {
-      constructor = t.functionExpression(null, [], constructorBody);
-    }
-
-    this.constructor = constructor;
+    this.constructor    = this.buildConstructor();
 
     //
 
@@ -153,26 +140,22 @@ class ClassTransformer {
     var closureArgs = [];
 
     //
-    if (this.hasSuper) {
+    if (this.isDerived) {
       closureArgs.push(superName);
 
       superName = this.scope.generateUidIdentifierBasedOnNode(superName);
       closureParams.push(superName);
 
       this.superName = superName;
-      body.push(t.expressionStatement(t.callExpression(file.addHelper("inherits"), [classRef, superName])));
     }
 
     //
     var decorators = this.node.decorators;
     if (decorators) {
-      // create a class reference to use later on
-      this.classRef = this.scope.generateUidIdentifier(classRef);
-
       // this is so super calls and the decorators have access to the raw function
-      body.push(t.variableDeclaration("var", [
-        t.variableDeclarator(this.classRef, classRef)
-      ]));
+      this.directRef = this.scope.generateUidIdentifier(this.classRef);
+    } else {
+      this.directRef = this.classRef;
     }
 
     //
@@ -181,46 +164,21 @@ class ClassTransformer {
     // make sure this class isn't directly called
     constructorBody.body.unshift(t.expressionStatement(t.callExpression(file.addHelper("class-call-check"), [
       t.thisExpression(),
-      this.classRef
+      this.directRef
     ])));
 
     //
-
-    if (decorators) {
-      // reverse the decorators so we execute them in the right order
-      decorators = decorators.reverse();
-
-      for (var i = 0; i < decorators.length; i++) {
-        var decorator = decorators[i];
-
-        var decoratorNode = util.template("class-decorator", {
-          DECORATOR: decorator.expression,
-          CLASS_REF: classRef
-        }, true);
-        decoratorNode.expression._ignoreModulesRemap = true;
-        body.push(decoratorNode);
-      }
-    }
+    this.pushDecorators();
 
     body = body.concat(this.staticPropBody);
 
-    if (this.className) {
+    if (this.classId) {
       // named class with only a constructor
       if (body.length === 1) return t.toExpression(body[0]);
-    } else {
-      // infer class name if this is a nameless class expression
-      constructor = nameMethod.bare(constructor, this.parent, this.scope) || constructor;
-
-      body.unshift(t.variableDeclaration("var", [
-        t.variableDeclarator(classRef, constructor)
-      ]));
-
-      t.inheritsComments(body[0], this.node);
     }
 
     //
-
-    body.push(t.returnStatement(classRef));
+    body.push(t.returnStatement(this.classRef));
 
     return t.callExpression(
       t.functionExpression(null, closureParams, t.blockStatement(body)),
@@ -228,6 +186,14 @@ class ClassTransformer {
     );
   }
 
+  /**
+   * Description
+   */
+
+  buildConstructor() {
+    return t.functionDeclaration(this.classRef, [], this.constructorBody);
+  }
+
   /**
    * Description
    */
@@ -258,23 +224,26 @@ class ClassTransformer {
    */
 
   constructorMeMaybe() {
-    if (!this.hasSuper) return;
-
     var hasConstructor = false;
     var paths = this.path.get("body.body");
-
     for (var path of (paths: Array)) {
       hasConstructor = path.equals("kind", "constructor");
       if (hasConstructor) break;
     }
+    if (hasConstructor) return;
 
-    if (!hasConstructor) {
-      this.path.get("body").unshiftContainer("body", t.methodDefinition(
-        t.identifier("constructor"),
-        util.template("class-derived-default-constructor"),
-        "constructor"
-      ));
+    var constructor;
+    if (this.isDerived) {
+      constructor = util.template("class-derived-default-constructor");
+    } else {
+      constructor = t.functionExpression(null, [], t.blockStatement([]));
     }
+
+    this.path.get("body").unshiftContainer("body", t.methodDefinition(
+      t.identifier("constructor"),
+      constructor,
+      "constructor"
+    ));
   }
 
   /**
@@ -283,10 +252,25 @@ class ClassTransformer {
 
   buildBody() {
     this.constructorMeMaybe();
+    this.pushBody();
+    this.placePropertyInitializers();
 
-    var constructorBody = this.constructorBody;
-    var classBodyPaths  = this.path.get("body.body");
-    var body            = this.body;
+    if (this.userConstructor) {
+      var constructorBody = this.constructorBody;
+      constructorBody.body = constructorBody.body.concat(this.userConstructor.body.body);
+      t.inherits(this.constructor, this.userConstructor);
+      t.inherits(constructorBody, this.userConstructor.body);
+    }
+
+    this.pushDescriptors();
+  }
+
+  /**
+   * Description
+   */
+
+  pushBody() {
+    var classBodyPaths = this.path.get("body.body");
 
     for (var path of (classBodyPaths: Array)) {
       var node = path.node;
@@ -302,7 +286,7 @@ class ClassTransformer {
         var replaceSupers = new ReplaceSupers({
           methodPath: path,
           methodNode: node,
-          objectRef:  this.classRef,
+          objectRef:  this.directRef,
           superRef:   this.superName,
           isStatic:   node.static,
           isLoose:    this.isLoose,
@@ -321,16 +305,28 @@ class ClassTransformer {
         this.pushProperty(node, path);
       }
     }
+  }
 
-    //
-    this.placePropertyInitializers();
+  /**
+   * Description
+   */
 
-    //
-    if (this.userConstructor) {
-      constructorBody.body = constructorBody.body.concat(this.userConstructor.body.body);
-      t.inherits(this.constructor, this.userConstructor);
-      t.inherits(this.constructorBody, this.userConstructor.body);
-    }
+  clearDescriptors() {
+    this.hasInstanceDescriptors = false;
+    this.hasStaticDescriptors   = false;
+
+    this.instanceMutatorMap = {};
+    this.staticMutatorMap   = {};
+  }
+
+  /**
+   * Description
+   */
+
+  pushDescriptors() {
+    this.pushInherits();
+
+    var body = this.body;
 
     var instanceProps;
     var staticProps;
@@ -378,8 +374,14 @@ class ClassTransformer {
         t.callExpression(this.file.addHelper(classHelper), args)
       ));
     }
+
+    this.clearDescriptors();
   }
 
+  /**
+   * Description
+   */
+
   buildObjectAssignment(id) {
     return t.variableDeclaration("var", [
       t.variableDeclarator(id, t.objectExpression([]))
@@ -405,18 +407,14 @@ class ClassTransformer {
         t.functionExpression(null, [], t.blockStatement(body))
       ), null, true);
 
-      if (this.hasSuper) {
+      if (this.isDerived) {
         this.bareSuper.insertAfter(call);
       } else {
         this.constructorBody.body.unshift(call);
       }
     } else {
-      if (this.hasSuper) {
-        if (this.hasConstructor) {
-          this.bareSuper.insertAfter(body);
-        } else {
-          this.constructorBody.body = this.constructorBody.body.concat(body);
-        }
+      if (this.isDerived) {
+        this.bareSuper.insertAfter(body);
       } else {
         this.constructorBody.body = body.concat(this.constructorBody.body);
       }
@@ -437,7 +435,7 @@ class ClassTransformer {
     }
 
     return false;
-   }
+  }
 
   /**
    * Description
@@ -445,17 +443,25 @@ class ClassTransformer {
 
    verifyConstructor(path: NodePath) {
     var state = {
-      hasBareSuper: false,
-      bareSuper:    null,
-      hasSuper:     this.hasSuper,
-      file:         this.file
+      constructorPath: path.get("value"),
+      hasBareSuper:    false,
+      bareSuper:       null,
+      isDerived:       this.isDerived,
+      file:            this.file,
     };
 
-    path.get("value").traverse(verifyConstructorVisitor, state);
+    state.constructorPath.traverse(verifyConstructorVisitor, state);
+
+    var thisAlias = state.constructorPath.getData("this");
+    if (thisAlias && state.bareSuper) {
+      state.bareSuper.insertAfter(t.variableDeclaration("var", [
+        t.variableDeclarator(thisAlias, t.thisExpression())
+      ]));
+    }
 
     this.bareSuper = state.bareSuper;
 
-    if (!state.hasBareSuper && this.hasSuper) {
+    if (!state.hasBareSuper && this.isDerived) {
       throw path.errorWithNode("Derived constructor must call super()");
     }
   }
@@ -471,24 +477,16 @@ class ClassTransformer {
 
     if (node.kind === "method") {
       nameMethod.property(node, this.file, path ? path.get("value").scope : this.scope);
-
-      if (this.isLoose && !node.decorators) {
-        // use assignments instead of define properties for loose classes
-
-        var classRef = this.classRef;
-        if (!node.static) classRef = t.memberExpression(classRef, t.identifier("prototype"));
-        var methodName = t.memberExpression(classRef, node.key, node.computed || t.isLiteral(node.key));
-
-        var expr = t.expressionStatement(t.assignmentExpression("=", methodName, node.value));
-        t.inheritsComments(expr, node);
-        this.body.push(expr);
-        return;
-      }
+      if (this._processMethod(node)) return;
     }
 
     this.pushToMap(node);
   }
 
+  _processMethod() {
+    return false;
+  }
+
   /**
    * Description
    */
@@ -568,5 +566,61 @@ class ClassTransformer {
     construct.params                = fn.params;
 
     t.inherits(construct.body, fn.body);
+
+    // push constructor to body
+    this._pushConstructor();
+  }
+
+  _pushConstructor() {
+    if (this.pushedConstructor) return;
+    this.pushedConstructor = true;
+
+    // we haven't pushed any descriptors yet
+    if (this.hasInstanceDescriptors || this.hasStaticDescriptors) {
+      this.pushDescriptors();
+    }
+
+    this.body.push(this.constructor);
+
+    this.pushInherits();
+  }
+
+  /**
+   * Push inherits helper to body.
+   */
+
+  pushInherits() {
+    if (!this.isDerived || this.pushedInherits) return;
+
+    this.pushedInherits = true;
+    this.body.push(t.expressionStatement(t.callExpression(
+      this.file.addHelper("inherits"),
+      [this.classRef, this.superName]
+    )));
+  }
+
+  /**
+   * Push decorators to body.
+   */
+
+  pushDecorators() {
+    var decorators = this.node.decorators;
+    if (!decorators) return;
+
+    this.body.push(t.variableDeclaration("var", [
+      t.variableDeclarator(this.directRef, this.classRef)
+    ]));
+
+    // reverse the decorators so we execute them in the right order
+    decorators = decorators.reverse();
+
+    for (var decorator of (decorators: Array)) {
+      var decoratorNode = util.template("class-decorator", {
+        DECORATOR: decorator.expression,
+        CLASS_REF: this.classRef
+      }, true);
+      decoratorNode.expression._ignoreModulesRemap = true;
+      this.body.push(decoratorNode);
+    }
   }
 }

src/babel/transformation/transformers/es6/destructuring.js

@@ -70,7 +70,15 @@ export var visitor = {
       let pattern = node.params[i];
       if (!t.isPattern(pattern)) continue;
 
-      var ref = node.params[i] = scope.generateUidIdentifier("ref");
+      var ref = scope.generateUidIdentifier("ref");
+      if (t.isAssignmentPattern(pattern)) {
+        var _pattern = pattern;
+        pattern = pattern.left;
+        _pattern.left = ref;
+      } else {
+        node.params[i] = ref;
+      }
+
       t.inherits(ref, pattern);
 
       var destructuring = new DestructuringTransformer({
@@ -308,9 +316,12 @@ class DestructuringTransformer {
 
     var left = pattern.left;
     if (t.isPattern(left)) {
-      this.nodes.push(t.expressionStatement(
+      var tempValueDefault = t.expressionStatement(
         t.assignmentExpression("=", tempValueRef, tempConditional)
-      ));
+      );
+      tempValueDefault._blockHoist = this.blockHoist;
+
+      this.nodes.push(tempValueDefault);
       this.push(left, tempValueRef);
     } else {
       this.nodes.push(this.buildVariableAssignment(left, tempConditional));

src/babel/transformation/transformers/es6/parameters/default.js

@@ -1,7 +1,7 @@
-import callDelegate from "../../helpers/call-delegate";
-import getFunctionArity from "../../helpers/get-function-arity";
-import * as util from  "../../../util";
-import * as t from "../../../types";
+import callDelegate from "../../../helpers/call-delegate";
+import getFunctionArity from "../../../helpers/get-function-arity";
+import * as util from  "../../../../util";
+import * as t from "../../../../types";
 
 var hasDefaults = function (node) {
   for (var i = 0; i < node.params.length; i++) {
@@ -38,7 +38,7 @@ export var visitor = {
 
     //
     var argsIdentifier = t.identifier("arguments");
-    argsIdentifier._shadowedFunctionLiteral = true;
+    argsIdentifier._shadowedFunctionLiteral = this;
 
     // push a default parameter definition
     function pushDefNode(left, right, i) {

src/babel/transformation/transformers/es6/parameters/index.js

@@ -0,0 +1,10 @@
+import * as visitors from "../../../../traversal/visitors";
+
+import * as def from "./default";
+import * as rest from "./rest";
+
+export var metadata = {
+  group: "builtin-advanced"
+};
+
+export var visitor = visitors.merge([rest.visitor, def.visitor]);

src/babel/transformation/transformers/es6/parameters/rest.js

@@ -1,5 +1,5 @@
-import * as util from  "../../../util";
-import * as t from "../../../types";
+import * as util from  "../../../../util";
+import * as t from "../../../../types";
 
 var memberExpressionOptimisationVisitor = {
   Scope(node, parent, scope, state) {
@@ -9,6 +9,11 @@ var memberExpressionOptimisationVisitor = {
     }
   },
 
+  Flow() {
+    // don't touch reference in type annotations
+    this.skip();
+  },
+
   Function(node, parent, scope, state) {
     // skip over functions as whatever `arguments` we reference inside will refer
     // to the wrong function
@@ -84,7 +89,7 @@ export var visitor = {
     var argsId = t.identifier("arguments");
 
     // otherwise `arguments` will be remapped in arrow functions
-    argsId._shadowedFunctionLiteral = true;
+    argsId._shadowedFunctionLiteral = this;
 
     // support patterns
     if (t.isPattern(rest)) {

src/babel/transformation/transformers/es6/spread.js

@@ -43,6 +43,10 @@ function build(props, scope) {
   return nodes;
 }
 
+export var metadata = {
+  group: "builtin-advanced"
+};
+
 export var visitor = {
   ArrayExpression(node, parent, scope) {
     var elements = node.elements;

src/babel/transformation/transformers/es6/tail-call.js

@@ -211,7 +211,7 @@ class TailCallTransformer {
       var decl = t.variableDeclarator(this.argumentsId);
       if (this.argumentsId) {
         decl.init = t.identifier("arguments");
-        decl.init._shadowedFunctionLiteral = true;
+        decl.init._shadowedFunctionLiteral = this.path;
       }
       topVars.push(decl);
     }
@@ -337,10 +337,6 @@ class TailCallTransformer {
       args = t.arrayExpression(node.arguments);
     }
 
-    if (t.isArrayExpression(args) && args.elements.length > this.node.params.length) {
-      this.needsArguments = true;
-    }
-
     var argumentsId = this.getArgumentsId();
     var params      = this.getParams();
 
@@ -354,17 +350,30 @@ class TailCallTransformer {
 
     if (t.isArrayExpression(args)) {
       var elems = args.elements;
-      for (let i = 0; i < elems.length && i < params.length; i++) {
+
+      // pad out the args so all the function args are reset - https://github.com/babel/babel/issues/1938
+      while (elems.length < params.length) {
+        elems.push(t.identifier("undefined"));
+      }
+
+      for (let i = 0; i < elems.length; i++) {
         let param = params[i];
-        let elem = elems[i] || (elems[i] = t.identifier("undefined"));
-        if (!param._isDefaultPlaceholder) {
+        let elem  = elems[i];
+
+        if (param && !param._isDefaultPlaceholder) {
           elems[i] = t.assignmentExpression("=", param, elem);
+        } else {
+          // exceeds parameters but push it anyway to ensure correct execution
         }
       }
 
       if (!this.needsArguments) {
         for (let elem of (elems: Array)) {
-          body.push(t.expressionStatement(elem));
+          // only push expressions that we really need, this will skip pure arguments that exceed the
+          // parameter length of the current function
+          if (!this.scope.isPure(elem)) {
+            body.push(t.expressionStatement(elem));
+          }
         }
       }
     } else {

src/babel/transformation/transformers/es6/template-literals.js

@@ -60,20 +60,16 @@ export var visitor = {
       if (expr) nodes.push(expr);
     }
 
+    // filter out empty string literals
+    nodes = nodes.filter(n => !t.isLiteral(n, { value: "" }));
+
+    // since `+` is left-to-right associative
+    // ensure the first node is a string if first/second isn't
+    if (!isString(nodes[0]) && !isString(nodes[1])) {
+      nodes.unshift(t.literal(""));
+    }
+
     if (nodes.length > 1) {
-      // filter out empty string literals
-      nodes = nodes.filter(n => !t.isLiteral(n, { value: "" }));
-
-      if (nodes.length === 1 && isString(nodes[0])) {
-        return nodes[0];
-      }
-
-      // since `+` is left-to-right associative
-      // ensure the first node is a string if first/second isn't
-      if (!isString(nodes[0]) && !isString(nodes[1])) {
-        nodes.unshift(t.literal(""));
-      }
-
       var root = buildBinaryExpression(nodes.shift(), nodes.shift());
 
       for (let node of (nodes: Array)) {

src/babel/transformation/transformers/index.js

@@ -46,9 +46,6 @@ export default {
   "es6.regex.sticky":                      require("./es6/regex.sticky"),
   "es6.regex.unicode":                     require("./es6/regex.unicode"),
   "es6.constants":                         require("./es6/constants"),
-  "es6.parameters.rest":                   require("./es6/parameters.rest"),
-  "es6.spread":                            require("./es6/spread"),
-  "es6.parameters.default":                require("./es6/parameters.default"),
   "es7.exportExtensions":                  require("./es7/export-extensions"),
   "spec.protoToAssign":                    require("babel-plugin-proto-to-assign"),
   "es7.doExpressions":                     require("./es7/do-expressions"),
@@ -57,6 +54,8 @@ export default {
   "spec.undefinedToVoid":                  require("babel-plugin-undefined-to-void"),
 
   //- builtin-advanced
+  "es6.spread":                            require("./es6/spread"),
+  "es6.parameters":                        require("./es6/parameters"),
   "es6.destructuring":                     require("./es6/destructuring"),
   "es6.blockScoping":                      require("./es6/block-scoping"),
   "es6.spec.blockScoping":                 require("./es6/spec.block-scoping"),
@@ -85,4 +84,5 @@ export default {
   _blockHoist:                             require("./internal/block-hoist"),
   jscript:                                 require("babel-plugin-jscript"),
   flow:                                    require("./other/flow"),
+  "optimisation.modules.system":           require("./optimisation/modules.system"),
 };

src/babel/transformation/transformers/internal/shadow-functions.js

@@ -6,9 +6,33 @@ export var metadata = {
 
 function remap(path, key, create) {
   // ensure that we're shadowed
-  if (!path.inShadow()) return;
+  var shadowPath = path.inShadow();
+  if (!shadowPath || shadowPath.isArrowFunctionExpression()) return;
 
-  var fnPath = path.findParent((path) => !path.is("shadow") && (path.isFunction() || path.isProgram()));
+  var shadowFunction = path.node._shadowedFunctionLiteral;
+  var currentFunction;
+
+  var fnPath = path.findParent(function (path) {
+    if (path.isProgram() || path.isFunction()) {
+      // catch current function in case this is the shadowed one and we can ignore it
+      currentFunction = currentFunction || path;
+    }
+
+    if (path.isProgram()) {
+      return true;
+    } else if (path.isFunction()) {
+      if (shadowFunction) {
+        return path === shadowFunction || path.node === shadowFunction.node;
+      } else {
+        return !path.is("shadow");
+      }
+    }
+
+    return false;
+  });
+
+  // no point in realiasing if we're in this function
+  if (fnPath === currentFunction) return;
 
   var cached = fnPath.getData(key);
   if (cached) return cached;
@@ -28,7 +52,7 @@ export var visitor = {
   },
 
   ReferencedIdentifier(node) {
-    if (node.name === "arguments" && !node._shadowedFunctionLiteral) {
+    if (node.name === "arguments") {
       return remap(this, "arguments", () => t.identifier("arguments"));
     }
   }

src/babel/transformation/transformers/optimisation/modules.system.js

@@ -0,0 +1,50 @@
+import * as t from "../../../types";
+
+export var metadata = {
+  optional: true,
+  group: "builtin-trailing"
+};
+
+export var visitor = {
+  Program(node, parent, scope, file){
+    if (file.moduleFormatter._setters){
+      scope.traverse(file.moduleFormatter._setters, optimizeSettersVisitor, {
+        exportFunctionIdentifier: file.moduleFormatter.exportIdentifier
+      });
+    }
+  }
+};
+
+/**
+ * Setters are optimized to avoid slow export behavior in modules that rely on deep hierarchies
+ * of export-from declarations.
+ * More info in https://github.com/babel/babel/pull/1722 and
+ * https://github.com/ModuleLoader/es6-module-loader/issues/386.
+ *
+ * TODO: Ideally this would be optimized during construction of the setters, but the current
+ * architecture of the module formatters make that difficult.
+ */
+var optimizeSettersVisitor = {
+  FunctionExpression: {
+    enter: (node, parent, scope, state) => {
+      state.hasExports = false;
+      state.exportObjectIdentifier = scope.generateUidIdentifier("exportObj");
+    },
+    exit: (node, parent, scope, state) => {
+      if (!state.hasExports) return;
+
+      node.body.body.unshift(t.variableDeclaration("var", [
+        t.variableDeclarator(t.cloneDeep(state.exportObjectIdentifier), t.objectExpression([]))
+      ]));
+      node.body.body.push(t.expressionStatement(t.callExpression(
+        t.cloneDeep(state.exportFunctionIdentifier), [t.cloneDeep(state.exportObjectIdentifier)])));
+    }
+  },
+  CallExpression: (node, parent, scope, state) => {
+    if (!t.isIdentifier(node.callee, {name: state.exportFunctionIdentifier.name})) return;
+
+    state.hasExports = true;
+    var memberNode = t.memberExpression(t.cloneDeep(state.exportObjectIdentifier), node.arguments[0], true);
+    return t.assignmentExpression("=", memberNode, node.arguments[1]);
+  }
+};

src/babel/transformation/transformers/optimisation/react.inline-elements.js

@@ -49,7 +49,9 @@ export var visitor = {
     pushElemProp("ref", t.literal(null));
 
     if (node.children.length) {
-      pushProp(props.properties, t.identifier("children"), t.arrayExpression(react.buildChildren(node)));
+      var children = react.buildChildren(node);
+      children = children.length === 1 ? children[0] : t.arrayExpression(children);
+      pushProp(props.properties, t.identifier("children"), children);
     }
 
     // props

src/babel/transformation/transformers/other/flow.js

@@ -5,6 +5,14 @@ export var metadata = {
 };
 
 export var visitor = {
+  Program(node, parent, scope, file) {
+    for (var comment of (file.ast.comments: Array)) {
+      if (comment.value.indexOf("@flow") >= 0) {
+        comment._displayed = true;
+      }
+    }
+  },
+
   Flow() {
     this.dangerouslyRemove();
   },

src/babel/transformation/transformers/spec/function-name.js

@@ -4,8 +4,24 @@ export var metadata = {
   group: "builtin-basic"
 };
 
+// visit Property functions first - https://github.com/babel/babel/issues/1860
 export var visitor = {
   "ArrowFunctionExpression|FunctionExpression": {
-    exit: bare
+    exit() {
+      if (!this.parentPath.isProperty()) {
+        return bare.apply(this, arguments);
+      }
+    }
+  },
+
+  ObjectExpression() {
+    var props = this.get("properties");
+    for (var prop of (props: Array)) {
+      var value = prop.get("value");
+      if (value.isFunction()) {
+        var newNode = bare(value.node, prop.node, value.scope);
+        if (newNode) value.replaceWith(newNode);
+      }
+    }
   }
 };

src/babel/traversal/path/ancestry.js

@@ -2,20 +2,28 @@ import * as t from "../../types";
 import NodePath from "./index";
 
 /**
- * Description
+ * Call the provided `callback` with the `NodePath`s of all the parents.
+ * When the `callback` returns a truthy value, we return that node path.
  */
 
 export function findParent(callback) {
   var path = this;
-  while (path) {
+  while (path = path.parentPath) {
     if (callback(path)) return path;
-    path = path.parentPath;
   }
   return null;
 }
 
 /**
- * Description
+ * Get the parent function of the current path.
+ */
+
+export function getFunctionParent() {
+  return this.findParent((path) => path.isFunction() || path.isProgram());
+}
+
+/**
+ * Walk up the tree until we hit a parent node path in a list.
  */
 
 export function getStatementParent() {
@@ -139,7 +147,9 @@ export function getDeepestCommonAncestorFrom(paths: Array<NodePath>, filter?: Fu
 }
 
 /**
- * Description
+ * Build an array of node paths containing the entire ancestry of the current node path.
+ *
+ * NOTE: The current node path is included in this.
  */
 
 export function getAncestry() {
@@ -168,14 +178,14 @@ export function inType() {
 }
 
 /**
- * Description
+ * Check if we're inside a shadowed function.
  */
 
 export function inShadow() {
   var path = this;
   while (path) {
     if (path.isFunction()) {
-      if (path.node.shadow) {
+      if (path.node.shadow || path.isArrowFunctionExpression()) {
         return path;
       } else {
         return null;

src/babel/traversal/path/family.js

@@ -1,3 +1,4 @@
+import type TraversalContext from "../index";
 import NodePath from "./index";
 import * as t from "../../types";
 
@@ -39,7 +40,7 @@ export function getOpposite() {
  * Description
  */
 
-export function getCompletionRecords(): Array<NodePath> {
+export function getCompletionRecords(): Array {
   var paths = [];
 
   var add = function (path) {
@@ -80,12 +81,13 @@ export function getSibling(key) {
  * Description
  */
 
-export function get(key: string): NodePath {
+export function get(key: string, context?: boolean | TraversalContext): NodePath {
+  if (context === true) context = this.context;
   var parts = key.split(".");
   if (parts.length === 1) { // "foo"
-    return this._getKey(key);
+    return this._getKey(key, context);
   } else { // "foo.bar"
-    return this._getPattern(parts);
+    return this._getPattern(parts, context);
   }
 }
 
@@ -93,7 +95,7 @@ export function get(key: string): NodePath {
  * Description
  */
 
-export function _getKey(key) {
+export function _getKey(key, context?) {
   var node      = this.node;
   var container = node[key];
 
@@ -106,7 +108,7 @@ export function _getKey(key) {
         parent: node,
         container: container,
         key: i
-      }).setContext();
+      }).setContext(context);
     });
   } else {
     return NodePath.get({
@@ -114,7 +116,7 @@ export function _getKey(key) {
       parent: node,
       container: node,
       key: key
-    }).setContext();
+    }).setContext(context);
   }
 }
 
@@ -122,7 +124,7 @@ export function _getKey(key) {
  * Description
  */
 
-export function _getPattern(parts) {
+export function _getPattern(parts, context) {
   var path = this;
   for (var part of (parts: Array)) {
     if (part === ".") {
@@ -131,7 +133,7 @@ export function _getPattern(parts) {
       if (Array.isArray(path)) {
         path = path[part];
       } else {
-        path = path.get(part);
+        path = path.get(part, context);
       }
     }
   }
@@ -142,6 +144,6 @@ export function _getPattern(parts) {
  * Description
  */
 
-export function getBindingIdentifiers() {
-  return t.getBindingIdentifiers(this.node);
+export function getBindingIdentifiers(duplicates?) {
+  return t.getBindingIdentifiers(this.node, duplicates);
 }

src/babel/traversal/path/introspection.js

@@ -147,18 +147,6 @@ export function isCompletionRecord(allowInsideFunction?) {
   return true;
 }
 
-/**
- * Description
- */
-
-export function isDirective() {
-  if (this.isExpressionStatement()) {
-    return this.get("expression").isLiteral();
-  } else {
-    return this.isLiteral() && this.parentPath.isExpressionStatement();
-  }
-}
-
 /**
  * Check whether or not the current `key` allows either a single statement or block statement
  * so we can explode it if necessary.
@@ -172,22 +160,6 @@ export function isStatementOrBlock() {
   }
 }
 
-/**
- * Check whether this node was a part of the original AST.
- */
-
-export function isUser() {
-  return this.node && !!this.node.loc;
-}
-
-/**
- * Check whether this node was generated by us and not a part of the original AST.
- */
-
-export function isGenerated() {
-  return !this.isUser();
-}
-
 /**
  * Check if the currently assigned path references the `importName` of `moduleSource`.
  */
@@ -226,7 +198,7 @@ export function referencesImport(moduleSource, importName) {
 }
 
 /**
- * Description
+ * Get the source code associated with this node.
  */
 
 export function getSource() {

src/babel/traversal/path/lib/hoister.js

@@ -19,7 +19,7 @@ var referenceVisitor = {
       state.bindings[node.name] = binding;
     } else {
       for (var violationPath of (binding.constantViolations: Array)) {
-        state.breakOnScopePaths.push(violationPath.scope.path);
+        state.breakOnScopePaths = state.breakOnScopePaths.concat(violationPath.getAncestry());
       }
     }
   }
@@ -106,12 +106,15 @@ export default class PathHoister {
 
     this.getCompatibleScopes();
 
-    var path = this.getAttachmentPath();
-    if (!path) return;
+    var attachTo = this.getAttachmentPath();
+    if (!attachTo) return;
 
-    var uid = path.scope.generateUidIdentifier("ref");
+    // don't bother hoisting to the same function as this will cause multiple branches to be evaluated more than once leading to a bad optimisation
+    if (attachTo.getFunctionParent() === this.path.getFunctionParent()) return;
 
-    path.insertBefore([
+    var uid = attachTo.scope.generateUidIdentifier("ref");
+
+    attachTo.insertBefore([
       t.variableDeclaration("var", [
         t.variableDeclarator(uid, this.path.node)
       ])

src/babel/traversal/path/lib/virtual-types.js

@@ -25,6 +25,22 @@ export var BindingIdentifier = {
   }
 };
 
+export var Statement = {
+  types: ["Statement"],
+  checkPath({ node, parent }) {
+    if (t.isStatement(node)) {
+      if (t.isVariableDeclaration(node)) {
+        if (t.isForXStatement(parent, { left: node })) return false;
+        if (t.isForStatement(parent, { init: node })) return false;
+      }
+
+      return true;
+    } else {
+      return false;
+    }
+  }
+};
+
 export var Expression = {
   types: ["Expression"],
   checkPath(path) {
@@ -61,3 +77,29 @@ export var Var = {
     return t.isVar(path.node);
   }
 };
+
+export var DirectiveLiteral = {
+  types: ["Literal"],
+  checkPath(path) {
+    return path.isLiteral() && path.parentPath.isExpressionStatement();
+  }
+};
+
+export var Directive = {
+  types: ["ExpressionStatement"],
+  checkPath(path) {
+    return path.get("expression").isLiteral();
+  }
+};
+
+export var User = {
+  checkPath(path) {
+    return path.node && !!path.node.loc;
+  }
+};
+
+export var Generated = {
+  checkPath(path) {
+    return !path.isUser();
+  }
+};

src/babel/traversal/path/modification.js

@@ -3,7 +3,7 @@ import NodePath from "./index";
 import * as t from "../../types";
 
 /**
- * Description
+ * Insert the provided nodes before the current one.
  */
 
 export function insertBefore(nodes) {
@@ -75,7 +75,8 @@ export function _maybePopFromStatements(nodes) {
 }
 
 /**
- * Description
+ * Insert the provided nodes after the current one. When inserting nodes after an
+ * expression, ensure that the completion record is correct by pushing the current node.
  */
 
 export function insertAfter(nodes) {
@@ -108,7 +109,7 @@ export function insertAfter(nodes) {
 }
 
 /**
- * Description
+ * Update all sibling node paths after `fromIndex` by `incrementBy`.
  */
 
 export function updateSiblingKeys(fromIndex, incrementBy) {
@@ -121,10 +122,6 @@ export function updateSiblingKeys(fromIndex, incrementBy) {
   }
 }
 
-/**
- * Description
- */
-
 /**
  * Description
  */
@@ -200,7 +197,8 @@ export function pushContainer(listKey, nodes) {
 }
 
 /**
- * Description
+ * Hoist the current node to the highest scope possible and return a UID
+ * referencing it.
  */
 
 export function hoist(scope = this.scope) {

src/babel/traversal/path/removal.js

@@ -11,7 +11,8 @@ export function remove() {
 }
 
 /**
- * Description
+ * Dangerously remove the current node. This may sometimes result in a tainted
+ * invalid AST so use with caution.
  */
 
 export function dangerouslyRemove() {

src/babel/traversal/path/replacement.js

@@ -32,7 +32,11 @@ var hoistVariablesVisitor = {
 };
 
 /**
- * Description
+ * Replace a node with an array of multiple. This method performs the following steps:
+ *
+ *  - Inherit the comments of first provided node with that of the current node.
+ *  - Insert the provided nodes after the current node.
+ *  - Remove the current node.
  */
 
 export function replaceWithMultiple(nodes: Array<Object>) {
@@ -46,7 +50,11 @@ export function replaceWithMultiple(nodes: Array<Object>) {
 }
 
 /**
- * Description
+ * Parse a string as an expression and replace the current node with the result.
+ *
+ * NOTE: This is typically not a good idea to use. Building source strings when
+ * transforming ASTs is an antipattern and SHOULD NOT be encouraged. Even if it's
+ * easier to use, your transforms will be extremely brittle.
  */
 
 export function replaceWithSourceString(replacement) {
@@ -70,7 +78,7 @@ export function replaceWithSourceString(replacement) {
 }
 
 /**
- * Description
+ * Replace the current node with another.
  */
 
 export function replaceWith(replacement, whateverAllowed) {
@@ -147,7 +155,9 @@ export function replaceWith(replacement, whateverAllowed) {
 }
 
 /**
- * Description
+ * This method takes an array of statements nodes and then explodes it
+ * into expressions. This method retains completion records which is
+ * extremely important to retain original semantics.
  */
 
 export function replaceExpressionWithStatements(nodes: Array) {

src/babel/traversal/scope/index.js

@@ -146,7 +146,7 @@ export default class Scope {
     }
 
     var cached = path.getData("scope");
-    if (cached && cached.parent === parent) {
+    if (cached && cached.parent === parent && cached.block === path.node) {
       return cached;
     } else {
       path.setData("scope", this);
@@ -491,32 +491,32 @@ export default class Scope {
     }
 
     var parent = this.getProgramParent();
-    var ids = path.getBindingIdentifiers();
+    var ids = path.getBindingIdentifiers(true);
 
     for (var name in ids) {
-      var id = ids[name];
+      for (var id of (ids[name]: Array)) {
+        var local = this.getOwnBinding(name);
+        if (local) {
+          // don't ever let a type alias shadow a local binding
+          if (kind === "type") continue;
 
-      var local = this.getOwnBinding(name);
-      if (local) {
-        // don't ever let a type alias shadow a local binding
-        if (kind === "type") continue;
+          // same identifier so continue safely as we're likely trying to register it
+          // multiple times
+          if (local.identifier === id) continue;
 
-        // same identifier so continue safely as we're likely trying to register it
-        // multiple times
-        if (local.identifier === id) continue;
+          this.checkBlockScopedCollisions(local, kind, name, id);
+        }
 
-        this.checkBlockScopedCollisions(local, kind, name, id);
+        parent.references[name] = true;
+
+        this.bindings[name] = new Binding({
+          identifier: id,
+          existing:   local,
+          scope:      this,
+          path:       path,
+          kind:       kind
+        });
       }
-
-      parent.references[name] = true;
-
-      this.bindings[name] = new Binding({
-        identifier: id,
-        existing:   local,
-        scope:      this,
-        path:       path,
-        kind:       kind
-      });
     }
   }
 

src/babel/traversal/visitors.js

@@ -126,6 +126,8 @@ export function merge(visitors) {
   var rootVisitor = {};
 
   for (var visitor of (visitors: Array)) {
+    explode(visitor);
+
     for (var type in visitor) {
       var nodeVisitor = rootVisitor[type] = rootVisitor[type] || {};
       mergePair(nodeVisitor, visitor[type]);

src/babel/types/converters.js

@@ -261,18 +261,22 @@ export function toBlock(node: Object, parent: Object): Object {
  */
 
 export function valueToNode(value: any): Object {
+  // undefined
   if (value === undefined) {
     return t.identifier("undefined");
   }
 
+  // null, booleans, strings, numbers, regexs
   if (value === true || value === false || value === null || isString(value) || isNumber(value) || isRegExp(value)) {
     return t.literal(value);
   }
 
+  // array
   if (Array.isArray(value)) {
     return t.arrayExpression(value.map(t.valueToNode));
   }
 
+  // object
   if (isPlainObject(value)) {
     var props = [];
     for (var key in value) {

src/babel/types/retrievers.js

@@ -5,7 +5,7 @@ import * as t from "./index";
  * Return a list of binding identifiers associated with the input `node`.
  */
 
-export function getBindingIdentifiers(node: Object): Object {
+export function getBindingIdentifiers(node: Object, duplicates?): Object {
   var search = [].concat(node);
   var ids    = object();
 
@@ -16,7 +16,12 @@ export function getBindingIdentifiers(node: Object): Object {
     var key = t.getBindingIdentifiers.keys[id.type];
 
     if (t.isIdentifier(id)) {
-      ids[id.name] = id;
+      if (duplicates) {
+        var _ids = ids[id.name] = ids[id.name] || [];
+        _ids.push(id);
+      } else {
+        ids[id.name] = id;
+      }
     } else if (t.isExportDeclaration(id)) {
       if (t.isDeclaration(node.declaration)) {
         search.push(node.declaration);

src/babel/util.js

@@ -56,7 +56,7 @@ export function resolveRelative(loc: string) {
   }
 }
 
-export function list(val: string): Array<string> {
+export function list(val: string): Array {
   if (!val) {
     return [];
   } else if (Array.isArray(val)) {
@@ -114,18 +114,26 @@ export function shouldIgnore(filename, ignore, only) {
 
   if (only) {
     for (let pattern of (only: Array)) {
-      if (pattern.test(filename)) return false;
+      if (_shouldIgnore(pattern, filename)) return false;
     }
     return true;
   } else if (ignore.length) {
     for (let pattern of (ignore: Array)) {
-      if (pattern.test(filename)) return true;
+      if (_shouldIgnore(pattern, filename)) return true;
     }
   }
 
   return false;
 }
 
+function _shouldIgnore(pattern, filename) {
+  if (typeof pattern === "function") {
+    return pattern(filename);
+  } else {
+    return pattern.test(filename);
+  }
+}
+
 var templateVisitor = {
   noScope: true,
 

test/acorn/tests-flow.js

@@ -9965,6 +9965,95 @@ var fbTestFixture = {
         end: { line: 1, column: 49 }
       }
     },
+    'declare class IViewFactory { didAnimate(view:Object, prop:string) :void; }': {
+      start: 0,
+      id: {
+        start: 14,
+        name: "IViewFactory",
+        type: "Identifier",
+        end: 26
+      },
+      typeParameters: null,
+      extends: [],
+      body: {
+        start: 27,
+        callProperties: [],
+        properties: [
+          {
+            start: 29,
+            value: {
+              start: 29,
+              params: [
+                {
+                  start: 40,
+                  name: {
+                    start: 40,
+                    name: "view",
+                    type: "Identifier",
+                    end: 44
+                  },
+                  optional: false,
+                  typeAnnotation: {
+                    start: 45,
+                    typeParameters: null,
+                    id: {
+                      start: 45,
+                      name: "Object",
+                      type: "Identifier",
+                      end: 51
+                    },
+                    type: "GenericTypeAnnotation",
+                    end: 51
+                  },
+                  type: "FunctionTypeParam",
+                  end: 51
+                },
+                {
+                  start: 53,
+                  name: {
+                    start: 53,
+                    name: "prop",
+                    type: "Identifier",
+                    end: 57
+                  },
+                  optional: false,
+                  typeAnnotation: {
+                    start: 58,
+                    type: "StringTypeAnnotation",
+                    end: 64
+                  },
+                  type: "FunctionTypeParam",
+                  end: 64
+                }
+              ],
+              rest: null,
+              typeParameters: null,
+              returnType: {
+                start: 67,
+                type: "VoidTypeAnnotation",
+                end: 71
+              },
+              type: "FunctionTypeAnnotation",
+              end: 71
+            },
+            key: {
+              start: 29,
+              name: "didAnimate",
+              type: "Identifier",
+              end: 39
+            },
+            optional: false,
+            type: "ObjectTypeProperty",
+            end: 72
+          }
+        ],
+        indexers: [],
+        type: "ObjectTypeAnnotation",
+        end: 74
+      },
+      type: "DeclareClass",
+      end: 74
+    },
     'declare class A {}': {
       type: 'DeclareClass',
       id: {

test/core/api.js

@@ -70,7 +70,7 @@ suite("api", function () {
 
     assert.deepEqual(transform('import localName from "./array";', {
       resolveModuleSource: function() {
-        return 'override-source';
+        return "override-source";
       }
     }).metadata.modules.imports, [
       {

test/core/fixtures/transformation/.es7.do-expressions/do-while.js

@@ -1 +0,0 @@
-assert.ok(false);

test/core/fixtures/transformation/.es7.do-expressions/nested-dos.js

@@ -1,3 +0,0 @@
-assert.equal(do {
-  do { "foo" };
-}, "foo");

test/core/fixtures/transformation/.es7.do-expressions/while.js

@@ -1 +0,0 @@
-assert.ok(false);

test/core/fixtures/transformation/es6.arrow-functions/default-parameters/expected.js

@@ -1,9 +1,9 @@
 "use strict";
 
 var some = function some() {
-  var count = arguments[0] === undefined ? "30" : arguments[0];
+  var count = arguments.length <= 0 || arguments[0] === undefined ? "30" : arguments[0];
 
   console.log("count", count);
 };
 
-some();
+some();

test/core/fixtures/transformation/es6.classes-loose/constructor-order/actual.js

@@ -0,0 +1,13 @@
+class x {
+  f() {
+    1
+    2
+    3
+  }
+
+  constructor() {
+    4
+    5
+    6
+  }
+}

test/core/fixtures/transformation/es6.classes-loose/constructor-order/expected.js

@@ -0,0 +1,18 @@
+"use strict";
+
+var x = (function () {
+  x.prototype.f = function f() {
+    1;
+    2;
+    3;
+  };
+
+  function x() {
+    4;
+    5;
+    6;
+    babelHelpers.classCallCheck(this, x);
+  }
+
+  return x;
+})();

test/core/fixtures/transformation/es6.classes/constructor/actual.js

@@ -10,3 +10,12 @@ class Foo extends Bar {
     this.state = "test";
   }
 }
+
+class ConstructorScoping {
+  constructor(){
+    let bar;
+    {
+      let bar;
+    }
+  }
+}

test/core/fixtures/transformation/es6.classes/constructor/expected.js

@@ -16,4 +16,13 @@ var Foo = (function (_Bar) {
 
   babelHelpers.inherits(Foo, _Bar);
   return Foo;
-})(Bar);
+})(Bar);
+
+var ConstructorScoping = function ConstructorScoping() {
+  babelHelpers.classCallCheck(this, ConstructorScoping);
+
+  var bar = undefined;
+  {
+    var _bar = undefined;
+  }
+};

test/core/fixtures/transformation/es6.classes/delay-arrow-function-for-bare-super-derived/actual.js

@@ -0,0 +1,7 @@
+class Foo extends Bar {
+  constructor() {
+    super(() => {
+      this.test;
+    });
+  }
+}

test/core/fixtures/transformation/es6.classes/delay-arrow-function-for-bare-super-derived/expected.js

@@ -0,0 +1,16 @@
+"use strict";
+
+var Foo = (function (_Bar) {
+  function Foo() {
+    babelHelpers.classCallCheck(this, Foo);
+
+    babelHelpers.get(Object.getPrototypeOf(Foo.prototype), "constructor", this).call(this, function () {
+      _this.test;
+    });
+
+    var _this = this;
+  }
+
+  babelHelpers.inherits(Foo, _Bar);
+  return Foo;
+})(Bar);

test/core/fixtures/transformation/es6.classes/statement/expected.js

@@ -6,26 +6,22 @@ var BaseView = function BaseView() {
   this.autoRender = true;
 };
 
-var BaseView = (function () {
-  var _class = function BaseView() {
-    babelHelpers.classCallCheck(this, _class);
+var BaseView = function BaseView() {
+  babelHelpers.classCallCheck(this, BaseView);
 
-    this.autoRender = true;
-  };
-
-  return _class;
-})();
+  this.autoRender = true;
+};
 
 var BaseView = (function () {
-  var _class2 = function BaseView() {
-    babelHelpers.classCallCheck(this, _class2);
-  };
+  function BaseView() {
+    babelHelpers.classCallCheck(this, BaseView);
+  }
 
-  babelHelpers.createClass(_class2, [{
+  babelHelpers.createClass(BaseView, [{
     key: "foo",
     value: function foo() {
       this.autoRender = true;
     }
   }]);
-  return _class2;
+  return BaseView;
 })();

test/core/fixtures/transformation/es6.classes/super-class-anonymous/expected.js

@@ -9,9 +9,9 @@ var TestEmpty = (function (_ref) {
   babelHelpers.inherits(TestEmpty, _ref);
   return TestEmpty;
 })((function () {
-  var _class = function _class() {
+  function _class() {
     babelHelpers.classCallCheck(this, _class);
-  };
+  }
 
   return _class;
 })());
@@ -25,9 +25,9 @@ var TestConstructorOnly = (function (_ref2) {
   babelHelpers.inherits(TestConstructorOnly, _ref2);
   return TestConstructorOnly;
 })((function () {
-  var _class2 = function _class2() {
+  function _class2() {
     babelHelpers.classCallCheck(this, _class2);
-  };
+  }
 
   return _class2;
 })());
@@ -41,9 +41,9 @@ var TestMethodOnly = (function (_ref3) {
   babelHelpers.inherits(TestMethodOnly, _ref3);
   return TestMethodOnly;
 })((function () {
-  var _class3 = function _class3() {
+  function _class3() {
     babelHelpers.classCallCheck(this, _class3);
-  };
+  }
 
   babelHelpers.createClass(_class3, [{
     key: "method",
@@ -61,9 +61,9 @@ var TestConstructorAndMethod = (function (_ref4) {
   babelHelpers.inherits(TestConstructorAndMethod, _ref4);
   return TestConstructorAndMethod;
 })((function () {
-  var _class4 = function _class4() {
+  function _class4() {
     babelHelpers.classCallCheck(this, _class4);
-  };
+  }
 
   babelHelpers.createClass(_class4, [{
     key: "method",
@@ -81,9 +81,9 @@ var TestMultipleMethods = (function (_ref5) {
   babelHelpers.inherits(TestMultipleMethods, _ref5);
   return TestMultipleMethods;
 })((function () {
-  var _class5 = function _class5() {
+  function _class5() {
     babelHelpers.classCallCheck(this, _class5);
-  };
+  }
 
   babelHelpers.createClass(_class5, [{
     key: "m1",

test/core/fixtures/transformation/es6.classes/super-reference-before-bare-super/actual.js

@@ -0,0 +1,6 @@
+class Foo extends Bar {
+  constructor() {
+    super.foo();
+    super();
+  }
+}

test/core/fixtures/transformation/es6.classes/super-reference-before-bare-super/options.json

@@ -0,0 +1,3 @@
+{
+  "throws": "'super.*' is not allowed before super()"
+}

test/core/fixtures/transformation/es6.destructuring/parameters/actual.js

@@ -1,4 +1,4 @@
-function somethingAdvanced({topLeft: {x: x1, y: y1}, bottomRight: {x: x2, y: y2}}){
+function somethingAdvanced({topLeft: {x: x1, y: y1} = {}, bottomRight: {x: x2, y: y2} = {}}, p2, p3){
 
 }
 

test/core/fixtures/transformation/es6.destructuring/parameters/expected.js

@@ -1,10 +1,12 @@
 "use strict";
 
-function somethingAdvanced(_ref) {
+function somethingAdvanced(_ref, p2, p3) {
   var _ref$topLeft = _ref.topLeft;
+  _ref$topLeft = _ref$topLeft === undefined ? {} : _ref$topLeft;
   var x1 = _ref$topLeft.x;
   var y1 = _ref$topLeft.y;
   var _ref$bottomRight = _ref.bottomRight;
+  _ref$bottomRight = _ref$bottomRight === undefined ? {} : _ref$bottomRight;
   var x2 = _ref$bottomRight.x;
   var y2 = _ref$bottomRight.y;
 }

test/core/fixtures/transformation/es6.modules-amd/exports-default/expected.js

@@ -9,13 +9,9 @@ define(["exports", "module"], function (exports, module) {
 
   module.exports = function () {};
 
-  var _default = (function () {
-    var _class = function _default() {
-      babelHelpers.classCallCheck(this, _class);
-    };
-
-    return _class;
-  })();
+  var _default = function _default() {
+    babelHelpers.classCallCheck(this, _default);
+  };
 
   module.exports = _default;
 

test/core/fixtures/transformation/es6.modules-common/exports-default/expected.js

@@ -11,13 +11,9 @@ exports["default"] = foo;
 
 exports["default"] = function () {};
 
-var _default = (function () {
-  var _class = function _default() {
-    babelHelpers.classCallCheck(this, _class);
-  };
-
-  return _class;
-})();
+var _default = function _default() {
+  babelHelpers.classCallCheck(this, _default);
+};
 
 exports["default"] = _default;
 

test/core/fixtures/transformation/es6.modules-ignore/exports-default/expected.js

@@ -2,13 +2,9 @@
 
 function _classCallCheck(instance, Constructor) { if (!(instance instanceof Constructor)) { throw new TypeError("Cannot call a class as a function"); } }
 
-var _default = (function () {
-  var _class = function _default() {
-    _classCallCheck(this, _class);
-  };
-
-  return _class;
-})();
+var _default = function _default() {
+  _classCallCheck(this, _default);
+};
 
 function foo() {}
 

test/core/fixtures/transformation/es6.modules-system/exports-default/expected.js

@@ -22,13 +22,9 @@ System.register([], function (_export) {
 
       _export("default", function () {});
 
-      _default = (function () {
-        var _class = function _default() {
-          _classCallCheck(this, _class);
-        };
-
-        return _class;
-      })();
+      _default = function _default() {
+        _classCallCheck(this, _default);
+      };
 
       _export("default", _default);
 
@@ -43,4 +39,4 @@ System.register([], function (_export) {
       })());
     }
   };
-});
+});

test/core/fixtures/transformation/es6.modules-system/hoist-function-exports/actual.js

@@ -8,6 +8,8 @@ export var p = 5;
 
 for (var a in b) ;
 
+for (var i = 0, j = 0;;) ;
+
 export var isOdd = (function (isEven) {
   return function (n) {
     return !isEven(n);

test/core/fixtures/transformation/es6.modules-system/hoist-function-exports/expected.js

@@ -1,7 +1,7 @@
 System.register(["./evens"], function (_export) {
   "use strict";
 
-  var isEven, p, a, isOdd;
+  var isEven, p, a, i, j, isOdd;
 
   _export("nextOdd", nextOdd);
 
@@ -20,6 +20,8 @@ System.register(["./evens"], function (_export) {
 
       for (a in b);
 
+      for (i = 0, j = 0;;);
+
       isOdd = (function (isEven) {
         return function (n) {
           return !isEven(n);

test/core/fixtures/transformation/es6.modules-umd/exports-default/expected.js

@@ -21,13 +21,9 @@
 
   module.exports = function () {};
 
-  var _default = (function () {
-    var _class = function _default() {
-      babelHelpers.classCallCheck(this, _class);
-    };
-
-    return _class;
-  })();
+  var _default = function _default() {
+    babelHelpers.classCallCheck(this, _default);
+  };
 
   module.exports = _default;
 

test/core/fixtures/transformation/es6.parameters.default/multiple/expected.js

@@ -1,14 +0,0 @@
-"use strict";
-
-var t = function t() {
-  var e = arguments[0] === undefined ? "foo" : arguments[0];
-  var f = arguments[1] === undefined ? 5 : arguments[1];
-
-  return e + " bar " + f;
-};
-
-var a = function a(e) {
-  var f = arguments[1] === undefined ? 5 : arguments[1];
-
-  return e + " bar " + f;
-};

test/core/fixtures/transformation/es6.parameters.default/single/expected.js

@@ -1,7 +0,0 @@
-"use strict";
-
-var t = function t() {
-  var f = arguments[0] === undefined ? "foo" : arguments[0];
-
-  return f + " bar";
-};

test/core/fixtures/transformation/es6.parameters/default-eval/expected.js

@@ -2,7 +2,7 @@
 
 var x = "outside";
 function outer() {
-  var a = arguments[0] === undefined ? function () {
+  var a = arguments.length <= 0 || arguments[0] === undefined ? function () {
     return eval("x");
   } : arguments[0];
   return (function () {
@@ -10,4 +10,4 @@ function outer() {
     return a();
   })();
 }
-outer();
+outer();

test/core/fixtures/transformation/es6.parameters/default-multiple/expected.js

@@ -0,0 +1,14 @@
+"use strict";
+
+var t = function t() {
+  var e = arguments.length <= 0 || arguments[0] === undefined ? "foo" : arguments[0];
+  var f = arguments.length <= 1 || arguments[1] === undefined ? 5 : arguments[1];
+
+  return e + " bar " + f;
+};
+
+var a = function a(e) {
+  var f = arguments.length <= 1 || arguments[1] === undefined ? 5 : arguments[1];
+
+  return e + " bar " + f;
+};

test/core/fixtures/transformation/es6.parameters/default-single/expected.js

@@ -0,0 +1,7 @@
+"use strict";
+
+var t = function t() {
+  var f = arguments.length <= 0 || arguments[0] === undefined ? "foo" : arguments[0];
+
+  return f + " bar";
+};

test/core/fixtures/transformation/es6.parameters/rest-flow-parameter-reference/actual.js

@@ -0,0 +1,9 @@
+function foo(
+  x: Object,
+  ...types
+): { types: Array<number>, x: Object } {
+  return {
+    types: types,
+    x: x,
+  };
+}

test/core/fixtures/transformation/es6.parameters/rest-flow-parameter-reference/expected.js

@@ -0,0 +1,12 @@
+"use strict";
+
+function foo(x) {
+  for (var _len = arguments.length, types = Array(_len > 1 ? _len - 1 : 0), _key = 1; _key < _len; _key++) {
+    types[_key - 1] = arguments[_key];
+  }
+
+  return {
+    types: types,
+    x: x
+  };
+}