Rewrite method matching with TS query

Author: lionel-

crates/ark/src/lsp/indexer.rs

@@ -314,61 +314,39 @@ fn index_r6_class(
     node: &Node,
     entries: &mut Vec<IndexEntry>,
 ) -> anyhow::Result<()> {
-    let Some(args_node) = node.child_by_field_name("arguments") else {
-        return Ok(());
-    };
-
-    let mut cursor = args_node.walk();
-    for arg in args_node.children(&mut cursor) {
-        // Only consider `public = ` and `private = ` arguments
-        let Some(arg_name) = arg.child_by_field_name("name") else {
-            continue;
-        };
-        if !arg_name.is_identifier_or_string() {
-            continue;
-        }
-        let arg_name_str = contents.node_slice(&arg_name)?;
-        if arg_name_str != "public" && arg_name_str != "private" {
-            continue;
-        }
-
-        let Some(list_node) = arg.child_by_field_name("value") else {
-            continue;
-        };
-        if !list_node.is_call() {
-            continue;
-        }
-
-        let Some(list_args) = list_node.child_by_field_name("arguments") else {
-            return Ok(());
-        };
-
-        let mut cursor = list_args.walk();
-        for arg in list_args.children(&mut cursor) {
-            if !arg.is_argument() {
-                continue;
-            }
+    // Tree-sitter query to match individual methods in R6Class public/private lists
+    let query_str = r#"
+    (argument
+      name: (identifier) @access
+      value: (call
+        function: (identifier) @_list_fn
+        arguments: (arguments
+          (argument
+            name: (identifier) @method_name
+            value: (function_definition) @method_fn
+          )
+        )
+      )
+      (#match? @access "public|private")
+      (#eq? @_list_fn "list")
+    )
+    "#;
+    let mut ts_query = crate::treesitter::TSQuery::new(query_str)?;
 
-            let (Some(mtd_name), Some(mtd_value)) = (
-                arg.child_by_field_name("name"),
-                arg.child_by_field_name("value"),
-            ) else {
-                continue;
-            };
-            if !mtd_name.is_identifier_or_string() || !mtd_value.is_function_definition() {
-                continue;
-            }
+    // We'll switch from Rope to String in the near future so let's not
+    // worry about this conversion now
+    let contents_str = contents.to_string();
 
-            let name = contents.node_slice(&mtd_name)?.to_string();
-            let start = convert_point_to_position(contents, mtd_name.start_position());
-            let end = convert_point_to_position(contents, mtd_name.end_position());
+    for method_node in ts_query.captures_for(*node, "method_name", contents_str.as_bytes()) {
+        let name = contents.node_slice(&method_node)?.to_string();
+        let start = convert_point_to_position(contents, method_node.start_position());
+        let end = convert_point_to_position(contents, method_node.end_position());
 
-            entries.push(IndexEntry {
-                key: name.clone(),
-                range: Range { start, end },
-                data: IndexEntryData::Method { name },
-            });
-        }
+        entries.push(IndexEntry {
+            key: name.clone(),
+            range: Range { start, end },
+            data: IndexEntryData::Method { name },
+        });
     }
 
     Ok(())

crates/ark/src/treesitter.rs

@@ -1,3 +1,4 @@
+use anyhow::anyhow;
 use tree_sitter::Node;
 
 use crate::lsp::traits::node::NodeExt;
@@ -603,3 +604,42 @@ pub(crate) fn node_find_containing_call<'tree>(node: Node<'tree>) -> Option<Node
 
     None
 }
+
+pub(crate) struct TSQuery {
+    query: tree_sitter::Query,
+    cursor: tree_sitter::QueryCursor,
+}
+
+impl TSQuery {
+    pub(crate) fn new(query_str: &str) -> anyhow::Result<Self> {
+        let language = &tree_sitter_r::LANGUAGE.into();
+        let query = tree_sitter::Query::new(language, query_str)
+            .map_err(|err| anyhow!("Failed to compile query: {err}"))?;
+
+        let cursor = tree_sitter::QueryCursor::new();
+
+        Ok(Self { query, cursor })
+    }
+
+    /// Match query against `contents` and collect all nodes captured with the
+    /// given capture name
+    pub(crate) fn captures_for<'tree>(
+        &mut self,
+        node: tree_sitter::Node<'tree>,
+        capture_name: &str,
+        contents: &[u8],
+    ) -> Vec<tree_sitter::Node<'tree>> {
+        let mut result = Vec::new();
+
+        for m in self.cursor.matches(&self.query, node, contents) {
+            for cap in m.captures.iter() {
+                let cap_name = &self.query.capture_names()[cap.index as usize];
+                if *cap_name == capture_name {
+                    result.push(cap.node);
+                }
+            }
+        }
+
+        result
+    }
+}