@sun-asterisk/sunlint 1.0.6 → 1.1.3

package/core/eslint-engine-service.js

@@ -1,312 +0,0 @@
-/**
- * ESLint Engine Service
- * Handles ESLint integration for TypeScript analysis
- * Following Rule C005: Single responsibility - only handle ESLint execution
- * Following Rule C014: Dependency injection for configuration
- */
-
-const { execSync } = require('child_process');
-const path = require('path');
-const fs = require('fs');
-const chalk = require('chalk');
-
-class ESLintEngineService {
-  constructor() {
-    this.eslintConfigPath = path.join(__dirname, '../config/typescript/eslint.config.js');
-    this.eslintPackagePath = path.join(__dirname, '../config/typescript');
-  }
-
-  /**
-   * Query: Check if ESLint is available
-   */
-  isAvailable() {
-    try {
-      // Check if config exists
-      if (!fs.existsSync(this.eslintConfigPath)) {
-        return false;
-      }
-
-      // Check if ESLint is available
-      execSync('npx eslint --version', { 
-        stdio: 'ignore',
-        cwd: this.eslintPackagePath
-      });
-      return true;
-    } catch (error) {
-      return false;
-    }
-  }
-
-  /**
-   * Command: Ensure ESLint dependencies are installed
-   */
-  async ensureDependencies() {
-    try {
-      console.log(chalk.blue('🔧 Ensuring ESLint dependencies...'));
-      
-      const packageJsonPath = path.join(this.eslintPackagePath, 'package.json');
-      if (fs.existsSync(packageJsonPath)) {
-        execSync('npm install', { 
-          cwd: this.eslintPackagePath,
-          stdio: 'inherit'
-        });
-      }
-      
-      console.log(chalk.green('✅ ESLint dependencies ready'));
-    } catch (error) {
-      throw new Error(`Failed to ensure ESLint dependencies: ${error.message}`);
-    }
-  }
-
-  /**
-   * Command: Run ESLint analysis
-   */
-  async runAnalysis(rulesToRun, options) {
-    try {
-      // Map SunLint rules to ESLint rules
-      const eslintRules = this.mapSunLintRulesToESLint(rulesToRun);
-      
-      // Build ESLint command
-      const command = this.buildESLintCommand(eslintRules, options);
-      
-      if (options.debug) {
-        console.log(chalk.yellow('ESLint command:'), command);
-      }
-
-      // Execute ESLint
-      const output = execSync(command, { 
-        cwd: this.eslintPackagePath,
-        encoding: 'utf-8'
-      });
-
-      // Parse results
-      return this.parseESLintOutput(output, options);
-
-    } catch (error) {
-      // ESLint might exit with non-zero code when violations are found
-      if (error.stdout) {
-        return this.parseESLintOutput(error.stdout, options);
-      }
-      throw new Error(`ESLint execution failed: ${error.message}`);
-    }
-  }
-
-  /**
-   * Query: Map SunLint rule IDs to ESLint rule IDs
-   */
-  mapSunLintRulesToESLint(rules) {
-    const ruleMapping = {
-      // Quality rules (C-rules)
-      'C002': 'custom/c002',
-      'C003': 'custom/c003',
-      'C006': 'custom/c006',
-      'C010': 'custom/c010',
-      'C013': 'custom/c013',
-      'C014': 'custom/c014',
-      'C017': 'custom/c017',
-      'C018': 'custom/c018',
-      'C023': 'custom/c023',
-      'C027': 'custom/c027',
-      'C029': 'custom/c029',
-      'C030': 'custom/c030',
-      'C034': 'custom/c034',
-      'C035': 'custom/c035',
-      'C041': 'custom/c041',
-      'C042': 'custom/c042',
-      'C043': 'custom/c043',
-      'C047': 'custom/c047',
-      'C048': 'custom/c048',
-      
-      // Security rules (S-rules)
-      'S005': 'custom/typescript_s005',
-      'S006': 'custom/typescript_s006',
-      'S008': 'custom/typescript_s008',
-      'S009': 'custom/typescript_s009',
-      'S010': 'custom/typescript_s010',
-      'S011': 'custom/typescript_s011',
-      'S012': 'custom/typescript_s012',
-      'S014': 'custom/typescript_s014',
-      'S015': 'custom/typescript_s015',
-      'S016': 'custom/typescript_s016',
-      'S017': 'custom/typescript_s017',
-      'S018': 'custom/typescript_s018',
-      'S019': 'custom/typescript_s019',
-      'S020': 'custom/typescript_s020',
-      'S022': 'custom/typescript_s022',
-      'S023': 'custom/typescript_s023',
-      'S025': 'custom/typescript_s025',
-      'S026': 'custom/typescript_s026',
-      'S027': 'custom/typescript_s027',
-      'S029': 'custom/typescript_s029',
-      'S030': 'custom/typescript_s030',
-      'S033': 'custom/typescript_s033',
-      'S034': 'custom/typescript_s034',
-      'S035': 'custom/typescript_s035',
-      'S036': 'custom/typescript_s036',
-      'S037': 'custom/typescript_s037',
-      'S038': 'custom/typescript_s038',
-      'S039': 'custom/typescript_s039',
-      'S041': 'custom/typescript_s041',
-      'S042': 'custom/typescript_s042',
-      'S043': 'custom/typescript_s043',
-      'S044': 'custom/typescript_s044',
-      'S045': 'custom/typescript_s045',
-      'S046': 'custom/typescript_s046',
-      'S048': 'custom/typescript_s048',
-      'S050': 'custom/typescript_s050',
-      'S052': 'custom/typescript_s052',
-      'S054': 'custom/typescript_s054',
-      'S057': 'custom/typescript_s057',
-      'S058': 'custom/typescript_s058'
-    };
-
-    return rules
-      .filter(rule => ruleMapping[rule.id])
-      .map(rule => ruleMapping[rule.id]);
-  }
-
-  /**
-   * Command: Build ESLint command
-   */
-  buildESLintCommand(eslintRules, options) {
-    const parts = [
-      'npx eslint',
-      `--config ${this.eslintConfigPath}`,
-      '--format json',
-      `"${options.input}"`
-    ];
-
-    // Add specific rules if provided
-    if (eslintRules.length > 0) {
-      const rulesFlag = eslintRules.map(rule => `${rule}:error`).join(' ');
-      parts.push(`--rule "${rulesFlag}"`);
-    }
-
-    // Add file extensions
-    parts.push('--ext .ts,.tsx,.js,.jsx');
-
-    return parts.join(' ');
-  }
-
-  /**
-   * Query: Parse ESLint JSON output to SunLint format
-   */
-  parseESLintOutput(output, options) {
-    try {
-      const eslintResults = JSON.parse(output);
-      
-      const results = {
-        results: [],
-        filesAnalyzed: eslintResults.length,
-        engine: 'eslint'
-      };
-
-      eslintResults.forEach(file => {
-        if (file.messages.length > 0) {
-          const violations = file.messages.map(msg => ({
-            ruleId: this.mapESLintRuleToSunLint(msg.ruleId),
-            severity: this.mapESLintSeverity(msg.severity),
-            message: msg.message,
-            line: msg.line,
-            column: msg.column,
-            file: file.filePath
-          }));
-
-          results.results.push({
-            file: file.filePath,
-            violations: violations
-          });
-        }
-      });
-
-      return results;
-    } catch (error) {
-      throw new Error(`Failed to parse ESLint output: ${error.message}`);
-    }
-  }
-
-  /**
-   * Query: Map ESLint rule ID back to SunLint rule ID
-   */
-  mapESLintRuleToSunLint(eslintRuleId) {
-    const reverseMapping = {
-      // Quality rules (C-rules)
-      'custom/c002': 'C002',
-      'custom/c003': 'C003',
-      'custom/c006': 'C006',
-      'custom/c010': 'C010',
-      'custom/c013': 'C013',
-      'custom/c014': 'C014',
-      'custom/c017': 'C017',
-      'custom/c018': 'C018',
-      'custom/c023': 'C023',
-      'custom/c027': 'C027',
-      'custom/c029': 'C029',
-      'custom/c030': 'C030',
-      'custom/c034': 'C034',
-      'custom/c035': 'C035',
-      'custom/c041': 'C041',
-      'custom/c042': 'C042',
-      'custom/c043': 'C043',
-      'custom/c047': 'C047',
-      'custom/c048': 'C048',
-      
-      // Security rules (S-rules)
-      'custom/typescript_s005': 'S005',
-      'custom/typescript_s006': 'S006',
-      'custom/typescript_s008': 'S008',
-      'custom/typescript_s009': 'S009',
-      'custom/typescript_s010': 'S010',
-      'custom/typescript_s011': 'S011',
-      'custom/typescript_s012': 'S012',
-      'custom/typescript_s014': 'S014',
-      'custom/typescript_s015': 'S015',
-      'custom/typescript_s016': 'S016',
-      'custom/typescript_s017': 'S017',
-      'custom/typescript_s018': 'S018',
-      'custom/typescript_s019': 'S019',
-      'custom/typescript_s020': 'S020',
-      'custom/typescript_s022': 'S022',
-      'custom/typescript_s023': 'S023',
-      'custom/typescript_s025': 'S025',
-      'custom/typescript_s026': 'S026',
-      'custom/typescript_s027': 'S027',
-      'custom/typescript_s029': 'S029',
-      'custom/typescript_s030': 'S030',
-      'custom/typescript_s033': 'S033',
-      'custom/typescript_s034': 'S034',
-      'custom/typescript_s035': 'S035',
-      'custom/typescript_s036': 'S036',
-      'custom/typescript_s037': 'S037',
-      'custom/typescript_s038': 'S038',
-      'custom/typescript_s039': 'S039',
-      'custom/typescript_s041': 'S041',
-      'custom/typescript_s042': 'S042',
-      'custom/typescript_s043': 'S043',
-      'custom/typescript_s044': 'S044',
-      'custom/typescript_s045': 'S045',
-      'custom/typescript_s046': 'S046',
-      'custom/typescript_s048': 'S048',
-      'custom/typescript_s050': 'S050',
-      'custom/typescript_s052': 'S052',
-      'custom/typescript_s054': 'S054',
-      'custom/typescript_s057': 'S057',
-      'custom/typescript_s058': 'S058'
-    };
-
-    return reverseMapping[eslintRuleId] || eslintRuleId;
-  }
-
-  /**
-   * Query: Map ESLint severity to SunLint severity
-   */
-  mapESLintSeverity(eslintSeverity) {
-    switch (eslintSeverity) {
-      case 1: return 'warning';
-      case 2: return 'error';
-      default: return 'info';
-    }
-  }
-}
-
-module.exports = ESLintEngineService;

