remove check

Author: BenjaminBrienen

crates/rust-analyzer/src/handlers/request.rs

@@ -20,9 +20,9 @@ use lsp_types::{
     CallHierarchyOutgoingCall, CallHierarchyOutgoingCallsParams, CallHierarchyPrepareParams,
     CodeLens, CompletionItem, Contents, FoldingRange, FoldingRangeParams, InlayHint,
     InlayHintParams, Location, LocationLink, Position, PrepareRenameResult, Range, RenameParams,
-    ResourceOperationKind, SemanticTokens, SemanticTokensDeltaParams,
-    SemanticTokensDeltaRequestResponse, SemanticTokensParams, SemanticTokensRangeParams,
-    SymbolInformation, SymbolTag, TextDocumentIdentifier, Uri, WorkspaceEdit,
+    SemanticTokens, SemanticTokensDeltaParams, SemanticTokensDeltaRequestResponse,
+    SemanticTokensParams, SemanticTokensRangeParams, SymbolInformation, SymbolTag,
+    TextDocumentIdentifier, Uri, WorkspaceEdit,
 };
 use paths::Utf8PathBuf;
 use project_model::{CargoWorkspace, ManifestPath, ProjectWorkspaceKind, TargetKind};
@@ -33,9 +33,7 @@ use triomphe::Arc;
 use vfs::{AbsPath, AbsPathBuf, FileId, VfsPath};
 
 use crate::{
-    config::{
-        ClientCommandsConfig, Config, HoverActionsConfig, RustfmtConfig, WorkspaceSymbolConfig,
-    },
+    config::{ClientCommandsConfig, HoverActionsConfig, RustfmtConfig, WorkspaceSymbolConfig},
     diagnostics::convert_diagnostic,
     global_state::{FetchWorkspaceRequest, GlobalState, GlobalStateSnapshot},
     line_index::LineEndings,
@@ -1414,12 +1412,6 @@ pub(crate) fn handle_rename(
 
     let workspace_edit = to_proto::workspace_edit(&snap, change)?;
 
-    if let Some(changes) = workspace_edit.document_changes.as_ref() {
-        for change in changes {
-            resource_ops_supported(&snap.config, resolve_resource_op(change))?;
-        }
-    }
-
     Ok(Some(workspace_edit))
 }
 
@@ -1537,17 +1529,6 @@ pub(crate) fn handle_code_action(
             None
         };
         let code_action = to_proto::code_action(&snap, &client_commands, assist, resolve_data)?;
-
-        // Check if the client supports the necessary `ResourceOperation`s.
-        let changes = code_action.edit.as_ref().and_then(|it| it.document_changes.as_ref());
-        if let Some(changes) = changes {
-            for change in changes {
-                if let lsp_ext::SnippetDocumentChangeOperation::Change(res_op) = change {
-                    resource_ops_supported(&snap.config, resolve_resource_op(res_op))?
-                }
-            }
-        }
-
         res.push(code_action)
     }
 
@@ -1640,17 +1621,6 @@ pub(crate) fn handle_code_action_resolve(
     let ca = to_proto::code_action(&snap, &snap.config.client_commands(), assist.clone(), None)?;
     code_action.edit = ca.edit;
     code_action.command = ca.command;
-
-    if let Some(edit) = code_action.edit.as_ref()
-        && let Some(changes) = edit.document_changes.as_ref()
-    {
-        for change in changes {
-            if let lsp_ext::SnippetDocumentChangeOperation::Change(res_op) = change {
-                resource_ops_supported(&snap.config, resolve_resource_op(res_op))?
-            }
-        }
-    }
-
     Ok(code_action)
 }
 
@@ -2669,35 +2639,6 @@ fn crate_path(root_file_path: &VfsPath) -> Option<VfsPath> {
     None
 }
 
-fn resource_ops_supported(config: &Config, kind: ResourceOperationKind) -> anyhow::Result<()> {
-    if !matches!(config.workspace_edit_resource_operations(), Some(resops) if resops.contains(&kind))
-    {
-        return Err(LspError::new(
-            ErrorCode::RequestFailed as i32,
-            format!(
-                "Client does not support {} capability.",
-                match kind {
-                    ResourceOperationKind::Create => "create",
-                    ResourceOperationKind::Rename => "rename",
-                    ResourceOperationKind::Delete => "delete",
-                }
-            ),
-        )
-        .into());
-    }
-
-    Ok(())
-}
-
-fn resolve_resource_op(op: &lsp_types::DocumentChange) -> ResourceOperationKind {
-    match op {
-        lsp_types::DocumentChange::CreateFile(_) => ResourceOperationKind::Create,
-        lsp_types::DocumentChange::RenameFile(_) => ResourceOperationKind::Rename,
-        lsp_types::DocumentChange::DeleteFile(_) => ResourceOperationKind::Delete,
-        lsp_types::DocumentChange::TextDocumentEdit(_) => todo!(),
-    }
-}
-
 pub(crate) fn diff(left: &str, right: &str) -> TextEdit {
     use dissimilar::Chunk;