feature: @putout/plugin-regexp: remove-useless-regexp: transformRegExp

Author: coderaiser

packages/plugin-regexp/lib/remove-useless-regexp/index.js

@@ -1,14 +1,21 @@
-import regexpTree from 'regexp-tree';
 import {
     template,
     types,
     operator,
 } from 'putout';
-import {isAlternative, isChar} from '../types.js';
+import * as regexp from './regexp.js';
 
-const {replaceWith} = operator;
+const {
+    replaceWith,
+    transformRegExp,
+} = operator;
 const {stringLiteral} = types;
 
+const cut = (a) => a.slice(START, END);
+const wrap = (a) => `/${a}/`;
+const START = 1;
+const END = -1;
+
 export const report = () => `Remove useless RegExp, use strict equal operator instead`;
 
 const build = template(`A === B`);
@@ -33,42 +40,15 @@ export const traverse = ({push}) => ({
 
         const [arg] = path.node.arguments;
         const to = cut(str);
-        const is = isOnlyChars(wrap(to));
+        const [, places] = transformRegExp(wrap(to), regexp);
+        
+        if (!places.length)
+            return;
 
-        if (is)
-            push({
-                path,
-                arg,
-                to,
-            });
+        push({
+            path,
+            arg,
+            to,
+        });
     },
 });
-
-const not = (f) => (...a) => !f(...a);
-const START = 1;
-const END = -1;
-const cut = (a) => a.slice(START, END);
-const wrap = (a) => `/${a}/`;
-
-function isOnlyChars(str) {
-    const ast = regexpTree.parse(str);
-    let is = false;
-    
-    regexpTree.traverse(ast, {
-        RegExp({node}) {
-            const {body} = node;
-            
-            if (isChar(body) && !body.value.includes('\\')) {
-                is = true;
-                return;
-            }
-            
-            if (isAlternative(body)) {
-                is = !body.expressions.filter(not(isChar)).length;
-                return;
-            }
-        },
-    });
-    
-    return is;
-}

packages/plugin-regexp/lib/remove-useless-regexp/regexp.js

@@ -0,0 +1,24 @@
+import {isAlternative, isChar} from '../types.js';
+
+const not = (f) => (...a) => !f(...a);
+
+export const report = () => `Remove useless RegExp, use strict equal operator instead`;
+export const fix = () => {};
+export const traverse = ({push}) => ({
+    RegExp(path) {
+        const {node} = path;
+        const {body} = node;
+        
+        if (isChar(body) && !body.value.includes('\\')) {
+            push(path);
+            return;
+        }
+        
+        if (isAlternative(body)) {
+            if (!body.expressions.filter(not(isChar)).length)
+                push(path);
+            
+            return;
+        }
+    },
+});