@aiready/consistency 0.21.13 → 0.21.14

package/dist/cli.js

@@ -794,7 +794,7 @@ async function analyzeNamingGeneralized(files) {
             "HEAD"
           ].includes(exp.name))
             continue;
-          if (conventions.constantPattern.test(exp.name) || conventions.variablePattern.test(exp.name)) {
+          if (conventions.constantPattern.test(exp.name) || conventions.variablePattern.test(exp.name) || /^[A-Z][a-zA-Z0-9]*$/.test(exp.name)) {
             continue;
           }
           pattern = conventions.constantPattern;
@@ -907,6 +907,23 @@ async function analyzePatterns(filePaths) {
 var import_core5 = require("@aiready/core");
 function calculateConsistencyScore(issues, totalFilesAnalyzed, costConfig) {
   void costConfig;
+  if (!Array.isArray(issues)) {
+    return {
+      toolName: import_core5.ToolName.NamingConsistency,
+      score: 100,
+      rawMetrics: {
+        totalIssues: 0,
+        criticalIssues: 0,
+        majorIssues: 0,
+        minorIssues: 0,
+        issuesPerFile: 0,
+        avgWeightedIssuesPerFile: 0,
+        estimatedDeveloperHours: 0
+      },
+      factors: [],
+      recommendations: []
+    };
+  }
   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;

package/dist/cli.mjs

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

package/dist/index.js

@@ -813,7 +813,7 @@ async function analyzeNamingGeneralized(files) {
             "HEAD"
           ].includes(exp.name))
             continue;
-          if (conventions.constantPattern.test(exp.name) || conventions.variablePattern.test(exp.name)) {
+          if (conventions.constantPattern.test(exp.name) || conventions.variablePattern.test(exp.name) || /^[A-Z][a-zA-Z0-9]*$/.test(exp.name)) {
             continue;
           }
           pattern = conventions.constantPattern;
@@ -926,6 +926,23 @@ async function analyzePatterns(filePaths) {
 var import_core5 = require("@aiready/core");
 function calculateConsistencyScore(issues, totalFilesAnalyzed, costConfig) {
   void costConfig;
+  if (!Array.isArray(issues)) {
+    return {
+      toolName: import_core5.ToolName.NamingConsistency,
+      score: 100,
+      rawMetrics: {
+        totalIssues: 0,
+        criticalIssues: 0,
+        majorIssues: 0,
+        minorIssues: 0,
+        issuesPerFile: 0,
+        avgWeightedIssuesPerFile: 0,
+        estimatedDeveloperHours: 0
+      },
+      factors: [],
+      recommendations: []
+    };
+  }
   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;

package/dist/index.mjs

@@ -6,7 +6,7 @@ import {
   analyzePatterns,
   calculateConsistencyScore,
   detectNamingConventions
-} from "./chunk-7DKVZNY7.mjs";
+} from "./chunk-R2WYI6DR.mjs";
 
 // src/index.ts
 import { ToolRegistry } from "@aiready/core";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aiready/consistency",
-  "version": "0.21.13",
+  "version": "0.21.14",
   "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": "^14.0.0",
-    "@aiready/core": "0.24.14"
+    "@aiready/core": "0.24.15"
   },
   "devDependencies": {
     "@types/node": "^24.0.0",

package/src/__tests__/analyzer.test.ts

@@ -247,6 +247,22 @@ describe('naming-generalized const naming', () => {
     }
   });
 
+  it('should allow PascalCase for exported constants (schemas/mocks)', () => {
+    // This test verifies that PascalCase constants like UserSchema or MockData are allowed.
+    const pascalCaseConstants = [
+      'UserSchema',
+      'MockData',
+      'ConfigOptions',
+      'AppRoutes',
+      'FeatureToggle',
+    ];
+
+    const pascalCasePattern = /^[A-Z][a-zA-Z0-9]*$/;
+    for (const name of pascalCaseConstants) {
+      expect(pascalCasePattern.test(name)).toBe(true);
+    }
+  });
+
   it('should NOT flag SCREAMING_SNAKE_CASE constants as naming issues', async () => {
     // After the fix, SCREAMING_SNAKE_CASE constants should not generate naming issues
     const report = await analyzeConsistency({

package/src/analyzers/naming-generalized.ts

@@ -112,12 +112,12 @@ export async function analyzeNamingGeneralized(
           )
             continue;
 
-          // Allow both SCREAMING_SNAKE_CASE and camelCase for exported constants.
-          // Module-level constants (config objects, paths, etc.) often use SCREAMING_SNAKE_CASE
-          // which is a valid convention for constants, regardless of whether they are primitives.
+          // Allow SCREAMING_SNAKE_CASE, camelCase, or PascalCase for exported constants.
+          // PascalCase is common for Zod schemas (UserSchema), mocks (MockData), or configuration objects.
           if (
             conventions.constantPattern.test(exp.name) ||
-            conventions.variablePattern.test(exp.name)
+            conventions.variablePattern.test(exp.name) ||
+            /^[A-Z][a-zA-Z0-9]*$/.test(exp.name)
           ) {
             continue;
           }

package/src/scoring.ts

@@ -18,6 +18,26 @@ export function calculateConsistencyScore(
 ): ToolScoringOutput {
   // Parameter reserved for future configuration; reference to avoid lint warnings
   void costConfig;
+
+  // Defensive check for issues array to prevent crashes
+  if (!Array.isArray(issues)) {
+    return {
+      toolName: ToolName.NamingConsistency,
+      score: 100,
+      rawMetrics: {
+        totalIssues: 0,
+        criticalIssues: 0,
+        majorIssues: 0,
+        minorIssues: 0,
+        issuesPerFile: 0,
+        avgWeightedIssuesPerFile: 0,
+        estimatedDeveloperHours: 0,
+      },
+      factors: [],
+      recommendations: [],
+    };
+  }
+
   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;