feature: @putout/plugin-promises: convert to ESM

coderaiser · coderaiser · commit c30003a77e40 · 2025-03-27T15:18:35.000+02:00
diff --git a/packages/plugin-promises/.madrun.js b/packages/plugin-promises/.madrun.js
diff --git a/packages/plugin-promises/eslint.config.js b/packages/plugin-promises/eslint.config.js
diff --git a/packages/plugin-promises/lib/add-missing-async/index.js b/packages/plugin-promises/lib/add-missing-async/index.js
@@ -1,12 +1,10 @@
-'use strict';
+export const report = () => `Add missing 'async'`;
 
-module.exports.report = () => `Add missing 'async'`;
-
-module.exports.fix = (path) => {
+export const fix = (path) => {
     path.node.async = true;
 };
 
-module.exports.traverse = ({push}) => ({
+export const traverse = ({push}) => ({
     AwaitExpression(path) {
         const fnPath = path.getFunctionParent();
         
diff --git a/packages/plugin-promises/lib/add-missing-async/index.spec.js b/packages/plugin-promises/lib/add-missing-async/index.spec.js
@@ -1,9 +1,7 @@
-'use strict';
+import {createTest} from '@putout/test';
+import * as convert from './index.js';
 
-const {createTest} = require('@putout/test');
-const convert = require('.');
-
-const test = createTest(__dirname, {
+const test = createTest(import.meta.url, {
     plugins: [
         ['promises/add-missing-async', convert],
     ],
diff --git a/packages/plugin-promises/lib/add-missing-await/index.js b/packages/plugin-promises/lib/add-missing-await/index.js
@@ -1,14 +1,13 @@
-'use strict';
+import {types} from 'putout';
 
-const {types} = require('putout');
 const {
     isIdentifier,
     awaitExpression,
 } = types;
 
-module.exports.report = () => `Call async functions using 'await'`;
+export const report = () => `Call async functions using 'await'`;
 
-module.exports.match = () => ({
+export const match = () => ({
     '__a(__args)': ({__a}, path) => {
         if (!isIdentifier(__a))
             return false;
@@ -41,7 +40,7 @@ module.exports.match = () => ({
     },
 });
 
-module.exports.replace = () => ({
+export const replace = () => ({
     '__a(__args)': (vars, path) => {
         const fnPath = path.getFunctionParent();
         
diff --git a/packages/plugin-promises/lib/add-missing-await/index.spec.js b/packages/plugin-promises/lib/add-missing-await/index.spec.js
@@ -1,9 +1,7 @@
-'use strict';
+import {createTest} from '@putout/test';
+import * as convert from './index.js';
 
-const {createTest} = require('@putout/test');
-const convert = require('.');
-
-const test = createTest(__dirname, {
+const test = createTest(import.meta.url, {
     plugins: [
         ['promises/add-missing-await', convert],
     ],
diff --git a/packages/plugin-promises/lib/apply-await-import/index.js b/packages/plugin-promises/lib/apply-await-import/index.js
@@ -1,16 +1,14 @@
-'use strict';
-
-const {types, operator} = require('putout');
+import {types, operator} from 'putout';
 
 const {replaceWith} = operator;
 const {
     isFunction,
     awaitExpression,
 } = types;
 
-module.exports.report = () => `Use 'await' near 'import' call`;
+export const report = () => `Use 'await' near 'import' call`;
 
-module.exports.match = () => ({
+export const match = () => ({
     'import(__a)'(vars, path) {
         if (!path.parentPath.isVariableDeclarator())
             return false;
@@ -19,7 +17,7 @@ module.exports.match = () => ({
     },
 });
 
-module.exports.replace = () => ({
+export const replace = () => ({
     'import(__a)'(vars, path) {
         const fnPath = path.findParent(isFunction);
         
diff --git a/packages/plugin-promises/lib/apply-await-import/index.spec.js b/packages/plugin-promises/lib/apply-await-import/index.spec.js
@@ -1,9 +1,7 @@
-'use strict';
+import {createTest} from '@putout/test';
+import * as awaitImport from './index.js';
 
-const {createTest} = require('@putout/test');
-const awaitImport = require('.');
-
-const test = createTest(__dirname, {
+const test = createTest(import.meta.url, {
     plugins: [
         ['await-import', awaitImport],
     ],
diff --git a/packages/plugin-promises/lib/apply-top-level-await/index.js b/packages/plugin-promises/lib/apply-top-level-await/index.js
@@ -1,7 +1,7 @@
-'use strict';
+import fullstore from 'fullstore';
+import {operator} from 'putout';
 
-const fullstore = require('fullstore');
-const {replaceWithMultiple} = require('putout').operator;
+const {replaceWithMultiple} = operator;
 
 const add = ({push, isImports, isExports}) => (path) => {
     const calleePath = path.get('callee');
@@ -17,14 +17,14 @@ const add = ({push, isImports, isExports}) => (path) => {
     push(path);
 };
 
-module.exports.report = () => `Use top level 'await'`;
+export const report = () => `Use top level 'await'`;
 
-module.exports.fix = (path) => {
+export const fix = (path) => {
     const {body} = path.get('callee.body').node;
     replaceWithMultiple(path, body);
 };
 
-module.exports.traverse = ({push}) => {
+export const traverse = ({push}) => {
     const isExports = fullstore();
     const isImports = fullstore();
     
diff --git a/packages/plugin-promises/lib/apply-top-level-await/index.spec.js b/packages/plugin-promises/lib/apply-top-level-await/index.spec.js
@@ -1,9 +1,7 @@
-'use strict';
+import {createTest} from '@putout/test';
+import * as applyTopLevelAwait from './index.js';
 
-const {createTest} = require('@putout/test');
-const applyTopLevelAwait = require('.');
-
-const test = createTest(__dirname, {
+const test = createTest(import.meta.url, {
     plugins: [
         ['apply-top-level-await', applyTopLevelAwait],
     ],
diff --git a/packages/plugin-promises/lib/apply-with-resolvers/index.js b/packages/plugin-promises/lib/apply-with-resolvers/index.js
@@ -1,6 +1,5 @@
-'use strict';
+import {operator} from 'putout';
 
-const {operator} = require('putout');
 const {remove, getBindingPath} = operator;
 
 const PROMISE = `
@@ -10,9 +9,9 @@ const PROMISE = `
     });
 `;
 
-module.exports.report = () => `Apply 'Promise.withResolvers()'`;
+export const report = () => `Apply 'Promise.withResolvers()'`;
 
-module.exports.match = () => ({
+export const match = () => ({
     [PROMISE]: ({__c, __d}, path) => {
         if (!getBindingPath(path, __c.name))
             return false;
@@ -21,7 +20,7 @@ module.exports.match = () => ({
     },
 });
 
-module.exports.replace = () => ({
+export const replace = () => ({
     [PROMISE]: ({__c, __d, __p, __a, __b}, path) => {
         remove(getBindingPath(path, __c.name));
         remove(getBindingPath(path, __d.name));
diff --git a/packages/plugin-promises/lib/apply-with-resolvers/index.spec.js b/packages/plugin-promises/lib/apply-with-resolvers/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: [
         ['apply-with-resolvers', plugin],
     ],
diff --git a/packages/plugin-promises/lib/convert-new-promise-to-async/index.js b/packages/plugin-promises/lib/convert-new-promise-to-async/index.js
@@ -1,6 +1,5 @@
-'use strict';
+import {operator, types} from 'putout';
 
-const {operator, types} = require('putout');
 const {
     returnStatement,
     throwStatement,
@@ -18,9 +17,9 @@ const isPassedToFn = (path) => {
     return parentPath.get('callee') !== path;
 };
 
-module.exports.report = () => `Async functions should be used instead of 'new Promise()'`;
+export const report = () => `Async functions should be used instead of 'new Promise()'`;
 
-module.exports.match = () => ({
+export const match = () => ({
     'return new Promise(__a)': (vars, path) => {
         const {scope} = path.get('argument.arguments.0');
         const {resolve, reject} = scope.bindings;
@@ -48,7 +47,7 @@ module.exports.match = () => ({
     },
 });
 
-module.exports.replace = () => ({
+export const replace = () => ({
     'return new Promise(__a)'({__a}, path) {
         const {scope} = path.get('argument.arguments.0');
         
diff --git a/packages/plugin-promises/lib/convert-new-promise-to-async/index.spec.js b/packages/plugin-promises/lib/convert-new-promise-to-async/index.spec.js
@@ -1,9 +1,7 @@
-'use strict';
+import {createTest} from '@putout/test';
+import * as convert from './index.js';
 
-const {createTest} = require('@putout/test');
-const convert = require('.');
-
-const test = createTest(__dirname, {
+const test = createTest(import.meta.url, {
     plugins: [
         ['promises/convert-new-promise-to-async', convert],
     ],
diff --git a/packages/plugin-promises/lib/convert-reject-to-throw/index.js b/packages/plugin-promises/lib/convert-reject-to-throw/index.js
@@ -1,16 +1,15 @@
-'use strict';
+import {types} from 'putout';
 
-const {types} = require('putout');
 const {isFunction} = types;
 
-module.exports.report = () => 'Reject is useless in async functions, use throw instead';
+export const report = () => 'Reject is useless in async functions, use throw instead';
 
-module.exports.filter = (path) => {
+export const filter = (path) => {
     const fnPath = path.find(isFunction);
     return fnPath?.node.async;
 };
 
-module.exports.replace = () => ({
+export const replace = () => ({
     'return Promise.reject(__a)': 'throw __a',
     'return await Promise.reject(__a)': 'throw __a',
 });
diff --git a/packages/plugin-promises/lib/convert-reject-to-throw/index.spec.js b/packages/plugin-promises/lib/convert-reject-to-throw/index.spec.js
@@ -1,9 +1,7 @@
-'use strict';
+import {createTest} from '@putout/test';
+import * as convert from './index.js';
 
-const {createTest} = require('@putout/test');
-const convert = require('.');
-
-const test = createTest(__dirname, {
+const test = createTest(import.meta.url, {
     plugins: [
         ['promises/convert-reject-to-throw', convert],
     ],
diff --git a/packages/plugin-promises/lib/index.js b/packages/plugin-promises/lib/index.js
@@ -1,18 +1,16 @@
-'use strict';
+import * as removeUselessResolve from './remove-useless-resolve/index.js';
+import * as convertRejectToThrow from './convert-reject-to-throw/index.js';
+import * as convertNewPromiseToAsync from './convert-new-promise-to-async/index.js';
+import * as addMissingAwait from './add-missing-await/index.js';
+import * as addMissingAsync from './add-missing-async/index.js';
+import * as applyAwaitImport from './apply-await-import/index.js';
+import * as applyTopLevelAwait from './apply-top-level-await/index.js';
+import * as removeUselessAsync from './remove-useless-async/index.js';
+import * as removeUselessAwait from './remove-useless-await/index.js';
+import * as removeUselessVariables from './remove-useless-variables/index.js';
+import * as applyWithResolvers from './apply-with-resolvers/index.js';
 
-const removeUselessResolve = require('./remove-useless-resolve');
-const convertRejectToThrow = require('./convert-reject-to-throw');
-const convertNewPromiseToAsync = require('./convert-new-promise-to-async');
-const addMissingAwait = require('./add-missing-await');
-const addMissingAsync = require('./add-missing-async');
-const applyAwaitImport = require('./apply-await-import');
-const applyTopLevelAwait = require('./apply-top-level-await');
-const removeUselessAsync = require('./remove-useless-async');
-const removeUselessAwait = require('./remove-useless-await');
-const removeUselessVariables = require('./remove-useless-variables');
-const applyWithResolvers = require('./apply-with-resolvers');
-
-module.exports.rules = {
+export const rules = {
     'remove-useless-resolve': removeUselessResolve,
     'convert-reject-to-throw': convertRejectToThrow,
     'convert-new-promise-to-async': convertNewPromiseToAsync,
diff --git a/packages/plugin-promises/lib/remove-useless-async/index.js b/packages/plugin-promises/lib/remove-useless-async/index.js
@@ -1,19 +1,18 @@
-'use strict';
+import {operator} from 'putout';
 
-const {operator} = require('putout');
 const {contains} = operator;
 
-module.exports.report = () => `Avoid useless 'async'`;
+export const report = () => `Avoid useless 'async'`;
 
-module.exports.fix = (path) => {
+export const fix = (path) => {
     path.node.async = false;
 };
 
-module.exports.exclude = () => ['__nop'];
+export const exclude = () => ['__nop'];
 
-module.exports.include = () => [
+export const include = () => [
     'async function __(__args) {}',
     'async (__args) => __body',
 ];
 
-module.exports.filter = (path) => !contains(path, ['throw __', 'await __', 'for await (__ of __) __']);
+export const filter = (path) => !contains(path, ['throw __', 'await __', 'for await (__ of __) __']);
diff --git a/packages/plugin-promises/lib/remove-useless-async/index.spec.js b/packages/plugin-promises/lib/remove-useless-async/index.spec.js
@@ -1,9 +1,7 @@
-'use strict';
+import {createTest} from '@putout/test';
+import * as removeUselessAsync from './index.js';
 
-const {createTest} = require('@putout/test');
-const removeUselessAsync = require('.');
-
-const test = createTest(__dirname, {
+const test = createTest(import.meta.url, {
     plugins: [
         ['remove-useless-async', removeUselessAsync],
     ],
diff --git a/packages/plugin-promises/lib/remove-useless-await/index.js b/packages/plugin-promises/lib/remove-useless-await/index.js
@@ -1,11 +1,10 @@
-'use strict';
+import {operator} from 'putout';
 
-const {operator} = require('putout');
 const {traverse} = operator;
 
-module.exports.report = () => `Avoid useless 'await'`;
+export const report = () => `Avoid useless 'await'`;
 
-module.exports.exclude = () => [
+export const exclude = () => [
     'await new Promise(__args)',
     'await __a.__b?.(__args)',
     'await __a?.(__args)',
@@ -15,12 +14,12 @@ module.exports.exclude = () => [
     'await import(__a, __b)',
 ];
 
-module.exports.replace = () => ({
+export const replace = () => ({
     'await await __a': 'await __a',
     'await __a': '__a',
 });
 
-module.exports.filter = (path) => {
+export const filter = (path) => {
     const argumentPath = path.get('argument');
     
     if (argumentPath.isTaggedTemplateExpression())
diff --git a/packages/plugin-promises/lib/remove-useless-await/index.spec.js b/packages/plugin-promises/lib/remove-useless-await/index.spec.js
@@ -1,9 +1,7 @@
-'use strict';
+import {createTest} from '@putout/test';
+import * as removeUselessAwait from './index.js';
 
-const {createTest} = require('@putout/test');
-const removeUselessAwait = require('.');
-
-const test = createTest(__dirname, {
+const test = createTest(import.meta.url, {
     plugins: [
         ['remove-useless-await', removeUselessAwait],
     ],
diff --git a/packages/plugin-promises/lib/remove-useless-resolve/index.js b/packages/plugin-promises/lib/remove-useless-resolve/index.js
diff --git a/packages/plugin-promises/lib/remove-useless-resolve/index.spec.js b/packages/plugin-promises/lib/remove-useless-resolve/index.spec.js
diff --git a/packages/plugin-promises/lib/remove-useless-variables/index.js b/packages/plugin-promises/lib/remove-useless-variables/index.js
diff --git a/packages/plugin-promises/lib/remove-useless-variables/index.spec.js b/packages/plugin-promises/lib/remove-useless-variables/index.spec.js
diff --git a/packages/plugin-promises/package.json b/packages/plugin-promises/package.json
diff --git a/packages/plugin-promises/test/apply-with-resolvers.js b/packages/plugin-promises/test/apply-with-resolvers.js
diff --git a/packages/plugin-promises/test/promises.js b/packages/plugin-promises/test/promises.js
