Improve performance of parentheses related check (#2794)

Author: fisker

rules/explicit-length-check.js

@@ -1,7 +1,11 @@
-import {isParenthesized, getStaticValue} from '@eslint-community/eslint-utils';
-import {checkVueTemplate} from './utils/rule.js';
-import isLogicalExpression from './utils/is-logical-expression.js';
-import {isBooleanNode, getBooleanAncestor} from './utils/boolean.js';
+import {getStaticValue} from '@eslint-community/eslint-utils';
+import {
+	isParenthesized,
+	checkVueTemplate,
+	isLogicalExpression,
+	isBooleanNode,
+	getBooleanAncestor,
+} from './utils/index.js';
 import {fixSpaceAroundKeyword} from './fix/index.js';
 import {isLiteral, isMemberExpression, isNumericLiteral} from './ast/index.js';
 
@@ -114,7 +118,7 @@ function create(context) {
 
 		let fixed = `${sourceCode.getText(lengthNode)} ${code}`;
 		if (
-			!isParenthesized(node, sourceCode)
+			!isParenthesized(node, context)
 			&& node.type === 'UnaryExpression'
 			&& (node.parent.type === 'UnaryExpression' || node.parent.type === 'AwaitExpression')
 		) {

rules/fix/fix-space-around-keywords.js

@@ -1,4 +1,4 @@
-import {getParenthesizedRange} from '../utils/parentheses.js';
+import {getParenthesizedRange} from '../utils/index.js';
 
 /**
 @import {TSESTree as ESTree} from '@typescript-eslint/types';

rules/fix/remove-argument.js

@@ -1,5 +1,5 @@
 import {isCommaToken} from '@eslint-community/eslint-utils';
-import {getParentheses} from '../utils/parentheses.js';
+import {getParentheses} from '../utils/index.js';
 
 /**
 @import {TSESTree as ESTree} from '@typescript-eslint/types';

rules/fix/remove-method-call.js

@@ -1,4 +1,4 @@
-import {getParenthesizedRange} from '../utils/parentheses.js';
+import {getParenthesizedRange} from '../utils/index.js';
 import {removeMemberExpressionProperty} from './replace-member-expression-property.js';
 
 /**

rules/fix/remove-parentheses.js

@@ -1,4 +1,4 @@
-import {getParentheses} from '../utils/parentheses.js';
+import {getParentheses} from '../utils/index.js';
 
 /**
 @import {TSESTree as ESTree} from '@typescript-eslint/types';

rules/fix/replace-argument.js

@@ -1,4 +1,4 @@
-import {getParenthesizedRange} from '../utils/parentheses.js';
+import {getParenthesizedRange} from '../utils/index.js';
 
 /**
 @import {TSESTree as ESTree} from '@typescript-eslint/types';

rules/fix/replace-member-expression-property.js

@@ -1,4 +1,4 @@
-import {getParenthesizedRange} from '../utils/parentheses.js';
+import {getParenthesizedRange} from '../utils/index.js';
 
 /**
 @import {TSESTree as ESTree} from '@typescript-eslint/types';

rules/fix/replace-node-or-token-and-spaces-before.js

@@ -1,4 +1,4 @@
-import {getParentheses} from '../utils/parentheses.js';
+import {getParentheses} from '../utils/index.js';
 /**
 @import {TSESTree as ESTree} from '@typescript-eslint/types';
 @import * as ESLint from 'eslint';

rules/fix/switch-call-expression-to-new-expression.js

@@ -1,5 +1,7 @@
-import {isParenthesized} from '../utils/parentheses.js';
-import shouldAddParenthesesToNewExpressionCallee from '../utils/should-add-parentheses-to-new-expression-callee.js';
+import {
+	isParenthesized,
+	shouldAddParenthesesToNewExpressionCallee,
+} from '../utils/index.js';
 import fixSpaceAroundKeyword from './fix-space-around-keywords.js';
 
 /**
@@ -20,7 +22,7 @@ export default function * switchCallExpressionToNewExpression(node, context, fix
 
 	const {callee} = node;
 	if (
-		!isParenthesized(callee, context.sourceCode)
+		!isParenthesized(callee, context)
 		&& shouldAddParenthesesToNewExpressionCallee(callee)
 	) {
 		yield fixer.insertTextBefore(callee, '(');

rules/fix/switch-new-expression-to-call-expression.js

@@ -1,6 +1,8 @@
-import isNewExpressionWithParentheses from '../utils/is-new-expression-with-parentheses.js';
-import {isParenthesized} from '../utils/parentheses.js';
-import isOnSameLine from '../utils/is-on-same-line.js';
+import {
+	isNewExpressionWithParentheses,
+	isParenthesized,
+	isOnSameLine,
+} from '../utils/index.js';
 import addParenthesizesToReturnOrThrowExpression from './add-parenthesizes-to-return-or-throw-expression.js';
 import removeSpaceAfter from './remove-spaces-after.js';
 
@@ -34,7 +36,7 @@ export default function * switchNewExpressionToCallExpression(newExpression, con
 			}
 		```
 	*/
-	if (!isOnSameLine(newToken, newExpression.callee, context) && !isParenthesized(newExpression, context.sourceCode)) {
+	if (!isOnSameLine(newToken, newExpression.callee, context) && !isParenthesized(newExpression, context)) {
 		// Ideally, we should use first parenthesis of the `callee`, and should check spaces after the `new` token
 		// But adding extra parentheses is harmless, no need to be too complicated
 		yield addParenthesizesToReturnOrThrowExpression(fixer, newExpression.parent, context);