@aiready/pattern-detect 0.17.8 → 0.17.12

package/dist/analyzer-entry/index.mjs

@@ -2,9 +2,10 @@ import {
   analyzePatterns,
   generateSummary,
   getSmartDefaults
-} from "../chunk-K7BO57OO.mjs";
-import "../chunk-NQBYYWHJ.mjs";
-import "../chunk-J2G742QF.mjs";
+} from "../chunk-P3BOCGVV.mjs";
+import "../chunk-IPBGVPUX.mjs";
+import "../chunk-2P7BQHGR.mjs";
+import "../chunk-PHJE6A3J.mjs";
 export {
   analyzePatterns,
   generateSummary,

package/dist/chunk-2P7BQHGR.mjs

@@ -0,0 +1,306 @@
+// 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 isCliFile = file.includes("/commands/") || file.includes("/cli/") || file.endsWith(".command.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"
+  }
+];
+
+// src/rules/categories/logic-rules.ts
+import { Severity as Severity4 } from "@aiready/core";
+var LOGIC_RULES = [
+  // 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"
+  },
+  // 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"
+  }
+];
+
+// 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
+};

package/dist/{chunk-VGMM3L3O.mjs → chunk-3EORD7DC.mjs}

@@ -1,6 +1,6 @@
 import {
   calculateSeverity
-} from "./chunk-I6ETJC7L.mjs";
+} from "./chunk-AQIP4JGM.mjs";
 
 // src/detector.ts
 import {

package/dist/{chunk-GREN7X5H.mjs → chunk-4PVPQMRT.mjs}

@@ -1,6 +1,6 @@
 import {
   calculateSeverity
-} from "./chunk-I6ETJC7L.mjs";
+} from "./chunk-X553BOMI.mjs";
 
 // src/detector.ts
 import {
@@ -18,7 +18,7 @@ function normalizeCode(code, isPython = false) {
   } else {
     normalized = normalized.replace(/\/\/.*$/gm, "").replace(/\/\*[\s\S]*?\*\//g, "");
   }
-  return normalized.replace(/"[^"]*"/g, '"STR"').replace(/'[^']*'/g, "'STR'").replace(/`[^`]*`/g, "`STR`").replace(/\b\d+\b/g, "NUM").replace(/\s+/g, " ").trim();
+  return normalized.replace(/"[^"]*"/g, '"STR"').replace(/'[^']*'/g, "'STR'").replace(/`[^`]*`/g, "`STR`").replace(/\b\d+\b/g, "NUM").replace(/\s+/g, " ").trim().toLowerCase();
 }
 
 // src/detector.ts

package/dist/{chunk-RS73WLNI.mjs → chunk-6VDL7TAS.mjs}

@@ -1,12 +1,12 @@
 import {
   calculateSeverity
-} from "./chunk-I6ETJC7L.mjs";
+} from "./chunk-PQXOORR4.mjs";
 
 // src/detector.ts
 import {
-  estimateTokens,
   calculateStringSimilarity,
-  calculateHeuristicConfidence
+  calculateHeuristicConfidence,
+  extractCodeBlocks
 } from "@aiready/core";
 
 // src/core/normalizer.ts
