v5.0.9

babel-node --print: don't mangle percent characters (%)

CHANGELOG.md

@@ -13,6 +13,33 @@ _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.9
+
+ * **Polish**
+  * Use `moduleId` for UMD global name if available.
+ * **Bug Fix**
+  * Fix UMD global `module` variable shadowing the `amd`/`common` `module` variable.
+  * Fix Flow param type annotation regression.
+  * Fix function name collision `toString` wrapper. Thanks [@alawatthe](https://github.com/alawatthe)!
+
+## 5.0.8
+
+ * **Bug Fix**
+  * Fix falsy static class properties not being writable.
+  * Fix block scoping collisions not properly detecting modules and function clashes.
+  * Skip `this` before `super` for derived constructors on functions.
+
+## 5.0.7
+
+ * **New Feature**
+  * Add `--ignore` and `--only` support to the CLI.
+ * **Bug Fix**
+  * Remove `HOMEPATH` environment variable from home resolution in `babel/register` cache.
+ * **Internal**
+  * Disable WIP path resolution introducing infinite recursion in some code examples.
+ * **Polish**
+  * Add live binding to CommonJS default imports.
+
 ## 5.0.6
 
  * **Bug Fix**

package.json

@@ -1,7 +1,7 @@
 {
   "name": "babel-core",
   "description": "Turn ES6 code into readable vanilla ES5 with source maps",
-  "version": "5.0.7",
+  "version": "5.0.9",
   "author": "Sebastian McKenzie <sebmck@gmail.com>",
   "homepage": "https://babeljs.io/",
   "repository": "babel/babel",

packages/babel-cli/bin/_babel-node

@@ -4,6 +4,7 @@ var pathIsAbsolute = require("path-is-absolute");
 var commander      = require("commander");
 var Module         = require("module");
 var babel          = require("babel-core");
+var inspect        = require("util").inspect;
 var path           = require("path");
 var repl           = require("repl");
 var util           = require("babel-core").util;
@@ -71,7 +72,10 @@ if (program.eval || program.print) {
   global.require = module.require.bind(module);
 
   var result = _eval(code, global.__filename);
-  if (program.print) console.log(result);
+  if (program.print) {
+    var output = _.isString(result) ? result : inspect(result);
+    process.stdout.write(output + "\n");
+  }
 } else {
   if (program.args.length) {
     // slice all arguments up to the first filename since they're babel args that we handle

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.6",
+  "version": "5.0.8",
   "author": "Sebastian McKenzie <sebmck@gmail.com>",
   "homepage": "https://babeljs.io/",
   "repository": "babel/babel",
   "preferGlobal": true,
   "dependencies": {
     "chokidar": "^0.12.6",
-    "babel-core": "^5.0.6",
+    "babel-core": "^5.0.8",
     "commander": "^2.6.0",
     "fs-readdir-recursive": "^0.1.0",
     "output-file-sync": "^1.1.0",
@@ -22,4 +22,4 @@
     "babel-node": "./bin/babel-node",
     "babel-external-helpers": "./bin/babel-external-helpers"
   }
-}
+}

packages/babel-runtime/package.json

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

src/acorn/src/expression.js

@@ -250,7 +250,7 @@ pp.parseExprAtom = function(refShorthandDefaultPos) {
     return this.finishNode(node, type)
 
   case tt._yield:
-    if (this.inGenerator) unexpected()
+    if (this.inGenerator) this.unexpected()
 
   case tt._do:
     if (this.options.features["es7.doExpressions"]) {

src/acorn/src/lval.js

@@ -125,7 +125,9 @@ pp.parseBindingList = function(close, allowEmpty, allowTrailingComma) {
       this.expect(close)
       break
     } else {
-      elts.push(this.parseAssignableListItemTypes(this.parseMaybeDefault()))
+      var left = this.parseMaybeDefault()
+      this.parseAssignableListItemTypes(left)
+      elts.push(this.parseMaybeDefault(null, left))
     }
   }
   return elts

src/babel/transformation/modules/umd.js

@@ -47,8 +47,8 @@ export default class UMDFormatter extends AMDFormatter {
 
     // globals
 
-    var browserArgs = [t.memberExpression(t.identifier("module"), t.identifier("exports"))];
-    if (this.passModuleArg) browserArgs.push(t.identifier("module"));
+    var browserArgs = [];
+    if (this.passModuleArg) browserArgs.push(t.identifier("mod"));
 
     for (let name in this.ids) {
       var id = this.defaultIds[name] || t.identifier(t.toIdentifier(name));
@@ -60,12 +60,17 @@ export default class UMDFormatter extends AMDFormatter {
     var moduleName = this.getModuleName();
     if (moduleName) defineArgs.unshift(t.literal(moduleName));
 
+    //
+    var globalArg = this.file.opts.basename;
+    if (moduleName) globalArg = moduleName;
+    globalArg = t.identifier(t.toIdentifier(globalArg));
+
     var runner = util.template("umd-runner-body", {
       AMD_ARGUMENTS: defineArgs,
       COMMON_TEST: commonTests,
       COMMON_ARGUMENTS: commonArgs,
       BROWSER_ARGUMENTS: browserArgs,
-      GLOBAL_ARG: t.identifier(t.toIdentifier(this.file.opts.basename))
+      GLOBAL_ARG: globalArg
     });
 
     //

src/babel/transformation/templates/helper-create-class.js

@@ -4,7 +4,7 @@
       var descriptor = props[i];
       descriptor.enumerable = descriptor.enumerable || false;
       descriptor.configurable = true;
-      if (descriptor.value) descriptor.writable = true;
+      if ("value" in descriptor) descriptor.writable = true;
       Object.defineProperty(target, descriptor.key, descriptor);
     }
   }

src/babel/transformation/templates/property-method-assignment-wrapper-generator.js

@@ -4,7 +4,7 @@
   }
 
   FUNCTION_ID.toString = function () {
-    return FUNCTION_ID.toString();
+    return FUNCTION_KEY.toString();
   };
 
   return FUNCTION_ID;

src/babel/transformation/templates/property-method-assignment-wrapper.js

@@ -4,7 +4,7 @@
   }
 
   FUNCTION_ID.toString = function () {
-    return FUNCTION_ID.toString();
+    return FUNCTION_KEY.toString();
   }
 
   return FUNCTION_ID;

src/babel/transformation/templates/umd-runner-body.js

@@ -4,8 +4,8 @@
   } else if (COMMON_TEST) {
     factory(COMMON_ARGUMENTS);
   } else {
-    var module = { exports: {} };
-    factory(BROWSER_ARGUMENTS);
-    global.GLOBAL_ARG = module.exports;
+    var mod = { exports: {} };
+    factory(mod.exports, BROWSER_ARGUMENTS);
+    global.GLOBAL_ARG = mod.exports;
   }
 });

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

