@putout/plugin-variables 1.0.0 → 1.1.0

package/README.md

@@ -155,6 +155,34 @@ function DestructuringErrors(a, b) {
 bc = b.c.replace('x', 'y');
 ```
 
+## remove-unreferenced
+
+> A **variable** is a named reference to a **value**.
+>
+> (c) [MDN](https://developer.mozilla.org/en-US/docs/Glossary/Variable)
+
+### ❌ Example of incorrect code
+
+```js
+let a;
+let b;
+
+a = 5;
+b = 6;
+
+console.log(a);
+```
+
+### ✅ Example of correct code
+
+```js
+let a;
+
+a = 5;
+
+console.log(a);
+```
+
 ## License
 
 MIT

package/lib/index.js

@@ -3,6 +3,7 @@ import * as removeUselessAssignment from './remove-useless-assignment/index.js';
 import * as removeUselessDeclarations from './remove-useless-declarations/index.js';
 import * as removeUselessDuplicates from './remove-useless-duplicates/index.js';
 import * as removeUselessRename from './remove-useless-rename/index.js';
+import * as removeUnreferenced from './remove-unreferenced/index.js';
 
 export const rules = {
     'remove-useless': removeUseless,
@@ -10,4 +11,5 @@ export const rules = {
     'remove-useless-declarations': removeUselessDeclarations,
     'remove-useless-duplicates': removeUselessDuplicates,
     'remove-useless-rename': removeUselessRename,
+    'remove-unreferenced': removeUnreferenced,
 };

package/lib/remove-unreferenced/index.js

@@ -0,0 +1,98 @@
+import {operator} from 'putout';
+
+const {remove} = operator;
+
+export const report = () => 'Avoid unreferenced variables';
+
+export const fix = (path) => remove(path);
+
+export const traverse = ({push}) => ({
+    'return __a'(path) {
+        const arg = path.get('argument');
+        
+        if (!arg.isIdentifier())
+            return false;
+        
+        const binding = path.scope.bindings[arg.node.name];
+        
+        if (!binding)
+            return false;
+        
+        if (binding.constantViolations.length)
+            return false;
+        
+        if (binding.referencePaths.length !== 1)
+            return false;
+        
+        if (!binding.path.isVariableDeclarator())
+            return false;
+        
+        if (binding.path.node.init)
+            return false;
+        
+        push(path);
+    },
+    '__identifier = __a'(path) {
+        const {parentPath} = path;
+        
+        if (parentPath.isMemberExpression())
+            return;
+        
+        if (parentPath.isConditionalExpression())
+            return;
+        
+        const {name} = path.node.left;
+        const binding = path.scope.getAllBindings()[name];
+        
+        if (!binding)
+            return;
+        
+        if (path.find(isInsideForOf))
+            return;
+        
+        const {referenced} = binding;
+        
+        if (referenced)
+            return;
+        
+        if (binding.path.isObjectPattern()) {
+            const propPath = getPropertyPath(binding.path, name);
+            
+            push(path);
+            push(propPath);
+            
+            return;
+        }
+        
+        const idPath = binding.path.get('id');
+        
+        if (binding.path.isVariableDeclarator() && idPath.isObjectPattern()) {
+            const propPath = getPropertyPath(idPath, name);
+            
+            push(path);
+            push(propPath);
+            
+            return;
+        }
+        
+        push(path);
+        push(binding.path);
+    },
+});
+
+function getPropertyPath(path, name) {
+    let propPath;
+    
+    for (propPath of path.get('properties')) {
+        const {key, shorthand} = propPath.node;
+        
+        if (shorthand && key.name !== name)
+            continue;
+        
+        break;
+    }
+    
+    return propPath;
+}
+
+const isInsideForOf = (path) => path.__putout_for_of_reduce;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-variables",
-  "version": "1.0.0",
+  "version": "1.1.0",
   "type": "module",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin adds ability to find and remove useless",
@@ -37,7 +37,9 @@
   "devDependencies": {
     "@putout/eslint-flat": "^3.0.0",
     "@putout/plugin-declare": "*",
+    "@putout/plugin-for-of": "*",
     "@putout/plugin-maybe": "*",
+    "@putout/plugin-minify": "*",
     "@putout/plugin-remove-useless-array-from": "*",
     "@putout/plugin-reuse-duplicate-init": "*",
     "@putout/test": "^14.0.0",