feature: @putout/plugin-remove-useless-variables: declaration: re-ass…

…ign (coderaiser/minify#128)
coderaiser · Sep 26, 2024 · 559fcd1 · 559fcd1
1 parent 05a2b37
commit 559fcd1
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 0 deletions.
diff --git a/packages/plugin-remove-useless-variables/lib/declaration/fixture/re-assign.js b/packages/plugin-remove-useless-variables/lib/declaration/fixture/re-assign.js
@@ -0,0 +1,7 @@
+let newVal, oldAddr = oldState.config.address[id];
+
+if (oldAddr != null) {
+    newVal = getter(state);
+}
+
+state.values[idx] = newVal;
diff --git a/packages/plugin-remove-useless-variables/lib/declaration/index.js b/packages/plugin-remove-useless-variables/lib/declaration/index.js
@@ -55,6 +55,9 @@ module.exports.match = ({options}) => ({
         if (binding.referencePaths.length !== 1)
             return false;
 
+        if (binding.constantViolations.length)
+            return false;
+
         if (!binding.path.isVariableDeclarator())
             return false;
 

diff --git a/packages/plugin-remove-useless-variables/lib/declaration/index.spec.js b/packages/plugin-remove-useless-variables/lib/declaration/index.spec.js
@@ -96,3 +96,8 @@ test('remove usless variables: declaration: no transform: other-scope', (t) => {
     t.noTransform('other-scope');
     t.end();
 });
+
+test('remove usless variables: declaration: no transform: re-assigned', (t) => {
+    t.noTransform('re-assign');
+    t.end();
+});