-
Notifications
You must be signed in to change notification settings - Fork 3
Feat ag UI api #80
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
base: main
Are you sure you want to change the base?
Feat ag UI api #80
Conversation
cmd/agui/types.go
Outdated
| AnalysisStatusPending AnalysisStatus = "pending" | ||
| AnalysisStatusProcessing AnalysisStatus = "processing" | ||
| AnalysisStatusCompleted AnalysisStatus = "completed" | ||
| AnalysisStatusFailed AnalysisStatus = "failed" |
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.
we don't need to redefine status, see simple-content docs about status.
| // Step 5.5: Get object metadata from storage and save it | ||
| storageMetadata, err := backend.GetObjectMeta(ctx, objectKey) |
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.
@tendant could you review this part?
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.
@sgao640 could you use a different PR for this?
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.
@sgao640 could you use a different PR for this?
removed
livefire2015
left a comment
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.
could you add OpenAPI yaml: ag-ui-conent-v5.yaml ?
used the name |
livefire2015
left a comment
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.
Any localhost screenshots?
cmd/agui/ag-ui-content-v5.yaml
Outdated
| '500': | ||
| $ref: '#/components/responses/InternalServerError' | ||
|
|
||
| /health: |
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.
no need of this health route
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.
should I remove from the code too or only from the yaml?
523fe10 to
4c42a51
Compare
3ba69e7 to
4fca2b0
Compare
|
Item 7: do you need to rebase, or is it new addition? |
|
Please show local test results, showing that the API are ready for integration. |
Rebase already done before the summary. I'll double check with Claude. Sometimes it include previous commits in the summary. It is not a new addition. |
AG-UI API Implementation - 7 Files ChangedNew AG-UI Server (5 files in
|
97530e7 to
2cc395f
Compare
|
@sgao640 could you add support for owner? Also, how can we let API caller use content_id to their domains, say, a conversation message, a card transaction, a reimbursement, etc? Ask AI. |
@livefire2015 support owner as in using |
|
@sgao640 This set of API does not seem to be related to AG-UI: https://docs.ag-ui.com/concepts/architecture |









@livefire2015
@tendant