eslint-cdk-plugin 0.4.0 → 1.0.0

package/dist/no-mutable-public-fields.mjs

@@ -1,12 +1,6 @@
 import { AST_NODE_TYPES, ESLintUtils } from "@typescript-eslint/utils";
-import { isConstructOrStackType } from "./utils/isConstructOrStackType.mjs";
-/**
- * Disallow mutable public class fields
- * @param context - The rule context provided by ESLint
- * @returns An object containing the AST visitor functions
- * @see {@link https://eslint-cdk-plugin.dev/rules/no-mutable-public-fields} - Documentation
- */
-export var noMutablePublicFields = ESLintUtils.RuleCreator.withoutDocs({
+import { isConstructOrStackType } from "./utils/typeCheck.mjs";
+export const noMutablePublicFields = ESLintUtils.RuleCreator.withoutDocs({
     meta: {
         type: "problem",
         docs: {
@@ -19,52 +13,45 @@ export var noMutablePublicFields = ESLintUtils.RuleCreator.withoutDocs({
         schema: [],
     },
     defaultOptions: [],
-    create: function (context) {
-        var parserServices = ESLintUtils.getParserServices(context);
-        var checker = parserServices.program.getTypeChecker();
-        var sourceCode = context.sourceCode;
+    create(context) {
+        const parserServices = ESLintUtils.getParserServices(context);
+        const typeChecker = parserServices.program.getTypeChecker();
+        const sourceCode = context.sourceCode;
         return {
-            ClassDeclaration: function (node) {
-                var _a;
-                var type = checker.getTypeAtLocation(parserServices.esTreeNodeToTSNodeMap.get(node));
+            ClassDeclaration(node) {
+                const type = typeChecker.getTypeAtLocation(parserServices.esTreeNodeToTSNodeMap.get(node));
                 if (!isConstructOrStackType(type)) {
                     return;
                 }
-                var _loop_1 = function (member) {
-                    // NOTE: check property definition
+                for (const member of node.body.body) {
                     if (member.type !== AST_NODE_TYPES.PropertyDefinition ||
                         member.key.type !== AST_NODE_TYPES.Identifier) {
-                        return "continue";
+                        continue;
                     }
-                    // NOTE: Skip private and protected fields
-                    if (["private", "protected"].includes((_a = member.accessibility) !== null && _a !== void 0 ? _a : "")) {
-                        return "continue";
+                    if (["private", "protected"].includes(member.accessibility ?? "")) {
+                        continue;
                     }
-                    // NOTE: Skip if readonly is present
                     if (member.readonly)
-                        return "continue";
+                        continue;
                     context.report({
                         node: member,
                         messageId: "noMutablePublicFields",
                         data: {
                             propertyName: member.key.name,
                         },
-                        fix: function (fixer) {
-                            var accessibility = member.accessibility ? "public " : "";
-                            var paramText = sourceCode.getText(member);
-                            var _a = paramText.split(":"), key = _a[0], value = _a[1];
-                            var replacedKey = key.startsWith("public ")
+                        fix: (fixer) => {
+                            const accessibility = member.accessibility ? "public " : "";
+                            const paramText = sourceCode.getText(member);
+                            const [key, value] = paramText.split(":");
+                            const replacedKey = key.startsWith("public ")
                                 ? key.replace("public ", "")
                                 : key;
-                            return fixer.replaceText(member, "".concat(accessibility, "readonly ").concat(replacedKey, ":").concat(value));
+                            return fixer.replaceText(member, `${accessibility}readonly ${replacedKey}:${value}`);
                         },
                     });
-                };
-                for (var _i = 0, _b = node.body.body; _i < _b.length; _i++) {
-                    var member = _b[_i];
-                    _loop_1(member);
                 }
             },
         };
     },
 });
+//# sourceMappingURL=no-mutable-public-fields.mjs.map

package/dist/no-mutable-public-fields.mjs.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-mutable-public-fields.mjs","sourceRoot":"","sources":["../src/no-mutable-public-fields.mts"],"names":[],"mappings":"AAAA,OAAO,EAAE,cAAc,EAAE,WAAW,EAAE,MAAM,0BAA0B,CAAC;AAEvE,OAAO,EAAE,sBAAsB,EAAE,MAAM,uBAAuB,CAAC;AAQ/D,MAAM,CAAC,MAAM,qBAAqB,GAAG,WAAW,CAAC,WAAW,CAAC,WAAW,CAAC;IACvE,IAAI,EAAE;QACJ,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACJ,WAAW,EAAE,sCAAsC;SACpD;QACD,OAAO,EAAE,MAAM;QACf,QAAQ,EAAE;YACR,qBAAqB,EACnB,gGAAgG;SACnG;QACD,MAAM,EAAE,EAAE;KACX;IACD,cAAc,EAAE,EAAE;IAClB,MAAM,CAAC,OAAO;QACZ,MAAM,cAAc,GAAG,WAAW,CAAC,iBAAiB,CAAC,OAAO,CAAC,CAAC;QAC9D,MAAM,WAAW,GAAG,cAAc,CAAC,OAAO,CAAC,cAAc,EAAE,CAAC;QAC5D,MAAM,UAAU,GAAG,OAAO,CAAC,UAAU,CAAC;QAEtC,OAAO;YACL,gBAAgB,CAAC,IAAI;gBACnB,MAAM,IAAI,GAAG,WAAW,CAAC,iBAAiB,CACxC,cAAc,CAAC,qBAAqB,CAAC,GAAG,CAAC,IAAI,CAAC,CAC/C,CAAC;gBACF,IAAI,CAAC,sBAAsB,CAAC,IAAI,CAAC,EAAE,CAAC;oBAClC,OAAO;gBACT,CAAC;gBAED,KAAK,MAAM,MAAM,IAAI,IAAI,CAAC,IAAI,CAAC,IAAI,EAAE,CAAC;oBAEpC,IACE,MAAM,CAAC,IAAI,KAAK,cAAc,CAAC,kBAAkB;wBACjD,MAAM,CAAC,GAAG,CAAC,IAAI,KAAK,cAAc,CAAC,UAAU,EAC7C,CAAC;wBACD,SAAS;oBACX,CAAC;oBAGD,IAAI,CAAC,SAAS,EAAE,WAAW,CAAC,CAAC,QAAQ,CAAC,MAAM,CAAC,aAAa,IAAI,EAAE,CAAC,EAAE,CAAC;wBAClE,SAAS;oBACX,CAAC;oBAGD,IAAI,MAAM,CAAC,QAAQ;wBAAE,SAAS;oBAE9B,OAAO,CAAC,MAAM,CAAC;wBACb,IAAI,EAAE,MAAM;wBACZ,SAAS,EAAE,uBAAuB;wBAClC,IAAI,EAAE;4BACJ,YAAY,EAAE,MAAM,CAAC,GAAG,CAAC,IAAI;yBAC9B;wBACD,GAAG,EAAE,CAAC,KAAK,EAAE,EAAE;4BACb,MAAM,aAAa,GAAG,MAAM,CAAC,aAAa,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,EAAE,CAAC;4BAC5D,MAAM,SAAS,GAAG,UAAU,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC;4BAC7C,MAAM,CAAC,GAAG,EAAE,KAAK,CAAC,GAAG,SAAS,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;4BAC1C,MAAM,WAAW,GAAG,GAAG,CAAC,UAAU,CAAC,SAAS,CAAC;gCAC3C,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,SAAS,EAAE,EAAE,CAAC;gCAC5B,CAAC,CAAC,GAAG,CAAC;4BAER,OAAO,KAAK,CAAC,WAAW,CACtB,MAAM,EACN,GAAG,aAAa,YAAY,WAAW,IAAI,KAAK,EAAE,CACnD,CAAC;wBACJ,CAAC;qBACF,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;SACF,CAAC;IACJ,CAAC;CACF,CAAC,CAAC"}

package/dist/no-parent-name-construct-id-match.d.mts

@@ -0,0 +1,2 @@
+import { ESLintUtils } from "@typescript-eslint/utils";
+export declare const noParentNameConstructIdMatch: ESLintUtils.RuleModule<"noParentNameConstructIdMatch", [], unknown, ESLintUtils.RuleListener>;

package/dist/no-parent-name-construct-id-match.mjs

@@ -1,12 +1,6 @@
 import { AST_NODE_TYPES, ESLintUtils, } from "@typescript-eslint/utils";
 import { toPascalCase } from "./utils/convertString.mjs";
-/**
- * Enforce that construct IDs does not match the parent construct name.
- * @param context - The rule context provided by ESLint
- * @returns An object containing the AST visitor functions
- * @see {@link https://eslint-cdk-plugin.dev/rules/no-parent-name-construct-id-match} - Documentation
- */
-export var noParentNameConstructIdMatch = ESLintUtils.RuleCreator.withoutDocs({
+export const noParentNameConstructIdMatch = ESLintUtils.RuleCreator.withoutDocs({
     meta: {
         type: "problem",
         docs: {
@@ -18,87 +12,75 @@ export var noParentNameConstructIdMatch = ESLintUtils.RuleCreator.withoutDocs({
         schema: [],
     },
     defaultOptions: [],
-    create: function (context) {
+    create(context) {
         return {
-            ClassBody: function (node) {
-                var _a;
-                var parent = node.parent;
-                if ((parent === null || parent === void 0 ? void 0 : parent.type) !== AST_NODE_TYPES.ClassDeclaration)
+            ClassBody(node) {
+                const parent = node.parent;
+                if (parent?.type !== AST_NODE_TYPES.ClassDeclaration)
                     return;
-                var parentClassName = (_a = parent.id) === null || _a === void 0 ? void 0 : _a.name;
+                const parentClassName = parent.id?.name;
                 if (!parentClassName)
                     return;
-                for (var _i = 0, _b = node.body; _i < _b.length; _i++) {
-                    var body = _b[_i];
-                    // NOTE: Ignore if neither method nor constructor.
+                for (const body of node.body) {
                     if (body.type !== AST_NODE_TYPES.MethodDefinition ||
                         !["method", "constructor"].includes(body.kind) ||
                         body.value.type !== AST_NODE_TYPES.FunctionExpression) {
                         continue;
                     }
                     validateConstructorBody({
-                        node: node,
+                        node,
                         expression: body.value,
-                        parentClassName: parentClassName,
-                        context: context,
+                        parentClassName,
+                        context,
                     });
                 }
             },
         };
     },
 });
-/**
- * Validate the constructor body for the parent class
- * - validate each statement in the constructor body
- */
-var validateConstructorBody = function (_a) {
-    var _b;
-    var node = _a.node, expression = _a.expression, parentClassName = _a.parentClassName, context = _a.context;
-    for (var _i = 0, _c = expression.body.body; _i < _c.length; _i++) {
-        var statement = _c[_i];
+const validateConstructorBody = ({ node, expression, parentClassName, context, }) => {
+    for (const statement of expression.body.body) {
         switch (statement.type) {
             case AST_NODE_TYPES.VariableDeclaration: {
-                var newExpression = statement.declarations[0].init;
-                if ((newExpression === null || newExpression === void 0 ? void 0 : newExpression.type) !== AST_NODE_TYPES.NewExpression)
+                const newExpression = statement.declarations[0].init;
+                if (newExpression?.type !== AST_NODE_TYPES.NewExpression)
                     continue;
                 validateConstructId({
-                    node: node,
-                    context: context,
+                    node,
+                    context,
                     expression: newExpression,
-                    parentClassName: parentClassName,
+                    parentClassName,
                 });
                 break;
             }
             case AST_NODE_TYPES.ExpressionStatement: {
-                if (((_b = statement.expression) === null || _b === void 0 ? void 0 : _b.type) !== AST_NODE_TYPES.NewExpression)
+                if (statement.expression?.type !== AST_NODE_TYPES.NewExpression)
                     break;
                 validateStatement({
-                    node: node,
-                    statement: statement,
-                    parentClassName: parentClassName,
-                    context: context,
+                    node,
+                    statement,
+                    parentClassName,
+                    context,
                 });
                 break;
             }
             case AST_NODE_TYPES.IfStatement: {
                 traverseStatements({
-                    node: node,
-                    context: context,
-                    parentClassName: parentClassName,
+                    node,
+                    context,
+                    parentClassName,
                     statement: statement.consequent,
                 });
                 break;
             }
             case AST_NODE_TYPES.SwitchStatement: {
-                for (var _d = 0, _e = statement.cases; _d < _e.length; _d++) {
-                    var switchCase = _e[_d];
-                    for (var _f = 0, _g = switchCase.consequent; _f < _g.length; _f++) {
-                        var statement_1 = _g[_f];
+                for (const switchCase of statement.cases) {
+                    for (const statement of switchCase.consequent) {
                         traverseStatements({
-                            node: node,
-                            context: context,
-                            parentClassName: parentClassName,
-                            statement: statement_1,
+                            node,
+                            context,
+                            parentClassName,
+                            statement,
                         });
                     }
                 }
@@ -107,154 +89,124 @@ var validateConstructorBody = function (_a) {
         }
     }
 };
-/**
- * Recursively traverse and validate statements in the AST
- * - Handles BlockStatement, ExpressionStatement, and VariableDeclaration
- * - Validates construct IDs against parent class name
- */
-var traverseStatements = function (_a) {
-    var node = _a.node, statement = _a.statement, parentClassName = _a.parentClassName, context = _a.context;
+const traverseStatements = ({ node, statement, parentClassName, context, }) => {
     switch (statement.type) {
         case AST_NODE_TYPES.BlockStatement: {
-            for (var _i = 0, _b = statement.body; _i < _b.length; _i++) {
-                var body = _b[_i];
+            for (const body of statement.body) {
                 validateStatement({
-                    node: node,
+                    node,
                     statement: body,
-                    parentClassName: parentClassName,
-                    context: context,
+                    parentClassName,
+                    context,
                 });
             }
             break;
         }
         case AST_NODE_TYPES.ExpressionStatement: {
-            var newExpression = statement.expression;
-            if ((newExpression === null || newExpression === void 0 ? void 0 : newExpression.type) !== AST_NODE_TYPES.NewExpression)
+            const newExpression = statement.expression;
+            if (newExpression?.type !== AST_NODE_TYPES.NewExpression)
                 break;
             validateStatement({
-                node: node,
-                statement: statement,
-                parentClassName: parentClassName,
-                context: context,
+                node,
+                statement,
+                parentClassName,
+                context,
             });
             break;
         }
         case AST_NODE_TYPES.VariableDeclaration: {
-            var newExpression = statement.declarations[0].init;
-            if ((newExpression === null || newExpression === void 0 ? void 0 : newExpression.type) !== AST_NODE_TYPES.NewExpression)
+            const newExpression = statement.declarations[0].init;
+            if (newExpression?.type !== AST_NODE_TYPES.NewExpression)
                 break;
             validateConstructId({
-                node: node,
-                context: context,
+                node,
+                context,
                 expression: newExpression,
-                parentClassName: parentClassName,
+                parentClassName,
             });
             break;
         }
     }
 };
-/**
- * Validate a single statement in the AST
- * - Handles different types of statements (Variable, Expression, If, Switch)
- * - Extracts and validates construct IDs from new expressions
- */
-var validateStatement = function (_a) {
-    var node = _a.node, statement = _a.statement, parentClassName = _a.parentClassName, context = _a.context;
+const validateStatement = ({ node, statement, parentClassName, context, }) => {
     switch (statement.type) {
         case AST_NODE_TYPES.VariableDeclaration: {
-            var newExpression = statement.declarations[0].init;
-            if ((newExpression === null || newExpression === void 0 ? void 0 : newExpression.type) !== AST_NODE_TYPES.NewExpression)
+            const newExpression = statement.declarations[0].init;
+            if (newExpression?.type !== AST_NODE_TYPES.NewExpression)
                 break;
             validateConstructId({
-                node: node,
-                context: context,
+                node,
+                context,
                 expression: newExpression,
-                parentClassName: parentClassName,
+                parentClassName,
             });
             break;
         }
         case AST_NODE_TYPES.ExpressionStatement: {
-            var newExpression = statement.expression;
-            if ((newExpression === null || newExpression === void 0 ? void 0 : newExpression.type) !== AST_NODE_TYPES.NewExpression)
+            const newExpression = statement.expression;
+            if (newExpression?.type !== AST_NODE_TYPES.NewExpression)
                 break;
             validateConstructId({
-                node: node,
-                context: context,
+                node,
+                context,
                 expression: newExpression,
-                parentClassName: parentClassName,
+                parentClassName,
             });
             break;
         }
         case AST_NODE_TYPES.IfStatement: {
             validateIfStatement({
-                node: node,
-                statement: statement,
-                parentClassName: parentClassName,
-                context: context,
+                node,
+                statement,
+                parentClassName,
+                context,
             });
             break;
         }
         case AST_NODE_TYPES.SwitchStatement: {
             validateSwitchStatement({
-                node: node,
-                statement: statement,
-                parentClassName: parentClassName,
-                context: context,
+                node,
+                statement,
+                parentClassName,
+                context,
             });
             break;
         }
     }
 };
-/**
- * Validate the `if` statement
- * - Validate recursively if `if` statements are nested
- */
-var validateIfStatement = function (_a) {
-    var node = _a.node, statement = _a.statement, parentClassName = _a.parentClassName, context = _a.context;
+const validateIfStatement = ({ node, statement, parentClassName, context, }) => {
     traverseStatements({
-        node: node,
-        context: context,
-        parentClassName: parentClassName,
+        node,
+        context,
+        parentClassName,
         statement: statement.consequent,
     });
 };
-/**
- * Validate the `switch` statement
- * - Validate recursively if `switch` statements are nested
- */
-var validateSwitchStatement = function (_a) {
-    var node = _a.node, statement = _a.statement, parentClassName = _a.parentClassName, context = _a.context;
-    for (var _i = 0, _b = statement.cases; _i < _b.length; _i++) {
-        var caseStatement = _b[_i];
-        for (var _c = 0, _d = caseStatement.consequent; _c < _d.length; _c++) {
-            var _consequent = _d[_c];
+const validateSwitchStatement = ({ node, statement, parentClassName, context, }) => {
+    for (const caseStatement of statement.cases) {
+        for (const _consequent of caseStatement.consequent) {
             traverseStatements({
-                node: node,
-                context: context,
-                parentClassName: parentClassName,
+                node,
+                context,
+                parentClassName,
                 statement: _consequent,
             });
         }
     }
 };
-/**
- * Validate that parent construct name and child id do not match
- */
-var validateConstructId = function (_a) {
-    var node = _a.node, context = _a.context, expression = _a.expression, parentClassName = _a.parentClassName;
+const validateConstructId = ({ node, context, expression, parentClassName, }) => {
     if (expression.arguments.length < 2)
         return;
-    // NOTE: Treat the second argument as ID
-    var secondArg = expression.arguments[1];
+    const secondArg = expression.arguments[1];
     if (secondArg.type !== AST_NODE_TYPES.Literal ||
         typeof secondArg.value !== "string") {
         return;
     }
-    var formattedConstructId = toPascalCase(secondArg.value);
-    var formattedParentClassName = toPascalCase(parentClassName);
+    const formattedConstructId = toPascalCase(secondArg.value);
+    const formattedParentClassName = toPascalCase(parentClassName);
     if (formattedParentClassName === formattedConstructId) {
         context.report({
-            node: node,
+            node,
             messageId: "noParentNameConstructIdMatch",
             data: {
                 constructId: secondArg.value,
@@ -263,3 +215,4 @@ var validateConstructId = function (_a) {
         });
     }
 };
+//# sourceMappingURL=no-parent-name-construct-id-match.mjs.map

package/dist/no-parent-name-construct-id-match.mjs.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-parent-name-construct-id-match.mjs","sourceRoot":"","sources":["../src/no-parent-name-construct-id-match.mts"],"names":[],"mappings":"AAAA,OAAO,EACL,cAAc,EACd,WAAW,GAGZ,MAAM,0BAA0B,CAAC;AAElC,OAAO,EAAE,YAAY,EAAE,MAAM,2BAA2B,CAAC;AAwBzD,MAAM,CAAC,MAAM,4BAA4B,GAAG,WAAW,CAAC,WAAW,CAAC,WAAW,CAC7E;IACE,IAAI,EAAE;QACJ,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACJ,WAAW,EACT,sEAAsE;SACzE;QACD,QAAQ,EAAE;YACR,4BAA4B,EAC1B,sIAAsI;SACzI;QACD,MAAM,EAAE,EAAE;KACX;IACD,cAAc,EAAE,EAAE;IAClB,MAAM,CAAC,OAAO;QACZ,OAAO;YACL,SAAS,CAAC,IAAI;gBACZ,MAAM,MAAM,GAAG,IAAI,CAAC,MAAM,CAAC;gBAC3B,IAAI,MAAM,EAAE,IAAI,KAAK,cAAc,CAAC,gBAAgB;oBAAE,OAAO;gBAE7D,MAAM,eAAe,GAAG,MAAM,CAAC,EAAE,EAAE,IAAI,CAAC;gBACxC,IAAI,CAAC,eAAe;oBAAE,OAAO;gBAE7B,KAAK,MAAM,IAAI,IAAI,IAAI,CAAC,IAAI,EAAE,CAAC;oBAE7B,IACE,IAAI,CAAC,IAAI,KAAK,cAAc,CAAC,gBAAgB;wBAC7C,CAAC,CAAC,QAAQ,EAAE,aAAa,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC;wBAC9C,IAAI,CAAC,KAAK,CAAC,IAAI,KAAK,cAAc,CAAC,kBAAkB,EACrD,CAAC;wBACD,SAAS;oBACX,CAAC;oBACD,uBAAuB,CAAC;wBACtB,IAAI;wBACJ,UAAU,EAAE,IAAI,CAAC,KAAK;wBACtB,eAAe;wBACf,OAAO;qBACR,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;SACF,CAAC;IACJ,CAAC;CACF,CACF,CAAC;AAMF,MAAM,uBAAuB,GAAG,CAAC,EAC/B,IAAI,EACJ,UAAU,EACV,eAAe,EACf,OAAO,GAC6C,EAAQ,EAAE;IAC9D,KAAK,MAAM,SAAS,IAAI,UAAU,CAAC,IAAI,CAAC,IAAI,EAAE,CAAC;QAC7C,QAAQ,SAAS,CAAC,IAAI,EAAE,CAAC;YACvB,KAAK,cAAc,CAAC,mBAAmB,CAAC,CAAC,CAAC;gBACxC,MAAM,aAAa,GAAG,SAAS,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;gBACrD,IAAI,aAAa,EAAE,IAAI,KAAK,cAAc,CAAC,aAAa;oBAAE,SAAS;gBACnE,mBAAmB,CAAC;oBAClB,IAAI;oBACJ,OAAO;oBACP,UAAU,EAAE,aAAa;oBACzB,eAAe;iBAChB,CAAC,CAAC;gBACH,MAAM;YACR,CAAC;YACD,KAAK,cAAc,CAAC,mBAAmB,CAAC,CAAC,CAAC;gBACxC,IAAI,SAAS,CAAC,UAAU,EAAE,IAAI,KAAK,cAAc,CAAC,aAAa;oBAAE,MAAM;gBACvE,iBAAiB,CAAC;oBAChB,IAAI;oBACJ,SAAS;oBACT,eAAe;oBACf,OAAO;iBACR,CAAC,CAAC;gBACH,MAAM;YACR,CAAC;YACD,KAAK,cAAc,CAAC,WAAW,CAAC,CAAC,CAAC;gBAChC,kBAAkB,CAAC;oBACjB,IAAI;oBACJ,OAAO;oBACP,eAAe;oBACf,SAAS,EAAE,SAAS,CAAC,UAAU;iBAChC,CAAC,CAAC;gBACH,MAAM;YACR,CAAC;YACD,KAAK,cAAc,CAAC,eAAe,CAAC,CAAC,CAAC;gBACpC,KAAK,MAAM,UAAU,IAAI,SAAS,CAAC,KAAK,EAAE,CAAC;oBACzC,KAAK,MAAM,SAAS,IAAI,UAAU,CAAC,UAAU,EAAE,CAAC;wBAC9C,kBAAkB,CAAC;4BACjB,IAAI;4BACJ,OAAO;4BACP,eAAe;4BACf,SAAS;yBACV,CAAC,CAAC;oBACL,CAAC;gBACH,CAAC;gBACD,MAAM;YACR,CAAC;QACH,CAAC;IACH,CAAC;AACH,CAAC,CAAC;AAOF,MAAM,kBAAkB,GAAG,CAAC,EAC1B,IAAI,EACJ,SAAS,EACT,eAAe,EACf,OAAO,GACmC,EAAE,EAAE;IAC9C,QAAQ,SAAS,CAAC,IAAI,EAAE,CAAC;QACvB,KAAK,cAAc,CAAC,cAAc,CAAC,CAAC,CAAC;YACnC,KAAK,MAAM,IAAI,IAAI,SAAS,CAAC,IAAI,EAAE,CAAC;gBAClC,iBAAiB,CAAC;oBAChB,IAAI;oBACJ,SAAS,EAAE,IAAI;oBACf,eAAe;oBACf,OAAO;iBACR,CAAC,CAAC;YACL,CAAC;YACD,MAAM;QACR,CAAC;QACD,KAAK,cAAc,CAAC,mBAAmB,CAAC,CAAC,CAAC;YACxC,MAAM,aAAa,GAAG,SAAS,CAAC,UAAU,CAAC;YAC3C,IAAI,aAAa,EAAE,IAAI,KAAK,cAAc,CAAC,aAAa;gBAAE,MAAM;YAChE,iBAAiB,CAAC;gBAChB,IAAI;gBACJ,SAAS;gBACT,eAAe;gBACf,OAAO;aACR,CAAC,CAAC;YACH,MAAM;QACR,CAAC;QACD,KAAK,cAAc,CAAC,mBAAmB,CAAC,CAAC,CAAC;YACxC,MAAM,aAAa,GAAG,SAAS,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;YACrD,IAAI,aAAa,EAAE,IAAI,KAAK,cAAc,CAAC,aAAa;gBAAE,MAAM;YAChE,mBAAmB,CAAC;gBAClB,IAAI;gBACJ,OAAO;gBACP,UAAU,EAAE,aAAa;gBACzB,eAAe;aAChB,CAAC,CAAC;YACH,MAAM;QACR,CAAC;IACH,CAAC;AACH,CAAC,CAAC;AAOF,MAAM,iBAAiB,GAAG,CAAC,EACzB,IAAI,EACJ,SAAS,EACT,eAAe,EACf,OAAO,GACmC,EAAQ,EAAE;IACpD,QAAQ,SAAS,CAAC,IAAI,EAAE,CAAC;QACvB,KAAK,cAAc,CAAC,mBAAmB,CAAC,CAAC,CAAC;YACxC,MAAM,aAAa,GAAG,SAAS,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;YACrD,IAAI,aAAa,EAAE,IAAI,KAAK,cAAc,CAAC,aAAa;gBAAE,MAAM;YAChE,mBAAmB,CAAC;gBAClB,IAAI;gBACJ,OAAO;gBACP,UAAU,EAAE,aAAa;gBACzB,eAAe;aAChB,CAAC,CAAC;YACH,MAAM;QACR,CAAC;QACD,KAAK,cAAc,CAAC,mBAAmB,CAAC,CAAC,CAAC;YACxC,MAAM,aAAa,GAAG,SAAS,CAAC,UAAU,CAAC;YAC3C,IAAI,aAAa,EAAE,IAAI,KAAK,cAAc,CAAC,aAAa;gBAAE,MAAM;YAChE,mBAAmB,CAAC;gBAClB,IAAI;gBACJ,OAAO;gBACP,UAAU,EAAE,aAAa;gBACzB,eAAe;aAChB,CAAC,CAAC;YACH,MAAM;QACR,CAAC;QACD,KAAK,cAAc,CAAC,WAAW,CAAC,CAAC,CAAC;YAChC,mBAAmB,CAAC;gBAClB,IAAI;gBACJ,SAAS;gBACT,eAAe;gBACf,OAAO;aACR,CAAC,CAAC;YACH,MAAM;QACR,CAAC;QACD,KAAK,cAAc,CAAC,eAAe,CAAC,CAAC,CAAC;YACpC,uBAAuB,CAAC;gBACtB,IAAI;gBACJ,SAAS;gBACT,eAAe;gBACf,OAAO;aACR,CAAC,CAAC;YACH,MAAM;QACR,CAAC;IACH,CAAC;AACH,CAAC,CAAC;AAMF,MAAM,mBAAmB,GAAG,CAAC,EAC3B,IAAI,EACJ,SAAS,EACT,eAAe,EACf,OAAO,GACqC,EAAQ,EAAE;IACtD,kBAAkB,CAAC;QACjB,IAAI;QACJ,OAAO;QACP,eAAe;QACf,SAAS,EAAE,SAAS,CAAC,UAAU;KAChC,CAAC,CAAC;AACL,CAAC,CAAC;AAMF,MAAM,uBAAuB,GAAG,CAAC,EAC/B,IAAI,EACJ,SAAS,EACT,eAAe,EACf,OAAO,GACyC,EAAQ,EAAE;IAC1D,KAAK,MAAM,aAAa,IAAI,SAAS,CAAC,KAAK,EAAE,CAAC;QAC5C,KAAK,MAAM,WAAW,IAAI,aAAa,CAAC,UAAU,EAAE,CAAC;YACnD,kBAAkB,CAAC;gBACjB,IAAI;gBACJ,OAAO;gBACP,eAAe;gBACf,SAAS,EAAE,WAAW;aACvB,CAAC,CAAC;QACL,CAAC;IACH,CAAC;AACH,CAAC,CAAC;AAKF,MAAM,mBAAmB,GAAG,CAAC,EAC3B,IAAI,EACJ,OAAO,EACP,UAAU,EACV,eAAe,GACgC,EAAQ,EAAE;IACzD,IAAI,UAAU,CAAC,SAAS,CAAC,MAAM,GAAG,CAAC;QAAE,OAAO;IAG5C,MAAM,SAAS,GAAG,UAAU,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC;IAC1C,IACE,SAAS,CAAC,IAAI,KAAK,cAAc,CAAC,OAAO;QACzC,OAAO,SAAS,CAAC,KAAK,KAAK,QAAQ,EACnC,CAAC;QACD,OAAO;IACT,CAAC;IAED,MAAM,oBAAoB,GAAG,YAAY,CAAC,SAAS,CAAC,KAAe,CAAC,CAAC;IACrE,MAAM,wBAAwB,GAAG,YAAY,CAAC,eAAe,CAAC,CAAC;IAE/D,IAAI,wBAAwB,KAAK,oBAAoB,EAAE,CAAC;QACtD,OAAO,CAAC,MAAM,CAAC;YACb,IAAI;YACJ,SAAS,EAAE,8BAA8B;YACzC,IAAI,EAAE;gBACJ,WAAW,EAAE,SAAS,CAAC,KAAK;gBAC5B,mBAAmB,EAAE,eAAe;aACrC;SACF,CAAC,CAAC;IACL,CAAC;AACH,CAAC,CAAC"}

package/dist/no-public-class-fields.d.mts

@@ -0,0 +1,2 @@
+import { ESLintUtils } from "@typescript-eslint/utils";
+export declare const noPublicClassFields: ESLintUtils.RuleModule<"noPublicClassFields", [], unknown, ESLintUtils.RuleListener>;

package/dist/no-public-class-fields.mjs

@@ -1,13 +1,7 @@
 import { AST_NODE_TYPES, ESLintUtils, } from "@typescript-eslint/utils";
 import { SymbolFlags } from "typescript";
-import { isConstructOrStackType } from "./utils/isConstructOrStackType.mjs";
-/**
- * Disallow class types in public class fields
- * @param context - The rule context provided by ESLint
- * @returns An object containing the AST visitor functions
- * @see {@link https://eslint-cdk-plugin.dev/rules/no-public-class-fields} - Documentation
- */
-export var noPublicClassFields = ESLintUtils.RuleCreator.withoutDocs({
+import { isConstructOrStackType } from "./utils/typeCheck.mjs";
+export const noPublicClassFields = ESLintUtils.RuleCreator.withoutDocs({
     meta: {
         type: "problem",
         docs: {
@@ -19,67 +13,54 @@ export var noPublicClassFields = ESLintUtils.RuleCreator.withoutDocs({
         schema: [],
     },
     defaultOptions: [],
-    create: function (context) {
-        var parserServices = ESLintUtils.getParserServices(context);
-        var typeChecker = parserServices.program.getTypeChecker();
+    create(context) {
+        const parserServices = ESLintUtils.getParserServices(context);
+        const typeChecker = parserServices.program.getTypeChecker();
         return {
-            ClassDeclaration: function (node) {
-                var type = typeChecker.getTypeAtLocation(parserServices.esTreeNodeToTSNodeMap.get(node));
+            ClassDeclaration(node) {
+                const type = typeChecker.getTypeAtLocation(parserServices.esTreeNodeToTSNodeMap.get(node));
                 if (!isConstructOrStackType(type)) {
                     return;
                 }
-                // NOTE: Check class members
                 validateClassMember({
-                    node: node,
-                    context: context,
-                    parserServices: parserServices,
-                    typeChecker: typeChecker,
-                });
-                // NOTE: Check constructor parameter properties
-                var constructor = node.body.body.find(function (member) {
-                    return member.type === AST_NODE_TYPES.MethodDefinition &&
-                        member.kind === "constructor";
+                    node,
+                    context,
+                    parserServices,
+                    typeChecker,
                 });
+                const constructor = node.body.body.find((member) => member.type === AST_NODE_TYPES.MethodDefinition &&
+                    member.kind === "constructor");
                 if (!constructor ||
                     constructor.value.type !== AST_NODE_TYPES.FunctionExpression) {
                     return;
                 }
                 validateConstructorParameterProperty({
-                    constructor: constructor,
-                    context: context,
-                    parserServices: parserServices,
-                    typeChecker: typeChecker,
+                    constructor,
+                    context,
+                    parserServices,
+                    typeChecker,
                 });
             },
         };
     },
 });
-/**
- * check the public variable of the class
- * - if it is a class type, report an error
- */
-var validateClassMember = function (_a) {
-    var _b;
-    var node = _a.node, context = _a.context, parserServices = _a.parserServices, typeChecker = _a.typeChecker;
-    for (var _i = 0, _c = node.body.body; _i < _c.length; _i++) {
-        var member = _c[_i];
+const validateClassMember = ({ node, context, parserServices, typeChecker, }) => {
+    for (const member of node.body.body) {
         if (member.type !== AST_NODE_TYPES.PropertyDefinition ||
             member.key.type !== AST_NODE_TYPES.Identifier) {
             continue;
         }
-        // NOTE: Skip private and protected fields
-        if (["private", "protected"].includes((_b = member.accessibility) !== null && _b !== void 0 ? _b : "")) {
+        if (["private", "protected"].includes(member.accessibility ?? "")) {
             continue;
         }
-        // NOTE: Skip fields without type annotation
         if (!member.typeAnnotation) {
             continue;
         }
-        var tsNode = parserServices.esTreeNodeToTSNodeMap.get(member);
-        var type = typeChecker.getTypeAtLocation(tsNode);
+        const tsNode = parserServices.esTreeNodeToTSNodeMap.get(member);
+        const type = typeChecker.getTypeAtLocation(tsNode);
         if (!type.symbol)
             continue;
-        var isClass = type.symbol.flags === SymbolFlags.Class;
+        const isClass = type.symbol.flags === SymbolFlags.Class;
         if (!isClass)
             continue;
         context.report({
@@ -92,32 +73,23 @@ var validateClassMember = function (_a) {
         });
     }
 };
-/**
- * check the constructor parameter property
- * - if it is a class type, report an error
- */
-var validateConstructorParameterProperty = function (_a) {
-    var _b;
-    var constructor = _a.constructor, context = _a.context, parserServices = _a.parserServices, typeChecker = _a.typeChecker;
-    for (var _i = 0, _c = constructor.value.params; _i < _c.length; _i++) {
-        var param = _c[_i];
+const validateConstructorParameterProperty = ({ constructor, context, parserServices, typeChecker, }) => {
+    for (const param of constructor.value.params) {
         if (param.type !== AST_NODE_TYPES.TSParameterProperty ||
             param.parameter.type !== AST_NODE_TYPES.Identifier) {
             continue;
         }
-        // NOTE: Skip private and protected parameters
-        if (["private", "protected"].includes((_b = param.accessibility) !== null && _b !== void 0 ? _b : "")) {
+        if (["private", "protected"].includes(param.accessibility ?? "")) {
             continue;
         }
-        // NOTE: Skip parameters without type annotation
         if (!param.parameter.typeAnnotation) {
             continue;
         }
-        var tsNode = parserServices.esTreeNodeToTSNodeMap.get(param);
-        var type = typeChecker.getTypeAtLocation(tsNode);
+        const tsNode = parserServices.esTreeNodeToTSNodeMap.get(param);
+        const type = typeChecker.getTypeAtLocation(tsNode);
         if (!type.symbol)
             continue;
-        var isClass = type.symbol.flags === SymbolFlags.Class;
+        const isClass = type.symbol.flags === SymbolFlags.Class;
         if (!isClass)
             continue;
         context.report({
@@ -130,3 +102,4 @@ var validateConstructorParameterProperty = function (_a) {
         });
     }
 };
+//# sourceMappingURL=no-public-class-fields.mjs.map

package/dist/no-public-class-fields.mjs.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-public-class-fields.mjs","sourceRoot":"","sources":["../src/no-public-class-fields.mts"],"names":[],"mappings":"AAAA,OAAO,EACL,cAAc,EACd,WAAW,GAIZ,MAAM,0BAA0B,CAAC;AAClC,OAAO,EAAE,WAAW,EAAe,MAAM,YAAY,CAAC;AAEtD,OAAO,EAAE,sBAAsB,EAAE,MAAM,uBAAuB,CAAC;AAU/D,MAAM,CAAC,MAAM,mBAAmB,GAAG,WAAW,CAAC,WAAW,CAAC,WAAW,CAAC;IACrE,IAAI,EAAE;QACJ,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACJ,WAAW,EAAE,6CAA6C;SAC3D;QACD,QAAQ,EAAE;YACR,mBAAmB,EACjB,kIAAkI;SACrI;QACD,MAAM,EAAE,EAAE;KACX;IACD,cAAc,EAAE,EAAE;IAClB,MAAM,CAAC,OAAO;QACZ,MAAM,cAAc,GAAG,WAAW,CAAC,iBAAiB,CAAC,OAAO,CAAC,CAAC;QAC9D,MAAM,WAAW,GAAG,cAAc,CAAC,OAAO,CAAC,cAAc,EAAE,CAAC;QAC5D,OAAO;YACL,gBAAgB,CAAC,IAAI;gBACnB,MAAM,IAAI,GAAG,WAAW,CAAC,iBAAiB,CACxC,cAAc,CAAC,qBAAqB,CAAC,GAAG,CAAC,IAAI,CAAC,CAC/C,CAAC;gBACF,IAAI,CAAC,sBAAsB,CAAC,IAAI,CAAC,EAAE,CAAC;oBAClC,OAAO;gBACT,CAAC;gBAGD,mBAAmB,CAAC;oBAClB,IAAI;oBACJ,OAAO;oBACP,cAAc;oBACd,WAAW;iBACZ,CAAC,CAAC;gBAGH,MAAM,WAAW,GAAG,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CACrC,CAAC,MAAM,EAAuC,EAAE,CAC9C,MAAM,CAAC,IAAI,KAAK,cAAc,CAAC,gBAAgB;oBAC/C,MAAM,CAAC,IAAI,KAAK,aAAa,CAChC,CAAC;gBACF,IACE,CAAC,WAAW;oBACZ,WAAW,CAAC,KAAK,CAAC,IAAI,KAAK,cAAc,CAAC,kBAAkB,EAC5D,CAAC;oBACD,OAAO;gBACT,CAAC;gBACD,oCAAoC,CAAC;oBACnC,WAAW;oBACX,OAAO;oBACP,cAAc;oBACd,WAAW;iBACZ,CAAC,CAAC;YACL,CAAC;SACF,CAAC;IACJ,CAAC;CACF,CAAC,CAAC;AAMH,MAAM,mBAAmB,GAAG,CAAC,EAC3B,IAAI,EACJ,OAAO,EACP,cAAc,EACd,WAAW,GAMZ,EAAE,EAAE;IACH,KAAK,MAAM,MAAM,IAAI,IAAI,CAAC,IAAI,CAAC,IAAI,EAAE,CAAC;QACpC,IACE,MAAM,CAAC,IAAI,KAAK,cAAc,CAAC,kBAAkB;YACjD,MAAM,CAAC,GAAG,CAAC,IAAI,KAAK,cAAc,CAAC,UAAU,EAC7C,CAAC;YACD,SAAS;QACX,CAAC;QAGD,IAAI,CAAC,SAAS,EAAE,WAAW,CAAC,CAAC,QAAQ,CAAC,MAAM,CAAC,aAAa,IAAI,EAAE,CAAC,EAAE,CAAC;YAClE,SAAS;QACX,CAAC;QAGD,IAAI,CAAC,MAAM,CAAC,cAAc,EAAE,CAAC;YAC3B,SAAS;QACX,CAAC;QAED,MAAM,MAAM,GAAG,cAAc,CAAC,qBAAqB,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC;QAChE,MAAM,IAAI,GAAG,WAAW,CAAC,iBAAiB,CAAC,MAAM,CAAC,CAAC;QAEnD,IAAI,CAAC,IAAI,CAAC,MAAM;YAAE,SAAS;QAE3B,MAAM,OAAO,GAAG,IAAI,CAAC,MAAM,CAAC,KAAK,KAAK,WAAW,CAAC,KAAK,CAAC;QACxD,IAAI,CAAC,OAAO;YAAE,SAAS;QAEvB,OAAO,CAAC,MAAM,CAAC;YACb,IAAI,EAAE,MAAM;YACZ,SAAS,EAAE,qBAAqB;YAChC,IAAI,EAAE;gBACJ,YAAY,EAAE,MAAM,CAAC,GAAG,CAAC,IAAI;gBAC7B,QAAQ,EAAE,IAAI,CAAC,MAAM,CAAC,IAAI;aAC3B;SACF,CAAC,CAAC;IACL,CAAC;AACH,CAAC,CAAC;AAMF,MAAM,oCAAoC,GAAG,CAAC,EAC5C,WAAW,EACX,OAAO,EACP,cAAc,EACd,WAAW,GAMZ,EAAE,EAAE;IACH,KAAK,MAAM,KAAK,IAAI,WAAW,CAAC,KAAK,CAAC,MAAM,EAAE,CAAC;QAC7C,IACE,KAAK,CAAC,IAAI,KAAK,cAAc,CAAC,mBAAmB;YACjD,KAAK,CAAC,SAAS,CAAC,IAAI,KAAK,cAAc,CAAC,UAAU,EAClD,CAAC;YACD,SAAS;QACX,CAAC;QAGD,IAAI,CAAC,SAAS,EAAE,WAAW,CAAC,CAAC,QAAQ,CAAC,KAAK,CAAC,aAAa,IAAI,EAAE,CAAC,EAAE,CAAC;YACjE,SAAS;QACX,CAAC;QAGD,IAAI,CAAC,KAAK,CAAC,SAAS,CAAC,cAAc,EAAE,CAAC;YACpC,SAAS;QACX,CAAC;QAED,MAAM,MAAM,GAAG,cAAc,CAAC,qBAAqB,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC;QAC/D,MAAM,IAAI,GAAG,WAAW,CAAC,iBAAiB,CAAC,MAAM,CAAC,CAAC;QAEnD,IAAI,CAAC,IAAI,CAAC,MAAM;YAAE,SAAS;QAE3B,MAAM,OAAO,GAAG,IAAI,CAAC,MAAM,CAAC,KAAK,KAAK,WAAW,CAAC,KAAK,CAAC;QACxD,IAAI,CAAC,OAAO;YAAE,SAAS;QAEvB,OAAO,CAAC,MAAM,CAAC;YACb,IAAI,EAAE,KAAK;YACX,SAAS,EAAE,qBAAqB;YAChC,IAAI,EAAE;gBACJ,YAAY,EAAE,KAAK,CAAC,SAAS,CAAC,IAAI;gBAClC,QAAQ,EAAE,IAAI,CAAC,MAAM,CAAC,IAAI;aAC3B;SACF,CAAC,CAAC;IACL,CAAC;AACH,CAAC,CAAC"}

package/dist/no-variable-construct-id.d.mts

@@ -0,0 +1,2 @@
+import { ESLintUtils } from "@typescript-eslint/utils";
+export declare const noVariableConstructId: ESLintUtils.RuleModule<"noVariableConstructId", [], unknown, ESLintUtils.RuleListener>;