Skip to content
This repository was archived by the owner on May 15, 2026. It is now read-only.

Update tree-sitter#73

Merged
neubig merged 10 commits into
OpenHands:mainfrom
enyst:update-tree-sitter
Feb 17, 2025
Merged

Update tree-sitter#73
neubig merged 10 commits into
OpenHands:mainfrom
enyst:update-tree-sitter

Conversation

@enyst

@enyst enyst commented Feb 15, 2025

Copy link
Copy Markdown
Member

Description

TestGenEval benchmark seems to need an upgraded tree-sitter. This PR updates tree-sitter to version 0.24.0.

Changes

  • Updated tree-sitter from 0.21.3 to 0.24.0
  • Added dynamic language imports to support any tree-sitter language package
  • Added Python, JavaScript, and TypeScript as default languages
  • Created a compatibility layer to handle the new tree-sitter API

Testing

  • All tests are passing
  • All pre-commit hooks pass

References

@neubig

neubig commented Feb 16, 2025

Copy link
Copy Markdown
Member

OK, I think with OpenHands's help I managed to make this work. It seems like the tree-sitter-languages has not been updated recently, and we should just build our own compatibility library.

@enyst

enyst commented Feb 16, 2025

Copy link
Copy Markdown
Member Author

Thank you!

@ryanhoangt ryanhoangt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks, this is interesting and LGTM! @enyst @neubig do you want to test this PR in the OH repo before we merge this?

@neubig

neubig commented Feb 17, 2025

Copy link
Copy Markdown
Member

Thanks @ryanhoangt , I tried it quickly in OpenHands by providing this prompt:

We are upgrading a dependency of the OpenHands library (openhands-aci) here: https://github.com/All-Hands-AI/openhands-aci/pull/73

Please modify pyproject.toml in the OpenHands folder to install openhands-aci from the branch associated with this PR, instead of from where it is normally installed (pypi). Then follow the Python unit testing procedure described in py-unit-tests.yml exactly.

and it worked after OpenHands upgraded the tree-sitter version in OpenHands as well, so I think we're good.

@neubig neubig merged commit 707d6da into OpenHands:main Feb 17, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants