@putout/plugin-nodejs 12.0.0 → 13.0.0

package/README.md

@@ -33,12 +33,13 @@ npm i putout @putout/plugin-nodejs -D
 - ✅ [convert-url-to-dirname](#convert-url-to-dirname);
 - ✅ [declare](#declare);
 - ✅ [declare-after-require](#declare-after-require);
+- ✅ [group-require-by-id](#group-require-by-id);
 - ✅ [remove-process-exit](#remove-process-exit);
 - ✅ [remove-useless-promisify](#remove-useless-promisify);
 - ✅ [rename-file-cjs-to-js](#rename-file-cjs-to-js);
 - ✅ [rename-file-mjs-to-js](#rename-file-mjs-to-js);
 - ✅ [remove-useless-strict-mode](#remove-useless-strict-mode);
-- ✅ [remove-illigal-strict-mode](#remove-useless-strict-mode);
+- ✅ [remove-illegal-strict-mode](#remove-useless-strict-mode);
 - ✅ [cjs-file](#cjs-file);
 - ✅ [mjs-file](#mjs-file);
 
@@ -63,9 +64,11 @@ npm i putout @putout/plugin-nodejs -D
         "nodejs/convert-top-level-return": "on",
         "nodejs/declare": "on",
         "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"
     }
 }
@@ -300,7 +303,7 @@ When you want to skip some declaration use `dismiss`:
 >
 > (c) [Nodejs.org](https://nodejs.org/en/knowledge/getting-started/what-is-require/)
 
-Check out in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/https://putout.cloudcmd.io/#/gist/ddf5731ae829beec4d3018d4d9ac2150/342738b63337bfa9b4fc08c5b301483ea2b5ba9c).
+For ESM use [declare-imports-first](https://github.com/coderaiser/putout/tree/master/packages/plugin-declare-imports-first#readme). Check out in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/https://putout.cloudcmd.io/#/gist/ddf5731ae829beec4d3018d4d9ac2150/342738b63337bfa9b4fc08c5b301483ea2b5ba9c).
 
 ### ❌ Example of incorrect code
 
@@ -347,6 +350,37 @@ const args = minimist({
 });
 ```
 
+## group-require-by-id
+
+Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/ff39c5d912d836a25b96772d8045dacb/fa8d8e1ebf8ac5f19a536247536f4bccf4fdac3d).
+
+### ❌ Example of incorrect code
+
+```js
+const ss = require('../../bb/ss');
+const d = require('../hello');
+const react = require('react');
+const {lodash} = require('lodash');
+const fs = require('node:fs');
+const b = require('./ss');
+const m = require(x);
+const c = 5;
+```
+
+### ✅ Example of correct code
+
+```js
+const fs = require('node:fs');
+const react = require('react');
+const {lodash} = require('lodash');
+const ss = require('../../bb/ss');
+const d = require('../hello');
+
+const b = require('./ss');
+const m = require(x);
+const c = 5;
+```
+
 ### exports
 
 ### ❌ Example of incorrect code
@@ -487,7 +521,7 @@ import a from 'b';
 import a from 'b';
 ```
 
-## remove-illigal-strict-mode
+## remove-illegal-strict-mode
 
 > `SyntaxError: "use strict" not allowed in function with non-simple parameters`
 > The JavaScript exception `"use strict" not allowed in function` occurs when a `use strict` directive is used at the top of a function with default parameters, rest parameters, or destructuring parameters.

package/lib/add-node-prefix/index.js

@@ -1,7 +1,8 @@
 'use strict';
 
-const {types, operator} = require('putout');
 const {isBuiltin} = require('node:module');
+const {types, operator} = require('putout');
+
 const {
     setLiteralValue,
     getTemplateValues,

package/lib/group-require-by-id/index.js

@@ -0,0 +1,71 @@
+'use strict';
+
+const {isDeepStrictEqual} = require('node:util');
+const {operator} = require('putout');
+const {
+    replaceWithMultiple,
+    remove,
+} = operator;
+
+module.exports.report = () => 'Group require by id';
+
+module.exports.fix = ({grouped}) => {
+    const [first, ...others] = grouped;
+    const nodes = [first.node];
+    
+    for (const current of others) {
+        const {node} = current;
+        remove(current);
+        nodes.push(node);
+    }
+    
+    replaceWithMultiple(first, nodes);
+};
+
+module.exports.traverse = ({pathStore, push}) => ({
+    'const __ = require(__)': (path) => {
+        if (!path.parentPath.isProgram())
+            return;
+        
+        pathStore(path);
+    },
+    'Program': {
+        exit(path) {
+            const external = [];
+            const internal = [];
+            const builtin = [];
+            const all = pathStore();
+            
+            for (const current of all) {
+                const [declaration] = current.node.declarations;
+                const {value} = declaration.init.arguments[0];
+                
+                if (!value || value.startsWith('.')) {
+                    internal.push(current);
+                    continue;
+                }
+                
+                if (value.startsWith('node:')) {
+                    builtin.push(current);
+                    continue;
+                }
+                
+                external.push(current);
+            }
+            
+            const grouped = [
+                ...builtin,
+                ...external,
+                ...internal,
+            ];
+            
+            if (isDeepStrictEqual(all, grouped))
+                return;
+            
+            push({
+                path,
+                grouped,
+            });
+        },
+    },
+});

package/lib/index.js

@@ -26,6 +26,7 @@ const renameFileMjsToJs = require('./rename-file-mjs-to-js');
 
 const strictMode = require('./strict-mode');
 const removeUselessPromisify = require('./remove-useless-promisify');
+const groupRequireById = require('./group-require-by-id');
 
 module.exports.rules = {
     'convert-buffer-to-buffer-alloc': convertBufferToBufferAlloc,
@@ -53,6 +54,7 @@ module.exports.rules = {
     
     'add-missing-strict-mode': strictMode.rules['add-missing'],
     'remove-useless-strict-mode': strictMode.rules['remove-useless'],
-    'remove-illigal-strict-mode': strictMode.rules['remove-illigal'],
+    'remove-illegal-strict-mode': strictMode.rules['remove-illegal'],
     'remove-useless-promisify': removeUselessPromisify,
+    'group-require-by-id': groupRequireById,
 };

package/lib/strict-mode/index.js

@@ -2,10 +2,10 @@
 
 const addMissing = require('./add-missing');
 const removeUseless = require('./remove-useless');
-const removeIlligal = require('./remove-illigal');
+const removeIllegal = require('./remove-illegal');
 
 module.exports.rules = {
     'add-missing': addMissing,
     'remove-useless': removeUseless,
-    'remove-illigal': removeIlligal,
+    'remove-illegal': removeIllegal,
 };

package/lib/strict-mode/{remove-illigal → remove-illegal}/index.js

@@ -11,7 +11,7 @@ const {
 
 const {remove} = operator;
 
-module.exports.report = () => `Avoid illigal 'use strict'`;
+module.exports.report = () => `Avoid illegal 'use strict'`;
 
 module.exports.fix = (path) => {
     remove(path);
@@ -23,12 +23,12 @@ module.exports.traverse = ({push}) => ({
         if (!isFunction(fnPath))
             return;
         
-        if (isIlligal(fnPath))
+        if (isIllegal(fnPath))
             push(path.parentPath);
     },
 });
 
-function isIlligal(fnPath) {
+function isIllegal(fnPath) {
     const params = fnPath.get('params');
     
     for (const param of params) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-nodejs",
-  "version": "12.0.0",
+  "version": "13.0.0",
   "type": "commonjs",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin adds ability to transform code to new API of Node.js",
@@ -54,7 +54,7 @@
     "nodemon": "^3.0.1"
   },
   "peerDependencies": {
-    "putout": ">=36"
+    "putout": ">=37"
   },
   "license": "MIT",
   "engines": {