@aiready/pattern-detect 0.16.22 → 0.16.23

package/dist/analyzer-entry/index.mjs

@@ -2,9 +2,9 @@ import {
   analyzePatterns,
   generateSummary,
   getSmartDefaults
-} from "../chunk-DNZS4ESD.mjs";
-import "../chunk-VGMM3L3O.mjs";
-import "../chunk-I6ETJC7L.mjs";
+} from "../chunk-SUUZMLPS.mjs";
+import "../chunk-NQBYYWHJ.mjs";
+import "../chunk-J2G742QF.mjs";
 export {
   analyzePatterns,
   generateSummary,

package/dist/chunk-J2G742QF.mjs

@@ -0,0 +1,162 @@
+// src/context-rules.ts
+import {
+  IssueType,
+  getSeverityLabel,
+  filterBySeverity,
+  Severity
+} from "@aiready/core";
+var CONTEXT_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"
+  },
+  // 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: Severity.Minor,
+    reason: "Template duplication may be intentional for maintainability and branding consistency",
+    suggestion: "Extract shared structure only if templates become hard to maintain"
+  },
+  // 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.Minor,
+    reason: "E2E test duplication ensures test independence and reduces coupling",
+    suggestion: "Consider page object pattern only if duplication causes maintenance issues"
+  },
+  // 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: Severity.Minor,
+    reason: "Configuration files often have similar structure by design",
+    suggestion: "Consider shared config base only if configurations become hard to maintain"
+  },
+  // 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 hasTypeDefinitions = code.includes("interface ") || code.includes("type ") || code.includes("enum ");
+      return isTypeFile && hasTypeDefinitions;
+    },
+    severity: Severity.Info,
+    reason: "Type duplication may be intentional for module independence and type safety",
+    suggestion: "Extract to shared types package only if causing maintenance burden"
+  },
+  // 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: Severity.Info,
+    reason: "Migration scripts are typically one-off and intentionally similar",
+    suggestion: "Duplication is acceptable for migration scripts"
+  },
+  // 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"
+  },
+  // 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: Severity.Info,
+    reason: "Tool implementations share structural boilerplate but have distinct business logic",
+    suggestion: "Tool duplication is acceptable for boilerplate interface wrappers"
+  }
+];
+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: Severity.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: Severity.Major,
+      reason: "Nearly identical code should be consolidated",
+      suggestion: "Move to shared utility file"
+    };
+  } else if (similarity >= 0.85) {
+    return {
+      severity: Severity.Major,
+      reason: "High similarity indicates significant duplication",
+      suggestion: "Extract common logic to shared function"
+    };
+  } else if (similarity >= 0.7) {
+    return {
+      severity: Severity.Minor,
+      reason: "Moderate similarity detected",
+      suggestion: "Consider extracting shared patterns if code evolves together"
+    };
+  } else {
+    return {
+      severity: Severity.Minor,
+      reason: "Minor similarity detected",
+      suggestion: "Monitor but refactoring may not be worthwhile"
+    };
+  }
+}
+function getSeverityThreshold(severity) {
+  const thresholds = {
+    [Severity.Critical]: 0.95,
+    [Severity.Major]: 0.85,
+    [Severity.Minor]: 0.5,
+    [Severity.Info]: 0
+  };
+  return thresholds[severity] || 0;
+}
+
+export {
+  IssueType,
+  getSeverityLabel,
+  filterBySeverity,
+  CONTEXT_RULES,
+  calculateSeverity,
+  getSeverityThreshold
+};

package/dist/chunk-J5CW6NYY.mjs

@@ -0,0 +1,64 @@
+import {
+  analyzePatterns
+} from "./chunk-SUUZMLPS.mjs";
+import {
+  calculatePatternScore
+} from "./chunk-WBBO35SC.mjs";
+
+// src/index.ts
+import { ToolRegistry, Severity } from "@aiready/core";
+
+// src/provider.ts
+import {
+  ToolName,
+  SpokeOutputSchema,
+  GLOBAL_SCAN_OPTIONS
+} from "@aiready/core";
+var PatternDetectProvider = {
+  id: ToolName.PatternDetect,
+  alias: ["patterns", "duplicates", "duplication"],
+  async analyze(options) {
+    const results = await analyzePatterns(options);
+    return SpokeOutputSchema.parse({
+      results: results.results,
+      summary: {
+        totalFiles: results.files.length,
+        totalIssues: results.results.reduce(
+          (sum, r) => sum + r.issues.length,
+          0
+        ),
+        duplicates: results.duplicates,
+        // Keep the raw duplicates for score calculation
+        clusters: results.clusters,
+        config: Object.fromEntries(
+          Object.entries(results.config).filter(
+            ([key]) => !GLOBAL_SCAN_OPTIONS.includes(key) || key === "rootDir"
+          )
+        )
+      },
+      metadata: {
+        toolName: ToolName.PatternDetect,
+        version: "0.12.5",
+        timestamp: (/* @__PURE__ */ new Date()).toISOString()
+      }
+    });
+  },
+  score(output, options) {
+    const duplicates = output.summary.duplicates || [];
+    const totalFiles = output.summary.totalFiles || output.results.length;
+    return calculatePatternScore(
+      duplicates,
+      totalFiles,
+      options.costConfig
+    );
+  },
+  defaultWeight: 22
+};
+
+// src/index.ts
+ToolRegistry.register(PatternDetectProvider);
+
+export {
+  PatternDetectProvider,
+  Severity
+};

