@aiready/consistency 0.8.31 → 0.8.32

package/src/analyzers/naming-constants.ts

@@ -6,75 +6,387 @@
 // Common short English words that are NOT abbreviations (full, valid words)
 export 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',
+  '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',
   // Prepositions and conjunctions
-  'and', 'from', 'how', 'pad', 'bar', 'non',
+  'and',
+  'from',
+  'how',
+  'pad',
+  'bar',
+  'non',
   // 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'
+  '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
 export const ACCEPTABLE_ABBREVIATIONS = new Set([
   // Standard identifiers
-  'id', 'uid', 'gid', 'pid',
+  'id',
+  'uid',
+  'gid',
+  'pid',
   // Loop counters and iterators
-  'i', 'j', 'k', 'n', 'm',
+  'i',
+  'j',
+  'k',
+  'n',
+  'm',
   // Web/Network
-  'url', 'uri', 'api', 'cdn', 'dns', 'ip', 'tcp', 'udp', 'http', 'ssl', 'tls',
-  'utm', 'seo', 'rss', 'xhr', 'ajax', 'cors', 'ws', 'wss',
+  'url',
+  'uri',
+  'api',
+  'cdn',
+  'dns',
+  'ip',
+  'tcp',
+  'udp',
+  'http',
+  'ssl',
+  'tls',
+  'utm',
+  'seo',
+  'rss',
+  'xhr',
+  'ajax',
+  'cors',
+  'ws',
+  'wss',
   // Data formats
-  'json', 'xml', 'yaml', 'csv', 'html', 'css', 'svg', 'pdf',
+  'json',
+  'xml',
+  'yaml',
+  'csv',
+  'html',
+  'css',
+  'svg',
+  'pdf',
   // File types & extensions
-  'img', 'txt', 'doc', 'docx', 'xlsx', 'ppt', 'md', 'rst', 'jpg', 'png', 'gif',
+  'img',
+  'txt',
+  'doc',
+  'docx',
+  'xlsx',
+  'ppt',
+  'md',
+  'rst',
+  'jpg',
+  'png',
+  'gif',
   // Databases
-  'db', 'sql', 'orm', 'dao', 'dto', 'ddb', 'rds', 'nosql',
+  'db',
+  'sql',
+  'orm',
+  'dao',
+  'dto',
+  'ddb',
+  'rds',
+  'nosql',
   // File system
-  'fs', 'dir', 'tmp', 'src', 'dst', 'bin', 'lib', 'pkg',
+  'fs',
+  'dir',
+  'tmp',
+  'src',
+  'dst',
+  'bin',
+  'lib',
+  'pkg',
   // Operating system
-  'os', 'env', 'arg', 'cli', 'cmd', 'exe', 'cwd', 'pwd',
+  'os',
+  'env',
+  'arg',
+  'cli',
+  'cmd',
+  'exe',
+  'cwd',
+  'pwd',
   // UI/UX
-  'ui', 'ux', 'gui', 'dom', 'ref',
+  'ui',
+  'ux',
+  'gui',
+  'dom',
+  'ref',
   // Request/Response
-  'req', 'res', 'ctx', 'err', 'msg', 'auth',
+  'req',
+  'res',
+  'ctx',
+  'err',
+  'msg',
+  'auth',
   // Mathematics/Computing
-  'max', 'min', 'avg', 'sum', 'abs', 'cos', 'sin', 'tan', 'log', 'exp',
-  'pow', 'sqrt', 'std', 'var', 'int', 'num', 'idx',
+  'max',
+  'min',
+  'avg',
+  'sum',
+  'abs',
+  'cos',
+  'sin',
+  'tan',
+  'log',
+  'exp',
+  'pow',
+  'sqrt',
+  'std',
+  'var',
+  'int',
+  'num',
+  'idx',
   // Time
-  'now', 'utc', 'tz', 'ms', 'sec', 'hr', 'min', 'yr', 'mo',
+  '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',
+  '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',
+  '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',
-  'ses', 'gst', 'cdk', 'btn', 'buf', 'agg', 'ocr', 'ai', 'cf', 'cfn', 'ga',
+  's3',
+  'ec2',
+  'sqs',
+  'sns',
+  'vpc',
+  'ami',
+  'iam',
+  'acl',
+  'elb',
+  'alb',
+  'nlb',
+  'aws',
+  'ses',
+  'gst',
+  'cdk',
+  'btn',
+  'buf',
+  'agg',
+  'ocr',
+  'ai',
+  'cf',
+  'cfn',
+  'ga',
   // Metrics/Performance
-  'fcp', 'lcp', 'cls', 'ttfb', 'tti', 'fid', 'fps', 'qps', 'rps', 'tps', 'wpm',
+  'fcp',
+  'lcp',
+  'cls',
+  'ttfb',
+  'tti',
+  'fid',
+  'fps',
+  'qps',
+  'rps',
+  'tps',
+  'wpm',
   // Testing & i18n
-  'po', 'e2e', 'a11y', 'i18n', 'l10n', 'spy',
+  'po',
+  'e2e',
+  'a11y',
+  'i18n',
+  'l10n',
+  'spy',
   // Domain-specific abbreviations (context-aware)
-  'sk', 'fy', 'faq', 'og', 'seo', 'cta', 'roi', 'kpi', 'ttl', 'pct',
+  'sk',
+  'fy',
+  'faq',
+  'og',
+  'seo',
+  'cta',
+  'roi',
+  'kpi',
+  'ttl',
+  'pct',
   // Technical abbreviations
-  'mac', 'hex', 'esm', 'git', 'rec', 'loc', 'dup',
+  'mac',
+  'hex',
+  'esm',
+  'git',
+  'rec',
+  'loc',
+  'dup',
   // 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',
+  'd',
+  't',
+  'dt',
   // Coverage metrics (industry standard: statements/branches/functions/lines)
-  's', 'b', 'f', 'l',
+  's',
+  'b',
+  'f',
+  'l',
   // Common media/content abbreviations
-  'vid', 'pic', 'img', 'doc', 'msg'
+  'vid',
+  'pic',
+  'img',
+  'doc',
+  'msg',
 ]);
 
 /**
@@ -88,14 +400,16 @@ export function snakeCaseToCamelCase(str: string): string {
  * Detect naming convention patterns across the codebase
  */
 export function detectNamingConventions(
-  files: string[], 
+  files: string[],
   allIssues: Array<{ type: string; [key: string]: any }>
 ): {
   dominantConvention: 'camelCase' | 'snake_case' | 'PascalCase' | 'mixed';
   conventionScore: number;
 } {
   // Count conventions
-  const camelCaseCount = allIssues.filter(i => i.type === 'convention-mix').length;
+  const camelCaseCount = allIssues.filter(
+    (i) => i.type === 'convention-mix'
+  ).length;
   const totalChecks = files.length * 10; // Rough estimate
 
   if (camelCaseCount / totalChecks > 0.3) {

package/src/analyzers/naming-python.ts

@@ -1,17 +1,19 @@
 /**
  * Python Naming Analyzer - PEP 8 Compliant
- * 
+ *
  * Analyzes Python code for PEP 8 naming convention violations
  * https://peps.python.org/pep-0008/#naming-conventions
  */
 
-import { getParser, Language } from '@aiready/core';
+import { getParser } from '@aiready/core';
 import type { NamingIssue } from '../types';
 
 /**
  * Analyze Python files for PEP 8 naming violations
  */
-export async function analyzePythonNaming(files: string[]): Promise<NamingIssue[]> {
+export async function analyzePythonNaming(
+  files: string[]
+): Promise<NamingIssue[]> {
   const issues: NamingIssue[] = [];
   const parser = getParser('dummy.py'); // Get Python parser instance
 
@@ -21,7 +23,7 @@ export async function analyzePythonNaming(files: string[]): Promise<NamingIssue[
   }
 
   // Filter to only Python files
-  const pythonFiles = files.filter(f => f.toLowerCase().endsWith('.py'));
+  const pythonFiles = files.filter((f) => f.toLowerCase().endsWith('.py'));
 
   for (const file of pythonFiles) {
     try {
@@ -31,7 +33,12 @@ export async function analyzePythonNaming(files: string[]): Promise<NamingIssue[
 
       // Analyze each export for naming violations
       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);
         }
@@ -41,7 +48,12 @@ export async function analyzePythonNaming(files: string[]): Promise<NamingIssue[
       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);
             }
@@ -127,7 +139,10 @@ function checkPythonNaming(
       // Regular variables should be snake_case
       if (!conventions.variablePattern.test(identifier)) {
         // Check if it's using camelCase (common mistake from JS/TS developers)
-        if (/^[a-z][a-zA-Z0-9]*$/.test(identifier) && /[A-Z]/.test(identifier)) {
+        if (
+          /^[a-z][a-zA-Z0-9]*$/.test(identifier) &&
+          /[A-Z]/.test(identifier)
+        ) {
           return {
             type: 'convention-mix',
             identifier,
@@ -162,16 +177,19 @@ function toSnakeCase(str: string): string {
 function toPascalCase(str: string): string {
   return str
     .split('_')
-    .map(word => word.charAt(0).toUpperCase() + word.slice(1).toLowerCase())
+    .map((word) => word.charAt(0).toUpperCase() + word.slice(1).toLowerCase())
     .join('');
 }
 
 /**
  * Detect common Python anti-patterns in naming
  */
-export function detectPythonNamingAntiPatterns(files: string[]): NamingIssue[] {
+export function detectPythonNamingAntiPatterns(_files: string[]): NamingIssue[] {
   const issues: NamingIssue[] = [];
 
+  // Parameter currently unused; reference to avoid lint warnings
+  void _files;
+
   // Anti-pattern 1: Using camelCase in Python (common for JS/TS developers)
   // Anti-pattern 2: Using PascalCase for functions
   // Anti-pattern 3: Not using leading underscore for private methods