@sun-asterisk/sunlint 1.2.1 → 1.3.0

package/README.md

@@ -87,10 +87,17 @@ For advanced TypeScript analysis with ESLint integration:
 npm install --save-dev @sun-asterisk/sunlint eslint @typescript-eslint/parser @typescript-eslint/eslint-plugin typescript
 ```
 
+### **Full ESLint Integration Support**
+For complete ESLint integration with import analysis:
+
+```bash
+npm install --save-dev @sun-asterisk/sunlint eslint @typescript-eslint/parser @typescript-eslint/eslint-plugin eslint-plugin-import typescript
+```
+
 ### **What's Included by Default**
 - ✅ **JavaScript Analysis**: High-accuracy AST analysis out of the box
 - ✅ **Basic TypeScript**: Works with built-in Babel parser  
-- ✅ **97+ Rules**: All quality and security rules available
+- ✅ **256+ Rules**: All quality and security rules available
 - ✅ **Heuristic Engine**: Pattern-based analysis for all languages
 
 ### **Optional Dependencies (Install as needed)**
@@ -98,16 +105,22 @@ npm install --save-dev @sun-asterisk/sunlint eslint @typescript-eslint/parser @t
 # For ESLint engine integration
 npm install eslint --save-dev
 
+# For import/module analysis (recommended with ESLint)
+npm install eslint-plugin-import --save-dev
+
 # For enhanced TypeScript analysis  
 npm install @typescript-eslint/parser @typescript-eslint/eslint-plugin --save-dev
 
 # For TypeScript compiler integration
 npm install typescript --save-dev
+
+# For import/module analysis (recommended)
+npm install eslint-plugin-import --save-dev
 ```
 
 **Quick setup for TypeScript projects:**
 ```bash
-npm install --save-dev @sun-asterisk/sunlint eslint @typescript-eslint/parser @typescript-eslint/eslint-plugin typescript
+npm install --save-dev @sun-asterisk/sunlint eslint @typescript-eslint/parser @typescript-eslint/eslint-plugin eslint-plugin-import typescript
 ```
 
 > 💡 **Note**: SunLint gracefully handles missing dependencies. Install only what your project needs. See [docs/DEPENDENCIES.md](docs/DEPENDENCIES.md) for detailed guidance.
@@ -135,6 +148,7 @@ npm install --save-dev @sun-asterisk/sunlint eslint @typescript-eslint/parser @t
     "eslint": "^8.50.0",
     "@typescript-eslint/parser": "^7.2.0",
     "@typescript-eslint/eslint-plugin": "^7.18.0",
+    "eslint-plugin-import": "^2.32.0",
     "typescript": "^5.0.0"
   }
 }

package/config/engines/engines-enhanced.json

@@ -0,0 +1,86 @@
+{
+  "semantic-engine": {
+    "enabled": true,
+    "description": "TypeScript/JavaScript semantic analysis engine using ts-morph",
+    "features": [
+      "cross-file-analysis",
+      "symbol-table-caching", 
+      "type-checking",
+      "semantic-rule-support"
+    ],
+    "configPath": "config/engines/semantic-config.json",
+    "dependencies": {
+      "required": [],
+      "optional": ["ts-morph"]
+    },
+    "performance": {
+      "enableCaching": true,
+      "maxCacheSize": 100,
+      "memoryLimit": "500MB",
+      "timeout": 30000
+    },
+    "rules": {
+      "classification": {
+        "semantic": ["C047", "C029", "C031", "C048", "C050"],
+        "hybrid": ["C019", "C035"]
+      },
+      "autoDetect": true
+    }
+  },
+  "eslint": {
+    "enabled": true,
+    "description": "ESLint integration for JavaScript/TypeScript linting",
+    "features": [
+      "syntax-checking",
+      "style-enforcement", 
+      "best-practices",
+      "custom-rules"
+    ],
+    "configPath": "config/engines/eslint-config.json",
+    "dependencies": {
+      "required": ["eslint"],
+      "optional": ["@typescript-eslint/parser", "@typescript-eslint/eslint-plugin"]
+    }
+  },
+  "heuristic": {
+    "enabled": true,
+    "description": "Enhanced pattern-based analysis with AST and semantic capabilities",
+    "features": [
+      "pattern-matching",
+      "ast-analysis",
+      "semantic-analysis",
+      "multi-language-support",
+      "rule-classification"
+    ],
+    "configPath": "config/engines/heuristic-config.json",
+    "dependencies": {
+      "required": [],
+      "optional": ["tree-sitter", "ts-morph"]
+    },
+    "modes": {
+      "traditional": {
+        "description": "Pattern + AST analysis only",
+        "semantic": false
+      },
+      "semantic": {
+        "description": "Pattern + AST + semantic analysis",
+        "semantic": true,
+        "requires": ["semantic-engine"]
+      }
+    }
+  },
+  "openai": {
+    "enabled": true,
+    "description": "AI-powered code analysis using OpenAI API",
+    "features": [
+      "context-aware-analysis",
+      "natural-language-descriptions",
+      "intelligent-suggestions"
+    ],
+    "configPath": "config/engines/openai-config.json",
+    "dependencies": {
+      "required": ["openai"],
+      "optional": []
+    }
+  }
+}