package/dist/chunk-NQBYYWHJ.mjs

@@ -0,0 +1,143 @@
+import {
+  calculateSeverity
+} from "./chunk-J2G742QF.mjs";
+
+// src/detector.ts
+import {
+  calculateStringSimilarity,
+  calculateHeuristicConfidence,
+  extractCodeBlocks
+} from "@aiready/core";
+
+// src/core/normalizer.ts
+function normalizeCode(code, isPython = false) {
+  if (!code) return "";
+  let normalized = code;
+  if (isPython) {
+    normalized = normalized.replace(/#.*/g, "");
+  } 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().toLowerCase();
+}
+
+// src/detector.ts
+function extractBlocks(file, content) {
+  return extractCodeBlocks(file, content);
+}
+function calculateSimilarity(a, b) {
+  return calculateStringSimilarity(a, b);
+}
+function calculateConfidence(similarity, tokens, lines) {
+  return calculateHeuristicConfidence(similarity, tokens, lines);
+}
+async function detectDuplicatePatterns(fileContents, options) {
+  const {
+    minSimilarity,
+    minLines,
+    streamResults,
+    onProgress,
+    excludePatterns = [],
+    confidenceThreshold = 0,
+    ignoreWhitelist = []
+  } = options;
+  const allBlocks = [];
+  const excludeRegexes = excludePatterns.map((p) => new RegExp(p, "i"));
+  for (const { file, content } of fileContents) {
+    const blocks = extractBlocks(file, content);
+    for (const b of blocks) {
+      if (b.endLine - b.startLine + 1 < minLines) continue;
+      const isExcluded = excludeRegexes.some((regex) => regex.test(b.code));
+      if (isExcluded) continue;
+      allBlocks.push(b);
+    }
+  }
+  const duplicates = [];
+  const totalBlocks = allBlocks.length;
+  let comparisons = 0;
+  const totalComparisons = totalBlocks * (totalBlocks - 1) / 2;
+  if (onProgress) {
+    onProgress(
+      0,
+      totalComparisons,
+      `Starting duplicate detection on ${totalBlocks} blocks...`
+    );
+  }
+  for (let i = 0; i < allBlocks.length; i++) {
+    if (i % 50 === 0 && i > 0) {
+      await new Promise((resolve) => setImmediate(resolve));
+      if (onProgress) {
+        onProgress(
+          comparisons,
+          totalComparisons,
+          `Analyzing blocks (${i}/${totalBlocks})...`
+        );
+      }
+    }
+    const b1 = allBlocks[i];
+    const isPython1 = b1.file.toLowerCase().endsWith(".py");
+    const norm1 = normalizeCode(b1.code, isPython1);
+    for (let j = i + 1; j < allBlocks.length; j++) {
+      comparisons++;
+      const b2 = allBlocks[j];
+      if (b1.file === b2.file) continue;
+      const isWhitelisted = ignoreWhitelist.some((pattern) => {
+        return b1.file.includes(pattern) && b2.file.includes(pattern) || pattern === `${b1.file}::${b2.file}` || pattern === `${b2.file}::${b1.file}`;
+      });
+      if (isWhitelisted) continue;
+      const isPython2 = b2.file.toLowerCase().endsWith(".py");
+      const norm2 = normalizeCode(b2.code, isPython2);
+      const sim = calculateSimilarity(norm1, norm2);
+      if (sim >= minSimilarity) {
+        const confidence = calculateConfidence(
+          sim,
+          b1.tokens,
+          b1.endLine - b1.startLine + 1
+        );
+        if (confidence < confidenceThreshold) continue;
+        const { severity, reason, suggestion, matchedRule } = calculateSeverity(
+          b1.file,
+          b2.file,
+          b1.code,
+          sim,
+          b1.endLine - b1.startLine + 1
+        );
+        const dup = {
+          file1: b1.file,
+          line1: b1.startLine,
+          endLine1: b1.endLine,
+          file2: b2.file,
+          line2: b2.startLine,
+          endLine2: b2.endLine,
+          code1: b1.code,
+          code2: b2.code,
+          similarity: sim,
+          confidence,
+          patternType: b1.patternType,
+          tokenCost: b1.tokens + b2.tokens,
+          severity,
+          reason,
+          suggestion,
+          matchedRule
+        };
+        duplicates.push(dup);
+        if (streamResults)
+          console.log(
+            `[DUPLICATE] ${dup.file1}:${dup.line1} <-> ${dup.file2}:${dup.line2} (${Math.round(sim * 100)}%, conf: ${Math.round(confidence * 100)}%)`
+          );
+      }
+    }
+  }
+  if (onProgress) {
+    onProgress(
+      totalComparisons,
+      totalComparisons,
+      `Duplicate detection complete. Found ${duplicates.length} patterns.`
+    );
+  }
+  return duplicates.sort((a, b) => b.similarity - a.similarity);
+}
+
+export {
+  detectDuplicatePatterns
+};