@sun-asterisk/sunlint 1.2.1 → 1.3.0

package/config/README.md

@@ -1,88 +0,0 @@
-# SunLint Configuration Structure
-
-This folder contains all configuration files for SunLint, organized for clarity and maintainability.
-
-## 📁 Structure Overview
-
-```
-config/
-├── schemas/                    # JSON schemas for validation
-│   └── sunlint-schema.json    # Main SunLint config schema
-├── engines/                    # Analysis engine configurations
-│   ├── engines.json           # Available engines (ESLint, TypeScript, etc.)
-│   └── eslint-rule-mapping.json  # ESLint rule mappings
-├── presets/                    # Pre-defined rule configurations
-│   ├── beginner.json          # Beginner-friendly preset
-│   ├── ci.json                # CI/CD optimized preset
-│   ├── recommended.json       # Recommended preset
-│   └── strict.json            # Strict coding standards
-├── integrations/              # Integration-specific configs
-│   └── eslint/
-│       ├── base.config.js     # Base ESLint configuration
-│       ├── typescript.config.js  # TypeScript ESLint config
-│       └── simple.config.js   # Simplified ESLint config
-├── rules/                     # Rule definitions and registry
-│   └── rules-registry.json   # Master rule registry
-├── defaults/                  # Default configurations
-│   ├── default.json          # Default SunLint settings
-│   └── ai-rules-context.json # AI analysis context
-└── testing/                   # Test configurations and samples
-    └── test-s005-working.ts  # Test file for S005 rule
-```
-
-## 🎯 Key Improvements
-
-### ✅ Eliminated Duplicates
-- **Before**: ESLint configs in both `config/typescript/` and `integrations/eslint/`
-- **After**: All ESLint configs consolidated in `config/integrations/eslint/`
-
-### ✅ Logical Organization
-- **Schemas**: All JSON schemas in one place
-- **Engines**: Engine-specific configurations separated
-- **Presets**: User-facing preset configurations grouped
-- **Integrations**: Third-party integration configs organized by tool
-
-### ✅ Reduced Complexity
-- **Before**: 10+ files scattered in root config/
-- **After**: Organized into 6 logical categories
-
-## 📋 Usage
-
-### For ESLint Integration
-```bash
-# Use the consolidated TypeScript ESLint config
-npx eslint --config config/integrations/eslint/typescript.config.js src/
-
-# Use the base ESLint config
-npx eslint --config config/integrations/eslint/base.config.js src/
-```
-
-### For Rule Presets
-```json
-{
-  "extends": "config/presets/recommended.json"
-}
-```
-
-### For Schema Validation
-```json
-{
-  "$schema": "config/schemas/sunlint-schema.json"
-}
-```
-
-## 🔧 Migration Notes
-
-- **Old `config/typescript/`**: ❌ Removed (duplicated functionality)
-- **ESLint configs**: ✅ Moved to `config/integrations/eslint/`
-- **Default configs**: ✅ Moved to `config/defaults/`
-- **Engine configs**: ✅ Moved to `config/engines/`
-
-## 🚀 Next Steps
-
-1. Update documentation references to new paths
-2. Update CI/CD scripts to use new config locations
-3. Consider adding more integration-specific configs as needed
-
----
-*Last updated: July 21, 2025 | SunLint Config Refactor*

package/config/engines/eslint-rule-mapping.json

