@aiready/consistency 0.20.19 → 0.20.21

package/dist/chunk-CLWNLHDB.mjs

@@ -0,0 +1,909 @@
+// src/analyzers/naming-ast.ts
+import { Severity as Severity2 } from "@aiready/core";
+
+// src/utils/ast-parser.ts
+import { parse } from "@typescript-eslint/typescript-estree";
+import { readFileSync } from "fs";
+function parseFile(filePath, content) {
+  try {
+    const code = content ?? readFileSync(filePath, "utf-8");
+    const isTypeScript = filePath.match(/\.tsx?$/);
+    return parse(code, {
+      jsx: filePath.match(/\.[jt]sx$/i) !== null,
+      loc: true,
+      range: true,
+      comment: false,
+      tokens: false,
+      // Relaxed parsing for JavaScript files
+      sourceType: "module",
+      ecmaVersion: "latest",
+      // Only use TypeScript parser features for .ts/.tsx files
+      filePath: isTypeScript ? filePath : void 0
+    });
+  } catch (error) {
+    void error;
+    return null;
+  }
+}
+function traverseAST(node, visitor, parent = null) {
+  if (!node) return;
+  visitor.enter?.(node, parent);
+  for (const key of Object.keys(node)) {
+    const value = node[key];
+    if (Array.isArray(value)) {
+      for (const child of value) {
+        if (child && typeof child === "object" && "type" in child) {
+          traverseAST(child, visitor, node);
+        }
+      }
+    } else if (value && typeof value === "object" && "type" in value) {
+      traverseAST(value, visitor, node);
+    }
+  }
+  visitor.leave?.(node, parent);
+}
+function isLoopStatement(node) {
+  return [
+    "ForStatement",
+    "ForInStatement",
+    "ForOfStatement",
+    "WhileStatement",
+    "DoWhileStatement"
+  ].includes(node.type);
+}
+function getLineNumber(node) {
+  return node.loc?.start.line ?? 0;
+}
+
+// src/utils/context-detector.ts
+import { Severity } from "@aiready/core";
+function detectFileType(filePath, ast) {
+  void ast;
+  const path = filePath.toLowerCase();
+  if (path.match(/\.(test|spec)\.(ts|tsx|js|jsx)$/) || path.includes("__tests__")) {
+    return "test";
+  }
+  if (path.endsWith(".d.ts") || path.includes("types")) {
+    return "types";
+  }
+  if (path.match(/config|\.config\.|rc\.|setup/) || path.includes("configuration")) {
+    return "config";
+  }
+  return "production";
+}
+function detectCodeLayer(ast) {
+  let hasAPIIndicators = 0;
+  let hasBusinessIndicators = 0;
+  let hasDataIndicators = 0;
+  let hasUtilityIndicators = 0;
+  traverseAST(ast, {
+    enter: (node) => {
+      if (node.type === "ImportDeclaration") {
+        const source = node.source.value;
+        if (source.match(/express|fastify|koa|@nestjs|axios|fetch|http/i)) {
+          hasAPIIndicators++;
+        }
+        if (source.match(/database|prisma|typeorm|sequelize|mongoose|pg|mysql/i)) {
+          hasDataIndicators++;
+        }
+      }
+      if (node.type === "FunctionDeclaration" && node.id) {
+        const name = node.id.name;
+        if (name.match(
+          /^(get|post|put|delete|patch|handle|api|route|controller)/i
+        )) {
+          hasAPIIndicators++;
+        }
+        if (name.match(/^(calculate|process|validate|transform|compute|analyze)/i)) {
+          hasBusinessIndicators++;
+        }
+        if (name.match(/^(find|create|update|delete|save|fetch|query|insert)/i)) {
+          hasDataIndicators++;
+        }
+        if (name.match(
+          /^(format|parse|convert|normalize|sanitize|encode|decode)/i
+        )) {
+          hasUtilityIndicators++;
+        }
+      }
+      if (node.type === "ExportNamedDeclaration" || node.type === "ExportDefaultDeclaration") {
+        if (node.type === "ExportNamedDeclaration" && node.declaration) {
+          if (node.declaration.type === "FunctionDeclaration" && node.declaration.id) {
+            const name = node.declaration.id.name;
+            if (name.match(/handler|route|api|controller/i)) {
+              hasAPIIndicators += 2;
+            }
+          }
+        }
+      }
+    }
+  });
+  const scores = {
+    api: hasAPIIndicators,
+    business: hasBusinessIndicators,
+    data: hasDataIndicators,
+    utility: hasUtilityIndicators
+  };
+  const maxScore = Math.max(...Object.values(scores));
+  if (maxScore === 0) {
+    return "unknown";
+  }
+  if (scores.api === maxScore) return "api";
+  if (scores.data === maxScore) return "data";
+  if (scores.business === maxScore) return "business";
+  if (scores.utility === maxScore) return "utility";
+  return "unknown";
+}
+function calculateComplexity(node) {
+  let complexity = 1;
+  traverseAST(node, {
+    enter: (childNode) => {
+      switch (childNode.type) {
+        case "IfStatement":
+        case "ConditionalExpression":
+        // ternary
+        case "SwitchCase":
+        case "ForStatement":
+        case "ForInStatement":
+        case "ForOfStatement":
+        case "WhileStatement":
+        case "DoWhileStatement":
+        case "CatchClause":
+          complexity++;
+          break;
+        case "LogicalExpression":
+          if (childNode.operator === "&&" || childNode.operator === "||") {
+            complexity++;
+          }
+          break;
+      }
+    }
+  });
+  return complexity;
+}
+function buildCodeContext(filePath, ast) {
+  const fileType = detectFileType(filePath, ast);
+  const codeLayer = detectCodeLayer(ast);
+  let totalComplexity = 0;
+  let functionCount = 0;
+  traverseAST(ast, {
+    enter: (node) => {
+      if (node.type === "FunctionDeclaration" || node.type === "FunctionExpression" || node.type === "ArrowFunctionExpression") {
+        totalComplexity += calculateComplexity(node);
+        functionCount++;
+      }
+    }
+  });
+  const avgComplexity = functionCount > 0 ? totalComplexity / functionCount : 1;
+  return {
+    fileType,
+    codeLayer,
+    complexity: Math.round(avgComplexity),
+    isTestFile: fileType === "test",
+    isTypeDefinition: fileType === "types"
+  };
+}
+function adjustSeverity(baseSeverity, context, issueType) {
+  const getEnum = (s) => {
+    if (s === Severity.Critical || s === "critical") return Severity.Critical;
+    if (s === Severity.Major || s === "major") return Severity.Major;
+    if (s === Severity.Minor || s === "minor") return Severity.Minor;
+    return Severity.Info;
+  };
+  let currentSev = getEnum(baseSeverity);
+  if (context.isTestFile) {
+    if (currentSev === Severity.Minor) currentSev = Severity.Info;
+    if (currentSev === Severity.Major) currentSev = Severity.Minor;
+  }
+  if (context.isTypeDefinition) {
+    if (currentSev === Severity.Minor) currentSev = Severity.Info;
+  }
+  if (context.codeLayer === "api") {
+    if (currentSev === Severity.Info && issueType === "unclear")
+      currentSev = Severity.Minor;
+    if (currentSev === Severity.Minor && issueType === "unclear")
+      currentSev = Severity.Major;
+  }
+  if (context.complexity > 10) {
+    if (currentSev === Severity.Info) currentSev = Severity.Minor;
+  }
+  if (context.codeLayer === "utility") {
+    if (currentSev === Severity.Minor && issueType === "abbreviation")
+      currentSev = Severity.Info;
+  }
+  return currentSev;
+}
+function isAcceptableInContext(name, context, options) {
+  if (options.isLoopVariable && ["i", "j", "k", "l", "n", "m"].includes(name)) {
+    return true;
+  }
+  if (context.isTestFile) {
+    if (["a", "b", "c", "x", "y", "z"].includes(name) && options.isParameter) {
+      return true;
+    }
+  }
+  if (context.codeLayer === "utility" && ["x", "y", "z"].includes(name)) {
+    return true;
+  }
+  if (options.isDestructured) {
+    if (["s", "b", "f", "l"].includes(name)) {
+      return true;
+    }
+  }
+  if (options.isParameter && (options.complexity ?? context.complexity) < 3) {
+    if (name.length >= 2) {
+      return true;
+    }
+  }
+  return false;
+}
+
+// src/analyzers/naming-ast.ts
+async function analyzeNamingAST(filePaths) {
+  const allIssues = [];
+  for (const filePath of filePaths) {
+    try {
+      const ast = parseFile(filePath);
+      if (!ast) continue;
+      const context = buildCodeContext(filePath, ast);
+      const issues = analyzeIdentifiers(ast, filePath, context);
+      allIssues.push(...issues);
+    } catch (err) {
+      void err;
+    }
+  }
+  return allIssues;
+}
+function analyzeIdentifiers(ast, filePath, context) {
+  const issues = [];
+  const scopeTracker = new ScopeTracker();
+  traverseAST(ast, {
+    enter: (node) => {
+      if (node.type === "VariableDeclarator" && node.id.type === "Identifier") {
+        const isParameter = false;
+        const isLoopVariable = isLoopStatement(node.parent?.parent);
+        scopeTracker.declareVariable(
+          node.id.name,
+          node.id,
+          getLineNumber(node.id),
+          { isParameter, isLoopVariable }
+        );
+      }
+      if (node.type === "FunctionDeclaration" || node.type === "FunctionExpression" || node.type === "ArrowFunctionExpression") {
+        const isArrowParameter = node.type === "ArrowFunctionExpression";
+        node.params.forEach((param) => {
+          if (param.type === "Identifier") {
+            scopeTracker.declareVariable(
+              param.name,
+              param,
+              getLineNumber(param),
+              { isParameter: true, isArrowParameter }
+            );
+          } else if (param.type === "ObjectPattern") {
+            extractDestructuredIdentifiers(param, scopeTracker, {
+              isParameter: true,
+              isArrowParameter
+            });
+          }
+        });
+      }
+      if ((node.type === "ClassDeclaration" || node.type === "TSInterfaceDeclaration" || node.type === "TSTypeAliasDeclaration") && node.id) {
+        checkNamingConvention(
+          node.id.name,
+          "PascalCase",
+          node.id,
+          filePath,
+          issues,
+          context
+        );
+      }
+    }
+  });
+  for (const varInfo of scopeTracker.getVariables()) {
+    checkVariableNaming(varInfo, filePath, issues, context);
+  }
+  return issues;
+}
+function checkNamingConvention(name, convention, node, file, issues, context) {
+  let isValid = true;
+  if (convention === "PascalCase") {
+    isValid = /^[A-Z][a-zA-Z0-9]*$/.test(name);
+  } else if (convention === "camelCase") {
+    isValid = /^[a-z][a-zA-Z0-9]*$/.test(name);
+  } else if (convention === "UPPER_CASE") {
+    isValid = /^[A-Z][A-Z0-9_]*$/.test(name);
+  }
+  if (!isValid) {
+    const severity = adjustSeverity(Severity2.Info, context, "convention-mix");
+    issues.push({
+      file,
+      line: getLineNumber(node),
+      type: "convention-mix",
+      identifier: name,
+      severity,
+      suggestion: `Follow ${convention} for this identifier`
+    });
+  }
+}
+function checkVariableNaming(varInfo, file, issues, context) {
+  const { name, line, options } = varInfo;
+  if (isAcceptableInContext(name, context, options)) {
+    return;
+  }
+  if (name.length === 1 && !options.isLoopVariable && !options.isArrowParameter) {
+    const severity = adjustSeverity(Severity2.Minor, context, "poor-naming");
+    issues.push({
+      file,
+      line,
+      type: "poor-naming",
+      identifier: name,
+      severity,
+      suggestion: "Use a more descriptive name than a single letter"
+    });
+  }
+  const vagueNames = [
+    "data",
+    "info",
+    "item",
+    "obj",
+    "val",
+    "tmp",
+    "temp",
+    "thing",
+    "stuff"
+  ];
+  if (vagueNames.includes(name.toLowerCase())) {
+    const severity = adjustSeverity(Severity2.Minor, context, "poor-naming");
+    issues.push({
+      file,
+      line,
+      type: "poor-naming",
+      identifier: name,
+      severity,
+      suggestion: `Avoid vague names like '${name}'. What does this data represent?`
+    });
+  }
+  if (name.length > 1 && name.length <= 3 && !options.isLoopVariable && !isCommonAbbreviation(name)) {
+    const severity = adjustSeverity(Severity2.Info, context, "abbreviation");
+    issues.push({
+      file,
+      line,
+      type: "abbreviation",
+      identifier: name,
+      severity,
+      suggestion: "Avoid non-standard abbreviations"
+    });
+  }
+}
+function isCommonAbbreviation(name) {
+  const common = [
+    "id",
+    "db",
+    "fs",
+    "os",
+    "ip",
+    "ui",
+    "ux",
+    "api",
+    "env",
+    "url",
+    "req",
+    "res",
+    "err",
+    "ctx",
+    "cb",
+    "idx",
+    "src",
+    "dir",
+    "app",
+    "dev",
+    "qa",
+    "dto",
+    "dao",
+    "ref",
+    "ast",
+    "dom",
+    "log",
+    "msg",
+    "pkg",
+    "req",
+    "err",
+    "res",
+    "css",
+    "html",
+    "xml",
+    "jsx",
+    "tsx",
+    "ts",
+    "js"
+  ];
+  return common.includes(name.toLowerCase());
+}
+var ScopeTracker = class {
+  constructor() {
+    this.variables = [];
+  }
+  declareVariable(name, node, line, options = {}) {
+    this.variables.push({ name, node, line, options });
+  }
+  getVariables() {
+    return this.variables;
+  }
+};
+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") {
+        scopeTracker.declareVariable(
+          prop.value.name,
+          prop.value,
+          getLineNumber(prop.value),
+          {
+            isParameter,
+            isDestructured: true,
+            isArrowParameter
+          }
+        );
+      }
+    });
+  } else if (node.type === "ArrayPattern") {
+    for (const element of node.elements) {
+      if (element?.type === "Identifier") {
+        scopeTracker.declareVariable(
+          element.name,
+          element,
+          getLineNumber(element),
+          {
+            isParameter,
+            isDestructured: true,
+            isArrowParameter
+          }
+        );
+      }
+    }
+  }
+}
+
+// src/analyzers/patterns.ts
+import { readFileSync as readFileSync2 } from "fs";
+import { Severity as Severity3 } from "@aiready/core";
+async function analyzePatterns(filePaths) {
+  const issues = [];
+  const contents = /* @__PURE__ */ new Map();
+  const tryCatchPattern = /try\s*\{/g;
+  const styleStats = {
+    tryCatch: 0,
+    thenCatch: 0,
+    asyncAwait: 0,
+    commonJs: 0,
+    esm: 0
+  };
+  for (const filePath of filePaths) {
+    try {
+      const content = readFileSync2(filePath, "utf-8");
+      contents.set(filePath, content);
+      if (content.match(tryCatchPattern)) styleStats.tryCatch++;
+      if (content.match(/\.catch\s*\(/)) styleStats.thenCatch++;
+      if (content.match(/\bawait\b/)) styleStats.asyncAwait++;
+      if (content.match(/\brequire\s*\(/)) styleStats.commonJs++;
+      if (content.match(/\bimport\b.*\bfrom\b/)) styleStats.esm++;
+    } catch (err) {
+      void err;
+    }
+  }
+  if (styleStats.tryCatch > 0 && styleStats.thenCatch > 0) {
+    const dominant = styleStats.tryCatch >= styleStats.thenCatch ? "try-catch" : ".catch()";
+    const minority = dominant === "try-catch" ? ".catch()" : "try-catch";
+    issues.push({
+      files: filePaths.filter((f) => {
+        const c = contents.get(f) || "";
+        return minority === "try-catch" ? c.match(tryCatchPattern) : c.match(/\.catch\s*\(/);
+      }),
+      type: "pattern-inconsistency",
+      description: `Mixed error handling styles: codebase primarily uses ${dominant}, but found ${minority} in some files.`,
+      examples: [dominant, minority],
+      severity: Severity3.Minor
+    });
+  }
+  if (styleStats.commonJs > 0 && styleStats.esm > 0) {
+    const minority = styleStats.esm >= styleStats.commonJs ? "CommonJS (require)" : "ESM (import)";
+    issues.push({
+      files: filePaths.filter((f) => {
+        const c = contents.get(f) || "";
+        return minority === "CommonJS (require)" ? c.match(/\brequire\s*\(/) : c.match(/\bimport\b/);
+      }),
+      type: "pattern-inconsistency",
+      description: `Mixed module systems: found both ESM and CommonJS.`,
+      examples: ['import X from "y"', 'const X = require("y")'],
+      severity: Severity3.Major
+    });
+  }
+  return issues;
+}
+
+// src/scoring.ts
+import { calculateProductivityImpact, ToolName } from "@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 = calculateProductivityImpact(issues);
+  return {
+    toolName: 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
+import {
+  scanFiles,
+  Severity as Severity5,
+  IssueType,
+  getSeverityLevel
+} from "@aiready/core";
+
+// src/analyzers/naming-generalized.ts
+import { getParser, Severity as Severity4 } from "@aiready/core";
+import { readFileSync as readFileSync3 } from "fs";
+var COMMON_ABBREVIATIONS = /* @__PURE__ */ new Set([
+  "id",
+  "db",
+  "fs",
+  "os",
+  "ip",
+  "ui",
+  "ux",
+  "api",
+  "env",
+  "url",
+  "req",
+  "res",
+  "err",
+  "ctx",
+  "cb",
+  "idx",
+  "src",
+  "dir",
+  "app",
+  "dev",
+  "qa",
+  "dto",
+  "dao",
+  "ref",
+  "ast",
+  "dom",
+  "log",
+  "msg",
+  "pkg",
+  "css",
+  "html",
+  "xml",
+  "jsx",
+  "tsx",
+  "ts",
+  "js"
+]);
+async function analyzeNamingGeneralized(files) {
+  const issues = [];
+  for (const file of files) {
+    const parser = await getParser(file);
+    if (!parser) continue;
+    try {
+      const code = readFileSync3(file, "utf-8");
+      if (!code.trim()) continue;
+      await parser.initialize();
+      const result = parser.parse(code, file);
+      const conventions = parser.getNamingConventions();
+      const exceptions = new Set(conventions.exceptions || []);
+      for (const exp of result.exports) {
+        if (!exp.name || exp.name === "default") continue;
+        if (exceptions.has(exp.name)) continue;
+        if (COMMON_ABBREVIATIONS.has(exp.name.toLowerCase())) continue;
+        let pattern;
+        if (exp.type === "class") {
+          pattern = conventions.classPattern;
+        } else if (exp.type === "interface" && conventions.interfacePattern) {
+          pattern = conventions.interfacePattern;
+        } else if (exp.type === "type" && conventions.typePattern) {
+          pattern = conventions.typePattern;
+        } else if (exp.type === "function") {
+          if (/^[A-Z][a-zA-Z0-9]*$/.test(exp.name) || /^[A-Z][A-Z0-9_]*$/.test(exp.name)) {
+            continue;
+          }
+          pattern = conventions.functionPattern;
+        } else if (exp.type === "const") {
+          if ([
+            "handler",
+            "GET",
+            "POST",
+            "PUT",
+            "DELETE",
+            "PATCH",
+            "OPTIONS",
+            "HEAD"
+          ].includes(exp.name))
+            continue;
+          pattern = exp.isPrimitive ? conventions.constantPattern : conventions.variablePattern;
+        } else {
+          pattern = conventions.variablePattern;
+        }
+        if (pattern && !pattern.test(exp.name)) {
+          issues.push({
+            type: "naming-inconsistency",
+            identifier: exp.name,
+            file,
+            line: exp.loc?.start.line || 1,
+            column: exp.loc?.start.column || 0,
+            // Recalibrate naming issues to Minor to differentiate from structural/architectural issues
+            severity: Severity4.Minor,
+            category: "naming",
+            suggestion: `Follow ${parser.language} ${exp.type} naming convention: ${pattern.toString()}`
+          });
+        }
+      }
+      for (const imp of result.imports) {
+        for (const spec of imp.specifiers) {
+          if (!spec || spec === "*" || spec === "default") continue;
+          if (exceptions.has(spec)) continue;
+          if (COMMON_ABBREVIATIONS.has(spec.toLowerCase())) continue;
+          if (!conventions.variablePattern.test(spec) && !conventions.classPattern.test(spec) && (!conventions.constantPattern || !conventions.constantPattern.test(spec)) && (!conventions.typePattern || !conventions.typePattern.test(spec)) && (!conventions.interfacePattern || !conventions.interfacePattern.test(spec)) && !/^[A-Z][A-Z0-9_]*$/.test(spec)) {
+            issues.push({
+              type: "naming-inconsistency",
+              identifier: spec,
+              file,
+              line: imp.loc?.start.line || 1,
+              column: imp.loc?.start.column || 0,
+              severity: Severity4.Info,
+              // Reduced from Minor to Info for imports
+              category: "naming",
+              suggestion: `Imported identifier '${spec}' may not follow standard conventions for this language.`
+            });
+          }
+        }
+      }
+    } catch (error) {
+      const errorMessage = error instanceof Error ? error.message : String(error);
+      console.debug(
+        `Consistency: Skipping unparseable file ${file}: ${errorMessage.split("\\n")[0]}`
+      );
+    }
+  }
+  return issues;
+}
+
+// src/analyzer.ts
+async function analyzeConsistency(options) {
+  const {
+    checkNaming = true,
+    checkPatterns = true,
+    checkArchitecture = false,
+    // Not implemented yet
+    minSeverity = Severity5.Info,
+    ...scanOptions
+  } = options;
+  void checkArchitecture;
+  const filePaths = await scanFiles(scanOptions);
+  let namingIssues = [];
+  if (checkNaming) {
+    namingIssues = await analyzeNamingGeneralized(filePaths);
+    const tsJsFiles = filePaths.filter((f) => /\.(ts|tsx|js|jsx)$/i.test(f));
+    if (tsJsFiles.length > 0) {
+      const deepTsIssues = await analyzeNamingAST(tsJsFiles);
+      namingIssues = [...namingIssues, ...deepTsIssues];
+    }
+  }
+  const patternIssues = checkPatterns ? await analyzePatterns(filePaths) : [];
+  const results = [];
+  const fileIssuesMap = /* @__PURE__ */ new Map();
+  for (const issue of namingIssues) {
+    if (!shouldIncludeSeverity(issue.severity, minSeverity)) continue;
+    const fileName = issue.fileName || issue.file || issue.filePath || "unknown";
+    if (!fileIssuesMap.has(fileName)) fileIssuesMap.set(fileName, []);
+    fileIssuesMap.get(fileName).push(issue);
+  }
+  for (const issue of patternIssues) {
+    if (!shouldIncludeSeverity(issue.severity, minSeverity)) continue;
+    const fileName = issue.fileName || issue.file || issue.filePath || (Array.isArray(issue.files) ? issue.files[0] : "unknown");
+    if (!fileIssuesMap.has(fileName)) fileIssuesMap.set(fileName, []);
+    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: scoreResult.score / 100
+      }
+    });
+  }
+  const recommendations = [];
+  if (namingIssues.length > 0) {
+    recommendations.push("Standardize naming conventions across the codebase");
+  }
+  if (patternIssues.length > 0) {
+    recommendations.push("Consolidate repetitive implementation patterns");
+  }
+  if (results.some((r) => (r.metrics?.consistencyScore ?? 1) < 0.8)) {
+    recommendations.push(
+      "Improve cross-module consistency to reduce AI confusion"
+    );
+  }
+  return {
+    results,
+    summary: {
+      filesAnalyzed: filePaths.length,
+      totalIssues: results.reduce((acc, r) => acc + r.issues.length, 0),
+      namingIssues: namingIssues.length,
+      patternIssues: patternIssues.length,
+      architectureIssues: 0
+    },
+    recommendations,
+    metadata: {
+      toolName: "naming-consistency",
+      timestamp: (/* @__PURE__ */ new Date()).toISOString()
+    }
+  };
+}
+function shouldIncludeSeverity(severity, minSeverity) {
+  return getSeverityLevel(severity) >= getSeverityLevel(minSeverity);
+}
+function getIssueType(type) {
+  if (!type) return IssueType.NamingInconsistency;
+  const typeMap = {
+    "naming-inconsistency": IssueType.NamingInconsistency,
+    "naming-quality": IssueType.NamingQuality,
+    "pattern-inconsistency": IssueType.PatternInconsistency,
+    "architecture-inconsistency": IssueType.ArchitectureInconsistency,
+    "error-handling": IssueType.PatternInconsistency,
+    "async-style": IssueType.PatternInconsistency,
+    "import-style": IssueType.PatternInconsistency,
+    "api-design": IssueType.PatternInconsistency
+  };
+  return typeMap[type] || IssueType.NamingInconsistency;
+}
+function transformToIssue(i) {
+  if (i.message && i.location) {
+    return {
+      type: getIssueType(i.type),
+      severity: i.severity,
+      message: i.message,
+      location: i.location,
+      suggestion: i.suggestion
+    };
+  }
+  if (i.identifier || i.type) {
+    const line = i.line || 1;
+    const column = i.column || 1;
+    return {
+      type: getIssueType(i.type),
+      severity: i.severity,
+      message: i.suggestion ? `Naming issue: ${i.suggestion}` : `Naming issue for '${i.identifier || "unknown"}'`,
+      location: {
+        file: i.file || i.fileName || "",
+        line,
+        column,
+        endLine: line,
+        endColumn: column + (i.identifier?.length || 10)
+      },
+      suggestion: i.suggestion
+    };
+  }
+  if (i.description || i.files) {
+    const fileName = Array.isArray(i.files) ? i.files[0] : i.file || "";
+    return {
+      type: getIssueType(i.type),
+      severity: i.severity,
+      message: i.description || "Pattern inconsistency found",
+      location: {
+        file: fileName,
+        line: 1,
+        column: 1,
+        endLine: 1,
+        endColumn: 10
+      },
+      suggestion: i.examples?.[0]
+    };
+  }
+  return {
+    type: getIssueType(i.type),
+    severity: i.severity,
+    message: i.message || "Unknown issue",
+    location: i.location || { file: "", line: 1, column: 1 },
+    suggestion: i.suggestion
+  };
+}
+
+export {
+  analyzeNamingAST,
+  analyzePatterns,
+  calculateConsistencyScore,
+  analyzeConsistency
+};