-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Explore whether stack graphs may be useful in this tool #69
Comments
Hi, thanks for the tips and the reading list, I already had read all of them but LSP and LSIF are still pending. We are definitely looking into the option of using tree-sitter-graphs. I'm currently weighing two options:
I think I'm going to explore the integration option, try to make it run, and if it's not too difficult, that's the route I'll go. |
I was checking LSP and seems to be a really power full tool. But it does a lot more of what I really need in this stage, because we are focused in give LLM a tool to navigate and understand code base repositories in a easy way. For that I think the best is to implement 'tree-sitter-graph' and later see in what could help LSP. |
@berrazuriz1 I would make sure to look at both eg.
@berrazuriz1 Yeah, that's fair enough. There are theoretically ways to call/embed rust within python as well I believe, but that may end up being even more complex than shelling out to the CLI e.g. Some random related links:
@berrazuriz1 Yeah, I think LSP is likely not relevant at this stage. I really only included the links there as part of the "why we aren't using" context. |
I think running Rust on Python is a good solution. I'm going to give it a try. |
Not really a feature request per se, but came across this project after it was mentioned in another issue (Ref), and figured I would create an issue to share the same info here in case it's useful.
I notice that you're already using
tree-sitter
, so it may not be a lot of extra effort to make use oftree-sitter-graph
and/or thestack-graphs
project (eg.tree-sitter-stack-graphs-javascript
, etc)From watching the demo video on linkedin + briefly skimming this repo, it looks like there might be some useful crossovers, and it may allow you to add support for a whole bunch more languages without needing to re-invent the wheel to do so.
A few notes/links/references I recently collated RE: stack graphs + related libs:
The text was updated successfully, but these errors were encountered: