feature(@putout/plugin-eslint) add convert-node-to-n

Author: coderaiser

packages/plugin-eslint/.eslintrc.json

@@ -1,10 +1,10 @@
 {
     "extends": [
-        "plugin:node/recommended",
+        "plugin:n/recommended",
         "plugin:putout/recommended"
     ],
     "plugins": [
-        "node",
+        "n",
         "putout"
     ]
 }

packages/plugin-eslint/README.md

@@ -25,7 +25,7 @@ npm i @putout/plugin-eslint -D
         "eslint/move-putout-to-end-of-extends": "on",
         "eslint/convert-ide-to-safe": "on",
         "eslint/convert-require-to-import": "on",
-        "eslint/convert-import-to-require": "on",
+        "eslint/convert-node-to-n": "on",
         "eslint/remove-no-missing": "on",
         "eslint/remove-no-unpublished-require": "on",
         "eslint/remove-no-unsupported-features": "on",
@@ -271,6 +271,25 @@ And ofcourse remove only elements with empty `rules`:
 };
 ```
 
+## convert-node-to-n
+
+`eslint-plugin-node` [is no longer supported](https://github.com/mysticatea/eslint-plugin-node/issues/300). Better to use [`eslint-plugin-n`](https://github.com/weiran-zsd/eslint-plugin-node).
+
+```diff
+{
+    "extends": [
+        "plugin:putout/safe+align",
+-       "plugin:node/recommended"
++       "plugin:n/recommended"
+    ],
+    "plugins": [
+        "putout",
+-       "node"
++       "n"
+    ]
+}
+```
+
 ## remove-no-missing
 
 `node/remove-no-missing-require` and `node/remove-no-missing-import` doesn't supports [`exports`](https://nodejs.org/dist/latest-v18.x/docs/api/packages.html#exports)

packages/plugin-eslint/lib/convert-node-to-n/fixture/node-fix.js

@@ -0,0 +1,10 @@
+__putout_processor_json({
+    "extends": [
+        'plugin:n/recommended',
+        "plugin:putout/ide",
+    ],
+    "plugins": [
+        "putout",
+        'n'
+    ]
+});

packages/plugin-eslint/lib/convert-node-to-n/fixture/node.js

@@ -0,0 +1,10 @@
+__putout_processor_json({
+    "extends": [
+        "plugin:node/recommended",
+        "plugin:putout/ide",
+    ],
+    "plugins": [
+        "putout",
+        "node"
+    ]
+});

packages/plugin-eslint/lib/convert-node-to-n/fixture/plugins-fix.js

@@ -0,0 +1,6 @@
+__putout_processor_json({
+    "plugins": [
+        "putout",
+        'n'
+    ]
+});

packages/plugin-eslint/lib/convert-node-to-n/fixture/plugins.js

@@ -0,0 +1,6 @@
+__putout_processor_json({
+    "plugins": [
+        "putout",
+        "node"
+    ]
+});

packages/plugin-eslint/lib/convert-node-to-n/index.js

@@ -0,0 +1,48 @@
+'use strict';
+
+const {getExtends, getPlugins} = require('../get');
+
+module.exports.report = () => 'Use "n" instead of "node"';
+
+module.exports.match = () => ({
+    '__putout_processor_json(__a)': ({__a}) => {
+        const elements = getExtends(__a);
+        const plugins = getPlugins(__a);
+        
+        for (const {value} of elements) {
+            if (value.includes('node/recommended'))
+                return true;
+        }
+        
+        for (const {value} of plugins) {
+            if (value === 'node')
+                return true;
+        }
+        
+        return false;
+    },
+});
+
+module.exports.replace = () => ({
+    '__putout_processor_json(__a)': ({__a}, path) => {
+        const elements = getExtends(__a);
+        const plugins = getPlugins(__a);
+        
+        for (const element of elements) {
+            const {value} = element;
+            
+            if (value.includes('node/recommended'))
+                element.value = 'plugin:n/recommended';
+        }
+        
+        for (const element of plugins) {
+            const {value} = element;
+            
+            if (value === 'node')
+                element.value = 'n';
+        }
+        
+        return path;
+    },
+});
+

packages/plugin-eslint/lib/convert-node-to-n/index.spec.js

@@ -0,0 +1,24 @@
+'use strict';
+
+const {createTest} = require('@putout/test');
+const convertIdeToSafe = require('.');
+
+const test = createTest(__dirname, {
+    'eslint/convert-node-to-n': convertIdeToSafe,
+});
+
+test('putout: plugin-eslint: convert-node-to-n: report', (t) => {
+    t.report('node', 'Use "n" instead of "node"');
+    t.end();
+});
+
+test('putout: plugin-eslint: convert-node-to-n: transform', (t) => {
+    t.transform('node');
+    t.end();
+});
+
+test('putout: plugin-eslint: convert-node-to-n: transform: plugins', (t) => {
+    t.transform('plugins');
+    t.end();
+});
+

packages/plugin-eslint/lib/index.js

@@ -10,6 +10,7 @@ module.exports.rules = {
     ...getRule('move-putout-to-end-of-extends'),
     ...getRule('convert-ide-to-safe'),
     ...getRule('convert-require-to-import'),
+    ...getRule('convert-node-to-n'),
     ...getRule('remove-no-unpublished-require'),
     ...getRule('remove-no-unsupported-features'),
     ...getRule('remove-overrides-with-empty-rules'),

packages/plugin-eslint/test/eslint.js

@@ -57,3 +57,8 @@ test('putout: plugin-eslint: transform: add-putout', (t) => {
     t.end();
 });
 
+test('putout: plugin-eslint: transform: convert-node-to-n', (t) => {
+    t.transform('convert-node-to-n');
+    t.end();
+});
+