feature: @putout/plugin-remove-useless-variabes: migrate to ESM

coderaiser · coderaiser · commit 62dc1ff0e37a · 2025-05-04T18:33:38.000+03:00
diff --git a/packages/plugin-remove-useless-variables/.madrun.js b/packages/plugin-remove-useless-variables/.madrun.js
diff --git a/packages/plugin-remove-useless-variables/eslint.config.js b/packages/plugin-remove-useless-variables/eslint.config.js
diff --git a/packages/plugin-remove-useless-variables/lib/assignment/index.js b/packages/plugin-remove-useless-variables/lib/assignment/index.js
@@ -1,13 +1,11 @@
-'use strict';
-
-const {types, operator} = require('putout');
+import {types, operator} from 'putout';
 
 const {getBinding} = operator;
 const {isIdentifier} = types;
 
-module.exports.report = () => `Avoid useless assignment`;
+export const report = () => `Avoid useless assignment`;
 
-module.exports.match = () => ({
+export const match = () => ({
     '(__a = __b).__c': ({__a}, path) => {
         if (!isIdentifier(__a))
             return false;
@@ -23,6 +21,6 @@ module.exports.match = () => ({
     },
 });
 
-module.exports.replace = () => ({
+export const replace = () => ({
     '(__a = __b).__c': '__b.__c',
 });
diff --git a/packages/plugin-remove-useless-variables/lib/assignment/index.spec.js b/packages/plugin-remove-useless-variables/lib/assignment/index.spec.js
@@ -1,9 +1,7 @@
-'use strict';
+import {createTest} from '@putout/test';
+import * as plugin from './index.js';
 
-const {createTest} = require('@putout/test');
-const plugin = require('.');
-
-const test = createTest(__dirname, {
+const test = createTest(import.meta.url, {
     plugins: [
         ['assignment', plugin],
     ],
diff --git a/packages/plugin-remove-useless-variables/lib/declaration/index.js b/packages/plugin-remove-useless-variables/lib/declaration/index.js
@@ -1,6 +1,5 @@
-'use strict';
+import {types, operator} from 'putout';
 
-const {types, operator} = require('putout');
 const {
     compare,
     remove,
@@ -15,9 +14,9 @@ const getBody = (path) => {
     return body.body || body;
 };
 
-module.exports.report = () => `Avoid useless declarations`;
+export const report = () => `Avoid useless declarations`;
 
-module.exports.match = ({options}) => ({
+export const match = ({options}) => ({
     'return __a': ({__a}, path) => {
         const binding = path.scope.getAllBindings()[__a.name];
         
@@ -95,7 +94,7 @@ module.exports.match = ({options}) => ({
     },
 });
 
-module.exports.replace = () => ({
+export const replace = () => ({
     'return __a': ({__a}, path) => {
         const binding = path.scope.getAllBindings()[__a.name];
         const [ref] = binding.referencePaths;
diff --git a/packages/plugin-remove-useless-variables/lib/declaration/index.spec.js b/packages/plugin-remove-useless-variables/lib/declaration/index.spec.js
@@ -1,9 +1,7 @@
-'use strict';
+import {createTest} from '@putout/test';
+import * as plugin from './index.js';
 
-const {createTest} = require('@putout/test');
-const plugin = require('.');
-
-const test = createTest(__dirname, {
+const test = createTest(import.meta.url, {
     plugins: [
         ['remove-useless-variables/declarations', plugin],
     ],
diff --git a/packages/plugin-remove-useless-variables/lib/destruct/index.js b/packages/plugin-remove-useless-variables/lib/destruct/index.js
@@ -1,6 +1,4 @@
-'use strict';
-
-const {types, operator} = require('putout');
+import {types, operator} from 'putout';
 
 const {
     isIdentifier,
@@ -26,11 +24,11 @@ const getKeyLength = (a) => {
 
 const sum = (a, b) => a + getKeyLength(b);
 
-module.exports.report = (path) => {
+export const report = (path) => {
     return `Remove useless variable '${path.node.declarations[0].init.name}'`;
 };
 
-module.exports.match = () => ({
+export const match = () => ({
     'const __object = __a': ({__a, __object}, path) => {
         const {parentPath} = path.parentPath;
         
@@ -68,7 +66,7 @@ module.exports.match = () => ({
     },
 });
 
-module.exports.replace = () => ({
+export const replace = () => ({
     'const __object = __a': ({__object}, path) => {
         replaceWith(path.parentPath.parentPath.get('params.0'), __object);
     },
diff --git a/packages/plugin-remove-useless-variables/lib/destruct/index.spec.js b/packages/plugin-remove-useless-variables/lib/destruct/index.spec.js
@@ -1,9 +1,7 @@
-'use strict';
+import {createTest} from '@putout/test';
+import * as destruct from './index.js';
 
-const {createTest} = require('@putout/test');
-const destruct = require('.');
-
-const test = createTest(__dirname, {
+const test = createTest(import.meta.url, {
     plugins: [
         ['remove-useless-variables/destruct', destruct],
     ],
diff --git a/packages/plugin-remove-useless-variables/lib/duplicate/index.js b/packages/plugin-remove-useless-variables/lib/duplicate/index.js
@@ -1,14 +1,13 @@
-'use strict';
+import {operator} from 'putout';
 
-const {operator} = require('putout');
 const {getTemplateValues} = operator;
 const FN = 'const __a = function __a(__args) {__body}';
 
-module.exports.report = (path) => {
+export const report = (path) => {
     const {__a} = getTemplateValues(path, FN);
     return `Avoid duplicate declaration of '${__a.name}'`;
 };
 
-module.exports.replace = () => ({
+export const replace = () => ({
     'const __a = function __a(__args) {__body}': 'function __a(__args){__body}',
 });
diff --git a/packages/plugin-remove-useless-variables/lib/duplicate/index.spec.js b/packages/plugin-remove-useless-variables/lib/duplicate/index.spec.js
@@ -1,9 +1,7 @@
-'use strict';
+import {createTest} from '@putout/test';
+import * as plugin from './index.js';
 
-const {createTest} = require('@putout/test');
-const plugin = require('.');
-
-const test = createTest(__dirname, {
+const test = createTest(import.meta.url, {
     plugins: [
         ['duplicate', plugin],
     ],
diff --git a/packages/plugin-remove-useless-variables/lib/index.js b/packages/plugin-remove-useless-variables/lib/index.js
@@ -1,13 +1,11 @@
-'use strict';
+import * as rename from './rename/index.js';
+import * as remove from './remove/index.js';
+import * as destruct from './destruct/index.js';
+import * as declaration from './declaration/index.js';
+import * as duplicate from './duplicate/index.js';
+import * as assignment from './assignment/index.js';
 
-const rename = require('./rename');
-const remove = require('./remove');
-const destruct = require('./destruct');
-const declaration = require('./declaration');
-const duplicate = require('./duplicate');
-const assignment = require('./assignment');
-
-module.exports.rules = {
+export const rules = {
     rename,
     remove,
     destruct,
diff --git a/packages/plugin-remove-useless-variables/lib/remove/index.js b/packages/plugin-remove-useless-variables/lib/remove/index.js
@@ -1,15 +1,14 @@
-'use strict';
+import {operator} from 'putout';
 
-const {operator} = require('putout');
 const {remove, compare} = operator;
 
 const isNestedMemberExpression = (path) => {
     return path.isMemberExpression() && path.get('object').isMemberExpression();
 };
 
-module.exports.report = ({idName}) => `Useless variable declaration with name "${idName}"`;
+export const report = ({idName}) => `Useless variable declaration with name "${idName}"`;
 
-module.exports.fix = ({mainPath, path}) => {
+export const fix = ({mainPath, path}) => {
     const {init} = path.node;
     remove(path);
     
@@ -21,7 +20,7 @@ module.exports.fix = ({mainPath, path}) => {
     mainPath.node.init = init;
 };
 
-module.exports.traverse = ({push}) => ({
+export const traverse = ({push}) => ({
     AssignmentExpression(mainPath) {
         const leftPath = mainPath.get('left');
         const rightPath = mainPath.get('right');
diff --git a/packages/plugin-remove-useless-variables/lib/remove/index.spec.js b/packages/plugin-remove-useless-variables/lib/remove/index.spec.js
@@ -1,11 +1,8 @@
-'use strict';
+import {createTest} from '@putout/test';
+import * as reuseDuplicateInit from '@putout/plugin-reuse-duplicate-init';
+import * as remove from './index.js';
 
-const {createTest} = require('@putout/test');
-
-const reuseDuplicateInit = require('@putout/plugin-reuse-duplicate-init');
-const remove = require('.');
-
-const test = createTest(__dirname, {
+const test = createTest(import.meta.url, {
     plugins: [
         ['remove-useless-variables/remove', remove],
     ],
diff --git a/packages/plugin-remove-useless-variables/lib/rename/index.js b/packages/plugin-remove-useless-variables/lib/rename/index.js
@@ -1,18 +1,16 @@
-'use strict';
-
-const {operator, types} = require('putout');
+import {operator, types} from 'putout';
 
 const {remove, rename} = operator;
 const {isIdentifier} = types;
 
-module.exports.report = ({idName}) => `Useless variable declaration with name "${idName}"`;
+export const report = ({idName}) => `Useless variable declaration with name "${idName}"`;
 
-module.exports.fix = ({path, bindingPath, initName, idName}) => {
+export const fix = ({path, bindingPath, initName, idName}) => {
     rename(bindingPath, initName, idName);
     remove(path);
 };
 
-module.exports.traverse = ({push}) => ({
+export const traverse = ({push}) => ({
     VariableDeclarator(path) {
         const {node, parentPath} = path;
         
diff --git a/packages/plugin-remove-useless-variables/lib/rename/index.spec.js b/packages/plugin-remove-useless-variables/lib/rename/index.spec.js
@@ -1,9 +1,7 @@
-'use strict';
+import {createTest} from '@putout/test';
+import * as rename from './index.js';
 
-const {createTest} = require('@putout/test');
-const rename = require('.');
-
-const test = createTest(__dirname, {
+const test = createTest(import.meta.url, {
     plugins: [
         ['remove-useless-variables/rename', rename],
     ],
diff --git a/packages/plugin-remove-useless-variables/package.json b/packages/plugin-remove-useless-variables/package.json
@@ -1,7 +1,7 @@
 {
   "name": "@putout/plugin-remove-useless-variables",
   "version": "13.0.2",
-  "type": "commonjs",
+  "type": "module",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin adds ability to find and remove useless",
   "homepage": "https://github.com/coderaiser/putout/tree/master/packages/plugin-remove-useless-variables#readme",
@@ -47,11 +47,11 @@
     "nodemon": "^3.0.1"
   },
   "peerDependencies": {
-    "putout": ">=38"
+    "putout": ">=40"
   },
   "license": "MIT",
   "engines": {
-    "node": ">=18"
+    "node": ">=20"
   },
   "publishConfig": {
     "access": "public"
diff --git a/packages/plugin-remove-useless-variables/test/remove-useless-variables.js b/packages/plugin-remove-useless-variables/test/remove-useless-variables.js
@@ -1,9 +1,7 @@
-'use strict';
+import {createTest} from '@putout/test';
+import * as removeUselessVariables from '../lib/index.js';
 
-const {createTest} = require('@putout/test');
-const removeUselessVariables = require('..');
-
-const test = createTest(__dirname, {
+const test = createTest(import.meta.url, {
     plugins: [
         ['remove-useless-variables', removeUselessVariables],
     ],
