@sun-asterisk/sunlint 1.1.7 → 1.2.0

package/.sunlint.json

@@ -1,5 +1,5 @@
 {
-  "extends": ["@sun/sunlint/recommended"],
+  "extends": "@sun/sunlint/recommended",
   "rules": {
     "C019": "warn",
     "C006": "warn", 

package/CHANGELOG.md

@@ -1,3 +1,86 @@
+# 🎉 SunLint Changelog
+
+---
+
+## 🔥 **v1.2.0 - Architecture Refactor (In Development)**
+
+**Release Date**: July 30, 2025  
+**Type**: Major Architecture Update (Adapter Pattern Implementation)
+
+### 🏗️ **Major Architecture Changes**
+
+#### **Unified Adapter Pattern**
+- **NEW**: `SunlintRuleAdapter` - Unified rule access layer for CLI
+- **IMPROVED**: Same adapter pattern as VSCode extension (`RuleReaderService`)
+- **ELIMINATED**: Direct parser/registry access across core modules
+- **PERFORMANCE**: 0.07ms average per rule query with singleton caching
+
+#### **Refactored Core Modules**
+- **UPDATED**: `core/rule-selection-service.js` - Now uses adapter exclusively
+- **UPDATED**: `core/config-manager.js` - Adapter-driven config validation
+- **UPDATED**: `core/analysis-orchestrator.js` - Unified rule initialization
+- **UPDATED**: `engines/heuristic-engine.js` - Adapter-based rule access
+
+#### **Enhanced Rule Management**
+- **IMPROVED**: 256 rules loaded from registry with fallback to origin-rules
+- **ADDED**: AI context generation via `generateAIContext()` method
+- **ENHANCED**: Engine compatibility checking (heuristic: 244, eslint: 17, ai: 256)
+- **OPTIMIZED**: Memory usage with singleton pattern
+
+### 🎯 **Benefits**
+- **No Rule Model Duplication**: Single source of truth across CLI and VSCode
+- **Extensible Architecture**: Easy to add new engines or rule sources
+- **Consistent OpenAI Integration**: Proper context extraction from origin-rules
+- **Maintainable Codebase**: Centralized rule logic through adapter layer
+
+### 📊 **Performance Metrics**
+- **Rule Loading**: 256 rules in ~10ms
+- **Query Performance**: 0.07ms average per `getAllRules()` call
+- **Engine Coverage**: Heuristic (95.3%), ESLint (6.6%), AI (100%)
+- **Memory Efficiency**: Singleton prevents duplicate instances
+
+### 🧪 **Testing & Validation**
+- **ADDED**: `test-adapter.js` - Comprehensive adapter testing
+- **UPDATED**: Integration tests now use adapter methods
+- **VERIFIED**: All 3/3 integration tests pass
+- **VALIDATED**: Engine orchestration and rule compatibility
+
+---
+
+## 🎉 **v1.1.8 Release Notes**
+
+**Release Date**: July 24, 2025  
+**Type**: Minor Release (ESLint 9.x Compatibility & Enhanced Error Handling)
+
+---
+
+## 🚀 **Key Improvements**
+
+### 🔧 **ESLint 9.x Full Compatibility**
+- **Fixed**: `context.getSource is not a function` error with React Hooks plugin
+- **Enhanced**: Robust plugin compatibility detection and fallback mechanisms
+- **Improved**: Legacy config to flat config conversion for ESLint 9.x projects
+- **Added**: Graceful degradation when plugins fail to load
+
+### 🛡️ **Enhanced Error Handling**
+- **Smart**: Plugin version detection with upgrade guidance
+- **Robust**: Fallback to minimal ESLint configuration when plugins fail
+- **Clear**: Detailed error messages for troubleshooting plugin issues
+- **Stable**: Continue analysis even with incompatible plugins
+
+### ✅ **Real-World Validation**
+- **Tested**: Successfully validated on 3 production projects (NestJS, Next.js)
+- **Verified**: 820+ files analyzed without crashes
+- **Proven**: Handles ESLint 8.x, 9.x, and mixed configurations
+
+### 🎯 **Plugin Compatibility**
+- **React Hooks**: Fixed compatibility issues with outdated versions
+- **TypeScript ESLint**: Enhanced support for v5.x and v8.x
+- **Security Plugins**: Graceful handling of missing security rules
+- **Custom Plugins**: Better error recovery for third-party plugins
+
+---
+
 # 🎉 SunLint v1.1.7 Release Notes
 
 **Release Date**: July 24, 2025  

package/README.md

@@ -7,15 +7,42 @@
 Sun Lint is a universal coding standards checker providing comprehensive code quality and security analysis. Built by Sun* Engineering Team with integrated security rules from OWASP and industry best practices.
 
 ### **✨ Key Features**
-- ✅ **97+ Coding Rules**: Quality (30), Security (47), TypeScript-specific
+- ✅ **256+ Coding Rules**: Quality (161), Security (70), Performance (25)
+- ✅ **Unified Architecture**: Same adapter pattern for CLI and VSCode extension
+- ✅ **Multi-Engine Support**: Heuristic (244 rules) + ESLint (17 rules) + AI (256 rules)
 - ✅ **Built-in AST Analysis**: JavaScript/TypeScript parsing out of the box
-- ✅ **Multi-Engine Architecture**: Heuristic + ESLint + OpenAI integration  
 - ✅ **Git Integration**: `--changed-files`, `--staged-files`, `--pr-mode`
 - ✅ **TypeScript Support**: Native TypeScript 5.8+ analysis
 - ✅ **Zero Config**: Works immediately after `npm install`
 - ✅ **CI/CD Ready**: Baseline comparison, fail-on-new-violations
 - ✅ **Advanced File Targeting**: Include/exclude patterns, language filtering
 
+### **🏗️ Architecture**
+
+SunLint uses a unified adapter pattern ensuring consistency between CLI and VSCode extension:
+
+```
+┌─────────────────┬─────────────────┐
+│   CLI Tools     │ VSCode Extension│
+├─────────────────┼─────────────────┤
+│ SunlintRule     │ RuleReader      │
+│ Adapter         │ Service         │
+├─────────────────┴─────────────────┤
+│     Rule Sources & Engines        │
+│ • Registry (auto-generated)       │
+│ • Origin Rules (markdown)         │
+│ • Heuristic Engine (244 rules)    │
+│ • ESLint Engine (17 rules)        │
+│ • AI Engine (256 rules)           │
+└───────────────────────────────────┘
+```
+
+**Benefits:**
+- **No Rule Model Duplication**: Single source of truth
+- **Extensible**: Easy to add new engines or rule sources
+- **Performance**: 0.07ms average per rule query
+- **AI Integration**: Consistent OpenAI context from origin-rules
+
 ### **🚀 Quick Start**
 ```bash
 # Install
@@ -113,8 +140,27 @@ npm install --save-dev @sun-asterisk/sunlint eslint @typescript-eslint/parser @t
 }
 ```
 
-## 🔗 **ESLint Integration**
+## 🔗 **Multi-Engine Architecture**
+
+SunLint automatically selects the best engine for each rule, providing comprehensive coverage:
+
+### **Engine Coverage & Performance**
+```bash
+# Show engine compatibility for specific rules
+sunlint --show-engines --rules=C010,R001,S005
 
+# Use specific engine
+sunlint --engine=heuristic --rules=C010,C020 --input=src
+sunlint --engine=eslint --rules=R001,R006 --input=src  
+sunlint --engine=ai --rules=C010,S001 --input=src
+```
+
+**Engine Stats:**
+- **Heuristic Engine**: 244/256 rules (95.3%) - Fast, universal
+- **ESLint Engine**: 17/256 rules (6.6%) - JavaScript/TypeScript focused
+- **AI Engine**: 256/256 rules (100%) - Context-aware analysis
+
+### **ESLint Integration**
 Seamlessly integrate with existing ESLint configurations:
 
 ```bash
@@ -129,7 +175,7 @@ sunlint --rules=C010,C006 --eslint-integration --input=src
 - ✅ **Multi-engine orchestration**: Rules automatically routed to optimal engine
 - ✅ **ESLint v8/v9 compatibility**: Production-ready with both major versions  
 - ✅ **TypeScript support**: Full TS/TSX parsing with custom rule implementation
-- ✅ **Custom rule integration**: 27+ SunLint custom rules via ESLint engine  
+- ✅ **Custom rule integration**: 17+ SunLint custom rules via ESLint engine  
 - ✅ **Smart fallback**: Automatic engine fallback for maximum rule coverage
 - ✅ **Production tested**: Successfully processes real projects with mixed violations
 
@@ -137,6 +183,7 @@ sunlint --rules=C010,C006 --eslint-integration --input=src
 - ✅ **No workflow disruption**: Existing ESLint continues working
 - ✅ **Engine flexibility**: Automatic best-engine selection per rule
 - ✅ **Combined reporting**: Unified violation tracking from multiple engines
+- ✅ **Adapter Pattern**: Same rule access layer as VSCode extension
 
 ## 🔀 **Git Integration**
 
@@ -298,6 +345,21 @@ sunlint --all --changed-files --fail-on-new-violations
 sunlint --all --staged-files --format=summary
 ```
 
+### **Testing & Debugging**
+```bash
+# Test adapter performance and coverage
+sunlint --test-adapter
+
+# Show detailed engine information
+sunlint --show-engines --verbose
+
+# Debug rule selection process
+sunlint --rules=C010,R001 --verbose --debug
+
+# Validate configuration
+sunlint --validate-config .sunlint.json
+```
+
 ## 📚 **Documentation**
 
 - **[Configuration Guide](./docs/CONFIGURATION.md)** - Complete config options with examples

package/config/presets/all.json

@@ -0,0 +1,125 @@
+{
+  "name": "@sun/sunlint/all",
+  "description": "Comprehensive configuration with all available activated rules (v1.0+)",
+  "rules": {
+    "C003": "warn",
+    "C006": "warn",
+    "C013": "warn",
+    "C014": "warn",
+    "C017": "warn",
+    "C018": "warn",
+    "C019": "warn",
+    "C023": "warn",
+    "C024": "warn",
+    "C029": "warn",
+    "C030": "warn",
+    "C031": "warn",
+    "C033": "warn",
+    "C035": "warn",
+    "C040": "warn",
+    "C041": "warn",
+    "C042": "warn",
+    "C043": "warn",
+    "C047": "warn",
+    "C048": "warn",
+    "C052": "warn",
+    "C056": "warn",
+    "C060": "warn",
+    "C061": "warn",
+    "C065": "warn",
+    "C067": "warn",
+    "C070": "warn",
+    "C072": "warn",
+    "C073": "warn",
+    "C075": "warn",
+    "C076": "warn",
+    "S001": "error",
+    "S002": "error",
+    "S003": "warn",
+    "S004": "warn",
+    "S005": "warn",
+    "S006": "warn",
+    "S007": "warn",
+    "S008": "warn",
+    "S009": "warn",
+    "S010": "error",
+    "S011": "error",
+    "S012": "warn",
+    "S013": "error",
+    "S014": "warn",
+    "S015": "warn",
+    "S016": "error",
+    "S017": "error",
+    "S018": "error",
+    "S019": "error",
+    "S020": "error",
+    "S021": "error",
+    "S022": "error",
+    "S023": "error",
+    "S024": "error",
+    "S025": "error",
+    "S026": "warn",
+    "S027": "error",
+    "S028": "error",
+    "S029": "error",
+    "S030": "error",
+    "S031": "error",
+    "S032": "error",
+    "S033": "error",
+    "S034": "error",
+    "S035": "error",
+    "S036": "error",
+    "S037": "error",
+    "S038": "error",
+    "S039": "error",
+    "S040": "error",
+    "S041": "error",
+    "S042": "error",
+    "S043": "error",
+    "S044": "warn",
+    "S045": "error",
+    "S046": "error",
+    "S047": "error",
+    "S048": "error",
+    "S049": "error",
+    "S050": "error",
+    "S051": "error",
+    "S052": "error",
+    "S053": "error",
+    "S054": "error",
+    "S055": "error",
+    "S056": "error",
+    "S057": "error",
+    "S058": "error",
+    "S059": "error"
+  },
+  "categories": {
+    "quality": "warn",
+    "security": "error",
+    "performance": "warn",
+    "maintainability": "warn",
+    "testability": "warn",
+    "documentation": "warn"
+  },
+  "languages": [
+    "typescript",
+    "javascript",
+    "dart",
+    "java",
+    "kotlin",
+    "swift"
+  ],
+  "exclude": [
+    "**/node_modules/**",
+    "**/build/**",
+    "**/dist/**",
+    "**/*.generated.*",
+    "**/*.min.*"
+  ],
+  "metadata": {
+    "totalRules": 90,
+    "removedRules": 0,
+    "lastUpdated": "2025-07-30T08:59:10.118Z",
+    "source": "origin-rules"
+  }
+}

package/config/presets/beginner.json

@@ -2,18 +2,20 @@
   "name": "@sun/sunlint/beginner",
   "description": "Beginner-friendly configuration with warnings only",
   "rules": {
-    "C019": "warning", 
-    "C029": "warning",
+    "C019": "warn",
+    "C029": "warn",
     "C006": "info"
   },
   "categories": {
-    "quality": "warning",
-    "security": "warning", 
-    "logging": "warning",
+    "quality": "warn",
+    "security": "warn",
+    "logging": "warn",
     "naming": "info",
-    "validation": "warning"
+    "validation": "warn"
   },
-  "languages": ["typescript"],
+  "languages": [
+    "typescript"
+  ],
   "exclude": [
     "**/node_modules/**",
     "**/build/**",
@@ -32,5 +34,11 @@
     "showProgress": true,
     "includeContext": true,
     "showFixSuggestions": true
+  },
+  "metadata": {
+    "totalRules": 3,
+    "removedRules": 0,
+    "lastUpdated": "2025-07-30T08:59:10.119Z",
+    "source": "origin-rules"
   }
-}
+}

