@babel/traverse 7.12.1 → 7.12.9

package/README.md

@@ -2,7 +2,7 @@
 
 > The Babel Traverse module maintains the overall tree state, and is responsible for replacing, removing, and adding nodes
 
-See our website [@babel/traverse](https://babeljs.io/docs/en/next/babel-traverse.html) for more information or the [issues](https://github.com/babel/babel/issues?utf8=%E2%9C%93&q=is%3Aissue+label%3A%22pkg%3A%20traverse%22+is%3Aopen) associated with this package.
+See our website [@babel/traverse](https://babeljs.io/docs/en/babel-traverse) for more information or the [issues](https://github.com/babel/babel/issues?utf8=%E2%9C%93&q=is%3Aissue+label%3A%22pkg%3A%20traverse%22+is%3Aopen) associated with this package.
 
 ## Install
 

package/lib/context.js

@@ -19,10 +19,6 @@ const testing = process.env.NODE_ENV === "test";
 
 class TraversalContext {
   constructor(scope, opts, state, parentPath) {
-    this.parentPath = void 0;
-    this.scope = void 0;
-    this.state = void 0;
-    this.opts = void 0;
     this.queue = null;
     this.parentPath = parentPath;
     this.scope = scope;
@@ -94,7 +90,7 @@ class TraversalContext {
   visitQueue(queue) {
     this.queue = queue;
     this.priorityQueue = [];
-    const visited = [];
+    const visited = new WeakSet();
     let stop = false;
 
     for (const path of queue) {
@@ -110,8 +106,11 @@ class TraversalContext {
         this.trap = true;
       }
 
-      if (visited.indexOf(path.node) >= 0) continue;
-      visited.push(path.node);
+      const {
+        node
+      } = path;
+      if (visited.has(node)) continue;
+      if (node) visited.add(node);
 
       if (path.visit()) {
         stop = true;

package/lib/path/family.js

@@ -141,7 +141,7 @@ function getSibling(key) {
     container: this.container,
     listKey: this.listKey,
     key: key
-  });
+  }).setContext(this.context);
 }
 
 function getPrevSibling() {
@@ -178,7 +178,7 @@ function getAllPrevSiblings() {
   return siblings;
 }
 
-function get(key, context) {
+function get(key, context = true) {
   if (context === true) context = this.context;
   const parts = key.split(".");
 

package/lib/path/index.js

@@ -57,41 +57,22 @@ exports.SHOULD_SKIP = SHOULD_SKIP;
 
 class NodePath {
   constructor(hub, parent) {
-    this.parent = void 0;
-    this.hub = void 0;
-    this.contexts = void 0;
-    this.data = void 0;
-    this.shouldSkip = void 0;
-    this.shouldStop = void 0;
-    this.removed = void 0;
-    this.state = void 0;
-    this.opts = void 0;
-    this._traverseFlags = void 0;
-    this.skipKeys = void 0;
-    this.parentPath = void 0;
-    this.context = void 0;
-    this.container = void 0;
-    this.listKey = void 0;
-    this.key = void 0;
-    this.node = void 0;
-    this.scope = void 0;
-    this.type = void 0;
-    this.parent = parent;
-    this.hub = hub;
     this.contexts = [];
-    this.data = null;
-    this._traverseFlags = 0;
     this.state = null;
     this.opts = null;
+    this._traverseFlags = 0;
     this.skipKeys = null;
     this.parentPath = null;
-    this.context = null;
     this.container = null;
     this.listKey = null;
     this.key = null;
     this.node = null;
-    this.scope = null;
     this.type = null;
+    this.parent = parent;
+    this.hub = hub;
+    this.data = null;
+    this.context = null;
+    this.scope = null;
   }
 
   static get({
@@ -111,26 +92,20 @@ class NodePath {
     }
 
     const targetNode = container[key];
-    const paths = _cache.path.get(parent) || [];
-
-    if (!_cache.path.has(parent)) {
-      _cache.path.set(parent, paths);
-    }
 
-    let path;
+    let paths = _cache.path.get(parent);
 
-    for (let i = 0; i < paths.length; i++) {
-      const pathCheck = paths[i];
+    if (!paths) {
+      paths = new Map();
 
-      if (pathCheck.node === targetNode) {
-        path = pathCheck;
-        break;
-      }
+      _cache.path.set(parent, paths);
     }
 
+    let path = paths.get(targetNode);
+
     if (!path) {
       path = new NodePath(hub, parent);
-      paths.push(path);
+      if (targetNode) paths.set(targetNode, path);
     }
 
     path.setup(parentPath, container, listKey, key);

package/lib/path/inference/index.js

@@ -25,9 +25,9 @@ function getTypeAnnotation() {
   return this.typeAnnotation = type;
 }
 
-function _getTypeAnnotation() {
-  var _inferer;
+const typeAnnotationInferringNodes = new WeakSet();
 
+function _getTypeAnnotation() {
   const node = this.node;
 
   if (!node) {
@@ -53,16 +53,28 @@ function _getTypeAnnotation() {
     return node.typeAnnotation;
   }
 
-  let inferer = inferers[node.type];
-
-  if (inferer) {
-    return inferer.call(this, node);
+  if (typeAnnotationInferringNodes.has(node)) {
+    return;
   }
 
-  inferer = inferers[this.parentPath.type];
+  typeAnnotationInferringNodes.add(node);
+
+  try {
+    var _inferer;
+
+    let inferer = inferers[node.type];
 
-  if ((_inferer = inferer) == null ? void 0 : _inferer.validParent) {
-    return this.parentPath.getTypeAnnotation();
+    if (inferer) {
+      return inferer.call(this, node);
+    }
+
+    inferer = inferers[this.parentPath.type];
+
+    if ((_inferer = inferer) == null ? void 0 : _inferer.validParent) {
+      return this.parentPath.getTypeAnnotation();
+    }
+  } finally {
+    typeAnnotationInferringNodes.delete(node);
   }
 }
 

package/lib/path/modification.js

@@ -136,9 +136,7 @@ function updateSiblingKeys(fromIndex, incrementBy) {
 
   const paths = _cache.path.get(this.parent);
 
-  for (let i = 0; i < paths.length; i++) {
-    const path = paths[i];
-
+  for (const [, path] of paths) {
     if (path.key >= fromIndex) {
       path.key += incrementBy;
     }

package/lib/path/removal.js

@@ -12,6 +12,8 @@ exports._assertUnremoved = _assertUnremoved;
 
 var _removalHooks = require("./lib/removal-hooks");
 
+var _cache = require("../cache");
+
 var _index = require("./index");
 
 function remove() {
@@ -60,6 +62,7 @@ function _remove() {
 
 function _markRemoved() {
   this._traverseFlags |= _index.SHOULD_SKIP | _index.REMOVED;
+  if (this.parent) _cache.path.get(this.parent).delete(this.node);
   this.node = null;
 }
 

package/lib/path/replacement.js

@@ -16,6 +16,8 @@ var _index = _interopRequireDefault(require("../index"));
 
 var _index2 = _interopRequireDefault(require("./index"));
 
+var _cache = require("../cache");
+
 var _parser = require("@babel/parser");
 
 var t = _interopRequireWildcard(require("@babel/types"));
@@ -55,10 +57,13 @@ const hoistVariablesVisitor = {
 };
 
 function replaceWithMultiple(nodes) {
+  var _pathCache$get;
+
   this.resync();
   nodes = this._verifyNodeList(nodes);
   t.inheritLeadingComments(nodes[0], this.node);
   t.inheritTrailingComments(nodes[nodes.length - 1], this.node);
+  (_pathCache$get = _cache.path.get(this.parent)) == null ? void 0 : _pathCache$get.delete(this.node);
   this.node = this.container[this.key] = null;
   const paths = this.insertAfter(nodes);
 
@@ -162,6 +167,8 @@ function replaceWith(replacement) {
 }
 
 function _replaceWith(node) {
+  var _pathCache$get2;
+
   if (!this.container) {
     throw new ReferenceError("Container is falsy");
   }
@@ -173,6 +180,7 @@ function _replaceWith(node) {
   }
 
   this.debug(`Replace with ${node == null ? void 0 : node.type}`);
+  (_pathCache$get2 = _cache.path.get(this.parent)) == null ? void 0 : _pathCache$get2.set(node, this).delete(this.node);
   this.node = this.container[this.key] = node;
 }
 

package/lib/scope/binding.js

@@ -12,23 +12,15 @@ class Binding {
     path,
     kind
   }) {
-    this.constantViolations = void 0;
-    this.constant = void 0;
-    this.referencePaths = void 0;
-    this.referenced = void 0;
-    this.references = void 0;
-    this.hasDeoptedValue = void 0;
-    this.hasValue = void 0;
-    this.value = void 0;
-    this.identifier = identifier;
-    this.scope = scope;
-    this.path = path;
-    this.kind = kind;
     this.constantViolations = [];
     this.constant = true;
     this.referencePaths = [];
     this.referenced = false;
     this.references = 0;
+    this.identifier = identifier;
+    this.scope = scope;
+    this.path = path;
+    this.kind = kind;
     this.clearValue();
   }
 

package/lib/scope/lib/renamer.js

@@ -45,9 +45,6 @@ const renameVisitor = {
 
 class Renamer {
   constructor(binding, oldName, newName) {
-    this.oldName = void 0;
-    this.newName = void 0;
-    this.binding = void 0;
     this.newName = newName;
     this.oldName = oldName;
     this.binding = binding;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@babel/traverse",
-  "version": "7.12.1",
+  "version": "7.12.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>",
   "homepage": "https://babeljs.io/",
@@ -16,11 +16,11 @@
   "main": "lib/index.js",
   "dependencies": {
     "@babel/code-frame": "^7.10.4",
-    "@babel/generator": "^7.12.1",
+    "@babel/generator": "^7.12.5",
     "@babel/helper-function-name": "^7.10.4",
     "@babel/helper-split-export-declaration": "^7.11.0",
-    "@babel/parser": "^7.12.1",
-    "@babel/types": "^7.12.1",
+    "@babel/parser": "^7.12.7",
+    "@babel/types": "^7.12.7",
     "debug": "^4.1.0",
     "globals": "^11.1.0",
     "lodash": "^4.17.19"