eslint-plugin-sonarjs
Version:
SonarJS rules for ESLint
106 lines (105 loc) • 4.91 kB
JavaScript
;
/*
* SonarQube JavaScript Plugin
* Copyright (C) 2011-2025 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 Sonar Source-Available License Version 1, as published by SonarSource SA.
*
* 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 Sonar Source-Available License for more details.
*
* You should have received a copy of the Sonar Source-Available License
* along with this program; if not, see https://sonarsource.com/license/ssal/
*/
// https://sonarsource.github.io/rspec/#/rspec/S1488
Object.defineProperty(exports, "__esModule", { value: true });
exports.rule = void 0;
const index_js_1 = require("../helpers/index.js");
const meta_js_1 = require("./meta.js");
exports.rule = {
meta: (0, index_js_1.generateMeta)(meta_js_1.meta, {
messages: {
doImmediateAction: 'Immediately {{action}} this expression instead of assigning it to the temporary variable "{{variable}}".',
},
fixable: 'code',
}),
create(context) {
return {
BlockStatement(node) {
processStatements(node, node.body);
},
SwitchCase(node) {
processStatements(node, node.consequent);
},
};
function processStatements(node, statements) {
if (statements.length > 1) {
const last = statements[statements.length - 1];
const returnedIdentifier = getOnlyReturnedVariable(last);
const lastButOne = statements[statements.length - 2];
const declaredIdentifier = getOnlyDeclaredVariable(lastButOne);
if (returnedIdentifier && declaredIdentifier) {
const sameVariable = getVariables(node, context).find(variable => {
return (variable.references.find(ref => ref.identifier === returnedIdentifier) !==
undefined &&
variable.references.find(ref => ref.identifier === declaredIdentifier.id) !==
undefined);
});
// there must be only one "read" - in `return` or `throw`
if (sameVariable && sameVariable.references.filter(ref => ref.isRead()).length === 1) {
context.report({
messageId: 'doImmediateAction',
data: {
action: last.type === 'ReturnStatement' ? 'return' : 'throw',
variable: returnedIdentifier.name,
},
node: declaredIdentifier.init,
fix: fixer => fix(fixer, last, lastButOne, declaredIdentifier.init, returnedIdentifier),
});
}
}
}
}
function fix(fixer, last, lastButOne, expressionToReturn, returnedExpression) {
const expressionText = context.sourceCode.getText(expressionToReturn);
const rangeToRemoveStart = lastButOne.range[0];
const commentsBetweenStatements = context.sourceCode.getCommentsAfter(lastButOne);
const rangeToRemoveEnd = commentsBetweenStatements.length > 0
? commentsBetweenStatements[0].range[0]
: last.range[0];
return [
fixer.removeRange([rangeToRemoveStart, rangeToRemoveEnd]),
fixer.replaceText(returnedExpression, expressionText),
];
}
function getOnlyReturnedVariable(node) {
return (node.type === 'ReturnStatement' || node.type === 'ThrowStatement') &&
node.argument &&
(0, index_js_1.isIdentifier)(node.argument)
? node.argument
: undefined;
}
function getOnlyDeclaredVariable(node) {
if (node.type === 'VariableDeclaration' && node.declarations.length === 1) {
const { id, init } = node.declarations[0];
if (id.type === 'Identifier' && init && !id.typeAnnotation) {
return { id, init };
}
}
return undefined;
}
function getVariables(node, context) {
const { variableScope, variables: currentScopeVariables } = context.sourceCode.getScope(node);
if (variableScope === context.sourceCode.getScope(node)) {
return currentScopeVariables;
}
else {
return currentScopeVariables.concat(variableScope.variables);
}
}
},
};