@babel/traverse 7.3.4 → 7.4.0

package/lib/path/context.js

@@ -169,7 +169,7 @@ function _resyncKey() {
       }
     }
   } else {
-    for (const key in this.container) {
+    for (const key of Object.keys(this.container)) {
       if (this.container[key] === this.node) {
         return this.setKey(key);
       }

package/lib/path/index.js

@@ -208,7 +208,7 @@ for (const type of t().TYPES) {
   };
 }
 
-for (const type in virtualTypes) {
+for (const type of Object.keys(virtualTypes)) {
   if (type[0] === "_") continue;
   if (t().TYPES.indexOf(type) < 0) t().TYPES.push(type);
   const virtualType = virtualTypes[type];

package/lib/path/inference/inferers.js

@@ -12,6 +12,7 @@ exports.BinaryExpression = BinaryExpression;
 exports.LogicalExpression = LogicalExpression;
 exports.ConditionalExpression = ConditionalExpression;
 exports.SequenceExpression = SequenceExpression;
+exports.ParenthesizedExpression = ParenthesizedExpression;
 exports.AssignmentExpression = AssignmentExpression;
 exports.UpdateExpression = UpdateExpression;
 exports.StringLiteral = StringLiteral;
@@ -128,6 +129,10 @@ function SequenceExpression() {
   return this.get("expressions").pop().getTypeAnnotation();
 }
 
+function ParenthesizedExpression() {
+  return this.get("expression").getTypeAnnotation();
+}
+
 function AssignmentExpression() {
   return this.get("right").getTypeAnnotation();
 }

package/lib/path/lib/hoister.js

@@ -54,7 +54,7 @@ class PathHoister {
   }
 
   isCompatibleScope(scope) {
-    for (const key in this.bindings) {
+    for (const key of Object.keys(this.bindings)) {
       const binding = this.bindings[key];
 
       if (!scope.bindingIdentifierEquals(key, binding.identifier)) {
@@ -92,7 +92,7 @@ class PathHoister {
     }
 
     if (targetScope.path.isProgram() || targetScope.path.isFunction()) {
-      for (const name in this.bindings) {
+      for (const name of Object.keys(this.bindings)) {
         if (!targetScope.hasOwnBinding(name)) continue;
         const binding = this.bindings[name];
 
@@ -154,7 +154,7 @@ class PathHoister {
   }
 
   hasOwnParamBindings(scope) {
-    for (const name in this.bindings) {
+    for (const name of Object.keys(this.bindings)) {
       if (!scope.hasOwnBinding(name)) continue;
       const binding = this.bindings[name];
       if (binding.kind === "param" && binding.constant) return true;

package/lib/path/lib/virtual-types.js

@@ -20,10 +20,12 @@ function _interopRequireWildcard(obj) { if (obj && obj.__esModule) { return obj;
 const ReferencedIdentifier = {
   types: ["Identifier", "JSXIdentifier"],
 
-  checkPath({
-    node,
-    parent
-  }, opts) {
+  checkPath(path, opts) {
+    const {
+      node,
+      parent
+    } = path;
+
     if (!t().isIdentifier(node, opts) && !t().isJSXMemberExpression(parent, opts)) {
       if (t().isJSXIdentifier(node, opts)) {
         if (t().react.isCompatTag(node.name)) return false;
@@ -32,7 +34,7 @@ const ReferencedIdentifier = {
       }
     }
 
-    return t().isReferenced(node, parent);
+    return t().isReferenced(node, parent, path.parentPath.parent);
   }
 
 };
@@ -52,11 +54,13 @@ exports.ReferencedMemberExpression = ReferencedMemberExpression;
 const BindingIdentifier = {
   types: ["Identifier"],
 
-  checkPath({
-    node,
-    parent
-  }) {
-    return t().isIdentifier(node) && t().isBinding(node, parent);
+  checkPath(path) {
+    const {
+      node,
+      parent
+    } = path;
+    const grandparent = path.parentPath.parent;
+    return t().isIdentifier(node) && t().isBinding(node, parent, grandparent);
   }
 
 };

package/lib/path/replacement.js

@@ -57,7 +57,7 @@ const hoistVariablesVisitor = {
     if (path.node.kind !== "var") return;
     const bindings = path.getBindingIdentifiers();
 
-    for (const key in bindings) {
+    for (const key of Object.keys(bindings)) {
       path.scope.push({
         id: bindings[key]
       });

package/lib/scope/index.js

@@ -152,9 +152,7 @@ const collectorVisitor = {
         if (binding) binding.reference(path);
       } else if (t().isVariableDeclaration(declar)) {
         for (const decl of declar.declarations) {
-          const ids = t().getBindingIdentifiers(decl);
-
-          for (const name in ids) {
+          for (const name of Object.keys(t().getBindingIdentifiers(decl))) {
             const binding = scope.getBinding(name);
             if (binding) binding.reference(path);
           }
@@ -371,7 +369,7 @@ class Scope {
     do {
       console.log("#", scope.block.type);
 
-      for (const name in scope.bindings) {
+      for (const name of Object.keys(scope.bindings)) {
         const binding = scope.bindings[name];
         console.log(" -", name, {
           constant: binding.constant,
@@ -472,7 +470,7 @@ class Scope {
   registerConstantViolation(path) {
     const ids = path.getBindingIdentifiers();
 
-    for (const name in ids) {
+    for (const name of Object.keys(ids)) {
       const binding = this.getBinding(name);
       if (binding) binding.reassign(path);
     }
@@ -492,9 +490,9 @@ class Scope {
     }
 
     const parent = this.getProgramParent();
-    const ids = path.getBindingIdentifiers(true);
+    const ids = path.getOuterBindingIdentifiers(true);
 
-    for (const name in ids) {
+    for (const name of Object.keys(ids)) {
       for (const id of ids[name]) {
         const local = this.getOwnBinding(name);
 
@@ -687,7 +685,7 @@ class Scope {
       const ids = path.getBindingIdentifiers();
       let programParent;
 
-      for (const name in ids) {
+      for (const name of Object.keys(ids)) {
         if (path.scope.getBinding(name)) continue;
         programParent = programParent || path.scope.getProgramParent();
         programParent.addGlobal(ids[name]);
@@ -800,7 +798,7 @@ class Scope {
       let scope = this;
 
       do {
-        for (const name in scope.bindings) {
+        for (const name of Object.keys(scope.bindings)) {
           const binding = scope.bindings[name];
           if (binding.kind === kind) ids[name] = binding;
         }

package/lib/visitors.js

@@ -37,7 +37,7 @@ function explode(visitor) {
   if (visitor._exploded) return visitor;
   visitor._exploded = true;
 
-  for (const nodeType in visitor) {
+  for (const nodeType of Object.keys(visitor)) {
     if (shouldIgnoreKey(nodeType)) continue;
     const parts = nodeType.split("|");
     if (parts.length === 1) continue;
@@ -60,7 +60,7 @@ function explode(visitor) {
     if (!wrapper) continue;
     const fns = visitor[nodeType];
 
-    for (const type in fns) {
+    for (const type of Object.keys(fns)) {
       fns[type] = wrapCheck(wrapper, fns[type]);
     }
 
@@ -79,7 +79,7 @@ function explode(visitor) {
     }
   }
 
-  for (const nodeType in visitor) {
+  for (const nodeType of Object.keys(visitor)) {
     if (shouldIgnoreKey(nodeType)) continue;
     const fns = visitor[nodeType];
     let aliases = t().FLIPPED_ALIAS_KEYS[nodeType];
@@ -104,7 +104,7 @@ function explode(visitor) {
     }
   }
 
-  for (const nodeType in visitor) {
+  for (const nodeType of Object.keys(visitor)) {
     if (shouldIgnoreKey(nodeType)) continue;
     ensureCallbackArrays(visitor[nodeType]);
   }
@@ -119,7 +119,7 @@ function verify(visitor) {
     throw new Error("You passed `traverse()` a function when it expected a visitor object, " + "are you sure you didn't mean `{ enter: Function }`?");
   }
 
-  for (const nodeType in visitor) {
+  for (const nodeType of Object.keys(visitor)) {
     if (nodeType === "enter" || nodeType === "exit") {
       validateVisitorMethods(nodeType, visitor[nodeType]);
     }
@@ -133,7 +133,7 @@ function verify(visitor) {
     const visitors = visitor[nodeType];
 
     if (typeof visitors === "object") {
-      for (const visitorKey in visitors) {
+      for (const visitorKey of Object.keys(visitors)) {
         if (visitorKey === "enter" || visitorKey === "exit") {
           validateVisitorMethods(`${nodeType}.${visitorKey}`, visitors[visitorKey]);
         } else {
@@ -164,7 +164,7 @@ function merge(visitors, states = [], wrapper) {
     const state = states[i];
     explode(visitor);
 
-    for (const type in visitor) {
+    for (const type of Object.keys(visitor)) {
       let visitorType = visitor[type];
 
       if (state || wrapper) {
@@ -182,7 +182,7 @@ function merge(visitors, states = [], wrapper) {
 function wrapWithStateOrWrapper(oldVisitor, state, wrapper) {
   const newVisitor = {};
 
-  for (const key in oldVisitor) {
+  for (const key of Object.keys(oldVisitor)) {
     let fns = oldVisitor[key];
     if (!Array.isArray(fns)) continue;
     fns = fns.map(function (fn) {
@@ -207,7 +207,7 @@ function wrapWithStateOrWrapper(oldVisitor, state, wrapper) {
 }
 
 function ensureEntranceObjects(obj) {
-  for (const key in obj) {
+  for (const key of Object.keys(obj)) {
     if (shouldIgnoreKey(key)) continue;
     const fns = obj[key];
 
@@ -248,7 +248,7 @@ function shouldIgnoreKey(key) {
 }
 
 function mergePair(dest, src) {
-  for (const key in src) {
+  for (const key of Object.keys(src)) {
     dest[key] = [].concat(dest[key] || [], src[key]);
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@babel/traverse",
-  "version": "7.3.4",
+  "version": "7.4.0",
   "description": "The Babel Traverse module maintains the overall tree state, and is responsible for replacing, removing, and adding nodes",
   "author": "Sebastian McKenzie <sebmck@gmail.com>",
   "homepage": "https://babeljs.io/",
@@ -12,11 +12,11 @@
   "main": "lib/index.js",
   "dependencies": {
     "@babel/code-frame": "^7.0.0",
-    "@babel/generator": "^7.3.4",
+    "@babel/generator": "^7.4.0",
     "@babel/helper-function-name": "^7.1.0",
-    "@babel/helper-split-export-declaration": "^7.0.0",
-    "@babel/parser": "^7.3.4",
-    "@babel/types": "^7.3.4",
+    "@babel/helper-split-export-declaration": "^7.4.0",
+    "@babel/parser": "^7.4.0",
+    "@babel/types": "^7.4.0",
     "debug": "^4.1.0",
     "globals": "^11.1.0",
     "lodash": "^4.17.11"
@@ -24,5 +24,5 @@
   "devDependencies": {
     "@babel/helper-plugin-test-runner": "^7.0.0"
   },
-  "gitHead": "1f6454cc90fe33e0a32260871212e2f719f35741"
+  "gitHead": "f1328fb913b5a93d54dfc6e3728b1f56c8f4a804"
 }