@@ -1,74 +0,0 @@
-{
-  "C005": [
-    "max-statements-per-line",
-    "complexity"
-  ],
-  "C006": [
-    "func-names",
-    "func-name-matching",
-    "@typescript-eslint/naming-convention"
-  ],
-  "C007": [
-    "spaced-comment",
-    "no-inline-comments",
-    "no-warning-comments"
-  ],
-  "C012": [
-    "consistent-return",
-    "no-void",
-    "@typescript-eslint/no-confusing-void-expression"
-  ],
-  "C014": [
-    "no-new",
-    "no-new-wrappers",
-    "@typescript-eslint/no-unnecessary-constructor"
-  ],
-  "C015": [
-    "@typescript-eslint/naming-convention",
-    "camelcase"
-  ],
-  "C019": [
-    "no-console",
-    "no-alert",
-    "no-debugger"
-  ],
-  "C031": [
-    "no-implicit-coercion",
-    "eqeqeq",
-    "@typescript-eslint/strict-boolean-expressions"
-  ],
-  "C032": [
-    "no-new",
-    "@typescript-eslint/no-floating-promises",
-    "no-constructor-return"
-  ],
-  "C033": [
-    "prefer-const",
-    "no-var",
-    "@typescript-eslint/prefer-readonly"
-  ],
-  "C034": [
-    "no-global-assign",
-    "no-implicit-globals",
-    "@typescript-eslint/no-namespace"
-  ],
-  "C035": [
-    "no-empty-catch",
-    "@typescript-eslint/no-unused-vars"
-  ],
-  "C037": [
-    "consistent-return",
-    "@typescript-eslint/explicit-function-return-type",
-    "@typescript-eslint/explicit-module-boundary-types"
-  ],
-  "C038": [
-    "import/no-dynamic-require",
-    "import/order",
-    "@typescript-eslint/no-var-requires"
-  ],
-  "C040": [
-    "no-duplicate-imports",
-    "import/no-duplicates",
-    "@typescript-eslint/no-duplicate-imports"
-  ]
-}

package/config/testing/test-s005-working.ts

@@ -1,22 +0,0 @@
-function doAdminTask() {}
-function doAction() {}
-function isAuthenticated(req: any): boolean {
-  return true;
-}
-
-function doPost(request: any, response: any) { // ❌
-  const origin = request.getHeader("Origin");
-
-  if (origin === "https://admin.example.com") {
-    doAdminTask();
-  }
-}
-
-function doPost_safe(request: any, response: any) { // ✅
-  const origin = request.getHeader("Origin");
-  console.log("Origin:", origin);
-
-  if (isAuthenticated(request)) {
-    doAction();
-  }
-}

package/scripts/merge-reports.js

