v5.0.12

Strip out comments before parsing babelrc

CHANGELOG.md

@@ -13,6 +13,20 @@ _Note: Gaps between patch versions are faulty/broken releases._
 
 See [CHANGELOG - 6to5](CHANGELOG-6to5.md) for the pre-4.0.0 version changelog.
 
+## 5.0.12
+
+ * **Bug Fix**
+  * Fix incorrect remapping of module references inside of a function id redirection container.
+
+## 5.0.11
+
+ * **Bug Fix**
+  * Fix new `for...of` loops not properly inheriting their original loop.
+ * **Internal**
+  * Disable scope instance cache.
+ * **Polish**
+  * Allow comments in `.babelrc` JSON.
+
 ## 5.0.9
 
  * **Polish**

package.json

@@ -1,7 +1,7 @@
 {
   "name": "babel-core",
   "description": "Turn ES6 code into readable vanilla ES5 with source maps",
-  "version": "5.0.10",
+  "version": "5.0.12",
   "author": "Sebastian McKenzie <sebmck@gmail.com>",
   "homepage": "https://babeljs.io/",
   "repository": "babel/babel",
@@ -53,6 +53,7 @@
     "slash": "^1.0.0",
     "source-map": "^0.4.0",
     "source-map-support": "^0.2.9",
+    "strip-json-comments": "^1.0.2",
     "to-fast-properties": "^1.0.0",
     "trim-right": "^1.0.0"
   },
@@ -65,7 +66,7 @@
     "esvalid": "^1.1.0",
     "istanbul": "^0.3.5",
     "matcha": "^0.6.0",
-    "mocha": "^2.1.0",
+    "mocha": "2.1.0",
     "rimraf": "^2.2.8",
     "uglify-js": "^2.4.16"
   }

packages/babel-cli/package.json

