fix(js): Show orgId option for all JS SDK categories#17724
Merged
Conversation
…K categories Both options were restricted to server/serverless via categorySupported, but they are defined in @sentry/core and available in browser SDKs too. Remove the categorySupported filter so these show on browser/frontend platform pages as well. Also removes a duplicate sentence in the orgId description.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
strictTraceContinuation guards against foreign inbound trace context, which is a server concern — browsers initiate requests rather than receive inbound traces from third parties.
sentry-junior
Bot
changed the title
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit c8dfb1d. Configure here.
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
The link targets an anchor that only exists on server/serverless pages, so wrap it so browser pages don't get a broken link.
andreiborza
approved these changes
May 12, 2026
Contributor
How so? |
sentrivana
pushed a commit
that referenced
this pull request
May 18, 2026
## DESCRIBE YOUR PR `orgId` was restricted to server/serverless categories via `categorySupported` on the JS SDK configuration options page, but it's defined in `@sentry/core` and available in browser SDKs too — the org ID is embedded in the DSC/baggage on outgoing requests from the browser. This PR removes the `categorySupported` filter from `orgId` so it appears on browser/frontend platform pages as well (e.g. `/platforms/javascript/configuration/options/`). `strictTraceContinuation` stays server/serverless-only since it guards against foreign inbound trace context, which is a server concern. Also cleans up a duplicate sentence in the `orgId` description. ## IS YOUR CHANGE URGENT? - [x] None: Not urgent, can wait up to 1 week+ ## PRE-MERGE CHECKLIST - [ ] Checked Vercel preview for correctness, including links - [ ] PR was reviewed and approved by any necessary SMEs (subject matter experts) - [ ] PR was reviewed and approved by a member of the [Sentry docs team](https://github.com/orgs/getsentry/teams/docs) --- *Opened on behalf of @Swatinem* --------- Co-authored-by: Junior <junior@sentry.io>
6 tasks
antonis
added a commit
that referenced
this pull request
May 19, 2026
orgId categorySupported was already removed in #17724. strictTraceContinuation stays server/serverless-only, aligned with RN approach — the feature is documented in the distributed tracing guide. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
antonis
added a commit
that referenced
this pull request
Jun 10, 2026
## DESCRIBE YOUR PR Add Capacitor-specific distributed tracing documentation with strict trace continuation section. **Changes:** - **`javascript.capacitor.mdx`** (new) — Capacitor-specific distributed tracing include with strict trace continuation section and code examples, aligned with the existing React Native include **Note:** The original PR also removed `categorySupported` from `orgId` and `strictTraceContinuation` in `options.mdx`. Those changes were dropped: - `orgId` was already fixed in #17724 - `strictTraceContinuation` intentionally stays server/serverless-only on the options page, aligned with the RN approach (documented in the distributed tracing guide instead) Related SDK PR: getsentry/sentry-capacitor#1166 ## IS YOUR CHANGE URGENT? - [ ] Urgent deadline (GA date, etc.): - [ ] Other deadline: - [x] None: Not urgent, can wait up to 1 week+ ## PRE-MERGE CHECKLIST - [ ] Checked Vercel preview for correctness, including links - [ ] PR was reviewed and approved by any necessary SMEs (subject matter experts) - [ ] PR was reviewed and approved by a member of the [Sentry docs team](https://github.com/orgs/getsentry/teams/docs) --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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
3 participants
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
orgIdwas restricted to server/serverless categories viacategorySupportedon the JS SDK configuration options page, but it's defined in@sentry/coreand available in browser SDKs too — the org ID is embedded in the DSC/baggage on outgoing requests from the browser.This PR removes the
categorySupportedfilter fromorgIdso it appears on browser/frontend platform pages as well (e.g./platforms/javascript/configuration/options/).strictTraceContinuationstays server/serverless-only since it guards against foreign inbound trace context, which is a server concern.Also cleans up a duplicate sentence in the
orgIddescription.IS YOUR CHANGE URGENT?
PRE-MERGE CHECKLIST
Opened on behalf of @cleptric