Uh oh!
There was an error while loading. Please reload this page.
Merge test to main - #172
Merged
Merged
Conversation
* feat: restrict account_id in GET /api/chats to ORG/RECOUP keys - Make account_id query parameter optional in /api/chats - Personal API keys can no longer specify account_id (403 error) - Org keys can only filter by account_id for org members - Recoup admin keys can filter by any account_id - Personal keys: Returns only their own chats - Org keys: Returns all org member chats - Recoup admin: Returns all chats Uses snake_case naming (account_id, artist_id, org_id) to match API docs. This matches the authorization pattern used in /api/pulses. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * fix: org key without account_id now returns all org member chats The rooms table has no FK from account_id to accounts, so the join approach was silently returning empty results. Changed to fetch org member account IDs via getAccountOrganizations first, then filter rooms. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
Contributor
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Uh oh!
There was an error while loading. Please reload this page.
Caution Review failedThe pull request is closed. 📝 WalkthroughWalkthroughThis PR refactors chat retrieval request handling by introducing comprehensive request validation and authorization logic supporting three key types (personal, organization, and Recoup admin). It replaces direct query validation with a layered approach that validates authentication, checks organizational membership, and builds appropriate database query parameters using an updated Changes
Sequence Diagram(s)sequenceDiagram
participant Client
participant Handler as getChatsHandler
participant ValidateReq as validateGetChatsRequest
participant BuildParams as buildGetChatsParams
participant Auth as validateAuthContext
participant OrgService as getAccountOrganizations
participant DB as selectRooms
Client->>Handler: GET /api/chats?account_id=...
Handler->>ValidateReq: validateGetChatsRequest(request)
ValidateReq->>Auth: validateAuthContext(request)
Auth-->>ValidateReq: {account_id, org_id, ...}
alt Personal Key (no org_id)
ValidateReq->>BuildParams: buildGetChatsParams({account_id, org_id: null, target_account_id})
BuildParams-->>ValidateReq: {params: {account_ids: [account_id]}, error: null}
else Organization Key
ValidateReq->>BuildParams: buildGetChatsParams({account_id, org_id, target_account_id})
BuildParams->>OrgService: getAccountOrganizations(org_id)
OrgService-->>BuildParams: [member_ids...]
BuildParams-->>ValidateReq: {params: {account_ids: [members]}, error: null}
else Recoup Admin Key
ValidateReq->>BuildParams: buildGetChatsParams({account_id, org_id: RECOUP_ORG_ID})
BuildParams-->>ValidateReq: {params: {artist_id}, error: null}
end
ValidateReq-->>Handler: SelectRoomsParams | NextResponse
Handler->>DB: selectRooms(params)
DB-->>Handler: Room[] | null
Handler-->>Client: JSON response
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Poem
✨ Finishing touches
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 |
Braintrust eval reportCatalog Opportunity Analysis Evaluation (HEAD-1769701381)
Catalog Songs Count Evaluation (HEAD-1769701381)
First Week Album Sales Evaluation (HEAD-1769701381)
Memory & Storage Tools Evaluation (HEAD-1769701381)
Monthly Listeners Tracking Evaluation (HEAD-1769701381)
Search Web Tool Evaluation (HEAD-1769701381)
Social Scraping Evaluation (HEAD-1769701381)
Spotify Followers Evaluation (HEAD-1769701381)
Spotify Tools Evaluation (HEAD-1769701381)
TikTok Analytics Questions Evaluation (HEAD-1769701381)
|
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 freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Merge test branch into main after PR #171
Summary by CodeRabbit
Release Notes
Tests
New Features
Refactor
✏️ Tip: You can customize this high-level summary in your review settings.