@putout/plugin-nodejs 4.8.1 → 6.0.0

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
@@ -192,6 +195,20 @@ 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/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'),

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

@@ -1,7 +1,6 @@
 'use strict';
 
 const {operator} = require('putout');
-
 const {
     remove,
     compareAny,
@@ -36,22 +35,19 @@ module.exports.fix = ({path, firstRequire, lastRequire}) => {
     lastRequire.insertAfter(node);
 };
 
-module.exports.traverse = ({push, listStore}) => ({
+module.exports.traverse = ({push, pathStore}) => ({
     'const __a = __b': (path) => {
         if (!path.parentPath.isProgram())
             return;
         
-        listStore(path);
+        pathStore(path);
     },
     'Program': {
         exit() {
             const requirePaths = [];
             const constPaths = [];
             
-            for (const path of listStore()) {
-                if (!path.node)
-                    continue;
-                
+            for (const path of pathStore()) {
                 if (path.node.__putoutNodeDeclareAfterRequire)
                     continue;
                 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-nodejs",
-  "version": "4.8.1",
+  "version": "6.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",
@@ -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",
+    "lerna": "^6.0.1",
     "madrun": "^9.0.0",
     "montag": "^1.2.1",
     "nodemon": "^2.0.1"
   },
   "peerDependencies": {
-    "putout": ">=25"
+    "putout": ">=29"
   },
   "license": "MIT",
   "engines": {