Repository navigation
feat(api): Read the API base path from the spec's server URL - #20787
Open
sentry-junior[bot] wants to merge 1 commit into
Open
sentry-junior[bot] wants to merge 1 commit into
sentry-junior[bot] wants to merge 1 commit into
Conversation
Sentry's OpenAPI spec is moving /api/0 out of every path and into its
server URL (https://{region}.sentry.io/api/0). The API reference ignored
the spec-level server, so pages and curl examples would lose /api/0.
Join the server URL's path to each operation path, and keep only the
origin as the server. Both spec shapes render the same pages.
Co-Authored-By: BYK <126780+BYK@users.noreply.github.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
|
Getting ready to monitor this change · Rollout details
|
BYK
added a commit
to getsentry/sentry-api-schema
that referenced
this pull request
Nov 13, 2026
…148) **Merge order:** [#148](#148) → [getsentry/sentry-docs#20787](getsentry/sentry-docs#20787) → [getsentry/sentry#131065](getsentry/sentry#131065) → the getsentry/toolkit bump, after `@sentry/api` 2.0.0 is published. Docs has to merge before sentry#131065. When that merges, the spec sync commits the new spec to sentry-api-schema, which auto-releases `@sentry/api` and auto-bumps the schema in sentry-docs. Without sentry-docs#20787 merged first, the docs would drop `/api/0` from paths and curl examples. This PR supersedes #147, which hard-coded `API_PREFIX`. Per @BYK, the server decides the base path. `/api/0` comes from the spec's server URL. It's exported as `API_BASE_PATH`, and every path in the package is relative to it. Nothing here hard-codes `0`. A future `/api/1` spec would produce `API_BASE_PATH = "/api/1"` with no code change. **Design** - `lib/api-base-path.mjs` reads the base path from the path of `servers[0].url`, makes `paths` relative, and trims server URLs to their origin. That includes the per-operation override on `listSeerModels`. - **Until the sentry PR lands,** the base path is the `/api/<version>` prefix that every path shares. It's marked TODO. Both spec shapes build the same `src/`; I checked this with `diff -r`. So the sentry change only syncs a byte-identical package through the bot. - **`baseUrl` stays the Sentry origin.** That's what users have, including self-hosted URLs and `links.regionUrl`. The patched `buildUrl` builds the request URL as origin + `API_BASE_PATH` + operation path. - **Exported** from `@sentry/api`, `@sentry/api/client` and `@sentry/api/operations`. - **The four `/api/0` copies are gone:** in `core.ts`, `buildSentryApiUrl`, `parseApiPathScope` and cache invalidation. So are the copies in the idempotent matcher, `normalize-spec` and `source-analysis`. Matching still finds the base path anywhere in the URL, as before. **Before / After** ```ts // @sentry/api/operations { functionName: "listOrganizations", path: "/api/0/organizations/" } { functionName: "listOrganizations", path: "/organizations/" } import { API_BASE_PATH } from "@sentry/api"; // "/api/0", from servers[0].url // raw requests: url is relative to API_BASE_PATH sentry.client.prepareRequest({ method: "GET", url: "/api/0/organizations/" }); // before sentry.client.prepareRequest({ method: "GET", url: "/organizations/" }); // after // both send https://sentry.io/api/0/organizations/ ``` **Breaking: major bump to 2.0.0** - `operations[].path` and `IdempotentOperation.path` are relative. Build-time lists derived from the catalog must be regenerated, such as toolkit's `idempotent-operations.json`. - Raw `client.request`, `client.get` and `prepareRequest` take relative `url`s. An old `/api/0/...` URL now gets a doubled base path. - Generated function names, types and the request URLs of typed operations don't change: `sentry-client.gen.ts` is byte-identical. Co-Authored-By: BYK <126780+BYK@users.noreply.github.com> <!-- junior-request-attribution:start --> via @BYK. <!-- junior-request-attribution:end --> <!-- junior-session-footer:start --> <!-- junior-conversation-id:agent%3A640c5343d03e547097aa9b8a711daaf5 --> -- [View Session](https://junior-prod.sentry.dev/conversations/agent%3A640c5343d03e547097aa9b8a711daaf5) [[Sentry]](https://sentry.sentry.io/explore/conversations/agent%3A640c5343d03e547097aa9b8a711daaf5/?project=4510944073809921) <!-- junior-session-footer:end --> --------- Co-authored-by: sentry-junior[bot] <264270552+sentry-junior[bot]@users.noreply.github.com> Co-authored-by: BYK <126780+BYK@users.noreply.github.com>
BYK
approved these changes
Nov 13, 2026
This branch was successfully deployed
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
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.
DESCRIBE YOUR PR
Sentry's OpenAPI spec is moving
/api/0out of every path and into its server URL, so clients read the base path (and the API version in it) from the spec (getsentry/sentry-api-schema#148, then the getsentry/sentry change). The API reference ignored the spec-levelservers, so after that change, pages and curl examples would drop/api/0.apiLocation()joins the server URL's path to each operation path. The server it shows is only the origin, stillhttps://sentry.ioby default, or the operation's own server.Before, with the new spec shape:
After, with either spec shape:
No visual change, so no screenshots. I ran
apiCategories()against a spec built from getsentry/sentry master and against the same spec built with the change. The page data was byte-identical, including the per-operationhttps://{region}.sentry.ioserver on/api/0/seer/models/.Checks: the
resolveOpenAPI.spec.tsvitest suite passes, with one new table test covering both spec shapes and an operation-level server. ESLint and Prettier pass on the changed files.IS YOUR CHANGE URGENT?
It does block the getsentry/sentry spec change from merging.
PRE-MERGE CHECKLIST
via @BYK.
--
View Session [Sentry]