Crash when rendering certain SQL files about helix HOT 7 OPEN

I can reproduce this and updating to the latest tree-sitter-sql revision fixes it

from helix.

This looks like an issue with the grammar. The SQL grammar has caused problems before. You would need to report this upstream to get that fixed (or maybe just updating the grammar could be enough)

from helix.

I couldn't reproduce it with Helix master.... mmm.

Regardless, I think we should update to https://github.com/DerekStride/tree-sitter-sql/releases/tag/v0.2.0 as there has been a lot of activity since the last Oct 4th Helix version, that was a pre-release version, first release version was v0.1.0 Nov 17, 2023.

@ds-cbo Do you feel like updating tree-sitter-sql again?

from helix.

Hm, I just tried again with an updated master, and it still crashes. I'd be happy to do anything else if it helps reproduce or figure out what's going wrong.

But if you just want to update tree-sitter-sql, I'd be happy to just try again when that's done.

from helix.

@David-Else I'll gladly bump it again if that's still necessary

from helix.

@ds-cbo Awesome! It seems the bump is needed to avoid this crash. I am afraid I don't have the knowledge needed for making any new new queries if they are required.

from helix.

I've found some oddities upstream while bumping: DerekStride/tree-sitter-sql#239

Play stay tuned while I resolve those first, otherwise I might be bumping again in a few days 😛

from helix.