Simplify dispatch_lsp_events parameter

Summary: Instead of accepting an impl, take &Server and add to the TspInterface for TSP.

Reviewed By: grievejia

Differential Revision: D92800157

fbshipit-source-id: 9e81dbd2faa4e04276d2b89969d84fc192296a4f

Author: samwgoldman

pyrefly/lib/lsp/non_wasm/server.rs

@@ -353,6 +353,8 @@ pub trait TspInterface: Send + Sync {
 
     fn stop_recheck_queue(&self);
 
+    fn dispatch_lsp_events(&self);
+
     /// Process an LSP event and return the next step
     fn process_event<'a>(
         &'a self,
@@ -740,7 +742,7 @@ pub fn initialize_finish<C: Serialize>(
 /// - priority_events includes those that should be handled as soon as possible (e.g. know that a
 ///   request is cancelled)
 /// - queued_events includes most of the other events.
-pub fn dispatch_lsp_events(server: &impl TspInterface) {
+pub fn dispatch_lsp_events(server: &Server) {
     for msg in &server.connection().receiver {
         match msg {
             Message::Request(x) => {
@@ -4812,6 +4814,10 @@ impl TspInterface for Server {
         &self.pending_watched_file_changes
     }
 
+    fn dispatch_lsp_events(&self) {
+        dispatch_lsp_events(self);
+    }
+
     fn run_recheck_queue(&self, telemetry: &impl Telemetry) {
         self.recheck_queue.run_until_stopped(self, telemetry);
     }

pyrefly/lib/tsp/server.rs

@@ -26,7 +26,6 @@ use crate::lsp::non_wasm::server::ProcessEvent;
 use crate::lsp::non_wasm::server::ServerCapabilitiesWithTypeHierarchy;
 use crate::lsp::non_wasm::server::TspInterface;
 use crate::lsp::non_wasm::server::capabilities;
-use crate::lsp::non_wasm::server::dispatch_lsp_events;
 use crate::lsp::non_wasm::transaction_manager::TransactionManager;
 
 /// TSP server that delegates to LSP server infrastructure while handling only TSP requests
@@ -149,7 +148,7 @@ pub fn tsp_loop(
         scope.spawn(|| server.inner.run_recheck_queue(telemetry));
 
         scope.spawn(|| {
-            dispatch_lsp_events(&server.inner);
+            server.inner.dispatch_lsp_events();
         });
 
         let mut ide_transaction_manager = TransactionManager::default();