improve match bracket matching

[pascalkuthe]

This PR fixes a couple of edgecases that were noticed in the match bracket implementation after the recent changes to match_bracket.

This should cover both the markdown cases discovered by @David-Else (although I think there are still some cases involving injections that may not be covered but all the examples discussed work now) and the python cases discoreved by @ thomasaarholt. I also included fixes for some issues for cases where in cases where multiple nested pairs are siblings. I think this probably shouldn't happen usually, but it can happen in case there are syntax error so it's nice to be a bit more robust

[woojiq]

I will rebase my changes on top of yours to see if they work together.
Unfortunately I can't review your PR properly because I don't really know how to work with the tree-sitter API, I used only helix's wrappers around TS in my commits)

[David-Else]

Brilliant! In Markdown words surrounded in backticks now work everywhere, and also all the individual matching pair elements ([] () "") inside a link work, for example:

My favorite search engine is [Duck Duck Go](https://duckduckgo.com "The best search engine for privacy").

For reasons unknown to me you can mim inside the " in the link above, but not in general text:

Try `mim in this backticked text`, no problem, but it doesn't work in "this quoted text"

Not that big a deal, but I thought I would mention it in case it was a bug.

[woojiq]

For reasons unknown to me you can mim inside the " in the link above, but not in general text:

Type :tree-sitter-subtree and you will notice that there are no nodes for default text, but there is a node for "link_title" that contains quotes.

[pascalkuthe]

Markdown seems to be just a weird case its grammar the quotes there are not actually part of the sytnax tree (they are really just plaintext). The inline grammar that markdown uses is pretty hacky and the way it seems to produce nodes... Frankly doesn't make much sense. I think it works this way because of TS limitations/because markdown really isn't all that suited to a proper syntax tree but its mostly a coincidence that is works sometimes at all that is down to implementation details of the language.

mim/mam/mm for quotes (and even brackets where they aren't part of links) just won't work with markdown. The fact that they sometimes do is pure coincidence.

[thomasaarholt]

Tested this with #10611 on top, only with Python. Everything works as expected. Nice work, and thanks for fixing!

[archseer]

Can we add some unit/integration tests for the bugs being fixed?

[pascalkuthe]

@archseer good point I added a couple integration tests both for past edge-cases and the new cases I fixed