@aiready/consistency 0.8.30 → 0.8.32

package/dist/cli.js

@@ -49,6 +49,7 @@ function parseFile(filePath, content) {
       filePath: isTypeScript ? filePath : void 0
     });
   } catch (error) {
+    void error;
     return null;
   }
 }
@@ -287,6 +288,7 @@ var ScopeTracker = class {
 
 // src/utils/context-detector.ts
 function detectFileType(filePath, ast) {
+  void ast;
   const path = filePath.toLowerCase();
   if (path.match(/\.(test|spec)\.(ts|tsx|js|jsx)$/) || path.includes("__tests__")) {
     return "test";
@@ -317,7 +319,9 @@ function detectCodeLayer(ast) {
       }
       if (node.type === "FunctionDeclaration" && node.id) {
         const name = node.id.name;
-        if (name.match(/^(get|post|put|delete|patch|handle|api|route|controller)/i)) {
+        if (name.match(
+          /^(get|post|put|delete|patch|handle|api|route|controller)/i
+        )) {
           hasAPIIndicators++;
         }
         if (name.match(/^(calculate|process|validate|transform|compute|analyze)/i)) {
@@ -326,7 +330,9 @@ function detectCodeLayer(ast) {
         if (name.match(/^(find|create|update|delete|save|fetch|query|insert)/i)) {
           hasDataIndicators++;
         }
-        if (name.match(/^(format|parse|convert|normalize|sanitize|encode|decode)/i)) {
+        if (name.match(
+          /^(format|parse|convert|normalize|sanitize|encode|decode)/i
+        )) {
           hasUtilityIndicators++;
         }
       }
@@ -845,10 +851,15 @@ async function loadNamingConfig(files) {
   const rootDir = files.length > 0 ? (0, import_path.dirname)(files[0]) : process.cwd();
   const config = await (0, import_core.loadConfig)(rootDir);
   const consistencyConfig = config?.tools?.["consistency"];
-  const customAbbreviations = new Set(consistencyConfig?.acceptedAbbreviations || []);
+  const customAbbreviations = new Set(
+    consistencyConfig?.acceptedAbbreviations || []
+  );
   const customShortWords = new Set(consistencyConfig?.shortWords || []);
   const disabledChecks = new Set(consistencyConfig?.disableChecks || []);
-  const allAbbreviations = /* @__PURE__ */ new Set([...ACCEPTABLE_ABBREVIATIONS, ...customAbbreviations]);
+  const allAbbreviations = /* @__PURE__ */ new Set([
+    ...ACCEPTABLE_ABBREVIATIONS,
+    ...customAbbreviations
+  ]);
   const allShortWords = /* @__PURE__ */ new Set([...COMMON_SHORT_WORDS, ...customShortWords]);
   return {
     customAbbreviations,
@@ -897,9 +908,14 @@ function analyzeFileNamingAST(file, ast, allAbbreviations, allShortWords, disabl
         if ("params" in node) {
           for (const param of node.params) {
             if (param.type === "Identifier") {
-              scopeTracker.declareVariable(param.name, param, getLineNumber(param), {
-                isParameter: true
-              });
+              scopeTracker.declareVariable(
+                param.name,
+                param,
+                getLineNumber(param),
+                {
+                  isParameter: true
+                }
+              );
             } else if (param.type === "ObjectPattern" || param.type === "ArrayPattern") {
               extractIdentifiersFromPattern(param, scopeTracker, true);
             }
@@ -914,7 +930,7 @@ function analyzeFileNamingAST(file, ast, allAbbreviations, allShortWords, disabl
       }
       if (node.type === "VariableDeclarator") {
         if (node.id.type === "Identifier") {
-          const isInCoverage = isCoverageContext(node, ancestors);
+          void isCoverageContext(node, ancestors);
           scopeTracker.declareVariable(
             node.id.name,
             node.id,
@@ -926,7 +942,12 @@ function analyzeFileNamingAST(file, ast, allAbbreviations, allShortWords, disabl
             }
           );
         } else if (node.id.type === "ObjectPattern" || node.id.type === "ArrayPattern") {
-          extractIdentifiersFromPattern(node.id, scopeTracker, false, ancestors);
+          extractIdentifiersFromPattern(
+            node.id,
+            scopeTracker,
+            false,
+            ancestors
+          );
         }
       }
       if (node.type === "Identifier" && parent) {
@@ -993,7 +1014,10 @@ function analyzeFileNamingAST(file, ast, allAbbreviations, allShortWords, disabl
     }
     if (!disabledChecks.has("convention-mix") && file.match(/\.(ts|tsx|js|jsx)$/)) {
       if (name.includes("_") && !name.startsWith("_") && name.toLowerCase() === name) {
-        const camelCase = name.replace(/_([a-z])/g, (_, letter) => letter.toUpperCase());
+        const camelCase = name.replace(
+          /_([a-z])/g,
+          (_, letter) => letter.toUpperCase()
+        );
         issues.push({
           file,
           line,
@@ -1013,16 +1037,37 @@ function analyzeFileNamingAST(file, ast, allAbbreviations, allShortWords, disabl
           if (!name) return;
           const line = getLineNumber(node);
           if (["main", "init", "setup", "bootstrap"].includes(name)) return;
-          const hasActionVerb = name.match(/^(get|set|is|has|can|should|create|update|delete|fetch|load|save|process|handle|validate|check|find|search|filter|map|reduce|make|do|run|start|stop|build|parse|format|render|calculate|compute|generate|transform|convert|normalize|sanitize|encode|decode|compress|extract|merge|split|join|sort|compare|test|verify|ensure|apply|execute|invoke|call|emit|dispatch|trigger|listen|subscribe|unsubscribe|add|remove|clear|reset|toggle|enable|disable|open|close|connect|disconnect|send|receive|read|write|import|export|register|unregister|mount|unmount|track|store|persist|upsert|derive|classify|combine|discover|activate|require|assert|expect|mask|escape|sign|put|list|complete|page|safe|mock|pick|pluralize|text|count|detect|select)/);
-          const isFactoryPattern = name.match(/(Factory|Builder|Creator|Generator|Provider|Adapter|Mock)$/);
+          const hasActionVerb = name.match(
+            /^(get|set|is|has|can|should|create|update|delete|fetch|load|save|process|handle|validate|check|find|search|filter|map|reduce|make|do|run|start|stop|build|parse|format|render|calculate|compute|generate|transform|convert|normalize|sanitize|encode|decode|compress|extract|merge|split|join|sort|compare|test|verify|ensure|apply|execute|invoke|call|emit|dispatch|trigger|listen|subscribe|unsubscribe|add|remove|clear|reset|toggle|enable|disable|open|close|connect|disconnect|send|receive|read|write|import|export|register|unregister|mount|unmount|track|store|persist|upsert|derive|classify|combine|discover|activate|require|assert|expect|mask|escape|sign|put|list|complete|page|safe|mock|pick|pluralize|text|count|detect|select)/
+          );
+          const isFactoryPattern = name.match(
+            /(Factory|Builder|Creator|Generator|Provider|Adapter|Mock)$/
+          );
           const isEventHandler = name.match(/^on[A-Z]/);
           const isDescriptiveLong = name.length > 15;
           const isReactHook = name.match(/^use[A-Z]/);
           const isHelperPattern = name.match(/^(to|from|with|without|for|as|into)\w+/) || name.match(/^\w+(To|From|With|Without|For|As|Into)\w*$/);
-          const isUtilityName = ["cn", "proxy", "sitemap", "robots", "gtag"].includes(name);
-          const isLanguageKeyword = ["constructor", "toString", "valueOf", "toJSON"].includes(name);
-          const isFrameworkPattern = name.match(/^(goto|fill|click|select|submit|wait|expect)\w*/);
-          const isDescriptivePattern = name.match(/^(default|total|count|sum|avg|max|min|initial|current|previous|next)\w+/) || name.match(/\w+(Count|Total|Sum|Average|List|Map|Set|Config|Settings|Options|Props|Data|Info|Details|State|Status|Response|Result)$/);
+          const isUtilityName = [
+            "cn",
+            "proxy",
+            "sitemap",
+            "robots",
+            "gtag"
+          ].includes(name);
+          const isLanguageKeyword = [
+            "constructor",
+            "toString",
+            "valueOf",
+            "toJSON"
+          ].includes(name);
+          const isFrameworkPattern = name.match(
+            /^(goto|fill|click|select|submit|wait|expect)\w*/
+          );
+          const isDescriptivePattern = name.match(
+            /^(default|total|count|sum|avg|max|min|initial|current|previous|next)\w+/
+          ) || name.match(
+            /\w+(Count|Total|Sum|Average|List|Map|Set|Config|Settings|Options|Props|Data|Info|Details|State|Status|Response|Result)$/
+          );
           const capitalCount = (name.match(/[A-Z]/g) || []).length;
           const isCompoundWord = capitalCount >= 3;
           if (!hasActionVerb && !isFactoryPattern && !isEventHandler && !isDescriptiveLong && !isReactHook && !isHelperPattern && !isUtilityName && !isDescriptivePattern && !isCompoundWord && !isLanguageKeyword && !isFrameworkPattern) {
@@ -1042,6 +1087,7 @@ function analyzeFileNamingAST(file, ast, allAbbreviations, allShortWords, disabl
   return issues;
 }
 function extractIdentifiersFromPattern(pattern, scopeTracker, isParameter, ancestors) {
+  void ancestors;
   if (pattern.type === "ObjectPattern") {
     for (const prop of pattern.properties) {
       if (prop.type === "Property" && prop.value.type === "Identifier") {
@@ -1099,7 +1145,12 @@ async function analyzePythonNaming(files) {
       const code = await fs.promises.readFile(file, "utf-8");
       const result = parser.parse(code, file);
       for (const exp of result.exports) {
-        const nameIssue = checkPythonNaming(exp.name, exp.type, file, exp.loc?.start.line || 0);
+        const nameIssue = checkPythonNaming(
+          exp.name,
+          exp.type,
+          file,
+          exp.loc?.start.line || 0
+        );
         if (nameIssue) {
           issues.push(nameIssue);
         }
@@ -1107,7 +1158,12 @@ async function analyzePythonNaming(files) {
       for (const imp of result.imports) {
         for (const spec of imp.specifiers) {
           if (spec !== "*" && spec !== "default") {
-            const nameIssue = checkPythonNaming(spec, "variable", file, imp.loc?.start.line || 0);
+            const nameIssue = checkPythonNaming(
+              spec,
+              "variable",
+              file,
+              imp.loc?.start.line || 0
+            );
             if (nameIssue) {
               issues.push(nameIssue);
             }
@@ -1230,15 +1286,19 @@ async function analyzeErrorHandling(files) {
     }
   }
   const issues = [];
-  const strategiesUsed = Object.values(patterns).filter((p) => p.length > 0).length;
+  const strategiesUsed = Object.values(patterns).filter(
+    (p) => p.length > 0
+  ).length;
   if (strategiesUsed > 2) {
     issues.push({
-      files: [.../* @__PURE__ */ new Set([
-        ...patterns.tryCatch,
-        ...patterns.throwsError,
-        ...patterns.returnsNull,
-        ...patterns.returnsError
-      ])],
+      files: [
+        .../* @__PURE__ */ new Set([
+          ...patterns.tryCatch,
+          ...patterns.throwsError,
+          ...patterns.returnsNull,
+          ...patterns.returnsError
+        ])
+      ],
       type: "error-handling",
       description: "Inconsistent error handling strategies across codebase",
       examples: [
@@ -1359,6 +1419,7 @@ async function analyzeConsistency(options) {
     minSeverity = "info",
     ...scanOptions
   } = options;
+  void checkArchitecture;
   const filePaths = await (0, import_core4.scanFiles)(scanOptions);
   const tsJsFiles = filePaths.filter((f) => /\.(ts|tsx|js|jsx)$/i.test(f));
   const pythonFiles = filePaths.filter((f) => /\.py$/i.test(f));
@@ -1428,10 +1489,14 @@ async function analyzeConsistency(options) {
   results.sort((fileResultA, fileResultB) => {
     const severityOrder = { critical: 0, major: 1, minor: 2, info: 3 };
     const maxSeverityA = Math.min(
-      ...fileResultA.issues.map((i) => severityOrder[i.severity])
+      ...fileResultA.issues.map(
+        (i) => severityOrder[i.severity]
+      )
     );
     const maxSeverityB = Math.min(
-      ...fileResultB.issues.map((i) => severityOrder[i.severity])
+      ...fileResultB.issues.map(
+        (i) => severityOrder[i.severity]
+      )
     );
     if (maxSeverityA !== maxSeverityB) {
       return maxSeverityA - maxSeverityB;
@@ -1439,8 +1504,12 @@ async function analyzeConsistency(options) {
     return fileResultB.issues.length - fileResultA.issues.length;
   });
   const recommendations = generateRecommendations(namingIssues, patternIssues);
-  const namingCountFiltered = namingIssues.filter((i) => shouldIncludeSeverity(i.severity, minSeverity)).length;
-  const patternCountFiltered = patternIssues.filter((i) => shouldIncludeSeverity(i.severity, minSeverity)).length;
+  const namingCountFiltered = namingIssues.filter(
+    (i) => shouldIncludeSeverity(i.severity, minSeverity)
+  ).length;
+  const patternCountFiltered = patternIssues.filter(
+    (i) => shouldIncludeSeverity(i.severity, minSeverity)
+  ).length;
   return {
     summary: {
       totalIssues: namingCountFiltered + patternCountFiltered,
@@ -1459,19 +1528,26 @@ function shouldIncludeSeverity(severity, minSeverity) {
 }
 function calculateConsistencyScore(issues) {
   const weights = { critical: 10, major: 5, minor: 2, info: 1 };
-  const totalWeight = issues.reduce((sum, issue) => sum + weights[issue.severity], 0);
+  const totalWeight = issues.reduce(
+    (sum, issue) => sum + weights[issue.severity],
+    0
+  );
   return Math.max(0, 1 - totalWeight / 100);
 }
 function generateRecommendations(namingIssues, patternIssues) {
   const recommendations = [];
   if (namingIssues.length > 0) {
-    const conventionMixCount = namingIssues.filter((i) => i.type === "convention-mix").length;
+    const conventionMixCount = namingIssues.filter(
+      (i) => i.type === "convention-mix"
+    ).length;
     if (conventionMixCount > 0) {
       recommendations.push(
         `Standardize naming conventions: Found ${conventionMixCount} snake_case variables in TypeScript/JavaScript (use camelCase)`
       );
     }
-    const poorNamingCount = namingIssues.filter((i) => i.type === "poor-naming").length;
+    const poorNamingCount = namingIssues.filter(
+      (i) => i.type === "poor-naming"
+    ).length;
     if (poorNamingCount > 0) {
       recommendations.push(
         `Improve variable naming: Found ${poorNamingCount} single-letter or unclear variable names`
@@ -1479,7 +1555,9 @@ function generateRecommendations(namingIssues, patternIssues) {
     }
   }
   if (patternIssues.length > 0) {
-    const errorHandlingIssues = patternIssues.filter((i) => i.type === "error-handling");
+    const errorHandlingIssues = patternIssues.filter(
+      (i) => i.type === "error-handling"
+    );
     if (errorHandlingIssues.length > 0) {
       recommendations.push(
         "Standardize error handling strategy across the codebase (prefer try-catch with typed errors)"
@@ -1499,7 +1577,9 @@ function generateRecommendations(namingIssues, patternIssues) {
     }
   }
   if (recommendations.length === 0) {
-    recommendations.push("No major consistency issues found! Your codebase follows good practices.");
+    recommendations.push(
+      "No major consistency issues found! Your codebase follows good practices."
+    );
   }
   return recommendations;
 }
@@ -1510,7 +1590,11 @@ var import_fs2 = require("fs");
 var import_path2 = require("path");
 var import_core5 = require("@aiready/core");
 var program = new import_commander.Command();
-program.name("aiready-consistency").description("Detect consistency patterns in naming, code structure, and architecture").version("0.1.0").addHelpText("after", `
+program.name("aiready-consistency").description(
+  "Detect consistency patterns in naming, code structure, and architecture"
+).version("0.1.0").addHelpText(
+  "after",
+  `
 LANGUAGE SUPPORT:
   Supported: TypeScript (.ts, .tsx), JavaScript (.js, .jsx)
   Note: Python, Java, and other language files will be safely ignored
@@ -1529,7 +1613,18 @@ EXAMPLES:
   aiready-consistency . --no-naming                   # Skip naming checks
   aiready-consistency . --min-severity major          # Only show major+ patterns
   aiready-consistency . --output json > report.json   # JSON export
-`).argument("<directory>", "Directory to analyze").option("--naming", "Check naming conventions and quality (default: true)").option("--no-naming", "Skip naming analysis").option("--patterns", "Check code pattern consistency (default: true)").option("--no-patterns", "Skip pattern analysis").option("--architecture", "Check architectural consistency (not yet implemented)").option("--min-severity <level>", "Minimum severity: info|minor|major|critical. Default: info").option("--include <patterns>", "File patterns to include (comma-separated)").option("--exclude <patterns>", "File patterns to exclude (comma-separated)").option("-o, --output <format>", "Output format: console|json|markdown", "console").option("--output-file <path>", "Output file path (for json/markdown)").action(async (directory, options) => {
+`
+).argument("<directory>", "Directory to analyze").option("--naming", "Check naming conventions and quality (default: true)").option("--no-naming", "Skip naming analysis").option("--patterns", "Check code pattern consistency (default: true)").option("--no-patterns", "Skip pattern analysis").option(
+  "--architecture",
+  "Check architectural consistency (not yet implemented)"
+).option(
+  "--min-severity <level>",
+  "Minimum severity: info|minor|major|critical. Default: info"
+).option("--include <patterns>", "File patterns to include (comma-separated)").option("--exclude <patterns>", "File patterns to exclude (comma-separated)").option(
+  "-o, --output <format>",
+  "Output format: console|json|markdown",
+  "console"
+).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_core5.loadConfig)(directory);
@@ -1595,7 +1690,11 @@ function displayConsoleReport(report, elapsedTime) {
   console.log(`Analysis Time: ${import_chalk.default.gray(elapsedTime + "s")}
 `);
   if (summary.totalIssues === 0) {
-    console.log(import_chalk.default.green("\u2728 No consistency patterns found! Your codebase is AI-friendly.\n"));
+    console.log(
+      import_chalk.default.green(
+        "\u2728 No consistency patterns found! Your codebase is AI-friendly.\n"
+      )
+    );
     return;
   }
   const namingResults = results.filter(

package/dist/cli.mjs

@@ -1,16 +1,24 @@
 #!/usr/bin/env node
 import {
   analyzeConsistency
-} from "./chunk-YEHXYHGY.mjs";
+} from "./chunk-J5IFYDVU.mjs";
 
 // src/cli.ts
 import { Command } from "commander";
 import chalk from "chalk";
 import { writeFileSync, mkdirSync, existsSync } from "fs";
 import { dirname } from "path";
-import { loadConfig, mergeConfigWithDefaults, resolveOutputPath } from "@aiready/core";
+import {
+  loadConfig,
+  mergeConfigWithDefaults,
+  resolveOutputPath
+} from "@aiready/core";
 var program = new Command();
-program.name("aiready-consistency").description("Detect consistency patterns in naming, code structure, and architecture").version("0.1.0").addHelpText("after", `
+program.name("aiready-consistency").description(
+  "Detect consistency patterns in naming, code structure, and architecture"
+).version("0.1.0").addHelpText(
+  "after",
+  `
 LANGUAGE SUPPORT:
   Supported: TypeScript (.ts, .tsx), JavaScript (.js, .jsx)
   Note: Python, Java, and other language files will be safely ignored
@@ -29,7 +37,18 @@ EXAMPLES:
   aiready-consistency . --no-naming                   # Skip naming checks
   aiready-consistency . --min-severity major          # Only show major+ patterns
   aiready-consistency . --output json > report.json   # JSON export
-`).argument("<directory>", "Directory to analyze").option("--naming", "Check naming conventions and quality (default: true)").option("--no-naming", "Skip naming analysis").option("--patterns", "Check code pattern consistency (default: true)").option("--no-patterns", "Skip pattern analysis").option("--architecture", "Check architectural consistency (not yet implemented)").option("--min-severity <level>", "Minimum severity: info|minor|major|critical. Default: info").option("--include <patterns>", "File patterns to include (comma-separated)").option("--exclude <patterns>", "File patterns to exclude (comma-separated)").option("-o, --output <format>", "Output format: console|json|markdown", "console").option("--output-file <path>", "Output file path (for json/markdown)").action(async (directory, options) => {
+`
+).argument("<directory>", "Directory to analyze").option("--naming", "Check naming conventions and quality (default: true)").option("--no-naming", "Skip naming analysis").option("--patterns", "Check code pattern consistency (default: true)").option("--no-patterns", "Skip pattern analysis").option(
+  "--architecture",
+  "Check architectural consistency (not yet implemented)"
+).option(
+  "--min-severity <level>",
+  "Minimum severity: info|minor|major|critical. Default: info"
+).option("--include <patterns>", "File patterns to include (comma-separated)").option("--exclude <patterns>", "File patterns to exclude (comma-separated)").option(
+  "-o, --output <format>",
+  "Output format: console|json|markdown",
+  "console"
+).option("--output-file <path>", "Output file path (for json/markdown)").action(async (directory, options) => {
   console.log(chalk.blue("\u{1F50D} Analyzing consistency...\n"));
   const startTime = Date.now();
   const config = await loadConfig(directory);
@@ -95,7 +114,11 @@ function displayConsoleReport(report, elapsedTime) {
   console.log(`Analysis Time: ${chalk.gray(elapsedTime + "s")}
 `);
   if (summary.totalIssues === 0) {
-    console.log(chalk.green("\u2728 No consistency patterns found! Your codebase is AI-friendly.\n"));
+    console.log(
+      chalk.green(
+        "\u2728 No consistency patterns found! Your codebase is AI-friendly.\n"
+      )
+    );
     return;
   }
   const namingResults = results.filter(