@putout/plugin-nodejs 18.16.2 → 19.0.1

package/README.md

@@ -19,7 +19,6 @@ npm i putout @putout/plugin-nodejs -D
 
 - ✅ [add-node-prefix](#add-node-prefix);
 - ✅ [add-missing-strict-mode](#add-missing-strict-mode);
-- ✅ [apply-global-this](#apply-global-this);
 - ✅ [convert-buffer-to-buffer-alloc](#convert-buffer-to-buffer-alloc);
 - ✅ [convert-commonjs-to-esm](#convert-commonjs-to-esm);
 - ✅ [convert-dirname-to-url](#convert-dirname-to-url);
@@ -48,7 +47,6 @@ npm i putout @putout/plugin-nodejs -D
     "rules": {
         "nodejs/add-missing-strict-mode": "on",
         "nodejs/add-node-prefix": "on",
-        "nodejs/apply-global-this": "off",
         "nodejs/convert-commonjs-to-esm": "off",
         "nodejs/convert-esm-to-commonjs": "off",
         "nodejs/cjs-file": "off",
@@ -106,33 +104,6 @@ Linter | Rule | Fix
 🐊 **Putout** | [`add-node-prefix`](https://github.com/coderaiser/putout/tree/master/packages/plugin-nodejs/add-node-prefix#readme) | ✅
 ⏣ **ESLint** | [`prefer-node-protocol`](https://github.com/eslint-community/eslint-plugin-n/blob/master/docs/rules/prefer-node-protocol.md#readme)  | ✅
 
-## apply-global-this
-
-> Legacy. Use `globalThis` instead.
->
-> (c) [nodejs.org](https://nodejs.org/api/globals.html#global)
-
-Check out in 🐊[Putout Editor](https://putout.cloudcmd.io/#/gist/c72c8e1b1d0c2b45fd0d15d837dcae52/c7f33d77d19efe962339debbcf20f68bf2159aee).
-
-### ❌ Example of incorrect code
-
-```js
-global.__putout_debug = debugFn;
-```
-
-### ✅ Example of correct code
-
-```js
-globalThis.__putout_debug = debugFn;
-```
-
-### Comparison
-
-Linter | Rule | Fix
--------|------|------------|
-🐊 **Putout** | [`apply-global-this`](https://github.com/coderaiser/putout/tree/master/packages/plugin-nodejs/apply-node-prefix#readme) | ✅
-⏣ **ESLint**  | [`no-node-globals`](https://docs.deno.com/lint/rules/no-node-globals/)  | ❌
-
 ## convert-buffer-to-buffer-alloc
 
 > The `Buffer()` function and `new Buffer()` constructor are **deprecated** due to API usability issues that can lead to accidental security issues.
@@ -347,7 +318,7 @@ const {readFile} = require('fs/promises');
 const name = 'hello.txt';
 ```
 
-## convert-commonjs-to-esm
+## convert-commonjs-to-esm/require
 
 Convert **CommonJS** **EcmaScript Modules**.
 
@@ -355,8 +326,6 @@ Convert **CommonJS** **EcmaScript Modules**.
 >
 > (c) [parceljs](https://parceljs.org/languages/javascript/)
 
-### require
-
 ### ❌ Example of incorrect code
 
 ```js
@@ -378,7 +347,7 @@ const args = minimist({
 });
 ```
 
-### exports
+## convert-commonjs-to-esm/exports
 
 Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/39dd730c35e488db3e2cf8d4b4df0c5a/feb042a49539e6b4b2165558a1c636b705726c47).
 
@@ -394,7 +363,7 @@ module.exports = () => {};
 export default () => {};
 ```
 
-### Commons
+## convert-commonjs-to-esm/commons
 
 ### ❌ Example of incorrect code
 

package/lib/{convert-commonjs-to-esm-commons → convert-commonjs-to-esm/commons}/index.js

@@ -27,11 +27,13 @@ export const include = () => [
     'require',
 ];
 
-export const filter = (path) => {
-    if (path.parentPath.isObjectProperty())
+export const filter = ({parentPath, scope}) => {
+    if (parentPath.isMemberExpression())
+        return false;
+    
+    if (parentPath.isObjectProperty())
         return false;
     
-    const {scope} = path;
     const isDirname = scope.hasBinding('__dirname');
     const isFilename = scope.hasBinding('__filename');
     const isRequire = scope.hasBinding('require');

package/lib/convert-commonjs-to-esm/index.js

@@ -0,0 +1,9 @@
+import * as exports from './exports/index.js';
+import * as commons from './commons/index.js';
+import * as require from './require/index.js';
+
+export const rules = {
+    exports,
+    commons,
+    require,
+};

package/lib/index.js

@@ -1,4 +1,3 @@
-import * as applyGlobalThis from './apply-global-this/index.js';
 import * as convertBufferToBufferAlloc from './convert-buffer-to-buffer-alloc/index.js';
 import * as convertFsPromises from './convert-fs-promises/index.js';
 import * as convertPromisifyToFsPromises from './convert-promisify-to-fs-promises/index.js';
@@ -11,9 +10,7 @@ import * as removeProcessExit from './remove-process-exit/index.js';
 import * as addNodePrefix from './add-node-prefix/index.js';
 import * as convertExportsToModuleExports from './convert-exports-to-module-exports/index.js';
 import * as convertEsmToCommonjs from './convert-esm-to-commonjs/index.js';
-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';
+import * as convertCommonjsToEsm from './convert-commonjs-to-esm/index.js';
 import * as cjsFile from './cjs-file/index.js';
 import * as mjsFile from './mjs-file/index.js';
 import * as renameFileCjsToJs from './rename-file-cjs-to-js/index.js';
@@ -36,9 +33,9 @@ export const rules = {
     'convert-exports-to-module-exports': convertExportsToModuleExports,
     
     'convert-esm-to-commonjs': ['off', convertEsmToCommonjs],
-    'convert-commonjs-to-esm-exports': ['off', convertCommonjsToEsmExports],
-    'convert-commonjs-to-esm-common': ['off', convertCommonjsToEsmCommons],
-    'convert-commonjs-to-esm-require': ['off', convertCommonjsToEsmRequire],
+    'convert-commonjs-to-esm/exports': ['off', convertCommonjsToEsm.rules.exports],
+    'convert-commonjs-to-esm/common': ['off', convertCommonjsToEsm.rules.commons],
+    'convert-commonjs-to-esm/require': ['off', convertCommonjsToEsm.rules.require],
     
     'cjs-file': ['off', cjsFile],
     'mjs-file': ['off', mjsFile],
@@ -51,5 +48,4 @@ export const rules = {
     'remove-illegal-strict-mode': strictMode.rules['remove-illegal'],
     'remove-useless-promisify': removeUselessPromisify,
     'group-require-by-id': groupRequireById,
-    'apply-global-this': applyGlobalThis,
 };

package/lib/mjs-file/index.js

@@ -1,5 +1,5 @@
 import {operator} from 'putout';
-import * as plugin from '../convert-commonjs-to-esm.js';
+import * as plugin from '../convert-commonjs-to-esm/index.js';
 
 const {matchFiles} = operator;
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-nodejs",
-  "version": "18.16.2",
+  "version": "19.0.1",
   "type": "module",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin adds ability to transform code to new API of Node.js",
@@ -10,7 +10,7 @@
     ".": "./lib/index.js",
     "./strict-mode": "./lib/strict-mode/index.js",
     "./convert-esm-to-commonjs": "./lib/convert-esm-to-commonjs/index.js",
-    "./convert-commonjs-to-esm": "./lib/convert-commonjs-to-esm.js"
+    "./convert-commonjs-to-esm": "./lib/convert-commonjs-to-esm/index.js"
   },
   "release": false,
   "tag": false,

package/lib/apply-global-this/index.js

@@ -1,6 +0,0 @@
-export const report = () => `Use 'globalThis' instead of 'global'`;
-
-export const replace = () => ({
-    'global.__a': 'globalThis.__a',
-    'const __a = global': 'const __a = globalThis',
-});

package/lib/convert-commonjs-to-esm.js

@@ -1,9 +0,0 @@
-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';
-
-export const rules = {
-    'convert-commonjs-to-esm-exports': convertCommonjsToEsmExports,
-    'convert-commonjs-to-esm-common': convertCommonjsToEsmCommons,
-    'convert-commonjs-to-esm-require': convertCommonjsToEsmRequire,
-};