-
Notifications
You must be signed in to change notification settings - Fork 850
improve UI and backend #366
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
Conversation
For this PR i'm going to wait until we're good with the whole stuff, meaning that until we have a clue about how we're going to handle the @cyclotruc if you got time, please review the current state of the PR, the swagger UI can be improved as you wish. |
@cyclotruc regarding comments from #24 This is way better for LLMs and we keep the possibility to use json for API integrations. Maybe we can do something like this : default -> plaintext, and "format=json" returns the json format. Still, everything will be easy to understand for LLMs since we have |
@ix-56h about exposing an argument format, I agree, but let's keep it for another pr and start with plain text for now we still need to decide the structure of the json, would it be "summary, tree, content" or keys for each files in the digest? things like that |
@ix-56h I think you can remove the additional tasks from the list for now and instead create an issue toward making the API public and documented For now, let's just get this PR tested in order to bring back main into a stable state and move forward from there thanks! |
Here's the list of features to add on another PR once main is back to a working state:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm besides this (but Im no good at js)
fd67762
to
554b611
Compare
…i error response, add api endpoints for ingestion and api config json, better docstring for endpoints, fix include/exclude frontend behavior, new download endpoint
554b611
to
09ba884
Compare
…i error response, add api endpoints for ingestion and api config json, better docstring for endpoints, fix include/exclude frontend behavior, new download endpoint (#366)
Closes: #365
Closes: #24
Closes: #376
Closes: #387
Topics
Improve frontend integration, better loading and results behavior and the html is back in jinja templates

Swagger UI integrated with gitinjest theme

remove
repo_url
from api error response (already available in the error message and it's empty most of the time)add
GET api.gitingest.com/{user}/{repo}
endpointadd
GET/POST/HEAD/OPTION/DELETE/PUT api.gitingest.com
returning theopenapi.json
content (only the GET is available in the swagger UI since we want LLMs to go there instead ofHEAD/DELETE/PUT/OPTION/POST
, but they still serves the same file).POST /api/ingest
now acceptapplication/json
header instead ofapplication/x-www-form
(better for LLMs and integrations with our API)POST /api/ingest
documentation do not include FastAPI internalrequest: Request
argument anymore since it's not something people (and LLMs) can't and must not send (error prone)Fix tree include/exclude bug(ui): selected/marked files not affected when changing between Exclude and Include pattern #376
New download endpoint used to download the full context
@cyclotruc if you think something should be removed from the list i can do it.