@@ -1,14 +1,14 @@
 {
   "name": "babel",
   "description": "Turn ES6 code into readable vanilla ES5 with source maps",
-  "version": "5.0.9",
+  "version": "5.0.11",
   "author": "Sebastian McKenzie <sebmck@gmail.com>",
   "homepage": "https://babeljs.io/",
   "repository": "babel/babel",
   "preferGlobal": true,
   "dependencies": {
     "chokidar": "^0.12.6",
-    "babel-core": "^5.0.9",
+    "babel-core": "^5.0.11",
     "commander": "^2.6.0",
     "fs-readdir-recursive": "^0.1.0",
     "output-file-sync": "^1.1.0",

packages/babel-runtime/package.json

@@ -1,7 +1,7 @@
 {
   "name": "babel-runtime",
   "description": "babel selfContained runtime",
-  "version": "5.0.9",
+  "version": "5.0.11",
   "repository": "babel/babel",
   "author": "Sebastian McKenzie <sebmck@gmail.com>",
   "dependencies": {

src/babel/generation/generators/expressions.js

@@ -169,4 +169,8 @@ export function MemberExpression(node, print) {
   }
 }
 
-export { MemberExpression as MetaProperty };
+export function MetaProperty(node, print) {
+  print(node.meta);
+  this.push(".");
+  print(node.property);
+}

src/babel/tools/resolve-rc.js

@@ -1,3 +1,4 @@
+import stripJsonComments from "strip-json-comments";
 import merge from "lodash/object/merge";
 import path from "path";
 import fs from "fs";
@@ -24,7 +25,7 @@ export default function (loc, opts = {}) {
       var json;
 
       try {
-        json = jsons[content] ||= JSON.parse(content);
+        json = jsons[content] ||= JSON.parse(stripJsonComments(content));
       } catch (err) {
         err.message = `${file}: ${err.message}`;
         throw err;

src/babel/transformation/helpers/name-method.js

@@ -27,6 +27,7 @@ var wrap = function (state, method, id, scope) {
       FUNCTION_ID: id,
       FUNCTION_KEY: scope.generateUidIdentifier(id.name)
     });
+    template.callee._skipModulesRemap = true;
 
     // shim in dummy params to retain function arity, if you try to read the
     // source then you'll get the original since it's proxied so it's all good

src/babel/transformation/modules/_default.js

@@ -46,6 +46,10 @@ var remapVisitor = {
       }
     }
 
+    if (node._skipModulesRemap) {
+      return this.skip();
+    }
+
     if (t.isAssignmentExpression(node) && !node._ignoreModulesRemap) {
       var exported = formatter.getExport(node.left, scope);
       if (exported) {

src/babel/transformation/transformers/es6/for-of.js

@@ -17,9 +17,6 @@ export function ForOfStatement(node, parent, scope, file) {
   var loop   = build.loop;
   var block  = loop.body;
 
-  // inherit comments from the original loop
-  t.inheritsComments(loop, node);
-
   // ensure that it's a block so we can take all its statements
   t.ensureBlock(node);
 
@@ -32,6 +29,7 @@ export function ForOfStatement(node, parent, scope, file) {
   block.body = block.body.concat(node.body.body);
 
   t.inherits(loop, node);
+  t.inherits(loop.body, node.body);
 
   if (build.replaceParent) {
     this.parentPath.replaceWithMultiple(build.node);

src/babel/traversal/scope.js

@@ -80,7 +80,7 @@ export default class Scope {
     if (cached && cached.parent === parent) {
       return cached;
     } else {
-      path.setData("scope", this);
+      //path.setData("scope", this);
     }
 
     this.parent = parent;

test/core/fixtures/transformation/misc/regression-1169/actual.js

@@ -0,0 +1,11 @@
+function foo() {
+  let input = ['a', 'b', 'c']
+  let output = {}
+
+  for (let c of input) {
+    let name = c
+    output[name] = name
+  }
+
+  return output
+}

test/core/fixtures/transformation/misc/regression-1169/expected.js

@@ -0,0 +1,34 @@
+'use strict';
+
+function foo() {
+  var input = ['a', 'b', 'c'];
+  var output = {};
+
+  var _iteratorNormalCompletion = true;
+  var _didIteratorError = false;
+  var _iteratorError = undefined;
+
+  try {
+    for (var _iterator = input[Symbol.iterator](), _step; !(_iteratorNormalCompletion = (_step = _iterator.next()).done); _iteratorNormalCompletion = true) {
+      var c = _step.value;
+
+      var _name = c;
+      output[_name] = _name;
+    }
+  } catch (err) {
+    _didIteratorError = true;
+    _iteratorError = err;
+  } finally {
+    try {
+      if (!_iteratorNormalCompletion && _iterator['return']) {
+        _iterator['return']();
+      }
+    } finally {
+      if (_didIteratorError) {
+        throw _iteratorError;
+      }
+    }
+  }
+
+  return output;
+}

test/core/fixtures/transformation/misc/regression-1199/actual.js

@@ -0,0 +1,8 @@
+const bug = 1;
+
+function foo() {}
+
+function bar() {
+  var bug;
+  bug = 2;
+}

test/core/fixtures/transformation/misc/regression-1199/expected.js

@@ -0,0 +1,10 @@
+"use strict";
+
+var bug = 1;
+
+function foo() {}
+
+function bar() {
+  var bug;
+  bug = 2;
+}

test/core/fixtures/transformation/spec.function-name/modules-2/actual.js

@@ -0,0 +1,8 @@
+import last from "lodash/array/last"
+
+export default class Container {
+  last(key) {
+    if (!this.has(key)) return
+    return last(this.tokens.get(key))
+  }
+}

test/core/fixtures/transformation/spec.function-name/modules-2/expected.js

@@ -0,0 +1,37 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+
+var _last2 = require("lodash/array/last");
+
+var _last3 = babelHelpers.interopRequireWildcard(_last2);
+
+var Container = (function () {
+  function Container() {
+    babelHelpers.classCallCheck(this, Container);
+  }
+
+  babelHelpers.createClass(Container, [{
+    key: "last",
+    value: (function (_last) {
+      function last(_x) {
+        return _last.apply(this, arguments);
+      }
+
+      last.toString = function () {
+        return _last.toString();
+      };
+
+      return last;
+    })(function (key) {
+      if (!this.has(key)) return;
+      return _last3["default"](this.tokens.get(key));
+    })
+  }]);
+  return Container;
+})();
+
+exports["default"] = Container;
+module.exports = exports["default"];

test/core/fixtures/transformation/spec.function-name/modules-3/actual.js

@@ -0,0 +1,7 @@
+import {getForm} from "./store"
+
+export default class Login extends React.Component {
+  getForm() {
+    return getForm().toJS()
+  }
+}

test/core/fixtures/transformation/spec.function-name/modules-3/expected.js

@@ -0,0 +1,39 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+
+var _getForm2 = require("./store");
+
+var Login = (function (_React$Component) {
+  function Login() {
+    babelHelpers.classCallCheck(this, Login);
+
+    if (_React$Component != null) {
+      _React$Component.apply(this, arguments);
+    }
+  }
+
+  babelHelpers.inherits(Login, _React$Component);
+  babelHelpers.createClass(Login, [{
+    key: "getForm",
+    value: (function (_getForm) {
+      function getForm() {
+        return _getForm.apply(this, arguments);
+      }
+
+      getForm.toString = function () {
+        return _getForm.toString();
+      };
+
+      return getForm;
+    })(function () {
+      return _getForm2.getForm().toJS();
+    })
+  }]);
+  return Login;
+})(React.Component);
+
+exports["default"] = Login;
+module.exports = exports["default"];

test/core/fixtures/transformation/spec.function-name/modules/actual.js

@@ -0,0 +1,9 @@
+import events from "events";
+
+class Template {
+  events() {
+    return events;
+  }
+}
+
+console.log(new Template().events());

test/core/fixtures/transformation/spec.function-name/modules/expected.js

@@ -0,0 +1,31 @@
+"use strict";
+
+var _events2 = require("events");
+
+var _events3 = babelHelpers.interopRequireWildcard(_events2);
+
+var Template = (function () {
+  function Template() {
+    babelHelpers.classCallCheck(this, Template);
+  }
+
+  babelHelpers.createClass(Template, [{
+    key: "events",
+    value: (function (_events) {
+      function events() {
+        return _events.apply(this, arguments);
+      }
+
+      events.toString = function () {
+        return _events.toString();
+      };
+
+      return events;
+    })(function () {
+      return _events3["default"];
+    })
+  }]);
+  return Template;
+})();
+
+console.log(new Template().events());

test/core/fixtures/transformation/spec.function-name/options.json

@@ -0,0 +1,3 @@
+{
+  "externalHelpers": true
+}