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

Add tools from LocAgent#108

Merged
ryanhoangt merged 6 commits into
mainfrom
add-locagent-tools
May 21, 2025
Merged

Add tools from LocAgent#108
ryanhoangt merged 6 commits into
mainfrom
add-locagent-tools

Conversation

@ryanhoangt
Copy link
Copy Markdown
Contributor

@ryanhoangt ryanhoangt commented Apr 9, 2025 •
edited
Loading

Description

This PR ports some of the indexing code from LocAgent to the ACI for maintainability.

@ryanhoangt ryanhoangt mentioned this pull request Apr 10, 2025
Merged
2 tasks
@ryanhoangt ryanhoangt marked this pull request as ready for review May 17, 2025 15:44
@czlll @ryanhoangt
Fix tree-sitter compatibility issue between LocAgent and OpenHands (#121
baf7830 
)

* Fix tree-sitter compatibility with version 0.24.0

* update query_file

* poetry lock

* fix test

* fix test & remote ts compat layer

* skip on CI

* remove tree-siter-languages & use pytest-forked

---------

Co-authored-by: Hoang Tran <descience.thh10@gmail.com>
@ryanhoangt ryanhoangt requested a review from xingyaoww May 20, 2025 08:33
xingyaoww
xingyaoww approved these changes May 20, 2025
View reviewed changes
Copy link
Copy Markdown
Collaborator

@xingyaoww xingyaoww left a comment

Choose a reason for hiding this comment

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

This is too much file for me to carefully review every line 😓 - took a quick look..
But since this is not used anywhere for OpenHands yet - let's get this in first and we can probably refactor these along the way while we try to use it

Comment thread pyproject.toml
@ryanhoangt ryanhoangt merged commit fb5fe56 into main May 21, 2025
9 checks passed
@ryanhoangt ryanhoangt deleted the add-locagent-tools branch May 21, 2025 18:07
@enyst
Copy link
Copy Markdown
Contributor

enyst commented May 26, 2025

This is used in OpenHands now

Should we make an aci release soon?

@ryanhoangt
Copy link
Copy Markdown
Contributor Author

ryanhoangt commented May 26, 2025

I think this is released in 0.2.14 and also updated in OpenHands as well: https://github.com/All-Hands-AI/OpenHands/blob/11e6d40c7a9e53c15a1936aee183fc612c624f43/pyproject.toml#L71

@enyst
Copy link
Copy Markdown
Contributor

enyst commented May 26, 2025

Ahh, okay. Weird, I got an error on missing imports, I guess I must have partially updated. 🤷

Thank you!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@xingyaoww xingyaoww xingyaoww approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ryanhoangt @enyst @xingyaoww @czlll