@aiready/consistency 0.3.3 → 0.3.4

package/src/analyzers/naming.ts

@@ -1,35 +1,69 @@
 import { readFileContent } from '@aiready/core';
 import type { NamingIssue } from '../types';
 
+// Common short English words that are NOT abbreviations (full, valid words)
+const COMMON_SHORT_WORDS = new Set([
+  // Full English words (1-3 letters)
+  'day', 'key', 'net', 'to', 'go', 'for', 'not', 'new', 'old', 'top', 'end',
+  'run', 'try', 'use', 'get', 'set', 'add', 'put', 'map', 'log', 'row', 'col',
+  'tab', 'box', 'div', 'nav', 'tag', 'any', 'all', 'one', 'two', 'out', 'off',
+  'on', 'yes', 'no', 'now', 'max', 'min', 'sum', 'avg', 'ref', 'src', 'dst',
+  'raw', 'def', 'sub', 'pub', 'pre', 'mid', 'alt', 'opt', 'tmp', 'ext', 'sep',
+  // Additional full words commonly flagged
+  'tax', 'cat', 'dog', 'car', 'bus', 'web', 'app', 'war', 'law', 'pay', 'buy',
+  'win', 'cut', 'hit', 'hot', 'pop', 'job', 'age', 'act', 'let', 'lot', 'bad',
+  'big', 'far', 'few', 'own', 'per', 'red', 'low', 'see', 'six', 'ten', 'way',
+  'who', 'why', 'yet', 'via', 'due', 'fee', 'fun', 'gas', 'gay', 'god', 'gun',
+  'guy', 'ice', 'ill', 'kid', 'mad', 'man', 'mix', 'mom', 'mrs', 'nor', 'odd',
+  'oil', 'pan', 'pet', 'pit', 'pot', 'pow', 'pro', 'raw', 'rep', 'rid', 'sad',
+  'sea', 'sit', 'sky', 'son', 'tea', 'tie', 'tip', 'van', 'war', 'win', 'won'
+]);
+
 // Comprehensive list of acceptable abbreviations and acronyms
 const ACCEPTABLE_ABBREVIATIONS = new Set([
   // Standard identifiers
   'id', 'uid', 'gid', 'pid',
+  // Loop counters and iterators
+  'i', 'j', 'k', 'n', 'm',
   // Web/Network
   'url', 'uri', 'api', 'cdn', 'dns', 'ip', 'tcp', 'udp', 'http', 'ssl', 'tls',
-  'utm', 'seo', 'rss', 'xhr', 'ajax',
+  'utm', 'seo', 'rss', 'xhr', 'ajax', 'cors', 'ws', 'wss',
   // Data formats
   'json', 'xml', 'yaml', 'csv', 'html', 'css', 'svg', 'pdf',
+  // File types & extensions
+  'img', 'txt', 'doc', 'docx', 'xlsx', 'ppt', 'md', 'rst', 'jpg', 'png', 'gif',
   // Databases
-  'db', 'sql', 'orm', 'dao', 'dto',
+  'db', 'sql', 'orm', 'dao', 'dto', 'ddb', 'rds', 'nosql',
   // File system
   'fs', 'dir', 'tmp', 'src', 'dst', 'bin', 'lib', 'pkg',
   // Operating system
-  'os', 'env', 'arg', 'cli', 'cmd', 'exe',
+  'os', 'env', 'arg', 'cli', 'cmd', 'exe', 'cwd', 'pwd',
   // UI/UX
   'ui', 'ux', 'gui', 'dom', 'ref',
   // Request/Response
-  'req', 'res', 'ctx', 'err', 'msg',
+  'req', 'res', 'ctx', 'err', 'msg', 'auth',
   // Mathematics/Computing
   'max', 'min', 'avg', 'sum', 'abs', 'cos', 'sin', 'tan', 'log', 'exp',
-  'pow', 'sqrt', 'std', 'var', 'int', 'num',
+  'pow', 'sqrt', 'std', 'var', 'int', 'num', 'idx',
   // Time
-  'now', 'utc', 'tz', 'ms', 'sec',
+  'now', 'utc', 'tz', 'ms', 'sec', 'hr', 'min', 'yr', 'mo',
   // Common patterns
   'app', 'cfg', 'config', 'init', 'len', 'val', 'str', 'obj', 'arr',
   'gen', 'def', 'raw', 'new', 'old', 'pre', 'post', 'sub', 'pub',
+  // Programming/Framework specific
+  'ts', 'js', 'jsx', 'tsx', 'py', 'rb', 'vue', 're', 'fn', 'fns', 'mod', 'opts', 'dev',
+  // Cloud/Infrastructure
+  's3', 'ec2', 'sqs', 'sns', 'vpc', 'ami', 'iam', 'acl', 'elb', 'alb', 'nlb', 'aws',
+  // Metrics/Performance
+  'fcp', 'lcp', 'cls', 'ttfb', 'tti', 'fid', 'fps', 'qps', 'rps', 'tps',
+  // Testing & i18n
+  'po', 'e2e', 'a11y', 'i18n', 'l10n',
+  // Domain-specific abbreviations (context-aware)
+  'sk', 'fy', 'faq', 'og', 'seo', 'cta', 'roi', 'kpi',
   // Boolean helpers (these are intentional short names)
-  'is', 'has', 'can', 'did', 'was', 'are'
+  'is', 'has', 'can', 'did', 'was', 'are',
+  // Date/Time context (when in date contexts)
+  'd', 't', 'dt'
 ]);
 
 /**
@@ -50,6 +84,9 @@ export async function analyzeNaming(files: string[]): Promise<NamingIssue[]> {
 function analyzeFileNaming(file: string, content: string): NamingIssue[] {
   const issues: NamingIssue[] = [];
 
+  // Check if this is a test file (more lenient rules)
+  const isTestFile = file.match(/\.(test|spec)\.(ts|tsx|js|jsx)$/);
+
   // Split into lines for line number tracking
   const lines = content.split('\n');
 
@@ -61,11 +98,31 @@ function analyzeFileNaming(file: string, content: string): NamingIssue[] {
     const singleLetterMatches = line.matchAll(/\b(?:const|let|var)\s+([a-hm-z])\s*=/gi);
     for (const match of singleLetterMatches) {
       const letter = match[1].toLowerCase();
-      // Skip if it's in a loop context or common iterator
-      const isInLoopContext = line.includes('for') || line.includes('.map') || 
-                              line.includes('.filter') || line.includes('.forEach') ||
-                              line.includes('.reduce');
-      if (!isInLoopContext && !['x', 'y', 'z', 'i', 'j', 'k', 'l', 'n', 'm'].includes(letter)) {
+      
+      // Enhanced loop/iterator context detection
+      const isInLoopContext = 
+        line.includes('for') || 
+        /\.(map|filter|forEach|reduce|find|some|every)\s*\(/.test(line) ||
+        line.includes('=>') || // Arrow function
+        /\w+\s*=>\s*/.test(line); // Callback pattern
+      
+      // Check for i18n/translation context
+      const isI18nContext = 
+        line.includes('useTranslation') ||
+        line.includes('i18n.t') ||
+        /\bt\s*\(['"]/.test(line); // t('key') pattern
+      
+      // Check for arrow function parameter (improved detection)
+      const isArrowFunctionParam = 
+        /\(\s*[a-z]\s*(?:,\s*[a-z]\s*)*\)\s*=>/.test(line) || // (s) => or (a, b) =>
+        /[a-z]\s*=>/.test(line); // s =>
+      
+      if (!isInLoopContext && !isI18nContext && !isArrowFunctionParam && !['x', 'y', 'z', 'i', 'j', 'k', 'l', 'n', 'm'].includes(letter)) {
+        // Skip in test files unless it's really unclear
+        if (isTestFile && ['a', 'b', 'c', 'd', 'e', 'f', 's'].includes(letter)) {
+          continue;
+        }
+        
         issues.push({
           file,
           line: lineNumber,
@@ -81,17 +138,48 @@ function analyzeFileNaming(file: string, content: string): NamingIssue[] {
     const abbreviationMatches = line.matchAll(/\b(?:const|let|var)\s+([a-z]{1,3})(?=[A-Z]|_|\s*=)/g);
     for (const match of abbreviationMatches) {
       const abbrev = match[1].toLowerCase();
+      
+      // Skip if it's a common short English word (full word, not abbreviation)
+      if (COMMON_SHORT_WORDS.has(abbrev)) {
+        continue;
+      }
+      
       // Skip acceptable abbreviations
-      if (!ACCEPTABLE_ABBREVIATIONS.has(abbrev)) {
-        issues.push({
-          file,
-          line: lineNumber,
-          type: 'abbreviation',
-          identifier: match[1],
-          severity: 'info',
-          suggestion: `Consider using full word instead of abbreviation '${match[1]}'`
-        });
+      if (ACCEPTABLE_ABBREVIATIONS.has(abbrev)) {
+        continue;
       }
+      
+      // Check for arrow function parameter context
+      const isArrowFunctionParam = 
+        /\(\s*[a-z]\s*(?:,\s*[a-z]\s*)*\)\s*=>/.test(line) || // (s) => or (a, b) =>
+        new RegExp(`\\b${abbrev}\\s*=>`).test(line); // s =>
+      
+      if (isArrowFunctionParam) {
+        continue;
+      }
+      
+      // For very short names (1-2 letters), check for date/time context
+      if (abbrev.length <= 2) {
+        const isDateTimeContext = /date|time|day|hour|minute|second|timestamp/i.test(line);
+        if (isDateTimeContext && ['d', 't', 'dt'].includes(abbrev)) {
+          continue;
+        }
+        
+        // Check for user/auth context
+        const isUserContext = /user|auth|account/i.test(line);
+        if (isUserContext && abbrev === 'u') {
+          continue;
+        }
+      }
+      
+      issues.push({
+        file,
+        line: lineNumber,
+        type: 'abbreviation',
+        identifier: match[1],
+        severity: 'info',
+        suggestion: `Consider using full word instead of abbreviation '${match[1]}'`
+      });
     }
 
     // Check for snake_case vs camelCase mixing in TypeScript/JavaScript
@@ -131,16 +219,42 @@ function analyzeFileNaming(file: string, content: string): NamingIssue[] {
     const functionMatches = line.matchAll(/function\s+([a-z][a-zA-Z0-9]*)/g);
     for (const match of functionMatches) {
       const name = match[1];
+      
+      // Skip JavaScript/TypeScript keywords that shouldn't be function names
+      const isKeyword = ['for', 'if', 'else', 'while', 'do', 'switch', 'case', 'break', 'continue', 'return', 'throw', 'try', 'catch', 'finally', 'with', 'yield', 'await'].includes(name);
+      if (isKeyword) {
+        continue;
+      }
+      
+      // Skip common entry point names
+      const isEntryPoint = ['main', 'init', 'setup', 'bootstrap'].includes(name);
+      if (isEntryPoint) {
+        continue;
+      }
+      
       // Functions should typically start with verbs, but allow:
       // 1. Factory/builder patterns (ends with Factory, Builder, etc.)
       // 2. Descriptive compound names that explain what they return
       // 3. Event handlers (onClick, onSubmit, etc.)
+      // 4. Descriptive aggregate/collection patterns
+      // 5. Very long descriptive names (>15 chars)
+      // 6. Compound words with 3+ capitals
+      
       const isFactoryPattern = name.match(/(Factory|Builder|Creator|Generator)$/);
       const isEventHandler = name.match(/^on[A-Z]/);
-      const isDescriptiveLong = name.length > 20; // Long names are usually descriptive enough
+      const isDescriptiveLong = name.length > 15; // Reduced from 20 to 15
+      
+      // Check for descriptive patterns
+      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)$/);
+      
+      // Count capital letters for compound detection
+      const capitalCount = (name.match(/[A-Z]/g) || []).length;
+      const isCompoundWord = capitalCount >= 3; // daysSinceLastCommit has 4 capitals
+      
       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)/);
       
-      if (!hasActionVerb && !isFactoryPattern && !isEventHandler && !isDescriptiveLong) {
+      if (!hasActionVerb && !isFactoryPattern && !isEventHandler && !isDescriptiveLong && !isDescriptivePattern && !isCompoundWord) {
         issues.push({
           file,
           line: lineNumber,

package/.turbo/turbo-build.log

@@ -1,24 +0,0 @@
-
-
-> @aiready/consistency@0.3.3 build /Users/pengcao/projects/aiready/packages/consistency
-> tsup src/index.ts src/cli.ts --format cjs,esm --dts
-
-CLI Building entry: src/cli.ts, src/index.ts
-CLI Using tsconfig: tsconfig.json
-CLI tsup v8.5.1
-CLI Target: es2020
-CJS Build start
-ESM Build start
-CJS dist/cli.js   25.23 KB
-CJS dist/index.js 16.34 KB
-CJS ⚡️ Build success in 79ms
-ESM dist/chunk-LUAREV6A.mjs 15.11 KB
-ESM dist/cli.mjs            8.54 KB
-ESM dist/index.mjs          220.00 B
-ESM ⚡️ Build success in 79ms
-DTS Build start
-DTS ⚡️ Build success in 587ms
-DTS dist/cli.d.ts    20.00 B
-DTS dist/index.d.ts  2.60 KB
-DTS dist/cli.d.mts   20.00 B
-DTS dist/index.d.mts 2.60 KB

package/.turbo/turbo-test.log

@@ -1,76 +0,0 @@
-
-
-> @aiready/consistency@0.3.3 test /Users/pengcao/projects/aiready/packages/consistency
-> vitest run
-
-
- RUN  v2.1.9 /Users/pengcao/projects/aiready/packages/consistency
-
-[?25l   ❯ analyzeConsistency (2)
-     ⠙ should analyze naming issues
-     · should detect minimum severity filtering
-   · analyzeNaming (5)
-     · should detect single letter variables
-     · should NOT flag acceptable abbreviations
-     · should detect snake_case in TypeScript files
-     · should detect unclear boolean names
-     · should allow common abbreviations
-   · analyzePatterns (3)
-     · should detect mixed error handling
-     · should detect mixed async patterns
-     · should detect mixed import styles
-   · consistency scoring (2)
-     · should calculate consistency score correctly
-     · should weight critical issues more than info
-   · recommendations (3)
-     · should generate relevant recommendations
-     · should suggest standardizing error handling
-     · should suggest using async/await consistently
-[?25l   ✓ analyzeConsistency (2)
-     ✓ should analyze naming issues
-     ✓ should detect minimum severity filtering
-   ✓ analyzeNaming (5)
-     ✓ should detect single letter variables
-     ✓ should NOT flag acceptable abbreviations
-     ✓ should detect snake_case in TypeScript files
-     ✓ should detect unclear boolean names
-     ✓ should allow common abbreviations
-   ✓ analyzePatterns (3)
-     ✓ should detect mixed error handling
-     ✓ should detect mixed async patterns
-     ✓ should detect mixed import styles
-   ✓ consistency scoring (2)
-     ✓ should calculate consistency score correctly
-     ✓ should weight critical issues more than info
-   ✓ recommendations (3)
-     ✓ should generate relevant recommendations
-     ✓ should suggest standardizing error handling
-     ✓ should suggest using async/await consistently
- ✓ src/__tests__/analyzer.test.ts (15)
-   ✓ analyzeConsistency (2)
-     ✓ should analyze naming issues
-     ✓ should detect minimum severity filtering
-   ✓ analyzeNaming (5)
-     ✓ should detect single letter variables
-     ✓ should NOT flag acceptable abbreviations
-     ✓ should detect snake_case in TypeScript files
-     ✓ should detect unclear boolean names
-     ✓ should allow common abbreviations
-   ✓ analyzePatterns (3)
-     ✓ should detect mixed error handling
-     ✓ should detect mixed async patterns
-     ✓ should detect mixed import styles
-   ✓ consistency scoring (2)
-     ✓ should calculate consistency score correctly
-     ✓ should weight critical issues more than info
-   ✓ recommendations (3)
-     ✓ should generate relevant recommendations
-     ✓ should suggest standardizing error handling
-     ✓ should suggest using async/await consistently
-
- Test Files  1 passed (1)
-      Tests  15 passed (15)
-   Start at  15:32:25
-   Duration  519ms (transform 74ms, setup 0ms, collect 251ms, tests 24ms, environment 0ms, prepare 54ms)
-
-[?25h[?25h