package/config/engines/semantic-config.json

@@ -0,0 +1,114 @@
+{
+  "semantic-engine": {
+    "compilerOptions": {
+      "target": "ES2020",
+      "module": "commonjs",
+      "lib": ["ES2020", "DOM"],
+      "allowJs": true,
+      "checkJs": false,
+      "skipLibCheck": true,
+      "skipDefaultLibCheck": true,
+      "moduleResolution": "node",
+      "esModuleInterop": true,
+      "allowSyntheticDefaultImports": true,
+      "strict": false,
+      "noImplicitAny": false
+    },
+    
+    "performance": {
+      "enableCaching": true,
+      "maxCacheSize": 100,
+      "memoryLimit": 524288000,
+      "timeout": 30000,
+      "crossFileAnalysis": true,
+      "enableTypeChecker": false
+    },
+    
+    "fileDiscovery": {
+      "patterns": [
+        "**/*.ts",
+        "**/*.tsx", 
+        "**/*.js",
+        "**/*.jsx"
+      ],
+      "excludePatterns": [
+        "**/node_modules/**",
+        "**/dist/**",
+        "**/build/**",
+        "**/.git/**",
+        "**/coverage/**",
+        "**/*.test.{ts,tsx,js,jsx}",
+        "**/*.spec.{ts,tsx,js,jsx}"
+      ],
+      "maxFiles": 1000
+    },
+    
+    "symbolTable": {
+      "extractImports": true,
+      "extractExports": true,
+      "extractFunctions": true,
+      "extractClasses": true,
+      "extractInterfaces": true,
+      "extractVariables": true,
+      "extractConstants": true,
+      "extractHooks": true,
+      "extractComponents": true,
+      "extractFunctionCalls": true,
+      "extractMethodCalls": true,
+      "crossFileReferences": true
+    },
+    
+    "rules": {
+      "C047": {
+        "enabled": true,
+        "crossFileAnalysis": true,
+        "retryPatterns": [
+          "retry",
+          "retries", 
+          "withRetry",
+          "retryWhen",
+          "attempt"
+        ],
+        "queryHooks": [
+          "useQuery",
+          "useMutation", 
+          "useInfiniteQuery",
+          "useSuspenseQuery"
+        ],
+        "contextAnalysis": {
+          "nearbyLines": 10,
+          "parentContext": true
+        }
+      },
+      
+      "C029": {
+        "enabled": false,
+        "crossFileAnalysis": true,
+        "description": "Future semantic rule for component lifecycle"
+      },
+      
+      "C031": {
+        "enabled": false,
+        "crossFileAnalysis": true, 
+        "description": "Future semantic rule for state management"
+      }
+    },
+    
+    "optimization": {
+      "incrementalAnalysis": true,
+      "watchMode": false,
+      "parallelProcessing": false,
+      "memoryCleanup": {
+        "interval": 60000,
+        "threshold": 0.8
+      }
+    },
+    
+    "debugging": {
+      "enableVerboseLogging": false,
+      "logSymbolTable": false,
+      "logPerformanceMetrics": true,
+      "outputStatsFile": false
+    }
+  }
+}

package/config/eslint-rule-mapping.json