@@ -62,6 +62,18 @@ var verifyConstructorVisitor = traverse.explode({
     }
   },
 
+  FunctionDeclaration: {
+    enter() {
+      this.skip();
+    }
+  },
+
+  FunctionExpression: {
+    enter() {
+      this.skip();
+    }
+  },
+
   ThisExpression: {
     enter(node, parent, scope, state) {
       if (state.hasSuper && !state.hasBareSuper) {
@@ -440,7 +452,7 @@ class ClassTransformer {
       file:         this.file
     };
 
-    path.traverse(verifyConstructorVisitor, state);
+    path.get("value").traverse(verifyConstructorVisitor, state);
 
     this.bareSuper = state.bareSuper;
 

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

@@ -332,7 +332,7 @@ class DestructuringTransformer {
       if (t.isSpreadProperty(prop)) continue;
 
       var key = prop.key;
-      if (t.isIdentifier(key)) key = t.literal(prop.key.name);
+      if (t.isIdentifier(key) && !prop.computed) key = t.literal(prop.key.name);
       keys.push(key);
     }
 

src/babel/traversal/scope.js

@@ -237,12 +237,12 @@ export default class Scope {
     var local = this.getOwnBindingInfo(name);
     if (!local) return;
 
+
     if (kind === "param") return;
     if (kind === "hoisted" && local.kind === "let") return;
 
-
     var duplicate = false;
-    duplicate ||= local.kind === "let" || local.kind === "const" || local.kind === "module";
+    duplicate ||= kind === "let" || kind === "const" || local.kind === "let" || local.kind === "const" || local.kind === "module";
     duplicate ||= local.kind === "param" && (kind === "let" || kind === "const");
 
     if (duplicate) {
@@ -477,6 +477,15 @@ export default class Scope {
       this.traverse(path.get("body").node, blockVariableVisitor, this);
     }
 
+    // Program, Function - var variables
+
+    if (path.isProgram() || path.isFunction()) {
+      this.traverse(path.node, functionVariableVisitor, {
+        blockId: path.get("id").node,
+        scope:   this
+      });
+    }
+
     // Program, BlockStatement, Function - let variables
 
     if (path.isBlockStatement() || path.isProgram()) {
@@ -495,15 +504,6 @@ export default class Scope {
       this.registerBinding("let", path);
     }
 
-    // Program, Function - var variables
-
-    if (path.isProgram() || path.isFunction()) {
-      this.traverse(path.node, functionVariableVisitor, {
-        blockId: path.get("id").node,
-        scope:   this
-      });
-    }
-
     // Program
 
     if (path.isProgram()) {

test/acorn/tests-flow.js

@@ -10959,7 +10959,7 @@ var fbTestFixture = {
       }
     },
   },
-    'Bounded Polymorphism': {
+  'Bounded Polymorphism': {
     'class A<T: Foo> {}': {
       type: 'ClassDeclaration',
       id: {

test/core/fixtures/transformation/es6.destructuring/es7-object-rest/actual.js

@@ -1,4 +1,5 @@
 var z = {};
 var { ...x } = z;
 var { x, ...y } = z;
+var { [x]: x, ...y } = z;
 (function({ x, ...y }) { })

test/core/fixtures/transformation/es6.destructuring/es7-object-rest/expected.js

@@ -4,6 +4,8 @@ var z = {};
 var x = babelHelpers.objectWithoutProperties(z, []);
 var x = z.x;
 var y = babelHelpers.objectWithoutProperties(z, ["x"]);
+var x = z[x];
+var y = babelHelpers.objectWithoutProperties(z, [x]);
 
 (function (_ref) {
   var x = _ref.x;

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

@@ -4,11 +4,11 @@
   } else if (typeof exports !== "undefined" && typeof module !== "undefined") {
     factory(exports, module);
   } else {
-    var module = {
+    var mod = {
       exports: {}
     };
-    factory(module.exports, module);
-    global.actual = module.exports;
+    factory(mod.exports, mod);
+    global.actual = mod.exports;
   }
 })(this, function (exports, module) {
   "use strict";
@@ -41,4 +41,4 @@
 
   module.exports = Foo;
   module.exports = foo;
-});
+});

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

@@ -4,11 +4,11 @@
   } else if (typeof exports !== "undefined") {
     factory(exports, require("foo"));
   } else {
-    var module = {
+    var mod = {
       exports: {}
     };
-    factory(module.exports, global.foo);
-    global.actual = module.exports;
+    factory(mod.exports, global.foo);
+    global.actual = mod.exports;
   }
 })(this, function (exports, _foo) {
   "use strict";

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

@@ -4,11 +4,11 @@
   } else if (typeof exports !== "undefined") {
     factory(exports);
   } else {
-    var module = {
+    var mod = {
       exports: {}
     };
-    factory(module.exports);
-    global.actual = module.exports;
+    factory(mod.exports);
+    global.actual = mod.exports;
   }
 })(this, function (exports) {
   "use strict";
@@ -23,4 +23,4 @@
   exports["default"] = foo;
   exports["default"] = foo;
   exports.bar = bar;
-});
+});

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

@@ -4,11 +4,11 @@
   } else if (typeof exports !== "undefined") {
     factory(exports);
   } else {
-    var module = {
+    var mod = {
       exports: {}
     };
-    factory(module.exports);
-    global.actual = module.exports;
+    factory(mod.exports);
+    global.actual = mod.exports;
   }
 })(this, function (exports) {
   "use strict";
@@ -43,4 +43,4 @@
   };
 
   exports.foo8 = foo8;
-});
+});

test/core/fixtures/transformation/es6.modules-umd/get-module-name-option/expected.js

@@ -4,12 +4,12 @@
   } else if (typeof exports !== "undefined") {
     factory(exports);
   } else {
-    var module = {
+    var mod = {
       exports: {}
     };
-    factory(module.exports);
-    global.actual = module.exports;
+    factory(mod.exports);
+    global.myCustomModuleName = mod.exports;
   }
 })(this, function (exports) {
   "use strict";
-});
+});

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

@@ -4,11 +4,11 @@
   } else if (typeof exports !== "undefined") {
     factory(exports, require("./evens"));
   } else {
-    var module = {
+    var mod = {
       exports: {}
     };
-    factory(module.exports, global.evens);
-    global.actual = module.exports;
+    factory(mod.exports, global.evens);
+    global.actual = mod.exports;
   }
 })(this, function (exports, _evens) {
   "use strict";
@@ -28,4 +28,4 @@
     };
   })(_evens.isEven);
   exports.isOdd = isOdd;
-});
+});

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

@@ -4,11 +4,11 @@
   } else if (typeof exports !== "undefined") {
     factory(exports, require("foo"));
   } else {
-    var module = {
+    var mod = {
       exports: {}
     };
-    factory(module.exports, global.foo);
-    global.actual = module.exports;
+    factory(mod.exports, global.foo);
+    global.actual = mod.exports;
   }
 })(this, function (exports, _foo) {
   "use strict";
@@ -21,4 +21,4 @@
 
   _foo2;
   _foo22;
-});
+});

