@putout/plugin-nodejs 20.2.0 → 20.3.0

package/README.md

@@ -33,7 +33,7 @@ npm i putout @putout/plugin-nodejs -D
 - ✅ [group-require-by-id](#group-require-by-id);
 - ✅ [remove-process-exit](#remove-process-exit);
 - ✅ [remove-useless-promisify](#remove-useless-promisify);
-- ✅ [remove-useless-process-exit](#remove-useless-process-exit);
+- ✅ [remove-top-level-process-exit](#remove-top-level-process-exit);
 - ✅ [remove-useless-strict-mode](#remove-useless-strict-mode);
 - ✅ [remove-illegal-strict-mode](#remove-useless-strict-mode);
 
@@ -71,7 +71,8 @@ npm i putout @putout/plugin-nodejs -D
         "nodejs/remove-process-exit": "on",
         "nodejs/remove-useless-strict-mode": "on",
         "nodejs/remove-illegal-strict-mode": "on",
-        "nodejs/remove-useless-promisify": "on"
+        "nodejs/remove-useless-promisify": "on",
+        "nodejs/remove-top-level-process-exit": "on"
     }
 }
 ```
@@ -224,10 +225,11 @@ In most cases `process.exit()` is called from `bin` directory, if not - disable
 -process.exit();
 ```
 
-## remove-useless-process-exit
+## remove-top-level-process-exit
 
-Top level `process.exit()` have no sense.
-Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/3aa8331269e49266d60914984027f9ed/6866eb78a0c1dd7f6e1f1e722c7c581b92f3b477).
+Top-level `process.exit()` has no sense (except debugging).
+
+Checkout in ✅[**Putout Editor**](https://putout.cloudcmd.io/#/gist/3aa8331269e49266d60914984027f9ed/9909ac2c07534299f64fee5ebf95cfc37ef04f11).
 
 ```diff
 -process.exit();

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

@@ -1,4 +1,13 @@
-import {template, operator} from 'putout';
+import {
+    template,
+    operator,
+    types,
+} from 'putout';
+
+const {
+    isProgram,
+    isCallExpression,
+} = types;
 
 const {
     getPathAfterImports,
@@ -24,14 +33,13 @@ export const fix = ({scope}) => {
 export const include = () => [
     '__filename',
     '__dirname',
-    'require',
+    'require(__b)',
 ];
 
-export const filter = ({parentPath, scope}) => {
-    if (parentPath.isMemberExpression())
-        return false;
+export const filter = (path) => {
+    const {scope} = path;
     
-    if (parentPath.isObjectProperty())
+    if (isCallExpression(path) && isProgram(scope.block))
         return false;
     
     const isDirname = scope.hasBinding('__dirname');

package/lib/index.js

@@ -1,4 +1,4 @@
-import * as removeUselessProcessExit from './remove-useless-process-exit/index.js';
+import * as removeTopLevelProcessExit from './remove-top-level-process-exit/index.js';
 import * as applyPrivatelyRequiredFiles from './apply-privately-required-file/index.js';
 import * as convertBufferToBufferAlloc from './convert-buffer-to-buffer-alloc/index.js';
 import * as convertFsPromises from './convert-fs-promises/index.js';
@@ -51,5 +51,5 @@ export const rules = {
     'remove-illegal-strict-mode': strictMode.rules['remove-illegal'],
     'remove-useless-promisify': removeUselessPromisify,
     'group-require-by-id': groupRequireById,
-    'remove-useless-process-exit': removeUselessProcessExit,
+    'remove-top-level-process-exit': removeTopLevelProcessExit,
 };

package/package.json

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