@aiready/consistency 0.20.13 → 0.20.14

package/dist/cli.js

@@ -301,16 +301,22 @@ function analyzeIdentifiers(ast, filePath, context) {
         );
       }
       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 }
+              { isParameter: true, isArrowParameter }
             );
           } else if (param.type === "ObjectPattern") {
-            extractDestructuredIdentifiers(param, true, scopeTracker);
+            extractDestructuredIdentifiers(
+              param,
+              true,
+              scopeTracker,
+              isArrowParameter
+            );
           }
         });
       }
@@ -357,7 +363,7 @@ function checkVariableNaming(varInfo, file, issues, context) {
   if (isAcceptableInContext(name, context, options)) {
     return;
   }
-  if (name.length === 1 && !options.isLoopVariable) {
+  if (name.length === 1 && !options.isLoopVariable && !options.isArrowParameter) {
     const severity = adjustSeverity(import_core2.Severity.Minor, context, "poor-naming");
     issues.push({
       file,
@@ -457,7 +463,7 @@ var ScopeTracker = class {
     return this.variables;
   }
 };
-function extractDestructuredIdentifiers(node, isParameter, scopeTracker) {
+function extractDestructuredIdentifiers(node, isParameter, scopeTracker, isArrowParameter = false) {
   if (node.type === "ObjectPattern") {
     node.properties.forEach((prop) => {
       if (prop.type === "Property" && prop.value.type === "Identifier") {
@@ -467,7 +473,8 @@ function extractDestructuredIdentifiers(node, isParameter, scopeTracker) {
           getLineNumber(prop.value),
           {
             isParameter,
-            isDestructured: true
+            isDestructured: true,
+            isArrowParameter
           }
         );
       }
@@ -481,7 +488,8 @@ function extractDestructuredIdentifiers(node, isParameter, scopeTracker) {
           getLineNumber(element),
           {
             isParameter,
-            isDestructured: true
+            isDestructured: true,
+            isArrowParameter
           }
         );
       }
@@ -554,8 +562,22 @@ async function analyzeNamingGeneralized(files) {
         } 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;
@@ -579,7 +601,7 @@ async function analyzeNamingGeneralized(files) {
           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.test(spec) && (!conventions.typePattern || !conventions.typePattern.test(spec)) && (!conventions.interfacePattern || !conventions.interfacePattern.test(spec))) {
+          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,

package/dist/cli.mjs

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

package/dist/index.js

@@ -309,16 +309,22 @@ function analyzeIdentifiers(ast, filePath, context) {
         );
       }
       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 }
+              { isParameter: true, isArrowParameter }
             );
           } else if (param.type === "ObjectPattern") {
-            extractDestructuredIdentifiers(param, true, scopeTracker);
+            extractDestructuredIdentifiers(
+              param,
+              true,
+              scopeTracker,
+              isArrowParameter
+            );
           }
         });
       }
@@ -365,7 +371,7 @@ function checkVariableNaming(varInfo, file, issues, context) {
   if (isAcceptableInContext(name, context, options)) {
     return;
   }
-  if (name.length === 1 && !options.isLoopVariable) {
+  if (name.length === 1 && !options.isLoopVariable && !options.isArrowParameter) {
     const severity = adjustSeverity(import_core2.Severity.Minor, context, "poor-naming");
     issues.push({
       file,
@@ -465,7 +471,7 @@ var ScopeTracker = class {
     return this.variables;
   }
 };
-function extractDestructuredIdentifiers(node, isParameter, scopeTracker) {
+function extractDestructuredIdentifiers(node, isParameter, scopeTracker, isArrowParameter = false) {
   if (node.type === "ObjectPattern") {
     node.properties.forEach((prop) => {
       if (prop.type === "Property" && prop.value.type === "Identifier") {
@@ -475,7 +481,8 @@ function extractDestructuredIdentifiers(node, isParameter, scopeTracker) {
           getLineNumber(prop.value),
           {
             isParameter,
-            isDestructured: true
+            isDestructured: true,
+            isArrowParameter
           }
         );
       }
@@ -489,7 +496,8 @@ function extractDestructuredIdentifiers(node, isParameter, scopeTracker) {
           getLineNumber(element),
           {
             isParameter,
-            isDestructured: true
+            isDestructured: true,
+            isArrowParameter
           }
         );
       }
@@ -562,8 +570,22 @@ async function analyzeNamingGeneralized(files) {
         } 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;
@@ -587,7 +609,7 @@ async function analyzeNamingGeneralized(files) {
           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.test(spec) && (!conventions.typePattern || !conventions.typePattern.test(spec)) && (!conventions.interfacePattern || !conventions.interfacePattern.test(spec))) {
+          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,

package/dist/index.mjs

@@ -2,7 +2,7 @@ import {
   analyzeConsistency,
   analyzeNamingAST,
   analyzePatterns
-} from "./chunk-LSXZH6X6.mjs";
+} from "./chunk-6H3JHDP7.mjs";
 
 // src/index.ts
 import { ToolRegistry } from "@aiready/core";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aiready/consistency",
-  "version": "0.20.13",
+  "version": "0.20.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.23.14"
+    "@aiready/core": "0.23.15"
   },
   "devDependencies": {
     "@types/node": "^24.0.0",

package/src/analyzers/naming-ast.ts

@@ -68,17 +68,23 @@ function analyzeIdentifiers(
         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 }
+              { isParameter: true, isArrowParameter }
             );
           } else if (param.type === 'ObjectPattern') {
             // Handle destructured parameters: { id, name }
-            extractDestructuredIdentifiers(param, true, scopeTracker);
+            extractDestructuredIdentifiers(
+              param,
+              true,
+              scopeTracker,
+              isArrowParameter
+            );
           }
         });
       }
@@ -160,7 +166,11 @@ function checkVariableNaming(
   }
 
   // 1. Single letter names
-  if (name.length === 1 && !options.isLoopVariable) {
+  if (
+    name.length === 1 &&
+    !options.isLoopVariable &&
+    !options.isArrowParameter
+  ) {
     const severity = adjustSeverity(Severity.Minor, context, 'poor-naming');
     issues.push({
       file,
@@ -290,7 +300,8 @@ class ScopeTracker {
 function extractDestructuredIdentifiers(
   node: TSESTree.ObjectPattern | TSESTree.ArrayPattern,
   isParameter: boolean,
-  scopeTracker: ScopeTracker
+  scopeTracker: ScopeTracker,
+  isArrowParameter: boolean = false
 ) {
   if (node.type === 'ObjectPattern') {
     node.properties.forEach((prop) => {
@@ -302,6 +313,7 @@ function extractDestructuredIdentifiers(
           {
             isParameter,
             isDestructured: true,
+            isArrowParameter,
           }
         );
       }
@@ -316,6 +328,7 @@ function extractDestructuredIdentifiers(
           {
             isParameter,
             isDestructured: true,
+            isArrowParameter,
           }
         );
       }

package/src/analyzers/naming-generalized.ts

@@ -88,8 +88,30 @@ export async function analyzeNamingGeneralized(
         } else if (exp.type === 'type' && conventions.typePattern) {
           pattern = conventions.typePattern;
         } else if (exp.type === 'function') {
+          // Allow PascalCase (React components) or UPPER_CASE (HTTP methods) for exported functions
+          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') {
+          // Exempt standard Next.js / API names
+          if (
+            [
+              'handler',
+              'GET',
+              'POST',
+              'PUT',
+              'DELETE',
+              'PATCH',
+              'OPTIONS',
+              'HEAD',
+            ].includes(exp.name)
+          )
+            continue;
+
           // Only enforce SCREAMING_SNAKE_CASE for primitive constants (strings, numbers,
           // booleans). Object literals, class instances, and tool definitions are
           // camelCase by convention (e.g. `logger`, `githubTools`, `RemediationSwarm`).
@@ -125,10 +147,12 @@ export async function analyzeNamingGeneralized(
           if (
             !conventions.variablePattern.test(spec) &&
             !conventions.classPattern.test(spec) &&
-            !conventions.constantPattern.test(spec) &&
+            (!conventions.constantPattern ||
+              !conventions.constantPattern.test(spec)) &&
             (!conventions.typePattern || !conventions.typePattern.test(spec)) &&
             (!conventions.interfacePattern ||
-              !conventions.interfacePattern.test(spec))
+              !conventions.interfacePattern.test(spec)) &&
+            !/^[A-Z][A-Z0-9_]*$/.test(spec)
           ) {
             // This is often a 'convention-mix' issue (e.g. importing snake_case into camelCase project)
             issues.push({