eslint-plugin-sonarjs 4.0.1 → 4.0.2

package/cjs/S1994/rule.js

@@ -51,7 +51,10 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.rule = void 0;
-const index_js_1 = require("../helpers/index.js");
+const equivalence_js_1 = require("../helpers/equivalence.js");
+const generate_meta_js_1 = require("../helpers/generate-meta.js");
+const ancestor_js_1 = require("../helpers/ancestor.js");
+const collection_js_1 = require("../helpers/collection.js");
 const meta = __importStar(require("./generated-meta.js"));
 class ForInfo {
     constructor(forLoop) {
@@ -61,7 +64,7 @@ class ForInfo {
     }
 }
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta, {
+    meta: (0, generate_meta_js_1.generateMeta)(meta, {
         messages: {
             misplacedCounter: `This loop's stop condition tests "{{test}}" but the incrementer updates "{{update}}".`,
         },
@@ -90,7 +93,7 @@ exports.rule = {
             return false;
         }
         function peekFor() {
-            return (0, index_js_1.last)(forLoopStack);
+            return (0, collection_js_1.last)(forLoopStack);
         }
         return {
             ForStatement: (node) => {
@@ -101,7 +104,7 @@ exports.rule = {
                 if (forInfo.updatedExpressions.length === 0 || !forInfo.forLoop.test) {
                     return;
                 }
-                const hasIntersection = forInfo.testedExpressions.some(testedExpr => forInfo.updatedExpressions.some(updatedExpr => (0, index_js_1.areEquivalent)(updatedExpr, testedExpr, context.sourceCode)));
+                const hasIntersection = forInfo.testedExpressions.some(testedExpr => forInfo.updatedExpressions.some(updatedExpr => (0, equivalence_js_1.areEquivalent)(updatedExpr, testedExpr, context.sourceCode)));
                 if (!hasIntersection) {
                     context.report({
                         loc: context.sourceCode.getFirstToken(forInfo.forLoop).loc,
@@ -140,7 +143,7 @@ exports.rule = {
             },
             'ForStatement Identifier': (node) => {
                 if (isInsideTest(node)) {
-                    const parent = (0, index_js_1.getParent)(context, node);
+                    const parent = (0, ancestor_js_1.getParent)(context, node);
                     if (parent.type !== 'MemberExpression' || parent.computed || parent.object === node) {
                         peekFor().testedExpressions.push(node);
                     }
@@ -148,8 +151,8 @@ exports.rule = {
             },
             'ForStatement MemberExpression': (node) => {
                 if (isInsideTest(node) &&
-                    (0, index_js_1.getParent)(context, node).type !== 'MemberExpression' &&
-                    (0, index_js_1.getParent)(context, node).type !== 'CallExpression') {
+                    (0, ancestor_js_1.getParent)(context, node).type !== 'MemberExpression' &&
+                    (0, ancestor_js_1.getParent)(context, node).type !== 'CallExpression') {
                     peekFor().testedExpressions.push(node);
                 }
             },

package/cjs/S2004/rule.js

@@ -51,7 +51,9 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.rule = void 0;
-const index_js_1 = require("../helpers/index.js");
+const generate_meta_js_1 = require("../helpers/generate-meta.js");
+const location_js_1 = require("../helpers/location.js");
+const test_frameworks_js_1 = require("../helpers/test-frameworks.js");
 const meta = __importStar(require("./generated-meta.js"));
 const DEFAULT_THRESHOLD = 4;
 /**
@@ -63,10 +65,10 @@ function isTestFrameworkCallback(node) {
     if (parent?.type !== 'CallExpression') {
         return false;
     }
-    return (0, index_js_1.isTestFrameworkCall)(parent);
+    return (0, test_frameworks_js_1.isTestFrameworkCall)(parent);
 }
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta),
+    meta: (0, generate_meta_js_1.generateMeta)(meta),
     create(context) {
         const max = context.options[0]?.threshold ?? DEFAULT_THRESHOLD;
         const nestedStack = [];
@@ -80,11 +82,11 @@ exports.rule = {
                 nestedStack.push(fn);
                 if (nestedStack.length === max + 1) {
                     const secondaries = nestedStack.slice(0, -1);
-                    (0, index_js_1.report)(context, {
-                        loc: (0, index_js_1.getMainFunctionTokenLocation)(fn, fn.parent, context),
+                    (0, location_js_1.report)(context, {
+                        loc: (0, location_js_1.getMainFunctionTokenLocation)(fn, fn.parent, context),
                         message: `Refactor this code to not nest functions more than ${max} levels deep.`,
-                    }, secondaries.map(n => (0, index_js_1.toSecondaryLocation)({
-                        loc: (0, index_js_1.getMainFunctionTokenLocation)(n, n.parent, context),
+                    }, secondaries.map(n => (0, location_js_1.toSecondaryLocation)({
+                        loc: (0, location_js_1.getMainFunctionTokenLocation)(n, n.parent, context),
                     }, 'Nesting +1')));
                 }
             },

package/cjs/S2068/rule.js

@@ -54,17 +54,27 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.rule = void 0;
-const index_js_1 = require("../helpers/index.js");
+const generate_meta_js_1 = require("../helpers/generate-meta.js");
+const ast_js_1 = require("../helpers/ast.js");
+const entropy_js_1 = require("../helpers/entropy.js");
 const node_path_1 = __importDefault(require("node:path"));
 const meta = __importStar(require("./generated-meta.js"));
 const DEFAULT_NAMES = ['password', 'pwd', 'passwd', 'passphrase'];
+const ENTROPY_THRESHOLD = 3;
+const MIN_PASSWORD_LENGTH = 5;
+const NON_CREDENTIAL_CHARS = /[\s/["'\]<>]/;
+const TEST_FILE_PATTERN = /\.(spec|test|mock)\.[jt]sx?$/;
 const messages = {
     reviewPassword: 'Review this potentially hard-coded password.',
 };
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta, { messages }),
+    meta: (0, generate_meta_js_1.generateMeta)(meta, { messages }),
     create(context) {
-        const dir = node_path_1.default.dirname(context.physicalFilename);
+        const filename = context.physicalFilename;
+        if (TEST_FILE_PATTERN.test(filename)) {
+            return {};
+        }
+        const dir = node_path_1.default.dirname(filename);
         const parts = dir.split(node_path_1.default.sep).map(part => part.toLowerCase());
         if (parts.includes('l10n')) {
             return {};
@@ -98,21 +108,74 @@ exports.rule = {
 };
 function checkAssignment(context, patterns, variable, initializer) {
     if (initializer &&
-        (0, index_js_1.isStringLiteral)(initializer) &&
-        initializer.value.length > 0 &&
-        patterns.some(pattern => context.sourceCode.getText(variable).toLowerCase().includes(pattern))) {
+        patterns.some(pattern => context.sourceCode.getText(variable).toLowerCase().includes(pattern)) &&
+        findValueSuspect(initializer)) {
         context.report({
             messageId: 'reviewPassword',
             node: initializer,
         });
     }
 }
+function findValueSuspect(node) {
+    if (!node) {
+        return false;
+    }
+    if ((0, ast_js_1.isStringLiteral)(node)) {
+        const value = node.value;
+        return (value.length >= MIN_PASSWORD_LENGTH &&
+            !NON_CREDENTIAL_CHARS.test(value) &&
+            hasHighEntropy(value));
+    }
+    if (node.type === 'ConditionalExpression') {
+        return findValueSuspect(node.consequent) || findValueSuspect(node.alternate);
+    }
+    if ((0, ast_js_1.isLogicalExpression)(node) &&
+        ['??', '||'].includes(node.operator)) {
+        return findValueSuspect(node.right);
+    }
+    return false;
+}
 function checkLiteral(context, patterns, literal) {
-    if ((0, index_js_1.isStringLiteral)(literal) &&
-        patterns.some(pattern => pattern.test(literal.value.toLowerCase()))) {
-        context.report({
-            messageId: 'reviewPassword',
-            node: literal,
-        });
+    if (!(0, ast_js_1.isStringLiteral)(literal)) {
+        return;
     }
+    const value = literal.value;
+    const lowerValue = value.toLowerCase();
+    for (const pattern of patterns) {
+        const match = pattern.exec(lowerValue);
+        if (!match) {
+            continue;
+        }
+        const eqIndex = value.indexOf('=', match.index);
+        if (eqIndex === -1) {
+            continue;
+        }
+        const passwordValue = extractPasswordValue(value, eqIndex);
+        if (passwordValue.length >= MIN_PASSWORD_LENGTH && hasHighEntropy(passwordValue)) {
+            context.report({
+                messageId: 'reviewPassword',
+                node: literal,
+            });
+            return;
+        }
+    }
+}
+function extractPasswordValue(value, eqIndex) {
+    const passwordPart = value.substring(eqIndex + 1);
+    const nextSep = findNextSeparator(passwordPart);
+    return nextSep === -1 ? passwordPart : passwordPart.substring(0, nextSep);
+}
+function findNextSeparator(str) {
+    const separators = ['&', ' ', ';'];
+    let minIndex = -1;
+    for (const sep of separators) {
+        const idx = str.indexOf(sep);
+        if (idx !== -1 && (minIndex === -1 || idx < minIndex)) {
+            minIndex = idx;
+        }
+    }
+    return minIndex;
+}
+function hasHighEntropy(value) {
+    return (0, entropy_js_1.shannonEntropy)(value) > ENTROPY_THRESHOLD;
 }

package/cjs/S2077/rule.js

@@ -51,7 +51,11 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.rule = void 0;
-const index_js_1 = require("../helpers/index.js");
+const generate_meta_js_1 = require("../helpers/generate-meta.js");
+const module_js_1 = require("../helpers/module.js");
+const module_ts_js_1 = require("../helpers/module-ts.js");
+const ast_js_1 = require("../helpers/ast.js");
+const parser_services_js_1 = require("../helpers/parser-services.js");
 const meta = __importStar(require("./generated-meta.js"));
 const sqlQuerySignatures = new Set([
     'pg.Client.query',
@@ -72,23 +76,23 @@ const sqlQuerySignatures = new Set([
     'better-sqlite3.prepare',
 ]);
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta, {
+    meta: (0, generate_meta_js_1.generateMeta)(meta, {
         messages: {
             safeQuery: `Make sure that executing SQL queries is safe here.`,
         },
     }),
     create(context) {
         const services = context.sourceCode.parserServices;
-        const hasTypeInformation = (0, index_js_1.isRequiredParserServices)(services);
+        const hasTypeInformation = (0, parser_services_js_1.isRequiredParserServices)(services);
         return {
             CallExpression(node) {
                 let fqn = null;
                 if (hasTypeInformation) {
                     const tsNode = services.esTreeNodeToTSNodeMap.get(node);
-                    fqn = (0, index_js_1.getFullyQualifiedNameTS)(services, tsNode);
+                    fqn = (0, module_ts_js_1.getFullyQualifiedNameTS)(services, tsNode);
                 }
                 else {
-                    fqn = (0, index_js_1.getFullyQualifiedName)(context, node.callee);
+                    fqn = (0, module_js_1.getFullyQualifiedName)(context, node.callee);
                 }
                 if (fqn && sqlQuerySignatures.has(fqn) && isQuestionable(node.arguments[0])) {
                     context.report({
@@ -110,7 +114,7 @@ function isQuestionable(sqlQuery) {
     if (isConcatenation(sqlQuery)) {
         return isVariableConcat(sqlQuery);
     }
-    return (sqlQuery.type === 'CallExpression' && (0, index_js_1.isMemberWithProperty)(sqlQuery.callee, 'concat', 'replace'));
+    return (sqlQuery.type === 'CallExpression' && (0, ast_js_1.isMemberWithProperty)(sqlQuery.callee, 'concat', 'replace'));
 }
 function isVariableConcat(node) {
     const { left, right } = node;

package/cjs/S2092/rule.js

@@ -52,10 +52,10 @@ var __importStar = (this && this.__importStar) || (function () {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.rule = void 0;
 const cookie_flag_check_js_1 = require("../helpers/cookie-flag-check.js");
-const index_js_1 = require("../helpers/index.js");
+const generate_meta_js_1 = require("../helpers/generate-meta.js");
 const meta = __importStar(require("./generated-meta.js"));
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta),
+    meta: (0, generate_meta_js_1.generateMeta)(meta),
     create(context) {
         return {
             CallExpression: (node) => new cookie_flag_check_js_1.CookieFlagCheck(context, 'secure').checkCookiesFromCallExpression(node),

package/cjs/S2123/rule.js

@@ -51,10 +51,10 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.rule = void 0;
-const index_js_1 = require("../helpers/index.js");
+const generate_meta_js_1 = require("../helpers/generate-meta.js");
 const meta = __importStar(require("./generated-meta.js"));
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta, {
+    meta: (0, generate_meta_js_1.generateMeta)(meta, {
         messages: {
             removeIncrement: 'Remove this {{updateOperator}}rement or correct the code not to waste it.',
         },

package/cjs/S2137/rule.js

@@ -55,7 +55,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.rule = void 0;
 const globals_1 = __importDefault(require("globals"));
-const index_js_1 = require("../helpers/index.js");
+const generate_meta_js_1 = require("../helpers/generate-meta.js");
 const meta = __importStar(require("./generated-meta.js"));
 const illegalNames = new Set(['arguments']);
 const objectPrototypeProperties = new Set([
@@ -77,7 +77,7 @@ const getModificationIssue = (functionName) => ({
     data: { symbol: functionName },
 });
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta, {
+    meta: (0, generate_meta_js_1.generateMeta)(meta, {
         messages: {
             removeModification: 'Remove the modification of "{{symbol}}".',
             forbidDeclaration: 'Do not use "{{symbol}}" to declare a {{type}} - use another name.',

package/cjs/S2138/rule.js

@@ -51,17 +51,18 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.rule = void 0;
-const index_js_1 = require("../helpers/index.js");
+const generate_meta_js_1 = require("../helpers/generate-meta.js");
+const ast_js_1 = require("../helpers/ast.js");
 const meta = __importStar(require("./generated-meta.js"));
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta, {
+    meta: (0, generate_meta_js_1.generateMeta)(meta, {
         messages: {
             useNull: 'Use null instead.',
         },
     }),
     create(context) {
         function raiseOnUndefined(node) {
-            if ((0, index_js_1.isUndefined)(node)) {
+            if ((0, ast_js_1.isUndefined)(node)) {
                 context.report({
                     messageId: 'useNull',
                     node,

package/cjs/S2187/rule.js

@@ -51,7 +51,7 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.rule = void 0;
-const index_js_1 = require("../helpers/index.js");
+const generate_meta_js_1 = require("../helpers/generate-meta.js");
 const meta = __importStar(require("./generated-meta.js"));
 const APIs = new Set([
     // Jasmine
@@ -114,7 +114,7 @@ const APIs = new Set([
     'ruleTester.run',
 ]);
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta, {
+    meta: (0, generate_meta_js_1.generateMeta)(meta, {
         messages: {
             missingTest: 'Add some tests to this file or delete it.',
         },

package/cjs/S2201/rule.js

@@ -55,7 +55,10 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.rule = void 0;
 const typescript_1 = __importDefault(require("typescript"));
-const index_js_1 = require("../helpers/index.js");
+const ancestor_js_1 = require("../helpers/ancestor.js");
+const generate_meta_js_1 = require("../helpers/generate-meta.js");
+const type_js_1 = require("../helpers/type.js");
+const parser_services_js_1 = require("../helpers/parser-services.js");
 const meta = __importStar(require("./generated-meta.js"));
 const METHODS_WITHOUT_SIDE_EFFECTS = {
     array: new Set([
@@ -205,7 +208,7 @@ const METHODS_WITHOUT_SIDE_EFFECTS = {
     ]),
 };
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta, {
+    meta: (0, generate_meta_js_1.generateMeta)(meta, {
         messages: {
             useForEach: `Consider using "forEach" instead of "map" as its return value is not being used here.`,
             returnValueMustBeUsed: 'The return value of "{{methodName}}" must be used.',
@@ -213,7 +216,7 @@ exports.rule = {
     }),
     create(context) {
         const services = context.sourceCode.parserServices;
-        if (!(0, index_js_1.isRequiredParserServices)(services)) {
+        if (!(0, parser_services_js_1.isRequiredParserServices)(services)) {
             return {};
         }
         return {
@@ -244,7 +247,7 @@ const isFunctionTypeNode = (candidate) => {
 };
 function isReplaceWithCallback(methodName, callArguments, services) {
     if (methodName === 'replace' && callArguments.length > 1) {
-        const type = (0, index_js_1.getTypeFromTreeNode)(callArguments[1], services);
+        const type = (0, type_js_1.getTypeFromTreeNode)(callArguments[1], services);
         const typeNode = services.program.getTypeChecker().typeToTypeNode(type, undefined, undefined);
         return typeNode && isFunctionTypeNode(typeNode);
     }
@@ -284,7 +287,7 @@ function containsAssignment(node, visitorKeys) {
     if (FUNCTION_BOUNDARIES.has(node.type)) {
         return false;
     }
-    return (0, index_js_1.childrenOf)(node, visitorKeys).some(child => containsAssignment(child, visitorKeys));
+    return (0, ancestor_js_1.childrenOf)(node, visitorKeys).some(child => containsAssignment(child, visitorKeys));
 }
 function reportDescriptor(methodName, node) {
     if (methodName === 'map') {

package/cjs/S2208/rule.js

@@ -51,10 +51,10 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.rule = void 0;
-const index_js_1 = require("../helpers/index.js");
+const generate_meta_js_1 = require("../helpers/generate-meta.js");
 const meta = __importStar(require("./generated-meta.js"));
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta, {
+    meta: (0, generate_meta_js_1.generateMeta)(meta, {
         messages: {
             wildcardImport: 'Explicitly {{xPort}} the specific member needed.',
         },

package/cjs/S2234/rule.js

@@ -51,13 +51,17 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.rule = void 0;
-const index_js_1 = require("../helpers/index.js");
+const ast_js_1 = require("../helpers/ast.js");
+const generate_meta_js_1 = require("../helpers/generate-meta.js");
+const type_js_1 = require("../helpers/type.js");
+const parser_services_js_1 = require("../helpers/parser-services.js");
+const location_js_1 = require("../helpers/location.js");
 const meta = __importStar(require("./generated-meta.js"));
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta),
+    meta: (0, generate_meta_js_1.generateMeta)(meta),
     create(context) {
         const services = context.sourceCode.parserServices;
-        const canResolveType = (0, index_js_1.isRequiredParserServices)(services);
+        const canResolveType = (0, parser_services_js_1.isRequiredParserServices)(services);
         function checkArguments(functionCall) {
             // Extract argument names first (cheap operation)
             const argumentNames = functionCall.arguments.map(arg => {
@@ -141,11 +145,11 @@ exports.rule = {
                 return resolveFromTSSignature(node);
             }
             let functionDeclaration = null;
-            if ((0, index_js_1.isFunctionNode)(node.callee)) {
+            if ((0, ast_js_1.isFunctionNode)(node.callee)) {
                 functionDeclaration = node.callee;
             }
             else if (node.callee.type === 'Identifier') {
-                functionDeclaration = (0, index_js_1.resolveFromFunctionReference)(context, node.callee);
+                functionDeclaration = (0, ast_js_1.resolveFromFunctionReference)(context, node.callee);
             }
             if (!functionDeclaration) {
                 return null;
@@ -156,7 +160,7 @@ exports.rule = {
             };
         }
         function resolveFromTSSignature(node) {
-            const signature = (0, index_js_1.getSignatureFromCallee)(node, services);
+            const signature = (0, type_js_1.getSignatureFromCallee)(node, services);
             if (signature?.declaration) {
                 return {
                     params: signature.parameters.map(param => param.name),
@@ -179,14 +183,14 @@ exports.rule = {
         }
         function haveCompatibleTypes(arg1, arg2) {
             if (canResolveType) {
-                const type1 = normalizeType((0, index_js_1.getTypeAsString)(arg1, services));
-                const type2 = normalizeType((0, index_js_1.getTypeAsString)(arg2, services));
+                const type1 = normalizeType((0, type_js_1.getTypeAsString)(arg1, services));
+                const type2 = normalizeType((0, type_js_1.getTypeAsString)(arg2, services));
                 return type1 === type2;
             }
             return true;
         }
         function raiseIssue(arg1, arg2, functionDeclaration, node) {
-            (0, index_js_1.report)(context, {
+            (0, location_js_1.report)(context, {
                 message: `Arguments '${arg1}' and '${arg2}' have the same names but not the same order as the function parameters.`,
                 loc: getParametersClauseLocation(node.arguments),
             }, getSecondaryLocations(functionDeclaration));
@@ -243,7 +247,7 @@ function isCryptoCyclicRotation(functionCall, functionParameters) {
 }
 function extractFunctionParameters(functionDeclaration) {
     return functionDeclaration.params.map(param => {
-        const identifiers = (0, index_js_1.resolveIdentifiers)(param);
+        const identifiers = (0, ast_js_1.resolveIdentifiers)(param);
         if (identifiers.length === 1 && identifiers[0]) {
             return identifiers[0].name;
         }
@@ -253,7 +257,7 @@ function extractFunctionParameters(functionDeclaration) {
 function getSecondaryLocations(functionDeclaration) {
     if (functionDeclaration?.params && functionDeclaration.params.length > 0) {
         const { start, end } = getParametersClauseLocation(functionDeclaration.params);
-        return [(0, index_js_1.toSecondaryLocation)({ loc: { start, end } }, 'Formal parameters')];
+        return [(0, location_js_1.toSecondaryLocation)({ loc: { start, end } }, 'Formal parameters')];
     }
     return [];
 }

package/cjs/S2245/rule.js

@@ -51,10 +51,11 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.rule = void 0;
-const index_js_1 = require("../helpers/index.js");
+const generate_meta_js_1 = require("../helpers/generate-meta.js");
+const module_js_1 = require("../helpers/module.js");
 const meta = __importStar(require("./generated-meta.js"));
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta, {
+    meta: (0, generate_meta_js_1.generateMeta)(meta, {
         messages: {
             safeGenerator: 'Make sure that using this pseudorandom number generator is safe here.',
         },
@@ -62,7 +63,7 @@ exports.rule = {
     create(context) {
         return {
             CallExpression(node) {
-                const fqn = (0, index_js_1.getFullyQualifiedName)(context, node);
+                const fqn = (0, module_js_1.getFullyQualifiedName)(context, node);
                 if (fqn === 'Math.random') {
                     context.report({
                         messageId: 'safeGenerator',

package/cjs/S2251/rule.js

@@ -51,10 +51,11 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.rule = void 0;
-const index_js_1 = require("../helpers/index.js");
+const generate_meta_js_1 = require("../helpers/generate-meta.js");
+const location_js_1 = require("../helpers/location.js");
 const meta = __importStar(require("./generated-meta.js"));
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta),
+    meta: (0, generate_meta_js_1.generateMeta)(meta),
     create(context) {
         return {
             ForStatement: (node) => {
@@ -67,10 +68,10 @@ exports.rule = {
                 const wrongDirection = getWrongDirection(test, loopIncrement);
                 if (wrongDirection !== 0 && wrongDirection === loopIncrement.direction) {
                     const movement = wrongDirection > 0 ? 'incremented' : 'decremented';
-                    (0, index_js_1.report)(context, {
+                    (0, location_js_1.report)(context, {
                         message: `"${loopIncrement.identifier.name}" is ${movement} and will never reach its stop condition.`,
                         node: forStatement.update,
-                    }, [(0, index_js_1.toSecondaryLocation)(test)]);
+                    }, [(0, location_js_1.toSecondaryLocation)(test)]);
                 }
             },
         };

package/cjs/S2255/rule.js

@@ -51,10 +51,11 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.rule = void 0;
-const index_js_1 = require("../helpers/index.js");
+const generate_meta_js_1 = require("../helpers/generate-meta.js");
+const ast_js_1 = require("../helpers/ast.js");
 const meta = __importStar(require("./generated-meta.js"));
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta, {
+    meta: (0, generate_meta_js_1.generateMeta)(meta, {
         messages: {
             safeCookie: 'Make sure that cookie is written safely here.',
         },
@@ -75,7 +76,7 @@ exports.rule = {
                 const { left } = node;
                 if (left.type === 'MemberExpression') {
                     const { object, property } = left;
-                    if ((0, index_js_1.isIdentifier)(object, 'document') && (0, index_js_1.isIdentifier)(property, 'cookie')) {
+                    if ((0, ast_js_1.isIdentifier)(object, 'document') && (0, ast_js_1.isIdentifier)(property, 'cookie')) {
                         context.report({
                             messageId: 'safeCookie',
                             node: left,
@@ -87,14 +88,14 @@ exports.rule = {
                 const { callee, arguments: args } = node;
                 if (callee.type === 'MemberExpression' &&
                     usingExpressFramework &&
-                    (0, index_js_1.isIdentifier)(callee.property, 'cookie', 'cookies')) {
+                    (0, ast_js_1.isIdentifier)(callee.property, 'cookie', 'cookies')) {
                     context.report({
                         messageId: 'safeCookie',
                         node,
                     });
                 }
                 if (callee.type === 'MemberExpression' &&
-                    (0, index_js_1.isIdentifier)(callee.property, 'setHeader') &&
+                    (0, ast_js_1.isIdentifier)(callee.property, 'setHeader') &&
                     isLiteral(args[0], 'Set-Cookie')) {
                     context.report({
                         messageId: 'safeCookie',