@babel/traverse 7.17.0 → 7.17.3

package/lib/path/modification.js

@@ -30,7 +30,13 @@ const {
   callExpression,
   cloneNode,
   expressionStatement,
-  isExpression
+  isAssignmentExpression,
+  isCallExpression,
+  isExpression,
+  isIdentifier,
+  isSequenceExpression,
+  isSuper,
+  thisExpression
 } = _t;
 
 function insertBefore(nodes_) {
@@ -96,9 +102,28 @@ function _containerInsertAfter(nodes) {
   return this._containerInsert(this.key + 1, nodes);
 }
 
+const last = arr => arr[arr.length - 1];
+
+function isHiddenInSequenceExpression(path) {
+  return isSequenceExpression(path.parent) && (last(path.parent.expressions) !== path.node || isHiddenInSequenceExpression(path.parentPath));
+}
+
+function isAlmostConstantAssignment(node, scope) {
+  if (!isAssignmentExpression(node) || !isIdentifier(node.left)) {
+    return false;
+  }
+
+  const blockScope = scope.getBlockParent();
+  return blockScope.hasOwnBinding(node.left.name) && blockScope.getOwnBinding(node.left.name).constantViolations.length <= 1;
+}
+
 function insertAfter(nodes_) {
   this._assertUnremoved();
 
+  if (this.isSequenceExpression()) {
+    return last(this.get("expressions")).insertAfter(nodes_);
+  }
+
   const nodes = this._verifyNodeList(nodes_);
 
   const {
@@ -123,16 +148,28 @@ function insertAfter(nodes_) {
         return [this];
       }
 
-      if (parentPath.isMethod({
-        computed: true,
-        key: node
-      })) {
-        scope = scope.parent;
+      if (isHiddenInSequenceExpression(this)) {
+        nodes.unshift(node);
+      } else if (isCallExpression(node) && isSuper(node.callee)) {
+        nodes.unshift(node);
+        nodes.push(thisExpression());
+      } else if (isAlmostConstantAssignment(node, scope)) {
+        nodes.unshift(node);
+        nodes.push(cloneNode(node.left));
+      } else if (scope.isPure(node, true)) {
+        nodes.push(node);
+      } else {
+        if (parentPath.isMethod({
+          computed: true,
+          key: node
+        })) {
+          scope = scope.parent;
+        }
+
+        const temp = scope.generateDeclaredUidIdentifier();
+        nodes.unshift(expressionStatement(assignmentExpression("=", cloneNode(temp), node)));
+        nodes.push(expressionStatement(cloneNode(temp)));
       }
-
-      const temp = scope.generateDeclaredUidIdentifier();
-      nodes.unshift(expressionStatement(assignmentExpression("=", cloneNode(temp), node)));
-      nodes.push(expressionStatement(cloneNode(temp)));
     }
 
     return this.replaceExpressionWithStatements(nodes);

package/lib/scope/index.js

@@ -848,8 +848,8 @@ class Scope {
     }
 
     const declarator = variableDeclarator(opts.id, opts.init);
-    declarPath.node.declarations.push(declarator);
-    this.registerBinding(kind, declarPath.get("declarations").pop());
+    const len = declarPath.node.declarations.push(declarator);
+    path.scope.registerBinding(kind, declarPath.get("declarations")[len - 1]);
   }
 
   getProgramParent() {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@babel/traverse",
-  "version": "7.17.0",
+  "version": "7.17.3",
   "description": "The Babel Traverse module maintains the overall tree state, and is responsible for replacing, removing, and adding nodes",
   "author": "The Babel Team (https://babel.dev/team)",
   "homepage": "https://babel.dev/docs/en/next/babel-traverse",
@@ -17,12 +17,12 @@
   "main": "./lib/index.js",
   "dependencies": {
     "@babel/code-frame": "^7.16.7",
-    "@babel/generator": "^7.17.0",
+    "@babel/generator": "^7.17.3",
     "@babel/helper-environment-visitor": "^7.16.7",
     "@babel/helper-function-name": "^7.16.7",
     "@babel/helper-hoist-variables": "^7.16.7",
     "@babel/helper-split-export-declaration": "^7.16.7",
-    "@babel/parser": "^7.17.0",
+    "@babel/parser": "^7.17.3",
     "@babel/types": "^7.17.0",
     "debug": "^4.1.0",
     "globals": "^11.1.0"