@putout/plugin-nodejs 5.0.0 → 6.0.1

package/README.md

@@ -26,6 +26,7 @@ npm i putout @putout/plugin-nodejs -D
         "nodejs/convert-dirname-to-url": "on",
         "nodejs/convert-url-to-dirname": "on",
         "nodejs/convert-top-level-return": "on",
+        "nodejs/declare": "on",
         "nodejs/declare-after-require": "on",
         "nodejs/remove-process-exit": "on"
     }
@@ -179,6 +180,8 @@ Add declarations to built-in node.js modules:
 - [util](https://nodejs.org/dist/latest-v18.x/docs/api/util.html);
 - [zlib](https://nodejs.org/dist/latest-v18.x/docs/api/zlib.html);
 
+Based on [`@putout/operator-declare`](https://github.com/coderaiser/putout/tree/master/packages/operator-declare#putoutoperator-declare-).
+
 #### ❌ Example of incorrect code
 
 ```js
@@ -189,9 +192,22 @@ await readFile('hello.txt', 'utf8');
 
 ```js
 import {readFile} from 'fs/promises';
+
 await readFile('hello.txt', 'utf8');
 ```
 
+When you want to skip some declaration use `dismiss`:
+
+```json
+{
+    "rules": {
+        "nodejs/declare": ["on", {
+            "dismiss": ["readFile"]
+        }]
+    }
+}
+```
+
 ### declare-after-require
 
 > **Node.js** follows the **CommonJS** module system, and the builtin `require` function is the easiest way to include modules that exist in separate files. The basic functionality of `require` is that it reads a JavaScript file, executes the file, and then proceeds to return the `exports` object.

package/lib/convert-buffer-to-buffer-alloc/index.js

@@ -30,4 +30,3 @@ function transform(vars, path) {
     
     return 'Buffer.from(__a)';
 }
-

package/lib/convert-fs-promises/index.js

@@ -5,4 +5,3 @@ module.exports.report = () => '"fs/promises" should be used instead of "fs.promi
 module.exports.replace = () => ({
     'const __a = require("fs").promises': 'const __a = require("fs/promises")',
 });
-

package/lib/convert-promisify-to-fs-promises/index.js

@@ -22,11 +22,17 @@ module.exports.fix = ({path, promisified}) => {
         const [declarator] = path.node.declarations;
         const {name} = declarator.id;
         
-        props.push(t.ObjectProperty(t.Identifier(name), t.Identifier(name), NOT_COMPUTED, SHORTHAND));
+        props.push(t.ObjectProperty(
+            t.Identifier(name),
+            t.Identifier(name),
+            NOT_COMPUTED,
+            SHORTHAND,
+        ));
         remove(path);
     }
     
     const {init} = path.node;
+    
     init.arguments[0].value = 'fs/promises';
     
     replaceWith(path.get('id'), t.ObjectPattern(props));
@@ -58,4 +64,3 @@ module.exports.find = (ast, {push, traverse}) => {
         promisified,
     });
 };
-

package/lib/convert-top-level-return/index.js

@@ -12,4 +12,3 @@ module.exports.replace = () => ({
     'return __a': 'process.exit()',
     'return': 'process.exit()',
 });
-

package/lib/declare/index.js

@@ -1,9 +1,6 @@
 'use strict';
 
-const {operator} = require('putout');
-const {declare} = operator;
-
-module.exports = declare({
+module.exports.declare = () => ({
     ...require('./modules/events'),
     ...require('./modules/fs'),
     ...require('./modules/fs-promises'),
@@ -17,4 +14,3 @@ module.exports = declare({
     ...require('./modules/util'),
     ...require('./modules/child_process'),
 });
-

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

@@ -1,7 +1,6 @@
 'use strict';
 
 const {operator} = require('putout');
-
 const {
     remove,
     compareAny,
@@ -22,7 +21,7 @@ module.exports.report = ({path}) => {
 
 module.exports.fix = ({path, firstRequire, lastRequire}) => {
     const {node} = path;
-    const {comments} = path.node;
+    const {comments} = node;
     
     delete node.loc;
     node.__putoutNodeDeclareAfterRequire = true;
@@ -124,4 +123,3 @@ function getReferenceLine(path) {
     
     return firstReference.node.loc.start.line;
 }
-

package/lib/index.js

@@ -15,4 +15,3 @@ module.exports.rules = {
     ...getRule('declare-after-require'),
     ...getRule('remove-process-exit'),
 };
-

package/lib/remove-process-exit/index.js

@@ -6,4 +6,3 @@ module.exports.replace = () => ({
     'process.exit()': '',
     'process["exit"]()': '',
 });
-

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-nodejs",
-  "version": "5.0.0",
+  "version": "6.0.1",
   "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",
@@ -34,18 +34,18 @@
     "@putout/plugin-convert-commonjs-to-esm": "*",
     "@putout/plugin-convert-esm-to-commonjs": "*",
     "@putout/plugin-putout": "*",
-    "@putout/test": "^5.0.0",
+    "@putout/test": "^6.0.0",
     "c8": "^7.5.0",
     "eslint": "^8.0.1",
-    "eslint-plugin-n": "^15.2.4",
-    "eslint-plugin-putout": "^16.0.0",
-    "lerna": "^5.0.0",
+    "eslint-plugin-n": "^16.0.0",
+    "eslint-plugin-putout": "^17.0.0",
+    "lerna": "^6.0.1",
     "madrun": "^9.0.0",
     "montag": "^1.2.1",
     "nodemon": "^2.0.1"
   },
   "peerDependencies": {
-    "putout": ">=27"
+    "putout": ">=29"
   },
   "license": "MIT",
   "engines": {