fix renaming variables in the same scope

lorenzoferre · Sep 18, 2024 · 9cc4900 · 9cc4900
1 parent 877d53c
commit 9cc4900
Showing 1 changed file with 16 additions and 12 deletions.
diff --git a/src/techniques/statics/rename-variable-same-scope.js b/src/techniques/statics/rename-variable-same-scope.js
@@ -2,19 +2,23 @@ export default function () {
   return {
     name: "rename-variable-same-scope",
     visitor: {
-      VariableDeclarator: {
+      Program: {
         enter(path) {
-          const { node, scope } = path;
-          const { id } = node;
-          const { name } = id;
-          const { parent } = scope;
-          if (!parent) return;
-          for (const binding in parent.bindings) {
-            if (binding === name) {
-              const newName = scope.generateUidIdentifier(name);
-              scope.rename(name, newName.name);
-            }
-          }
+          path.traverse({
+            VariableDeclarator(innerPath) {
+              const { node, scope } = innerPath;
+              const { id } = node;
+              const { name } = id;
+              const { parent } = scope;
+              if (!parent) return;
+              for (const binding in parent.bindings) {
+                if (binding === name) {
+                  const newName = scope.generateUidIdentifier(name);
+                  scope.rename(name, newName.name);
+                }
+              }
+            },
+          });
         },
       },
     },