@@ -18,120 +18,12 @@ function normalizeCode(code, isPython = false) {
   } else {
     normalized = normalized.replace(/\/\/.*$/gm, "").replace(/\/\*[\s\S]*?\*\//g, "");
   }
-  return normalized.replace(/"[^"]*"/g, '"STR"').replace(/'[^']*'/g, "'STR'").replace(/`[^`]*`/g, "`STR`").replace(/\b\d+\b/g, "NUM").replace(/\s+/g, " ").trim();
+  return normalized.replace(/"[^"]*"/g, '"STR"').replace(/'[^']*'/g, "'STR'").replace(/`[^`]*`/g, "`STR`").replace(/\b\d+\b/g, "NUM").replace(/\s+/g, " ").trim().toLowerCase();
 }
 
 // src/detector.ts
 function extractBlocks(file, content) {
-  const isPython = file.toLowerCase().endsWith(".py");
-  if (isPython) {
-    return extractBlocksPython(file, content);
-  }
-  const blocks = [];
-  const lines = content.split("\n");
-  const blockRegex = /^\s*(?:export\s+)?(?:async\s+)?(?:public\s+|private\s+|protected\s+|internal\s+|static\s+|readonly\s+|virtual\s+|abstract\s+|override\s+)*(function|class|interface|type|enum|record|struct|void|func|[a-zA-Z0-9_<>[]]+)\s+([a-zA-Z0-9_]+)(?:\s*\(|(?:\s+extends|\s+implements|\s+where)?\s*\{)|^\s*(?:export\s+)?const\s+([a-zA-Z0-9_]+)\s*=\s*[a-zA-Z0-9_.]+\.object\(|^\s*(app\.(?:get|post|put|delete|patch|use))\(/gm;
-  let match;
-  while ((match = blockRegex.exec(content)) !== null) {
-    const startLine = content.substring(0, match.index).split("\n").length;
-    let type;
-    let name;
-    if (match[1]) {
-      type = match[1];
-      name = match[2];
-    } else if (match[3]) {
-      type = "const";
-      name = match[3];
-    } else {
-      type = "handler";
-      name = match[4];
-    }
-    let endLine = -1;
-    let openBraces = 0;
-    let foundStart = false;
-    for (let i = match.index; i < content.length; i++) {
-      if (content[i] === "{") {
-        openBraces++;
-        foundStart = true;
-      } else if (content[i] === "}") {
-        openBraces--;
-      }
-      if (foundStart && openBraces === 0) {
-        endLine = content.substring(0, i + 1).split("\n").length;
-        break;
-      }
-    }
-    if (endLine === -1) {
-      const remaining = content.slice(match.index);
-      const nextLineMatch = remaining.indexOf("\n");
-      if (nextLineMatch !== -1) {
-        endLine = startLine;
-      } else {
-        endLine = lines.length;
-      }
-    }
-    endLine = Math.max(startLine, endLine);
-    const blockCode = lines.slice(startLine - 1, endLine).join("\n");
-    const tokens = estimateTokens(blockCode);
-    blocks.push({
-      file,
-      startLine,
-      endLine,
-      code: blockCode,
-      tokens,
-      patternType: inferPatternType(type, name)
-    });
-  }
-  return blocks;
-}
-function extractBlocksPython(file, content) {
-  const blocks = [];
-  const lines = content.split("\n");
-  const blockRegex = /^\s*(?:async\s+)?(def|class)\s+([a-zA-Z0-9_]+)/gm;
-  let match;
-  while ((match = blockRegex.exec(content)) !== null) {
-    const startLinePos = content.substring(0, match.index).split("\n").length;
-    const startLineIdx = startLinePos - 1;
-    const initialIndent = lines[startLineIdx].search(/\S/);
-    let endLineIdx = startLineIdx;
-    for (let i = startLineIdx + 1; i < lines.length; i++) {
-      const line = lines[i];
-      if (line.trim().length === 0) {
-        endLineIdx = i;
-        continue;
-      }
-      const currentIndent = line.search(/\S/);
-      if (currentIndent <= initialIndent) {
-        break;
-      }
-      endLineIdx = i;
-    }
-    while (endLineIdx > startLineIdx && lines[endLineIdx].trim().length === 0) {
-      endLineIdx--;
-    }
-    const blockCode = lines.slice(startLineIdx, endLineIdx + 1).join("\n");
-    const tokens = estimateTokens(blockCode);
-    blocks.push({
-      file,
-      startLine: startLinePos,
-      endLine: endLineIdx + 1,
-      code: blockCode,
-      tokens,
-      patternType: inferPatternType(match[1], match[2])
-    });
-  }
-  return blocks;
-}
-function inferPatternType(keyword, name) {
-  const n = name.toLowerCase();
-  if (keyword === "handler" || n.includes("handler") || n.includes("controller") || n.startsWith("app.")) {
-    return "api-handler";
-  }
-  if (n.includes("validate") || n.includes("schema")) return "validator";
-  if (n.includes("util") || n.includes("helper")) return "utility";
-  if (keyword === "class") return "class-method";
-  if (n.match(/^[A-Z]/)) return "component";
-  if (keyword === "function") return "function";
-  return "unknown";
+  return extractCodeBlocks(file, content);
 }
 function calculateSimilarity(a, b) {
   return calculateStringSimilarity(a, b);