@aiready/pattern-detect 0.17.13 → 0.17.15

package/dist/chunk-ATXO4JL7.mjs

@@ -0,0 +1,404 @@
+// src/context-rules.ts
+import {
+  IssueType,
+  getSeverityLabel,
+  filterBySeverity,
+  Severity as Severity5
+} from "@aiready/core";
+
+// src/rules/categories/test-rules.ts
+import { Severity } from "@aiready/core";
+var TEST_RULES = [
+  // Test Fixtures - Intentional duplication for test isolation
+  {
+    name: "test-fixtures",
+    detect: (file, code) => {
+      const isTestFile = file.includes(".test.") || file.includes(".spec.") || file.includes("__tests__") || file.includes("/test/") || file.includes("/tests/");
+      const hasTestFixtures = code.includes("beforeAll") || code.includes("afterAll") || code.includes("beforeEach") || code.includes("afterEach") || code.includes("setUp") || code.includes("tearDown");
+      return isTestFile && hasTestFixtures;
+    },
+    severity: Severity.Info,
+    reason: "Test fixture duplication is intentional for test isolation",
+    suggestion: "Consider if shared test setup would improve maintainability without coupling tests"
+  },
+  // E2E/Integration Test Page Objects - Test independence
+  {
+    name: "e2e-page-objects",
+    detect: (file, code) => {
+      const isE2ETest = file.includes("e2e/") || file.includes("/e2e/") || file.includes(".e2e.") || file.includes("/playwright/") || file.includes("playwright/") || file.includes("/cypress/") || file.includes("cypress/") || file.includes("/integration/") || file.includes("integration/");
+      const hasPageObjectPatterns = code.includes("page.") || code.includes("await page") || code.includes("locator") || code.includes("getBy") || code.includes("selector") || code.includes("click(") || code.includes("fill(");
+      return isE2ETest && hasPageObjectPatterns;
+    },
+    severity: Severity.Info,
+    reason: "E2E test duplication ensures test independence and reduces coupling",
+    suggestion: "Consider page object pattern only if duplication causes maintenance issues"
+  },
+  // Mock Data - Test data intentionally duplicated
+  {
+    name: "mock-data",
+    detect: (file, code) => {
+      const isMockFile = file.includes("/mocks/") || file.includes("/__mocks__/") || file.includes("/fixtures/") || file.includes(".mock.") || file.includes(".fixture.");
+      const hasMockData = code.includes("mock") || code.includes("Mock") || code.includes("fixture") || code.includes("stub") || code.includes("export const");
+      return isMockFile && hasMockData;
+    },
+    severity: Severity.Info,
+    reason: "Mock data duplication is expected for comprehensive test coverage",
+    suggestion: "Consider shared factories only for complex mock generation"
+  }
+];
+
+// src/rules/categories/web-rules.ts
+import { Severity as Severity2 } from "@aiready/core";
+var WEB_RULES = [
+  // Email/Document Templates - Often intentionally similar for consistency
+  {
+    name: "templates",
+    detect: (file, code) => {
+      const isTemplate = file.includes("/templates/") || file.includes("-template") || file.includes("/email-templates/") || file.includes("/emails/");
+      const hasTemplateContent = (code.includes("return") || code.includes("export")) && (code.includes("html") || code.includes("subject") || code.includes("body"));
+      return isTemplate && hasTemplateContent;
+    },
+    severity: Severity2.Info,
+    reason: "Template duplication may be intentional for maintainability and branding consistency",
+    suggestion: "Extract shared structure only if templates become hard to maintain"
+  },
+  // Common UI Event Handlers - Very specific patterns only
+  {
+    name: "common-ui-handlers",
+    detect: (file, code) => {
+      const isUIFile = file.includes("/components/") || file.includes(".tsx") || file.includes(".jsx") || file.includes("/hooks/");
+      const hasCommonHandler = code.includes("handleClickOutside") && code.includes("dropdownRef.current") && code.includes("!dropdownRef.current.contains") || code.includes("handleEscape") && code.includes("event.key") && code.includes("=== 'Escape'") || code.includes("handleClickInside") && code.includes("event.stopPropagation");
+      return isUIFile && hasCommonHandler;
+    },
+    severity: Severity2.Info,
+    reason: "Common UI event handlers are boilerplate patterns that repeat across components",
+    suggestion: "Consider extracting to shared hooks (useClickOutside, useEscapeKey) only if causing maintenance issues"
+  },
+  // Next.js Route Handler Patterns - Boilerplate API route patterns
+  {
+    name: "nextjs-route-handlers",
+    detect: (file, code) => {
+      const isRouteFile = file.includes("/api/") && (file.endsWith("/route.ts") || file.endsWith("/route.js"));
+      const hasRoutePattern = code.includes("export async function POST") || code.includes("export async function GET") || code.includes("export async function PUT") || code.includes("export async function DELETE") || code.includes("NextResponse.json") || code.includes("NextRequest");
+      return isRouteFile && hasRoutePattern;
+    },
+    severity: Severity2.Info,
+    reason: "Next.js route handlers follow standard patterns and are intentionally similar across endpoints",
+    suggestion: "Route handler duplication is acceptable for API endpoint boilerplate"
+  }
+];
+
+// src/rules/categories/infra-rules.ts
+import { Severity as Severity3 } from "@aiready/core";
+var INFRA_RULES = [
+  // Configuration Files - Often necessarily similar by design
+  {
+    name: "config-files",
+    detect: (file) => {
+      return file.endsWith(".config.ts") || file.endsWith(".config.js") || file.includes("jest.config") || file.includes("vite.config") || file.includes("webpack.config") || file.includes("rollup.config") || file.includes("tsconfig");
+    },
+    severity: Severity3.Info,
+    reason: "Configuration files often have similar structure by design",
+    suggestion: "Consider shared config base only if configurations become hard to maintain"
+  },
+  // Migration Scripts - One-off scripts that are similar by nature
+  {
+    name: "migration-scripts",
+    detect: (file) => {
+      return file.includes("/migrations/") || file.includes("/migrate/") || file.includes(".migration.");
+    },
+    severity: Severity3.Info,
+    reason: "Migration scripts are typically one-off and intentionally similar",
+    suggestion: "Duplication is acceptable for migration scripts"
+  },
+  // Tool Implementations - Structural Boilerplate
+  {
+    name: "tool-implementations",
+    detect: (file, code) => {
+      const isToolFile = file.includes("/tools/") || file.endsWith(".tool.ts") || code.includes("toolDefinitions");
+      const hasToolStructure = code.includes("execute") && (code.includes("try") || code.includes("catch"));
+      return isToolFile && hasToolStructure;
+    },
+    severity: Severity3.Info,
+    reason: "Tool implementations share structural boilerplate but have distinct business logic",
+    suggestion: "Tool duplication is acceptable for boilerplate interface wrappers"
+  },
+  // CLI Command Definitions - Commander.js boilerplate patterns
+  {
+    name: "cli-command-definitions",
+    detect: (file, code) => {
+      const basename = file.split("/").pop() || "";
+      const isCliFile = file.includes("/commands/") || file.includes("/cli/") || file.endsWith(".command.ts") || basename === "cli.ts" || basename === "cli.js" || basename === "cli.tsx" || basename === "cli-action.ts";
+      const hasCommandPattern = (code.includes(".command(") || code.includes("defineCommand")) && (code.includes(".description(") || code.includes(".option(")) && (code.includes(".action(") || code.includes("async"));
+      return isCliFile && hasCommandPattern;
+    },
+    severity: Severity3.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: Severity3.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: Severity3.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
+import { Severity as Severity4 } from "@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: Severity4.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: Severity4.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",
+    detect: (file, code) => {
+      const isIndexFile = file.endsWith("/index.ts") || file.endsWith("/index.js") || file.endsWith("/index.tsx") || file.endsWith("/index.jsx");
+      const lines = code.split("\n").filter((l) => l.trim());
+      if (lines.length === 0) return false;
+      const reExportLines = lines.filter(
+        (l) => /^export\s+(\{[^}]+\}|\*)\s+from\s+/.test(l.trim()) || /^export\s+\*\s+as\s+\w+\s+from\s+/.test(l.trim())
+      ).length;
+      return isIndexFile && reExportLines > 0 && reExportLines / lines.length > 0.5;
+    },
+    severity: Severity4.Info,
+    reason: "Barrel/index files intentionally re-export for API surface consolidation",
+    suggestion: "Re-exports in barrel files are expected and not true duplication"
+  },
+  // Type Definitions - Duplication for type safety and module independence
+  {
+    name: "type-definitions",
+    detect: (file, code) => {
+      const isTypeFile = file.endsWith(".d.ts") || file.includes("/types/");
+      const hasOnlyTypeDefinitions = (code.includes("interface ") || code.includes("type ") || code.includes("enum ")) && !code.includes("function ") && !code.includes("class ") && !code.includes("const ") && !code.includes("let ") && !code.includes("export default");
+      const isInterfaceOnlySnippet = code.trim().startsWith("interface ") && code.includes("{") && code.includes("}") && !code.includes("function ") && !code.includes("const ") && !code.includes("return ");
+      return isTypeFile && hasOnlyTypeDefinitions || isInterfaceOnlySnippet;
+    },
+    severity: Severity4.Info,
+    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: Severity4.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",
+    detect: (file, code) => {
+      const isUtilFile = file.endsWith(".util.ts") || file.endsWith(".helper.ts") || file.endsWith(".utils.ts");
+      const hasUtilPattern = code.includes("function format") || code.includes("function parse") || code.includes("function sanitize") || code.includes("function normalize") || code.includes("function convert");
+      return isUtilFile && hasUtilPattern;
+    },
+    severity: Severity4.Info,
+    reason: "Utility functions in dedicated utility files may be intentionally similar",
+    suggestion: "Consider extracting to shared utilities only if causing significant duplication"
+  },
+  // React/Vue Hooks - Standard patterns
+  {
+    name: "shared-hooks",
+    detect: (file, code) => {
+      const isHookFile = file.includes("/hooks/") || file.endsWith(".hook.ts") || file.endsWith(".hook.tsx");
+      const hasHookPattern = code.includes("function use") || code.includes("export function use") || code.includes("const use") || code.includes("export const use");
+      return isHookFile && hasHookPattern;
+    },
+    severity: Severity4.Info,
+    reason: "Hooks follow standard patterns and are often intentionally similar across components",
+    suggestion: "Consider extracting common hook logic only if hooks become complex"
+  },
+  // Score/Rating Helper Functions - Common threshold patterns
+  {
+    name: "score-helpers",
+    detect: (file, code) => {
+      const isHelperFile = file.includes("/utils/") || file.includes("/helpers/") || file.endsWith(".util.ts");
+      const hasScorePattern = (code.includes("if (score >=") || code.includes("if (score >")) && code.includes("return") && code.includes("'") && code.split("if (score").length >= 3;
+      return isHelperFile && hasScorePattern;
+    },
+    severity: Severity4.Info,
+    reason: "Score/rating helper functions use common threshold patterns that are intentionally similar",
+    suggestion: "Score formatting duplication is acceptable for consistent UI display"
+  },
+  // D3/Canvas Event Handlers - Standard visualization patterns
+  {
+    name: "visualization-handlers",
+    detect: (file, code) => {
+      const isVizFile = file.includes("/visualizer/") || file.includes("/charts/") || file.includes("GraphCanvas") || file.includes("ForceDirected");
+      const hasVizPattern = (code.includes("dragstarted") || code.includes("dragged") || code.includes("dragended")) && (code.includes("simulation") || code.includes("d3.") || code.includes("alphaTarget"));
+      return isVizFile && hasVizPattern;
+    },
+    severity: Severity4.Info,
+    reason: "D3/visualization event handlers follow standard patterns and are intentionally similar",
+    suggestion: "Visualization boilerplate duplication is acceptable for interactive charts"
+  },
+  // Icon/Switch Statement Helpers - Common enum-to-value patterns
+  {
+    name: "switch-helpers",
+    detect: (file, code) => {
+      const hasSwitchPattern = code.includes("switch (") && code.includes("case '") && code.includes("return") && code.split("case ").length >= 4;
+      const hasIconPattern = code.includes("getIcon") || code.includes("getColor") || code.includes("getLabel") || code.includes("getRating");
+      return hasSwitchPattern && hasIconPattern;
+    },
+    severity: Severity4.Info,
+    reason: "Switch statement helpers for enum-to-value mapping are inherently similar",
+    suggestion: "Switch duplication is acceptable for mapping enums to display values"
+  },
+  // Common API/Utility Functions - Legitimate duplication across modules
+  {
+    name: "common-api-functions",
+    detect: (file, code) => {
+      const isApiFile = file.includes("/api/") || file.includes("/lib/") || file.includes("/utils/") || file.endsWith(".ts");
+      const hasCommonApiPattern = code.includes("getStripe") && code.includes("process.env.STRIPE_SECRET_KEY") || code.includes("getUserByEmail") && code.includes("queryItems") || code.includes("updateUser") && code.includes("buildUpdateExpression") || code.includes("listUserRepositories") && code.includes("queryItems") || code.includes("listTeamRepositories") && code.includes("queryItems") || code.includes("getRemediation") && code.includes("queryItems") || code.includes("formatBreakdownKey") && code.includes(".replace(/([A-Z])/g") || code.includes("queryItems") && code.includes("KeyConditionExpression") || code.includes("putItem") && code.includes("createdAt") || code.includes("updateItem") && code.includes("buildUpdateExpression");
+      return isApiFile && hasCommonApiPattern;
+    },
+    severity: Severity4.Info,
+    reason: "Common API/utility functions are legitimately duplicated across modules for clarity and independence",
+    suggestion: "Consider extracting to shared utilities only if causing significant duplication"
+  },
+  // Validation Functions - Inherently similar patterns
+  {
+    name: "validation-functions",
+    detect: (file, code) => {
+      const hasValidationPattern = code.includes("isValid") || code.includes("validate") || code.includes("checkValid") || code.includes("isEmail") || code.includes("isPhone") || code.includes("isUrl") || code.includes("isNumeric") || code.includes("isAlpha") || code.includes("isAlphanumeric") || code.includes("isEmpty") || code.includes("isNotEmpty") || code.includes("isRequired") || code.includes("isOptional");
+      return hasValidationPattern;
+    },
+    severity: Severity4.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: Severity4.Info,
+    reason: "Singleton getter functions follow standard initialization pattern and are intentionally similar",
+    suggestion: "Singleton getters are boilerplate and acceptable duplication for lazy initialization"
+  }
+];
+
+// src/context-rules.ts
+var CONTEXT_RULES = [
+  ...TEST_RULES,
+  ...WEB_RULES,
+  ...INFRA_RULES,
+  ...LOGIC_RULES
+];
+function calculateSeverity(file1, file2, code, similarity, linesOfCode) {
+  for (const rule of CONTEXT_RULES) {
+    if (rule.detect(file1, code) || rule.detect(file2, code)) {
+      return {
+        severity: rule.severity,
+        reason: rule.reason,
+        suggestion: rule.suggestion,
+        matchedRule: rule.name
+      };
+    }
+  }
+  if (similarity >= 0.95 && linesOfCode >= 30) {
+    return {
+      severity: Severity5.Critical,
+      reason: "Large nearly-identical code blocks waste tokens and create maintenance burden",
+      suggestion: "Extract to shared utility module immediately"
+    };
+  } else if (similarity >= 0.95 && linesOfCode >= 15) {
+    return {
+      severity: Severity5.Major,
+      reason: "Nearly identical code should be consolidated",
+      suggestion: "Move to shared utility file"
+    };
+  } else if (similarity >= 0.85) {
+    return {
+      severity: Severity5.Major,
+      reason: "High similarity indicates significant duplication",
+      suggestion: "Extract common logic to shared function"
+    };
+  } else if (similarity >= 0.7) {
+    return {
+      severity: Severity5.Minor,
+      reason: "Moderate similarity detected",
+      suggestion: "Consider extracting shared patterns if code evolves together"
+    };
+  } else {
+    return {
+      severity: Severity5.Minor,
+      reason: "Minor similarity detected",
+      suggestion: "Monitor but refactoring may not be worthwhile"
+    };
+  }
+}
+function getSeverityThreshold(severity) {
+  const thresholds = {
+    [Severity5.Critical]: 0.95,
+    [Severity5.Major]: 0.85,
+    [Severity5.Minor]: 0.5,
+    [Severity5.Info]: 0
+  };
+  return thresholds[severity] || 0;
+}
+
+export {
+  getSeverityLabel,
+  filterBySeverity,
+  calculateSeverity,
+  getSeverityThreshold
+};