@putout/plugin-variables 1.1.0 → 1.3.0

package/lib/remove-unused/index.js

@@ -0,0 +1,40 @@
+import {operator} from 'putout';
+import getVars from './get-vars/index.js';
+import transform from './transform.js';
+import getUnused from './get-unused.js';
+
+const {
+    replaceWith,
+    compare,
+    remove,
+} = operator;
+
+export const report = ({name}) => `'${name}' is defined but never used`;
+
+export const fix = ({path}) => {
+    if (compare(path, 'const __a = __b = __c'))
+        return replaceWith(path.parentPath, path.node.init);
+    
+    if (isOneImport(path))
+        return remove(path.parentPath);
+    
+    remove(path);
+};
+
+export const find = (ast, {traverse}) => {
+    const vars = getVars(ast, {
+        setPath: true,
+        traverse,
+    });
+    
+    const transformed = transform(vars);
+    
+    return getUnused(transformed);
+};
+
+function isOneImport({parentPath}) {
+    if (!parentPath.isImportDeclaration())
+        return false;
+    
+    return parentPath.node.specifiers.length === 1;
+}

package/lib/remove-unused/transform.js

@@ -0,0 +1,18 @@
+export default (items) => {
+    const result = [];
+    
+    for (const item of items) {
+        const entries = Object.entries(item);
+        
+        for (const entry of entries) {
+            result.push(transform(entry));
+        }
+    }
+    
+    return result;
+};
+
+const transform = ([name, value]) => ({
+    name,
+    ...value,
+});

package/lib/remove-useless-rename/index.js

@@ -3,7 +3,7 @@ import {operator, types} from 'putout';
 const {remove, rename} = operator;
 const {isIdentifier} = types;
 
-export const report = ({idName}) => `Useless variable declaration with name "${idName}"`;
+export const report = ({idName}) => `Avoid useless variable declaration with name '${idName}'`;
 
 export const fix = ({path, bindingPath, initName, idName}) => {
     rename(bindingPath, initName, idName);

package/lib/split-declarations/index.js

@@ -0,0 +1,67 @@
+import {types, operator} from 'putout';
+
+const {
+    isForStatement,
+    variableDeclaration,
+} = types;
+
+const {
+    replaceWithMultiple,
+    isKeyword,
+} = operator;
+
+export const report = () => 'Variables should be declared separately';
+
+export const fix = (path) => {
+    const {node} = path;
+    const varNodes = getVarNodes(node);
+    
+    replaceWithMultiple(path, varNodes);
+};
+
+export const traverse = ({push}) => ({
+    VariableDeclaration(path) {
+        const {
+            node,
+            parent,
+            parentPath,
+        } = path;
+        
+        const {declarations} = node;
+        
+        if (parentPath.isExportDeclaration())
+            return;
+        
+        if (declarations.length === 1)
+            return;
+        
+        for (const declaration of declarations) {
+            const {name} = declaration.id;
+            
+            if (isKeyword(name))
+                return;
+        }
+        
+        const init = node;
+        
+        if (isForStatement(parent, {init}))
+            return;
+        
+        push(path);
+    },
+});
+
+function getVarNodes({kind, declarations}) {
+    const result = [];
+    
+    for (const declaration of declarations) {
+        const declarations = [declaration];
+        
+        result.push(variableDeclaration(
+            kind,
+            declarations,
+        ));
+    }
+    
+    return result;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-variables",
-  "version": "1.1.0",
+  "version": "1.3.0",
   "type": "module",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin adds ability to find and remove useless",
@@ -40,6 +40,8 @@
     "@putout/plugin-for-of": "*",
     "@putout/plugin-maybe": "*",
     "@putout/plugin-minify": "*",
+    "@putout/plugin-nodejs": "*",
+    "@putout/plugin-putout": "*",
     "@putout/plugin-remove-useless-array-from": "*",
     "@putout/plugin-reuse-duplicate-init": "*",
     "@putout/test": "^14.0.0",
@@ -47,8 +49,11 @@
     "eslint": "v10.0.0-alpha.0",
     "eslint-plugin-n": "^17.0.0",
     "eslint-plugin-putout": "^29.0.0",
+    "just-camel-case": "^6.2.0",
     "madrun": "^11.0.0",
-    "nodemon": "^3.0.1"
+    "nodemon": "^3.0.1",
+    "supertape": "^11.3.1",
+    "try-catch": "^3.0.1"
   },
   "peerDependencies": {
     "putout": ">=41"