feature: @putout/plugin-for-of: convert to ESM

Author: coderaiser

packages/plugin-for-of/lib/add-missing-declaration/index.js

@@ -1,16 +1,15 @@
-'use strict';
+import {types} from 'putout';
 
-const {types} = require('putout');
 const {
     isVariableDeclaration,
     isIdentifier,
     isArrayPattern,
     isObjectPattern,
 } = types;
 
-module.exports.report = () => `Add missing declaration`;
+export const report = () => `Add missing declaration`;
 
-module.exports.match = () => ({
+export const match = () => ({
     'for (__a of __b) __c': ({__a}, path) => {
         if (isVariableDeclaration(__a))
             return false;
@@ -42,6 +41,6 @@ module.exports.match = () => ({
     },
 });
 
-module.exports.replace = () => ({
+export const replace = () => ({
     'for (__a of __b) __c': 'for (const __a of __b) __c',
 });

packages/plugin-for-of/lib/add-missing-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: [
         ['add-missing-declaration', plugin],
     ],

packages/plugin-for-of/lib/for-each/index.js

@@ -1,10 +1,8 @@
-'use strict';
-
-const {
+import {
     template,
     operator,
     types,
-} = require('putout');
+} from 'putout';
 
 const {
     isIdentifier,
@@ -30,9 +28,9 @@ const forOfEntriesTemplate = template(`
 const {keys} = Object;
 const isRoot = (path) => path.isFunction() || path.isProgram();
 
-module.exports.report = () => `Use 'for-of' instead of 'forEach()'`;
+export const report = () => `Use 'for-of' instead of 'forEach()'`;
 
-module.exports.replace = () => ({
+export const replace = () => ({
     '__.forEach.call(__a, (__b) => __body)': 'for (const __b of __a) __body',
     '__.forEach(__args)': (vars, path) => {
         const {params, body} = path.node.arguments[0];
@@ -76,7 +74,7 @@ module.exports.replace = () => ({
     },
 });
 
-module.exports.match = () => ({
+export const match = () => ({
     '__.forEach(__args)': (vars, path) => {
         const {parentPath} = path;
 

packages/plugin-for-of/lib/for-each/index.spec.js

@@ -1,15 +1,14 @@
-'use strict';
-
-const {createTest} = require('@putout/test');
-
-const convertConstToLet = require('@putout/plugin-convert-const-to-let');
-const removeUselessContinue = require('@putout/plugin-remove-useless-continue');
-const plugin = require('.');
-
-const removeUselessVariables = require('../remove-useless-variables');
+import {createRequire} from 'node:module';
+import {createTest} from '@putout/test';
+import * as convertConstToLet from '@putout/plugin-convert-const-to-let';
+import * as removeUselessContinue from '@putout/plugin-remove-useless-continue';
+import * as plugin from './index.js';
+import * as removeUselessVariables from '../remove-useless-variables/index.js';
+
+const require = createRequire(import.meta.url);
 const convertComparisonToBoolean = require('@putout/plugin-conditions').rules['convert-comparison-to-boolean'];
 
-const test = createTest(__dirname, {
+const test = createTest(import.meta.url, {
     plugins: [
         ['for-of/for-each', plugin],
     ],

packages/plugin-for-of/lib/for-entries-n/index.js

@@ -1,10 +1,8 @@
-'use strict';
-
-const {
+import {
     operator,
     template,
     types,
-} = require('putout');
+} from 'putout';
 
 const {
     compare,
@@ -13,14 +11,14 @@ const {
 
 const {isBlockStatement} = types;
 
-module.exports.report = () => `Use 'for...of' instead of 'for'`;
+export const report = () => `Use 'for...of' instead of 'for'`;
 
 const forLoopToN = 'for (let __i = 0; __i < __n; __i++) __c';
 const getForOfLoop = template(`for (const [INDEX, LEFT] of RIGHT.entries()) BODY`);
 const assignIterable = (__i) => `const __a = __b[${__i.name}]`;
 const assignN = (__n) => `const ${__n.name} = __e.length`;
 
-module.exports.filter = (path) => {
+export const filter = (path) => {
     const {node} = path;
     const prevPath = path.getPrevSibling();
 
@@ -57,7 +55,7 @@ module.exports.filter = (path) => {
     return references > 3;
 };
 
-module.exports.replace = () => ({
+export const replace = () => ({
     [forLoopToN]: ({__c, __i}) => {
         const [node] = __c.body;
         const {__a, __b} = getTemplateValues(node, assignIterable(__i));

packages/plugin-for-of/lib/for-entries-n/index.spec.js

@@ -1,10 +1,8 @@
-'use strict';
+import montag from 'montag';
+import {createTest} from '@putout/test';
+import * as entriesN from './index.js';
 
-const montag = require('montag');
-const {createTest} = require('@putout/test');
-const entriesN = require('./index.js');
-
-const test = createTest(__dirname, {
+const test = createTest(import.meta.url, {
     plugins: [
         ['convert-for-to-for-of/entries-n', entriesN],
     ],

packages/plugin-for-of/lib/for-entries/index.js

@@ -1,10 +1,8 @@
-'use strict';
-
-const {
+import {
     operator,
     types,
     template,
-} = require('putout');
+} from 'putout';
 
 const {
     compare,
@@ -13,14 +11,14 @@ const {
 
 const {isBlockStatement} = types;
 
-module.exports.report = () => `Use 'for...of' instead of 'for'`;
+export const report = () => `Use 'for...of' instead of 'for'`;
 
 const forLoop = 'for (let __i = 0; __i < __e.length; __i++) __c';
 const getForOfLoop = template(`for (const [INDEX, ITEM] of ITEMS.entries()) BODY`);
 const assignIterable = (__i) => `const __a = __b[${__i.name}]`;
 const assignIterableWithName = (__i, __e) => `const __a = ${__e.name}[${__i.name}]`;
 
-module.exports.filter = (path) => {
+export const filter = (path) => {
     const {node} = path;
     const {body} = node;
 
@@ -43,7 +41,7 @@ module.exports.filter = (path) => {
     return compare(first, assignIterableWithName(__i, __e));
 };
 
-module.exports.replace = () => ({
+export const replace = () => ({
     [forLoop]: ({__c, __i}) => {
         const [node] = __c.body;
         const {__a, __b} = getTemplateValues(node, assignIterable(__i));

packages/plugin-for-of/lib/for-entries/index.spec.js

@@ -1,10 +1,8 @@
-'use strict';
+import removeUnusedVariables from '@putout/plugin-remove-unused-variables';
+import {createTest} from '@putout/test';
+import * as forEntries from './index.js';
 
-const removeUnusedVariables = require('@putout/plugin-remove-unused-variables');
-const {createTest} = require('@putout/test');
-const forEntries = require('.');
-
-const test = createTest(__dirname, {
+const test = createTest(import.meta.url, {
     plugins: [
         ['for-of/for-entries', forEntries],
     ],

packages/plugin-for-of/lib/for-in-negative/index.js

@@ -1,15 +1,13 @@
-'use strict';
-
-const {generate, operator} = require('putout');
+import {generate, operator} from 'putout';
 
 const {
     contains,
     getTemplateValues,
 } = operator;
 
-module.exports.report = () => `for-of should be used instead of for-in`;
+export const report = () => `for-of should be used instead of for-in`;
 
-module.exports.match = () => ({
+export const match = () => ({
     'for (__a in __b) __body': ({__a, __b, __body}) => {
         const declaration = getTemplateValues(__a, 'var __a');
 
@@ -22,7 +20,7 @@ module.exports.match = () => ({
     },
 });
 
-module.exports.replace = () => ({
+export const replace = () => ({
     'for (__a in __b) __body': ({__b, __body}) => {
         const [first] = __body.body;
         const condition = getTemplateValues(first, 'if (!__b.hasOwnProperty(__a)) __c');

packages/plugin-for-of/lib/for-in-negative/index.spec.js

@@ -1,9 +1,7 @@
-'use strict';
+import {createTest} from '@putout/test';
+import * as negative from './index.js';
 
-const {createTest} = require('@putout/test');
-const negative = require('.');
-
-const test = createTest(__dirname, {
+const test = createTest(import.meta.url, {
     plugins: [
         ['convert-for-in-to-for-of/negative', negative],
     ],

packages/plugin-for-of/lib/for-in-positive/index.js

@@ -1,15 +1,13 @@
-'use strict';
-
-const {generate, operator} = require('putout');
+import {generate, operator} from 'putout';
 
 const {
     contains,
     getTemplateValues,
 } = operator;
 
-module.exports.report = () => `for-of should be used instead of for-in`;
+export const report = () => `for-of should be used instead of for-in`;
 
-module.exports.match = () => ({
+export const match = () => ({
     'for (__a in __b) __body': ({__a, __b, __body}) => {
         const declaration = getTemplateValues(__a, 'var __a');
 
@@ -22,7 +20,7 @@ module.exports.match = () => ({
     },
 });
 
-module.exports.replace = () => ({
+export const replace = () => ({
     'for (__a in __b) __body': ({__b, __body}) => {
         const [first] = __body.body;
         const condition = getTemplateValues(first, 'if (__b.hasOwnProperty(__a)) __body');

packages/plugin-for-of/lib/for-in-positive/index.spec.js

@@ -1,9 +1,7 @@
-'use strict';
+import {createTest} from '@putout/test';
+import * as convertForInToForOf from './index.js';
 
-const {createTest} = require('@putout/test');
-const convertForInToForOf = require('.');
-
-const test = createTest(__dirname, {
+const test = createTest(import.meta.url, {
     plugins: [
         ['convert-for-in-to-for-of', convertForInToForOf],
     ],

packages/plugin-for-of/lib/for-length/index.js

@@ -1,10 +1,8 @@
-'use strict';
-
-const {
+import {
     operator,
     types,
     template,
-} = require('putout');
+} from 'putout';
 
 const {
     compare,
@@ -13,14 +11,14 @@ const {
 
 const {isBlockStatement} = types;
 
-module.exports.report = () => `Use 'for...of' instead of 'for'`;
+export const report = () => `Use 'for...of' instead of 'for'`;
 
 const forLoop = 'for (let __i = 0; __i < __e.length; __i++) __c';
 const getForOfLoop = template(`for (const LEFT of RIGHT) BODY`);
 const assignIterable = (__i) => `const __a = __b[${__i.name}]`;
 const assignIterableWithName = (__i, __e) => `const __a = ${__e.name}[${__i.name}]`;
 
-module.exports.filter = (path) => {
+export const filter = (path) => {
     const {node} = path;
     const {body} = node;
 
@@ -37,7 +35,7 @@ module.exports.filter = (path) => {
     return compare(first, assignIterableWithName(__i, __e));
 };
 
-module.exports.replace = () => ({
+export const replace = () => ({
     [forLoop]: ({__c, __i}) => {
         const [node] = __c.body;
         const {__a, __b} = getTemplateValues(node, assignIterable(__i));

packages/plugin-for-of/lib/for-length/index.spec.js

@@ -1,10 +1,8 @@
-'use strict';
+import removeUnusedVariables from '@putout/plugin-remove-unused-variables';
+import {createTest} from '@putout/test';
+import * as convertForToForOf from './index.js';
 
-const removeUnusedVariables = require('@putout/plugin-remove-unused-variables');
-const {createTest} = require('@putout/test');
-const convertForToForOf = require('./index.js');
-
-const test = createTest(__dirname, {
+const test = createTest(import.meta.url, {
     plugins: [
         ['for-of/length', convertForToForOf],
     ],

packages/plugin-for-of/lib/for-n/index.js

@@ -1,10 +1,8 @@
-'use strict';
-
-const {
+import {
     operator,
     template,
     types,
-} = require('putout');
+} from 'putout';
 
 const {
     compare,
@@ -13,14 +11,14 @@ const {
 
 const {isBlockStatement} = types;
 
-module.exports.report = () => `Use 'for...of' instead of 'for'`;
+export const report = () => `Use 'for...of' instead of 'for'`;
 
 const forLoopToN = 'for (let __i = 0; __i < __n; __i++) __c';
 const getForOfLoop = template(`for (const LEFT of RIGHT) BODY`);
 const assignIterable = (__i) => `const __a = __b[${__i.name}]`;
 const assignN = (__n) => `const ${__n.name} = __e.length`;
 
-module.exports.filter = (path) => {
+export const filter = (path) => {
     const {node} = path;
     const prevPath = path.getPrevSibling();
 
@@ -46,7 +44,7 @@ module.exports.filter = (path) => {
     return references <= 3;
 };
 
-module.exports.replace = () => ({
+export const replace = () => ({
     [forLoopToN]: ({__c, __i}) => {
         const [node] = __c.body;
         const {__a, __b} = getTemplateValues(node, assignIterable(__i));

packages/plugin-for-of/lib/for-n/index.spec.js

@@ -1,9 +1,10 @@
-'use strict';
+import {createRequire} from 'node:module';
+import {createTest} from '@putout/test';
+import * as forN from './index.js';
 
-const {createTest} = require('@putout/test');
-const forN = require('.');
+const require = createRequire(import.meta.url);
 
-const test = createTest(__dirname, {
+const test = createTest(import.meta.url, {
     plugins: [
         ['for-of/n', forN],
     ],