feat: add list loaded threads to app server#8902
Merged
Conversation
owenlin0
reviewed
Jan 8, 2026
| #[ts(export_to = "v2/")] | ||
| pub struct ThreadLoadedListResponse { | ||
| /// Thread ids for sessions currently loaded in memory. | ||
| pub data: Vec<String>, |
Collaborator
There was a problem hiding this comment.
Should this return Vec<Thread>? i.e. would the metadata on Thread be useful?
We don't have to populate all the turns & items since we only do that for specific APIs (we've already documented the behavior on the Thread object)
Collaborator
Author
There was a problem hiding this comment.
No because sometimes we don't have them and don't need them
Collaborator
There was a problem hiding this comment.
Also, would be nice to support pagination for consistency with our other list APIs, even though in practice you'd probably want to return everything always. Mainly because adding pagination later is a breaking change
pub struct ThreadLoadedListRequest {
/// Opaque pagination cursor returned by a previous call.
pub cursor: Option<String>,
/// Optional page size; defaults to no limit
pub limit: Option<u32>,
}
pub struct ThreadLoadedListResponse {
pub data: Vec<String>,
/// Opaque cursor to pass to the next call to continue after the last item.
/// if None, there are no more items to return.
pub next_cursor: Option<String>,
}
owenlin0
approved these changes
Jan 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.