@@ -1,424 +0,0 @@
-#!/usr/bin/env node
-
-/**
- * Report Merger - Combines ESLint and SunLint results into unified report
- * Usage: node merge-reports.js eslint-results.json sunlint-results.json --format=summary --output=unified.json
- */
-
-const fs = require('fs');
-const path = require('path');
-const { Command } = require('commander');
-const chalk = require('chalk');
-
-class ReportMerger {
-  constructor() {
-    this.version = '1.0.0';
-  }
-
-  async mergeReports(eslintPath, sunlintPath, options = {}) {
-    const eslintReport = this.loadReport(eslintPath, 'eslint');
-    const sunlintReport = this.loadReport(sunlintPath, 'sunlint');
-
-    const unifiedReport = {
-      metadata: {
-        timestamp: new Date().toISOString(),
-        merger_version: this.version,
-        input_files: {
-          eslint: eslintPath,
-          sunlint: sunlintPath
-        },
-        tools: {
-          eslint: this.extractESLintMetadata(eslintReport),
-          sunlint: this.extractSunLintMetadata(sunlintReport)
-        }
-      },
-      summary: this.generateSummary(eslintReport, sunlintReport),
-      violations: this.mergeViolations(eslintReport, sunlintReport),
-      files: this.mergeFileAnalysis(eslintReport, sunlintReport),
-      rules: this.mergeRuleAnalysis(eslintReport, sunlintReport),
-      raw_reports: {
-        eslint: eslintReport,
-        sunlint: sunlintReport
-      }
-    };
-
-    return unifiedReport;
-  }
-
-  loadReport(filePath, toolName) {
-    if (!fs.existsSync(filePath)) {
-      console.warn(chalk.yellow(`⚠️  ${toolName} report not found: ${filePath}`));
-      return this.getEmptyReport(toolName);
-    }
-
-    try {
-      const content = fs.readFileSync(filePath, 'utf8');
-      return JSON.parse(content);
-    } catch (error) {
-      console.error(chalk.red(`❌ Failed to parse ${toolName} report: ${error.message}`));
-      return this.getEmptyReport(toolName);
-    }
-  }
-
-  getEmptyReport(toolName) {
-    return {
-      tool: toolName,
-      files: [],
-      violations: [],
-      summary: { total: 0, errors: 0, warnings: 0, info: 0 }
-    };
-  }
-
-  extractESLintMetadata(report) {
-    return {
-      version: report.version || 'unknown',
-      rules_count: report.rules ? Object.keys(report.rules).length : 0,
-      files_analyzed: report.files ? report.files.length : 0
-    };
-  }
-
-  extractSunLintMetadata(report) {
-    return {
-      version: report.version || 'unknown',
-      rules_count: report.rulesRun || 0,
-      files_analyzed: report.filesAnalyzed || 0,
-      ai_enabled: report.aiEnabled || false
-    };
-  }
-
-  generateSummary(eslintReport, sunlintReport) {
-    const eslintSummary = this.getSummaryFromReport(eslintReport, 'eslint');
-    const sunlintSummary = this.getSummaryFromReport(sunlintReport, 'sunlint');
-
-    return {
-      total: {
-        files: eslintSummary.files + sunlintSummary.files,
-        violations: eslintSummary.violations + sunlintSummary.violations,
-        errors: eslintSummary.errors + sunlintSummary.errors,
-        warnings: eslintSummary.warnings + sunlintSummary.warnings,
-        info: eslintSummary.info + sunlintSummary.info
-      },
-      by_tool: {
-        eslint: eslintSummary,
-        sunlint: sunlintSummary
-      },
-      breakdown: {
-        overlap_files: this.countOverlapFiles(eslintReport, sunlintReport),
-        unique_eslint_rules: this.getUniqueRules(eslintReport, 'eslint'),
-        unique_sunlint_rules: this.getUniqueRules(sunlintReport, 'sunlint')
-      }
-    };
-  }
-
-  getSummaryFromReport(report, toolName) {
-    if (toolName === 'eslint') {
-      // ESLint format
-      const violations = this.extractESLintViolations(report);
-      return {
-        files: report.files ? report.files.length : 0,
-        violations: violations.length,
-        errors: violations.filter(v => v.severity === 'error').length,
-        warnings: violations.filter(v => v.severity === 'warning').length,
-        info: violations.filter(v => v.severity === 'info').length
-      };
-    } else {
-      // SunLint format
-      const violations = report.results ? 
-        report.results.flatMap(r => r.violations || []) : [];
-      return {
-        files: report.filesAnalyzed || 0,
-        violations: violations.length,
-        errors: violations.filter(v => v.severity === 'error').length,
-        warnings: violations.filter(v => v.severity === 'warning').length,
-        info: violations.filter(v => v.severity === 'info').length
-      };
-    }
-  }
-
-  mergeViolations(eslintReport, sunlintReport) {
-    const eslintViolations = this.extractESLintViolations(eslintReport);
-    const sunlintViolations = this.extractSunLintViolations(sunlintReport);
-
-    const allViolations = [
-      ...eslintViolations.map(v => ({ ...v, tool: 'eslint' })),
-      ...sunlintViolations.map(v => ({ ...v, tool: 'sunlint' }))
-    ];
-
-    // Sort by severity and file
-    return allViolations.sort((a, b) => {
-      const severityOrder = { error: 0, warning: 1, info: 2 };
-      if (a.severity !== b.severity) {
-        return severityOrder[a.severity] - severityOrder[b.severity];
-      }
-      return a.file.localeCompare(b.file);
-    });
-  }
-
-  extractESLintViolations(report) {
-    if (!report.files || !Array.isArray(report.files)) return [];
-    
-    return report.files.flatMap(file => 
-      (file.messages || []).map(msg => ({
-        file: file.filePath || file.file,
-        line: msg.line,
-        column: msg.column,
-        ruleId: msg.ruleId,
-        message: msg.message,
-        severity: this.mapESLintSeverity(msg.severity)
-      }))
-    );
-  }
-
-  extractSunLintViolations(report) {
-    if (!report.results || !Array.isArray(report.results)) return [];
-    
-    return report.results.flatMap(result =>
-      (result.violations || []).map(violation => ({
-        file: violation.file,
-        line: violation.line,
-        column: violation.column,
-        ruleId: violation.ruleId,
-        message: violation.message,
-        severity: violation.severity
-      }))
-    );
-  }
-
-  mapESLintSeverity(severity) {
-    switch (severity) {
-      case 1: return 'warning';
-      case 2: return 'error';
-      default: return 'info';
-    }
-  }
-
-  mergeFileAnalysis(eslintReport, sunlintReport) {
-    const fileMap = new Map();
-
-    // Process ESLint files
-    if (eslintReport.files) {
-      eslintReport.files.forEach(file => {
-        const fileName = file.filePath || file.file;
-        fileMap.set(fileName, {
-          file: fileName,
-          eslint: {
-            violations: file.messages ? file.messages.length : 0,
-            rules_triggered: [...new Set((file.messages || []).map(m => m.ruleId))]
-          },
-          sunlint: { violations: 0, rules_triggered: [] }
-        });
-      });
-    }
-
-    // Process SunLint files
-    if (sunlintReport.results) {
-      sunlintReport.results.forEach(result => {
-        const fileName = result.file;
-        if (!fileMap.has(fileName)) {
-          fileMap.set(fileName, {
-            file: fileName,
-            eslint: { violations: 0, rules_triggered: [] },
-            sunlint: { violations: 0, rules_triggered: [] }
-          });
-        }
-        
-        const fileData = fileMap.get(fileName);
-        fileData.sunlint = {
-          violations: result.violations ? result.violations.length : 0,
-          rules_triggered: [...new Set((result.violations || []).map(v => v.ruleId))]
-        };
-      });
-    }
-
-    return Array.from(fileMap.values());
-  }
-
-  mergeRuleAnalysis(eslintReport, sunlintReport) {
-    const ruleMap = new Map();
-
-    // Analyze ESLint rules
-    this.extractESLintViolations(eslintReport).forEach(violation => {
-      if (!ruleMap.has(violation.ruleId)) {
-        ruleMap.set(violation.ruleId, {
-          rule: violation.ruleId,
-          tool: 'eslint',
-          violations: 0,
-          files: new Set()
-        });
-      }
-      const rule = ruleMap.get(violation.ruleId);
-      rule.violations++;
-      rule.files.add(violation.file);
-    });
-
-    // Analyze SunLint rules
-    this.extractSunLintViolations(sunlintReport).forEach(violation => {
-      if (!ruleMap.has(violation.ruleId)) {
-        ruleMap.set(violation.ruleId, {
-          rule: violation.ruleId,
-          tool: 'sunlint',
-          violations: 0,
-          files: new Set()
-        });
-      }
-      const rule = ruleMap.get(violation.ruleId);
-      rule.violations++;
-      rule.files.add(violation.file);
-    });
-
-    // Convert Sets to arrays and sort
-    return Array.from(ruleMap.values())
-      .map(rule => ({
-        ...rule,
-        files: Array.from(rule.files),
-        files_count: rule.files.size
-      }))
-      .sort((a, b) => b.violations - a.violations);
-  }
-
-  countOverlapFiles(eslintReport, sunlintReport) {
-    const eslintFiles = new Set();
-    const sunlintFiles = new Set();
-
-    if (eslintReport.files) {
-      eslintReport.files.forEach(f => eslintFiles.add(f.filePath || f.file));
-    }
-    
-    if (sunlintReport.results) {
-      sunlintReport.results.forEach(r => sunlintFiles.add(r.file));
-    }
-
-    return [...eslintFiles].filter(file => sunlintFiles.has(file)).length;
-  }
-
-  getUniqueRules(report, toolName) {
-    if (toolName === 'eslint') {
-      return [...new Set(this.extractESLintViolations(report).map(v => v.ruleId))];
-    } else {
-      return [...new Set(this.extractSunLintViolations(report).map(v => v.ruleId))];
-    }
-  }
-
-  formatReport(unifiedReport, format) {
-    switch (format) {
-      case 'json':
-        return JSON.stringify(unifiedReport, null, 2);
-      case 'summary':
-        return this.formatSummary(unifiedReport);
-      case 'github':
-        return this.formatGitHub(unifiedReport);
-      case 'detailed':
-        return this.formatDetailed(unifiedReport);
-      default:
-        return this.formatSummary(unifiedReport);
-    }
-  }
-
-  formatSummary(report) {
-    const { summary } = report;
-    return `
-🎯 Sun* Engineering - Unified Code Quality Report
-════════════════════════════════════════════════
-
-📊 Overall Summary:
-  Files analyzed: ${summary.total.files}
-  Total violations: ${summary.total.violations}
-  
-🚨 By Severity:
-  Errors: ${summary.total.errors}
-  Warnings: ${summary.total.warnings}  
-  Info: ${summary.total.info}
-
-🔧 By Tool:
-  ESLint: ${summary.by_tool.eslint.violations} violations (${summary.by_tool.eslint.files} files)
-  SunLint: ${summary.by_tool.sunlint.violations} violations (${summary.by_tool.sunlint.files} files)
-
-📁 Coverage:
-  Overlap files: ${summary.breakdown.overlap_files}
-  ESLint rules: ${summary.breakdown.unique_eslint_rules.length}
-  SunLint rules: ${summary.breakdown.unique_sunlint_rules.length}
-
-Generated: ${report.metadata.timestamp}
-`;
-  }
-
-  formatGitHub(report) {
-    const annotations = report.violations
-      .filter(v => v.severity === 'error')
-      .slice(0, 10) // GitHub limits annotations
-      .map(v => `::error file=${v.file},line=${v.line}::${v.message} (${v.ruleId})`)
-      .join('\n');
-
-    return annotations + '\n' + this.formatSummary(report);
-  }
-
-  formatDetailed(report) {
-    let output = this.formatSummary(report);
-    
-    output += '\n🔍 Top Violated Rules:\n';
-    report.rules.slice(0, 10).forEach(rule => {
-      output += `  ${rule.rule}: ${rule.violations} violations (${rule.files_count} files) [${rule.tool}]\n`;
-    });
-
-    output += '\n📁 Most Problematic Files:\n';
-    const filesByViolations = report.files
-      .map(f => ({
-        ...f,
-        total: f.eslint.violations + f.sunlint.violations
-      }))
-      .sort((a, b) => b.total - a.total)
-      .slice(0, 10);
-
-    filesByViolations.forEach(file => {
-      output += `  ${file.file}: ${file.total} violations (ESLint: ${file.eslint.violations}, SunLint: ${file.sunlint.violations})\n`;
-    });
-
-    return output;
-  }
-}
-
-// CLI
-const program = new Command();
-
-program
-  .name('merge-reports')
-  .description('Merge ESLint and SunLint reports into unified report')
-  .version('1.0.0')
-  .argument('<eslint-report>', 'ESLint JSON report file')
-  .argument('<sunlint-report>', 'SunLint JSON report file')
-  .option('-f, --format <format>', 'Output format (json,summary,github,detailed)', 'summary')
-  .option('-o, --output <file>', 'Output file (default: console)')
-  .option('--verbose', 'Verbose logging');
-
-program.action(async (eslintReport, sunlintReport, options) => {
-  try {
-    const merger = new ReportMerger();
-    
-    if (options.verbose) {
-      console.log(chalk.blue('🔄 Merging reports...'));
-      console.log(`ESLint: ${eslintReport}`);
-      console.log(`SunLint: ${sunlintReport}`);
-    }
-
-    const unifiedReport = await merger.mergeReports(eslintReport, sunlintReport, options);
-    const formattedOutput = merger.formatReport(unifiedReport, options.format);
-
-    if (options.output) {
-      fs.writeFileSync(options.output, formattedOutput);
-      console.log(chalk.green(`✅ Unified report saved: ${options.output}`));
-    } else {
-      console.log(formattedOutput);
-    }
-
-  } catch (error) {
-    console.error(chalk.red('❌ Report merge failed:'), error.message);
-    if (options.verbose) {
-      console.error(error.stack);
-    }
-    process.exit(1);
-  }
-});
-
-program.parse();
-
-module.exports = ReportMerger;

package/scripts/test-scripts/README.md

@@ -1,22 +0,0 @@
-# Test Scripts
-
-This directory contains various test scripts for SunLint development and validation.
-
-## Files
-
-- `test-eslint-rules.js` - Tests ESLint rule implementations
-- `test-rules-on-real-projects.js` - Tests rules on real project samples
-- `test-real-world.js` - Real-world testing scenarios
-- `test-c041-eslint.js` - Specific tests for C041 ESLint rule
-- `test-c041-comparison.js` - Comparison tests between Heuristic and ESLint for C041
-
-## Usage
-
-Run any test script from the project root:
-
-```bash
-node scripts/test-scripts/test-eslint-rules.js
-node scripts/test-scripts/test-rules-on-real-projects.js
-```
-
-These scripts are used for development, validation, and debugging of SunLint rules.