eslint-cdk-plugin 0.1.0 → 0.4.0

package/README.md

@@ -1,4 +1,4 @@
-# eslint-plugin-cdk
+# eslint-cdk-plugin
 
 ESLint plugin for [AWS CDK](https://github.com/aws/aws-cdk).
 
@@ -6,13 +6,13 @@ ESLint plugin for [AWS CDK](https://github.com/aws/aws-cdk).
 
 ```bash
 # npm
-npm install -D @nigg/eslint-plugin-cdk
+npm install --save-dev eslint-cdk-plugin
 
 # yarn
-yarn add -D @nigg/eslint-plugin-cdk
+yarn add -D eslint-cdk-plugin
 
 # pnpm
-pnpm install -D @nigg/eslint-plugin-cdk
+pnpm install -D eslint-cdk-plugin
 ```
 
 ## Usage
@@ -21,17 +21,17 @@ pnpm install -D @nigg/eslint-plugin-cdk
 
 ```js
 // eslint.config.mjs
-import eslintPluginCdk from "@nigg/eslint-plugin-cdk";
+import eslintCdkPlugin from "eslint-cdk-plugin";
 export default [
   {
     plugins: {
-      cdk: eslintPluginCdk,
+      cdk: eslintCdkPlugin,
     },
     rules: {
-      ...eslintPluginCdk.configs.recommended.rules,
+      ...eslintCdkPlugin.configs.recommended.rules,
     },
   },
 ];
 ```
 
-### For more detailed documentation, see [docs for eslint-plugin-cdk](https://eslint-plugin-cdk.dev/)
+### For more detailed documentation, see [docs for eslint-cdk-plugin](https://eslint-cdk-plugin.dev/)

package/dist/index.mjs

@@ -1,4 +1,4 @@
-import { noClassInInterfaceProps } from "./no-class-in-interface.mjs";
+import { noClassInInterfaceProps } from "./no-class-in-interface-props.mjs";
 import { noConstructStackSuffix } from "./no-construct-stack-suffix.mjs";
 import { noImportPrivate } from "./no-import-private.mjs";
 import { noMutablePropsInterface } from "./no-mutable-props-interface.mjs";

package/dist/{no-class-in-interface.mjs → no-class-in-interface-props.mjs}

@@ -1,5 +1,11 @@
-import { ESLintUtils } from "@typescript-eslint/utils";
+import { AST_NODE_TYPES, ESLintUtils } from "@typescript-eslint/utils";
 import { SymbolFlags } from "typescript";
+/**
+ * Enforces the use of interface types instead of class in interface properties
+ * @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-class-in-interface} - Documentation
+ */
 export var noClassInInterfaceProps = ESLintUtils.RuleCreator.withoutDocs({
     meta: {
         type: "problem",
@@ -19,8 +25,9 @@ export var noClassInInterfaceProps = ESLintUtils.RuleCreator.withoutDocs({
             TSInterfaceDeclaration: function (node) {
                 for (var _i = 0, _a = node.body.body; _i < _a.length; _i++) {
                     var property = _a[_i];
-                    if (property.type !== "TSPropertySignature" ||
-                        property.key.type !== "Identifier") {
+                    // NOTE: check property signature
+                    if (property.type !== AST_NODE_TYPES.TSPropertySignature ||
+                        property.key.type !== AST_NODE_TYPES.Identifier) {
                         continue;
                     }
                     var tsNode = parserServices.esTreeNodeToTSNodeMap.get(property);

package/dist/no-construct-stack-suffix.mjs

@@ -1,5 +1,12 @@
-import { ESLintUtils } from "@typescript-eslint/utils";
+import { AST_NODE_TYPES, ESLintUtils, } from "@typescript-eslint/utils";
 import { toPascalCase } from "./utils/convertString.mjs";
+import { isConstructOrStackType } from "./utils/isConstructOrStackType.mjs";
+/**
+ * Enforces that Construct IDs do not end with 'Construct' or 'Stack' suffix
+ * @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-construct-stack-suffix} - Documentation
+ */
 export var noConstructStackSuffix = ESLintUtils.RuleCreator.withoutDocs({
     meta: {
         type: "problem",
@@ -13,148 +20,21 @@ export var noConstructStackSuffix = ESLintUtils.RuleCreator.withoutDocs({
     },
     defaultOptions: [],
     create: function (context) {
+        var parserServices = ESLintUtils.getParserServices(context);
+        var checker = parserServices.program.getTypeChecker();
         return {
-            ClassBody: function (node) {
-                var _a;
-                var parent = node.parent;
-                if ((parent === null || parent === void 0 ? void 0 : parent.type) !== "ClassDeclaration")
-                    return;
-                var className = (_a = parent.id) === null || _a === void 0 ? void 0 : _a.name;
-                if (!className)
+            NewExpression: function (node) {
+                var type = checker.getTypeAtLocation(parserServices.esTreeNodeToTSNodeMap.get(node));
+                if (!isConstructOrStackType(type)) {
                     return;
-                for (var _i = 0, _b = node.body; _i < _b.length; _i++) {
-                    var body = _b[_i];
-                    if (body.type !== "MethodDefinition" ||
-                        !["method", "constructor"].includes(body.kind) ||
-                        body.value.type !== "FunctionExpression") {
-                        continue;
-                    }
-                    validateConstructorBody(node, body.value, context);
                 }
+                if (node.arguments.length < 2)
+                    return;
+                validateConstructId(node, context, node);
             },
         };
     },
 });
-/**
- * Validate the constructor body for the parent class
- * - validate each statement in the constructor body
- */
-var validateConstructorBody = function (node, expression, context) {
-    var _a;
-    for (var _i = 0, _b = expression.body.body; _i < _b.length; _i++) {
-        var statement = _b[_i];
-        switch (statement.type) {
-            case "VariableDeclaration": {
-                var newExpression = statement.declarations[0].init;
-                if ((newExpression === null || newExpression === void 0 ? void 0 : newExpression.type) !== "NewExpression")
-                    continue;
-                validateConstructId(node, context, newExpression);
-                break;
-            }
-            case "ExpressionStatement": {
-                if (((_a = statement.expression) === null || _a === void 0 ? void 0 : _a.type) !== "NewExpression")
-                    break;
-                validateStatement(node, statement, context);
-                break;
-            }
-            case "IfStatement": {
-                traverseStatements(node, statement.consequent, context);
-                break;
-            }
-            case "SwitchStatement": {
-                for (var _c = 0, _d = statement.cases; _c < _d.length; _c++) {
-                    var switchCase = _d[_c];
-                    for (var _e = 0, _f = switchCase.consequent; _e < _f.length; _e++) {
-                        var statement_1 = _f[_e];
-                        traverseStatements(node, statement_1, context);
-                    }
-                }
-                break;
-            }
-        }
-    }
-};
-/**
- * Recursively traverse and validate statements in the AST
- * - Handles BlockStatement, ExpressionStatement, and VariableDeclaration
- * - Validates construct IDs
- */
-var traverseStatements = function (node, statement, context) {
-    switch (statement.type) {
-        case "BlockStatement": {
-            for (var _i = 0, _a = statement.body; _i < _a.length; _i++) {
-                var body = _a[_i];
-                validateStatement(node, body, context);
-            }
-            break;
-        }
-        case "ExpressionStatement": {
-            var newExpression = statement.expression;
-            if ((newExpression === null || newExpression === void 0 ? void 0 : newExpression.type) !== "NewExpression")
-                break;
-            validateStatement(node, statement, context);
-            break;
-        }
-        case "VariableDeclaration": {
-            var newExpression = statement.declarations[0].init;
-            if ((newExpression === null || newExpression === void 0 ? void 0 : newExpression.type) !== "NewExpression")
-                break;
-            validateConstructId(node, context, newExpression);
-            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 (node, body, context) {
-    switch (body.type) {
-        case "VariableDeclaration": {
-            var newExpression = body.declarations[0].init;
-            if ((newExpression === null || newExpression === void 0 ? void 0 : newExpression.type) !== "NewExpression")
-                break;
-            validateConstructId(node, context, newExpression);
-            break;
-        }
-        case "ExpressionStatement": {
-            var newExpression = body.expression;
-            if ((newExpression === null || newExpression === void 0 ? void 0 : newExpression.type) !== "NewExpression")
-                break;
-            validateConstructId(node, context, newExpression);
-            break;
-        }
-        case "IfStatement": {
-            validateIfStatement(node, body, context);
-            break;
-        }
-        case "SwitchStatement": {
-            validateSwitchStatement(node, body, context);
-            break;
-        }
-    }
-};
-/**
- * Validate the `if` statement
- * - Validate recursively if `if` statements are nested
- */
-var validateIfStatement = function (node, ifStatement, context) {
-    traverseStatements(node, ifStatement.consequent, context);
-};
-/**
- * Validate the `switch` statement
- * - Validate recursively if `switch` statements are nested
- */
-var validateSwitchStatement = function (node, switchStatement, context) {
-    for (var _i = 0, _a = switchStatement.cases; _i < _a.length; _i++) {
-        var statement = _a[_i];
-        for (var _b = 0, _c = statement.consequent; _b < _c.length; _b++) {
-            var _consequent = _c[_b];
-            traverseStatements(node, _consequent, context);
-        }
-    }
-};
 /**
  * Validate that construct ID does not end with "Construct" or "Stack"
  */
@@ -163,7 +43,8 @@ var validateConstructId = function (node, context, expression) {
         return;
     // NOTE: Treat the second argument as ID
     var secondArg = expression.arguments[1];
-    if (secondArg.type !== "Literal" || typeof secondArg.value !== "string") {
+    if (secondArg.type !== AST_NODE_TYPES.Literal ||
+        typeof secondArg.value !== "string") {
         return;
     }
     var formattedConstructId = toPascalCase(secondArg.value);

package/dist/no-import-private.mjs

@@ -1,12 +1,10 @@
 import * as path from "path";
 /**
- * Split the directory path into segments (split at `/`)
- * @param dirPath - The directory path to split
- * @returns The segments of the directory path
+ * Disallow importing modules from private directories at different levels of the hierarchy.
+ * @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-import-private} - Documentation
  */
-var getDirSegments = function (dirPath) {
-    return dirPath.split(path.sep).filter(function (segment) { return segment !== ""; });
-};
 export var noImportPrivate = {
     meta: {
         type: "problem",
@@ -41,3 +39,11 @@ export var noImportPrivate = {
         };
     },
 };
+/**
+ * Split the directory path into segments (split at `/`)
+ * @param dirPath - The directory path to split
+ * @returns The segments of the directory path
+ */
+var getDirSegments = function (dirPath) {
+    return dirPath.split(path.sep).filter(function (segment) { return segment !== ""; });
+};

package/dist/no-mutable-props-interface.mjs

@@ -1,4 +1,10 @@
-import { ESLintUtils } from "@typescript-eslint/utils";
+import { AST_NODE_TYPES, ESLintUtils } from "@typescript-eslint/utils";
+/**
+ * Disallow mutable properties in Props interfaces
+ * @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-props-interface} - Documentation
+ */
 export var noMutablePropsInterface = ESLintUtils.RuleCreator.withoutDocs({
     meta: {
         type: "problem",
@@ -20,8 +26,9 @@ export var noMutablePropsInterface = ESLintUtils.RuleCreator.withoutDocs({
                 if (!node.id.name.endsWith("Props"))
                     return;
                 var _loop_1 = function (property) {
-                    if (property.type !== "TSPropertySignature" ||
-                        property.key.type !== "Identifier") {
+                    // NOTE: check property signature
+                    if (property.type !== AST_NODE_TYPES.TSPropertySignature ||
+                        property.key.type !== AST_NODE_TYPES.Identifier) {
                         return "continue";
                     }
                     // NOTE: Skip if already readonly

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

@@ -1,4 +1,11 @@
-import { ESLintUtils } from "@typescript-eslint/utils";
+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({
     meta: {
         type: "problem",
@@ -13,13 +20,20 @@ export var noMutablePublicFields = ESLintUtils.RuleCreator.withoutDocs({
     },
     defaultOptions: [],
     create: function (context) {
+        var parserServices = ESLintUtils.getParserServices(context);
+        var checker = parserServices.program.getTypeChecker();
         var sourceCode = context.sourceCode;
         return {
             ClassDeclaration: function (node) {
                 var _a;
+                var type = checker.getTypeAtLocation(parserServices.esTreeNodeToTSNodeMap.get(node));
+                if (!isConstructOrStackType(type)) {
+                    return;
+                }
                 var _loop_1 = function (member) {
-                    if (member.type !== "PropertyDefinition" ||
-                        member.key.type !== "Identifier") {
+                    // NOTE: check property definition
+                    if (member.type !== AST_NODE_TYPES.PropertyDefinition ||
+                        member.key.type !== AST_NODE_TYPES.Identifier) {
                         return "continue";
                     }
                     // NOTE: Skip private and protected fields

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

@@ -1,5 +1,11 @@
-import { ESLintUtils } from "@typescript-eslint/utils";
+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({
     meta: {
         type: "problem",
@@ -17,16 +23,17 @@ export var noParentNameConstructIdMatch = ESLintUtils.RuleCreator.withoutDocs({
             ClassBody: function (node) {
                 var _a;
                 var parent = node.parent;
-                if ((parent === null || parent === void 0 ? void 0 : parent.type) !== "ClassDeclaration")
+                if ((parent === null || parent === void 0 ? void 0 : parent.type) !== AST_NODE_TYPES.ClassDeclaration)
                     return;
                 var parentClassName = (_a = parent.id) === null || _a === void 0 ? void 0 : _a.name;
                 if (!parentClassName)
                     return;
                 for (var _i = 0, _b = node.body; _i < _b.length; _i++) {
                     var body = _b[_i];
-                    if (body.type !== "MethodDefinition" ||
+                    // NOTE: Ignore if neither method nor constructor.
+                    if (body.type !== AST_NODE_TYPES.MethodDefinition ||
                         !["method", "constructor"].includes(body.kind) ||
-                        body.value.type !== "FunctionExpression") {
+                        body.value.type !== AST_NODE_TYPES.FunctionExpression) {
                         continue;
                     }
                     validateConstructorBody({
@@ -50,9 +57,9 @@ var validateConstructorBody = function (_a) {
     for (var _i = 0, _c = expression.body.body; _i < _c.length; _i++) {
         var statement = _c[_i];
         switch (statement.type) {
-            case "VariableDeclaration": {
+            case AST_NODE_TYPES.VariableDeclaration: {
                 var newExpression = statement.declarations[0].init;
-                if ((newExpression === null || newExpression === void 0 ? void 0 : newExpression.type) !== "NewExpression")
+                if ((newExpression === null || newExpression === void 0 ? void 0 : newExpression.type) !== AST_NODE_TYPES.NewExpression)
                     continue;
                 validateConstructId({
                     node: node,
@@ -62,18 +69,18 @@ var validateConstructorBody = function (_a) {
                 });
                 break;
             }
-            case "ExpressionStatement": {
-                if (((_b = statement.expression) === null || _b === void 0 ? void 0 : _b.type) !== "NewExpression")
+            case AST_NODE_TYPES.ExpressionStatement: {
+                if (((_b = statement.expression) === null || _b === void 0 ? void 0 : _b.type) !== AST_NODE_TYPES.NewExpression)
                     break;
                 validateStatement({
                     node: node,
-                    body: statement,
+                    statement: statement,
                     parentClassName: parentClassName,
                     context: context,
                 });
                 break;
             }
-            case "IfStatement": {
+            case AST_NODE_TYPES.IfStatement: {
                 traverseStatements({
                     node: node,
                     context: context,
@@ -82,7 +89,7 @@ var validateConstructorBody = function (_a) {
                 });
                 break;
             }
-            case "SwitchStatement": {
+            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++) {
@@ -108,33 +115,33 @@ var validateConstructorBody = function (_a) {
 var traverseStatements = function (_a) {
     var node = _a.node, statement = _a.statement, parentClassName = _a.parentClassName, context = _a.context;
     switch (statement.type) {
-        case "BlockStatement": {
+        case AST_NODE_TYPES.BlockStatement: {
             for (var _i = 0, _b = statement.body; _i < _b.length; _i++) {
                 var body = _b[_i];
                 validateStatement({
                     node: node,
-                    body: body,
+                    statement: body,
                     parentClassName: parentClassName,
                     context: context,
                 });
             }
             break;
         }
-        case "ExpressionStatement": {
+        case AST_NODE_TYPES.ExpressionStatement: {
             var newExpression = statement.expression;
-            if ((newExpression === null || newExpression === void 0 ? void 0 : newExpression.type) !== "NewExpression")
+            if ((newExpression === null || newExpression === void 0 ? void 0 : newExpression.type) !== AST_NODE_TYPES.NewExpression)
                 break;
             validateStatement({
                 node: node,
-                body: statement,
+                statement: statement,
                 parentClassName: parentClassName,
                 context: context,
             });
             break;
         }
-        case "VariableDeclaration": {
+        case AST_NODE_TYPES.VariableDeclaration: {
             var newExpression = statement.declarations[0].init;
-            if ((newExpression === null || newExpression === void 0 ? void 0 : newExpression.type) !== "NewExpression")
+            if ((newExpression === null || newExpression === void 0 ? void 0 : newExpression.type) !== AST_NODE_TYPES.NewExpression)
                 break;
             validateConstructId({
                 node: node,
@@ -152,11 +159,11 @@ var traverseStatements = function (_a) {
  * - Extracts and validates construct IDs from new expressions
  */
 var validateStatement = function (_a) {
-    var node = _a.node, body = _a.body, parentClassName = _a.parentClassName, context = _a.context;
-    switch (body.type) {
-        case "VariableDeclaration": {
-            var newExpression = body.declarations[0].init;
-            if ((newExpression === null || newExpression === void 0 ? void 0 : newExpression.type) !== "NewExpression")
+    var node = _a.node, statement = _a.statement, parentClassName = _a.parentClassName, context = _a.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)
                 break;
             validateConstructId({
                 node: node,
@@ -166,9 +173,9 @@ var validateStatement = function (_a) {
             });
             break;
         }
-        case "ExpressionStatement": {
-            var newExpression = body.expression;
-            if ((newExpression === null || newExpression === void 0 ? void 0 : newExpression.type) !== "NewExpression")
+        case AST_NODE_TYPES.ExpressionStatement: {
+            var newExpression = statement.expression;
+            if ((newExpression === null || newExpression === void 0 ? void 0 : newExpression.type) !== AST_NODE_TYPES.NewExpression)
                 break;
             validateConstructId({
                 node: node,
@@ -178,19 +185,19 @@ var validateStatement = function (_a) {
             });
             break;
         }
-        case "IfStatement": {
+        case AST_NODE_TYPES.IfStatement: {
             validateIfStatement({
                 node: node,
-                ifStatement: body,
+                statement: statement,
                 parentClassName: parentClassName,
                 context: context,
             });
             break;
         }
-        case "SwitchStatement": {
+        case AST_NODE_TYPES.SwitchStatement: {
             validateSwitchStatement({
                 node: node,
-                switchStatement: body,
+                statement: statement,
                 parentClassName: parentClassName,
                 context: context,
             });
@@ -203,12 +210,12 @@ var validateStatement = function (_a) {
  * - Validate recursively if `if` statements are nested
  */
 var validateIfStatement = function (_a) {
-    var node = _a.node, ifStatement = _a.ifStatement, parentClassName = _a.parentClassName, context = _a.context;
+    var node = _a.node, statement = _a.statement, parentClassName = _a.parentClassName, context = _a.context;
     traverseStatements({
         node: node,
         context: context,
         parentClassName: parentClassName,
-        statement: ifStatement.consequent,
+        statement: statement.consequent,
     });
 };
 /**
@@ -216,10 +223,10 @@ var validateIfStatement = function (_a) {
  * - Validate recursively if `switch` statements are nested
  */
 var validateSwitchStatement = function (_a) {
-    var node = _a.node, switchStatement = _a.switchStatement, parentClassName = _a.parentClassName, context = _a.context;
-    for (var _i = 0, _b = switchStatement.cases; _i < _b.length; _i++) {
-        var statement = _b[_i];
-        for (var _c = 0, _d = statement.consequent; _c < _d.length; _c++) {
+    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];
             traverseStatements({
                 node: node,
@@ -239,7 +246,8 @@ var validateConstructId = function (_a) {
         return;
     // NOTE: Treat the second argument as ID
     var secondArg = expression.arguments[1];
-    if (secondArg.type !== "Literal" || typeof secondArg.value !== "string") {
+    if (secondArg.type !== AST_NODE_TYPES.Literal ||
+        typeof secondArg.value !== "string") {
         return;
     }
     var formattedConstructId = toPascalCase(secondArg.value);

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

@@ -1,5 +1,12 @@
-import { ESLintUtils, } from "@typescript-eslint/utils";
+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({
     meta: {
         type: "problem",
@@ -17,6 +24,10 @@ export var noPublicClassFields = ESLintUtils.RuleCreator.withoutDocs({
         var typeChecker = parserServices.program.getTypeChecker();
         return {
             ClassDeclaration: function (node) {
+                var type = typeChecker.getTypeAtLocation(parserServices.esTreeNodeToTSNodeMap.get(node));
+                if (!isConstructOrStackType(type)) {
+                    return;
+                }
                 // NOTE: Check class members
                 validateClassMember({
                     node: node,
@@ -26,9 +37,11 @@ export var noPublicClassFields = ESLintUtils.RuleCreator.withoutDocs({
                 });
                 // NOTE: Check constructor parameter properties
                 var constructor = node.body.body.find(function (member) {
-                    return member.type === "MethodDefinition" && member.kind === "constructor";
+                    return member.type === AST_NODE_TYPES.MethodDefinition &&
+                        member.kind === "constructor";
                 });
-                if (!constructor || constructor.value.type !== "FunctionExpression") {
+                if (!constructor ||
+                    constructor.value.type !== AST_NODE_TYPES.FunctionExpression) {
                     return;
                 }
                 validateConstructorParameterProperty({
@@ -41,13 +54,17 @@ export var noPublicClassFields = ESLintUtils.RuleCreator.withoutDocs({
         };
     },
 });
+/**
+ * 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];
-        if (member.type !== "PropertyDefinition" ||
-            member.key.type !== "Identifier") {
+        if (member.type !== AST_NODE_TYPES.PropertyDefinition ||
+            member.key.type !== AST_NODE_TYPES.Identifier) {
             continue;
         }
         // NOTE: Skip private and protected fields
@@ -75,13 +92,17 @@ 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];
-        if (param.type !== "TSParameterProperty" ||
-            param.parameter.type !== "Identifier") {
+        if (param.type !== AST_NODE_TYPES.TSParameterProperty ||
+            param.parameter.type !== AST_NODE_TYPES.Identifier) {
             continue;
         }
         // NOTE: Skip private and protected parameters

package/dist/pascal-case-construct-id.mjs

@@ -1,8 +1,57 @@
+import { AST_NODE_TYPES, ESLintUtils, } from "@typescript-eslint/utils";
 import { toPascalCase } from "./utils/convertString.mjs";
+import { isConstructOrStackType } from "./utils/isConstructOrStackType.mjs";
 var QUOTE_TYPE = {
     SINGLE: "'",
     DOUBLE: '"',
 };
+/**
+/**
+ * Enforce PascalCase for Construct ID.
+ * @param context - The rule context provided by ESLint
+ * @returns An object containing the AST visitor functions
+ * @see {@link https://eslint-cdk-plugin.dev/rules/pascal-case-construct-id} - Documentation
+ */
+export var pascalCaseConstructId = ESLintUtils.RuleCreator.withoutDocs({
+    meta: {
+        type: "problem",
+        docs: {
+            description: "Enforce PascalCase for Construct ID.",
+        },
+        messages: {
+            pascalCaseConstructId: "Construct ID must be PascalCase.",
+        },
+        schema: [],
+        fixable: "code",
+    },
+    defaultOptions: [],
+    create: function (context) {
+        var parserServices = ESLintUtils.getParserServices(context);
+        var checker = parserServices.program.getTypeChecker();
+        return {
+            // ExpressionStatement(node) {
+            //   if (node.expression.type !== AST_NODE_TYPES.NewExpression) return;
+            //   validateConstructId(node, context, node.expression.arguments);
+            // },
+            // VariableDeclaration(node) {
+            //   if (!node.declarations.length) return;
+            //   for (const declaration of node.declarations) {
+            //     if (declaration.init?.type !== AST_NODE_TYPES.NewExpression) return;
+            //     validateConstructId(node, context, declaration.init.arguments);
+            //   }
+            // },
+            NewExpression: function (node) {
+                var type = checker.getTypeAtLocation(parserServices.esTreeNodeToTSNodeMap.get(node));
+                if (!isConstructOrStackType(type)) {
+                    return;
+                }
+                if (node.arguments.length < 2)
+                    return;
+                validateConstructId(node, context, node);
+            },
+        };
+    },
+});
 /**
  * check if the string is PascalCase
  * @param str - The string to check
@@ -11,13 +60,17 @@ var QUOTE_TYPE = {
 var isPascalCase = function (str) {
     return /^[A-Z][a-zA-Z0-9]*$/.test(str);
 };
-var validateConstructId = function (node, context, args) {
+/**
+ * Check the construct ID is PascalCase
+ */
+var validateConstructId = function (node, context, expression) {
     var _a;
-    if (args.length < 2)
+    if (expression.arguments.length < 2)
         return;
     // NOTE: Treat the second argument as ID
-    var secondArg = args[1];
-    if (secondArg.type !== "Literal" || typeof secondArg.value !== "string") {
+    var secondArg = expression.arguments[1];
+    if (secondArg.type !== AST_NODE_TYPES.Literal ||
+        typeof secondArg.value !== "string") {
         return;
     }
     var quote = ((_a = secondArg.raw) === null || _a === void 0 ? void 0 : _a.startsWith('"'))
@@ -34,36 +87,3 @@ var validateConstructId = function (node, context, args) {
         });
     }
 };
-export var pascalCaseConstructId = {
-    meta: {
-        type: "problem",
-        docs: {
-            description: "Enforce PascalCase for Construct ID.",
-        },
-        messages: {
-            pascalCaseConstructId: "Construct ID must be PascalCase.",
-        },
-        schema: [],
-        fixable: "code",
-    },
-    create: function (context) {
-        return {
-            ExpressionStatement: function (node) {
-                if (node.expression.type !== "NewExpression")
-                    return;
-                validateConstructId(node, context, node.expression.arguments);
-            },
-            VariableDeclaration: function (node) {
-                var _a;
-                if (!node.declarations.length)
-                    return;
-                for (var _i = 0, _b = node.declarations; _i < _b.length; _i++) {
-                    var declaration = _b[_i];
-                    if (((_a = declaration.init) === null || _a === void 0 ? void 0 : _a.type) !== "NewExpression")
-                        return;
-                    validateConstructId(node, context, declaration.init.arguments);
-                }
-            },
-        };
-    },
-};

package/dist/utils/isConstructOrStackType.mjs

@@ -0,0 +1,19 @@
+export var SUPPORTED_SUPER_CLASS_SUFFIXES = ["Construct", "Stack"];
+/**
+ * Check if the type extends Construct or Stack
+ * @param type - The type to check
+ * @returns True if the type extends Construct or Stack, otherwise false
+ */
+export var isConstructOrStackType = function (type) {
+    if (!type.symbol)
+        return false;
+    // NOTE: Check if the current type ends in Construct or Stack
+    if (SUPPORTED_SUPER_CLASS_SUFFIXES.some(function (suffix) {
+        return type.symbol.name.endsWith(suffix);
+    })) {
+        return true;
+    }
+    // NOTE: Check the base type
+    var baseTypes = type.getBaseTypes() || [];
+    return baseTypes.some(function (baseType) { return isConstructOrStackType(baseType); });
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-cdk-plugin",
-  "version": "0.1.0",
+  "version": "0.4.0",
   "description": "eslint plugin for AWS CDK projects",
   "main": "dist/index.mjs",
   "type": "module",
@@ -45,10 +45,10 @@
     "aws",
     "cdk"
   ],
-  "homepage": "https://eslint-plugin-cdk.dev/",
+  "homepage": "https://eslint-cdk-plugin.dev/",
   "repository": {
     "type": "git",
-    "url": "https://github.com/ren-yamanashi/eslint-plugin-cdk.git"
+    "url": "https://github.com/ren-yamanashi/eslint-cdk-plugin.git"
   },
   "engines": {
     "node": "^18.18.0 || ^20.9.0 || >=21.1.0"