@@ -3,8 +3,8 @@
   "version": "1.0.0",
   "stats": {
     "commonRulesTotal": 76,
-    "commonRulesImplemented": 19,
-    "commonRulesMissing": 57,
+    "commonRulesImplemented": 27,
+    "commonRulesMissing": 49,
     "securityRulesTotal": 59, 
     "securityRulesImplemented": 43,
     "securityRulesMissing": 16,
@@ -12,46 +12,58 @@
     "typescriptRulesImplemented": 8
   },
   "missingCommonRules": [
-    "C001", "C004", "C005", "C007", "C008", "C009", "C011", "C012", "C015", "C016", 
-    "C019", "C020", "C021", "C022", "C024", "C025", "C026", "C027", "C028", "C031",
-    "C032", "C033", "C034", "C036", "C037", "C038", "C039", "C040", "C044", "C045",
-    "C046", "C048", "C049", "C050", "C051", "C052", "C053", "C054", "C055", "C056",
-    "C057", "C058", "C059", "C060", "C061", "C062", "C063", "C064", "C065", "C066",
-    "C067", "C068", "C069", "C070", "C071", "C073", "C074"
+    "C001", "C004", "C008", "C009", "C011", "C016", 
+    "C020", "C021", "C022", "C024", "C025", "C026", "C027", "C028", "C036",
+    "C039", "C044", "C045", "C046", "C048", "C049", "C050", "C051", "C052", 
+    "C053", "C054", "C055", "C056", "C057", "C058", "C059", "C060", "C061", 
+    "C062", "C063", "C064", "C065", "C066", "C067", "C068", "C069", "C070", 
+    "C071", "C073", "C074"
   ],
   "missingSecurityRules": [
     "S004", "S021", "S024", "S028", "S031", "S032", "S040", "S049", "S051", "S053", 
     "S056", "S059", "S060", "S061", "S062", "S063"
   ],
   "mappings": {
-    "C002": ["custom/no-duplicate-code"],
-    "C003": ["custom/no-vague-abbreviations"],
-    "C006": ["custom/function-name-verb-noun"],
-    "C010": ["custom/limit-block-nesting"],
-    "C013": ["custom/no-dead-code"],
-    "C014": ["custom/abstract-dependency-preferred"],
-    "C017": ["custom/limit-constructor-logic"],
-    "C018": ["custom/no-generic-throw"],
-    "C023": ["custom/no-duplicate-variable-name-in-scope"],
-    "C029": ["custom/catch-block-logging"],
-    "C030": ["custom/use-custom-error-classes"],
-    "C035": ["custom/no-empty-catch"],
-    "C041": ["custom/no-config-inline"],
-    "C042": ["custom/boolean-name-prefix"],
-    "C043": ["custom/no-console-or-print"],
-    "C047": ["custom/no-duplicate-retry-logic"],
-    "C072": ["custom/one-assert-per-test"],
-    "C075": ["custom/explicit-function-return-types"],
-    "C076": ["custom/single-behavior-per-test"],
+    "C002": ["custom/c002-no-duplicate-code"],
+    "C003": ["custom/c003-no-vague-abbreviations"],
+    "C005": ["max-statements-per-line", "complexity"],
+    "C006": ["custom/c006-function-name-verb-noun"],
+    "C007": ["spaced-comment", "no-inline-comments", "no-warning-comments"],
+    "C010": ["custom/c010-limit-block-nesting"],
+    "C012": ["consistent-return", "no-void", "@typescript-eslint/no-confusing-void-expression"],
+    "C013": ["custom/c013-no-dead-code"],
+    "C014": ["custom/c014-abstract-dependency-preferred"],
+    "C015": ["@typescript-eslint/naming-convention", "camelcase"],
+    "C017": ["custom/c017-limit-constructor-logic"],
+    "C018": ["custom/c018-no-generic-throw"],
+    "C019": ["no-console", "no-alert", "no-debugger"],
+    "C023": ["custom/c023-no-duplicate-variable-name-in-scope"],
+    "C029": ["custom/c029-catch-block-logging"],
+    "C030": ["custom/c030-use-custom-error-classes"],
+    "C031": ["no-implicit-coercion", "eqeqeq"],
+    "C032": ["no-new", "no-constructor-return"],
+    "C033": ["prefer-const", "no-var"],
+    "C034": ["no-global-assign", "no-implicit-globals", "@typescript-eslint/no-namespace"],
+    "C035": ["custom/c035-no-empty-catch"],
+    "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"],
+    "C041": ["custom/c041-no-config-inline"],
+    "C042": ["custom/c042-boolean-name-prefix"],
+    "C043": ["custom/c043-no-console-or-print"],
+    "C047": ["custom/c047-no-duplicate-retry-logic"],
+    "C072": ["custom/c072-one-assert-per-test"],
+    "C075": ["custom/c075-explicit-function-return-types"],
+    "C076": ["custom/c076-single-behavior-per-test"],
     