test/core/fixtures/transformation/es6.modules-umd/imports-glob/expected.js

@@ -4,14 +4,14 @@
   } else if (typeof exports !== "undefined") {
     factory(exports, require("foo"));
   } else {
-    var module = {
+    var mod = {
       exports: {}
     };
-    factory(module.exports, global.foo);
-    global.actual = module.exports;
+    factory(mod.exports, global.foo);
+    global.actual = mod.exports;
   }
 })(this, function (exports, _foo) {
   "use strict";
 
   _foo;
-});
+});

test/core/fixtures/transformation/es6.modules-umd/imports-mixing/expected.js

@@ -4,11 +4,11 @@
   } else if (typeof exports !== "undefined") {
     factory(exports, require("foo"));
   } else {
-    var module = {
+    var mod = {
       exports: {}
     };
-    factory(module.exports, global.foo);
-    global.actual = module.exports;
+    factory(mod.exports, global.foo);
+    global.actual = mod.exports;
   }
 })(this, function (exports, _foo) {
   "use strict";
@@ -18,4 +18,4 @@
   var _foo2 = _interopRequire(_foo);
 
   _foo.baz;
-});
+});

test/core/fixtures/transformation/es6.modules-umd/imports-named/expected.js

@@ -4,11 +4,11 @@
   } else if (typeof exports !== "undefined") {
     factory(exports, require("foo"));
   } else {
-    var module = {
+    var mod = {
       exports: {}
     };
-    factory(module.exports, global.foo);
-    global.actual = module.exports;
+    factory(mod.exports, global.foo);
+    global.actual = mod.exports;
   }
 })(this, function (exports, _foo) {
   "use strict";
@@ -19,4 +19,4 @@
   _foo.bar;
   _foo.bar;
   _foo.xyz;
-});
+});

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

@@ -4,12 +4,12 @@
   } else if (typeof exports !== "undefined") {
     factory(exports, require("foo"), require("foo-bar"), require("./directory/foo-bar"));
   } else {
-    var module = {
+    var mod = {
       exports: {}
     };
-    factory(module.exports, global.foo, global.fooBar, global.directoryFooBar);
-    global.actual = module.exports;
+    factory(mod.exports, global.foo, global.fooBar, global.directoryFooBar);
+    global.actual = mod.exports;
   }
 })(this, function (exports, _foo, _fooBar, _directoryFooBar) {
   "use strict";
-});
+});

test/core/fixtures/transformation/es6.modules-umd/module-id/actual.js

@@ -0,0 +1 @@
+foobar();

test/core/fixtures/transformation/es6.modules-umd/module-id/expected.js

@@ -0,0 +1,17 @@
+(function (global, factory) {
+  if (typeof define === "function" && define.amd) {
+    define("MyLib", ["exports"], factory);
+  } else if (typeof exports !== "undefined") {
+    factory(exports);
+  } else {
+    var mod = {
+      exports: {}
+    };
+    factory(mod.exports);
+    global.MyLib = mod.exports;
+  }
+})(this, function (exports) {
+  "use strict";
+
+  foobar();
+});

test/core/fixtures/transformation/es6.modules-umd/module-id/options.json

@@ -0,0 +1,3 @@
+{
+  "moduleId": "MyLib"
+}

test/core/fixtures/transformation/es6.modules-umd/module-name/expected.js

@@ -4,14 +4,14 @@
   } else if (typeof exports !== "undefined") {
     factory(exports);
   } else {
-    var module = {
+    var mod = {
       exports: {}
     };
-    factory(module.exports);
-    global.actual = module.exports;
+    factory(mod.exports);
+    global.es6ModulesUmdModuleNameExpected = mod.exports;
   }
 })(this, function (exports) {
   "use strict";
 
   foobar();
-});
+});

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

@@ -4,11 +4,11 @@
   } else if (typeof exports !== "undefined") {
     factory(exports, require("foo"), require("foo-bar"), require("./directory/foo-bar"));
   } else {
-    var module = {
+    var mod = {
       exports: {}
     };
-    factory(module.exports, global.foo2, global.fooBar, global.directoryFooBar);
-    global.actual = module.exports;
+    factory(mod.exports, global.foo2, global.fooBar, global.directoryFooBar);
+    global.actual = mod.exports;
   }
 })(this, function (exports, _foo, _fooBar, _directoryFooBar) {
   "use strict";
@@ -29,4 +29,4 @@
 
   _foo.bar;
   _foo.foo;
-});
+});

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

