@sun-asterisk/sunlint 1.3.39 → 1.3.41

package/config/rules/rules-registry-generated.json

@@ -1848,9 +1848,7 @@
         "code-quality"
       ],
       "tools": [
-        "`dart lint` (flutter_lints",
-        "very_good_analysis",
-        "lints)"
+        "Custom analyzer (D001)"
       ],
       "framework": "All",
       "principles": [
@@ -1858,8 +1856,8 @@
       ]
     },
     "D002": {
-      "name": "Avoid using single cascade (..) operators",
-      "description": "Write clear, readable code",
+      "name": "Always Dispose Resources and Remove Listeners",
+      "description": "Prevent memory leaks by ensuring proper resource disposal",
       "category": "Common",
       "severity": "major",
       "languages": [
@@ -1873,18 +1871,19 @@
         "code-quality"
       ],
       "tools": [
-        "`dart lint` (`avoid_single_cascade_in_expression_statements`)"
+        "Custom analyzer (D002)"
       ],
       "framework": "All",
       "principles": [
-        "CODE_QUALITY"
+        "CODE_QUALITY",
+        "PERFORMANCE"
       ]
     },
     "D003": {
-      "name": "Avoid calling methods/accessing properties on dynamic types",
-      "description": "Prevent runtime errors due to lack of type checking",
+      "name": "Prefer Widgets Over Methods Returning Widgets",
+      "description": "Improve performance and maintainability by extracting widget-returning methods into widget classes",
       "category": "Common",
-      "severity": "critical",
+      "severity": "major",
       "languages": [
         "dart"
       ],
@@ -1896,17 +1895,17 @@
         "code-quality"
       ],
       "tools": [
-        "`dart lint` (`avoid_dynamic_calls`)"
+        "Custom analyzer (D003)"
       ],
       "framework": "All",
       "principles": [
         "CODE_QUALITY",
-        "SECURITY"
+        "PERFORMANCE"
       ]
     },
     "D004": {
-      "name": "Use standard `package:` imports",
-      "description": "Reduce confusion in imports",
+      "name": "Avoid shrinkWrap in ListView",
+      "description": "Prevent performance issues caused by shrinkWrap in scrollable widgets",
       "category": "Common",
       "severity": "major",
       "languages": [
@@ -1920,16 +1919,17 @@
         "code-quality"
       ],
       "tools": [
-        "`dart lint` (`always_use_package_imports`)"
+        "Custom analyzer (D004)"
       ],
       "framework": "All",
       "principles": [
-        "CODE_QUALITY"
+        "CODE_QUALITY",
+        "PERFORMANCE"
       ]
     },
     "D005": {
-      "name": "Always declare function return types",
-      "description": "Clarify logic and increase reliability in type checking",
+      "name": "Limit Widget Nesting Depth to 6",
+      "description": "Maintain code readability and prevent performance issues caused by deeply nested widgets",
       "category": "Common",
       "severity": "major",
       "languages": [
@@ -1943,18 +1943,20 @@
         "code-quality"
       ],
       "tools": [
-        "`dart lint` (`always_declare_return_types`)"
+        "Custom analyzer (D005)"
       ],
       "framework": "All",
       "principles": [
-        "CODE_QUALITY"
+        "CODE_QUALITY",
+        "MAINTAINABILITY",
+        "PERFORMANCE"
       ]
     },
     "D006": {
-      "name": "Do not override `==` and `hashCode` in mutable classes",
-      "description": "Prevent logic errors when using mutable objects in collections.",
+      "name": "Prefer Extracting Large Callbacks from Build",
+      "description": "Improve code readability and testability by extracting large callback functions",
       "category": "Common",
-      "severity": "critical",
+      "severity": "major",
       "languages": [
         "dart"
       ],
@@ -1966,16 +1968,18 @@
         "code-quality"
       ],
       "tools": [
-        "`dart lint` (`avoid_equals_and_hash_code_on_mutable_classes`)"
+        "Custom analyzer (D006)"
       ],
       "framework": "All",
       "principles": [
-        "CODE_QUALITY"
+        "CODE_QUALITY",
+        "MAINTAINABILITY",
+        "TESTABILITY"
       ]
     },
     "D007": {
-      "name": "Do not pass default values when calling functions",
-      "description": "Avoid redundancy and clarify intent",
+      "name": "Prefer Init First, Dispose Last",
+      "description": "Ensure proper lifecycle method ordering in StatefulWidget",
       "category": "Common",
       "severity": "major",
       "languages": [
@@ -1989,17 +1993,18 @@
         "code-quality"
       ],
       "tools": [
-        "`dart lint` (`avoid_redundant_argument_values`)"
+        "Custom analyzer (D007)"
       ],
       "framework": "All",
       "principles": [
         "CODE_QUALITY",
-        "PERFORMANCE"
+        "MAINTAINABILITY",
+        "LIFECYCLE_MANAGEMENT"
       ]
     },
     "D008": {
-      "name": "Avoid slow async functions in `dart:io`",
-      "description": "Optimize I/O performance",
+      "name": "Avoid Long Functions",
+      "description": "Improve code readability and maintainability by limiting function length",
       "category": "Common",
       "severity": "major",
       "languages": [
@@ -2013,19 +2018,20 @@
         "code-quality"
       ],
       "tools": [
-        "`dart lint` (`avoid_slow_async_io`)"
+        "Custom analyzer (D008)"
       ],
       "framework": "All",
       "principles": [
         "CODE_QUALITY",
-        "PERFORMANCE"
+        "MAINTAINABILITY",
+        "READABILITY"
       ]
     },
     "D009": {
-      "name": "Do not use throw or control flow in `finally`",
-      "description": "Avoid unexpected behavior",
+      "name": "Limit Function Parameters",
+      "description": "Improve code readability by limiting the number of function parameters",
       "category": "Common",
-      "severity": "critical",
+      "severity": "major",
       "languages": [
         "dart"
       ],
@@ -2037,17 +2043,18 @@
         "code-quality"
       ],
       "tools": [
-        "`dart lint` (`control_flow_in_finally`",
-        "`throw_in_finally`)"
+        "Custom analyzer (D009)"
       ],
       "framework": "All",
       "principles": [
-        "CODE_QUALITY"
+        "CODE_QUALITY",
+        "MAINTAINABILITY",
+        "READABILITY"
       ]
     },
     "D010": {
-      "name": "Handle all cases when using `switch` with enums or enum-like classes",
-      "description": "Avoid missing cases",
+      "name": "Limit Cyclomatic Complexity",
+      "description": "Improve code readability and maintainability by limiting cyclomatic complexity",
       "category": "Common",
       "severity": "major",
       "languages": [
@@ -2061,16 +2068,18 @@
         "code-quality"
       ],
       "tools": [
-        "`dart lint` (`exhaustive_cases`)"
+        "Custom analyzer (D010)"
       ],
       "framework": "All",
       "principles": [
-        "CODE_QUALITY"
+        "CODE_QUALITY",
+        "MAINTAINABILITY",
+        "READABILITY"
       ]
     },
     "D011": {
-      "name": "Avoid importing `.dart` files from `lib/src` of other packages",
-      "description": "Avoid unstable dependencies that cause breaking changes.",
+      "name": "Prefer Named Parameters",
+      "description": "Improve code readability and prevent parameter confusion",
       "category": "Common",
       "severity": "major",
       "languages": [
@@ -2084,17 +2093,18 @@
         "code-quality"
       ],
       "tools": [
-        "`dart lint` (`implementation_imports`)"
+        "Custom analyzer (D011)"
       ],
       "framework": "All",
       "principles": [
         "CODE_QUALITY",
-        "SECURITY"
+        "READABILITY",
+        "MAINTAINABILITY"
       ]
     },
     "D012": {
-      "name": "Avoid passing null to closure parameters",
-      "description": "Prevent runtime exceptions",
+      "name": "Prefer Named Boolean Parameters",
+      "description": "Improve code readability by avoiding unclear boolean parameters",
       "category": "Common",
       "severity": "major",
       "languages": [
@@ -2108,17 +2118,18 @@
         "code-quality"
       ],
       "tools": [
-        "`dart lint` (`null_closures`)"
+        "Custom analyzer (D012)"
       ],
       "framework": "All",
       "principles": [
         "CODE_QUALITY",
-        "SECURITY"
+        "READABILITY",
+        "MAINTAINABILITY"
       ]
     },
     "D013": {
-      "name": "Use adjacent strings or interpolation to create strings",
-      "description": "Easier to read and more efficient",
+      "name": "Prefer a Single Public Class Per File",
+      "description": "Improve code organization and maintainability",
       "category": "Common",
       "severity": "major",
       "languages": [
@@ -2132,18 +2143,18 @@
         "code-quality"
       ],
       "tools": [
-        "`dart lint` (`prefer_adjacent_string_concatenation`",
-        "`prefer_interpolation_to_compose_strings`)"
+        "Custom analyzer (D013)"
       ],
       "framework": "All",
       "principles": [
         "CODE_QUALITY",
-        "PERFORMANCE"
+        "ORGANIZATION",
+        "MAINTAINABILITY"
       ]
     },
     "D014": {
-      "name": "Use conditional assignment `??=` instead of `if-null-then-assign`",
-      "description": "More concise and clear meaning",
+      "name": "Avoid Unsafe Collection Access",
+      "description": "Prevent runtime errors from accessing empty collections",
       "category": "Common",
       "severity": "major",
       "languages": [
@@ -2157,17 +2168,18 @@
         "code-quality"
       ],
       "tools": [
-        "`dart lint` (`prefer_conditional_assignment`)"
+        "Custom analyzer (D014)"
       ],
       "framework": "All",
       "principles": [
         "CODE_QUALITY",
-        "MAINTAINABILITY"
+        "SAFETY",
+        "ERROR_PREVENTION"
       ]
     },
     "D015": {
-      "name": "Use `final`, `const` for immutable variables",
-      "description": "Prevent bugs from unintended value changes",
+      "name": "Ensure copyWith includes all constructor parameters",
+      "description": "Maintain data integrity and completeness in immutable objects",
       "category": "Common",
       "severity": "major",
       "languages": [
@@ -2181,18 +2193,18 @@
         "code-quality"
       ],
       "tools": [
-        "`dart lint` (`prefer_final_fields`",
-        "`prefer_const_declarations`",
-        "`prefer_const_constructors`)"
+        "Custom analyzer (D015)"
       ],
       "framework": "All",
       "principles": [
-        "CODE_QUALITY"
+        "CODE_QUALITY",
+        "IMMUTABILITY",
+        "DATA_INTEGRITY"
       ]
     },
     "D016": {
-      "name": "Use explicit definitions for function types in parameters",
-      "description": "Increase clarity and accurate type checking",
+      "name": "Project should have tests",
+      "description": "Ensure code quality and prevent regressions through automated testing",
       "category": "Common",
       "severity": "major",
       "languages": [
@@ -2206,16 +2218,18 @@
         "code-quality"
       ],
       "tools": [
-        "`dart lint` (`use_function_type_syntax_for_parameters`)"
+        "Custom analyzer (D016)"
       ],
       "framework": "All",
       "principles": [
-        "CODE_QUALITY"
+        "CODE_QUALITY",
+        "TESTING",
+        "MAINTAINABILITY"
       ]
     },
     "D017": {
-      "name": "Ensure simple and correct Regex syntax",
-      "description": "Prevent logic errors from invalid expressions",
+      "name": "Pubspec dependencies should be reviewed regularly",
+      "description": "Ensure dependencies are kept up-to-date for security and stability",
       "category": "Common",
       "severity": "major",
       "languages": [
@@ -2229,16 +2243,18 @@
         "code-quality"
       ],
       "tools": [
-        "`dart lint` (`valid_regexps`)"
+        "Custom analyzer (D017)"
       ],
       "framework": "All",
       "principles": [
+        "SECURITY",
+        "MAINTAINABILITY",
         "CODE_QUALITY"
       ]
     },
     "D018": {
-      "name": "Use `rethrow` instead of `throw` when re-throwing errors",
-      "description": "Preserve original error stack trace",
+      "name": "Remove Commented-Out Code",
+      "description": "Keep codebase clean by removing commented-out code",
       "category": "Common",
       "severity": "major",
       "languages": [
@@ -2252,16 +2268,18 @@
         "code-quality"
       ],
       "tools": [
-        "`dart lint` (`use_rethrow_when_possible`)"
+        "Custom analyzer (D018)"
       ],
       "framework": "All",
       "principles": [
-        "CODE_QUALITY"
+        "CODE_QUALITY",
+        "MAINTAINABILITY",
+        "READABILITY"
       ]
     },
     "D019": {
-      "name": "Use `isEmpty` / `isNotEmpty` for String, Iterable and Map",
-      "description": "Clear meaning and better performance",
+      "name": "Avoid Single Child in Multi-Child Widget",
+      "description": "Use appropriate widget types for the number of children",
       "category": "Common",
       "severity": "major",
       "languages": [
@@ -2275,17 +2293,18 @@
         "code-quality"
       ],
       "tools": [
-        "`dart lint` (`prefer_is_empty`",
-        "`prefer_is_not_empty`)"
+        "Custom analyzer (D019)"
       ],
       "framework": "All",
       "principles": [
-        "CODE_QUALITY"
+        "CODE_QUALITY",
+        "PERFORMANCE",
+        "MAINTAINABILITY"
       ]
     },
     "D020": {
-      "name": "Ensure valid URLs in `pubspec.yaml`",
-      "description": "Avoid metadata errors and poor security",
+      "name": "Limit If/Else Branches",
+      "description": "Reduce complexity by limiting the number of if/else branches",
       "category": "Common",
       "severity": "major",
       "languages": [
@@ -2299,18 +2318,20 @@
         "code-quality"
       ],
       "tools": [
-        "`dart lint` (`secure_pubspec_urls`)"
+        "Custom analyzer (D020)"
       ],
       "framework": "All",
       "principles": [
-        "SECURITY"
+        "CODE_QUALITY",
+        "MAINTAINABILITY",
+        "READABILITY"
       ]
     },
     "D021": {
-      "name": "Use `BuildContext` synchronously",
-      "description": "Prevent errors when `context` changes after `await`",
+      "name": "Avoid Negated Boolean Checks",
+      "description": "Improve code readability by avoiding inverted or negated boolean conditions",
       "category": "Common",
-      "severity": "critical",
+      "severity": "major",
       "languages": [
         "dart"
       ],
@@ -2322,16 +2343,17 @@
         "code-quality"
       ],
       "tools": [
-        "`flutter_lints` (`use_build_context_synchronously`)"
+        "Custom analyzer (D021)"
       ],
       "framework": "All",
       "principles": [
-        "CODE_QUALITY"
+        "CODE_QUALITY",
+        "READABILITY"
       ]
     },
     "D022": {
-      "name": "Place `child:` at the end when constructing widgets",
-      "description": "Help readability of widget tree and UI structure",
+      "name": "Use setState Correctly",
+      "description": "Ensure setState is used correctly in StatefulWidget to avoid performance issues and bugs",
       "category": "Common",
       "severity": "major",
       "languages": [
@@ -2345,16 +2367,18 @@
         "code-quality"
       ],
       "tools": [
-        "`flutter_lints` (`sort_child_properties_last`)"
+        "Custom analyzer (D022)"
       ],
       "framework": "All",
       "principles": [
-        "CODE_QUALITY"
+        "CODE_QUALITY",
+        "PERFORMANCE",
+        "BEST_PRACTICES"
       ]
     },
     "D023": {
-      "name": "Prefer using `contains` for `List` and `String`",
-      "description": "Easier to read and more efficient",
+      "name": "Avoid Unnecessary Method Overrides",
+      "description": "Remove methods that only call super with the same parameters as they add no value",
       "category": "Common",
       "severity": "major",
       "languages": [
@@ -2368,17 +2392,18 @@
         "code-quality"
       ],
       "tools": [
-        "`dart lint` (`prefer_contains`)"
+        "Custom analyzer (D023)"
       ],
       "framework": "All",
       "principles": [
         "CODE_QUALITY",
-        "SECURITY"
+        "MAINTAINABILITY",
+        "READABILITY"
       ]
     },
     "D024": {
-      "name": "Use `??` to convert `null` to `bool`",
-      "description": "Write concisely and avoid null exceptions",
+      "name": "Avoid Unnecessary StatefulWidget",
+      "description": "Use StatelessWidget when no state management is needed to improve performance",
       "category": "Common",
       "severity": "major",
       "languages": [
@@ -2392,17 +2417,18 @@
         "code-quality"
       ],
       "tools": [
-        "`dart lint` (`use_if_null_to_convert_nulls_to_bools`)"
+        "Custom analyzer (D024)"
       ],
       "framework": "All",
       "principles": [
         "CODE_QUALITY",
-        "SECURITY"
+        "PERFORMANCE",
+        "BEST_PRACTICES"
       ]
     },
     "D025": {
-      "name": "Include `Key` in Widget constructors",
-      "description": "Help Flutter identify widgets, ensure efficient rebuilds and prevent errors when reordering widgets.",
+      "name": "Avoid Nested Conditional Expressions",
+      "description": "Improve code readability by avoiding nested ternary operators",
       "category": "Common",
       "severity": "major",
       "languages": [
@@ -2416,13 +2442,13 @@
         "code-quality"
       ],
       "tools": [
-        "`flutter_lints` (`use_key_in_widget_constructors`)"
+        "Custom analyzer (D025)"
       ],
       "framework": "All",
       "principles": [
         "CODE_QUALITY",
-        "USABILITY",
-        "PERFORMANCE"
+        "READABILITY",
+        "MAINTAINABILITY"
       ]
     },
     "J001": {

package/core/rule-selection-service.js

@@ -126,6 +126,17 @@ class RuleSelectionService {
       }
     }
 
+    // Filter out disabled rules from config (rules set to 'off', false, or 0)
+    const disabledRules = new Set(
+      Object.entries(allRules)
+        .filter(([, value]) => value === 'off' || value === false || value === 0)
+        .map(([ruleId]) => ruleId)
+    );
+
+    if (disabledRules.size > 0) {
+      selectedRules = selectedRules.filter(ruleId => !disabledRules.has(ruleId));
+    }
+
     // Convert to rule objects
     return selectedRules.map(ruleId => {
       const adapterRule = this.ruleAdapter.getRuleById(ruleId);