@babel/traverse 7.14.2 → 7.14.9

package/lib/context.js

@@ -9,8 +9,6 @@ var _path = require("./path");
 
 var t = require("@babel/types");
 
-const testing = process.env.NODE_ENV === "test";
-
 class TraversalContext {
   constructor(scope, opts, state, parentPath) {
     this.queue = null;
@@ -46,10 +44,6 @@ class TraversalContext {
   }
 
   maybeQueue(path, notPriority) {
-    if (this.trap) {
-      throw new Error("Infinite cycle detected");
-    }
-
     if (this.queue) {
       if (notPriority) {
         this.queue.push(path);
@@ -96,11 +90,6 @@ class TraversalContext {
       }
 
       if (path.key === null) continue;
-
-      if (testing && queue.length >= 10000) {
-        this.trap = true;
-      }
-
       const {
         node
       } = path;

package/lib/path/conversion.js

@@ -166,7 +166,15 @@ function hoistFunctionEnvironment(fnPath, noNewArrows = true, allowInsertArrow =
   }
 
   if (argumentsPaths.length > 0) {
-    const argumentsBinding = getBinding(thisEnvFn, "arguments", () => t.identifier("arguments"));
+    const argumentsBinding = getBinding(thisEnvFn, "arguments", () => {
+      const args = () => t.identifier("arguments");
+
+      if (thisEnvFn.scope.path.isProgram()) {
+        return t.conditionalExpression(t.binaryExpression("===", t.unaryExpression("typeof", args()), t.stringLiteral("undefined")), thisEnvFn.scope.buildUndefinedNode(), args());
+      } else {
+        return args();
+      }
+    });
     argumentsPaths.forEach(argumentsChild => {
       const argsRef = t.identifier(argumentsBinding);
       argsRef.loc = argumentsChild.node.loc;
@@ -398,6 +406,19 @@ function getScopeInformation(fnPath) {
 
     ReferencedIdentifier(child) {
       if (child.node.name !== "arguments") return;
+      let curr = child.scope;
+
+      do {
+        if (curr.hasOwnBinding("arguments")) {
+          curr.rename("arguments");
+          return;
+        }
+
+        if (curr.path.isFunction() && !curr.path.isArrowFunctionExpression()) {
+          break;
+        }
+      } while (curr = curr.parent);
+
       argumentsPaths.push(child);
     },
 

package/lib/path/family.js

@@ -158,7 +158,14 @@ function getStatementListCompletion(paths, context) {
       }
     }
   } else if (paths.length) {
-    completions = completions.concat(_getCompletionRecords(paths[paths.length - 1], context));
+    for (let i = paths.length - 1; i >= 0; i--) {
+      const pathCompletions = _getCompletionRecords(paths[i], context);
+
+      if (pathCompletions.length > 1 || pathCompletions.length === 1 && !pathCompletions[0].path.isVariableDeclaration()) {
+        completions = completions.concat(pathCompletions);
+        break;
+      }
+    }
   }
 
   return completions;

package/lib/path/replacement.js

@@ -22,33 +22,7 @@ var _parser = require("@babel/parser");
 
 var t = require("@babel/types");
 
-const hoistVariablesVisitor = {
-  Function(path) {
-    path.skip();
-  },
-
-  VariableDeclaration(path) {
-    if (path.node.kind !== "var") return;
-    const bindings = path.getBindingIdentifiers();
-
-    for (const key of Object.keys(bindings)) {
-      path.scope.push({
-        id: bindings[key]
-      });
-    }
-
-    const exprs = [];
-
-    for (const declar of path.node.declarations) {
-      if (declar.init) {
-        exprs.push(t.expressionStatement(t.assignmentExpression("=", declar.id, declar.init)));
-      }
-    }
-
-    path.replaceWithMultiple(exprs);
-  }
-
-};
+var _helperHoistVariables = require("@babel/helper-hoist-variables");
 
 function replaceWithMultiple(nodes) {
   var _pathCache$get;
@@ -191,7 +165,12 @@ function replaceExpressionWithStatements(nodes) {
   const isParentGenerator = functionParent == null ? void 0 : functionParent.is("generator");
   const container = t.arrowFunctionExpression([], t.blockStatement(nodes));
   this.replaceWith(t.callExpression(container, []));
-  this.traverse(hoistVariablesVisitor);
+  const callee = this.get("callee");
+  (0, _helperHoistVariables.default)(callee.get("body"), id => {
+    this.scope.push({
+      id
+    });
+  }, "var");
   const completionRecords = this.get("callee").getCompletionRecords();
 
   for (const path of completionRecords) {
@@ -202,7 +181,6 @@ function replaceExpressionWithStatements(nodes) {
       let uid = loop.getData("expressionReplacementReturnUid");
 
       if (!uid) {
-        const callee = this.get("callee");
         uid = callee.scope.generateDeclaredUidIdentifier("ret");
         callee.get("body").pushContainer("body", t.returnStatement(t.cloneNode(uid)));
         loop.setData("expressionReplacementReturnUid", uid);
@@ -216,15 +194,15 @@ function replaceExpressionWithStatements(nodes) {
     }
   }
 
-  const callee = this.get("callee");
   callee.arrowFunctionToExpression();
+  const newCallee = callee;
 
   const needToAwaitFunction = isParentAsync && _index.default.hasType(this.get("callee.body").node, "AwaitExpression", t.FUNCTION_TYPES);
 
   const needToYieldFunction = isParentGenerator && _index.default.hasType(this.get("callee.body").node, "YieldExpression", t.FUNCTION_TYPES);
 
   if (needToAwaitFunction) {
-    callee.set("async", true);
+    newCallee.set("async", true);
 
     if (!needToYieldFunction) {
       this.replaceWith(t.awaitExpression(this.node));
@@ -232,11 +210,11 @@ function replaceExpressionWithStatements(nodes) {
   }
 
   if (needToYieldFunction) {
-    callee.set("generator", true);
+    newCallee.set("generator", true);
     this.replaceWith(t.yieldExpression(this.node, true));
   }
 
-  return callee.get("body.body");
+  return newCallee.get("body.body");
 }
 
 function replaceInline(nodes) {

package/lib/scope/index.js

@@ -172,11 +172,17 @@ const collectorVisitor = {
 
   Declaration(path) {
     if (path.isBlockScoped()) return;
+    if (path.isImportDeclaration()) return;
     if (path.isExportDeclaration()) return;
     const parent = path.scope.getFunctionParent() || path.scope.getProgramParent();
     parent.registerDeclaration(path);
   },
 
+  ImportDeclaration(path) {
+    const parent = path.scope.getBlockParent();
+    parent.registerDeclaration(path);
+  },
+
   ReferencedIdentifier(path, state) {
     state.references.push(path);
   },

package/package.json

@@ -1,8 +1,8 @@
 {
   "name": "@babel/traverse",
-  "version": "7.14.2",
+  "version": "7.14.9",
   "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>",
+  "author": "The Babel Team (https://babel.dev/team)",
   "homepage": "https://babel.dev/docs/en/next/babel-traverse",
   "bugs": "https://github.com/babel/babel/issues?utf8=%E2%9C%93&q=is%3Aissue+label%3A%22pkg%3A%20traverse%22+is%3Aopen",
   "license": "MIT",
@@ -14,18 +14,22 @@
     "url": "https://github.com/babel/babel.git",
     "directory": "packages/babel-traverse"
   },
-  "main": "lib/index.js",
+  "main": "./lib/index.js",
   "dependencies": {
-    "@babel/code-frame": "^7.12.13",
-    "@babel/generator": "^7.14.2",
-    "@babel/helper-function-name": "^7.14.2",
-    "@babel/helper-split-export-declaration": "^7.12.13",
-    "@babel/parser": "^7.14.2",
-    "@babel/types": "^7.14.2",
+    "@babel/code-frame": "^7.14.5",
+    "@babel/generator": "^7.14.9",
+    "@babel/helper-function-name": "^7.14.5",
+    "@babel/helper-hoist-variables": "^7.14.5",
+    "@babel/helper-split-export-declaration": "^7.14.5",
+    "@babel/parser": "^7.14.9",
+    "@babel/types": "^7.14.9",
     "debug": "^4.1.0",
     "globals": "^11.1.0"
   },
   "devDependencies": {
-    "@babel/helper-plugin-test-runner": "7.13.10"
+    "@babel/helper-plugin-test-runner": "7.14.5"
+  },
+  "engines": {
+    "node": ">=6.9.0"
   }
 }

package/scripts/generators/validators.js

@@ -9,6 +9,7 @@ export default function generateValidators() {
  */
 import * as t from "@babel/types";
 import NodePath from "../index";
+import type { VirtualTypeAliases } from "./virtual-types";
 
 export interface NodePathValidators {
 `;
@@ -18,10 +19,18 @@ export interface NodePathValidators {
   }
 
   for (const type of Object.keys(virtualTypes)) {
+    const { types } = virtualTypes[type];
     if (type[0] === "_") continue;
     if (definitions.NODE_FIELDS[type] || definitions.FLIPPED_ALIAS_KEYS[type]) {
       output += `is${type}(opts?: object): this is NodePath<t.${type}>;`;
+    } else if (types /* in VirtualTypeAliases */) {
+      output += `is${type}(opts?: object): this is NodePath<VirtualTypeAliases["${type}"]>;`;
     } else {
+      // if it don't have types, then VirtualTypeAliases[type] is t.Node
+      // which TS marked as always true
+      // eg. if (path.isBlockScope()) return;
+      //     path resolved to `never` here
+      // so we have to return boolean instead of this is NodePath<t.Node> here
       output += `is${type}(opts?: object): boolean;`;
     }
   }