reset

Author: juleswritescode

crates/pgt_hover/src/hovered_node.rs

@@ -64,6 +64,17 @@ impl HoveredNode {
                 Some(HoveredNode::Role(NodeIdentification::Name(node_content)))
             }
 
+            "policy_table" | "revoke_table" | "grant_table" => {
+                if let Some(schema) = ctx.schema_or_alias_name.as_ref() {
+                    Some(HoveredNode::Table(NodeIdentification::SchemaAndName((
+                        schema.clone(),
+                        node_content,
+                    ))))
+                } else {
+                    Some(HoveredNode::Table(NodeIdentification::Name(node_content)))
+                }
+            }
+
             _ => None,
         }
     }

crates/pgt_hover/src/lib.rs

@@ -20,6 +20,10 @@ pub struct OnHoverParams<'a> {
     pub ts_tree: &'a tree_sitter::Tree,
 }
 
+#[tracing::instrument(level = "debug", skip_all, fields(
+    text = params.stmt_sql,
+    position = params.position.to_string()
+))]
 pub fn on_hover(params: OnHoverParams) -> Vec<String> {
     let ctx = pgt_treesitter::context::TreesitterContext::new(TreeSitterContextParams {
         position: params.position,

crates/pgt_lsp/src/handlers/hover.rs

@@ -3,6 +3,7 @@ use tower_lsp::lsp_types::{self, MarkedString, MarkupContent};
 
 use crate::{adapters::get_cursor_position, diagnostics::LspError, session::Session};
 
+#[tracing::instrument(level = "debug", skip(session), err)]
 pub(crate) fn on_hover(
     session: &Session,
     params: lsp_types::HoverParams,

crates/pgt_workspace/src/workspace/server.rs

@@ -719,6 +719,11 @@ impl Workspace for WorkspaceServer {
         }
     }
 
+    #[ignored_path(path=&params.path)]
+    #[tracing::instrument(level = "debug", skip_all, fields(
+        path = params.path.as_os_str().to_str(),
+        position = params.position.to_string()
+    ), err)]
     fn on_hover(&self, params: OnHoverParams) -> Result<OnHoverResult, WorkspaceError> {
         let documents = self.documents.read().unwrap();
         let doc = documents
@@ -742,7 +747,7 @@ impl Workspace for WorkspaceServer {
             .next()
         {
             Some((stmt_id, range, ts_tree, maybe_ast)) => {
-                let position_in_stmt = params.position + range.start();
+                let position_in_stmt = params.position - range.start();
 
                 let markdown_blocks = pgt_hover::on_hover(pgt_hover::OnHoverParams {
                     ts_tree: &ts_tree,