eslint-plugin-sonarjs
Version:
SonarJS rules for ESLint
96 lines (95 loc) • 4 kB
JavaScript
;
/*
* SonarQube JavaScript Plugin
* Copyright (C) 2011-2024 SonarSource SA
* mailto:info AT sonarsource DOT com
*
* This program is free software; you can redistribute it and/or
* modify it under the terms of the GNU Lesser General Public
* License as published by the Free Software Foundation; either
* version 3 of the License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
* Lesser General Public License for more details.
*
* You should have received a copy of the GNU Lesser General Public License
* along with this program; if not, write to the Free Software Foundation,
* Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
*/
// https://sonarsource.github.io/rspec/#/rspec/S1110/javascript
Object.defineProperty(exports, "__esModule", { value: true });
exports.rule = void 0;
const helpers_1 = require("../helpers");
const meta_1 = require("./meta");
/**
* Parts of the grammar that are required to have parentheses.
*/
const parenthesized = {
DoWhileStatement: 'test',
IfStatement: 'test',
SwitchStatement: 'discriminant',
WhileStatement: 'test',
WithStatement: 'object',
ArrowFunctionExpression: 'body',
ImportExpression: 'source',
};
exports.rule = {
meta: (0, helpers_1.generateMeta)(meta_1.meta, { hasSuggestions: true }, true),
create(context) {
return {
'*'(node) {
checkRedundantParentheses(context.sourceCode, node, context);
},
};
},
};
function checkRedundantParentheses(sourceCode, node, context) {
const parenthesesPairsAroundNode = getParenthesesPairsAround(sourceCode, node, node);
const parent = (0, helpers_1.getParent)(context, node);
// Ignore parentheses pair from the parent node
if (!!parent && isInParentNodeParentheses(node, parent)) {
parenthesesPairsAroundNode.pop();
}
// One pair of parentheses is allowed for readability purposes
parenthesesPairsAroundNode.shift();
parenthesesPairsAroundNode.forEach(parentheses => {
(0, helpers_1.report)(context, {
message: `Remove these redundant parentheses.`,
loc: parentheses.openingParenthesis.loc,
suggest: [
{
desc: 'Remove these redundant parentheses',
fix(fixer) {
return [
fixer.remove(parentheses.openingParenthesis),
fixer.remove(parentheses.closingParenthesis),
];
},
},
],
}, [(0, helpers_1.toSecondaryLocation)(parentheses.closingParenthesis)]);
});
}
function getParenthesesPairsAround(sourceCode, start, end) {
const tokenBefore = sourceCode.getTokenBefore(start);
const tokenAfter = sourceCode.getTokenAfter(end);
if (!!tokenBefore && !!tokenAfter && tokenBefore.value === '(' && tokenAfter.value === ')') {
return [
{ openingParenthesis: tokenBefore, closingParenthesis: tokenAfter },
...getParenthesesPairsAround(sourceCode, tokenBefore, tokenAfter),
];
}
return [];
}
function isInParentNodeParentheses(node, parent) {
// Applying same logic as https://github.com/eslint/eslint/blob/main/lib/rules/no-sequences.js#L81
// both rules (S1110 and S878) can contradict each other, so better use the same logic
const parentAttribute = parenthesized[parent.type];
const nodeIsInConditionOfParent = parentAttribute &&
node === parent[parentAttribute];
const nodeIsArgumentOfCallExpression = (parent.type === 'CallExpression' || parent.type === 'NewExpression') &&
parent.arguments.includes(node);
return nodeIsInConditionOfParent || nodeIsArgumentOfCallExpression;
}