eslint-plugin-sonarjs 4.0.0 → 4.0.2

package/cjs/S6759/rule.js

@@ -54,12 +54,16 @@ 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 parser_services_js_1 = require("../helpers/parser-services.js");
+const generate_meta_js_1 = require("../helpers/generate-meta.js");
+const type_js_1 = require("../helpers/type.js");
+const ast_js_1 = require("../helpers/ast.js");
+const collection_js_1 = require("../helpers/collection.js");
 const ts_api_utils_1 = require("ts-api-utils");
 const typescript_1 = __importDefault(require("typescript"));
 const meta = __importStar(require("./generated-meta.js"));
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta, {
+    meta: (0, generate_meta_js_1.generateMeta)(meta, {
         hasSuggestions: true,
         messages: {
             readOnlyProps: 'Mark the props of the component as read-only.',
@@ -68,7 +72,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 {};
         }
         const functionInfo = [];
@@ -112,7 +116,7 @@ exports.rule = {
                 }
             },
             ReturnStatement(node) {
-                const current = (0, index_js_1.last)(functionInfo);
+                const current = (0, collection_js_1.last)(functionInfo);
                 if (current) {
                     current.returns.push(node);
                 }
@@ -144,7 +148,7 @@ exports.rule = {
                 if (!ret.argument) {
                     continue;
                 }
-                const value = (0, index_js_1.getUniqueWriteUsageOrNode)(context, ret.argument);
+                const value = (0, ast_js_1.getUniqueWriteUsageOrNode)(context, ret.argument);
                 if (value.type.startsWith('JSX')) {
                     return true;
                 }
@@ -159,7 +163,7 @@ exports.rule = {
  * resolved properties are read-only (regardless of how the type was constructed).
  */
 function isReadOnly(props, services) {
-    const type = (0, index_js_1.getTypeFromTreeNode)(props, services);
+    const type = (0, type_js_1.getTypeFromTreeNode)(props, services);
     const checker = services.program.getTypeChecker();
     /* Readonly utility type */
     if (type.aliasSymbol?.escapedName === 'Readonly') {

package/cjs/S6958/rule.js

@@ -34,7 +34,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 NODES = new Set([
     'ArrayExpression',
@@ -44,7 +44,7 @@ const NODES = new Set([
     'TemplateLiteral',
 ]);
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta, {
+    meta: (0, generate_meta_js_1.generateMeta)(meta, {
         messages: {
             asFunction: 'Literal should not be used as function.',
             asTagFunction: 'Literal should not be used as tag function.',

package/cjs/S6959/rule.js

@@ -51,10 +51,13 @@ 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 type_js_1 = require("../helpers/type.js");
+const parser_services_js_1 = require("../helpers/parser-services.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: {
             message: 'Add an initial value to this "reduce()" call.',
         },
@@ -62,16 +65,16 @@ exports.rule = {
     create(context) {
         const services = context.sourceCode.parserServices;
         function isArray(node) {
-            if ((0, index_js_1.isRequiredParserServices)(services)) {
-                return (0, index_js_1.isArray)(node, services);
+            if ((0, parser_services_js_1.isRequiredParserServices)(services)) {
+                return (0, type_js_1.isArray)(node, services);
             }
             else {
-                return (0, index_js_1.isArrayExpression)((0, index_js_1.getUniqueWriteUsageOrNode)(context, node));
+                return (0, ast_js_1.isArrayExpression)((0, ast_js_1.getUniqueWriteUsageOrNode)(context, node));
             }
         }
         return {
             CallExpression(node) {
-                if ((0, index_js_1.isCallingMethod)(node, 1, 'reduce') && isArray(node.callee.object)) {
+                if ((0, ast_js_1.isCallingMethod)(node, 1, 'reduce') && isArray(node.callee.object)) {
                     context.report({
                         node: node.callee.property,
                         messageId: 'message',

package/cjs/S7059/generated-meta.js

@@ -39,7 +39,7 @@ exports.meta = {
         description: 'Constructors should not contain asynchronous operations',
         recommended: true,
         url: 'https://sonarsource.github.io/rspec/#/rspec/S7059/javascript',
-        requiresTypeChecking: false,
+        requiresTypeChecking: true,
     },
     fixable: undefined,
     deprecated: false,

package/cjs/S7059/rule.js

@@ -51,18 +51,21 @@ 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 parser_services_js_1 = require("../helpers/parser-services.js");
+const generate_meta_js_1 = require("../helpers/generate-meta.js");
+const type_js_1 = require("../helpers/type.js");
+const ast_js_1 = require("../helpers/ast.js");
 const meta = __importStar(require("./generated-meta.js"));
 const flaggedStatements = new Set();
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta, {
+    meta: (0, generate_meta_js_1.generateMeta)(meta, {
         messages: {
             noAsyncConstructor: 'Refactor this asynchronous operation outside of the constructor.',
         },
     }),
     create(context) {
         const services = context.sourceCode.parserServices;
-        if (!(0, index_js_1.isRequiredParserServices)(services)) {
+        if (!(0, parser_services_js_1.isRequiredParserServices)(services)) {
             return {};
         }
         /**
@@ -88,7 +91,7 @@ exports.rule = {
                     statement = ancestor;
                 }
                 // If we find a function declaration it should not be considered as part of the constructor
-                if (classConstructor && statement && (0, index_js_1.isFunctionNode)(ancestor)) {
+                if (classConstructor && statement && (0, ast_js_1.isFunctionNode)(ancestor)) {
                     statement = undefined;
                     classConstructor = undefined;
                 }
@@ -97,7 +100,7 @@ exports.rule = {
         }
         return {
             CallExpression(node) {
-                if (!(0, index_js_1.isThenable)(node, services)) {
+                if (!(0, type_js_1.isThenable)(node, services)) {
                     return;
                 }
                 // we want to raise on the parent statement

package/cjs/S7639/generated-meta.js

@@ -46,6 +46,6 @@ exports.meta = {
     defaultOptions: [],
 };
 exports.sonarKey = 'S7639';
-exports.scope = 'All';
+exports.scope = 'Main';
 exports.languages = ['js', 'ts'];
 exports.requiredDependency = [];

package/cjs/S7639/rule.js

@@ -51,12 +51,13 @@ 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"));
 const BLOCKCHAIN_MODULES = ['ethers', 'viem/accounts', 'tronweb'];
 const MNEMONIC_FUNCTIONS = ['fromPhrase', 'mnemonicToAccount', 'fromMnemonic'];
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta, {
+    meta: (0, generate_meta_js_1.generateMeta)(meta, {
         messages: {
             reviewBlockchainSeedPhrase: `Revoke and change this seed phrase, as it is compromised.`,
         },
@@ -87,7 +88,7 @@ exports.rule = {
             return expr;
         }
         function isMnemonicFunction(callee) {
-            return MNEMONIC_FUNCTIONS.some(func => (0, index_js_1.isMemberWithProperty)(callee, func) || (0, index_js_1.isIdentifier)(callee, func));
+            return MNEMONIC_FUNCTIONS.some(func => (0, ast_js_1.isMemberWithProperty)(callee, func) || (0, ast_js_1.isIdentifier)(callee, func));
         }
         return {
             Program() {
@@ -109,7 +110,7 @@ exports.rule = {
                 }
             },
             CallExpression(node) {
-                if ((0, index_js_1.isRequireModule)(node, ...BLOCKCHAIN_MODULES)) {
+                if ((0, ast_js_1.isRequireModule)(node, ...BLOCKCHAIN_MODULES)) {
                     isBlockchainModuleImported = true;
                     return;
                 }

package/cjs/S7790/generated-meta.js

@@ -46,6 +46,6 @@ exports.meta = {
     defaultOptions: [],
 };
 exports.sonarKey = 'S7790';
-exports.scope = 'All';
+exports.scope = 'Main';
 exports.languages = ['js', 'ts'];
 exports.requiredDependency = [];

package/cjs/S7790/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 module_js_1 = require("../helpers/module.js");
+const ast_js_1 = require("../helpers/ast.js");
 const meta = __importStar(require("./generated-meta.js"));
 const templatingFqns = new Set([
     'pug.compile',
@@ -60,7 +62,7 @@ const templatingFqns = new Set([
     'ejs.render',
 ]);
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta, {
+    meta: (0, generate_meta_js_1.generateMeta)(meta, {
         messages: {
             reviewDynamicTemplate: `Make sure this dynamically formatted template is safe here.`,
         },
@@ -69,7 +71,7 @@ exports.rule = {
         return {
             CallExpression: (node) => {
                 const callExpression = node;
-                const fqn = (0, index_js_1.getFullyQualifiedName)(context, callExpression);
+                const fqn = (0, module_js_1.getFullyQualifiedName)(context, callExpression);
                 if (fqn &&
                     templatingFqns.has(fqn) &&
                     !isCallingFunctionResult(context, callExpression) &&
@@ -93,11 +95,11 @@ function isCallingFunctionResult(context, callExpression) {
     if (callee.type !== 'Identifier') {
         return false;
     }
-    const variable = (0, index_js_1.getVariableFromScope)(context.sourceCode.getScope(callee), callee.name);
+    const variable = (0, ast_js_1.getVariableFromScope)(context.sourceCode.getScope(callee), callee.name);
     if (!variable || variable.defs.some(def => def.type === 'ImportBinding')) {
         return false;
     }
-    const writeRef = (0, index_js_1.getUniqueWriteReference)(variable);
+    const writeRef = (0, ast_js_1.getUniqueWriteReference)(variable);
     return writeRef?.type === 'CallExpression';
 }
 function isQuestionable(node, index = 0) {

package/cjs/S8441/generated-meta.js

@@ -46,6 +46,6 @@ exports.meta = {
     defaultOptions: [],
 };
 exports.sonarKey = 'S8441';
-exports.scope = 'All';
+exports.scope = 'Main';
 exports.languages = ['js', 'ts'];
 exports.requiredDependency = [];

package/cjs/S8441/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 express_js_1 = require("../helpers/express.js");
+const ast_js_1 = require("../helpers/ast.js");
+const generate_meta_js_1 = require("../helpers/generate-meta.js");
+const module_js_1 = require("../helpers/module.js");
+const location_js_1 = require("../helpers/location.js");
 // If a rule has a schema, use this to extract it.
 // import { FromSchema } from 'json-schema-to-ts';
 const meta = __importStar(require("./generated-meta.js"));
@@ -63,7 +67,7 @@ const messages = {
 const SESSION_MIDDLEWARES = ['express-session', 'cookie-session'];
 const STATIC_MIDDLEWARES = ['express.static'];
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta, { messages }),
+    meta: (0, generate_meta_js_1.generateMeta)(meta, { messages }),
     create(context) {
         let app = null;
         let lastSessionMiddleware = null;
@@ -72,7 +76,7 @@ exports.rule = {
             if (node.type !== 'CallExpression') {
                 return false;
             }
-            const fqn = (0, index_js_1.getFullyQualifiedName)(context, node);
+            const fqn = (0, module_js_1.getFullyQualifiedName)(context, node);
             return fqn !== null && names.includes(fqn);
         }
         return {
@@ -83,7 +87,7 @@ exports.rule = {
             },
             ':function'(node) {
                 scopeStack.push({ app, lastSessionMiddleware });
-                const injectedApp = index_js_1.Express.attemptFindAppInjection(node, context, node);
+                const injectedApp = (0, express_js_1.attemptFindAppInjection)(node, context, node);
                 if (injectedApp) {
                     app = injectedApp;
                     lastSessionMiddleware = null;
@@ -98,7 +102,7 @@ exports.rule = {
             },
             VariableDeclarator(node) {
                 const varDecl = node;
-                const instantiatedApp = index_js_1.Express.attemptFindAppInstantiation(varDecl, context);
+                const instantiatedApp = (0, express_js_1.attemptFindAppInstantiation)(varDecl, context);
                 if (instantiatedApp) {
                     app = instantiatedApp;
                     lastSessionMiddleware = null;
@@ -109,21 +113,21 @@ exports.rule = {
                     return;
                 }
                 const callExpr = node;
-                if (!(0, index_js_1.isMethodInvocation)(callExpr, app.name, 'use', 1)) {
+                if (!(0, ast_js_1.isMethodInvocation)(callExpr, app.name, 'use', 1)) {
                     return;
                 }
-                const flattenedArgs = (0, index_js_1.flattenArgs)(context, callExpr.arguments);
+                const flattenedArgs = (0, ast_js_1.flattenArgs)(context, callExpr.arguments);
                 for (const middlewareNode of flattenedArgs) {
                     if (isMiddleware(context, middlewareNode, SESSION_MIDDLEWARES)) {
                         lastSessionMiddleware = callExpr;
                         continue;
                     }
                     if (lastSessionMiddleware && isMiddleware(context, middlewareNode, STATIC_MIDDLEWARES)) {
-                        (0, index_js_1.report)(context, {
+                        (0, location_js_1.report)(context, {
                             node: callExpr,
                             messageId: 'moveStaticBeforeSession',
                             message: messages.moveStaticBeforeSession,
-                        }, [(0, index_js_1.toSecondaryLocation)(lastSessionMiddleware, messages.sessionSecondaryLocation)]);
+                        }, [(0, location_js_1.toSecondaryLocation)(lastSessionMiddleware, messages.sessionSecondaryLocation)]);
                     }
                 }
             },

package/cjs/S881/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: {
             extractOperation: 'Extract this {{incrementType}} operation into a dedicated statement.',
         },

package/cjs/S888/rule.js

@@ -51,13 +51,14 @@ 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"));
 const allEqualityOperators = new Set(['!=', '==', '!==', '===']);
 const notEqualOperators = new Set(['!==', '!=']);
 const plusMinusOperators = new Set(['+=', '-=']);
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta, {
+    meta: (0, generate_meta_js_1.generateMeta)(meta, {
         messages: {
             replaceOperator: "Replace '{{operator}}' operator with one of '<=', '>=', '<', or '>' comparison operators.",
         },
@@ -164,7 +165,7 @@ function isUpdateByOne(update, loopBody, context) {
 }
 function isUsedInsideBody(id, loopBody, context) {
     if (id.type === 'Identifier') {
-        const variable = (0, index_js_1.getVariableFromName)(context, id.name, id);
+        const variable = (0, ast_js_1.getVariableFromName)(context, id.name, id);
         const bodyRange = loopBody.range;
         if (variable && bodyRange) {
             return variable.references.some(ref => isInBody(ref.identifier, bodyRange));

package/cjs/S930/rule.js

@@ -51,11 +51,13 @@ 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 location_js_1 = require("../helpers/location.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"));
 const message = 'This function expects {{expectedArguments}}, but {{providedArguments}} provided.';
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta, {
+    meta: (0, generate_meta_js_1.generateMeta)(meta, {
         messages: {
             tooManyArguments: message,
         },
@@ -66,7 +68,7 @@ exports.rule = {
         const emptyFunctions = new Set();
         return {
             CallExpression(callExpr) {
-                if ((0, index_js_1.isIdentifier)(callExpr.callee)) {
+                if ((0, ast_js_1.isIdentifier)(callExpr.callee)) {
                     const reference = context.sourceCode
                         .getScope(callExpr)
                         .references.find(ref => ref.identifier === callExpr.callee);
@@ -77,14 +79,14 @@ exports.rule = {
                         }
                         else if (definition.type === 'Variable') {
                             const { init } = definition.node;
-                            if (init && ((0, index_js_1.isFunctionExpression)(init) || (0, index_js_1.isArrowFunctionExpression)(init))) {
+                            if (init && ((0, ast_js_1.isFunctionExpression)(init) || (0, ast_js_1.isArrowFunctionExpression)(init))) {
                                 checkFunction(callExpr, init);
                             }
                         }
                     }
                 }
-                else if ((0, index_js_1.isArrowFunctionExpression)(callExpr.callee) ||
-                    (0, index_js_1.isFunctionExpression)(callExpr.callee)) {
+                else if ((0, ast_js_1.isArrowFunctionExpression)(callExpr.callee) ||
+                    (0, ast_js_1.isFunctionExpression)(callExpr.callee)) {
                     // IIFE
                     checkFunction(callExpr, callExpr.callee);
                 }
@@ -120,8 +122,8 @@ exports.rule = {
                 // special `arguments` variable has no definition
                 if (!definition) {
                     const ancestors = context.sourceCode.getAncestors(identifier).reverse();
-                    const fn = ancestors.find(node => (0, index_js_1.isFunctionDeclaration)(node) ||
-                        (0, index_js_1.isFunctionExpression)(node));
+                    const fn = ancestors.find(node => (0, ast_js_1.isFunctionDeclaration)(node) ||
+                        (0, ast_js_1.isFunctionExpression)(node));
                     if (fn) {
                         usingArguments.add(fn);
                     }
@@ -145,7 +147,7 @@ exports.rule = {
             const providedArguments = argsLength === 0 ? "none was" :
                 argsLength === 1 ? "1 was" :
                     `${argsLength} were`;
-            (0, index_js_1.report)(context, {
+            (0, location_js_1.report)(context, {
                 message,
                 messageId: 'tooManyArguments',
                 data: {
@@ -159,13 +161,13 @@ exports.rule = {
             const paramLength = functionNode.params.length;
             const secondaryLocations = [];
             if (paramLength > 0) {
-                secondaryLocations.push((0, index_js_1.toSecondaryLocation)(functionNode.params[0], functionNode.params[paramLength - 1], 'Formal parameters'));
+                secondaryLocations.push((0, location_js_1.toSecondaryLocation)(functionNode.params[0], functionNode.params[paramLength - 1], 'Formal parameters'));
             }
             else {
                 // as we're not providing parent node, `getMainFunctionTokenLocation` may return `undefined`
-                const fnToken = (0, index_js_1.getMainFunctionTokenLocation)(functionNode, undefined, context);
+                const fnToken = (0, location_js_1.getMainFunctionTokenLocation)(functionNode, undefined, context);
                 if (fnToken) {
-                    secondaryLocations.push((0, index_js_1.toSecondaryLocation)({ loc: fnToken }, 'Formal parameters'));
+                    secondaryLocations.push((0, location_js_1.toSecondaryLocation)({ loc: fnToken }, 'Formal parameters'));
                 }
             }
             // find actual extra arguments to highlight

package/cjs/helpers/ast.js

@@ -56,7 +56,10 @@ exports.isProperty = isProperty;
 exports.isUnresolved = isUnresolved;
 exports.hasParent = hasParent;
 exports.hasTypePredicateReturn = hasTypePredicateReturn;
-const index_js_1 = require("./index.js");
+const ancestor_js_1 = require("./ancestor.js");
+const collection_js_1 = require("./collection.js");
+const module_js_1 = require("./module.js");
+const location_js_1 = require("./location.js");
 const MODULE_DECLARATION_NODES = new Set([
     'ImportDeclaration',
     'ExportNamedDeclaration',
@@ -256,7 +259,7 @@ function isNodeType(node, type) {
  */
 function getLhsVariable(context, node) {
     const ancestors = context.sourceCode.getAncestors(node);
-    const parent = (0, index_js_1.last)(ancestors);
+    const parent = (0, collection_js_1.last)(ancestors);
     let formIdentifier;
     if (parent.type === 'VariableDeclarator' && parent.id.type === 'Identifier') {
         formIdentifier = parent.id;
@@ -304,13 +307,13 @@ function flattenArgs(context, args) {
     function recHelper(nodePossiblyIdentifier) {
         const n = getUniqueWriteUsageOrNode(context, nodePossiblyIdentifier);
         if (n.type === 'ArrayExpression') {
-            return (0, index_js_1.flatMap)(n.elements, recHelper);
+            return (0, collection_js_1.flatMap)(n.elements, recHelper);
         }
         else {
             return [n];
         }
     }
-    return (0, index_js_1.flatMap)(args, recHelper);
+    return (0, collection_js_1.flatMap)(args, recHelper);
 }
 function resolveIdentifiers(node, acceptShorthand = false) {
     const identifiers = [];
@@ -363,7 +366,7 @@ function getPropertyWithValue(context, objectExpression, propertyName, propertyV
 }
 function getPropertyFromSpreadElement(spreadElement, key, ctx) {
     const props = getValueOfExpression(ctx, spreadElement.argument, 'ObjectExpression');
-    const recursiveDefinition = (0, index_js_1.findFirstMatchingAncestor)(spreadElement.argument, node => node === props);
+    const recursiveDefinition = (0, ancestor_js_1.findFirstMatchingAncestor)(spreadElement.argument, node => node === props);
     if (recursiveDefinition || props === undefined) {
         return undefined;
     }
@@ -436,10 +439,10 @@ function checkSensitiveCall(context, callExpression, sensitiveArgumentIndex, sen
     }
     const unsafeProperty = getPropertyWithValue(context, options, sensitiveProperty, sensitivePropertyValue);
     if (unsafeProperty) {
-        (0, index_js_1.report)(context, {
+        (0, location_js_1.report)(context, {
             node: callExpression.callee,
             message,
-        }, [(0, index_js_1.toSecondaryLocation)(unsafeProperty)]);
+        }, [(0, location_js_1.toSecondaryLocation)(unsafeProperty)]);
     }
 }
 function isStringLiteral(node) {
@@ -509,7 +512,7 @@ function isProperty(node) {
  * @param ctx Rule context
  */
 function isUnresolved(node, ctx) {
-    if (!node || (0, index_js_1.getFullyQualifiedName)(ctx, node) || isUndefined(node)) {
+    if (!node || (0, module_js_1.getFullyQualifiedName)(ctx, node) || isUndefined(node)) {
         return false;
     }
     let nodeToCheck = node;

package/cjs/helpers/aws/s3.js

@@ -5,7 +5,10 @@ exports.isS3BucketConstructor = isS3BucketConstructor;
 exports.isS3BucketDeploymentConstructor = isS3BucketDeploymentConstructor;
 exports.getBucketProperty = getBucketProperty;
 exports.findPropagatedSetting = findPropagatedSetting;
-const index_js_1 = require("../index.js");
+const location_js_1 = require("../location.js");
+const module_js_1 = require("../module.js");
+const ancestor_js_1 = require("../ancestor.js");
+const ast_js_1 = require("../ast.js");
 const cdk_js_1 = require("./cdk.js");
 /**
  * A rule template for AWS S3 Buckets
@@ -40,7 +43,7 @@ function S3BucketTemplate(callback, meta = {}) {
  * new s3.Bucket();
  */
 function isS3BucketConstructor(context, node) {
-    return (0, cdk_js_1.normalizeFQN)((0, index_js_1.getFullyQualifiedName)(context, node)) === 'aws_cdk_lib.aws_s3.Bucket';
+    return (0, cdk_js_1.normalizeFQN)((0, module_js_1.getFullyQualifiedName)(context, node)) === 'aws_cdk_lib.aws_s3.Bucket';
 }
 /**
  * Detects S3 BucketDeployment's constructor invocation from 'aws-cdk-lib/aws-s3-deployment':
@@ -49,7 +52,7 @@ function isS3BucketConstructor(context, node) {
  * new s3.BucketDeployment();
  */
 function isS3BucketDeploymentConstructor(context, node) {
-    return ((0, cdk_js_1.normalizeFQN)((0, index_js_1.getFullyQualifiedName)(context, node)) ===
+    return ((0, cdk_js_1.normalizeFQN)((0, module_js_1.getFullyQualifiedName)(context, node)) ===
         'aws_cdk_lib.aws_s3_deployment.BucketDeployment');
 }
 /**
@@ -71,11 +74,11 @@ function isS3BucketDeploymentConstructor(context, node) {
 function getBucketProperty(context, bucket, key) {
     const args = bucket.arguments;
     const optionsArg = args[2];
-    const options = (0, index_js_1.getValueOfExpression)(context, optionsArg, 'ObjectExpression');
+    const options = (0, ast_js_1.getValueOfExpression)(context, optionsArg, 'ObjectExpression');
     if (options == null) {
         return null;
     }
-    return options.properties.find(property => (0, index_js_1.isProperty)(property) && (0, index_js_1.isIdentifier)(property.key, key));
+    return options.properties.find(property => (0, ast_js_1.isProperty)(property) && (0, ast_js_1.isIdentifier)(property.key, key));
 }
 /**
  * Finds the propagated setting of a sensitive property
@@ -83,7 +86,7 @@ function getBucketProperty(context, bucket, key) {
 function findPropagatedSetting(sensitiveProperty, propagatedValue) {
     const isPropagatedProperty = sensitiveProperty.value !== propagatedValue;
     if (isPropagatedProperty) {
-        return (0, index_js_1.toSecondaryLocation)((0, index_js_1.getNodeParent)(propagatedValue), 'Propagated setting.');
+        return (0, location_js_1.toSecondaryLocation)((0, ancestor_js_1.getNodeParent)(propagatedValue), 'Propagated setting.');
     }
     return undefined;
 }