mirrors/helix
1
0
Fork 0
mirror of https://github.com/helix-editor/helix.git synced 2025-04-04 19:37:54 +03:00
Code Issues Projects Releases Packages Wiki Activity

LSP: No-op client/registerCapability requests (#6258)

Browse source
This commit is contained in:
Michael Davis 2023-03-12 19:29:58 -05:00 • committed by GitHub
parent 1126af53b4
commit 4f066b1cc6
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 16 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

5
helix-lsp/src/lib.rs
View file

@ -526,6 +526,7 @@ pub enum MethodCall {
ApplyWorkspaceEdit(lsp::ApplyWorkspaceEditParams), ApplyWorkspaceEdit(lsp::ApplyWorkspaceEditParams),
WorkspaceFolders, WorkspaceFolders,
WorkspaceConfiguration(lsp::ConfigurationParams), WorkspaceConfiguration(lsp::ConfigurationParams),
RegisterCapability(lsp::RegistrationParams),
} }
impl MethodCall { impl MethodCall {
@ -545,6 +546,10 @@ impl MethodCall {
let params: lsp::ConfigurationParams = params.parse()?; let params: lsp::ConfigurationParams = params.parse()?;
Self::WorkspaceConfiguration(params) Self::WorkspaceConfiguration(params)
} }
lsp::request::RegisterCapability::METHOD => {
let params: lsp::RegistrationParams = params.parse()?;
Self::RegisterCapability(params)
}
_ => { _ => {
return Err(Error::Unhandled); return Err(Error::Unhandled);
} }

11
helix-term/src/application.rs
View file

@ -1014,6 +1014,17 @@ impl Application {
.collect(); .collect();
Ok(json!(result)) Ok(json!(result))
} }
Ok(MethodCall::RegisterCapability(_params)) => {
log::warn!("Ignoring a client/registerCapability request because dynamic capability registration is not enabled. Please report this upstream to the language server");
// Language Servers based on the `vscode-languageserver-node` library often send
// client/registerCapability even though we do not enable dynamic registration
// for any capabilities. We should send a MethodNotFound JSONRPC error in this
// case but that rejects the registration promise in the server which causes an
// exit. So we work around this by ignoring the request and sending back an OK
// response.
Ok(serde_json::Value::Null)
}
}; };
let language_server = match self.editor.language_servers.get_by_id(server_id) { let language_server = match self.editor.language_servers.get_by_id(server_id) {