feat(frontier): add upper bounds to pagination page_size and rql limit fields#492
Conversation
…t fields Caps page_size on the six paginated list request messages and the shared RQLRequest limit at 5000 via buf.validate lte rules. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
The latest Buf updates on your PR. Results from workflow Validate / validate (pull_request).
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThe protobuf definitions now cap pagination-related values at 5000. Six list request Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Adds a maximum bound of 5000, using
buf.validatelterules, to:page_sizeinListUsersRequest,ListProjectsByCurrentUserRequest,ListOrganizationsRequest(frontier.proto)page_sizeinListAllUsersRequest,ListAllOrganizationsRequest,ListAllInvoicesRequest(admin.proto)limitin the sharedRQLRequest(models.proto)Zero still means unset and keeps server-side defaults. Values from 1 to 5000 are accepted as before.
🤖 Generated with Claude Code