Skip to content

highlighter: Update tree-sitter to remove workaround - #2990

Merged
huacnlee merged 2 commits into
longbridge:mainfrom
zanmato:remove-workaround
Sep 7, 2026
Merged

highlighter: Update tree-sitter to remove workaround#2990
huacnlee merged 2 commits into
longbridge:mainfrom
zanmato:remove-workaround

Conversation

@zanmato

@zanmato zanmato commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Description

Removes a workaround for large nodes introduced in #2128, fixed in upstream tree-sitter/tree-sitter#5801
Backported to 0.26 tree-sitter/tree-sitter#5855

How to Test

Open large broken sql file in the editor example:

$ (echo ""; for i in $(seq 1 9999); do echo "('$i'),"; done; echo "('10000')"; echo ') as x(ix);') > test_broken.sql
$ cargo run --release -p example-editor

Checklist

  • I have read the CONTRIBUTING document and followed the guidelines.
  • Reviewed the changes in this PR and confirmed AI generated code (If any) is accurate.
  • Passed cargo run for story tests related to the changes.
  • Tested macOS, Windows and Linux platforms performance (if the change is platform-specific)

@huacnlee
huacnlee merged commit 5670451 into longbridge:main Sep 7, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants