feature: @putout/plugin-nodejs: group-require-by-id: changed node before exit: add support

Author: coderaiser

packages/plugin-nodejs/lib/group-require-by-id/fixture/no-argument-fix.js

@@ -0,0 +1 @@
+const a = () => {};

packages/plugin-nodejs/lib/group-require-by-id/fixture/no-argument.js

@@ -0,0 +1 @@
+const a = require('b');

packages/plugin-nodejs/lib/group-require-by-id/index.js

@@ -1,13 +1,17 @@
 import {isDeepStrictEqual} from 'node:util';
-import {operator, types} from 'putout';
-
-const {isVariableDeclaration} = types;
+import {operator} from 'putout';
 
 const {
     replaceWithMultiple,
     remove,
+    compare,
 } = operator;
 
+const REQUIRE = 'const __ = require(__)';
+
+const compareWith = (a) => (b) => compare(b, a);
+const compareWithRequire = compareWith(REQUIRE);
+
 export const report = () => 'Group require by id';
 
 export const fix = ({grouped}) => {
@@ -27,18 +31,18 @@ export const fix = ({grouped}) => {
 };
 
 export const traverse = ({pathStore, push}) => ({
-    'const __ = require(__)': (path) => {
+    [REQUIRE]: (path) => {
         if (!path.parentPath.isProgram())
             return;
 
         pathStore(path);
     },
-    'Program': {
+    Program: {
         exit(path) {
             const external = [];
             const internal = [];
             const builtin = [];
-            const all = pathStore().filter(isVariableDeclaration);
+            const all = pathStore().filter(compareWithRequire);
 
             for (const current of all) {
                 const [declaration] = current.node.declarations;

packages/plugin-nodejs/lib/group-require-by-id/index.spec.js

@@ -54,6 +54,20 @@ test('nodejs: group-require-by-id: merge-declaration-with-export', (t) => {
     t.end();
 });
 
+test('nodejs: group-require-by-id: no-argument', (t) => {
+    const replacer = {
+        report: () => `Replace`,
+        replace: () => ({
+            'const __a = require(__b)': 'const __a = () => {}',
+        }),
+    };
+    
+    t.transform('no-argument', {
+        replacer,
+    });
+    t.end();
+});
+
 test('nodejs: group-require-by-id: merge', (t) => {
     const merge = {
         report: () => `Inline export`,