-    "T002": ["custom/interface-prefix-i"],
-    "T003": ["custom/ts-ignore-reason"],
-    "T004": ["custom/no-empty-type"],
-    "T007": ["custom/no-fn-in-constructor"],
-    "T010": ["custom/no-nested-union-tuple"],
-    "T019": ["custom/no-this-assign"],
-    "T020": ["custom/no-default-multi-export"],
-    "T021": ["custom/limit-nested-generics"],
+    "T002": ["custom/t002-interface-prefix-i"],
+    "T003": ["custom/t003-ts-ignore-reason"],
+    "T004": ["custom/t004-no-empty-type"],
+    "T007": ["custom/t007-no-fn-in-constructor"],
+    "T010": ["custom/t010-no-nested-union-tuple"],
+    "T019": ["custom/t019-no-this-assign"],
+    "T020": ["custom/t020-no-default-multi-export"],
+    "T021": ["custom/t021-limit-nested-generics"],
     
     "S001": ["custom/s001-fail-securely"],
     "S002": ["custom/s002-idor-check"],
@@ -114,9 +126,9 @@
   
   "implementationTodo": {
     "commonRules": {
-      "priority1": ["C001", "C004", "C005", "C007", "C008", "C009", "C011", "C012"],
-      "priority2": ["C015", "C016", "C019", "C020", "C021", "C022", "C024", "C025"],  
-      "priority3": ["C026", "C027", "C028", "C031", "C032", "C033", "C034", "C036"]
+      "priority1": ["C001", "C004", "C008", "C009", "C011"],
+      "priority2": ["C016", "C020", "C021", "C022", "C024", "C025"],  
+      "priority3": ["C026", "C027", "C028", "C036", "C039", "C044", "C045", "C046"]
     },
     "securityRules": {
       "needImplementation": ["S004", "S021", "S024", "S028", "S031", "S032", "S040"],

package/config/rule-analysis-strategies.js

@@ -12,9 +12,9 @@ module.exports = {
       accuracy: { ast: 95, regex: 75 }
     },
     'C012': {
-      reason: 'Cyclomatic complexity needs control flow analysis',
+      reason: 'Command Query Separation requires function behavior analysis',
       methods: ['ast', 'regex'],
-      accuracy: { ast: 90, regex: 60 }
+      accuracy: { ast: 95, regex: 80 }
     },
     'C015': {
       reason: 'Function parameter counting benefits from AST',
@@ -25,6 +25,16 @@ module.exports = {
       reason: 'Constructor logic analysis needs AST context',
       methods: ['ast', 'regex'],
       accuracy: { ast: 90, regex: 70 }
+    },
+    'S015': {
+      reason: 'TLS certificate validation requires AST context analysis',
+      methods: ['ast', 'regex'],
+      accuracy: { ast: 95, regex: 80 }
+    },
+    'S023': {
+      reason: 'JSON injection detection requires AST context analysis',
+      methods: ['ast', 'regex'],
+      accuracy: { ast: 95, regex: 60 }
     }
   },
 
@@ -55,6 +65,12 @@ module.exports = {
       strategy: 'ast-primary-regex-fallback',
       accuracy: { ast: 90, regex: 75, combined: 95 }
     },
+    'C041': {
+      reason: 'Hardcoded secrets need AST literal analysis like ESLint',
+      methods: ['ast', 'regex'],
+      strategy: 'ast-primary-regex-fallback',
+      accuracy: { ast: 95, regex: 70, combined: 95 }
+    },
     'C047': {
       reason: 'Retry logic detection needs pattern + structure',
       methods: ['regex', 'ast'],