ref(organizations): list orgs from control silo in a single call - #1212
Merged
Conversation
GET /organizations/ is now served from the control silo and returns every org the user belongs to across all regions in one paginated response (getsentry/sentry#112622, #115513). Replace the /users/me/regions/ discovery + per-region fan-out with a single call, routing subsequent org-scoped requests using each org's own links.regionUrl. Self-hosted/monolith deployments serve the same endpoint, so the separate fallback branch is no longer needed. Updates the multi-region mock server to model the control silo's combined org listing (previously it only modeled per-region listings), and rewrites the fan-out-specific unit tests accordingly. Co-authored-by: Cursor <cursoragent@cursor.com>
Contributor
|
Contributor
Codecov Results 📊✅ Patch coverage is 100.00%. Project has 5336 uncovered lines. Coverage diff@@ Coverage Diff @@
## main #PR +/-##
==========================================
+ Coverage 81.81% 81.81% —%
==========================================
Files 423 423 —
Lines 29358 29338 -20
Branches 19126 19113 -13
==========================================
+ Hits 24019 24002 -17
- Misses 5339 5336 -3
- Partials 1988 1987 -1Generated by Codecov Action |
…ate lore orgRole is already typed directly on SentryOrganization, so the Record<string, unknown> cast in the new regionEntries mapping was dead weight left over from before that field existed. Also add missing @param/@returns/@throws to listOrganizationsPage. Updates the now-stale .lore.md entry that documented the Promise.allSettled/hasSuccessfulRegion fan-out pattern removed in the previous commit, so it doesn't mislead future sessions searching for that logic. Co-authored-by: Cursor <cursoragent@cursor.com>
listOrganizationsInRegion had zero production callers left after the control-silo refactor -- listOrganizationsPage (the paginated primitive it wrapped) is the only thing anything still needs. Removed it and its barrel re-export; repointed its dedicated CLI-1CQ guard tests and the multiregion describe block at listOrganizationsPage directly. listOrganizationsPage is now a normal public export (not @internal) since it fills that role. Also reverts the .lore.md edit from the previous commit -- that file is meant to be read for context, not edited by hand. Co-authored-by: Cursor <cursoragent@cursor.com>
betegon
marked this pull request as ready for review
July 9, 2026 11:47
betegon
added a commit
that referenced
this pull request
Jul 9, 2026
The Promise.allSettled/hasSuccessfulRegion pattern this entry documented no longer exists in listOrganizationsUncached. Update it to point at the control-silo refactor (#1212) instead of leaving stale guidance that references removed code. Co-authored-by: Cursor <cursoragent@cursor.com>
betegon
added a commit
that referenced
this pull request
Jul 9, 2026
## Summary Follow-up to #1212 (already merged) — that PR removed the `Promise.allSettled`/`hasSuccessfulRegion`/`lastScopeError` fan-out logic this `.lore.md` entry documented, but the entry landed in a commit pushed after the PR had already merged, so it never made it into `main`. Updates the entry to point at the control-silo refactor instead of leaving guidance that references removed code. ## Test plan - N/A (docs-only change to `.lore.md`) Made with [Cursor](https://cursor.com) Co-authored-by: Cursor <cursoragent@cursor.com>
5 tasks
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 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.
Summary
GET /organizations/is now served from the control silo and returns every org the user belongs to across all regions in one paginated response (getsentry/sentry#112622, getsentry/sentry#115513). This replaces the/users/me/regions/discovery + per-region fan-out inlistOrganizationsUncached()with a single call, routing subsequent org-scoped requests using each org's ownlinks.regionUrlfrom the response.Promise.allSettledtracking) since there's only one auth check now instead of N per-region ones.Mirrors the same change the Rust
sentry-clijust made in getsentry/sentry-cli#3352.Test plan
pnpm exec tsc --noEmitpnpm exec biome checkpnpm vitest run test/lib/api-client.multiregion.test.ts test/lib/api/organizations.test.ts test/e2e/multiregion.test.ts— rewrote the fan-out-specific unit tests and confirmed the e2e suite (US + EU orgs, region routing fororg view/project list/issue list, self-hosted fallback) still passes end to endpnpm vitest run— no new failures vs.main(confirmed viagit stashcomparison)Made with Cursor