package/config/presets/ci.json

@@ -8,13 +8,15 @@
   },
   "categories": {
     "quality": "error",
-    "security": "error", 
+    "security": "error",
     "logging": "error",
     "naming": "off",
     "validation": "error"
   },
-  },
-  "languages": ["typescript", "dart"],
+  "languages": [
+    "typescript",
+    "dart"
+  ],
   "exclude": [
     "**/node_modules/**",
     "**/build/**",
@@ -43,5 +45,11 @@
     "showProgress": false,
     "includeContext": false,
     "showFixSuggestions": false
+  },
+  "metadata": {
+    "totalRules": 3,
+    "removedRules": 0,
+    "lastUpdated": "2025-07-30T08:59:10.120Z",
+    "source": "origin-rules"
   }
-}
+}

package/config/presets/maintainability.json

@@ -0,0 +1,38 @@
+{
+  "name": "@sun/sunlint/maintainability",
+  "description": "Maintainability and clean code focused configuration",
+  "rules": {
+    "C024": "warn",
+    "C031": "warn",
+    "C048": "warn",
+    "C052": "warn",
+    "C061": "warn",
+    "C067": "warn",
+    "C076": "warn",
+    "S008": "warn"
+  },
+  "categories": {
+    "maintainability": "warn"
+  },
+  "languages": [
+    "typescript",
+    "javascript",
+    "dart",
+    "java",
+    "kotlin",
+    "swift"
+  ],
+  "exclude": [
+    "**/node_modules/**",
+    "**/build/**",
+    "**/dist/**",
+    "**/*.generated.*",
+    "**/*.min.*"
+  ],
+  "metadata": {
+    "totalRules": 8,
+    "removedRules": 0,
+    "lastUpdated": "2025-07-30T08:59:10.120Z",
+    "source": "origin-rules"
+  }
+}

