docs: keep rule review decisions local

docs/00 INBOX/ai/nix-statix-deadnix-rules.md

@@ -17,7 +17,7 @@ Use the `Decision` line on each rule for `keep`, `discard`, `update`, or `need-m
 
 ### `bool_comparison`
 
-Decision: discard
+Decision: keep
 
 Source: `statix` `W01`.
 Status at collection time: enabled.
@@ -50,7 +50,7 @@ Direct predicates make conditions easier to scan.
 
 ### `empty_let_in`
 
-Decision: discard
+Decision: keep
 
 Source: `statix` `W02`.
 Status at collection time: enabled.
@@ -80,7 +80,7 @@ It usually remains after editing or debugging.
 
 ### `manual_inherit`
 
-Decision: keep
+Decision: Keep
 
 Source: `statix` `W03`.
 Status at collection time: enabled.
@@ -113,7 +113,7 @@ It removes duplicated names.
 
 ### `manual_inherit_from`
 
-Decision: keep
+Decision: 
 
 Source: `statix` `W04`.
 Status at collection time: enabled.
@@ -142,7 +142,7 @@ It avoids repeated selector names.
 
 ### `legacy_let_syntax`
 
-Decision: discard
+Decision: keep
 
 Source: `statix` `W05`.
 Status at collection time: enabled.
@@ -180,7 +180,7 @@ Documented forms are clearer to tools and readers.
 
 ### `collapsible_let_in`
 
-Decision: discard
+Decision: keep
 
 Source: `statix` `W06`.
 Status at collection time: enabled.
@@ -220,7 +220,7 @@ A single binding group shows the local dependencies together.
 
 ### `eta_reduction`
 
-Decision: discard
+Decision: keep
 
 Source: `statix` `W07`.
 Status at collection time: enabled.
@@ -250,7 +250,7 @@ The direct function reference exposes the operation.
 
 ### `useless_parens`
 
-Decision: discard
+Decision: keep
 
 Source: `statix` `W08`.
 Status at collection time: enabled.
@@ -285,7 +285,7 @@ Required precedence parentheses remain valid.
 
 ### `empty_pattern`
 
-Decision: discard
+Decision: keep
 
 Source: `statix` `W10`.
 Status at collection time: enabled.
@@ -315,7 +315,7 @@ An empty variadic pattern looks like partial destructuring.
 
 ### `redundant_pattern_bind`
 
-Decision: discard
+Decision: keep
 
 Source: `statix` `W11`.
 Status at collection time: enabled.
@@ -375,7 +375,7 @@ They avoid ambiguity with punctuation after the URI.
 
 ### `empty_inherit`
 
-Decision: discard
+Decision: keep
 
 Source: `statix` `W14`.
 Status at collection time: enabled.
@@ -400,7 +400,7 @@ It usually remains after a refactor.
 
 ### `deprecated_to_path`
 
-Decision: discard
+Decision: keep
 
 Source: `statix` `W17`.
 Status at collection time: enabled.
@@ -431,7 +431,7 @@ Explicit path construction states whether the result is absolute or relative.
 
 ### `bool_simplification`
 
-Decision: discard
+Decision: keep
 
 Source: `statix` `W18`.
 Status at collection time: enabled.
@@ -459,7 +459,7 @@ The direct operator expresses inequality without nested boolean syntax.
 
 ### `useless_has_attr`
 
-Decision: discard
+Decision: keep
 
 Source: `statix` `W19`.
 Status at collection time: enabled.
@@ -488,7 +488,7 @@ It avoids repeating the attribute path.
 
 ### `repeated_keys`
 
-Decision: discard
+Decision: update so that it doesn't conflict with the custom module convention.
 
 Source: `statix` `W20`.
 Status at collection time: disabled in `statix.toml`.
@@ -529,7 +529,7 @@ The local config disables this rule, so it needs an explicit decision before pro
 
 ### `empty_list_concat`
 
-Decision: discard
+Decision: keep
 
 Source: `statix` `W23`.
 Status at collection time: enabled.
@@ -561,7 +561,7 @@ It usually remains after conditional list construction changed.
 
 ### Unused `let` bindings
 
-Decision: discard
+Decision: keep
 
 Source: `deadnix` default check.
 Status at collection time: active in `nix run .#fix`.
@@ -595,7 +595,7 @@ They also make refactors look unfinished.
 
 ### Unused `inherit` names
 
-Decision: discard
+Decision: keep
 
 Source: `deadnix` default check.
 Status at collection time: active in `nix run .#fix`.
@@ -628,7 +628,7 @@ An empty result also overlaps with `statix` `empty_inherit`.
 
 ### Unused lambda arguments
 
-Decision: discard
+Decision: keep
 
 Source: `deadnix` default check.
 Status at collection time: active in `nix run .#fix`.
@@ -658,7 +658,7 @@ A named unused argument implies missing logic.
 
 ### Unused lambda pattern names
 
-Decision: discard
+Decision: keep
 
 Source: `deadnix` default check.
 Status at collection time: active in `nix run .#fix`.
@@ -689,7 +689,7 @@ Unused names overstate the function's inputs.
 
 ### Unused lambda pattern aliases
 
-Decision: discard
+Decision: keep
 
 Source: `deadnix` default check.
 Status at collection time: active in `nix run .#fix`.
@@ -719,7 +719,7 @@ Keeping it unused makes the function contract look wider than it is.
 
 ### Ignored underscore bindings
 
-Decision: discard
+Decision: keep
 
 Source: `deadnix` `--no-underscore` option.
 Status at collection time: inactive; current fixer does not pass `--no-underscore`.
@@ -748,7 +748,7 @@ Skipping all underscore bindings can hide real dead code.
 
 ### Used underscore bindings
 
-Decision: discard
+Decision: keep
 
 Source: `deadnix` `--warn-used-underscore` option.
 Status at collection time: inactive; current fixer does not pass `--warn-used-underscore`.