From 3869d7713e2dfd2621f7e5a656ebac0b13d10542 Mon Sep 17 00:00:00 2001 From: Jan Hrastnik Date: Sun, 7 Mar 2021 19:41:49 +0100 Subject: [PATCH] added goto functions in helix-term --- Cargo.lock | 1 + helix-term/src/commands.rs | 97 +++++++++++++++++++++++++++++--------- helix-view/src/document.rs | 2 +- 3 files changed, 76 insertions(+), 24 deletions(-) diff --git a/Cargo.lock b/Cargo.lock index 5fac33918..ac1f172a1 100644 --- a/Cargo.lock +++ b/Cargo.lock @@ -550,6 +550,7 @@ dependencies = [ "helix-view", "ignore", "log", + "lsp-types", "num_cpus", "once_cell", "pulldown-cmark", diff --git a/helix-term/src/commands.rs b/helix-term/src/commands.rs index 9b48e8034..87d7ea79e 100644 --- a/helix-term/src/commands.rs +++ b/helix-term/src/commands.rs @@ -13,6 +13,11 @@ use once_cell::sync::Lazy; use crate::compositor::Compositor; use crate::ui::{self, Popup, Prompt, PromptEvent}; +use lsp_types as lsp; +use std::path::PathBuf; + +use smol::Executor; + use helix_view::{ document::Mode, view::{View, PADDING}, @@ -846,34 +851,13 @@ pub fn exit_select_mode(cx: &mut Context) { cx.doc().mode = Mode::Normal; } -pub fn goto_definition(cx: &mut Context) { +pub fn goto_generic(cx: &mut Context, res: Vec) { let doc = cx.doc(); - let language_server = match doc.language_server.as_ref() { - Some(language_server) => language_server, - None => return, - }; - - // TODO: blocking here is not ideal - let pos = helix_lsp::util::pos_to_lsp_pos(doc.text().slice(..), doc.selection().cursor()); - - // TODO: handle fails - let res = - smol::block_on(language_server.goto_definition(doc.identifier(), pos)).unwrap_or_default(); - - /* - if res.len() == 1 { - let definition_pos = res.get(0).unwrap().range.start; - let new_pos = helix_lsp::util::lsp_pos_to_pos(doc.text().slice(..), definition_pos); - doc.set_selection(Selection::point(new_pos)); - } else { - // show menu picker i guess - }*/ - doc.mode = Mode::Normal; log::info!("{:?}", res); - let filepath = doc.path.clone().unwrap(); + let filepath = doc.path().unwrap(); log::info!("{:?}", filepath); match &res.as_slice() { @@ -884,6 +868,9 @@ pub fn goto_definition(cx: &mut Context) { doc.set_selection(Selection::point(new_pos)); } else { // open new file + cx.editor + .open(PathBuf::from(location.uri.path()), cx.executor); + // TODO: go to position } } [] => (), // maybe show user message that no definition was found? @@ -907,6 +894,70 @@ pub fn goto_definition(cx: &mut Context) { } } +pub fn goto_definition(cx: &mut Context) { + let doc = cx.doc(); + let language_server = match doc.language_server.as_ref() { + Some(language_server) => language_server, + None => return, + }; + + // TODO: blocking here is not ideal + let pos = helix_lsp::util::pos_to_lsp_pos(doc.text().slice(..), doc.selection().cursor()); + + // TODO: handle fails + let res = + smol::block_on(language_server.goto_definition(doc.identifier(), pos)).unwrap_or_default(); + goto_generic(cx, res); +} + +pub fn goto_type_definition(cx: &mut Context) { + let doc = cx.doc(); + let language_server = match doc.language_server.as_ref() { + Some(language_server) => language_server, + None => return, + }; + + // TODO: blocking here is not ideal + let pos = helix_lsp::util::pos_to_lsp_pos(doc.text().slice(..), doc.selection().cursor()); + + // TODO: handle fails + let res = smol::block_on(language_server.goto_type_definition(doc.identifier(), pos)) + .unwrap_or_default(); + goto_generic(cx, res); +} + +pub fn goto_implementation(cx: &mut Context) { + let doc = cx.doc(); + let language_server = match doc.language_server.as_ref() { + Some(language_server) => language_server, + None => return, + }; + + // TODO: blocking here is not ideal + let pos = helix_lsp::util::pos_to_lsp_pos(doc.text().slice(..), doc.selection().cursor()); + + // TODO: handle fails + let res = smol::block_on(language_server.goto_implementation(doc.identifier(), pos)) + .unwrap_or_default(); + goto_generic(cx, res); +} + +pub fn goto_reference(cx: &mut Context) { + let doc = cx.doc(); + let language_server = match doc.language_server.as_ref() { + Some(language_server) => language_server, + None => return, + }; + + // TODO: blocking here is not ideal + let pos = helix_lsp::util::pos_to_lsp_pos(doc.text().slice(..), doc.selection().cursor()); + + // TODO: handle fails + let res = + smol::block_on(language_server.goto_reference(doc.identifier(), pos)).unwrap_or_default(); + goto_generic(cx, res); +} + // NOTE: Transactions in this module get appended to history when we switch back to normal mode. pub mod insert { use super::*; diff --git a/helix-view/src/document.rs b/helix-view/src/document.rs index 9ec70023b..82258bde9 100644 --- a/helix-view/src/document.rs +++ b/helix-view/src/document.rs @@ -19,7 +19,7 @@ pub enum Mode { pub struct Document { pub state: State, // rope + selection /// File path on disk. - pub path: Option, // pub for testing + path: Option, /// Current editing mode. pub mode: Mode,