Skip to content

docs(js): use notSupportedCategories for browser-only integrations - #19499

Merged
mydea merged 2 commits into
masterfrom
guide-cat/08-browser-integrations
Sep 21, 2026
Merged

mydea merged 2 commits into
masterfrom
guide-cat/08-browser-integrations

Conversation

@mydea

@mydea mydea commented Sep 18, 2026 •

Copy link
Copy Markdown
Member

DESCRIBE YOUR PR

Stacked PR. Part of the JavaScript guide-categories series (#19492 → #19506). Depends on #19498; review/merge bottom-up.

Convert the browser-only integration pages (breadcrumbs, browserapierrors, browserprofiling, replay, contextlines, globalhandlers, webworker, …) and best-practices/web-workers to notSupportedCategories: [server-only].

  • Keeps non-category exclusions (cordova/wasm/electron/capacitor) explicit; auto-covers all server-only SDKs and picks up the stale-omitted backends.

IS YOUR CHANGE URGENT?

Help us prioritize incoming PRs by letting us know when the change needs to go live.
Select exactly one option. For deadlines, replace YYYY-MM-DD with the due date. You can update this information later by editing the PR description.

  • Urgent deadline (GA date, etc.): YYYY-MM-DD
  • Other deadline: YYYY-MM-DD
  • No deadline: Not urgent, can wait up to 1 week+

SLA

  • Teamwork makes the dream work, so please add a reviewer to your PRs.
  • Please give the docs team up to 1 week to review your PR unless you've supplied a deadline.

Thanks in advance for your help!

PRE-MERGE CHECKLIST

Make sure you've checked the following before merging your changes:

  • 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

🤖 Generated with Claude Code

@vercel

vercel Bot commented Sep 18, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
sentry-docs Ready Ready Preview Sep 21, 2026 8:36am UTC
1 Skipped Deployment
Project Deployment Actions Updated
develop-docs Ignored Ignored Preview Sep 21, 2026 8:36am UTC

Request Review

@github-actions github-actions Bot added the Priority: Needs Triage The PR description is missing valid priority information label Sep 18, 2026
@mydea
mydea added this pull request to stack #19507 September 18, 2026 10:24
@github-actions github-actions Bot added Priority: Normal Docs review has no urgent deadline and removed Priority: Needs Triage The PR description is missing valid priority information labels Sep 18, 2026
@mydea
mydea force-pushed the guide-cat/08-browser-integrations branch from 7199b3a to e6c5d05 Compare September 18, 2026 10:49
@mydea
mydea marked this pull request as ready for review September 18, 2026 10:50
@codeowner-assignment
codeowner-assignment Bot requested a review from a team September 18, 2026 10:50

@chargome chargome left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mydea
mydea force-pushed the guide-cat/08-browser-integrations branch from e6c5d05 to d72430a Compare September 21, 2026 08:06

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit d72430a. Configure here.

Comment thread docs/platforms/javascript/common/configuration/integrations/globalhandlers.mdx Outdated
mydea and others added 2 commits September 21, 2026 10:30
Replace the enumerated backend-framework `notSupported` lists on browser-only
integration pages (breadcrumbs, browserapierrors, browserprofiling,
browsersession, browsertracing, contextlines, elementtiming, globalhandlers,
graphqlclient, httpclient, httpcontext, modulemetadata, replay, replaycanvas,
reportingobserver, webworker) and best-practices/web-workers with
`notSupportedCategories: [server-only]`, keeping non-category exclusions
(cordova/wasm/electron/capacitor) as explicit keys.

Auto-covers all server-only SDKs and picks up the stale-omitted backend guides
(effect, elysia, firebase, mastra; globalhandlers also cloudflare/deno) that
were previously showing these browser-only integration docs.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
GlobalHandlers works in the Browser and Deno runtimes, but the
`notSupportedCategories: [server-only]` switch hid it on Deno (Deno is
server-only). Use a `browser` category allowlist plus an explicit
`javascript.deno` entry so the page shows on browser platforms and Deno,
while staying hidden on the other server runtimes.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@mydea
mydea force-pushed the guide-cat/08-browser-integrations branch from 666c7ff to bc71ecc Compare September 21, 2026 08:30
@mydea
mydea merged commit 1dcaeb2 into master Sep 21, 2026
22 checks passed
@mydea
mydea deleted the guide-cat/08-browser-integrations branch September 21, 2026 09:04
mydea added a commit that referenced this pull request Sep 21, 2026
…19500)

## DESCRIBE YOUR PR

> **Stacked PR.** Part of the JavaScript guide-categories series (#19492
→ #19506). Depends on #19499; review/merge bottom-up.

Convert the feature-flag integration pages (launchdarkly, openfeature,
statsig, unleash) to `notSupportedCategories: [server-only]`, keeping
browser/desktop exclusions explicit.

- These integrations ship in the browser bundle only. The change also
makes the four pages consistent (statsig previously omitted hono/nitro).

## IS YOUR CHANGE URGENT?

Help us prioritize incoming PRs by letting us know when the change needs
to go live.
Select exactly one option. For deadlines, replace `YYYY-MM-DD` with the
due date. You can update this information later by editing the PR
description.

- [ ] Urgent deadline (GA date, etc.): YYYY-MM-DD
- [ ] Other deadline: YYYY-MM-DD
- [x] No deadline: Not urgent, can wait up to 1 week+

## SLA

- Teamwork makes the dream work, so please add a reviewer to your PRs.
- Please give the docs team up to 1 week to review your PR unless you've
supplied a deadline.

Thanks in advance for your help!

## PRE-MERGE CHECKLIST

_Make sure you've checked the following before merging your changes:_

- [ ] 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)

---

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

This branch was successfully deployed

2 active (1 outdated) deployments
Preview – sentry-docs — bc71ecc5 Deployed Sep 21, 2026 by vercel[bot]
Preview – develop-docs — 7199b3a8 Deployed Sep 18, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Priority: Normal Docs review has no urgent deadline

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants