@sun-asterisk/sunlint 1.3.3 → 1.3.4

package/CHANGELOG.md

@@ -2,7 +2,35 @@
 
 ---
 
-## 🚀 **v1.3.3 - Performance & File Limits Optimization (September 4, 2025)**
+## � **v1.3.4 - Engine Auto Hotfix (September 5, 2025)**
+
+**Release Date**: September 5, 2025  
+**Type**: Critical Hotfix
+
+### 🚨 **Critical Bug Fix**
+- **FIXED**: Engine "auto" validation and selection logic
+  - **Issue**: `--engine=auto` causing "Invalid engine: auto" error in v1.3.3
+  - **Root Cause**: Missing auto engine support in validation and orchestrator
+  - **Solution**: Comprehensive auto engine implementation
+    - Added "auto" case to engine factory with heuristic fallback
+    - Updated CLI validation to include "auto" in valid engines
+    - Enhanced orchestrator to resolve "auto" to actual engines (heuristic + eslint)
+    - Fixed CLI action handler auto-detection logic
+
+### 🧪 **Validation Results**
+- **✅ Auto engine**: Works correctly (auto-selects heuristic + eslint)
+- **✅ Heuristic engine**: Unchanged, working properly
+- **✅ ESLint engine**: Unchanged, working properly
+- **✅ CLI help**: Shows all engines including auto option
+
+### 📦 **Upgrade Notes**
+- **Zero breaking changes** - seamless upgrade from v1.3.3
+- **Default `--engine=auto`** now works as intended
+- **All existing commands** continue to work unchanged
+
+---
+
+## �🚀 **v1.3.3 - Performance & File Limits Optimization (September 4, 2025)**
 
 **Release Date**: September 4, 2025  
 **Type**: Performance Enhancement & User Experience
@@ -21,7 +49,14 @@
   - **Auto-detection**: Smart defaults for 90% of use cases
   - **Manual tuning**: Fine control for enterprise projects
 
-### 📚 **Documentation Expansion**
+### � **Bug Fixes**
+- **FIXED**: Engine "auto" validation and selection logic
+  - **Engine Factory**: Added "auto" case with fallback to heuristic engine
+  - **CLI Validation**: Added "auto" to valid engines list
+  - **Orchestrator**: Auto-resolve "auto" to actual engines (heuristic + eslint)
+  - **Engine Selection**: Auto-detection works correctly for rule preferences
+
+### �📚 **Documentation Expansion**
 - **NEW**: [FILE_LIMITS_EXPLANATION.md](./docs/FILE_LIMITS_EXPLANATION.md) - Comprehensive guide (5.7KB)
 - **NEW**: [QUICK_FILE_LIMITS.md](./docs/QUICK_FILE_LIMITS.md) - Quick reference (1.8KB)
 - **ENHANCED**: CLI help with clear usage examples

package/core/analysis-orchestrator.js

@@ -345,6 +345,12 @@ class AnalysisOrchestrator {
   getEnginePreference(rule, config) {
     // If user specified a specific engine via --engine option, use only that engine
     if (config.requestedEngine) {
+      // Handle "auto" engine selection
+      if (config.requestedEngine === 'auto') {
+        // Auto-select best engines: default to heuristic, add eslint for JS/TS
+        return ['heuristic', 'eslint'];
+      }
+      
       return [config.requestedEngine];
     }
 

package/core/cli-action-handler.js

@@ -164,6 +164,20 @@ class CliActionHandler {
   determineEnabledEngines(config) {
     // If specific engine is requested via --engine option, use only that engine
     if (this.options.engine) {
+      // Handle "auto" engine selection
+      if (this.options.engine === 'auto') {
+        // Auto-select best engines: default to heuristic for compatibility
+        const autoEngines = ['heuristic'];
+        
+        // Add ESLint for JS/TS files if available
+        if (this.hasJavaScriptTypeScriptFiles() || config.eslint?.enabled !== false) {
+          autoEngines.push('eslint');
+        }
+        
+        return autoEngines;
+      }
+      
+      // Return specific engine as requested
       return [this.options.engine];
     }
 
@@ -272,7 +286,7 @@ class CliActionHandler {
   validateInput(config) {
     // Validate engine option if specified (check this first, always)
     if (this.options.engine) {
-      const validEngines = ['eslint', 'heuristic', 'openai'];
+      const validEngines = ['auto', 'eslint', 'heuristic', 'openai'];
       if (!validEngines.includes(this.options.engine)) {
         throw new Error(
           chalk.red(`❌ Invalid engine: ${this.options.engine}\n`) +

package/engines/engine-factory.js

@@ -51,6 +51,13 @@ class EngineFactory {
     let engine = null;
 
     switch (engineType.toLowerCase()) {
+      case 'auto':
+        // Auto-detection: default to heuristic for best performance/compatibility
+        engine = new HeuristicEngine();
+        if (this.verbose) {
+          console.log('🤖 Auto-detected engine: heuristic (best performance/compatibility)');
+        }
+        break;
       case 'heuristic':
         engine = new HeuristicEngine();
         break;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sun-asterisk/sunlint",
-  "version": "1.3.3",
+  "version": "1.3.4",
   "description": "☀️ SunLint - Multi-language static analysis tool for code quality and security | Sun* Engineering Standards",
   "main": "cli.js",
   "bin": {