fix: renameVisitor should skip when a pattern is a scope

packages/babel-traverse/src/scope/lib/renamer.js

@@ -9,8 +9,9 @@ const renameVisitor = {
     }
   },
 
-  Scope(path, state) {
+  "Pattern|Scope"(path, state) {
     if (
+      path.isScope() &&
       !path.scope.bindingIdentifierEquals(
         state.oldName,
         state.binding.identifier,

packages/babel-traverse/test/fixtures/rename/parameter-default-1/input.js

@@ -0,0 +1,7 @@
+let a = "outside";
+
+function f(g = () => a) {
+  let a = "inside";
+  return g();
+}
+

packages/babel-traverse/test/fixtures/rename/parameter-default-1/options.json

@@ -0,0 +1,3 @@
+{
+  "plugins": ["./plugin"]
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-1/output.js

@@ -0,0 +1,6 @@
+let a = "outside";
+
+function f(g = () => a) {
+  let z = "inside";
+  return g();
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-1/plugin.js

@@ -0,0 +1,9 @@
+module.exports = function() {
+  return {
+    visitor: {
+      FunctionDeclaration(path) {
+        path.scope.rename("a", "z");
+      }
+    }
+  };
+};

packages/babel-traverse/test/fixtures/rename/parameter-default-10/input.js

@@ -0,0 +1,5 @@
+let a = "outside";
+
+function r({ a: b }, { [a]: { c } = a }) {
+  g(a);
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-10/options.json

@@ -0,0 +1,3 @@
+{
+  "plugins": ["./plugin"]
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-10/output.js

@@ -0,0 +1,11 @@
+let z = "outside";
+
+function r({
+  a: b
+}, {
+  [z]: {
+    c
+  } = z
+}) {
+  g(z);
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-10/plugin.js

@@ -0,0 +1,9 @@
+module.exports = function() {
+  return {
+    visitor: {
+      FunctionDeclaration(path) {
+        path.scope.rename("a", "z");
+      }
+    }
+  };
+};

packages/babel-traverse/test/fixtures/rename/parameter-default-2/input.js

@@ -0,0 +1,5 @@
+let a = "outside";
+
+function h(a, g = () => a) {
+  return g();
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-2/options.json

@@ -0,0 +1,3 @@
+{
+  "plugins": ["./plugin"]
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-2/output.js

@@ -0,0 +1,5 @@
+let a = "outside";
+
+function h(z, g = () => z) {
+  return g();
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-2/plugin.js

@@ -0,0 +1,9 @@
+module.exports = function() {
+  return {
+    visitor: {
+      FunctionDeclaration(path) {
+        path.scope.rename("a", "z");
+      }
+    }
+  };
+};

packages/babel-traverse/test/fixtures/rename/parameter-default-3/input.js

@@ -0,0 +1,6 @@
+let a = "outside";
+
+function j(g = a) {
+  let a = "inside";
+  return g;
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-3/options.json

@@ -0,0 +1,3 @@
+{
+  "plugins": ["./plugin"]
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-3/output.js

@@ -0,0 +1,6 @@
+let a = "outside";
+
+function j(g = a) {
+  let z = "inside";
+  return g;
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-3/plugin.js

@@ -0,0 +1,9 @@
+module.exports = function() {
+  return {
+    visitor: {
+      FunctionDeclaration(path) {
+        path.scope.rename("a", "z");
+      }
+    }
+  };
+};

packages/babel-traverse/test/fixtures/rename/parameter-default-4/input.js

@@ -0,0 +1,8 @@
+let a = "outside";
+
+function k([{
+  g = a
+}]) {
+  let a = "inside";
+  return g;
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-4/options.json

@@ -0,0 +1,3 @@
+{
+  "plugins": ["./plugin"]
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-4/output.js

@@ -0,0 +1,8 @@
+let a = "outside";
+
+function k([{
+  g = a
+}]) {
+  let z = "inside";
+  return g;
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-4/plugin.js

@@ -0,0 +1,9 @@
+module.exports = function() {
+  return {
+    visitor: {
+      FunctionDeclaration(path) {
+        path.scope.rename("a", "z");
+      }
+    }
+  };
+};

packages/babel-traverse/test/fixtures/rename/parameter-default-5/input.js

@@ -0,0 +1,8 @@
+let a = "outside";
+
+function f([{
+  [a]: g
+}]) {
+  let a = "inside";
+  return g;
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-5/options.json

@@ -0,0 +1,3 @@
+{
+  "plugins": ["./plugin"]
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-5/output.js

@@ -0,0 +1,8 @@
+let a = "outside";
+
+function f([{
+  [a]: g
+}]) {
+  let z = "inside";
+  return g;
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-5/plugin.js

@@ -0,0 +1,9 @@
+module.exports = function() {
+  return {
+    visitor: {
+      FunctionDeclaration(path) {
+        path.scope.rename("a", "z");
+      }
+    }
+  };
+};

packages/babel-traverse/test/fixtures/rename/parameter-default-6/input.js

@@ -0,0 +1,5 @@
+let a = "outside";
+
+function n(g = (a = a) => {}) {
+  let a = "inside";
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-6/options.json

@@ -0,0 +1,3 @@
+{
+  "plugins": ["./plugin"]
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-6/output.js

@@ -0,0 +1,5 @@
+let a = "outside";
+
+function n(g = (a = a) => {}) {
+  let z = "inside";
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-6/plugin.js

@@ -0,0 +1,9 @@
+module.exports = function() {
+  return {
+    visitor: {
+      FunctionDeclaration(path) {
+        path.scope.rename("a", "z");
+      }
+    }
+  };
+};

packages/babel-traverse/test/fixtures/rename/parameter-default-7/input.js

@@ -0,0 +1,5 @@
+let a = "outside";
+
+function n(a, g = (a = a) => {}) {
+  a = "inside";
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-7/options.json

@@ -0,0 +1,3 @@
+{
+  "plugins": ["./plugin"]
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-7/output.js

@@ -0,0 +1,5 @@
+let a = "outside";
+
+function n(z, g = (a = a) => {}) {
+  z = "inside";
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-7/plugin.js

@@ -0,0 +1,9 @@
+module.exports = function() {
+  return {
+    visitor: {
+      FunctionDeclaration(path) {
+        path.scope.rename("a", "z");
+      }
+    }
+  };
+};

packages/babel-traverse/test/fixtures/rename/parameter-default-8/input.js

@@ -0,0 +1,5 @@
+let a = "outside";
+
+function q(a, g = (b = a) => b) {
+  g(a);
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-8/options.json

@@ -0,0 +1,3 @@
+{
+  "plugins": ["./plugin"]
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-8/output.js

@@ -0,0 +1,5 @@
+let a = "outside";
+
+function q(z, g = (b = z) => b) {
+  g(z);
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-8/plugin.js

@@ -0,0 +1,9 @@
+module.exports = function() {
+  return {
+    visitor: {
+      FunctionDeclaration(path) {
+        path.scope.rename("a", "z");
+      }
+    }
+  };
+};

packages/babel-traverse/test/fixtures/rename/parameter-default-9/input.js

@@ -0,0 +1,5 @@
+let a = "outside";
+
+function r(a, g = (a, b = a) => a) {
+  g(a);
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-9/options.json

@@ -0,0 +1,3 @@
+{
+  "plugins": ["./plugin"]
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-9/output.js

@@ -0,0 +1,5 @@
+let a = "outside";
+
+function r(z, g = (a, b = a) => a) {
+  g(z);
+}

packages/babel-traverse/test/fixtures/rename/parameter-default-9/plugin.js

@@ -0,0 +1,9 @@
+module.exports = function() {
+  return {
+    visitor: {
+      FunctionDeclaration(path) {
+        path.scope.rename("a", "z");
+      }
+    }
+  };
+};