@aiready/pattern-detect 0.17.14 → 0.17.15

package/dist/context-rules-entry/index.js

@@ -159,12 +159,67 @@ var INFRA_RULES = [
     severity: import_core3.Severity.Info,
     reason: "CLI command definitions follow standard Commander.js patterns and are intentionally similar",
     suggestion: "Command boilerplate duplication is acceptable for CLI interfaces"
+  },
+  // DynamoDB Single-Table Design - Standard single-table patterns with prefixed keys
+  {
+    name: "dynamodb-single-table",
+    detect: (file, code) => {
+      const hasDynamoDBPattern = code.includes("docClient") || code.includes("dynamodb") || code.includes("DynamoDB") || code.includes("queryItems") || code.includes("putItem") || code.includes("getItem") || code.includes("updateItem") || code.includes("deleteItem");
+      const hasKeyPrefix = code.includes("userId:") && code.includes("#") || code.includes("pk:") && code.includes("#") || code.includes("Key:") && code.includes("#") || /[A-Z]+#/.test(code);
+      const hasSingleTablePattern = code.includes("KeyConditionExpression") || code.includes("pk =") || code.includes("sk =") || code.includes("userId") && code.includes("timestamp");
+      return hasDynamoDBPattern && (hasKeyPrefix || hasSingleTablePattern);
+    },
+    severity: import_core3.Severity.Info,
+    reason: "DynamoDB single-table design with prefixed keys is a standard pattern for efficient data access",
+    suggestion: "Single-table query patterns are intentionally similar and should not be refactored"
+  },
+  // CLI Main Function Boilerplate - Standard argument parsing patterns
+  {
+    name: "cli-main-boilerplate",
+    detect: (file, code) => {
+      const basename = file.split("/").pop() || "";
+      const isCliFile = file.includes("/cli/") || file.includes("/commands/") || basename.startsWith("cli") || basename.endsWith(".cli.ts") || basename.endsWith(".cli.js");
+      const hasMainFunction = code.includes("function main()") || code.includes("async function main()") || code.includes("const main =") || code.includes("main()");
+      const hasArgParsing = code.includes("process.argv") || code.includes("yargs") || code.includes("commander") || code.includes("minimist") || code.includes(".parse(") || code.includes("args") && code.includes("._");
+      return isCliFile && hasMainFunction && hasArgParsing;
+    },
+    severity: import_core3.Severity.Info,
+    reason: "CLI main functions with argument parsing follow standard boilerplate patterns",
+    suggestion: "CLI argument parsing boilerplate is acceptable and should not be flagged as duplication"
   }
 ];
 
 // src/rules/categories/logic-rules.ts
 var import_core4 = require("@aiready/core");
 var LOGIC_RULES = [
+  // Enum Semantic Difference - Different enum names indicate different semantic meanings
+  {
+    name: "enum-semantic-difference",
+    detect: (file, code) => {
+      const enumRegex = /(?:export\s+)?(?:const\s+)?enum\s+([A-Z][a-zA-Z0-9]*)/g;
+      const enums = [];
+      let match;
+      while ((match = enumRegex.exec(code)) !== null) {
+        enums.push(match[1]);
+      }
+      return enums.length > 0;
+    },
+    severity: import_core4.Severity.Info,
+    reason: "Enums with different names represent different semantic domain concepts, even if they share similar values",
+    suggestion: "Different enums (e.g., EscalationPriority vs HealthSeverity) serve different purposes and should not be merged"
+  },
+  // Enum Value Similarity - Common enum values like LOW, MEDIUM, HIGH are standard
+  {
+    name: "enum-value-similarity",
+    detect: (file, code) => {
+      const hasCommonEnumValues = (code.includes("LOW = 'low'") || code.includes("LOW = 0") || code.includes("LOW = 'LOW'")) && (code.includes("HIGH = 'high'") || code.includes("HIGH = 2") || code.includes("HIGH = 'HIGH'")) && (code.includes("MEDIUM = 'medium'") || code.includes("MEDIUM = 1") || code.includes("MEDIUM = 'MEDIUM'"));
+      const isEnumDefinition = /(?:export\s+)?(?:const\s+)?enum\s+/.test(code) || code.includes("enum ") && code.includes("{") && code.includes("}");
+      return hasCommonEnumValues && isEnumDefinition;
+    },
+    severity: import_core4.Severity.Info,
+    reason: "Common enum values (LOW, MEDIUM, HIGH, CRITICAL) are standard patterns used across different domain enums",
+    suggestion: "Enum value similarity is expected for severity/priority enums and should not be flagged as duplication"
+  },
   // Re-export / Barrel files - Intentional API surface consolidation
   {
     name: "re-export-files",
@@ -194,6 +249,20 @@ var LOGIC_RULES = [
     reason: "Type/interface definitions are intentionally duplicated for module independence",
     suggestion: "Extract to shared types package only if causing maintenance burden"
   },
+  // Cross-Package Type Definitions - Different packages may have similar types
+  {
+    name: "cross-package-types",
+    detect: (file, code) => {
+      const hasTypeDefinition = code.includes("interface ") || code.includes("type ") || code.includes("enum ");
+      const isPackageOrApp = file.includes("/packages/") || file.includes("/apps/") || file.includes("/core/");
+      const packageMatch = file.match(/\/(packages|apps|core)\/([^/]+)\//);
+      const hasPackageStructure = packageMatch !== null;
+      return hasTypeDefinition && isPackageOrApp && hasPackageStructure;
+    },
+    severity: import_core4.Severity.Info,
+    reason: "Types in different packages/modules are often intentionally similar for module independence",
+    suggestion: "Cross-package type duplication is acceptable for decoupled module architecture"
+  },
   // Utility Functions - Small helpers in dedicated utility files
   {
     name: "utility-functions",
@@ -276,6 +345,22 @@ var LOGIC_RULES = [
     severity: import_core4.Severity.Info,
     reason: "Validation functions are inherently similar and often intentionally duplicated for domain clarity",
     suggestion: "Consider extracting to shared validators only if validation logic becomes complex"
+  },
+  // Singleton Getter Pattern - Standard singleton initialization pattern
+  {
+    name: "singleton-getter",
+    detect: (file, code) => {
+      const hasSingletonGetter = /(?:export\s+)?(?:async\s+)?function\s+get[A-Z][a-zA-Z0-9]*\s*\(/.test(
+        code
+      ) || /(?:export\s+)?const\s+get[A-Z][a-zA-Z0-9]*\s*=\s*(?:async\s+)?\(\)\s*=>/.test(
+        code
+      );
+      const hasSingletonPattern = code.includes("if (!") && code.includes("instance") && code.includes(" = ") || code.includes("if (!_") && code.includes(" = new ") || code.includes("if (") && code.includes(" === null") && code.includes(" = new ");
+      return hasSingletonGetter && hasSingletonPattern;
+    },
+    severity: import_core4.Severity.Info,
+    reason: "Singleton getter functions follow standard initialization pattern and are intentionally similar",
+    suggestion: "Singleton getters are boilerplate and acceptable duplication for lazy initialization"
   }
 ];
 

package/dist/context-rules-entry/index.mjs

@@ -3,7 +3,7 @@ import {
   filterBySeverity,
   getSeverityLabel,
   getSeverityThreshold
-} from "../chunk-UKQFCUQA.mjs";
+} from "../chunk-KDXWIT6W.mjs";
 export {
   calculateSeverity,
   filterBySeverity,

package/dist/detector-entry/index.js

@@ -159,12 +159,67 @@ var INFRA_RULES = [
     severity: import_core3.Severity.Info,
     reason: "CLI command definitions follow standard Commander.js patterns and are intentionally similar",
     suggestion: "Command boilerplate duplication is acceptable for CLI interfaces"
+  },
+  // DynamoDB Single-Table Design - Standard single-table patterns with prefixed keys
+  {
+    name: "dynamodb-single-table",
+    detect: (file, code) => {
+      const hasDynamoDBPattern = code.includes("docClient") || code.includes("dynamodb") || code.includes("DynamoDB") || code.includes("queryItems") || code.includes("putItem") || code.includes("getItem") || code.includes("updateItem") || code.includes("deleteItem");
+      const hasKeyPrefix = code.includes("userId:") && code.includes("#") || code.includes("pk:") && code.includes("#") || code.includes("Key:") && code.includes("#") || /[A-Z]+#/.test(code);
+      const hasSingleTablePattern = code.includes("KeyConditionExpression") || code.includes("pk =") || code.includes("sk =") || code.includes("userId") && code.includes("timestamp");
+      return hasDynamoDBPattern && (hasKeyPrefix || hasSingleTablePattern);
+    },
+    severity: import_core3.Severity.Info,
+    reason: "DynamoDB single-table design with prefixed keys is a standard pattern for efficient data access",
+    suggestion: "Single-table query patterns are intentionally similar and should not be refactored"
+  },
+  // CLI Main Function Boilerplate - Standard argument parsing patterns
+  {
+    name: "cli-main-boilerplate",
+    detect: (file, code) => {
+      const basename = file.split("/").pop() || "";
+      const isCliFile = file.includes("/cli/") || file.includes("/commands/") || basename.startsWith("cli") || basename.endsWith(".cli.ts") || basename.endsWith(".cli.js");
+      const hasMainFunction = code.includes("function main()") || code.includes("async function main()") || code.includes("const main =") || code.includes("main()");
+      const hasArgParsing = code.includes("process.argv") || code.includes("yargs") || code.includes("commander") || code.includes("minimist") || code.includes(".parse(") || code.includes("args") && code.includes("._");
+      return isCliFile && hasMainFunction && hasArgParsing;
+    },
+    severity: import_core3.Severity.Info,
+    reason: "CLI main functions with argument parsing follow standard boilerplate patterns",
+    suggestion: "CLI argument parsing boilerplate is acceptable and should not be flagged as duplication"
   }
 ];
 
 // src/rules/categories/logic-rules.ts
 var import_core4 = require("@aiready/core");
 var LOGIC_RULES = [
+  // Enum Semantic Difference - Different enum names indicate different semantic meanings
+  {
+    name: "enum-semantic-difference",
+    detect: (file, code) => {
+      const enumRegex = /(?:export\s+)?(?:const\s+)?enum\s+([A-Z][a-zA-Z0-9]*)/g;
+      const enums = [];
+      let match;
+      while ((match = enumRegex.exec(code)) !== null) {
+        enums.push(match[1]);
+      }
+      return enums.length > 0;
+    },
+    severity: import_core4.Severity.Info,
+    reason: "Enums with different names represent different semantic domain concepts, even if they share similar values",
+    suggestion: "Different enums (e.g., EscalationPriority vs HealthSeverity) serve different purposes and should not be merged"
+  },
+  // Enum Value Similarity - Common enum values like LOW, MEDIUM, HIGH are standard
+  {
+    name: "enum-value-similarity",
+    detect: (file, code) => {
+      const hasCommonEnumValues = (code.includes("LOW = 'low'") || code.includes("LOW = 0") || code.includes("LOW = 'LOW'")) && (code.includes("HIGH = 'high'") || code.includes("HIGH = 2") || code.includes("HIGH = 'HIGH'")) && (code.includes("MEDIUM = 'medium'") || code.includes("MEDIUM = 1") || code.includes("MEDIUM = 'MEDIUM'"));
+      const isEnumDefinition = /(?:export\s+)?(?:const\s+)?enum\s+/.test(code) || code.includes("enum ") && code.includes("{") && code.includes("}");
+      return hasCommonEnumValues && isEnumDefinition;
+    },
+    severity: import_core4.Severity.Info,
+    reason: "Common enum values (LOW, MEDIUM, HIGH, CRITICAL) are standard patterns used across different domain enums",
+    suggestion: "Enum value similarity is expected for severity/priority enums and should not be flagged as duplication"
+  },
   // Re-export / Barrel files - Intentional API surface consolidation
   {
     name: "re-export-files",
@@ -194,6 +249,20 @@ var LOGIC_RULES = [
     reason: "Type/interface definitions are intentionally duplicated for module independence",
     suggestion: "Extract to shared types package only if causing maintenance burden"
   },
+  // Cross-Package Type Definitions - Different packages may have similar types
+  {
+    name: "cross-package-types",
+    detect: (file, code) => {
+      const hasTypeDefinition = code.includes("interface ") || code.includes("type ") || code.includes("enum ");
+      const isPackageOrApp = file.includes("/packages/") || file.includes("/apps/") || file.includes("/core/");
+      const packageMatch = file.match(/\/(packages|apps|core)\/([^/]+)\//);
+      const hasPackageStructure = packageMatch !== null;
+      return hasTypeDefinition && isPackageOrApp && hasPackageStructure;
+    },
+    severity: import_core4.Severity.Info,
+    reason: "Types in different packages/modules are often intentionally similar for module independence",
+    suggestion: "Cross-package type duplication is acceptable for decoupled module architecture"
+  },
   // Utility Functions - Small helpers in dedicated utility files
   {
     name: "utility-functions",
@@ -276,6 +345,22 @@ var LOGIC_RULES = [
     severity: import_core4.Severity.Info,
     reason: "Validation functions are inherently similar and often intentionally duplicated for domain clarity",
     suggestion: "Consider extracting to shared validators only if validation logic becomes complex"
+  },
+  // Singleton Getter Pattern - Standard singleton initialization pattern
+  {
+    name: "singleton-getter",
+    detect: (file, code) => {
+      const hasSingletonGetter = /(?:export\s+)?(?:async\s+)?function\s+get[A-Z][a-zA-Z0-9]*\s*\(/.test(
+        code
+      ) || /(?:export\s+)?const\s+get[A-Z][a-zA-Z0-9]*\s*=\s*(?:async\s+)?\(\)\s*=>/.test(
+        code
+      );
+      const hasSingletonPattern = code.includes("if (!") && code.includes("instance") && code.includes(" = ") || code.includes("if (!_") && code.includes(" = new ") || code.includes("if (") && code.includes(" === null") && code.includes(" = new ");
+      return hasSingletonGetter && hasSingletonPattern;
+    },
+    severity: import_core4.Severity.Info,
+    reason: "Singleton getter functions follow standard initialization pattern and are intentionally similar",
+    suggestion: "Singleton getters are boilerplate and acceptable duplication for lazy initialization"
   }
 ];
 

package/dist/detector-entry/index.mjs

@@ -1,7 +1,7 @@
 import {
   detectDuplicatePatterns
-} from "../chunk-RH5JPWEC.mjs";
-import "../chunk-UKQFCUQA.mjs";
+} from "../chunk-JWP5TCDM.mjs";
+import "../chunk-KDXWIT6W.mjs";
 export {
   detectDuplicatePatterns
 };

package/dist/index.js

@@ -191,12 +191,67 @@ var INFRA_RULES = [
     severity: import_core3.Severity.Info,
     reason: "CLI command definitions follow standard Commander.js patterns and are intentionally similar",
     suggestion: "Command boilerplate duplication is acceptable for CLI interfaces"
+  },
+  // DynamoDB Single-Table Design - Standard single-table patterns with prefixed keys
+  {
+    name: "dynamodb-single-table",
+    detect: (file, code) => {
+      const hasDynamoDBPattern = code.includes("docClient") || code.includes("dynamodb") || code.includes("DynamoDB") || code.includes("queryItems") || code.includes("putItem") || code.includes("getItem") || code.includes("updateItem") || code.includes("deleteItem");
+      const hasKeyPrefix = code.includes("userId:") && code.includes("#") || code.includes("pk:") && code.includes("#") || code.includes("Key:") && code.includes("#") || /[A-Z]+#/.test(code);
+      const hasSingleTablePattern = code.includes("KeyConditionExpression") || code.includes("pk =") || code.includes("sk =") || code.includes("userId") && code.includes("timestamp");
+      return hasDynamoDBPattern && (hasKeyPrefix || hasSingleTablePattern);
+    },
+    severity: import_core3.Severity.Info,
+    reason: "DynamoDB single-table design with prefixed keys is a standard pattern for efficient data access",
+    suggestion: "Single-table query patterns are intentionally similar and should not be refactored"
+  },
+  // CLI Main Function Boilerplate - Standard argument parsing patterns
+  {
+    name: "cli-main-boilerplate",
+    detect: (file, code) => {
+      const basename = file.split("/").pop() || "";
+      const isCliFile = file.includes("/cli/") || file.includes("/commands/") || basename.startsWith("cli") || basename.endsWith(".cli.ts") || basename.endsWith(".cli.js");
+      const hasMainFunction = code.includes("function main()") || code.includes("async function main()") || code.includes("const main =") || code.includes("main()");
+      const hasArgParsing = code.includes("process.argv") || code.includes("yargs") || code.includes("commander") || code.includes("minimist") || code.includes(".parse(") || code.includes("args") && code.includes("._");
+      return isCliFile && hasMainFunction && hasArgParsing;
+    },
+    severity: import_core3.Severity.Info,
+    reason: "CLI main functions with argument parsing follow standard boilerplate patterns",
+    suggestion: "CLI argument parsing boilerplate is acceptable and should not be flagged as duplication"
   }
 ];
 
 // src/rules/categories/logic-rules.ts
 var import_core4 = require("@aiready/core");
 var LOGIC_RULES = [
+  // Enum Semantic Difference - Different enum names indicate different semantic meanings
+  {
+    name: "enum-semantic-difference",
+    detect: (file, code) => {
+      const enumRegex = /(?:export\s+)?(?:const\s+)?enum\s+([A-Z][a-zA-Z0-9]*)/g;
+      const enums = [];
+      let match;
+      while ((match = enumRegex.exec(code)) !== null) {
+        enums.push(match[1]);
+      }
+      return enums.length > 0;
+    },
+    severity: import_core4.Severity.Info,
+    reason: "Enums with different names represent different semantic domain concepts, even if they share similar values",
+    suggestion: "Different enums (e.g., EscalationPriority vs HealthSeverity) serve different purposes and should not be merged"
+  },
+  // Enum Value Similarity - Common enum values like LOW, MEDIUM, HIGH are standard
+  {
+    name: "enum-value-similarity",
+    detect: (file, code) => {
+      const hasCommonEnumValues = (code.includes("LOW = 'low'") || code.includes("LOW = 0") || code.includes("LOW = 'LOW'")) && (code.includes("HIGH = 'high'") || code.includes("HIGH = 2") || code.includes("HIGH = 'HIGH'")) && (code.includes("MEDIUM = 'medium'") || code.includes("MEDIUM = 1") || code.includes("MEDIUM = 'MEDIUM'"));
+      const isEnumDefinition = /(?:export\s+)?(?:const\s+)?enum\s+/.test(code) || code.includes("enum ") && code.includes("{") && code.includes("}");
+      return hasCommonEnumValues && isEnumDefinition;
+    },
+    severity: import_core4.Severity.Info,
+    reason: "Common enum values (LOW, MEDIUM, HIGH, CRITICAL) are standard patterns used across different domain enums",
+    suggestion: "Enum value similarity is expected for severity/priority enums and should not be flagged as duplication"
+  },
   // Re-export / Barrel files - Intentional API surface consolidation
   {
     name: "re-export-files",
@@ -226,6 +281,20 @@ var LOGIC_RULES = [
     reason: "Type/interface definitions are intentionally duplicated for module independence",
     suggestion: "Extract to shared types package only if causing maintenance burden"
   },
+  // Cross-Package Type Definitions - Different packages may have similar types
+  {
+    name: "cross-package-types",
+    detect: (file, code) => {
+      const hasTypeDefinition = code.includes("interface ") || code.includes("type ") || code.includes("enum ");
+      const isPackageOrApp = file.includes("/packages/") || file.includes("/apps/") || file.includes("/core/");
+      const packageMatch = file.match(/\/(packages|apps|core)\/([^/]+)\//);
+      const hasPackageStructure = packageMatch !== null;
+      return hasTypeDefinition && isPackageOrApp && hasPackageStructure;
+    },
+    severity: import_core4.Severity.Info,
+    reason: "Types in different packages/modules are often intentionally similar for module independence",
+    suggestion: "Cross-package type duplication is acceptable for decoupled module architecture"
+  },
   // Utility Functions - Small helpers in dedicated utility files
   {
     name: "utility-functions",
@@ -308,6 +377,22 @@ var LOGIC_RULES = [
     severity: import_core4.Severity.Info,
     reason: "Validation functions are inherently similar and often intentionally duplicated for domain clarity",
     suggestion: "Consider extracting to shared validators only if validation logic becomes complex"
+  },
+  // Singleton Getter Pattern - Standard singleton initialization pattern
+  {
+    name: "singleton-getter",
+    detect: (file, code) => {
+      const hasSingletonGetter = /(?:export\s+)?(?:async\s+)?function\s+get[A-Z][a-zA-Z0-9]*\s*\(/.test(
+        code
+      ) || /(?:export\s+)?const\s+get[A-Z][a-zA-Z0-9]*\s*=\s*(?:async\s+)?\(\)\s*=>/.test(
+        code
+      );
+      const hasSingletonPattern = code.includes("if (!") && code.includes("instance") && code.includes(" = ") || code.includes("if (!_") && code.includes(" = new ") || code.includes("if (") && code.includes(" === null") && code.includes(" = new ");
+      return hasSingletonGetter && hasSingletonPattern;
+    },
+    severity: import_core4.Severity.Info,
+    reason: "Singleton getter functions follow standard initialization pattern and are intentionally similar",
+    suggestion: "Singleton getters are boilerplate and acceptable duplication for lazy initialization"
   }
 ];
 

package/dist/index.mjs

@@ -12,16 +12,16 @@ import {
   getSmartDefaults,
   groupDuplicatesByFilePair,
   logConfiguration
-} from "./chunk-C4ZGC4KA.mjs";
+} from "./chunk-WQX7IHAN.mjs";
 import {
   detectDuplicatePatterns
-} from "./chunk-RH5JPWEC.mjs";
+} from "./chunk-JWP5TCDM.mjs";
+import {
+  filterBySeverity
+} from "./chunk-KDXWIT6W.mjs";
 import {
   calculatePatternScore
 } from "./chunk-G3GZFYRI.mjs";
-import {
-  filterBySeverity
-} from "./chunk-UKQFCUQA.mjs";
 
 // src/index.ts
 import { ToolRegistry } from "@aiready/core";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aiready/pattern-detect",
-  "version": "0.17.14",
+  "version": "0.17.15",
   "description": "Semantic duplicate pattern detection for AI-generated code - finds similar implementations that waste AI context tokens",
   "main": "./dist/index.js",
   "module": "./dist/index.mjs",
@@ -56,16 +56,16 @@
   "license": "MIT",
   "repository": {
     "type": "git",
-    "url": "https://github.com/caopengau/aiready-pattern-detect.git"
+    "url": "https://github.com/getaiready/aiready-pattern-detect.git"
   },
-  "homepage": "https://github.com/caopengau/aiready-pattern-detect",
+  "homepage": "https://github.com/getaiready/aiready-pattern-detect",
   "bugs": {
-    "url": "https://github.com/caopengau/aiready-pattern-detect/issues"
+    "url": "https://github.com/getaiready/aiready-pattern-detect/issues"
   },
   "dependencies": {
     "commander": "^14.0.0",
     "chalk": "^5.3.0",
-    "@aiready/core": "0.24.15"
+    "@aiready/core": "0.24.16"
   },
   "devDependencies": {
     "tsup": "^8.3.5",