@putout/plugin-nodejs 19.0.0 → 19.1.0

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.

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

@@ -163,12 +163,13 @@ function addExportToBinding(name, path) {
         return exportNamedDeclaration(null, [exportSpecifier(imported, imported)]);
     }
     
-    if (binding.references > 1)
-        return '';
-    
     if (isImportDefaultSpecifier(bindingPath)) {
+        if (binding.references > 1)
+            return '';
+        
         const {local} = bindingPath.node;
         const {source} = bindingPath.parentPath.node;
+        
         const specifiers = [
             exportNamespaceSpecifier(local),
         ];

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';
@@ -49,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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-nodejs",
-  "version": "19.0.0",
+  "version": "19.1.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",

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',
-});