v1.12.12

This reverts commit fdd1451d53.

CHANGELOG.md

@@ -1,3 +1,30 @@
+# 1.12.12
+
+ * Make scope tracker more reliable to handle all edgecases.
+
+# 1.12.11
+
+ * Block scope classes.
+ * Fix generation of integer `Literal`s in `MemberExpression`.
+
+# 1.12.10
+
+ * Fix let scoping var hoisting.
+
+# 1.12.9
+
+ * Escape unicode characters when generating string `Literal`s.
+ * Fix semicolons being output for statements in `ExportDeclaration`.
+ * Fix `WithStatement` missing parenthesis.
+
+# 1.12.8
+
+ * Temporarily forbid `AssignmentExpression` destructuring outside of `ExpressionStatement`.
+
+# 1.12.7
+
+ * Update to latest `acorn-6to5`.
+
 # 1.12.6
 
  * Update to latest `acorn-6to5`.

CONTRIBUTING.md

@@ -0,0 +1,31 @@
+# Contributing
+
+ * **General**
+   * No ES6 syntax features or methods, exclusively ES5.
+   * Max of five arguments for functions
+   * Max depth of four nested blocks
+   * 2-spaced soft tabs
+ * **Naming**
+   * CamelCase all class names
+   * camelBack all variable names
+ * **Spacing**
+   * 80 character line max
+   * Spaces after all keywords
+   * Spaces before all left curly braces
+ * **Comments**
+   * Use JSDoc-style comments for methods
+   * Single-line comments for ambiguous code
+ * **Quotes**
+   * Always use double quotes
+   * Only use single quotes when the string contains a double quote
+ * **Declaration**
+   * No unused variables
+   * No pollution of global variables and prototypes
+
+## Testing
+
+    $ make test
+
+## Linting
+
+    $ make lint

README.md

@@ -24,6 +24,6 @@
   </a>
 </p>
 
