-
Notifications
You must be signed in to change notification settings - Fork 903
Closed as not planned
@ix-56h
Description
Refer to:
#346 (comment)
#346 (comment)
Subject
Now that #346 is ready to merge, we need to discuss a bit about DTO.
Once the PR is merged, we going to have well defined parameters and response objects for our api endpoints.
For a potential new frontend, and mostly for anyone using the API (even LLMs), we needs to strictly define which data should be provided and when.
Ideas
/api/ingest
should return a way more normalized json response. For instance here's a draft for success request :
{ "status": OK | ERROR, "error_msg": STR | NONE, "result": { "repo_url": "string", "short_repo_url": "string", "summary": "string", "tree": "string", "content": "string" } }