@sun-asterisk/sunlint 1.3.30 → 1.3.32

package/rules/common/C006_function_naming/analyzer.js

@@ -45,11 +45,92 @@ class SmartC006Analyzer {
       // Side effect patterns indicate actions
       ACTION: [/console\./, /fetch\(/, /\.send\(/, /\.post\(/]
     };
+
+    // 📚 COMPREHENSIVE VERB LIST (from C006 rule specification)
+    this.acceptedVerbs = [
+      // Getters/Queries
+      'get', 'fetch', 'retrieve', 'find', 'search', 'query', 'load',
+      // Setters/Modifiers
+      'set', 'update', 'modify', 'change', 'edit', 'alter', 'transform',
+      // Creation
+      'create', 'build', 'make', 'generate', 'construct', 'produce',
+      // Deletion
+      'delete', 'remove', 'destroy', 'clean', 'clear', 'reset',
+      // Validation
+      'validate', 'verify', 'check', 'confirm', 'ensure', 'test', 'compare',
+      // Computation
+      'calculate', 'compute', 'parse', 'format', 'convert',
+      // Communication
+      'send', 'receive', 'transmit', 'broadcast', 'emit', 'publish',
+      // Collections
+      'map', 'filter', 'sort', 'group', 'merge', 'split', 'add', 'append', 'insert',
+      'push', 'pop', 'shift', 'splice', 'unshift', 'slice', 'concat', 'join',
+      // State checks (Boolean)
+      'is', 'has', 'can', 'should', 'will', 'does', 'contains', 'includes', 'exists',
+      // UI actions
+      'show', 'hide', 'display', 'render', 'draw', 'toggle', 'enable', 'disable',
+      'activate', 'deactivate', 'select', 'deselect', 'focus', 'blur',
+      // Lifecycle
+      'connect', 'disconnect', 'open', 'close', 'start', 'stop', 'run', 'refresh',
+      'initialize', 'init', 'setup', 'teardown', 'shutdown', 'restart', 'reload',
+      'restore', 'resume', 'pause', 'suspend',
+      // Event Handling
+      'on', 'trigger', 'fire', 'dispatch', 'invoke', 'call', 'emit', 'handle',
+      // Monitoring
+      'count', 'measure', 'monitor', 'watch', 'track', 'observe', 'log', 'record',
+      // Navigation
+      'navigate', 'redirect', 'route', 'go', 'move', 'scroll',
+      // Data Operations
+      'save', 'store', 'persist', 'cache', 'serialize', 'deserialize',
+      // Detection/Analysis
+      'detect', 'analyze', 'scan', 'inspect', 'evaluate', 'assess',
+      // Computation & Transformation
+      'calculate', 'compute', 'parse', 'format', 'convert',
+      'normalize', 'sanitize', 'encode', 'decode', 'encrypt', 'decrypt',
+      'compress', 'decompress', 'zip', 'unzip', 'tar', 'untar', // ← Add these
+      'stringify', 'objectify',
+      // State Management
+      'apply', 'revert', 'undo', 'redo', 'commit', 'rollback',
+      // Async Operations
+      'await', 'defer', 'debounce', 'throttle', 'delay',
+      // Error Handling
+      'throw', 'catch', 'handle', 'recover', 'retry',
+      // Copying/Cloning
+      'copy', 'clone', 'duplicate', 'replicate',
+      // Comparison
+      'equals', 'match', 'differ', 'compare',
+      // Registration
+      'register', 'unregister', 'subscribe', 'unsubscribe',
+      // Import/Export
+      'import', 'export', 'download', 'upload',
+      // Expansion/Collapse
+      'expand', 'collapse', 'maximize', 'minimize',
+      // Submission
+      'submit', 'cancel', 'abort', 'complete', 'finish',
+      // Preparation
+      'prepare', 'ready', 'preload', 'precache',
+      // Extraction
+      'extract', 'derive', 'obtain', 'acquire',
+      // Replacement
+      'replace', 'swap', 'substitute', 'override',
+      // Binding
+      'bind', 'unbind', 'attach', 'detach',
+      // Notification
+      'notify', 'alert', 'warn', 'inform',
+      // Execution
+      'execute', 'perform', 'process', 'run'
+    ];
+
+    // ⚠️ VAGUE/GENERIC VERBS TO FLAG (should have more specific verbs)
+    this.vagueVerbs = [
+      'do', 'handle', 'manage', 'process', 'execute', 'perform',
+      'something', 'stuff', 'thing', 'work', 'data', 'item'
+    ];
   }
 
   async analyze(files, language, config) {
     const violations = [];
-    
+
     if (config.verbose) {
       console.log(`🔧 [DEBUG] Starting Smart C006 Analysis on ${files.length} files...`);
     }
@@ -69,7 +150,7 @@ class SmartC006Analyzer {
     if (config.verbose) {
       console.log(`🔧 [DEBUG] Smart C006 Analysis complete: ${violations.length} violations found`);
     }
-    
+
     return violations;
   }
 
@@ -80,10 +161,10 @@ class SmartC006Analyzer {
 
     const violations = [];
     const lines = content.split('\n');
-    
+
     // 🏗️ TIER 1: ARCHITECTURAL CONTEXT ANALYSIS
     const architecturalContext = this.analyzeArchitecturalContext(filePath, content);
-    
+
     // Parse TypeScript/JavaScript code
     const sourceFile = ts.createSourceFile(
       filePath,
@@ -102,13 +183,13 @@ class SmartC006Analyzer {
           architecturalContext,
           content
         );
-        
+
         if (analysis.isViolation && analysis.confidence >= this.confidenceThresholds.LOW) {
           const namePosition = sourceFile.getLineAndCharacterOfPosition(node.name.getStart());
           const line = namePosition.line + 1;
           const column = namePosition.character + 1;
           const lineText = lines[line - 1]?.trim() || '';
-          
+
           violations.push({
             ruleId: this.ruleId,
             file: filePath,
@@ -134,13 +215,13 @@ class SmartC006Analyzer {
           architecturalContext,
           content
         );
-        
+
         if (analysis.isViolation && analysis.confidence >= this.confidenceThresholds.LOW) {
           const namePosition = sourceFile.getLineAndCharacterOfPosition(node.name.getStart());
           const line = namePosition.line + 1;
           const column = namePosition.character + 1;
           const lineText = lines[line - 1]?.trim() || '';
-          
+
           violations.push({
             ruleId: this.ruleId,
             file: filePath,
@@ -158,7 +239,7 @@ class SmartC006Analyzer {
       }
 
       // Analyze arrow functions assigned to variables
-      if (ts.isVariableDeclaration(node) && node.name && ts.isIdentifier(node.name) && 
+      if (ts.isVariableDeclaration(node) && node.name && ts.isIdentifier(node.name) &&
           node.initializer && ts.isArrowFunction(node.initializer)) {
         const analysis = this.smartAnalyzeFunctionName(
           node.name.text,
@@ -167,13 +248,13 @@ class SmartC006Analyzer {
           architecturalContext,
           content
         );
-        
+
         if (analysis.isViolation && analysis.confidence >= this.confidenceThresholds.LOW) {
           const namePosition = sourceFile.getLineAndCharacterOfPosition(node.name.getStart());
           const line = namePosition.line + 1;
           const column = namePosition.character + 1;
           const lineText = lines[line - 1]?.trim() || '';
-          
+
           violations.push({
             ruleId: this.ruleId,
             file: filePath,
@@ -189,7 +270,7 @@ class SmartC006Analyzer {
           });
         }
       }
-      
+
       ts.forEachChild(node, visit);
     };
 
@@ -204,7 +285,7 @@ class SmartC006Analyzer {
   analyzeArchitecturalContext(filePath, content) {
     const fileName = path.basename(filePath, path.extname(filePath)).toLowerCase();
     const fileDir = path.dirname(filePath).toLowerCase();
-    
+
     // Detect architectural layer
     let layer = 'UNKNOWN';
     for (const [layerName, patterns] of Object.entries(this.architecturalLayers)) {
@@ -213,12 +294,12 @@ class SmartC006Analyzer {
         break;
       }
     }
-    
+
     // Analyze imports for additional context
     const imports = this.extractImports(content);
     const isReactComponent = imports.some(imp => imp.includes('react')) || content.includes('JSX.Element');
     const isTestFile = fileName.includes('test') || fileName.includes('spec');
-    
+
     return {
       layer,
       isReactComponent,
@@ -234,83 +315,96 @@ class SmartC006Analyzer {
    */
   analyzeSemanticIntent(functionNode, sourceFile, content) {
     if (!functionNode.body) return 'UNKNOWN';
-    
+
     const functionText = content.substring(
       functionNode.body.getStart(),
       functionNode.body.getEnd()
     );
-    
+
     // Check for different semantic patterns
     for (const [intent, patterns] of Object.entries(this.semanticPatterns)) {
       if (patterns.some(pattern => pattern.test(functionText))) {
         return intent;
       }
     }
-    
+
     return 'UNKNOWN';
   }
 
   /**
-   * 🎯 TIER 3: INTELLIGENT VERB DETECTION
-   * Uses multiple strategies to detect verbs
+   * 🎯 ENHANCED VERB DETECTION - FIXED VERSION
+   * Properly detects verbs including compound verbs
    */
-  isVerbLikeName(functionName) {
-    // Strategy 0: REJECT generic/vague verbs that should be flagged
-    const genericVerbs = [
-      'do', 'handle', 'process', 'manage', 'execute',
-      'something', 'stuff', 'thing', 'work', 'data'
-    ];
-    
-    const isGenericVerb = genericVerbs.some(verb => {
-      const verbPattern = new RegExp(`^${verb}([A-Z].*|$)`, 'i');
-      return verbPattern.test(functionName);
-    });
-    
-    // Reject names starting with generic verbs
-    if (isGenericVerb) return false;
-    
-    // Strategy 1: Known verb prefixes (expanded beyond static list)
-    const verbPrefixes = [
-      'get', 'set', 'is', 'has', 'can', 'should', 'will', 'does',
-      'create', 'build', 'make', 'generate', 'construct', 'produce',
-      'update', 'modify', 'change', 'edit', 'alter', 'transform',
-      'delete', 'remove', 'destroy', 'clean', 'clear', 'reset',
-      'load', 'save', 'fetch', 'retrieve', 'find', 'search', 'query',
-      'validate', 'verify', 'check', 'confirm', 'ensure', 'test',
-      'calculate', 'compute', 'compare', 'parse', 'format', 'convert',
-      'send', 'receive', 'transmit', 'broadcast', 'emit', 'publish',
-      'map', 'filter', 'sort', 'group', 'merge', 'split',
-      'connect', 'disconnect', 'open', 'close', 'start', 'stop', 'run',
-      'show', 'hide', 'display', 'render', 'draw', 'paint', 'animate',
-      'add', 'append', 'insert', 'push', 'pop', 'shift', 'splice',
-      'count', 'measure', 'monitor', 'watch', 'track', 'observe',
-      'refresh', 'restore', 'reload', 'retry', 'resume', 'redirect',
-      'select', 'toggle', 'switch', 'enable', 'disable', 'activate',
-      'expand', 'collapse', 'scroll', 'navigate', 'submit', 'cancel',
-      'on', 'trigger', 'fire', 'dispatch', 'invoke', 'call'
-    ];
-    
-    // Strategy 2: Check if starts with known verb
-    const startsWithVerb = verbPrefixes.some(verb => {
-      const verbPattern = new RegExp(`^${verb}[A-Z]?`, 'i');
-      return verbPattern.test(functionName);
-    });
-    
-    if (startsWithVerb) return true;
-    
-    // Strategy 3: Common verb suffixes that indicate actions
-    const actionSuffixes = ['ize', 'ise', 'fy', 'ate', 'en'];
-    if (actionSuffixes.some(suffix => functionName.endsWith(suffix))) {
-      return true;
+  extractVerbFromName(functionName) {
+    // Convert to lowercase for comparison
+    const lowerName = functionName.toLowerCase();
+
+    // Find the longest matching verb from the accepted verbs list
+    let matchedVerb = null;
+    let maxLength = 0;
+
+    for (const verb of this.acceptedVerbs) {
+      // Check if function name starts with this verb
+      // Must be followed by uppercase letter, end of string, or be exact match
+      const verbPattern = new RegExp(`^${verb}([A-Z].*|$)`);
+
+      if (verbPattern.test(functionName) && verb.length > maxLength) {
+        matchedVerb = verb;
+        maxLength = verb.length;
+      }
     }
-    
-    // Strategy 4: English verb patterns (basic NLP)
-    const verbPatterns = [
-      /^(re|un|pre|de|dis)[A-Z]/, // prefixed verbs: revalidate, unload, preprocess
-      /^[a-z]+ly[A-Z]/, // adverb-verb patterns: quicklyProcess
-    ];
-    
-    return verbPatterns.some(pattern => pattern.test(functionName));
+
+    return matchedVerb;
+  }
+
+  /**
+   * 🎯 CHECK IF NAME FOLLOWS VERB-NOUN PATTERN
+   */
+  isVerbNounPattern(functionName) {
+    // Strategy 1: Extract verb from accepted verbs list
+    const verb = this.extractVerbFromName(functionName);
+
+    if (verb) {
+      // Check if it's a vague verb (should be flagged even if technically a verb)
+      const isVagueVerb = this.vagueVerbs.some(vague => {
+        const vaguePattern = new RegExp(`^${vague}([A-Z].*|$)`, 'i');
+        return vaguePattern.test(functionName);
+      });
+
+      if (isVagueVerb) {
+        return { isValid: false, verb: null, reason: 'vague_verb' };
+      }
+
+      // Valid verb found
+      return { isValid: true, verb, reason: null };
+    }
+
+    // Strategy 2: Check for verb-like patterns (morphology)
+    const verbSuffixes = ['ize', 'ise', 'fy', 'ate', 'en', 'ing', 'ed'];
+    for (const suffix of verbSuffixes) {
+      const suffixPattern = new RegExp(`^[a-z]+${suffix}([A-Z].*|$)`, 'i');
+      if (suffixPattern.test(functionName)) {
+        return { isValid: true, verb: 'morphological', reason: null };
+      }
+    }
+
+    // Strategy 3: Check for verb prefixes (re-, un-, pre-, de-, dis-)
+    const verbPrefixes = ['re', 'un', 'pre', 'de', 'dis', 'over', 'under', 'out', 'up'];
+    for (const prefix of verbPrefixes) {
+      // Must be followed by a known verb or capital letter
+      const prefixPattern = new RegExp(`^${prefix}[A-Z]`, 'i');
+      if (prefixPattern.test(functionName)) {
+        // Check if the part after prefix is a known verb
+        const remainder = functionName.substring(prefix.length);
+        const remainderVerb = this.extractVerbFromName(remainder);
+        if (remainderVerb) {
+          return { isValid: true, verb: `${prefix}${remainderVerb}`, reason: null };
+        }
+      }
+    }
+
+    // No verb pattern found
+    return { isValid: false, verb: null, reason: 'no_verb' };
   }
 
   /**
@@ -323,29 +417,29 @@ class SmartC006Analyzer {
       requiredPatterns: [],
       suggestions: []
     };
-    
+
     // React components have different naming conventions
     if (architecturalContext.isReactComponent) {
       rules.allowedPatterns.push(/^[A-Z][a-zA-Z]*$/); // PascalCase components
       rules.allowedPatterns.push(/^use[A-Z][a-zA-Z]*$/); // React hooks
       rules.allowedPatterns.push(/^handle[A-Z][a-zA-Z]*$/); // Event handlers
     }
-    
+
     // Test files have different patterns
     if (architecturalContext.isTestFile) {
       rules.allowedPatterns.push(/^(test|it|describe|should|expect)[A-Z]?/);
     }
-    
+
     // Data layer functions often have CRUD patterns
     if (architecturalContext.layer === 'DATA') {
-      rules.suggestions.push('Consider CRUD verbs: create, read, update, delete');
+      rules.suggestions.push('Consider CRUD verbs: create, read, update, delete, fetch, save');
     }
-    
+
     // UI layer functions often have interaction verbs
     if (architecturalContext.layer === 'UI') {
-      rules.suggestions.push('Consider UI verbs: show, hide, toggle, render, display');
+      rules.suggestions.push('Consider UI verbs: show, hide, toggle, render, display, handle');
     }
-    
+
     return rules;
   }
 
@@ -358,88 +452,103 @@ class SmartC006Analyzer {
     if (this.isSpecialFunction(functionName, architecturalContext)) {
       return { isViolation: false };
     }
-    
+
     // Get semantic intent
     const semanticIntent = this.analyzeSemanticIntent(functionNode, sourceFile, content);
-    
+
     // Get context-specific rules
     const contextRules = this.getContextSpecificRules(architecturalContext, semanticIntent);
-    
+
     // Check if allowed by context-specific patterns
     if (contextRules.allowedPatterns.some(pattern => pattern.test(functionName))) {
       return { isViolation: false };
     }
-    
+
     // Check if name follows verb-noun pattern
-    const isVerbLike = this.isVerbLikeName(functionName);
-    
-    if (isVerbLike) {
+    const verbCheck = this.isVerbNounPattern(functionName);
+
+    if (verbCheck.isValid) {
       return { isViolation: false };
     }
-    
+
     // 🧮 CONFIDENCE CALCULATION
     let confidence = 0.5; // Base confidence
-    
-    // Boost confidence for clearly generic/vague patterns
-    const vagueFunctionNames = [
-      'doSomething', 'handleStuff', 'processData', 'processInfo',
-      'executeWork', 'manageItems', 'doWork', 'handleData',
-      'something', 'stuff', 'thing', 'data', 'info', 'item'
-    ];
-    
-    if (vagueFunctionNames.some(vague => functionName.toLowerCase().includes(vague.toLowerCase()))) {
-      confidence += 0.4; // Strongly boost confidence for obviously vague names
-    }
-    
-    // Boost confidence for clear noun-only patterns
-    if (/^[a-z]+$/.test(functionName)) {
-      confidence += 0.3; // Simple lowercase nouns: user, data
-    }
-    
-    if (/^[a-z]+[A-Z][a-z]+$/.test(functionName)) {
-      confidence += 0.2; // Simple camelCase nouns: userData, userInfo
+    let violationType = 'no_verb';
+
+    if (verbCheck.reason === 'vague_verb') {
+      violationType = 'vague_verb';
+      confidence = 0.9; // High confidence for vague verbs
+    } else if (verbCheck.reason === 'no_verb') {
+      violationType = 'no_verb';
+
+      // Check for obviously wrong patterns
+      const commonNounPatterns = [
+        /^(user|data|info|item|list|config|settings|options|params|args|props|state|value|result|response|request|error|message|event|callback|handler)([A-Z][a-z]*)*$/i,
+        /^[a-z]+$/, // Simple lowercase words
+        /^[a-z]+[A-Z][a-z]+$/ // Simple camelCase nouns
+      ];
+
+      if (commonNounPatterns.some(pattern => pattern.test(functionName))) {
+        confidence = 0.8; // High confidence for obvious nouns
+      }
+
+      // Boost confidence for clearly generic patterns
+      const vagueFunctionNames = [
+        'something', 'stuff', 'thing', 'data', 'info', 'item',
+        'work', 'task', 'job', 'action', 'operation'
+      ];
+
+      if (vagueFunctionNames.some(vague => functionName.toLowerCase().includes(vague.toLowerCase()))) {
+        confidence = 0.95; // Very high confidence
+      }
     }
-    
-    // Reduce confidence for complex names (might have hidden verbs)
-    if (functionName.length > 15) {
-      confidence -= 0.1;
+
+    // Reduce confidence for complex names (might have hidden patterns)
+    if (functionName.length > 20) {
+      confidence -= 0.15;
     }
-    
+
     // Reduce confidence for utils/helpers (more flexible naming)
     if (architecturalContext.layer === 'UTILS') {
       confidence -= 0.2;
     }
-    
+
     // Reduce confidence for test files
     if (architecturalContext.isTestFile) {
       confidence -= 0.3;
     }
-    
+
     // Cap confidence
     confidence = Math.min(Math.max(confidence, 0.1), 1.0);
-    
+
     // 💬 INTELLIGENT MESSAGING
     const context = {
       layer: architecturalContext.layer,
       intent: semanticIntent,
       isReactComponent: architecturalContext.isReactComponent,
-      isTestFile: architecturalContext.isTestFile
+      isTestFile: architecturalContext.isTestFile,
+      violationType
     };
-    
-    let reason = `Function '${functionName}' should follow verb-noun naming pattern`;
-    let suggestion = this.generateSmartSuggestion(functionName, semanticIntent, architecturalContext);
+
+    let reason = '';
+    let suggestion = '';
+
+    if (violationType === 'vague_verb') {
+      reason = `Function '${functionName}' uses vague verb. Use more specific action verbs (see C006 accepted verbs list)`;
+      suggestion = this.generateSmartSuggestion(functionName, semanticIntent, architecturalContext, true);
+    } else {
+      reason = `Function '${functionName}' must start with a verb (verb-noun pattern required by C006)`;
+      suggestion = this.generateSmartSuggestion(functionName, semanticIntent, architecturalContext, false);
+    }
 
     if (architecturalContext.layer !== 'UNKNOWN') {
       reason += ` (${architecturalContext.layer} layer)`;
     }
 
-    // Add helpful note about accepted verbs
-    reason += `. Accepted verbs: get, set, create, update, delete, validate, check, compare, etc. See: https://coding-standards.sun-asterisk.vn/rules/rule/C006/`;
-    
     return {
       isViolation: true,
       reason,
-      type: 'smart_naming_violation',
+      type: `smart_naming_violation_${violationType}`,
       confidence,
       suggestion,
       context
@@ -449,27 +558,51 @@ class SmartC006Analyzer {
   /**
    * 💡 SMART SUGGESTION GENERATOR
    */
-  generateSmartSuggestion(functionName, semanticIntent, architecturalContext) {
-    const baseNoun = functionName.charAt(0).toUpperCase() + functionName.slice(1);
-    
+  generateSmartSuggestion(functionName, semanticIntent, architecturalContext, isVagueVerb) {
+    // If it's a vague verb, extract the noun part
+    let baseNoun = functionName;
+
+    if (isVagueVerb) {
+      // Try to extract the noun part after the vague verb
+      for (const vagueVerb of this.vagueVerbs) {
+        const pattern = new RegExp(`^${vagueVerb}([A-Z].*)$`, 'i');
+        const match = functionName.match(pattern);
+        if (match) {
+          baseNoun = match[1];
+          break;
+        }
+      }
+    }
+
+    // Ensure first letter is capitalized for camelCase
+    baseNoun = baseNoun.charAt(0).toUpperCase() + baseNoun.slice(1);
+
+    const suggestions = [];
+
     switch (semanticIntent) {
       case 'GETTER':
-        return `get${baseNoun}()`;
+        suggestions.push(`get${baseNoun}()`, `fetch${baseNoun}()`, `retrieve${baseNoun}()`);
+        break;
       case 'SETTER':
-        return `set${baseNoun}()`;
+        suggestions.push(`set${baseNoun}()`, `update${baseNoun}()`, `save${baseNoun}()`);
+        break;
       case 'CHECKER':
-        return `is${baseNoun}() or has${baseNoun}()`;
+        suggestions.push(`is${baseNoun}()`, `has${baseNoun}()`, `can${baseNoun}()`, `should${baseNoun}()`);
+        break;
       case 'ACTION':
-        return `process${baseNoun}() or handle${baseNoun}()`;
+        suggestions.push(`process${baseNoun}()`, `handle${baseNoun}()`, `execute${baseNoun}()`);
+        break;
       default:
         if (architecturalContext.layer === 'DATA') {
-          return `fetch${baseNoun}() or create${baseNoun}()`;
+          suggestions.push(`fetch${baseNoun}()`, `create${baseNoun}()`, `save${baseNoun}()`, `load${baseNoun}()`);
+        } else if (architecturalContext.layer === 'UI') {
+          suggestions.push(`render${baseNoun}()`, `show${baseNoun}()`, `display${baseNoun}()`, `draw${baseNoun}()`);
+        } else {
+          suggestions.push(`get${baseNoun}()`, `set${baseNoun}()`, `process${baseNoun}()`, `handle${baseNoun}()`);
         }
-        if (architecturalContext.layer === 'UI') {
-          return `render${baseNoun}() or show${baseNoun}()`;
-        }
-        return `get${baseNoun}() or process${baseNoun}()`;
     }
+
+    return suggestions.slice(0, 3).join(' or ');
   }
 
   /**
@@ -482,29 +615,40 @@ class SmartC006Analyzer {
       'onCreate', 'onDestroy', 'onStart', 'onStop',
       'onPause', 'onResume', 'onSaveInstanceState',
       'equals', 'hashCode', 'compareTo', 'clone',
-      'finalize', 'notify', 'notifyAll', 'wait'
+      'finalize', 'notify', 'notifyAll', 'wait', 'bootstrap',
     ];
 
+    // ignore by patterns
+    const patternsToIgnore = [
+      /^(setup|config|bootstrap).*/,
+      /^_.*/,
+      /Factory$/
+    ];
+
+    if (patternsToIgnore.some(pattern => pattern.test(name))) {
+      return true;
+    }
+
     // Basic special function check
     if (specialFunctions.includes(name) || name.startsWith('_') || name.startsWith('$')) {
       return true;
     }
-    
+
     // React component names (PascalCase)
     if (architecturalContext.isReactComponent && /^[A-Z][a-zA-Z]*$/.test(name)) {
       return true;
     }
-    
+
     // React hooks
     if (name.startsWith('use') && /^use[A-Z]/.test(name)) {
       return true;
     }
-    
+
     // Test function names
     if (architecturalContext.isTestFile && /^(test|it|describe|should|expect)/.test(name)) {
       return true;
     }
-    
+
     return false;
   }
 
@@ -515,11 +659,11 @@ class SmartC006Analyzer {
     const importRegex = /import\s+.*?\s+from\s+['"]([^'"]+)['"]/g;
     const imports = [];
     let match;
-    
+
     while ((match = importRegex.exec(content)) !== null) {
       imports.push(match[1]);
     }
-    
+
     return imports;
   }