eslint-plugin-sonarjs 4.0.1 → 4.0.2

package/cjs/S5958/rule.js

@@ -51,16 +51,19 @@ 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 reaching_definitions_js_1 = require("../helpers/reaching-definitions.js");
+const ast_js_1 = require("../helpers/ast.js");
+const mocha_js_1 = require("../helpers/mocha.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) {
         let catchWithDone = false;
         function isInsideTest(node) {
             return context.sourceCode
                 .getAncestors(node)
-                .some(n => n.type === 'CallExpression' && index_js_1.Mocha.isTestConstruct(n));
+                .some(n => n.type === 'CallExpression' && (0, mocha_js_1.isTestConstruct)(n));
         }
         return {
             'CatchClause CallExpression[callee.name="done"]': (_node) => {
@@ -73,7 +76,7 @@ exports.rule = {
                 catchWithDone = false;
                 const { param } = node;
                 if (param?.type === 'Identifier') {
-                    const exception = (0, index_js_1.getVariableFromIdentifier)(param, context.sourceCode.getScope(node));
+                    const exception = (0, reaching_definitions_js_1.getVariableFromIdentifier)(param, context.sourceCode.getScope(node));
                     if (exception?.references.length === 0) {
                         context.report({
                             node: param,
@@ -87,7 +90,7 @@ exports.rule = {
                 if (isInsideTest(node) &&
                     isThrowAssertWithoutNot(callExpr) &&
                     (callExpr.arguments.length === 0 ||
-                        (callExpr.arguments.length === 1 && (0, index_js_1.isIdentifier)(callExpr.arguments[0], 'Error')))) {
+                        (callExpr.arguments.length === 1 && (0, ast_js_1.isIdentifier)(callExpr.arguments[0], 'Error')))) {
                     context.report({
                         node: callExpr.callee.property,
                         message: 'Assert more concrete exception type or assert the message of exception.',
@@ -103,17 +106,17 @@ function isThrowAssertWithoutNot(node) {
         return false;
     }
     let { object, property } = node.callee;
-    if (!(0, index_js_1.isIdentifier)(property, 'throw')) {
+    if (!(0, ast_js_1.isIdentifier)(property, 'throw')) {
         return false;
     }
     while (object.type === 'MemberExpression') {
-        if ((0, index_js_1.isIdentifier)(object.property, 'not')) {
+        if ((0, ast_js_1.isIdentifier)(object.property, 'not')) {
             return false;
         }
-        if ((0, index_js_1.isIdentifier)(object.property, 'should')) {
+        if ((0, ast_js_1.isIdentifier)(object.property, 'should')) {
             return true;
         }
         object = object.object;
     }
-    return object.type === 'CallExpression' && (0, index_js_1.isIdentifier)(object.callee, 'expect');
+    return object.type === 'CallExpression' && (0, ast_js_1.isIdentifier)(object.callee, 'expect');
 }

package/cjs/S5973/rule.js

@@ -51,11 +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 module_js_1 = require("../helpers/module.js");
+const ast_js_1 = require("../helpers/ast.js");
+const mocha_js_1 = require("../helpers/mocha.js");
 const meta = __importStar(require("./generated-meta.js"));
 const dependencies_js_1 = require("../helpers/package-jsons/dependencies.js");
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta, {
+    meta: (0, generate_meta_js_1.generateMeta)(meta, {
         messages: {
             stable: 'Make your tests stable so that they pass on the first try, or remove the flaky ones.',
         },
@@ -73,7 +76,7 @@ exports.rule = {
                     report(context, node);
                     return;
                 }
-                if (index_js_1.Mocha.isDescribeCase(node)) {
+                if ((0, mocha_js_1.isDescribeCase)(node)) {
                     describes.push(node);
                     return;
                 }
@@ -82,7 +85,7 @@ exports.rule = {
                 }
             },
             'CallExpression:exit': (node) => {
-                if (index_js_1.Mocha.isDescribeCase(node)) {
+                if ((0, mocha_js_1.isDescribeCase)(node)) {
                     describes.pop();
                 }
             },
@@ -93,9 +96,9 @@ exports.rule = {
     },
 };
 function hasJestRetry(context, node, hasJest) {
-    const callExpressionName = (0, index_js_1.getFullyQualifiedName)(context, node);
+    const callExpressionName = (0, module_js_1.getFullyQualifiedName)(context, node);
     return (callExpressionName === 'jest.retryTimes' ||
-        (hasJest && (0, index_js_1.isMethodInvocation)(node, 'jest', 'retryTimes', 1)));
+        (hasJest && (0, ast_js_1.isMethodInvocation)(node, 'jest', 'retryTimes', 1)));
 }
 function hasJestDependency(context) {
     const dependencies = (0, dependencies_js_1.getDependenciesSanitizePaths)(context);
@@ -108,7 +111,7 @@ function checkMochaRetries(context, node) {
     const callee = node.callee;
     if (callee.type === 'MemberExpression' &&
         callee.object.type === 'ThisExpression' &&
-        (0, index_js_1.isIdentifier)(callee.property, 'retries')) {
+        (0, ast_js_1.isIdentifier)(callee.property, 'retries')) {
         report(context, node);
     }
 }

package/cjs/S6019/rule.js

@@ -51,7 +51,8 @@ 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 collection_js_1 = require("../helpers/collection.js");
 const meta = __importStar(require("./generated-meta.js"));
 const rule_template_js_1 = require("../helpers/regex/rule-template.js");
 exports.rule = (0, rule_template_js_1.createRegExpRule)(context => {
@@ -62,7 +63,7 @@ exports.rule = (0, rule_template_js_1.createRegExpRule)(context => {
             }
         },
     };
-}, (0, index_js_1.generateMeta)(meta));
+}, (0, generate_meta_js_1.generateMeta)(meta));
 function report(quantifier, context) {
     const ending = quantifier.min === 1 ? '' : 's';
     const message = `Fix this reluctant quantifier that will only ever match ${quantifier.min} repetition${ending}.`;
@@ -76,7 +77,7 @@ function checkElements(elements, context) {
     if (elements.length === 0) {
         return;
     }
-    const lastElement = (0, index_js_1.last)(elements);
+    const lastElement = (0, collection_js_1.last)(elements);
     if (lastElement.type === 'Quantifier' && !lastElement.greedy) {
         report(lastElement, context);
         return;

package/cjs/S6035/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 rule_template_js_1 = require("../helpers/regex/rule-template.js");
 exports.rule = (0, rule_template_js_1.createRegExpRule)(context => {
@@ -78,4 +78,4 @@ exports.rule = (0, rule_template_js_1.createRegExpRule)(context => {
             }
         },
     };
-}, (0, index_js_1.generateMeta)(meta));
+}, (0, generate_meta_js_1.generateMeta)(meta));

package/cjs/S6079/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 reaching_definitions_js_1 = require("../helpers/reaching-definitions.js");
+const mocha_js_1 = require("../helpers/mocha.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) {
         let currentDoneVariable;
         let doneCall;
@@ -63,7 +66,7 @@ exports.rule = {
         let currentCase;
         const segmentFirstStatement = new Map();
         function checkForTestCase(node) {
-            const testCase = index_js_1.Mocha.extractTestCase(node);
+            const testCase = (0, mocha_js_1.extractTestCase)(node);
             if (!testCase) {
                 return;
             }
@@ -82,7 +85,7 @@ exports.rule = {
             if (!callbackScope) {
                 return;
             }
-            currentDoneVariable = (0, index_js_1.getVariableFromIdentifier)(done, callbackScope);
+            currentDoneVariable = (0, reaching_definitions_js_1.getVariableFromIdentifier)(done, callbackScope);
         }
         function checkForDoneCall(node) {
             const { callee } = node;
@@ -92,10 +95,10 @@ exports.rule = {
             }
         }
         function report(statementAfterDone) {
-            (0, index_js_1.report)(context, {
+            (0, location_js_1.report)(context, {
                 node: statementAfterDone,
                 message: `Move this code before the call to "done".`,
-            }, [(0, index_js_1.toSecondaryLocation)(doneCall)]);
+            }, [(0, location_js_1.toSecondaryLocation)(doneCall)]);
             doneSegment = undefined;
             doneCall = undefined;
             currentDoneVariable = undefined;

package/cjs/S6080/rule.js

@@ -51,20 +51,23 @@ 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 chai_js_1 = require("../helpers/chai.js");
+const generate_meta_js_1 = require("../helpers/generate-meta.js");
+const ast_js_1 = require("../helpers/ast.js");
+const mocha_js_1 = require("../helpers/mocha.js");
 const meta = __importStar(require("./generated-meta.js"));
 const MESSAGE = 'Set this timeout to 0 if you want to disable it, otherwise use a value lower than 2147483648.';
 const MAX_DELAY_VALUE = 2_147_483_647;
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta),
+    meta: (0, generate_meta_js_1.generateMeta)(meta),
     create(context) {
-        if (!index_js_1.Chai.isImported(context)) {
+        if (!(0, chai_js_1.isImported)(context)) {
             return {};
         }
         const constructs = [];
         return {
             CallExpression: (node) => {
-                if (index_js_1.Mocha.isTestConstruct(node)) {
+                if ((0, mocha_js_1.isTestConstruct)(node)) {
                     constructs.push(node);
                     return;
                 }
@@ -73,7 +76,7 @@ exports.rule = {
                 }
             },
             'CallExpression:exit': (node) => {
-                if (index_js_1.Mocha.isTestConstruct(node)) {
+                if ((0, mocha_js_1.isTestConstruct)(node)) {
                     constructs.pop();
                 }
             },
@@ -81,10 +84,10 @@ exports.rule = {
     },
 };
 function checkTimeoutDisabling(node, context) {
-    if ((0, index_js_1.isMethodCall)(node) && node.arguments.length > 0) {
+    if ((0, ast_js_1.isMethodCall)(node) && node.arguments.length > 0) {
         const { callee: { object, property }, arguments: [value], } = node;
-        if ((0, index_js_1.isThisExpression)(object) &&
-            (0, index_js_1.isIdentifier)(property, 'timeout') &&
+        if ((0, ast_js_1.isThisExpression)(object) &&
+            (0, ast_js_1.isIdentifier)(property, 'timeout') &&
             isDisablingTimeout(value, context)) {
             context.report({
                 message: MESSAGE,
@@ -94,6 +97,6 @@ function checkTimeoutDisabling(node, context) {
     }
 }
 function isDisablingTimeout(timeout, context) {
-    const usage = (0, index_js_1.getUniqueWriteUsageOrNode)(context, timeout);
-    return (0, index_js_1.isNumberLiteral)(usage) && usage.value > MAX_DELAY_VALUE;
+    const usage = (0, ast_js_1.getUniqueWriteUsageOrNode)(context, timeout);
+    return (0, ast_js_1.isNumberLiteral)(usage) && usage.value > MAX_DELAY_VALUE;
 }

package/cjs/S6092/rule.js

@@ -51,20 +51,22 @@ 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 chai_js_1 = require("../helpers/chai.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 = 'Refactor this uncertain assertion; it can succeed for multiple reasons.';
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta),
+    meta: (0, generate_meta_js_1.generateMeta)(meta),
     create(context) {
-        if (!index_js_1.Chai.isImported(context)) {
+        if (!(0, chai_js_1.isImported)(context)) {
             return {};
         }
         return {
             ExpressionStatement: (node) => {
                 const elements = retrieveAssertionChainElements(node.expression);
                 if (elements.length > 1 &&
-                    ((0, index_js_1.isIdentifier)(elements[0].identifier, 'expect') ||
+                    ((0, ast_js_1.isIdentifier)(elements[0].identifier, 'expect') ||
                         getElementIndex(elements, 'should') >= 0)) {
                     checkNotThrow(context, elements);
                     checkNotInclude(context, elements);
@@ -135,7 +137,7 @@ function retrieveAssertionChainElements(node) {
     const result = [];
     let currentArguments = undefined;
     while (true) {
-        if ((0, index_js_1.isDotNotation)(currentNode)) {
+        if ((0, ast_js_1.isDotNotation)(currentNode)) {
             result.push({ identifier: currentNode.property, arguments: currentArguments });
             currentNode = currentNode.object;
             currentArguments = undefined;
@@ -144,7 +146,7 @@ function retrieveAssertionChainElements(node) {
             currentArguments = currentNode.arguments;
             currentNode = currentNode.callee;
         }
-        else if ((0, index_js_1.isIdentifier)(currentNode)) {
+        else if ((0, ast_js_1.isIdentifier)(currentNode)) {
             result.push({ identifier: currentNode, arguments: currentArguments });
             break;
         }
@@ -155,7 +157,7 @@ function retrieveAssertionChainElements(node) {
     return result.reverse();
 }
 function getElementIndex(elements, name) {
-    return elements.findIndex(element => (0, index_js_1.isIdentifier)(element.identifier, name));
+    return elements.findIndex(element => (0, ast_js_1.isIdentifier)(element.identifier, name));
 }
 function locFromTwoNodes(start, end) {
     return {

package/cjs/S6245/rule.js

@@ -51,32 +51,37 @@ 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 s3_js_1 = require("../helpers/aws/s3.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 cdk_js_1 = require("../helpers/aws/cdk.js");
+const location_js_1 = require("../helpers/location.js");
 const meta = __importStar(require("./generated-meta.js"));
 const ENCRYPTED_KEY = 'encryption';
 const messages = {
     unencrypted: 'Objects in the bucket are not encrypted. Make sure it is safe here.',
     omitted: 'Omitting "encryption" disables server-side encryption. Make sure it is safe here.',
 };
-exports.rule = (0, index_js_1.S3BucketTemplate)((bucket, context) => {
-    const encryptedProperty = (0, index_js_1.getBucketProperty)(context, bucket, ENCRYPTED_KEY);
+exports.rule = (0, s3_js_1.S3BucketTemplate)((bucket, context) => {
+    const encryptedProperty = (0, s3_js_1.getBucketProperty)(context, bucket, ENCRYPTED_KEY);
     if (encryptedProperty == null) {
-        (0, index_js_1.report)(context, {
+        (0, location_js_1.report)(context, {
             message: messages['omitted'],
             node: bucket.callee,
         });
         return;
     }
-    const encryptedValue = (0, index_js_1.getValueOfExpression)(context, encryptedProperty.value, 'MemberExpression');
+    const encryptedValue = (0, ast_js_1.getValueOfExpression)(context, encryptedProperty.value, 'MemberExpression');
     if (encryptedValue && isUnencrypted(encryptedValue)) {
-        const propagated = (0, index_js_1.findPropagatedSetting)(encryptedProperty, encryptedValue);
-        (0, index_js_1.report)(context, {
+        const propagated = (0, s3_js_1.findPropagatedSetting)(encryptedProperty, encryptedValue);
+        (0, location_js_1.report)(context, {
             message: messages['unencrypted'],
             node: encryptedProperty,
         }, propagated ? [propagated] : []);
     }
     function isUnencrypted(encrypted) {
-        return ((0, index_js_1.normalizeFQN)((0, index_js_1.getFullyQualifiedName)(context, encrypted)) ===
+        return ((0, cdk_js_1.normalizeFQN)((0, module_js_1.getFullyQualifiedName)(context, encrypted)) ===
             'aws_cdk_lib.aws_s3.BucketEncryption.UNENCRYPTED');
     }
-}, (0, index_js_1.generateMeta)(meta));
+}, (0, generate_meta_js_1.generateMeta)(meta));

package/cjs/S6249/rule.js

@@ -51,15 +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 generate_meta_js_1 = require("../helpers/generate-meta.js");
+const s3_js_1 = require("../helpers/aws/s3.js");
+const ast_js_1 = require("../helpers/ast.js");
 const meta = __importStar(require("./generated-meta.js"));
 const ENFORCE_SSL_KEY = 'enforceSSL';
 const messages = {
     authorized: 'Make sure authorizing HTTP requests is safe here.',
     omitted: "Omitting 'enforceSSL' authorizes HTTP requests. Make sure it is safe here.",
 };
-exports.rule = (0, index_js_1.S3BucketTemplate)((bucket, context) => {
-    const enforceSSLProperty = (0, index_js_1.getBucketProperty)(context, bucket, ENFORCE_SSL_KEY);
+exports.rule = (0, s3_js_1.S3BucketTemplate)((bucket, context) => {
+    const enforceSSLProperty = (0, s3_js_1.getBucketProperty)(context, bucket, ENFORCE_SSL_KEY);
     if (enforceSSLProperty == null) {
         context.report({
             message: messages['omitted'],
@@ -67,11 +69,11 @@ exports.rule = (0, index_js_1.S3BucketTemplate)((bucket, context) => {
         });
         return;
     }
-    const enforceSSLValue = (0, index_js_1.getValueOfExpression)(context, enforceSSLProperty.value, 'Literal');
+    const enforceSSLValue = (0, ast_js_1.getValueOfExpression)(context, enforceSSLProperty.value, 'Literal');
     if (enforceSSLValue?.value === false) {
         context.report({
             message: messages['authorized'],
             node: enforceSSLProperty,
         });
     }
-}, (0, index_js_1.generateMeta)(meta));
+}, (0, generate_meta_js_1.generateMeta)(meta));

package/cjs/S6252/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 location_js_1 = require("../helpers/location.js");
+const generate_meta_js_1 = require("../helpers/generate-meta.js");
+const s3_js_1 = require("../helpers/aws/s3.js");
+const ancestor_js_1 = require("../helpers/ancestor.js");
+const ast_js_1 = require("../helpers/ast.js");
 const meta = __importStar(require("./generated-meta.js"));
 const VERSIONED_KEY = 'versioned';
 const messages = {
@@ -59,25 +63,25 @@ const messages = {
     omitted: 'Omitting the "versioned" argument disables S3 bucket versioning. Make sure it is safe here.',
     secondary: 'Propagated setting',
 };
-exports.rule = (0, index_js_1.S3BucketTemplate)((bucketConstructor, context) => {
-    const versionedProperty = (0, index_js_1.getBucketProperty)(context, bucketConstructor, VERSIONED_KEY);
+exports.rule = (0, s3_js_1.S3BucketTemplate)((bucketConstructor, context) => {
+    const versionedProperty = (0, s3_js_1.getBucketProperty)(context, bucketConstructor, VERSIONED_KEY);
     if (versionedProperty == null) {
-        (0, index_js_1.report)(context, {
+        (0, location_js_1.report)(context, {
             message: messages.omitted,
             node: bucketConstructor.callee,
         });
         return;
     }
-    const propertyLiteralValue = (0, index_js_1.getValueOfExpression)(context, versionedProperty.value, 'Literal');
+    const propertyLiteralValue = (0, ast_js_1.getValueOfExpression)(context, versionedProperty.value, 'Literal');
     if (propertyLiteralValue?.value === false) {
         const secondaries = [];
         const isPropagatedProperty = versionedProperty.value !== propertyLiteralValue;
         if (isPropagatedProperty) {
-            secondaries.push((0, index_js_1.toSecondaryLocation)((0, index_js_1.getNodeParent)(propertyLiteralValue), messages.secondary));
+            secondaries.push((0, location_js_1.toSecondaryLocation)((0, ancestor_js_1.getNodeParent)(propertyLiteralValue), messages.secondary));
         }
-        (0, index_js_1.report)(context, {
+        (0, location_js_1.report)(context, {
             message: messages.unversioned,
             node: versionedProperty,
         }, secondaries);
     }
-}, (0, index_js_1.generateMeta)(meta));
+}, (0, generate_meta_js_1.generateMeta)(meta));

package/cjs/S6265/rule.js

@@ -51,7 +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 s3_js_1 = require("../helpers/aws/s3.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 collection_js_1 = require("../helpers/collection.js");
+const merger_js_1 = require("../helpers/decorators/merger.js");
+const cdk_js_1 = require("../helpers/aws/cdk.js");
+const location_js_1 = require("../helpers/location.js");
 const meta = __importStar(require("./generated-meta.js"));
 const messages = {
     accessLevel: (param) => `Make sure granting ${param} access is safe here.`,
@@ -62,12 +69,12 @@ const INVALID_ACCESS_CONTROL_VALUES = ['PUBLIC_READ', 'PUBLIC_READ_WRITE', 'AUTH
 const PUBLIC_READ_ACCESS_KEY = 'publicReadAccess';
 const INVALID_PUBLIC_READ_ACCESS_VALUE = true;
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta),
+    meta: (0, generate_meta_js_1.generateMeta)(meta),
     create(context) {
-        return (0, index_js_1.mergeRules)(s3BucketConstructorRule.create(context), s3BucketDeploymentConstructorRule.create(context), handleGrantPublicAccess.create(context));
+        return (0, merger_js_1.mergeRules)(s3BucketConstructorRule.create(context), s3BucketDeploymentConstructorRule.create(context), handleGrantPublicAccess.create(context));
     },
 };
-const s3BucketConstructorRule = (0, index_js_1.S3BucketTemplate)((bucketConstructor, context) => {
+const s3BucketConstructorRule = (0, s3_js_1.S3BucketTemplate)((bucketConstructor, context) => {
     for (const value of INVALID_ACCESS_CONTROL_VALUES) {
         checkConstantParam(context, bucketConstructor, ACCESS_CONTROL_KEY, [
             'BucketAccessControl',
@@ -80,7 +87,7 @@ const s3BucketDeploymentConstructorRule = {
     create(context) {
         return {
             NewExpression: (node) => {
-                if ((0, index_js_1.isS3BucketDeploymentConstructor)(context, node)) {
+                if ((0, s3_js_1.isS3BucketDeploymentConstructor)(context, node)) {
                     for (const value of INVALID_ACCESS_CONTROL_VALUES) {
                         checkConstantParam(context, node, ACCESS_CONTROL_KEY, ['BucketAccessControl', value]);
                     }
@@ -90,31 +97,31 @@ const s3BucketDeploymentConstructorRule = {
     },
 };
 function checkBooleanParam(context, bucketConstructor, propName, propValue) {
-    const property = (0, index_js_1.getBucketProperty)(context, bucketConstructor, propName);
+    const property = (0, s3_js_1.getBucketProperty)(context, bucketConstructor, propName);
     if (property == null) {
         return;
     }
-    const propertyLiteralValue = (0, index_js_1.getValueOfExpression)(context, property.value, 'Literal');
+    const propertyLiteralValue = (0, ast_js_1.getValueOfExpression)(context, property.value, 'Literal');
     if (propertyLiteralValue?.value === propValue) {
-        const secondary = (0, index_js_1.findPropagatedSetting)(property, propertyLiteralValue);
-        (0, index_js_1.report)(context, {
+        const secondary = (0, s3_js_1.findPropagatedSetting)(property, propertyLiteralValue);
+        (0, location_js_1.report)(context, {
             message: messages.unrestricted,
             node: property,
         }, secondary ? [secondary] : []);
     }
 }
 function checkConstantParam(context, bucketConstructor, propName, paramQualifiers) {
-    const property = (0, index_js_1.getBucketProperty)(context, bucketConstructor, propName);
+    const property = (0, s3_js_1.getBucketProperty)(context, bucketConstructor, propName);
     if (property == null) {
         return;
     }
-    const propertyLiteralValue = (0, index_js_1.getValueOfExpression)(context, property.value, 'MemberExpression');
+    const propertyLiteralValue = (0, ast_js_1.getValueOfExpression)(context, property.value, 'MemberExpression');
     if (propertyLiteralValue !== undefined &&
-        (0, index_js_1.normalizeFQN)((0, index_js_1.getFullyQualifiedName)(context, propertyLiteralValue)) ===
+        (0, cdk_js_1.normalizeFQN)((0, module_js_1.getFullyQualifiedName)(context, propertyLiteralValue)) ===
             `aws_cdk_lib.aws_s3.${paramQualifiers.join('.')}`) {
-        const secondary = (0, index_js_1.findPropagatedSetting)(property, propertyLiteralValue);
-        (0, index_js_1.report)(context, {
-            message: messages.accessLevel((0, index_js_1.last)(paramQualifiers)),
+        const secondary = (0, s3_js_1.findPropagatedSetting)(property, propertyLiteralValue);
+        (0, location_js_1.report)(context, {
+            message: messages.accessLevel((0, collection_js_1.last)(paramQualifiers)),
             node: property,
         }, secondary ? [secondary] : []);
     }
@@ -123,21 +130,21 @@ const handleGrantPublicAccess = {
     create(context) {
         return {
             CallExpression: (node) => {
-                if (!(0, index_js_1.isMethodCall)(node)) {
+                if (!(0, ast_js_1.isMethodCall)(node)) {
                     return;
                 }
                 const { object, property } = node.callee;
-                const isGrantPublicAccessMethodCall = (0, index_js_1.isIdentifier)(property, 'grantPublicAccess');
+                const isGrantPublicAccessMethodCall = (0, ast_js_1.isIdentifier)(property, 'grantPublicAccess');
                 if (!isGrantPublicAccessMethodCall) {
                     return;
                 }
-                const variableAssignment = (0, index_js_1.getUniqueWriteUsageOrNode)(context, object);
+                const variableAssignment = (0, ast_js_1.getUniqueWriteUsageOrNode)(context, object);
                 const isS3bucketInstance = variableAssignment.type === 'NewExpression' &&
-                    (0, index_js_1.isS3BucketConstructor)(context, variableAssignment);
+                    (0, s3_js_1.isS3BucketConstructor)(context, variableAssignment);
                 if (!isS3bucketInstance) {
                     return;
                 }
-                (0, index_js_1.report)(context, {
+                (0, location_js_1.report)(context, {
                     message: messages.unrestricted,
                     node: property,
                 });

package/cjs/S6268/rule.js

@@ -51,7 +51,8 @@ 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 bypassMethods = [
     'bypassSecurityTrustHtml',
@@ -61,7 +62,7 @@ const bypassMethods = [
     'bypassSecurityTrustResourceUrl',
 ];
 exports.rule = {
-    meta: (0, index_js_1.generateMeta)(meta, {
+    meta: (0, generate_meta_js_1.generateMeta)(meta, {
         messages: {
             checkAngularBypass: 'Make sure disabling Angular built-in sanitization is safe here.',
         },
@@ -70,7 +71,7 @@ exports.rule = {
         return {
             CallExpression: (node) => {
                 const { callee, arguments: args } = node;
-                if ((0, index_js_1.isMemberWithProperty)(callee, ...bypassMethods) &&
+                if ((0, ast_js_1.isMemberWithProperty)(callee, ...bypassMethods) &&
                     args.length === 1 &&
                     !isHardcodedLiteral(args[0])) {
                     context.report({
@@ -87,6 +88,6 @@ function isHardcodedLiteral(node) {
         return node.expressions.length === 0;
     }
     else {
-        return (0, index_js_1.isLiteral)(node);
+        return (0, ast_js_1.isLiteral)(node);
     }
 }