-
Notifications
You must be signed in to change notification settings - Fork 2
refactor: code generated from refactored openapi specs #71
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?
Conversation
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.
Pull request overview
This PR represents a comprehensive refactoring of the GetStream SDK, with code regenerated from refactored OpenAPI specifications. The changes primarily focus on improving type consistency, documentation, and API design across the feeds, chat, and video domains.
Key Changes
- Type System Refactoring: Separation of request/response types with consistent naming patterns (e.g.,
*Request,*Response,*Payloadsuffixes) - Model Consolidation: Removal of duplicate or deprecated types (e.g.,
Channel,Message,User,Poll,Banreplaced with dedicated response types) - API Method Updates: Renamed batch operations methods (
OwnCapabilitiesBatch→OwnBatch) and added new delete operations - Enhanced Documentation: Added comprehensive comments to events, request types, and response types
Reviewed changes
Copilot reviewed 8 out of 9 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
requests.go |
Renamed request types with consistent *Request and *Payload suffixes; added new fields and types for batch operations |
models.go |
Major refactoring: removed ~30 deprecated types, added ~50 new response types, enhanced event documentation, separated feeds/chat reaction types |
feeds.go |
Minor change with an unprofessional comment that needs removal |
feeds-v3.go |
Renamed OwnCapabilitiesBatch to OwnBatch, added DeleteFeedsBatch, updated DeleteFeedUserData endpoint from DELETE to POST |
feeds_test.go |
Updated tests to match renamed methods and added new test for DeleteFeedsBatch |
common_test.go |
Removed test skip statements for upload tests |
chat_test.go |
Removed test skip statements for upload tests |
chat.go |
Minor documentation update (duplicate event comment) |
call.go |
Reformatted imports to use grouped style |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| // | ||
| // Sends events: | ||
| // - * | ||
| // - * |
Copilot
AI
Dec 16, 2025
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.
The duplicated comment "// - *" on line 920 should be removed. It appears to be a documentation error where the same wildcard event marker was added twice, which provides no additional information and reduces documentation clarity.
| // - * |
Submit a pull request
CLA
Description of the pull request