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

Author: coderaiser

packages/plugin-nodejs/.putout.json

@@ -1,4 +1,8 @@
 {
+    "rules": {
+        "nodejs/convert-commonjs-to-esm": "on",
+        "nodejs/convert-esm-to-commonjs": "off"
+    },
     "match": {
         ".filesystem.json": {
             "eslint/convert-rc-to-flat": "on"

packages/plugin-nodejs/lib/add-node-prefix/index.js

@@ -1,7 +1,5 @@
-'use strict';
-
-const {isBuiltin} = require('node:module');
-const {types, operator} = require('putout');
+import {isBuiltin} from 'node:module';
+import {types, operator} from 'putout';
 
 const {
     setLiteralValue,
@@ -12,11 +10,11 @@ const {isCallExpression} = types;
 
 const REQUIRE = 'require("__a")';
 
-module.exports.report = ({value}) => {
+export const report = ({value}) => {
     return `Use 'node:${value}' instead of '${value}'`;
 };
 
-module.exports.fix = ({path, value}) => {
+export const fix = ({path, value}) => {
     if (isCallExpression(path)) {
         const arg = path.get('arguments.0');
         setLiteralValue(arg, `node:${value}`);
@@ -28,7 +26,7 @@ module.exports.fix = ({path, value}) => {
     setLiteralValue(source, `node:${value}`);
 };
 
-module.exports.traverse = ({push}) => ({
+export const traverse = ({push}) => ({
     [REQUIRE](path) {
         const {__a} = getTemplateValues(path, REQUIRE);
         const {value} = __a;

packages/plugin-nodejs/lib/add-node-prefix/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-node-prefix', plugin],
     ],

packages/plugin-nodejs/lib/cjs-file/index.js

@@ -1,10 +1,17 @@
-'use strict';
-
-const {operator} = require('putout');
-const plugin = require('../convert-esm-to-commonjs');
+import {operator} from 'putout';
+import * as plugin from '../convert-esm-to-commonjs/index.js';
 
 const {matchFiles} = operator;
-
-module.exports = matchFiles({
+const {
+    report,
+    fix,
+    scan,
+} = matchFiles({
     '*.cjs': plugin,
 });
+
+export {
+    report,
+    fix,
+    scan,
+};

packages/plugin-nodejs/lib/cjs-file/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: [
         ['cjs-file', plugin],
     ],

packages/plugin-nodejs/lib/convert-buffer-to-buffer-alloc/index.js

@@ -1,13 +1,12 @@
-'use strict';
+import {operator} from 'putout';
 
-const {operator} = require('putout');
 const {compute} = operator;
 
 const isNumber = (a) => typeof a === 'number';
 
-module.exports.report = () => `Use 'Buffer.alloc()' or 'Buffer.from()' instead of 'Buffer()' and 'new Buffer()'`;
+export const report = () => `Use 'Buffer.alloc()' or 'Buffer.from()' instead of 'Buffer()' and 'new Buffer()'`;
 
-module.exports.match = () => ({
+export const match = () => ({
     'new Buffer(__a)': (vars, path) => {
         const __aPath = path.get('arguments.0');
         const [is] = compute(__aPath);
@@ -16,7 +15,7 @@ module.exports.match = () => ({
     },
 });
 
-module.exports.replace = () => ({
+export const replace = () => ({
     'new Buffer(__a)': transform,
     'new Buffer(__a, __b)': transform,
     'Buffer(__a)': transform,

packages/plugin-nodejs/lib/convert-buffer-to-buffer-alloc/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: [
         ['nodejs/convert-buffer-to-buffer-alloc', convert],
     ],

packages/plugin-nodejs/lib/convert-commonjs-to-esm-commons/index.js

@@ -1,6 +1,4 @@
-'use strict';
-
-const {template, operator} = require('putout');
+import {template, operator} from 'putout';
 
 const {
     getPathAfterImports,
@@ -13,23 +11,23 @@ const initCommons = template.ast(`
     const require = createRequire(import.meta.url);
 `);
 
-module.exports.report = () => '"__filename", "__dirname" and "require" should be declared in ESM';
+export const report = () => '"__filename", "__dirname" and "require" should be declared in ESM';
 
-module.exports.fix = ({scope}) => {
+export const fix = ({scope}) => {
     const programScope = scope.getProgramParent();
     const body = programScope.path.get('body');
     const afterImportPath = getPathAfterImports(body);
 
     insertBefore(afterImportPath, initCommons);
 };
 
-module.exports.include = () => [
+export const include = () => [
     '__filename',
     '__dirname',
     'require',
 ];
 
-module.exports.filter = ({scope}) => {
+export const filter = ({scope}) => {
     const isDirname = scope.hasBinding('__dirname');
     const isFilename = scope.hasBinding('__filename');
     const isRequire = scope.hasBinding('require');

packages/plugin-nodejs/lib/convert-commonjs-to-esm-commons/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: [
         ['convert-commonjs-to-esm/commons', convert],
     ],

packages/plugin-nodejs/lib/convert-commonjs-to-esm-exports/index.js

@@ -1,6 +1,5 @@
-'use strict';
+import {types, operator} from 'putout';
 
-const {types, operator} = require('putout');
 const {
     exportSpecifier,
     isIdentifier,
@@ -10,11 +9,11 @@ const {
 
 const {replaceWith} = operator;
 
-module.exports.report = () => `Use 'ESM' instead of 'CommonJS'`;
+export const report = () => `Use 'ESM' instead of 'CommonJS'`;
 
-module.exports.exclude = () => ['__, __'];
+export const exclude = () => ['__, __'];
 
-module.exports.match = () => ({
+export const match = () => ({
     'module.exports.__a = __b': ({__a, __b}, path) => {
         const {name} = __a;
 
@@ -36,7 +35,7 @@ module.exports.match = () => ({
     },
 });
 
-module.exports.replace = () => ({
+export const replace = () => ({
     'module.exports = __a': 'export default __a',
     'module.exports.__a = __b': ({__a, __b}, path) => {
         const {name} = __a;

packages/plugin-nodejs/lib/convert-commonjs-to-esm-exports/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: [
         ['convert-commonjs-to-esm/exports', convert],
     ],

packages/plugin-nodejs/lib/convert-commonjs-to-esm-require/index.js

@@ -1,12 +1,10 @@
-'use strict';
-
-const {
+import {
     types,
     operator,
     template,
-} = require('putout');
+} from 'putout';
+import justCamelCase from 'just-camel-case';
 
-const justCamelCase = require('just-camel-case');
 const {
     awaitExpression,
     identifier,
@@ -23,7 +21,7 @@ const {replaceWith, insertBefore} = operator;
 
 const camelCase = (a) => justCamelCase(a.replace('@', ''));
 
-module.exports.report = () => `Use 'ESM' instead of 'CommonJS'`;
+export const report = () => `Use 'ESM' instead of 'CommonJS'`;
 
 const __B = 'declarations.0.init.arguments.0';
 
@@ -38,7 +36,7 @@ const createImport = ({name, source}) => {
 const createFnDeclaration = template('const NAME1 = FN(NAME2)');
 const isPackage = ({value}) => /package(\.json)?$/.test(value);
 
-module.exports.match = () => ({
+export const match = () => ({
     'const __a = require(__b)': ({__b}, path) => {
         // exclude jsons while not supported
         if (/\.json/.test(__b.value))
@@ -64,9 +62,9 @@ module.exports.match = () => ({
     'const __a = require("__b").__c(__args)': checkCall,
 });
 
-module.exports.replace = () => ({
+export const replace = () => ({
     'const __a = require(".")': 'import __a from "./index.js"',
-    'const __a = require(__b).default': 'import __a from "__b"',
+    'const __a = require("__b").default': 'import __a from "__b"',
     'const __a = require(__b).__c': `{
         const {__c} = require(__b);
         const __a = __c;

packages/plugin-nodejs/lib/convert-commonjs-to-esm-require/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: [
         ['convert-commonjs-to-esm/require', convert],
     ],

packages/plugin-nodejs/lib/convert-commonjs-to-esm.js

@@ -1,10 +1,8 @@
-'use strict';
+import * as convertCommonjsToEsmExports from './convert-commonjs-to-esm-exports/index.js';
+import * as convertCommonjsToEsmCommons from './convert-commonjs-to-esm-commons/index.js';
+import * as convertCommonjsToEsmRequire from './convert-commonjs-to-esm-require/index.js';
 
-const convertCommonjsToEsmExports = require('./convert-commonjs-to-esm-exports');
-const convertCommonjsToEsmCommons = require('./convert-commonjs-to-esm-commons');
-const convertCommonjsToEsmRequire = require('./convert-commonjs-to-esm-require');
-
-module.exports.rules = {
+export const rules = {
     'convert-commonjs-to-esm-exports': convertCommonjsToEsmExports,
     'convert-commonjs-to-esm-common': convertCommonjsToEsmCommons,
     'convert-commonjs-to-esm-require': convertCommonjsToEsmRequire,

packages/plugin-nodejs/lib/convert-dirname-to-url/index.js

@@ -1,13 +1,12 @@
-'use strict';
+import {operator} from 'putout';
 
-const {operator} = require('putout');
 const {isESM} = operator;
 
-module.exports.report = () => `Use 'import.meta.url' instead of '__dirname'`;
+export const report = () => `Use 'import.meta.url' instead of '__dirname'`;
 
-module.exports.filter = isESM;
+export const filter = isESM;
 
-module.exports.replace = () => ({
+export const replace = () => ({
     'join(__dirname, __a)': 'new URL(__a, import.meta.url).pathname',
     'path.join(__dirname, __a)': 'new URL(__a, import.meta.url).pathname',
 });

packages/plugin-nodejs/lib/convert-dirname-to-url/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: [
         ['nodejs/convert-dirname-to-url', convert],
     ],

packages/plugin-nodejs/lib/convert-esm-to-commonjs/index.js

@@ -1,6 +1,5 @@
-'use strict';
+import {types} from 'putout';
 
-const {types} = require('putout');
 const {isImportDefaultSpecifier} = types;
 
 const BODY = '{__body}';
@@ -9,9 +8,9 @@ const GEN_FN = `function* __a(__args) ${BODY}`;
 const ASYNC_FN = `async function __a(__args) ${BODY}`;
 const CLASS = `class __a ${BODY}`;
 
-module.exports.report = () => `Use 'CommonJS' instead of 'ESM'`;
+export const report = () => `Use 'CommonJS' instead of 'ESM'`;
 
-module.exports.replace = () => ({
+export const replace = () => ({
     'export default __a': 'module.exports = __a',
     [`export ${CLASS}`]: `module.exports.__a = ${CLASS}`,
     [`export ${FN}`]: `module.epxorts.__a = ${FN}`,

packages/plugin-nodejs/lib/convert-esm-to-commonjs/index.spec.js

@@ -1,12 +1,16 @@
-'use strict';
-
-const {createTest} = require('@putout/test');
-const putout = require('@putout/plugin-putout');
-const convert = require('.');
-
+import {fileURLToPath} from 'node:url';
+import {dirname} from 'node:path';
+import {createRequire} from 'node:module';
+import {createTest} from '@putout/test';
+import * as putout from '@putout/plugin-putout';
+import * as convert from './index.js';
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = dirname(__filename);
+const require = createRequire(import.meta.url);
 const nodejsDeclare = require('..').rules.declare;
 
-const test = createTest(__dirname, {
+const test = createTest(import.meta.url, {
     plugins: [
         ['nodejs/convert-esm-to-commonjs', convert],
     ],

packages/plugin-nodejs/lib/convert-exports-to-module-exports/index.js

@@ -1,7 +1,5 @@
-'use strict';
+export const report = () => `Use 'module.exports' instead of 'exports'`;
 
-module.exports.report = () => `Use 'module.exports' instead of 'exports'`;
-
-module.exports.replace = () => ({
+export const replace = () => ({
     'exports.__a': 'module.exports.__a',
 });

packages/plugin-nodejs/lib/convert-exports-to-module-exports/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: [
         ['convert-exports-to-module-exports', plugin],
     ],