[red-knot] Don't use latency-sensitive for handlers

crates/red_knot_server/src/server/api.rs

@@ -20,21 +20,19 @@ pub(super) fn request<'a>(req: server::Request) -> Task<'a> {
     let id = req.id.clone();
 
     match req.method.as_str() {
-        request::DocumentDiagnosticRequestHandler::METHOD => {
-            background_request_task::<request::DocumentDiagnosticRequestHandler>(
-                req,
-                BackgroundSchedule::LatencySensitive,
-            )
+        request::DocumentDiagnosticRequestHandler::METHOD => background_request_task::<
+            request::DocumentDiagnosticRequestHandler,
+        >(
+            req, BackgroundSchedule::Worker
+        ),
+        request::GotoTypeDefinitionRequestHandler::METHOD => background_request_task::<
+            request::GotoTypeDefinitionRequestHandler,
+        >(
+            req, BackgroundSchedule::Worker
+        ),
+        request::HoverRequestHandler::METHOD => {
+            background_request_task::<request::HoverRequestHandler>(req, BackgroundSchedule::Worker)
         }
-        request::GotoTypeDefinitionRequestHandler::METHOD => {
-            background_request_task::<request::GotoTypeDefinitionRequestHandler>(
-                req,
-                BackgroundSchedule::LatencySensitive,
-            )
-        }
-        request::HoverRequestHandler::METHOD => background_request_task::<
-            request::HoverRequestHandler,
-        >(req, BackgroundSchedule::LatencySensitive),
 
         method => {
             tracing::warn!("Received request {method} which does not have a handler");

crates/red_knot_server/src/server/schedule/task.rs

@@ -19,9 +19,10 @@ pub(in crate::server) enum BackgroundSchedule {
     /// for formatting actions. This is a high priority thread.
     Fmt,
     /// The task should be run on the general high-priority background
-    /// thread.
+    /// thread. Reserved for actions caused by the user typing (e.g.syntax highlighting).
     LatencySensitive,
     /// The task should be run on a regular-priority background thread.
+    /// The default for any request that isn't in the critical path of the user typing.
     #[default]
     Worker,
 }

crates/ruff_server/src/server/api.rs

@@ -29,18 +29,14 @@ pub(super) fn request<'a>(req: server::Request) -> Task<'a> {
     let id = req.id.clone();
 
     match req.method.as_str() {
-        request::CodeActions::METHOD => background_request_task::<request::CodeActions>(
-            req,
-            BackgroundSchedule::LatencySensitive,
-        ),
+        request::CodeActions::METHOD => {
+            background_request_task::<request::CodeActions>(req, BackgroundSchedule::Worker)
+        }
         request::CodeActionResolve::METHOD => {
             background_request_task::<request::CodeActionResolve>(req, BackgroundSchedule::Worker)
         }
         request::DocumentDiagnostic::METHOD => {
-            background_request_task::<request::DocumentDiagnostic>(
-                req,
-                BackgroundSchedule::LatencySensitive,
-            )
+            background_request_task::<request::DocumentDiagnostic>(req, BackgroundSchedule::Worker)
         }
         request::ExecuteCommand::METHOD => local_request_task::<request::ExecuteCommand>(req),
         request::Format::METHOD => {

crates/ruff_server/src/server/schedule/task.rs

@@ -19,9 +19,11 @@ pub(in crate::server) enum BackgroundSchedule {
     /// for formatting actions. This is a high priority thread.
     Fmt,
     /// The task should be run on the general high-priority background
-    /// thread.
+    /// thread. Reserved for actions caused by the user typing (e.g.syntax highlighting).
+    #[expect(dead_code)]
     LatencySensitive,
     /// The task should be run on a regular-priority background thread.
+    /// The default for any request that isn't in the critical path of the user typing.
     #[default]
     Worker,
 }