From dae676874fbad48db003250fee5a7fedcb093cf1 Mon Sep 17 00:00:00 2001 From: Jane Lewis Date: Mon, 10 Jun 2024 07:09:53 -0700 Subject: [PATCH] Warn about unavailable document instead of throwing error --- .../src/server/api/requests/execute_command.rs | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/crates/ruff_server/src/server/api/requests/execute_command.rs b/crates/ruff_server/src/server/api/requests/execute_command.rs index 1cf06cf1024b0..5dbcd104dab22 100644 --- a/crates/ruff_server/src/server/api/requests/execute_command.rs +++ b/crates/ruff_server/src/server/api/requests/execute_command.rs @@ -56,10 +56,10 @@ impl super::SyncRequestHandler for ExecuteCommand { let mut edit_tracker = WorkspaceEditTracker::new(session.resolved_client_capabilities()); for Argument { uri, version } in arguments { - let snapshot = session - .take_snapshot(uri.clone()) - .ok_or(anyhow::anyhow!("Document snapshot not available for {uri}",)) - .with_failure_code(ErrorCode::InternalError)?; + let Some(snapshot) = session.take_snapshot(uri.clone()) else { + tracing::warn!("Document at {uri} could not be opened"); + return Ok(None); + }; match command { Command::FixAll => { let fixes = super::code_action_resolve::fix_all_edit(