package/core/eslint-instance-manager.js

@@ -1,104 +0,0 @@
-const path = require('path');
-const fs = require('fs');
-const chalk = require('chalk');
-
-/**
- * Handles ESLint instance initialization and configuration detection
- * Rule C005: Single responsibility - only ESLint initialization
- * Rule C015: Domain language - EslintInstanceManager
- */
-class EslintInstanceManager {
-  constructor() {
-    this.eslintInstance = null;
-    this.isInitialized = false;
-    this.eslintModulePath = null;
-  }
-
-  /**
-   * Rule C006: initializeEslintInstance - verb-noun naming
-   * Rule C032: No external API calls in constructor - initialization method
-   */
-  async initializeEslintInstance(eslintModulePath = null) {
-    if (this.isInitialized) {
-      return this.eslintInstance;
-    }
-
-    try {
-      // Rule C014: Dependency injection from main package
-      const { ESLint } = require('eslint');
-      
-      const configPath = this.resolveConfigPath(eslintModulePath);
-      
-      this.eslintInstance = new ESLint({
-        overrideConfigFile: path.join(configPath, 'eslint.config.js'),
-        cache: false,
-        fix: false,
-        overrideConfig: {
-          linterOptions: {
-            reportUnusedDisableDirectives: 'error'
-          }
-        }
-      });
-
-      this.isInitialized = true;
-      this.eslintModulePath = configPath;
-      
-      return this.eslintInstance;
-    } catch (error) {
-      throw new Error(`Failed to initialize ESLint: ${error.message}`);
-    }
-  }
-
-  /**
-   * Rule C006: resolveConfigPath - verb-noun naming
-   * Rule C005: Single responsibility - config path resolution
-   */
-  resolveConfigPath(eslintModulePath) {
-    if (eslintModulePath) {
-      return eslintModulePath;
-    }
-
-    // Try primary config path, fallback to eslint-integration
-    const primaryPath = path.join(__dirname, '..', 'config', 'typescript');
-    const fallbackPath = path.join(__dirname, '..', 'eslint-integration');
-    const configFile = 'eslint.config.js';
-    
-    // Check if primary config exists
-    if (fs.existsSync(path.join(primaryPath, configFile))) {
-      return primaryPath;
-    } else {
-      // Fallback to eslint-integration
-      console.log(chalk.yellow('⚠️  Using fallback ESLint config from eslint-integration'));
-      return fallbackPath;
-    }
-  }
-
-  /**
-   * Rule C006: getEslintInstance - verb-noun naming
-   * Rule C012: Query method - returns instance without side effects
-   */
-  getEslintInstance() {
-    if (!this.isInitialized) {
-      throw new Error('ESLint instance not initialized. Call initializeEslintInstance() first.');
-    }
-    return this.eslintInstance;
-  }
-
-  /**
-   * Rule C006: getConfigPath - verb-noun naming
-   * Rule C012: Query method
-   */
-  getConfigPath() {
-    return this.eslintModulePath;
-  }
-
-  /**
-   * Rule C006: checkInstanceReady - verb-noun naming
-   * Rule C012: Query method
-   */
-  checkInstanceReady() {
-    return this.isInitialized && this.eslintInstance !== null;
-  }
-}
-
-module.exports = EslintInstanceManager;