@putout/plugin-nodejs 11.0.0 → 11.1.0

package/README.md

@@ -37,7 +37,8 @@ npm i putout @putout/plugin-nodejs -D
         "nodejs/declare-after-require": "on",
         "nodejs/remove-process-exit": "on",
         "nodejs/add-missing-strict-mode": "on",
-        "nodejs/remove-useless-strict-mode": "on"
+        "nodejs/remove-useless-strict-mode": "on",
+        "nodejs/remove-useless-promisify": "on"
     }
 }
 ```
@@ -62,6 +63,13 @@ import fs from 'fs';
 import fs from 'node:fs';
 ```
 
+### Comparison
+
+Linter | Rule | Fix
+--------|-------|------------|
+🐊 **Putout** | [`apply-node-prefix`](https://github.com/coderaiser/putout/tree/master/packages/plugin-nodejs/apply-node-prefix#readme) | ✅
+⏣ **ESLint** | [`prefer-node-protocol`](https://github.com/eslint-community/eslint-plugin-n/blob/master/docs/rules/prefer-node-protocol.md#readme)  | ✅
+
 ## convert-buffer-to-buffer-alloc
 
 > The `Buffer()` function and `new Buffer()` constructor are **deprecated** due to API usability issues that can lead to accidental security issues.
@@ -445,6 +453,26 @@ import a from 'b';
 import a from 'b';
 ```
 
+## remove-useless-promisify
+
+> Takes a function following the common error-first callback style, i.e. taking an (err, value) => ... callback as the last argument, and returns a version that returns promises.
+>
+> (c) [nodejs.org](https://nodejs.org/dist/latest-v21.x/docs/api/util.html#utilpromisifyoriginal)
+
+Remove useless [`promisify()`](https://nodejs.org/dist/latest-v21.x/docs/api/util.html#utilpromisifyoriginal). Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/31000391865a36dfec2f8db5c2e98601/ce8867f83a84ecbe073637b9ceae58a443817187).
+
+### ❌ Example of incorrect code
+
+```js
+export const readSize = promisify(async (dir, options, callback) => {});
+```
+
+### ✅ Example of correct code
+
+```js
+export const readSize = async (dir, options, callback) => {};
+```
+
 ## License
 
 MIT

package/lib/index.js

@@ -25,6 +25,7 @@ const renameFileCjsToJs = require('./rename-file-cjs-to-js');
 const renameFileMjsToJs = require('./rename-file-mjs-to-js');
 
 const strictMode = require('./strict-mode');
+const removeUselessPromisify = require('./remove-useless-promisify');
 
 module.exports.rules = {
     'convert-buffer-to-buffer-alloc': convertBufferToBufferAlloc,
@@ -52,4 +53,5 @@ module.exports.rules = {
     
     'add-missing-strict-mode': strictMode.rules['add-missing'],
     'remove-useless-strict-mode': strictMode.rules['remove-useless'],
+    'remove-useless-promisify': removeUselessPromisify,
 };

package/lib/remove-useless-promisify/index.js

@@ -0,0 +1,7 @@
+'use strict';
+
+module.exports.report = () => `Calling 'promisify' on a function that returns a Promise is likely a mistake`;
+
+module.exports.replace = () => ({
+    'promisify(async (__args) => __body)': 'async (__args) => __body',
+});

package/package.json

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