refactor(linter): fix dead code warnings when compiling napi/oxlint2 (#13132)

overlookmotel · overlookmotel · commit 0dd7908fe6da · 2025-08-15T23:39:16.000Z
`MessageCloner` (introduced in #13032) is only used when `language_server` feature is enabled, or in tests. Feature-gate this code, to avoid "dead code" warnings when compiling `napi/oxlint2`.
diff --git a/crates/oxc_linter/src/service/runtime.rs b/crates/oxc_linter/src/service/runtime.rs
@@ -184,6 +184,7 @@ impl RuntimeFileSystem for OsFileSystem {
 ///
 /// This type is wrapped in a module so that other code cannot access the inner `UnsafeAllocatorRef`
 /// directly, and must go via the [`MessageCloner::clone_message`] method.
+#[cfg(any(feature = "language_server", test))]
 mod message_cloner {
     use std::sync::Mutex;
 
@@ -237,6 +238,7 @@ mod message_cloner {
         }
     }
 }
+#[cfg(any(feature = "language_server", test))]
 use message_cloner::MessageCloner;
 
 impl Runtime {

Original file line number	Diff line number	Diff line change
`@@ -184,6 +184,7 @@ impl RuntimeFileSystem for OsFileSystem {`
`184`	`184`	`///`
`185`	`185`	/// This type is wrapped in a module so that other code cannot access the inner `UnsafeAllocatorRef`
`186`	`186`	/// directly, and must go via the [`MessageCloner::clone_message`] method.
	`187`	`+#[cfg(any(feature = "language_server", test))]`
`187`	`188`	`mod message_cloner {`
`188`	`189`	`use std::sync::Mutex;`
`189`	`190`
`@@ -237,6 +238,7 @@ mod message_cloner {`
`237`	`238`	`}`
`238`	`239`	`}`
`239`	`240`	`}`
	`241`	`+#[cfg(any(feature = "language_server", test))]`
`240`	`242`	`use message_cloner::MessageCloner;`
`241`	`243`
`242`	`244`	`impl Runtime {`