@aiready/consistency 0.6.0 → 0.6.2

package/dist/cli.js

@@ -818,7 +818,7 @@ var ACCEPTABLE_ABBREVIATIONS = /* @__PURE__ */ new Set([
 async function analyzeNamingAST(files) {
   const issues = [];
   const rootDir = files.length > 0 ? (0, import_path.dirname)(files[0]) : process.cwd();
-  const config = (0, import_core.loadConfig)(rootDir);
+  const config = await (0, import_core.loadConfig)(rootDir);
   const consistencyConfig = config?.tools?.["consistency"];
   const customAbbreviations = new Set(consistencyConfig?.acceptedAbbreviations || []);
   const customShortWords = new Set(consistencyConfig?.shortWords || []);
@@ -972,17 +972,19 @@ 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)/);
+          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+/);
+          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 capitalCount = (name.match(/[A-Z]/g) || []).length;
           const isCompoundWord = capitalCount >= 3;
-          if (!hasActionVerb && !isFactoryPattern && !isEventHandler && !isDescriptiveLong && !isReactHook && !isHelperPattern && !isUtilityName && !isDescriptivePattern && !isCompoundWord) {
+          if (!hasActionVerb && !isFactoryPattern && !isEventHandler && !isDescriptiveLong && !isReactHook && !isHelperPattern && !isUtilityName && !isDescriptivePattern && !isCompoundWord && !isLanguageKeyword && !isFrameworkPattern) {
             issues.push({
               file,
               line,
@@ -1254,6 +1256,19 @@ 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])
+    );
+    const maxSeverityB = Math.min(
+      ...fileResultB.issues.map((i) => severityOrder[i.severity])
+    );
+    if (maxSeverityA !== maxSeverityB) {
+      return maxSeverityA - maxSeverityB;
+    }
+    return fileResultB.issues.length - fileResultA.issues.length;
+  });
   const recommendations = generateRecommendations(namingIssues, patternIssues);
   return {
     summary: {
@@ -1342,7 +1357,7 @@ EXAMPLES:
 `).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 = (0, import_core4.loadConfig)(directory);
+  const config = await (0, import_core4.loadConfig)(directory);
   const defaults = {
     checkNaming: true,
     checkPatterns: true,

package/dist/cli.mjs

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 import {
   analyzeConsistency
-} from "./chunk-IVRBV7SE.mjs";
+} from "./chunk-ON73WHHU.mjs";
 
 // src/cli.ts
 import { Command } from "commander";
@@ -28,7 +28,7 @@ EXAMPLES:
 `).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 = loadConfig(directory);
+  const config = await loadConfig(directory);
   const defaults = {
     checkNaming: true,
     checkPatterns: true,

package/dist/index.js

@@ -820,7 +820,7 @@ var ACCEPTABLE_ABBREVIATIONS = /* @__PURE__ */ new Set([
 async function analyzeNamingAST(files) {
   const issues = [];
   const rootDir = files.length > 0 ? (0, import_path.dirname)(files[0]) : process.cwd();
-  const config = (0, import_core.loadConfig)(rootDir);
+  const config = await (0, import_core.loadConfig)(rootDir);
   const consistencyConfig = config?.tools?.["consistency"];
   const customAbbreviations = new Set(consistencyConfig?.acceptedAbbreviations || []);
   const customShortWords = new Set(consistencyConfig?.shortWords || []);
@@ -974,17 +974,19 @@ 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)/);
+          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+/);
+          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 capitalCount = (name.match(/[A-Z]/g) || []).length;
           const isCompoundWord = capitalCount >= 3;
-          if (!hasActionVerb && !isFactoryPattern && !isEventHandler && !isDescriptiveLong && !isReactHook && !isHelperPattern && !isUtilityName && !isDescriptivePattern && !isCompoundWord) {
+          if (!hasActionVerb && !isFactoryPattern && !isEventHandler && !isDescriptiveLong && !isReactHook && !isHelperPattern && !isUtilityName && !isDescriptivePattern && !isCompoundWord && !isLanguageKeyword && !isFrameworkPattern) {
             issues.push({
               file,
               line,
@@ -1256,6 +1258,19 @@ 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])
+    );
+    const maxSeverityB = Math.min(
+      ...fileResultB.issues.map((i) => severityOrder[i.severity])
+    );
+    if (maxSeverityA !== maxSeverityB) {
+      return maxSeverityA - maxSeverityB;
+    }
+    return fileResultB.issues.length - fileResultA.issues.length;
+  });
   const recommendations = generateRecommendations(namingIssues, patternIssues);
   return {
     summary: {
@@ -1708,7 +1723,7 @@ var ACCEPTABLE_ABBREVIATIONS2 = /* @__PURE__ */ new Set([
 async function analyzeNaming(files) {
   const issues = [];
   const rootDir = files.length > 0 ? (0, import_path2.dirname)(files[0]) : process.cwd();
-  const config = (0, import_core4.loadConfig)(rootDir);
+  const config = await (0, import_core4.loadConfig)(rootDir);
   const consistencyConfig = config?.tools?.["consistency"];
   const customAbbreviations = new Set(consistencyConfig?.acceptedAbbreviations || []);
   const customShortWords = new Set(consistencyConfig?.shortWords || []);

package/dist/index.mjs

@@ -2,7 +2,7 @@ import {
   analyzeConsistency,
   analyzeNamingAST,
   analyzePatterns
-} from "./chunk-IVRBV7SE.mjs";
+} from "./chunk-ON73WHHU.mjs";
 
 // src/analyzers/naming.ts
 import { readFileContent, loadConfig } from "@aiready/core";
@@ -392,7 +392,7 @@ var ACCEPTABLE_ABBREVIATIONS = /* @__PURE__ */ new Set([
 async function analyzeNaming(files) {
   const issues = [];
   const rootDir = files.length > 0 ? dirname(files[0]) : process.cwd();
-  const config = loadConfig(rootDir);
+  const config = await loadConfig(rootDir);
   const consistencyConfig = config?.tools?.["consistency"];
   const customAbbreviations = new Set(consistencyConfig?.acceptedAbbreviations || []);
   const customShortWords = new Set(consistencyConfig?.shortWords || []);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aiready/consistency",
-  "version": "0.6.0",
+  "version": "0.6.2",
   "description": "Detects consistency issues in naming, patterns, and architecture that confuse AI models",
   "main": "./dist/index.js",
   "module": "./dist/index.mjs",
@@ -43,7 +43,7 @@
     "@typescript-eslint/typescript-estree": "^8.53.0",
     "chalk": "^5.3.0",
     "commander": "^12.1.0",
-    "@aiready/core": "0.7.1"
+    "@aiready/core": "0.7.2"
   },
   "devDependencies": {
     "@types/node": "^22.10.5",

package/src/analyzer.ts

@@ -94,6 +94,27 @@ export async function analyzeConsistency(
     });
   }
 
+  // Sort results by severity first, then by issue count per file
+  results.sort((fileResultA, fileResultB) => {
+    const severityOrder = { critical: 0, major: 1, minor: 2, info: 3 };
+    
+    // Get highest severity in each file
+    const maxSeverityA = Math.min(
+      ...fileResultA.issues.map(i => severityOrder[(i as ConsistencyIssue).severity])
+    );
+    const maxSeverityB = Math.min(
+      ...fileResultB.issues.map(i => severityOrder[(i as ConsistencyIssue).severity])
+    );
+    
+    // Sort by severity first
+    if (maxSeverityA !== maxSeverityB) {
+      return maxSeverityA - maxSeverityB;
+    }
+    
+    // Then by issue count (descending)
+    return fileResultB.issues.length - fileResultA.issues.length;
+  });
+
   // Generate recommendations
   const recommendations = generateRecommendations(namingIssues, patternIssues);
 

package/src/analyzers/naming-ast.ts

@@ -74,7 +74,7 @@ export async function analyzeNamingAST(files: string[]): Promise<NamingIssue[]>
 
   // Load config
   const rootDir = files.length > 0 ? dirname(files[0]) : process.cwd();
-  const config = loadConfig(rootDir);
+  const config = await loadConfig(rootDir);
   const consistencyConfig = config?.tools?.['consistency'];
 
   // Merge custom configuration
@@ -289,14 +289,17 @@ function analyzeFileNamingAST(
           if (['main', 'init', 'setup', 'bootstrap'].includes(name)) return;
 
           // Check for action verbs and patterns
-          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)/);
+          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+/);
+          const isHelperPattern = name.match(/^(to|from|with|without|for|as|into)\w+/) || 
+                                  name.match(/^\w+(To|From|With|Without|For|As|Into)\w*$/); // xForY, xToY patterns
           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*/); // Page Object Model, test framework patterns
           
           // Descriptive patterns: countX, totalX, etc.
           const isDescriptivePattern = name.match(/^(default|total|count|sum|avg|max|min|initial|current|previous|next)\w+/) ||
@@ -306,7 +309,7 @@ function analyzeFileNamingAST(
           const capitalCount = (name.match(/[A-Z]/g) || []).length;
           const isCompoundWord = capitalCount >= 3; // daysSinceLastCommit has 4 capitals
 
-          if (!hasActionVerb && !isFactoryPattern && !isEventHandler && !isDescriptiveLong && !isReactHook && !isHelperPattern && !isUtilityName && !isDescriptivePattern && !isCompoundWord) {
+          if (!hasActionVerb && !isFactoryPattern && !isEventHandler && !isDescriptiveLong && !isReactHook && !isHelperPattern && !isUtilityName && !isDescriptivePattern && !isCompoundWord && !isLanguageKeyword && !isFrameworkPattern) {
             issues.push({
               file,
               line,

package/src/analyzers/naming.ts

@@ -82,7 +82,7 @@ export async function analyzeNaming(files: string[]): Promise<NamingIssue[]> {
 
   // Load config from the first file's directory (or project root)
   const rootDir = files.length > 0 ? dirname(files[0]) : process.cwd();
-  const config = loadConfig(rootDir);
+  const config = await loadConfig(rootDir);
   const consistencyConfig = config?.tools?.['consistency'];
 
   // Merge custom abbreviations and short words with defaults

package/src/cli.ts

@@ -47,7 +47,7 @@ EXAMPLES:
     const startTime = Date.now();
 
     // Load config file if it exists
-    const config = loadConfig(directory);
+    const config = await loadConfig(directory);
 
     // Define defaults
     const defaults = {