From 05ed60e25f1de9d1bb83d56c81a349130960bec8 Mon Sep 17 00:00:00 2001 From: Yassin Kammoun <52890329+yassin-kammoun-sonarsource@users.noreply.github.com> Date: Wed, 19 Apr 2023 15:33:15 +0200 Subject: [PATCH] fix(eslint-plugin): [prefer-regexp-exec] skip malformed regexes (#6935) Skip malformed regular expressions --- packages/eslint-plugin/src/rules/prefer-regexp-exec.ts | 7 ++++++- .../eslint-plugin/tests/rules/prefer-regexp-exec.test.ts | 6 ++++++ 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/packages/eslint-plugin/src/rules/prefer-regexp-exec.ts b/packages/eslint-plugin/src/rules/prefer-regexp-exec.ts index 60bf310947f..5de7ab892a5 100644 --- 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 index 2ac3c7a07ba..46f192ab0ca 100644 --- 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: [ {