perf(linter): remove unnecessary should_run check (#13639)

camchenry · camchenry · commit e6a25e78d5ac · 2025-09-10T04:25:48.000Z
`rules` was already filtered with `should_run`, so this check doesn't do anything.
diff --git a/crates/oxc_linter/src/lib.rs b/crates/oxc_linter/src/lib.rs
@@ -191,14 +191,12 @@ impl Linter {
                     let rule = *rule;
                     // Collect node type information for rules. In large files, benchmarking showed it was worth
                     // collecting rules into buckets by AST node type to avoid iterating over all rules for each node.
-                    if rule.should_run(&ctx_host) {
-                        if let Some(ast_types) = rule.types_info() {
-                            for ty in ast_types {
-                                rules_by_ast_type[ty as usize].push((rule, ctx));
-                            }
-                        } else {
-                            rules_any_ast_type.push((rule, ctx));
+                    if let Some(ast_types) = rule.types_info() {
+                        for ty in ast_types {
+                            rules_by_ast_type[ty as usize].push((rule, ctx));
                         }
+                    } else {
+                        rules_any_ast_type.push((rule, ctx));
                     }
 
                     rule.run_once(ctx);