-**6to5** turns ES6 code into vanilla ES5, so you can use ES6 features **today.**
+**6to5** turns ES6+ code into vanilla ES5, so you can use next generation features **today.**
 
 For more information view the [documentation](https://6to5.github.io).

doc/browser.md

@@ -3,12 +3,6 @@
 A browser version of 6to5 is available from `browser.js` inside the 6to5
 directory in an npm release.
 
-## API
-
-```javascript
-to5.transform("class Test {}").code;
-```
-
 ## Scripts
 
 While it's not recommended for serious use, when the browser version is included
@@ -47,3 +41,21 @@ To test 6to5 in your browser run:
     $ make test-browser
 
 And open `test/browser.html` in your browser if it doesn't open automatically.
+
+## API
+
+### to5.transform(code, [opts])
+
+See [options](usage.md#options) for additional documentation.
+
+```javascript
+to5.transform("class Test {}").code;
+```
+
+### to5.run(code, [opts])
+
+See [options](usage.md#options) for additional documentation.
+
+```javascript
+to5.run("class Test {}");
+```

doc/caveats.md

@@ -1,5 +1,10 @@
 # Caveats
 
+## Async/Generators
+
+The [regenerator runtime](https://github.com/facebook/regenerator/blob/master/runtime.js)
+and an [ES6 polyfill](polyfill.md) are required in order for generators to work.
+
 ## Classes
 
 Built-in classes such as `Date`, `Array` and `DOM` cannot be subclassed due to
@@ -32,11 +37,6 @@ A polyfill is required for for-of functionality that implements `Symbol` and
 adds `prototype[Symbol.iterator]` behaviour to built-ins. Using the polyfills
 specified in [polyfill](polyfill.md) suffices.
 
-## Generators
-
-The [regenerator runtime](https://github.com/facebook/regenerator/blob/master/runtime.js)
-and an [ES6 polyfill](polyfill.md) are required in order for generators to work.
-
 ## Spread
 
 An [ES6 polyfill](polyfill.md) is required in order for spread to work. More

doc/differences.md

@@ -64,7 +64,7 @@ better suited if you'd like a full ES6 environment with polyfills and all.
 | Browser support              | ✓    | ✓       | ✓      |        |                |             |
 | Array comprehension          | ✓    | ✓       |        |        | ✓              |             |
 | Arrow functions              | ✓    | ✓       | ✓      | ✓      | ✓              | ✓           |
-| Async functions              | ✓    |         | ✓      |        |                |             |
+| Async functions              | ✓    | ✓       | ✓      |        |                |             |
 | Classes                      | ✓    | ✓       | ✓      | ✓      | ✓              | ✓           |
 | Computed property names      | ✓    | ✓       | ✓      | ✓      | ✓              |             |
 | Constants                    | ✓    | ✓       |        |        | ✓              |             |

doc/features.md

@@ -3,7 +3,32 @@
 ## Array comprehension
 
 ```javascript
-[for (i of [1, 2, 3]) i * i]; // [1, 4, 9]
+var results = [for (c of customers) if (c.city == "Seattle") { name: c.name, age: c.age }]
+```
+
+## Arrow functions
+
+```javascript
+// Expression bodies
+var odds = evens.map(v => v + 1);
+var nums = evens.map((v, i) => v + i);
+
+// Statement bodies
+nums.forEach(v => {
+  if (v % 5 === 0)
+    fives.push(v);
+});
+
+// Lexical this
+var bob = {
+  _name: "Bob",
+  _friends: [],
+  printFriends() {
+    this._friends.forEach(f => {
+      console.log(this._name + " knows " + f);
+    });
+  }
+};
 ```
 
 ## Async functions
@@ -16,31 +41,27 @@ async function chainAnimationsAsync(elem, animations) {
 }
 ```
 
-## Arrow functions
-
-```javascript
-arr.map(x => x * x);
-```
-
-## Let scoping
-
-```javascript
-for (let i in arr) {
-  let v = arr[i];
-}
-```
-
 ## Classes
 
 ```javascript
-class Foo extends Bar {
-  constructor() { }
+class SkinnedMesh extends THREE.Mesh {
+  constructor(geometry, materials) {
+    super(geometry, materials);
 
-  foo() { }
+    this.idMatrix = SkinnedMesh.defaultMatrix();
+    this.bones = [];
+    this.boneMatrices = [];
+    //...
+  }
 
-  get bar() { }
+  update(camera) {
+    //...
+    super.update();
+  }
 
-  set bar() { }
+  static defaultMatrix() {
+    return new THREE.Matrix4();
+  }
 }
 ```
 
@@ -70,16 +91,35 @@ var MULTIPLIER; // error
 ## Default parameters
 
 ```javascript
-function foo(bar = "foo") {
-  return bar + "bar";
+function f(x, y = 12) {
+  // y is 12 if not passed (or passed as undefined)
+  return x + y;
 }
+f(3) == 15
 ```
 
 ## Destructuring
 
 ```javascript
-var [a, [b], c, d] = ["hello", [", ", "junk"], ["world"]];
-console.log(a + b + c); // hello, world
+// list matching
+var [a, , b] = [1,2,3];
+
+// object matching
+var { op: a, lhs: { op: b }, rhs: c } = getASTNode();
+
+// object matching shorthand
+// binds `op`, `lhs` and `rhs` in scope
+var { op, lhs, rhs } = getASTNode();
+
+// Can be used in parameter position
+function g({ name: x }) {
+  console.log(x);
+}
+g({ name: 5 });
+
+// Fail-soft destructuring
+var [a] = [];
+a === undefined;
 ```
 
 ## For-of
@@ -100,6 +140,14 @@ for (var i of [1, 2, 3]) {
 ```javascript
 ```
 
+## Let scoping
+
+```javascript
+for (let i in arr) {
+  let v = arr[i];
+}
+```
+
 ## Modules
 
 ```javascript
@@ -141,23 +189,21 @@ function f(x, y) {
 ## Rest parameters
 
 ```javascript
-function printList(name, ...items) {
-  console.log("list %s has the following items", name);
-  items.forEach(function (item) {
-    console.log(item);
-  });
+function f(x, ...y) {
+  // y is an Array
+  return x * y.length;
 }
+f(3, "hello", true) == 6
 ```
 
 ## Spread
 
 ```javascript
-function add(x, y) {
-  return x + y;
+function f(x, y, z) {
+  return x + y + z;
 }
-
-var numbers = [5, 10];
-add(...numbers); // 15
+// Pass each elem of array as argument
+f(...[1,2,3]) == 6
 ```
 
 ## Template literals

doc/index.md

@@ -1,4 +1,4 @@
-**6to5** turns ES6 code into vanilla ES5, so you can use ES6 features **today.**
+**6to5** turns ES6+ code into vanilla ES5, so you can use next generation features **today.**
 
  - **Readable** - formatting is retained if possible so your generated code is as similar as possible.
  - **Extensible** - with a large range of [plugins](plugins.md) and **browser support**.
@@ -11,6 +11,24 @@ It's as easy as:
 
     $ npm install -g 6to5
 
+## Usage
+
+Once you've installed 6to5, there are multiple paths you can take depending on
+how you want to use it.
+
+6to5 will simply compile your ES6+ script to ES5 if you pass it as an argument
+to the command-line tool `6to5`:
+
+    $ 6to5 script.js
+
+If you have a file written using ES6+ and you just want to run it, `6to5-node`
+has you covered:
+
+    $ 6to5-node script.js
+
+And it doesn't end here! To see all the ways you can use 6to5, check out the
+[Usage](http://6to5.github.io/usage.html) page.
+
 ## [Features](features.md)
 
  - [Array comprehension](features.md#array-comprehension)

doc/usage.md

@@ -50,16 +50,28 @@ Compile and run `test.js`.
 
 ```javascript
 var to5 = require("6to5");
+```
 
+### to5.transform(code, [opts]);
+
+```javascript
 var result = to5.transform("code();", options);
 result.code;
 result.map;
 result.ast;
+```
 
+### to5.transformFileSync(filename, [opts])
+
+```javascript
 to5.transformFileSync("filename.js", options).code;
+```
 
+### to5.transformFile(filename, [opts], callback)
+
+```javascript
 to5.transformFile("filename.js", options, function (err, result) {
-
+  result.code;
 });
 ```
 

lib/6to5/browser.js

@@ -2,13 +2,6 @@ var transform = module.exports = require("./transformation/transform");
 
 transform.transform = transform;
 
-transform.eval = function (code, opts) {
-  opts = opts || {};
-  opts.filename = opts.filename || "eval";
-  opts.sourceMap = "inline";
-  return eval(transform(code, opts).code);
-};
-
 transform.run = function (code, opts) {
   opts = opts || {};
   opts.sourceMap = "inline";

lib/6to5/generation/generators/classes.js

@@ -30,3 +30,11 @@ exports.ClassBody = function (node, print) {
     this.rightBrace();
   }
 };
+
+exports.MethodDefinition = function (node, print) {
+  if (node.static) {
+    this.push("static ");
+  }
+
+  this._method(node, print);
+};

lib/6to5/generation/generators/expressions.js

@@ -1,4 +1,5 @@
-var t = require("../../types");
+var util = require("../../util");
+var t    = require("../../types");
 
 exports.UnaryExpression = function (node, print) {
   var hasSpace = /[a-z]$/.test(node.operator);
@@ -105,6 +106,11 @@ exports.MemberExpression = function (node, print) {
     print(node.property);
     this.push("]");
   } else {
+    // 5..toFixed(2);
+    if (t.isLiteral(node.object) && util.isInteger(node.object.value)) {
+      this.push(".");
+    }
+
     this.push(".");
     print(node.property);
   }

lib/6to5/generation/generators/jsx.js

@@ -74,5 +74,5 @@ exports.XJSClosingElement = function (node, print) {
 };
 
 exports.XJSEmptyExpression = function () {
-  this.push("null");
+
 };

lib/6to5/generation/generators/methods.js

@@ -41,6 +41,8 @@ exports._method = function (node, print) {
     this.push(kind + " ");
   }
 
+  if (value.async) this.push("async ");
+
   if (node.computed) {
     this.push("[");
     print(key);
@@ -54,16 +56,9 @@ exports._method = function (node, print) {
   print(value.body);
 };
 
-exports.MethodDefinition = function (node, print) {
-  if (node.static) {
-    this.push("static ");
-  }
-
-  this._method(node, print);
-};
-
 exports.FunctionDeclaration =
 exports.FunctionExpression = function (node, print) {
+  if (node.async) this.push("async ");
   this.push("function");
   if (node.generator) this.push("*");
   this.space();
@@ -74,6 +69,8 @@ exports.FunctionExpression = function (node, print) {
 };
 
 exports.ArrowFunctionExpression = function (node, print) {
+  if (node.async) this.push("async ");
+
   if (node.params.length === 1 && !node.defaults.length && !node.rest && t.isIdentifier(node.params[0])) {
     print(node.params[0]);
   } else {

lib/6to5/generation/generators/modules.js

@@ -25,9 +25,10 @@ exports.ExportDeclaration = function (node, print) {
 
   if (node.declaration) {
     print(node.declaration);
+    if (t.isStatement(node.declaration)) return;
   } else {
     if (specifiers.length === 1 && t.isExportBatchSpecifier(specifiers[0])) {
-      this.push("*");
+      print(specifiers[0]);
     } else {
       this.push("{");
       if (specifiers.length) {

lib/6to5/generation/generators/statements.js

@@ -4,6 +4,7 @@ exports.WithStatement = function (node, print) {
   this.keyword("with");
   this.push("(");
   print(node.object);
+  this.push(")");
   print.block(node.body);
 };
 
@@ -134,15 +135,6 @@ exports.SwitchStatement = function (node, print) {
   this.push(") {");
   print.sequence(node.cases, { indent: true });
   this.push("}");
-
-  //if (node.cases.length) {
-  //  this.newline();
-  //  print.sequence(node.cases, { indent: true });
-  //  this.newline();
-  //  this.rightBrace();
-  //} else {
-  //  this.push("}");
-  //}
 };
 
 exports.SwitchCase = function (node, print) {
@@ -154,13 +146,8 @@ exports.SwitchCase = function (node, print) {
     this.push("default:");
   }
 
-  if (node.consequent.length === 1) {
-    this.space();
-    print(node.consequent[0]);
-  } else if (node.consequent.length > 1) {
-    this.newline();
-    print.sequence(node.consequent, { indent: true });
-  }
+  this.space();
+  print.sequence(node.consequent, { indent: true });
 };
 
 exports.DebuggerStatement = function () {

lib/6to5/generation/generators/types.js

@@ -74,13 +74,20 @@ exports.Literal = function (node) {
   var val  = node.value;
   var type = typeof val;
 
-  if (type === "boolean" || type === "number" || type === "string") {
+  if (type === "string") {
+    val = JSON.stringify(val);
+
+    // escape unicode characters
+    val = val.replace(/[\u007f-\uffff]/g, function(c) {
+      return "\\u" + ("0000" + c.charCodeAt(0).toString(16)).slice(-4);
+    });
+
+    this.push(val);
+  } else if (type === "boolean" || type === "number") {
     this.push(JSON.stringify(val));
   } else if (node.regex) {
     this.push("/" + node.regex.pattern + "/" + node.regex.flags);
   } else if (val === null) {
     this.push("null");
-  } else if (node.raw) {
-    this.push(node.raw);
   }
 };

lib/6to5/generation/node/index.js

@@ -2,8 +2,8 @@ module.exports = Node;
 
 var whitespace = require("./whitespace");
 var parens     = require("./parentheses");
-var t           = require("../../types");
-var _           = require("lodash");
+var t          = require("../../types");
+var _          = require("lodash");
 
 var find = function (obj, node, parent) {
   var result;
@@ -24,17 +24,14 @@ function Node(node, parent) {
 }
 
 Node.prototype.isUserWhitespacable = function () {
-  //var parent = this.parent;
+  var parent = this.parent;
   var node = this.node;
 
-  if (t.isUserWhitespacable(node)) {
+  if (t.isUserWhitespacable(node) ||
+      t.isSequenceExpression(parent)) {
     return true;
   }
 
-  //if (t.isArrayExpression(parent)) {
-  //  return true;
-  //}
-
   return false;
 };
 

lib/6to5/runtime.js

@@ -5,7 +5,7 @@ var t         = require("./types");
 var _         = require("lodash");
 
 module.exports = function (namespace) {
-  namespace = t.identifier(namespace || "to5Runtime");
+  namespace = t.identifier(t.toIdentifier(namespace || "to5Runtime"));
 
   var body      = [];
   var container = t.functionExpression(null, [], t.blockStatement(body));

lib/6to5/transformation/transform.js

@@ -44,8 +44,8 @@ _.each({
   defaultParameters:        require("./transformers/default-parameters"),
   restParameters:           require("./transformers/rest-parameters"),
   destructuring:            require("./transformers/destructuring"),
-  letScoping:               require("./transformers/let-scoping"),
   forOf:                    require("./transformers/for-of"),
+  letScoping:               require("./transformers/let-scoping"),
   unicodeRegex:             require("./transformers/unicode-regex"),
 
   react:                    require("./transformers/react"),

lib/6to5/transformation/transformers/classes.js

@@ -4,7 +4,7 @@ var t        = require("../../types");
 var _        = require("lodash");
 
 exports.ClassDeclaration = function (node, parent, file, scope) {
-  return t.variableDeclaration("var", [
+  return t.variableDeclaration("let", [
     t.variableDeclarator(node.id, new Class(node, file, scope).run())
   ]);
 };

lib/6to5/transformation/transformers/destructuring.js

@@ -146,6 +146,12 @@ exports.ExpressionStatement = function (node, parent, file, scope) {
   return nodes;
 };
 
+exports.AssignmentExpression = function (node, parent, file) {
+  if (parent.type === "ExpressionStatement") return;
+  if (!t.isPattern(node.left)) return;
+  throw file.errorWithNode(node, "AssignmentExpression destructuring outside of a ExpressionStatement is forbidden due to current 6to5 limitations");
+};
+
 exports.VariableDeclaration = function (node, parent, file, scope) {
   if (t.isForInStatement(parent) || t.isForOfStatement(parent)) return;
 

lib/6to5/transformation/transformers/for-of.js

@@ -5,17 +5,17 @@ exports.ForOfStatement = function (node, parent, file, scope) {
   var left = node.left;
   var declar;
 
-  var stepKey = t.identifier(file.generateUid("step", scope));
-  var stepValueId = t.memberExpression(stepKey, t.identifier("value"));
+  var stepKey   = t.identifier(file.generateUid("step", scope));
+  var stepValue = t.memberExpression(stepKey, t.identifier("value"));
 
   if (t.isIdentifier(left)) {
-    declar = t.expressionStatement(t.assignmentExpression("=", left, stepValueId));
+    declar = t.expressionStatement(t.assignmentExpression("=", left, stepValue));
   } else if (t.isVariableDeclaration(left)) {
     declar = t.variableDeclaration(left.kind, [
-      t.variableDeclarator(left.declarations[0].id, stepValueId)
+      t.variableDeclarator(left.declarations[0].id, stepValue)
     ]);
   } else {
-    return;
+    throw file.errorWithNode(left, "Unknown node type " + left.type + " in ForOfStatement");
   }
 
   var node2 = util.template("for-of", {

lib/6to5/transformation/transformers/let-scoping.js

@@ -132,7 +132,7 @@ LetScoping.prototype.run = function () {
 
 /**
  * There are no let references accessed within a closure so we can just traverse
- * through this block and replace all references that exist in a high scope to
+ * through this block and replace all references that exist in a higher scope to
  * their uids.
  */
 
@@ -211,7 +211,8 @@ LetScoping.prototype.getInfo = function () {
 /**
  * If we're inside of a `For*Statement` then traverse it and check if it has one
  * of the following node types `ReturnStatement`, `BreakStatement`,
- * `ContinueStatement` and replace it with a return value we can track later on.
+ * `ContinueStatement` and replace it with a return value that we can track
+ * later on.
  *
  * @returns {Object}
  */
@@ -251,7 +252,7 @@ LetScoping.prototype.checkFor = function () {
 
 /**
  * Hoist all var declarations in this block to before it so they retain scope
- * once we wrap everything is in a closure.
+ * once we wrap everything in a closure.
  */
 
 LetScoping.prototype.hoistVarDeclarations = function () {
@@ -340,7 +341,7 @@ LetScoping.prototype.getLetReferences = function () {
  * @returns {Array}
  */
 
-LetScoping.prototype.buildPushDeclar = function (node) {
+LetScoping.prototype.pushDeclar = function (node) {
   this.body.push(t.variableDeclaration(node.kind, node.declarations.map(function (declar) {
     return t.variableDeclarator(declar.id);
   })));

lib/6to5/traverse/scope.js

@@ -4,6 +4,8 @@ var traverse = require("./index");
 var t        = require("../types");
 var _        = require("lodash");
 
+var FOR_KEYS = ["left", "init"];
+
 function Scope(parent, block) {
   this.parent = parent;
   this.block  = block;
@@ -19,29 +21,59 @@ Scope.prototype.getIds = function () {
   var self = this;
   var ids  = block._scopeIds = {};
 
-  if (t.isBlockStatement(block)) {
-    _.each(block.body, function (node) {
-      if (t.isVariableDeclaration(node) && node.kind !== "var") {
-        self.add(node, ids);
-      }
-    });
-  } else if (t.isProgram(block) || t.isFunction(block)) {
-    traverse(block, function (node, parent) {
-      if (parent !== block && t.isVariableDeclaration(node) && node.kind !== "var") {
-        return;
-      }
+  // ForStatement - left, init
 
-      if (t.isDeclaration(node)) {
-        self.add(node, ids);
-      } else if (t.isFunction(node)) {
-        return false;
-      }
+  if (t.isFor(block)) {
+    _.each(FOR_KEYS, function (key) {
+      var node = block[key];
+      if (t.isLet(node)) self.add(node, ids);
     });
-  } else if (t.isCatchClause(block)) {
+
+    block = block.body;
+  }
+
+  // Program, BlockStatement - let variables
+
+  if (t.isBlockStatement(block) || t.isProgram(block)) {
+    _.each(block.body, function (node) {
+      // check for non-var `VariableDeclaration`s
+      if (t.isLet(node)) self.add(node, ids);
+    });
+  }
+
+  // CatchClause - param
+
+  if (t.isCatchClause(block)) {
     self.add(block.param, ids);
   }
 
+  // Program, Function - var variables
+
+  if (t.isProgram(block) || t.isFunction(block)) {
+    traverse(block, function (node) {
+      if (t.isFor(node)) {
+        _.each(FOR_KEYS, function (key) {
+          var declar = node[key];
+          if (t.isVar(declar)) self.add(declar, ids);
+        });
+      }
+
+      // this block is a function so we'll stop since none of the variables
+      // declared within are accessible
+      if (t.isFunction(node)) return false;
+
+      // we've ran into a declaration!
+      // we'll let the BlockStatement scope deal with `let` declarations
+      if (t.isDeclaration(node) && !t.isLet(node)) {
+        self.add(node, ids);
+      }
+    });
+  }
+
+  // Function - params
+
   if (t.isFunction(block)) {
+    this.add(block.rest, ids);
     _.each(block.params, function (param) {
       self.add(param, ids);
     });
@@ -51,6 +83,7 @@ Scope.prototype.getIds = function () {
 };
 
 Scope.prototype.add = function (node, ids) {
+  if (!node) return;
   _.merge(ids || this.ids, t.getIds(node, true));
 };
 

lib/6to5/types/alias-keys.json

@@ -35,12 +35,13 @@
   "ClassDeclaration": ["Statement", "Declaration", "Class"],
   "ClassExpression":  ["Class"],
 
-  "ForOfStatement": ["Statement", "For"],
-  "ForInStatement": ["Statement", "For"],
-  "ForStatement":   ["Statement", "For"],
+  "ForOfStatement": ["Statement", "For", "Scope"],
+  "ForInStatement": ["Statement", "For", "Scope"],
+  "ForStatement":   ["Statement", "For", "Scope"],
 
   "ObjectPattern": ["Pattern"],
   "ArrayPattern":  ["Pattern"],
 
-  "Property": ["UserWhitespacable"]
+  "Property": ["UserWhitespacable"],
+  "XJSElement": ["UserWhitespacable"]
 }

lib/6to5/types/index.js

@@ -106,6 +106,10 @@ t.ensureBlock = function (node) {
 };
 
 t.toStatement = function (node, ignore) {
+  if (t.isStatement(node)) {
+    return node;
+  }
+
   var mustHaveId = false;
   var newType;
 
@@ -115,8 +119,6 @@ t.toStatement = function (node, ignore) {
   } else if (t.isFunction(node)) {
     mustHaveId = true;
     newType = "FunctionDeclaration";
-  } else if (t.isStatement(node)) {
-    newType = node.type;
   }
 
   if (mustHaveId && !node.id) {
@@ -183,6 +185,14 @@ t.getIds = function (node, map, ignoreTypes) {
   return ids;
 };
 
+t.isLet = function (node) {
+  return t.isVariableDeclaration(node) && (node.kind !== "var" || node._let);
+};
+
+t.isVar = function (node) {
+  return t.isVariableDeclaration(node, { kind: "var" }) && !node._let;
+};
+
 t.getIds.nodes = {
   AssignmentExpression: "left",
   ImportSpecifier: "id",

lib/6to5/util.js

@@ -17,6 +17,10 @@ exports.canCompile = function (filename, altExts) {
   return _.contains(exts, ext);
 };
 
+exports.isInteger = function (i) {
+  return _.isNumber(i) && i % 1 === 0;
+};
+
 exports.resolve = function (loc) {
   try {
     return require.resolve(loc);
@@ -123,23 +127,17 @@ exports.template = function (name, nodes, keepExpression) {
 
   template = _.cloneDeep(template);
 
-  var inherits = false;
-  if (nodes) {
-    inherits = nodes.inherits;
-    delete nodes.inherits;
-
-    if (!_.isEmpty(nodes)) {
-      traverse(template, function (node) {
-        if (t.isIdentifier(node) && _.has(nodes, node.name)) {
-          var newNode = nodes[node.name];
-          if (_.isString(newNode)) {
-            node.name = newNode;
-          } else {
-            return newNode;
-          }
+  if (!_.isEmpty(nodes)) {
+    traverse(template, function (node) {
+      if (t.isIdentifier(node) && _.has(nodes, node.name)) {
+        var newNode = nodes[node.name];
+        if (_.isString(newNode)) {
+          node.name = newNode;
+        } else {
+          return newNode;
         }
-      });
-    }
+      }
+    });
   }
 
   var node = template.body[0];
@@ -150,10 +148,6 @@ exports.template = function (name, nodes, keepExpression) {
     if (t.isParenthesizedExpression(node)) node = node.expression;
   }
 
-  if (inherits) {
-    node = t.inherits(node, inherits);
-  }
-
   return node;
 };
 

package.json

@@ -1,7 +1,7 @@
 {
   "name": "6to5",
   "description": "Turn ES6 code into readable vanilla ES5 with source maps",
-  "version": "1.12.6",
+  "version": "1.12.12",
   "author": "Sebastian McKenzie <sebmck@gmail.com>",
   "homepage": "https://github.com/6to5/6to5",
   "repository": {
@@ -48,7 +48,7 @@
     "chokidar": "0.10.5",
     "source-map-support": "0.2.8",
     "esutils": "1.1.4",
-    "acorn-6to5": "https://github.com/6to5/acorn-6to5/archive/90db0ad3c6219015a36fc3c87b85231b117cf3ea.tar.gz",
+    "acorn-6to5": "https://github.com/6to5/acorn-6to5/archive/a6c34ba73e215bd16ee585f6794f6a6587820c58.tar.gz",
     "estraverse": "^1.7.0"
   },
   "devDependencies": {

test/api.js

@@ -0,0 +1,17 @@
+var transform = require("../lib/6to5/transformation/transform");
+var assert    = require("assert");
+var File      = require("../lib/6to5/file");
+
+suite("api", function () {
+  test("{ code: false }", function () {
+    var result = transform("foo('bar');", { code: false });
+    assert.ok(!result.code);
+  });
+
+  test("addDeclaration unknown", function () {
+    var file = new File;
+    assert.throws(function () {
+      file.addDeclaration("foob");
+    }, /unknown declaration foob/);
+  });
+});

test/fixtures/generation/basic/array-comprehensions/actual.js

@@ -1,3 +0,0 @@
-var multiples = [for (num of nums) num * num];
-var multiples = [for (num of nums) for (num2 of nums) num * num];
-var multiples = [for (num of nums) if (num > 5) num * num];

test/fixtures/generation/basic/array-comprehensions/expected.js

@@ -1,3 +0,0 @@
-var multiples = [for (num of nums) num * num];
-var multiples = [for (num of nums) for (num2 of nums) num * num];
-var multiples = [for (num of nums) if (num > 5) num * num];

test/fixtures/generation/basic/arrays/actual.js

@@ -1 +0,0 @@
-var foo = [1, 2, 3];

test/fixtures/generation/basic/arrays/expected.js

@@ -1 +0,0 @@
-var foo = [1, 2, 3];

test/fixtures/generation/basic/template-literals/actual.js

@@ -1,17 +0,0 @@
-var foo = `i
-am a multiline string`;
-
-var foo = `i am also
-   a multiline string
-foobar`;
-
-var foo = `im a single line string`;
-
-var foo = tagit`i
-am a multiline string`;
-
-var foo = tagit`i am also
-   a multiline string
-foobar`;
-
-var foo = tagit`im a single line string`;

test/fixtures/generation/basic/template-literals/expected.js

@@ -1,17 +0,0 @@
-var foo = `i
-am a multiline string`;
-
-var foo = `i am also
-   a multiline string
-foobar`;
-
-var foo = `im a single line string`;
-
-var foo = tagit`i
-am a multiline string`;
-
-var foo = tagit`i am also
-   a multiline string
-foobar`;
-
-var foo = tagit`im a single line string`;

test/fixtures/generation/harmony-edgecase/exports/expected.js

@@ -4,6 +4,6 @@ export { a as b, c as d } from "hello";
 export { a as b, c as d };
 export {};
 export default i = 20;
-export function test() {};
+export function test() {}
 export var i = 20;
 export let i = 42;

test/fixtures/generation/types/ArrayExpression-ArrayPattern/actual.js

@@ -0,0 +1,5 @@
+[a, b, c];
+[[], [b, c], []];
+[a,, b,];
+[a,,,, b];
+[a, b,, c];

test/fixtures/generation/types/ArrayExpression-ArrayPattern/expected.js

@@ -0,0 +1,5 @@
+[a, b, c];
+[[], [b, c], []];
+[a,, b];
+[a,,,, b];
+[a, b,, c];

test/fixtures/generation/types/ArrowFunctionExpression/actual.js

@@ -0,0 +1,4 @@
+var foo = x => x * x;
+var foo = (a, b) => a * b;
+var foo = async x => x * x;
+var foo = async (a, b) => a * b;

test/fixtures/generation/types/ArrowFunctionExpression/expected.js

@@ -0,0 +1,4 @@
+var foo = x => x * x;
+var foo = (a, b) => a * b;
+var foo = async x => x * x;
+var foo = async (a, b) => a * b;

test/fixtures/generation/types/AssignmentExpression-BinaryExpression-LogicalExpression/actual.js

@@ -0,0 +1,3 @@
+foo === bar;
+foo + bar;
+foo = bar;

test/fixtures/generation/types/AssignmentExpression-BinaryExpression-LogicalExpression/expected.js

@@ -0,0 +1,3 @@
+foo === bar;
+foo + bar;
+foo = bar;

test/fixtures/generation/types/AwaitExpression/actual.js

@@ -0,0 +1,7 @@
+async function foo() {
+  await bar();
+}
+
+async function bar() {
+  await* foo();
+}

test/fixtures/generation/types/AwaitExpression/expected.js

@@ -0,0 +1,7 @@
+async function foo() {
+  await bar();
+}
+
+async function bar() {
+  await* foo();
+}

test/fixtures/generation/types/BlockStatement/actual.js

@@ -0,0 +1,5 @@
+{}
+
+{
+  foo();
+}

test/fixtures/generation/types/BlockStatement/expected.js

@@ -0,0 +1,5 @@
+{}
+
+{
+  foo();
+}

test/fixtures/generation/types/BreakStatement/actual.js

@@ -0,0 +1,7 @@
+for (var i in foo) {
+  break;
+}
+
+foo: for (var i in foo) {
+  break foo;
+}

test/fixtures/generation/types/BreakStatement/expected.js

@@ -0,0 +1,7 @@
+for (var i in foo) {
+  break;
+}
+
+foo: for (var i in foo) {
+  break foo;
+}

test/fixtures/generation/types/CallExpression/actual.js

@@ -0,0 +1,5 @@
+foo();
+foo("foo");
+foo("foo", "bar");
+foo(bar());
+foo(bar("test"));

test/fixtures/generation/types/CallExpression/expected.js

@@ -0,0 +1,5 @@
+foo();
+foo("foo");
+foo("foo", "bar");
+foo(bar());
+foo(bar("test"));

test/fixtures/generation/types/ClassBody-MethodDefinition/actual.js

@@ -0,0 +1,13 @@
+class Foo {
+  async foo() {}
+  foo() {}
+  ["foo"]() {}
+  get foo() {}
+  set foo() {}
+
+  static async foo() {}
+  static foo() {}
+  static ["foo"]() {}
+  static get foo() {}
+  static set foo() {}
+}

test/fixtures/generation/types/ClassBody-MethodDefinition/expected.js

@@ -0,0 +1,13 @@
+class Foo {
+  async foo() {}
+  foo() {}
+  ["foo"]() {}
+  get foo() {}
+  set foo() {}
+
+  static async foo() {}
+  static foo() {}
+  static ["foo"]() {}
+  static get foo() {}
+  static set foo() {}
+}

test/fixtures/generation/types/ClassDeclaration/actual.js

@@ -0,0 +1,2 @@
+var foo = class Foo {};
+var foo = class Foo extends Bar {};

test/fixtures/generation/types/ClassDeclaration/expected.js

@@ -0,0 +1,2 @@
+var foo = class Foo {};
+var foo = class Foo extends Bar {};

test/fixtures/generation/types/ClassExpression/actual.js

@@ -0,0 +1,2 @@
+class Foo {}
+class Foo extends Bar {}

test/fixtures/generation/types/ClassExpression/expected.js

@@ -0,0 +1,2 @@
+class Foo {}
+class Foo extends Bar {}

test/fixtures/generation/types/ComprehensionBlock-ComprehensionExpression/actual.js

@@ -0,0 +1,9 @@
+[for (x of nums) x * x];
+[for (x of nums) if (x > 2) x * x];
+[for (nums of nums2) for (x of nums) x * x];
+[for (nums of nums2) for (x of nums) if (x > 2) x * x];
+
+(for (x of nums) x * x);
+(for (x of nums) if (x > 2) x * x);
+(for (nums of nums2) for (x of nums) x * x);
+(for (nums of nums2) for (x of nums) if (x > 2) x * x);

test/fixtures/generation/types/ComprehensionBlock-ComprehensionExpression/expected.js

@@ -0,0 +1,9 @@
+[for (x of nums) x * x];
+[for (x of nums) if (x > 2) x * x];
+[for (nums of nums2) for (x of nums) x * x];
+[for (nums of nums2) for (x of nums) if (x > 2) x * x];
+
+(for (x of nums) x * x);
+(for (x of nums) if (x > 2) x * x);
+(for (nums of nums2) for (x of nums) x * x);
+(for (nums of nums2) for (x of nums) if (x > 2) x * x);

test/fixtures/generation/types/ConditionalExpression/actual.js

@@ -0,0 +1 @@
+foo ? foo : bar;

test/fixtures/generation/types/ConditionalExpression/expected.js

@@ -0,0 +1 @@
+foo ? foo : bar;

test/fixtures/generation/types/ContinueStatement/actual.js

@@ -0,0 +1,7 @@
+for (var i in foo) {
+  continue;
+}
+
+foo: for (var i in foo) {
+  continue foo;
+}

test/fixtures/generation/types/ContinueStatement/expected.js

@@ -0,0 +1,7 @@
+for (var i in foo) {
+  continue;
+}
+
+foo: for (var i in foo) {
+  continue foo;
+}

test/fixtures/generation/types/DebuggerStatement/actual.js

@@ -0,0 +1 @@
+debugger;

test/fixtures/generation/types/DebuggerStatement/expected.js

@@ -0,0 +1 @@
+debugger;

test/fixtures/generation/types/DoWhileStatement/actual.js

@@ -0,0 +1,5 @@
+do {
+  test();
+} while (true);
+
+do {} while (true);

test/fixtures/generation/types/DoWhileStatement/expected.js

@@ -0,0 +1,5 @@
+do {
+  test();
+} while (true);
+
+do {} while (true);

test/fixtures/generation/types/EmptyStatement/actual.js

@@ -0,0 +1 @@
+;

test/fixtures/generation/types/EmptyStatement/expected.js

@@ -0,0 +1 @@
+;

test/fixtures/generation/types/ExportDeclaration-ExportSpecifier-ExportBatchSpecifier/actual.js

@@ -0,0 +1,27 @@
+export default 42;
+export default {};
+export default [];
+export default foo;
+export default function () {};
+export default class {};
+export default function foo() {};
+export default class Foo {};
+export * from "foo";
+export { foo } from "foo";
+export { foo, bar } from "foo";
+export { foo as bar } from "foo";
+export { foo as default } from "foo";
+export { foo as default, bar } from "foo";
+export { foo };
+export { foo, bar };
+export { foo as bar };
+export { foo as default };
+export { foo as default, bar };
+export var foo = 1;
+export var foo2 = function () {};
+export var foo3;
+export let foo4 = 2;
+export let foo5;
+export const foo6 = 3;
+export function foo7() {}
+export class foo8 {}

test/fixtures/generation/types/ExportDeclaration-ExportSpecifier-ExportBatchSpecifier/expected.js

@@ -0,0 +1,27 @@
+export default 42;
+export default {};
+export default [];
+export default foo;
+export default function () {};
+export default class {};
+export default function foo() {};
+export default class Foo {};
+export * from "foo";
+export { foo } from "foo";
+export { foo, bar } from "foo";
+export { foo as bar } from "foo";
+export { foo as default } from "foo";
+export { foo as default, bar } from "foo";
+export { foo };
+export { foo, bar };
+export { foo as bar };
+export { foo as default };
+export { foo as default, bar };
+export var foo = 1;
+export var foo2 = function () {};
+export var foo3;
+export let foo4 = 2;
+export let foo5;
+export const foo6 = 3;
+export function foo7() {}
+export class foo8 {}

test/fixtures/generation/types/ForInStatement/actual.js

@@ -0,0 +1,5 @@
+for (var i in nums) {}
+
+for (var i in nums) {
+  nums[i] = nums[i] * nums[i];
+}

test/fixtures/generation/types/ForInStatement/expected.js

@@ -0,0 +1,5 @@
+for (var i in nums) {}
+
+for (var i in nums) {
+  nums[i] = nums[i] * nums[i];
+}

test/fixtures/generation/types/ForOfStatement/actual.js

@@ -0,0 +1,5 @@
+for (var x of nums) {}
+
+for (var x of nums) {
+  console.log(x * x);
+}

test/fixtures/generation/types/ForOfStatement/expected.js

@@ -0,0 +1,5 @@
+for (var x of nums) {}
+
+for (var x of nums) {
+  console.log(x * x);
+}

test/fixtures/generation/types/ForStatement/actual.js

@@ -0,0 +1,3 @@
+for (var i = 0;;) {}
+for (var i = 0; i < 5;) {}
+for (var i = 0; i < 5; i++) {}

test/fixtures/generation/types/ForStatement/expected.js

@@ -0,0 +1,3 @@
+for (var i = 0;;) {}
+for (var i = 0; i < 5;) {}
+for (var i = 0; i < 5; i++) {}

test/fixtures/generation/types/FunctionDeclaration-FunctionExpression/actual.js

@@ -0,0 +1,23 @@
+function foo(bar = "bar") {}
+
+function foo(bar = "bar", ...items) {}
+
+function foo(...items) {}
+
+function* foo() {}
+
+function foo() {}
+
+async function foo() {}
+
+var foo = function* foo() {};
+
+var foo = function foo() {};
+
+var foo = async function foo() {};
+
+var foo = function* () {};
+
+var foo = function () {};
+
+var foo = async function () {};

test/fixtures/generation/types/FunctionDeclaration-FunctionExpression/expected.js

@@ -0,0 +1,23 @@
+function foo(bar = "bar") {}
+
+function foo(bar = "bar", ...items) {}
+
+function foo(...items) {}
+
+function* foo() {}
+
+function foo() {}
+
+async function foo() {}
+
+var foo = function* foo() {};
+
+var foo = function foo() {};
+
+var foo = async function foo() {};
+
+var foo = function* () {};
+
+var foo = function () {};
+
+var foo = async function () {};

test/fixtures/generation/types/Identifier/actual.js

@@ -0,0 +1,2 @@
+foo;
+undefined;

test/fixtures/generation/types/Identifier/expected.js

@@ -0,0 +1,2 @@
+foo;
+undefined;

test/fixtures/generation/types/IfStatement/actual.js

@@ -0,0 +1,25 @@
+if (foo) foobar();
+
+if (foo) {
+  foobar();
+}
+
+if (foo) {}
+
+if (foo) {
+  foo();
+} else bar();
+
+if (foo) {
+  foo();
+} else {
+  bar();
+}
+
+if (foo) {
+  foo();
+} else if (bar) {
+  foobar();
+} else {
+  bar();
+}

test/fixtures/generation/types/IfStatement/expected.js

@@ -0,0 +1,25 @@
+if (foo) foobar();
+
+if (foo) {
+  foobar();
+}
+
+if (foo) {}
+
+if (foo) {
+  foo();
+} else bar();
+
+if (foo) {
+  foo();
+} else {
+  bar();
+}
+
+if (foo) {
+  foo();
+} else if (bar) {
+  foobar();
+} else {
+  bar();
+}

test/fixtures/generation/types/ImportDeclaration-ImportSpecifier-ImportBatchSpecifier/actual.js

@@ -0,0 +1,9 @@
+import "foo";
+import foo from "foo";
+import { default as foo } from "foo";
+import * as foo from "foo";
+import foo, { baz as xyz } from "foo";
+import { bar } from "foo";
+import { bar, baz } from "foo";
+import { bar as baz } from "foo";
+import { bar as baz, xyz } from "foo";

test/fixtures/generation/types/ImportDeclaration-ImportSpecifier-ImportBatchSpecifier/expected.js

@@ -0,0 +1,9 @@
+import "foo";
+import foo from "foo";
+import { default as foo } from "foo";
+import * as foo from "foo";
+import foo, { baz as xyz } from "foo";
+import { bar } from "foo";
+import { bar, baz } from "foo";
+import { bar as baz } from "foo";
+import { bar as baz, xyz } from "foo";

test/fixtures/generation/types/LabeledStatement/actual.js

@@ -0,0 +1 @@
+label: for (var i in test) {}

test/fixtures/generation/types/LabeledStatement/expected.js

@@ -0,0 +1 @@
+label: for (var i in test) {}

test/fixtures/generation/types/Literal/actual.js

@@ -0,0 +1,7 @@
+5;
+"foobar";
+"\n\r\u2028\u2029";
+/foobar/g;
+null;
+true;
+false;