perf(langage_server): prebuild IsolatedLintHandler

Author: Sysix

crates/oxc_language_server/src/linter/isolated_lint_handler.rs

@@ -29,12 +29,12 @@ pub struct IsolatedLintHandlerOptions {
 }
 
 pub struct IsolatedLintHandler {
-    linter: Arc<Linter>,
-    options: Arc<IsolatedLintHandlerOptions>,
+    linter: Linter,
+    options: IsolatedLintHandlerOptions,
 }
 
 impl IsolatedLintHandler {
-    pub fn new(linter: Arc<Linter>, options: Arc<IsolatedLintHandlerOptions>) -> Self {
+    pub fn new(linter: Linter, options: IsolatedLintHandlerOptions) -> Self {
         Self { linter, options }
     }
 
@@ -121,7 +121,7 @@ impl IsolatedLintHandler {
         .with_cross_module(self.options.use_cross_module);
         // ToDo: do not clone the linter
         let path_arc = Arc::from(path.as_os_str());
-        let mut lint_service = LintService::new((*self.linter).clone(), lint_service_options);
+        let mut lint_service = LintService::new(self.linter.clone(), lint_service_options);
         let result = lint_service.run_source(allocator, &path_arc, &source_text);
 
         Some(result)

crates/oxc_language_server/src/linter/server_linter.rs

@@ -11,25 +11,28 @@ use super::isolated_lint_handler::IsolatedLintHandlerOptions;
 
 #[derive(Clone)]
 pub struct ServerLinter {
-    linter: Arc<Linter>,
-    options: Arc<IsolatedLintHandlerOptions>,
+    isolated_linter: Arc<IsolatedLintHandler>,
 }
 
 impl ServerLinter {
     pub fn new(options: IsolatedLintHandlerOptions) -> Self {
         let config_store =
             ConfigStoreBuilder::default().build().expect("Failed to build config store");
         let linter = Linter::new(LintOptions::default(), config_store).with_fix(FixKind::SafeFix);
-        Self { linter: Arc::new(linter), options: Arc::new(options) }
+
+        let isolated_linter = Arc::new(IsolatedLintHandler::new(linter, options));
+
+        Self { isolated_linter }
     }
 
     pub fn new_with_linter(linter: Linter, options: IsolatedLintHandlerOptions) -> Self {
-        Self { linter: Arc::new(linter), options: Arc::new(options) }
+        let isolated_linter = Arc::new(IsolatedLintHandler::new(linter, options));
+
+        Self { isolated_linter }
     }
 
     pub fn run_single(&self, uri: &Uri, content: Option<String>) -> Option<Vec<DiagnosticReport>> {
-        IsolatedLintHandler::new(Arc::clone(&self.linter), Arc::clone(&self.options))
-            .run_single(&uri.to_file_path().unwrap(), content)
+        self.isolated_linter.run_single(&uri.to_file_path().unwrap(), content)
     }
 }