Bug: Matching brackets not working in plain text.

[heliostatic]

Reproduction steps

Expected: Using mm on an eligible character will go to the matching character in the pair.
Actual: This only seems to work in files with a language server?

2021-11-15.at.16.40.35.-.Alacritty.-.CleanShot.mp4

Environment

• Platform: macOS
• Helix version: v0.5.0-114-g225e790

[Omnikar]

Parsing of matching brackets is done by Tree Sitter, so mm only works on files with a Tree Sitter grammar (not language server).

[heliostatic]

Right, I meant tree sitter, good correction. But I wasn't clear if @archseer was saying it should also work on raw text as well:

[archseer]

Yeah so mm specifically requires tree-sitter: https://docs.helix-editor.com/keymap.html#match-mode

@sudormrfbin recently labeled all the keybindings that require a LSP or a tree-sitter grammar

[heliostatic]

Interesting, thanks!

[heliostatic]

Also wondering about why some brackets match in tree-sitter supported files and others don't:

2021-11-16.at.11.57.37.-.Alacritty.-.CleanShot.mp4

[pickfire]

I think that's because it's not the first and last item in a named node.

https://github.com/ikatyang/tree-sitter-toml/blob/7cff70bbcbbc62001b465603ca1ea88edd668704/grammar.js#L48-L55

    table: $ =>
      seq(
        "[",
        choice($.dotted_key, $._key),
        "]",
        $._line_ending_or_eof,
        repeat(choice($.pair, newline)),
      ),

Our implementation.

helix/helix-core/src/match_brackets.rs

Lines 22 to 28 in ed76cdf

	let node = match tree
	.root_node()
	.named_descendant_for_byte_range(byte_pos, byte_pos)
	{
	Some(node) => node,
	None => return None,
	};

Notice the you showed that worked because it's the first and last item.

https://github.com/ikatyang/tree-sitter-toml/blob/7cff70bbcbbc62001b465603ca1ea88edd668704/grammar.js#L187-L200

    array: $ =>
      seq(
        "[",
        repeat(newline),
        optional(
          seq(
            $._inline_value,
            repeat(newline),
            repeat(seq(",", repeat(newline), $._inline_value, repeat(newline))),
            optional(seq(",", repeat(newline))),
          ),
        ),
        "]",
      ),

Not very familiar with this, not sure how to solve.

[sudormrfbin]

We need an implementation sans tree-sitter anyway for working with plaintext files or files without a grammar, so we could fall back to that if tree-sitter based matching returns no results.

[txtyash]

I too have noticed that mm doesn't work in many places.

[pascalkuthe]

This was adrwssed by #4288 and shortcoming in bracket matching when TS is available discussed in the comments were addressed I'm a seperate PR