@@ -4,11 +4,11 @@
   } else if (typeof exports !== "undefined") {
     factory(exports);
   } else {
-    var module = {
+    var mod = {
       exports: {}
     };
-    factory(module.exports);
-    global.actual = module.exports;
+    factory(mod.exports);
+    global.actual = mod.exports;
   }
 })(this, function (exports) {
   "use strict";
@@ -42,4 +42,4 @@
   exports.f = d;
 
   exports.f = exports.e = d = 4;
-});
+});

test/core/fixtures/transformation/es7.class-properties/static/exec.js

@@ -0,0 +1,9 @@
+class Foo {
+  static num = 0;
+  static str = "foo";
+}
+
+assert.equal(Foo.num, 0);
+assert.equal(Foo.num = 1, 1);
+assert.equal(Foo.str, "foo");
+assert.equal(Foo.str = "bar", "bar");

test/core/fixtures/transformation/flow/strip-type-annotations/actual.js

@@ -89,7 +89,7 @@ var a: Promise<bool>[]
 var a:(...rest:Array<number>) => number
 var identity: <T>(x: T) => T
 var identity: <T>(x: T, ...y:T[]) => T
-import type foo from "bar";
+import type foo4 from "bar";
 import type { foo2, bar } from "baz";
 import type { foo as bar2 } from "baz";
 import type from "foo";

test/core/fixtures/transformation/misc/options.json

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

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

@@ -0,0 +1,9 @@
+class Foo extends Bar {
+  constructor (options) {
+    let parentOptions = {};
+    parentOptions.init = function () {
+      this;
+    };
+    super(parentOptions);
+  }
+}

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

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

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

@@ -0,0 +1,3 @@
+function test(x: string = "hi"): string {
+  return x;
+}

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

@@ -0,0 +1,7 @@
+"use strict";
+
+function test() {
+  var x = arguments[0] === undefined ? "hi" : arguments[0];
+
+  return x;
+}

test/core/fixtures/transformation/misc/regression-lodash-tostring-import/expected.js

@@ -1,9 +1,7 @@
 "use strict";
 
-var _interopRequireWildcard = function (obj) { return obj && obj.__esModule ? obj : { "default": obj }; };
-
 var _toString = require("foo");
 
-var _toString2 = _interopRequireWildcard(_toString);
+var _toString2 = babelHelpers.interopRequireWildcard(_toString);
 
-_toString2["default"];
+_toString2["default"];

test/core/fixtures/transformation/runtime/modules-umd/expected.js

@@ -4,14 +4,14 @@
   } else if (typeof exports !== "undefined") {
     factory(exports, require("foo"), require("babel-runtime/helpers/interop-require"));
   } else {
-    var module = {
+    var mod = {
       exports: {}
     };
-    factory(module.exports, global.foo, global._interopRequire);
-    global.actual = module.exports;
+    factory(mod.exports, global.foo, global._interopRequire);
+    global.actual = mod.exports;
   }
 })(this, function (exports, _foo, _babelRuntimeHelpersInteropRequire) {
   "use strict";
 
   var _foo2 = _babelRuntimeHelpersInteropRequire["default"](_foo);
-});
+});

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

@@ -6,7 +6,7 @@ var i = (function (_i) {
   }
 
   i.toString = function () {
-    return i.toString();
+    return _i.toString();
   };
 
   return i;
@@ -17,4 +17,4 @@ var i = (function (_i) {
 var j = function j() {
   var _ = 5;
   j = _.j;
-};
+};

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

@@ -13,7 +13,7 @@ var obj = {
     }
 
     h.toString = function () {
-      return h.toString();
+      return _h.toString();
     };
 
     return h;
@@ -24,4 +24,4 @@ var obj = {
   m: function m() {
     doSmth();
   }
-};
+};

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

@@ -10,7 +10,7 @@ var f = (function (_f) {
   }
 
   f.toString = function () {
-    return f.toString();
+    return _f.toString();
   };
 
   return f;
@@ -23,9 +23,9 @@ var obj = {
     }
 
     f.toString = function () {
-      return f.toString();
+      return _f2.toString();
     };
 
     return f;
   })(function (f) {})
-};
+};

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

@@ -6,10 +6,10 @@ var f = (function (_f) {
   }
 
   f.toString = function () {
-    return f.toString();
+    return _f.toString();
   };
 
   return f;
 })(function () {
   console.log(f, g);
-});
+});