fix(linter): drop rules to allow mutable access to ctx_host in run_external_rules (#13832)

camc314 · camc314 · commit a2c91cd5104a · 2025-09-16T18:49:53.000Z
diff --git a/crates/oxc_linter/src/lib.rs b/crates/oxc_linter/src/lib.rs
@@ -288,6 +288,10 @@ impl Linter {
                 });
             }
 
+            // Drop `rules` to release its `Rc` clones of `ctx_host`, ensuring `run_external_rules`
+            // can mutably access `ctx_host` via `Rc::get_mut` without panicking due to multiple references.
+            drop(rules);
+
             self.run_external_rules(&external_rules, path, &mut ctx_host, allocator);
 
             if let Some(severity) = self.options.report_unused_directive {

Original file line number	Diff line number	Diff line change
`@@ -288,6 +288,10 @@ impl Linter {`
`288`	`288`	`});`
`289`	`289`	`}`
`290`	`290`
	`291`	+ // Drop `rules` to release its `Rc` clones of `ctx_host`, ensuring `run_external_rules`
	`292`	+ // can mutably access `ctx_host` via `Rc::get_mut` without panicking due to multiple references.
	`293`	`+ drop(rules);`
	`294`	`+`
`291`	`295`	`self.run_external_rules(&external_rules, path, &mut ctx_host, allocator);`
`292`	`296`
`293`	`297`	`if let Some(severity) = self.options.report_unused_directive {`