fix(eslint-plugin): [prefer-regexp-exec] skip malformed regexes (#6935)

Skip malformed regular expressions
typescript-eslint · Apr 19, 2023 · 05ed60e · 05ed60e
1 parent 4f3750f
commit 05ed60e
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/packages/eslint-plugin/src/rules/prefer-regexp-exec.ts b/packages/eslint-plugin/src/rules/prefer-regexp-exec.ts
@@ -125,7 +125,12 @@ export default createRule({
           argumentNode.type === AST_NODE_TYPES.Literal &&
           typeof argumentNode.value === 'string'
         ) {
-          const regExp = RegExp(argumentNode.value);
+          let regExp: RegExp;
+          try {
+            regExp = RegExp(argumentNode.value);
+          } catch {
+            return;
+          }
           return context.report({
             node: memberNode.property,
             messageId: 'regExpExecOverStringMatch',

diff --git a/packages/eslint-plugin/tests/rules/prefer-regexp-exec.test.ts b/packages/eslint-plugin/tests/rules/prefer-regexp-exec.test.ts
@@ -74,6 +74,12 @@ const matchCount = (str: string, re: RegExp) => {
   return (str.match(re) || []).length;
 };
     `,
+    // https://github.com/typescript-eslint/typescript-eslint/issues/6928
+    `
+function test(str: string) {
+  str.match('[a-z');
+}
+    `,
   ],
   invalid: [
     {