@aiready/consistency 0.20.19 → 0.20.21

package/dist/cli.js

@@ -27,7 +27,7 @@ var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__ge
 var import_commander = require("commander");
 
 // src/analyzer.ts
-var import_core5 = require("@aiready/core");
+var import_core6 = require("@aiready/core");
 
 // src/analyzers/naming-ast.ts
 var import_core2 = require("@aiready/core");
@@ -311,12 +311,10 @@ function analyzeIdentifiers(ast, filePath, context) {
               { isParameter: true, isArrowParameter }
             );
           } else if (param.type === "ObjectPattern") {
-            extractDestructuredIdentifiers(
-              param,
-              true,
-              scopeTracker,
+            extractDestructuredIdentifiers(param, scopeTracker, {
+              isParameter: true,
               isArrowParameter
-            );
+            });
           }
         });
       }
@@ -463,7 +461,8 @@ var ScopeTracker = class {
     return this.variables;
   }
 };
-function extractDestructuredIdentifiers(node, isParameter, scopeTracker, isArrowParameter = false) {
+function extractDestructuredIdentifiers(node, scopeTracker, options = {}) {
+  const { isParameter = false, isArrowParameter = false } = options;
   if (node.type === "ObjectPattern") {
     node.properties.forEach((prop) => {
       if (prop.type === "Property" && prop.value.type === "Identifier") {
@@ -541,7 +540,7 @@ var COMMON_ABBREVIATIONS = /* @__PURE__ */ new Set([
 async function analyzeNamingGeneralized(files) {
   const issues = [];
   for (const file of files) {
-    const parser = (0, import_core3.getParser)(file);
+    const parser = await (0, import_core3.getParser)(file);
     if (!parser) continue;
     try {
       const code = (0, import_fs2.readFileSync)(file, "utf-8");
@@ -683,6 +682,102 @@ async function analyzePatterns(filePaths) {
   return issues;
 }
 
+// src/scoring.ts
+var import_core5 = require("@aiready/core");
+function calculateConsistencyScore(issues, totalFilesAnalyzed, costConfig) {
+  void costConfig;
+  const criticalIssues = issues.filter((i) => i.severity === "critical").length;
+  const majorIssues = issues.filter((i) => i.severity === "major").length;
+  const minorIssues = issues.filter((i) => i.severity === "minor").length;
+  const totalIssues = issues.length;
+  const issuesPerFile = totalFilesAnalyzed > 0 ? totalIssues / totalFilesAnalyzed : 0;
+  const densityPenalty = Math.min(50, issuesPerFile * 15);
+  const weightedCount = criticalIssues * 10 + majorIssues * 3 + minorIssues * 0.5;
+  const avgWeightedIssuesPerFile = totalFilesAnalyzed > 0 ? weightedCount / totalFilesAnalyzed : 0;
+  const severityPenalty = Math.min(50, avgWeightedIssuesPerFile * 2);
+  const rawScore = 100 - densityPenalty - severityPenalty;
+  const score = Math.max(0, Math.min(100, Math.round(rawScore)));
+  const factors = [
+    {
+      name: "Issue Density",
+      impact: -Math.round(densityPenalty),
+      description: `${issuesPerFile.toFixed(2)} issues per file ${issuesPerFile < 1 ? "(excellent)" : issuesPerFile < 3 ? "(acceptable)" : "(high)"}`
+    }
+  ];
+  if (criticalIssues > 0) {
+    const criticalImpact = Math.min(30, criticalIssues * 10);
+    factors.push({
+      name: "Critical Issues",
+      impact: -criticalImpact,
+      description: `${criticalIssues} critical consistency issue${criticalIssues > 1 ? "s" : ""} (high AI confusion risk)`
+    });
+  }
+  if (majorIssues > 0) {
+    const majorImpact = Math.min(20, Math.round(majorIssues * 3));
+    factors.push({
+      name: "Major Issues",
+      impact: -majorImpact,
+      description: `${majorIssues} major issue${majorIssues > 1 ? "s" : ""} (moderate AI confusion risk)`
+    });
+  }
+  if (minorIssues > 0 && minorIssues >= totalFilesAnalyzed) {
+    const minorImpact = -Math.round(minorIssues * 0.5);
+    factors.push({
+      name: "Minor Issues",
+      impact: minorImpact,
+      description: `${minorIssues} minor issue${minorIssues > 1 ? "s" : ""} (slight AI confusion risk)`
+    });
+  }
+  const recommendations = [];
+  if (criticalIssues > 0) {
+    const estimatedImpact = Math.min(30, criticalIssues * 10);
+    recommendations.push({
+      action: "Fix critical naming/pattern inconsistencies (highest AI confusion risk)",
+      estimatedImpact,
+      priority: "high"
+    });
+  }
+  if (majorIssues > 5) {
+    const estimatedImpact = Math.min(15, Math.round(majorIssues / 2));
+    recommendations.push({
+      action: "Standardize naming conventions across codebase",
+      estimatedImpact,
+      priority: "medium"
+    });
+  }
+  if (issuesPerFile > 3) {
+    recommendations.push({
+      action: "Establish and enforce coding style guide to reduce inconsistencies",
+      estimatedImpact: 12,
+      priority: "medium"
+    });
+  }
+  if (totalIssues > 20 && minorIssues / totalIssues > 0.7) {
+    recommendations.push({
+      action: "Enable linter/formatter to automatically fix minor style issues",
+      estimatedImpact: 8,
+      priority: "low"
+    });
+  }
+  const productivityImpact = (0, import_core5.calculateProductivityImpact)(issues);
+  return {
+    toolName: import_core5.ToolName.NamingConsistency,
+    score,
+    rawMetrics: {
+      totalIssues,
+      criticalIssues,
+      majorIssues,
+      minorIssues,
+      issuesPerFile: Math.round(issuesPerFile * 100) / 100,
+      avgWeightedIssuesPerFile: Math.round(avgWeightedIssuesPerFile * 100) / 100,
+      // Business value metrics
+      estimatedDeveloperHours: productivityImpact.totalHours
+    },
+    factors,
+    recommendations
+  };
+}
+
 // src/analyzer.ts
 async function analyzeConsistency(options) {
   const {
@@ -690,11 +785,11 @@ async function analyzeConsistency(options) {
     checkPatterns = true,
     checkArchitecture = false,
     // Not implemented yet
-    minSeverity = import_core5.Severity.Info,
+    minSeverity = import_core6.Severity.Info,
     ...scanOptions
   } = options;
   void checkArchitecture;
-  const filePaths = await (0, import_core5.scanFiles)(scanOptions);
+  const filePaths = await (0, import_core6.scanFiles)(scanOptions);
   let namingIssues = [];
   if (checkNaming) {
     namingIssues = await analyzeNamingGeneralized(filePaths);
@@ -720,11 +815,15 @@ async function analyzeConsistency(options) {
     fileIssuesMap.get(fileName).push(issue);
   }
   for (const [fileName, issues] of fileIssuesMap.entries()) {
+    const scoreResult = calculateConsistencyScore(
+      issues,
+      filePaths.length
+    );
     results.push({
       fileName,
       issues: issues.map((i) => transformToIssue(i)),
       metrics: {
-        consistencyScore: calculateConsistencyScore(issues)
+        consistencyScore: scoreResult.score / 100
       }
     });
   }
@@ -757,21 +856,21 @@ async function analyzeConsistency(options) {
   };
 }
 function shouldIncludeSeverity(severity, minSeverity) {
-  return (0, import_core5.getSeverityLevel)(severity) >= (0, import_core5.getSeverityLevel)(minSeverity);
+  return (0, import_core6.getSeverityLevel)(severity) >= (0, import_core6.getSeverityLevel)(minSeverity);
 }
 function getIssueType(type) {
-  if (!type) return import_core5.IssueType.NamingInconsistency;
+  if (!type) return import_core6.IssueType.NamingInconsistency;
   const typeMap = {
-    "naming-inconsistency": import_core5.IssueType.NamingInconsistency,
-    "naming-quality": import_core5.IssueType.NamingQuality,
-    "pattern-inconsistency": import_core5.IssueType.PatternInconsistency,
-    "architecture-inconsistency": import_core5.IssueType.ArchitectureInconsistency,
-    "error-handling": import_core5.IssueType.PatternInconsistency,
-    "async-style": import_core5.IssueType.PatternInconsistency,
-    "import-style": import_core5.IssueType.PatternInconsistency,
-    "api-design": import_core5.IssueType.PatternInconsistency
+    "naming-inconsistency": import_core6.IssueType.NamingInconsistency,
+    "naming-quality": import_core6.IssueType.NamingQuality,
+    "pattern-inconsistency": import_core6.IssueType.PatternInconsistency,
+    "architecture-inconsistency": import_core6.IssueType.ArchitectureInconsistency,
+    "error-handling": import_core6.IssueType.PatternInconsistency,
+    "async-style": import_core6.IssueType.PatternInconsistency,
+    "import-style": import_core6.IssueType.PatternInconsistency,
+    "api-design": import_core6.IssueType.PatternInconsistency
   };
-  return typeMap[type] || import_core5.IssueType.NamingInconsistency;
+  return typeMap[type] || import_core6.IssueType.NamingInconsistency;
 }
 function transformToIssue(i) {
   if (i.message && i.location) {
@@ -824,35 +923,12 @@ function transformToIssue(i) {
     suggestion: i.suggestion
   };
 }
-function calculateConsistencyScore(issues) {
-  let totalWeight = 0;
-  for (const issue of issues) {
-    const val = (0, import_core5.getSeverityLevel)(issue.severity);
-    switch (val) {
-      case 4:
-        totalWeight += 10;
-        break;
-      case 3:
-        totalWeight += 5;
-        break;
-      case 2:
-        totalWeight += 2;
-        break;
-      case 1:
-        totalWeight += 1;
-        break;
-      default:
-        totalWeight += 1;
-    }
-  }
-  return Math.max(0, 1 - totalWeight / 100);
-}
 
 // src/cli.ts
 var import_chalk = __toESM(require("chalk"));
 var import_fs4 = require("fs");
 var import_path = require("path");
-var import_core6 = require("@aiready/core");
+var import_core7 = require("@aiready/core");
 var program = new import_commander.Command();
 program.name("aiready-consistency").description(
   "Detect consistency patterns in naming, code structure, and architecture"
@@ -891,7 +967,7 @@ EXAMPLES:
 ).option("--output-file <path>", "Output file path (for json/markdown)").action(async (directory, options) => {
   console.log(import_chalk.default.blue("\u{1F50D} Analyzing consistency...\n"));
   const startTime = Date.now();
-  const config = await (0, import_core6.loadConfig)(directory);
+  const config = await (0, import_core7.loadConfig)(directory);
   const defaults = {
     checkNaming: true,
     checkPatterns: true,
@@ -900,7 +976,7 @@ EXAMPLES:
     include: void 0,
     exclude: void 0
   };
-  const mergedConfig = (0, import_core6.mergeConfigWithDefaults)(config, defaults);
+  const mergedConfig = (0, import_core7.mergeConfigWithDefaults)(config, defaults);
   const finalOptions = {
     rootDir: directory,
     checkNaming: options.naming !== false && mergedConfig.checkNaming,
@@ -914,7 +990,7 @@ EXAMPLES:
   const elapsedTime = ((Date.now() - startTime) / 1e3).toFixed(2);
   if (options.output === "json") {
     const output = JSON.stringify(report, null, 2);
-    const outputPath = (0, import_core6.resolveOutputPath)(
+    const outputPath = (0, import_core7.resolveOutputPath)(
       options.outputFile,
       `consistency-report-${(/* @__PURE__ */ new Date()).toISOString().split("T")[0]}.json`,
       directory
@@ -927,7 +1003,7 @@ EXAMPLES:
     console.log(import_chalk.default.green(`\u2713 Report saved to ${outputPath}`));
   } else if (options.output === "markdown") {
     const markdown = generateMarkdownReport(report, elapsedTime);
-    const outputPath = (0, import_core6.resolveOutputPath)(
+    const outputPath = (0, import_core7.resolveOutputPath)(
       options.outputFile,
       `consistency-report-${(/* @__PURE__ */ new Date()).toISOString().split("T")[0]}.md`,
       directory

package/dist/cli.mjs

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 import {
   analyzeConsistency
-} from "./chunk-6H3JHDP7.mjs";
+} from "./chunk-CLWNLHDB.mjs";
 
 // src/cli.ts
 import { Command } from "commander";

package/dist/index.js

@@ -25,7 +25,7 @@ __export(index_exports, {
   analyzeNaming: () => analyzeNaming,
   analyzeNamingAST: () => analyzeNamingAST,
   analyzePatterns: () => analyzePatterns,
-  calculateConsistencyScore: () => calculateConsistencyScore2,
+  calculateConsistencyScore: () => calculateConsistencyScore,
   detectNamingConventions: () => detectNamingConventions
 });
 module.exports = __toCommonJS(index_exports);
@@ -35,7 +35,7 @@ var import_core9 = require("@aiready/core");
 var import_core7 = require("@aiready/core");
 
 // src/analyzer.ts
-var import_core5 = require("@aiready/core");
+var import_core6 = require("@aiready/core");
 
 // src/analyzers/naming-ast.ts
 var import_core2 = require("@aiready/core");
@@ -319,12 +319,10 @@ function analyzeIdentifiers(ast, filePath, context) {
               { isParameter: true, isArrowParameter }
             );
           } else if (param.type === "ObjectPattern") {
-            extractDestructuredIdentifiers(
-              param,
-              true,
-              scopeTracker,
+            extractDestructuredIdentifiers(param, scopeTracker, {
+              isParameter: true,
               isArrowParameter
-            );
+            });
           }
         });
       }
@@ -471,7 +469,8 @@ var ScopeTracker = class {
     return this.variables;
   }
 };
-function extractDestructuredIdentifiers(node, isParameter, scopeTracker, isArrowParameter = false) {
+function extractDestructuredIdentifiers(node, scopeTracker, options = {}) {
+  const { isParameter = false, isArrowParameter = false } = options;
   if (node.type === "ObjectPattern") {
     node.properties.forEach((prop) => {
       if (prop.type === "Property" && prop.value.type === "Identifier") {
@@ -549,7 +548,7 @@ var COMMON_ABBREVIATIONS = /* @__PURE__ */ new Set([
 async function analyzeNamingGeneralized(files) {
   const issues = [];
   for (const file of files) {
-    const parser = (0, import_core3.getParser)(file);
+    const parser = await (0, import_core3.getParser)(file);
     if (!parser) continue;
     try {
       const code = (0, import_fs2.readFileSync)(file, "utf-8");
@@ -691,6 +690,102 @@ async function analyzePatterns(filePaths) {
   return issues;
 }
 
+// src/scoring.ts
+var import_core5 = require("@aiready/core");
+function calculateConsistencyScore(issues, totalFilesAnalyzed, costConfig) {
+  void costConfig;
+  const criticalIssues = issues.filter((i) => i.severity === "critical").length;
+  const majorIssues = issues.filter((i) => i.severity === "major").length;
+  const minorIssues = issues.filter((i) => i.severity === "minor").length;
+  const totalIssues = issues.length;
+  const issuesPerFile = totalFilesAnalyzed > 0 ? totalIssues / totalFilesAnalyzed : 0;
+  const densityPenalty = Math.min(50, issuesPerFile * 15);
+  const weightedCount = criticalIssues * 10 + majorIssues * 3 + minorIssues * 0.5;
+  const avgWeightedIssuesPerFile = totalFilesAnalyzed > 0 ? weightedCount / totalFilesAnalyzed : 0;
+  const severityPenalty = Math.min(50, avgWeightedIssuesPerFile * 2);
+  const rawScore = 100 - densityPenalty - severityPenalty;
+  const score = Math.max(0, Math.min(100, Math.round(rawScore)));
+  const factors = [
+    {
+      name: "Issue Density",
+      impact: -Math.round(densityPenalty),
+      description: `${issuesPerFile.toFixed(2)} issues per file ${issuesPerFile < 1 ? "(excellent)" : issuesPerFile < 3 ? "(acceptable)" : "(high)"}`
+    }
+  ];
+  if (criticalIssues > 0) {
+    const criticalImpact = Math.min(30, criticalIssues * 10);
+    factors.push({
+      name: "Critical Issues",
+      impact: -criticalImpact,
+      description: `${criticalIssues} critical consistency issue${criticalIssues > 1 ? "s" : ""} (high AI confusion risk)`
+    });
+  }
+  if (majorIssues > 0) {
+    const majorImpact = Math.min(20, Math.round(majorIssues * 3));
+    factors.push({
+      name: "Major Issues",
+      impact: -majorImpact,
+      description: `${majorIssues} major issue${majorIssues > 1 ? "s" : ""} (moderate AI confusion risk)`
+    });
+  }
+  if (minorIssues > 0 && minorIssues >= totalFilesAnalyzed) {
+    const minorImpact = -Math.round(minorIssues * 0.5);
+    factors.push({
+      name: "Minor Issues",
+      impact: minorImpact,
+      description: `${minorIssues} minor issue${minorIssues > 1 ? "s" : ""} (slight AI confusion risk)`
+    });
+  }
+  const recommendations = [];
+  if (criticalIssues > 0) {
+    const estimatedImpact = Math.min(30, criticalIssues * 10);
+    recommendations.push({
+      action: "Fix critical naming/pattern inconsistencies (highest AI confusion risk)",
+      estimatedImpact,
+      priority: "high"
+    });
+  }
+  if (majorIssues > 5) {
+    const estimatedImpact = Math.min(15, Math.round(majorIssues / 2));
+    recommendations.push({
+      action: "Standardize naming conventions across codebase",
+      estimatedImpact,
+      priority: "medium"
+    });
+  }
+  if (issuesPerFile > 3) {
+    recommendations.push({
+      action: "Establish and enforce coding style guide to reduce inconsistencies",
+      estimatedImpact: 12,
+      priority: "medium"
+    });
+  }
+  if (totalIssues > 20 && minorIssues / totalIssues > 0.7) {
+    recommendations.push({
+      action: "Enable linter/formatter to automatically fix minor style issues",
+      estimatedImpact: 8,
+      priority: "low"
+    });
+  }
+  const productivityImpact = (0, import_core5.calculateProductivityImpact)(issues);
+  return {
+    toolName: import_core5.ToolName.NamingConsistency,
+    score,
+    rawMetrics: {
+      totalIssues,
+      criticalIssues,
+      majorIssues,
+      minorIssues,
+      issuesPerFile: Math.round(issuesPerFile * 100) / 100,
+      avgWeightedIssuesPerFile: Math.round(avgWeightedIssuesPerFile * 100) / 100,
+      // Business value metrics
+      estimatedDeveloperHours: productivityImpact.totalHours
+    },
+    factors,
+    recommendations
+  };
+}
+
 // src/analyzer.ts
 async function analyzeConsistency(options) {
   const {
@@ -698,11 +793,11 @@ async function analyzeConsistency(options) {
     checkPatterns = true,
     checkArchitecture = false,
     // Not implemented yet
-    minSeverity = import_core5.Severity.Info,
+    minSeverity = import_core6.Severity.Info,
     ...scanOptions
   } = options;
   void checkArchitecture;
-  const filePaths = await (0, import_core5.scanFiles)(scanOptions);
+  const filePaths = await (0, import_core6.scanFiles)(scanOptions);
   let namingIssues = [];
   if (checkNaming) {
     namingIssues = await analyzeNamingGeneralized(filePaths);
@@ -728,11 +823,15 @@ async function analyzeConsistency(options) {
     fileIssuesMap.get(fileName).push(issue);
   }
   for (const [fileName, issues] of fileIssuesMap.entries()) {
+    const scoreResult = calculateConsistencyScore(
+      issues,
+      filePaths.length
+    );
     results.push({
       fileName,
       issues: issues.map((i) => transformToIssue(i)),
       metrics: {
-        consistencyScore: calculateConsistencyScore(issues)
+        consistencyScore: scoreResult.score / 100
       }
     });
   }
@@ -765,21 +864,21 @@ async function analyzeConsistency(options) {
   };
 }
 function shouldIncludeSeverity(severity, minSeverity) {
-  return (0, import_core5.getSeverityLevel)(severity) >= (0, import_core5.getSeverityLevel)(minSeverity);
+  return (0, import_core6.getSeverityLevel)(severity) >= (0, import_core6.getSeverityLevel)(minSeverity);
 }
 function getIssueType(type) {
-  if (!type) return import_core5.IssueType.NamingInconsistency;
+  if (!type) return import_core6.IssueType.NamingInconsistency;
   const typeMap = {
-    "naming-inconsistency": import_core5.IssueType.NamingInconsistency,
-    "naming-quality": import_core5.IssueType.NamingQuality,
-    "pattern-inconsistency": import_core5.IssueType.PatternInconsistency,
-    "architecture-inconsistency": import_core5.IssueType.ArchitectureInconsistency,
-    "error-handling": import_core5.IssueType.PatternInconsistency,
-    "async-style": import_core5.IssueType.PatternInconsistency,
-    "import-style": import_core5.IssueType.PatternInconsistency,
-    "api-design": import_core5.IssueType.PatternInconsistency
+    "naming-inconsistency": import_core6.IssueType.NamingInconsistency,
+    "naming-quality": import_core6.IssueType.NamingQuality,
+    "pattern-inconsistency": import_core6.IssueType.PatternInconsistency,
+    "architecture-inconsistency": import_core6.IssueType.ArchitectureInconsistency,
+    "error-handling": import_core6.IssueType.PatternInconsistency,
+    "async-style": import_core6.IssueType.PatternInconsistency,
+    "import-style": import_core6.IssueType.PatternInconsistency,
+    "api-design": import_core6.IssueType.PatternInconsistency
   };
-  return typeMap[type] || import_core5.IssueType.NamingInconsistency;
+  return typeMap[type] || import_core6.IssueType.NamingInconsistency;
 }
 function transformToIssue(i) {
   if (i.message && i.location) {
@@ -832,125 +931,6 @@ function transformToIssue(i) {
     suggestion: i.suggestion
   };
 }
-function calculateConsistencyScore(issues) {
-  let totalWeight = 0;
-  for (const issue of issues) {
-    const val = (0, import_core5.getSeverityLevel)(issue.severity);
-    switch (val) {
-      case 4:
-        totalWeight += 10;
-        break;
-      case 3:
-        totalWeight += 5;
-        break;
-      case 2:
-        totalWeight += 2;
-        break;
-      case 1:
-        totalWeight += 1;
-        break;
-      default:
-        totalWeight += 1;
-    }
-  }
-  return Math.max(0, 1 - totalWeight / 100);
-}
-
-// src/scoring.ts
-var import_core6 = require("@aiready/core");
-function calculateConsistencyScore2(issues, totalFilesAnalyzed, costConfig) {
-  void costConfig;
-  const criticalIssues = issues.filter((i) => i.severity === "critical").length;
-  const majorIssues = issues.filter((i) => i.severity === "major").length;
-  const minorIssues = issues.filter((i) => i.severity === "minor").length;
-  const totalIssues = issues.length;
-  const issuesPerFile = totalFilesAnalyzed > 0 ? totalIssues / totalFilesAnalyzed : 0;
-  const densityPenalty = Math.min(50, issuesPerFile * 15);
-  const weightedCount = criticalIssues * 10 + majorIssues * 3 + minorIssues * 0.5;
-  const avgWeightedIssuesPerFile = totalFilesAnalyzed > 0 ? weightedCount / totalFilesAnalyzed : 0;
-  const severityPenalty = Math.min(50, avgWeightedIssuesPerFile * 2);
-  const rawScore = 100 - densityPenalty - severityPenalty;
-  const score = Math.max(0, Math.min(100, Math.round(rawScore)));
-  const factors = [
-    {
-      name: "Issue Density",
-      impact: -Math.round(densityPenalty),
-      description: `${issuesPerFile.toFixed(2)} issues per file ${issuesPerFile < 1 ? "(excellent)" : issuesPerFile < 3 ? "(acceptable)" : "(high)"}`
-    }
-  ];
-  if (criticalIssues > 0) {
-    const criticalImpact = Math.min(30, criticalIssues * 10);
-    factors.push({
-      name: "Critical Issues",
-      impact: -criticalImpact,
-      description: `${criticalIssues} critical consistency issue${criticalIssues > 1 ? "s" : ""} (high AI confusion risk)`
-    });
-  }
-  if (majorIssues > 0) {
-    const majorImpact = Math.min(20, Math.round(majorIssues * 3));
-    factors.push({
-      name: "Major Issues",
-      impact: -majorImpact,
-      description: `${majorIssues} major issue${majorIssues > 1 ? "s" : ""} (moderate AI confusion risk)`
-    });
-  }
-  if (minorIssues > 0 && minorIssues >= totalFilesAnalyzed) {
-    const minorImpact = -Math.round(minorIssues * 0.5);
-    factors.push({
-      name: "Minor Issues",
-      impact: minorImpact,
-      description: `${minorIssues} minor issue${minorIssues > 1 ? "s" : ""} (slight AI confusion risk)`
-    });
-  }
-  const recommendations = [];
-  if (criticalIssues > 0) {
-    const estimatedImpact = Math.min(30, criticalIssues * 10);
-    recommendations.push({
-      action: "Fix critical naming/pattern inconsistencies (highest AI confusion risk)",
-      estimatedImpact,
-      priority: "high"
-    });
-  }
-  if (majorIssues > 5) {
-    const estimatedImpact = Math.min(15, Math.round(majorIssues / 2));
-    recommendations.push({
-      action: "Standardize naming conventions across codebase",
-      estimatedImpact,
-      priority: "medium"
-    });
-  }
-  if (issuesPerFile > 3) {
-    recommendations.push({
-      action: "Establish and enforce coding style guide to reduce inconsistencies",
-      estimatedImpact: 12,
-      priority: "medium"
-    });
-  }
-  if (totalIssues > 20 && minorIssues / totalIssues > 0.7) {
-    recommendations.push({
-      action: "Enable linter/formatter to automatically fix minor style issues",
-      estimatedImpact: 8,
-      priority: "low"
-    });
-  }
-  const productivityImpact = (0, import_core6.calculateProductivityImpact)(issues);
-  return {
-    toolName: import_core6.ToolName.NamingConsistency,
-    score,
-    rawMetrics: {
-      totalIssues,
-      criticalIssues,
-      majorIssues,
-      minorIssues,
-      issuesPerFile: Math.round(issuesPerFile * 100) / 100,
-      avgWeightedIssuesPerFile: Math.round(avgWeightedIssuesPerFile * 100) / 100,
-      // Business value metrics
-      estimatedDeveloperHours: productivityImpact.totalHours
-    },
-    factors,
-    recommendations
-  };
-}
 
 // src/provider.ts
 var ConsistencyProvider = (0, import_core7.createProvider)({
@@ -971,7 +951,7 @@ var ConsistencyProvider = (0, import_core7.createProvider)({
     const results = output.results;
     const allIssues = results.flatMap((r) => r.issues);
     const totalFiles = output.summary.filesAnalyzed || results.length;
-    return calculateConsistencyScore2(
+    return calculateConsistencyScore(
       allIssues,
       totalFiles,
       options.costConfig