syntax: Use matches rather than captures in textobject queries

Previously, `TextObject::capture_nodes` and `capture_nodes_any`
collected the captures within a match for every capture which occurred,
so for a query like

    [
      (line_comment)
      (block_comment)
    ] @comment.inside

and text like

    // comment 1
    // comment 2

`capture_nodes` would produce `CaptureNode::Single` for the first
comment and also `CaptureNode::Grouped` for both comments.

Instead we want to have tree-sitter complete the full match and then
consider nodes from the requested capture. Then `capture_nodes` produces
a single `CaptureNode::Grouped` for both comments.

This fixes the behavior of `mac` in languages like Rust.

Author: the-mikedavis

helix-core/src/syntax.rs

@@ -1054,7 +1054,7 @@ impl TextObjectQuery {
         let mut cursor = InactiveQueryCursor::new(0..u32::MAX, TREE_SITTER_MATCH_LIMIT)
             .execute_query(&self.query, node, RopeInput::new(slice));
         let capture_node = iter::from_fn(move || {
-            let (mat, _) = cursor.next_matched_node()?;
+            let mat = cursor.next_match()?;
             Some(mat.nodes_for_capture(capture).cloned().collect())
         })
         .filter_map(move |nodes: Vec<_>| {
@@ -1241,8 +1241,13 @@ mod test {
         };
 
         test("quantified_nodes", 1..37);
-        // NOTE: Enable after implementing proper node group capturing
-        // test("quantified_nodes_grouped", 1..37);
+        test("quantified_nodes_grouped", 1..37);
+        // TODO: the query for this works instead as
+        // ```
+        // ((line_comment) @capture (line_comment) @capture)
+        // ```
+        // The query used in this test case only captures the first line_comment node.
+        // Determine if this behavior is intentional in tree-sitter.
         // test("multiple_nodes_grouped", 1..37);
     }