package/config/presets/performance.json

@@ -0,0 +1,32 @@
+{
+  "name": "@sun/sunlint/performance",
+  "description": "Performance-focused configuration for optimization",
+  "rules": {
+    "C043": "warn",
+    "C056": "warn"
+  },
+  "categories": {
+    "performance": "warn"
+  },
+  "languages": [
+    "typescript",
+    "javascript",
+    "dart",
+    "java",
+    "kotlin",
+    "swift"
+  ],
+  "exclude": [
+    "**/node_modules/**",
+    "**/build/**",
+    "**/dist/**",
+    "**/*.generated.*",
+    "**/*.min.*"
+  ],
+  "metadata": {
+    "totalRules": 2,
+    "removedRules": 0,
+    "lastUpdated": "2025-07-30T08:59:10.120Z",
+    "source": "origin-rules"
+  }
+}

package/config/presets/quality.json

@@ -0,0 +1,103 @@
+{
+  "name": "@sun/sunlint/quality",
+  "description": "Code quality and best practices focused configuration",
+  "rules": {
+    "C003": "warn",
+    "C006": "warn",
+    "C013": "warn",
+    "C014": "warn",
+    "C017": "warn",
+    "C018": "warn",
+    "C019": "warn",
+    "C023": "warn",
+    "C024": "warn",
+    "C029": "warn",
+    "C030": "warn",
+    "C031": "warn",
+    "C033": "warn",
+    "C035": "warn",
+    "C040": "warn",
+    "C042": "warn",
+    "C043": "warn",
+    "C047": "warn",
+    "C052": "warn",
+    "C060": "warn",
+    "C061": "warn",
+    "C065": "warn",
+    "C067": "warn",
+    "C070": "warn",
+    "C072": "warn",
+    "C073": "warn",
+    "C075": "warn",
+    "C076": "warn",
+    "S001": "error",
+    "S002": "error",
+    "S004": "warn",
+    "S005": "warn",
+    "S006": "warn",
+    "S007": "warn",
+    "S008": "warn",
+    "S009": "warn",
+    "S010": "error",
+    "S011": "error",
+    "S013": "error",
+    "S014": "warn",
+    "S017": "error",
+    "S021": "warn",
+    "S022": "warn",
+    "S023": "error",
+    "S025": "warn",
+    "S026": "warn",
+    "S027": "warn",
+    "S028": "warn",
+    "S029": "error",
+    "S030": "warn",
+    "S033": "warn",
+    "S034": "warn",
+    "S035": "warn",
+    "S036": "error",
+    "S041": "warn",
+    "S042": "warn",
+    "S043": "warn",
+    "S044": "warn",
+    "S045": "warn",
+    "S046": "warn",
+    "S047": "error",
+    "S048": "warn",
+    "S049": "warn",
+    "S050": "warn",
+    "S051": "warn",
+    "S052": "warn",
+    "S053": "warn",
+    "S054": "error",
+    "S055": "warn",
+    "S057": "warn",
+    "S058": "warn",
+    "S059": "warn"
+  },
+  "categories": {
+    "quality": "warn",
+    "security": "error"
+  },
+  "languages": [
+    "typescript",
+    "javascript",
+    "dart",
+    "java",
+    "kotlin",
+    "swift"
+  ],
+  "exclude": [
+    "**/node_modules/**",
+    "**/build/**",
+    "**/dist/**",
+    "**/*.generated.*",
+    "**/*.min.*"
+  ],
+  "metadata": {
+    "totalRules": 72,
+    "removedRules": 0,
+    "lastUpdated": "2025-07-30T08:59:10.121Z",
+    "source": "origin-rules"
+  }
+}

package/config/presets/recommended.json

@@ -1,24 +1,48 @@
 {
   "name": "@sun/sunlint/recommended",
-  "description": "Sun* Engineering recommended configuration",
+  "description": "Sun* Engineering recommended configuration - essential rules from core files (common-en.md + security-en.md)",
   "rules": {
-    "C019": "warning",
-    "C029": "error",
-    "C006": "off"
+    "C003": "warn",
+    "C006": "warn",
+    "C010": "warn",
+    "C019": "warn",
+    "C029": "warn",
+    "C031": "warn",
+    "C032": "warn",
+    "C041": "error",
+    "S001": "error",
+    "S002": "error",
+    "S003": "warn",
+    "S005": "error",
+    "S007": "error",
+    "S012": "error",
+    "S013": "error"
   },
   "categories": {
-    "quality": "warning",
-    "security": "error", 
-    "logging": "warning",
-    "naming": "off",
-    "validation": "error"
+    "quality": "warn",
+    "security": "error"
   },
-  "languages": ["typescript", "dart"],
+  "languages": [
+    "typescript",
+    "javascript",
+    "dart",
+    "java",
+    "kotlin",
+    "swift"
+  ],
   "exclude": [
     "**/node_modules/**",
     "**/build/**",
     "**/dist/**",
     "**/*.generated.*",
     "**/*.min.*"
-  ]
-}
+  ],
+  "metadata": {
+    "totalRules": 15,
+    "coreRules": 15,
+    "approach": "core-files-only",
+    "source": "common-en.md + security-en.md",
+    "lastUpdated": "2025-07-30T09:30:00.000Z",
+    "version": "2.0.0"
+  }
+}