eslint-plugin-sonarjs
Version:
SonarJS rules for ESLint
80 lines (79 loc) • 3.47 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/S2392/javascript
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, undefined, true),
create(context) {
let scopeRanges = [];
let reported = [];
function enterScope(node) {
scopeRanges.push(node.range);
}
function exitScope() {
scopeRanges.pop();
}
return {
Program(node) {
scopeRanges = [node.range];
reported = [];
},
BlockStatement: enterScope,
'BlockStatement:exit': exitScope,
ForStatement: enterScope,
'ForStatement:exit': exitScope,
ForInStatement: enterScope,
'ForInStatement:exit': exitScope,
ForOfStatement: enterScope,
'ForOfStatement:exit': exitScope,
SwitchStatement: enterScope,
'SwitchStatement:exit': exitScope,
VariableDeclaration: (node) => {
const varDeclaration = node;
if (varDeclaration.kind !== 'var') {
return;
}
const scopeRange = scopeRanges[scopeRanges.length - 1];
function isOutsideOfScope(reference) {
const idRange = reference.range;
return idRange[0] < scopeRange[0] || idRange[1] > scopeRange[1];
}
context.sourceCode.getDeclaredVariables(node).forEach(variable => {
const referencesOutside = variable.references
.map(ref => ref.identifier)
.filter(isOutsideOfScope);
if (referencesOutside.length === 0) {
return;
}
const definition = variable.defs.find(def => varDeclaration.declarations.includes(def.node));
if (definition && !reported.includes(definition.name)) {
(0, index_js_1.report)(context, {
node: definition.name,
message: `Consider moving declaration of '${variable.name}' ` +
`as it is referenced outside current binding context.`,
}, referencesOutside.map(node => (0, index_js_1.toSecondaryLocation)(node, 'Outside reference.')));
variable.defs.map(def => def.name).forEach(defId => reported.push(defId));
}
});
},
};
},
};