@putout/plugin-nodejs 18.0.1 → 18.2.0

package/README.md

@@ -19,6 +19,7 @@ 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);
@@ -45,13 +46,15 @@ npm i putout @putout/plugin-nodejs -D
 ```json
 {
     "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",
         "nodejs/mjs-file": "off",
         "nodejs/rename-file-cjs-to-js": "off",
         "nodejs/rename-file-mjs-to-js": "off",
-        "nodejs/add-node-prefix": "on",
         "nodejs/convert-buffer-to-buffer-alloc": "on",
         "nodejs/convert-fs-promises": "on",
         "nodejs/convert-promisify-to-fs-promises": "on",
@@ -63,7 +66,6 @@ npm i putout @putout/plugin-nodejs -D
         "nodejs/declare-after-require": "on",
         "nodejs/group-require-by-id": "on",
         "nodejs/remove-process-exit": "on",
-        "nodejs/add-missing-strict-mode": "on",
         "nodejs/remove-useless-strict-mode": "on",
         "nodejs/remove-illegal-strict-mode": "on",
         "nodejs/remove-useless-promisify": "on"
@@ -101,9 +103,36 @@ await import('node:path');
 
 Linter | Rule | Fix
 --------|-------|------------|
-🐊 **Putout** | [`apply-node-prefix`](https://github.com/coderaiser/putout/tree/master/packages/plugin-nodejs/apply-node-prefix#readme) | ✅
+🐊 **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.
@@ -254,6 +283,7 @@ process.exit();
 
 Add declarations to built-in node.js modules:
 
+- [buffer](https://nodejs.org/dist/latest-v24.x/docs/api/buffer.html);
 - [child_process](https://nodejs.org/dist/latest-v24.x/docs/api/child_process.html);
 - [events](https://nodejs.org/dist/latest-v24.x/docs/api/events.html);
 - [fs](https://nodejs.org/dist/latest-v24.x/docs/api/fs.html);

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

@@ -0,0 +1,6 @@
+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-exports/index.js

@@ -5,6 +5,7 @@ const {
     isIdentifier,
     isImportSpecifier,
     exportNamedDeclaration,
+    isObjectExpression,
 } = types;
 
 const {replaceWith} = operator;
@@ -39,7 +40,25 @@ export const match = () => ({
 });
 
 export const replace = () => ({
-    'module.exports = __a': 'export default __a',
+    'module.exports = __a': ({__a}) => {
+        if (!isObjectExpression(__a))
+            return 'export default __a';
+        
+        const result = ['export {'];
+        
+        for (const {key, value} of __a.properties) {
+            if (key.name === value.name) {
+                result.push(`${key.name},`);
+                continue;
+            }
+            
+            result.push(`${key.name} as ${value.name}`);
+        }
+        
+        result.push('};');
+        
+        return result.join('\n');
+    },
     'module.exports.__a = __b': ({__a, __b}, path) => {
         const {name} = __a;
         

package/lib/declare/index.js

@@ -1,3 +1,4 @@
+import buffer from './modules/buffer.js';
 import child_process from './modules/child_process.js';
 import util from './modules/util.js';
 import url from './modules/url.js';
@@ -13,6 +14,7 @@ import events from './modules/events.js';
 import timers from './modules/timers.js';
 
 export const declare = () => ({
+    ...buffer,
     ...events,
     ...fs,
     ...fsPromises,

package/lib/declare/modules/buffer.js

@@ -0,0 +1,9 @@
+export default {
+    Blob: `import {Blob} from 'node:buffer'`,
+    Buffer: `import {Buffer} from 'node:buffer'`,
+    File: `import {File} from 'node:buffer'`,
+    isAscii: `import {isAscii} from 'node:buffer'`,
+    isUtf8: `import {isUtf8} from 'node:buffer'`,
+    resolveObjectURL: `import {resolveObjectURL} from 'node:buffer'`,
+    transcode: `import {transcode} from 'node:buffer'`,
+};

package/lib/index.js

@@ -1,3 +1,4 @@
+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';
@@ -50,4 +51,5 @@ 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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-nodejs",
-  "version": "18.0.1",
+  "version": "18.2.0",
   "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",
@@ -44,7 +44,7 @@
     "@putout/plugin-declare-before-reference": "*",
     "@putout/plugin-esm": "*",
     "@putout/plugin-putout": "*",
-    "@putout/plugin-reuse-duplicate-init": "*",
+    "@putout/plugin-variables": "*",
     "@putout/plugin-typescript": "*",
     "@putout/test": "^14.0.0",
     "c8": "^10.0.0",