@putout/plugin-nodejs 13.1.0 → 14.0.0

package/README.md

@@ -303,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/)
 
-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).
+Check out in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/https://putout.cloudcmd.io/#/gist/ddf5731ae829beec4d3018d4d9ac2150/342738b63337bfa9b4fc08c5b301483ea2b5ba9c).For **ESM** use [`esm/declare-imports-first`](https://github.com/coderaiser/putout/tree/master/packages/plugin-esm#declare-imports-first).
 
 ### ❌ Example of incorrect code
 
@@ -350,37 +350,6 @@ 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
@@ -415,6 +384,37 @@ const __filename = fileURLToPath(import.meta.url);
 await readFile(__filename);
 ```
 
+## group-require-by-id
+
+Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/ff39c5d912d836a25b96772d8045dacb/fa8d8e1ebf8ac5f19a536247536f4bccf4fdac3d). For **ESM** use [`esm/group-imports-by-sources`](https://github.com/coderaiser/putout/tree/master/packages/plugin-esm#group-imports-by-source).
+
+### ❌ 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;
+```
+
 ## convert-esm-to-commonjs
 
 > **CommonJS** is a module system supported in Node, it provides a `require` function, which can be used to access the `exports` object exposed by another file.

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

@@ -73,9 +73,7 @@ module.exports.replace = () => ({
     }`,
     'require("__a")': 'import("__a")',
     'const __a = require(__b)': ({__a}, path) => {
-        let {value} = path
-            .get(__B)
-            .evaluate();
+        let {value} = path.get(__B).evaluate();
         
         if (value.includes('./') && !/\.m?js(on)?$/.test(value) && !value.endsWith('..'))
             value += '.js';

package/lib/declare-after-require/index.js

@@ -20,18 +20,12 @@ module.exports.report = ({path}) => {
     return `Declare '${id}' after last 'require()'`;
 };
 
-module.exports.fix = ({path, firstRequire, lastRequire}) => {
+module.exports.fix = ({path, lastRequire}) => {
     const {node} = path;
-    const {comments} = node;
     
     delete node.loc;
     node.__putoutNodeDeclareAfterRequire = true;
     
-    if (comments) {
-        firstRequire.node.comments = comments;
-        delete path.node.comments;
-    }
-    
     remove(path);
     insertAfter(lastRequire, node);
 };

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

@@ -15,7 +15,9 @@ module.exports.fix = ({grouped}) => {
     
     for (const current of others) {
         const {node} = current;
+        
         delete node.declarations[0].loc;
+        
         remove(current);
         nodes.push(node);
     }

package/lib/strict-mode/remove-useless/index.js

@@ -31,13 +31,6 @@ module.exports.traverse = ({push, store}) => ({
     },
     'Program': {
         exit(path) {
-            const [strictPath, ...paths] = path.get('body');
-            
-            for (const path of paths) {
-                if (path.isExpressionStatement() && path.node.expression.value === 'use strict')
-                    push(path);
-            }
-            
             const directives = path.get('directives');
             
             if (directives.length)
@@ -52,9 +45,6 @@ module.exports.traverse = ({push, store}) => ({
             directives
                 .filter(isStrictMode)
                 .forEach(push);
-            
-            if (strictPath.isExpressionStatement() && strictPath.node.expression.value === 'use strict')
-                push(strictPath);
         },
     },
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-nodejs",
-  "version": "13.1.0",
+  "version": "14.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",
@@ -49,14 +49,14 @@
     "c8": "^10.0.0",
     "eslint": "^9.0.0",
     "eslint-plugin-n": "^17.0.0",
-    "eslint-plugin-putout": "^23.0.0",
+    "eslint-plugin-putout": "^24.0.0",
     "lerna": "^6.0.1",
     "madrun": "^10.0.0",
     "montag": "^1.2.1",
     "nodemon": "^3.0.1"
   },
   "peerDependencies": {
-    "putout": ">=37"
+    "putout": ">=38"
   },
   "license": "MIT",
   "engines": {