Skip to content

[hotfix][plan] Upgrade tree-sitter to 0.26.6 - #1293

Merged
wenjin272 merged 1 commit into
apache:mainfrom
sangkyoonnam:hotfix/tree-sitter-0.26.6
Oct 20, 2026
Merged

wenjin272 merged 1 commit into
apache:mainfrom
sangkyoonnam:hotfix/tree-sitter-0.26.6

Conversation

@sangkyoonnam

Copy link
Copy Markdown
Contributor

Linked issue: #1289

Purpose of change

When tree-sitter 0.25.3 has to extract its native library, it writes it under FileChannel.lock(), and that lock throws on an already interrupted thread. TSParser's static initializer then fails and the class stays unusable in that class loader (#1289). ut-java fails this way when BashToolTest#interruptionPropagatesInsteadOfReturningToolResult is the first test to create a parser and the library has to be extracted. 0.26.6 extracts the library to a temporary file without the lock (bonede/tree-sitter-ng#133), as @wenjin272 suggested in #1289. Neither version declares dependencies, so META-INF/NOTICE only gets the new version. This supersedes the test-only warm-up in #1290.

Tests

Locally on macOS arm64:

  • The interruption test run on its own, as the first parser initialization in a fresh JVM, passes on JDK 11.0.32.1, 17.0.14 and 21.0.11. With 0.25.3 the same run fails on JDK 11 and 17 with FileLockInterruptionException and ExceptionInInitializerError.
  • BashToolTest (8) and BashValidatorTest (16) pass on all three JDKs with tree-sitter-bash:0.23.3.
  • The full plan module on JDK 17: 649 tests, 0 failures, 3 skipped, and 8 errors, all in AgentPlanDeclareMCPServerTest (MCP initialization returns HTTP 404 locally), which fails the same way on main with 0.25.3.

API

No.

Documentation

  • doc-needed
  • doc-not-needed
  • doc-included

Was this patch authored or co-authored using generative AI tooling?

  • Yes
  • No

Generated-by: Claude Code 2.1.303 (Claude Opus 5.5)

Generated-by: Claude Code 2.1.303 (Claude Opus 5.5)
@github-actions github-actions Bot added doc-not-needed Your PR changes do not impact docs fixVersion/0.4.0 priority/major Default priority of the PR or issue. labels Oct 20, 2026

@wenjin272 wenjin272 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wenjin272
wenjin272 merged commit 212f7c2 into apache:main Oct 20, 2026
54 of 57 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-not-needed Your PR changes do not impact docs fixVersion/0.4.0 priority/major Default priority of the PR or issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants