Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
mydea
added this pull request to stack #19507
September 18, 2026 10:24
This was referenced Sep 18, 2026
mydea
force-pushed
the
guide-cat/15-async-context
branch
from
September 18, 2026 10:50
e18e4a8 to
869498c
Compare
mydea
marked this pull request as ready for review
September 18, 2026 11:10
6 tasks
chargome
approved these changes
Sep 21, 2026
mydea
force-pushed
the
guide-cat/15-async-context
branch
from
September 21, 2026 08:07
869498c to
438d27c
Compare
mydea
force-pushed
the
guide-cat/15-async-context
branch
from
September 21, 2026 08:18
438d27c to
e2b2c9f
Compare
mydea
added a commit
that referenced
this pull request
Sep 21, 2026
…#19492) ## DESCRIBE YOUR PR > **Stacked PR.** Bottom of the JavaScript guide-categories series (#19492 → #19506), based on `master`. Review/merge bottom-up. Remove the `javascript`, `server-node`, and `server-edge` values from the `categories` frontmatter of all JavaScript platform pages and guides. - These values were declared only in frontmatter and never referenced by any filtering logic (`PlatformCategorySection`, `SdkOption`, `platformLink`, `migrationGuide`), nor were they part of the `PlatformCategory` type. - Removing them aligns the frontmatter data with the type. No functional change — every guide keeps its meaningful tags (`browser`, `server`, `serverless`, `mobile`, `desktop`). ## 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>
mydea
added a commit
that referenced
this pull request
Sep 21, 2026
## DESCRIBE YOUR PR > **Stacked PR.** Part of the JavaScript guide-categories series (#19492 → #19506). Depends on #19492; review/merge bottom-up. Introduce two new `PlatformCategory` values, `browser-only` and `server-only`, and apply them across the JavaScript platform and guides. - `browser-only`: guides that run only in the browser (angular, react, vue, svelte, ember, gatsby, solid, wasm, capacitor, cordova) + the base Browser JavaScript platform. - `server-only`: guides that run only server-side (node, express, nestjs, and the serverless/edge runtimes). - Meta-frameworks that run in both (astro, nextjs, nuxt, remix, react-router, solidstart, sveltekit, tanstackstart-react) and electron get neither. - Also drops the redundant duplicate `categories` blocks from the effect and tanstackstart-react guide `config.yml` files (guide categories are read from `index.mdx`). ## 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>
mydea
added a commit
that referenced
this pull request
Sep 21, 2026
## DESCRIBE YOUR PR > **Stacked PR.** Part of the JavaScript guide-categories series (#19492 → #19506). Depends on #19493; review/merge bottom-up. Node, Bun, and Deno are long-running server runtimes, not serverless/FaaS platforms, so the `serverless` category was semantically wrong for them. - The genuinely serverless guides (aws-lambda, azure-functions, gcp-functions, cloudflare, firebase) keep it. - No rendered content changes: `serverless` is always paired with `server` in every filter these guides retain, and the only `serverless`-alone consumers (migration items) are also gated by `frameworks:`. ## 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>
mydea
added a commit
that referenced
this pull request
Sep 21, 2026
## DESCRIBE YOUR PR > **Stacked PR.** Part of the JavaScript guide-categories series (#19492 → #19506). Depends on #19494; review/merge bottom-up. Extract the repeated `categories.includes('server') || includes('serverless')` logic into shared `hasServerCategory` / `hasBrowserCategory` helpers (`src/categories.ts`) and use them in `sdkOption` and `platformLink`. - Behavior-preserving cleanup; no rendered output changes. - The `nextjs`/`sveltekit` env-var exclusion in `shouldShowEnvVar` is intentionally left as a framework-specific special case. ## 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> Co-authored-by: getsantry[bot] <66042841+getsantry[bot]@users.noreply.github.com>
mydea
added a commit
that referenced
this pull request
Sep 21, 2026
…ts (#19496) ## DESCRIBE YOUR PR > **Stacked PR.** Part of the JavaScript guide-categories series (#19492 → #19506). Depends on #19495; review/merge bottom-up. Key the “Only available on: Client/Server” hint in `getPlatformHints` off the new `isBrowserOnly` / `isServerOnly` category checks instead of hand-rolled runtime detection. - The hint is only informative on dual-runtime (meta-framework) platforms; on a single-runtime platform the option's runtime is already implied, so it's suppressed there. - Note: this also surfaces the hint on `desktop` (electron), which runs both a browser and a Node runtime. ## 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>
mydea
added a commit
that referenced
this pull request
Sep 21, 2026
## DESCRIBE YOUR PR > **Stacked PR.** Part of the JavaScript guide-categories series (#19492 → #19506). Depends on #19496; review/merge bottom-up. Add `supportedCategories` / `notSupportedCategories` frontmatter fields that gate common-content inheritance by platform category, alongside the existing guide-name `supported` / `notSupported` lists. - A page with `supportedCategories` is shown only for platforms/guides whose categories match; `notSupportedCategories` hides matching ones. - Lets pages target e.g. all server SDKs with `notSupportedCategories: [browser]` instead of enumerating every framework, and auto-covers new guides. - Both `isSupported` implementations (doc-tree collector and search collector) resolve categories and apply the new lists. Covered by `commonContentInheritance.spec.ts`. - This is the feature the remaining PRs in the stack build on. ## 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>
mydea
added a commit
that referenced
this pull request
Sep 21, 2026
## DESCRIBE YOUR PR > **Stacked PR.** Part of the JavaScript guide-categories series (#19492 → #19506). Depends on #19497; review/merge bottom-up. Replace the enumerated backend-framework `notSupported` lists on the Session Replay pages with `notSupportedCategories: [server-only]`, keeping non-category exclusions (`cordova`, `wasm`) explicit. - Session Replay is browser-only; the category form auto-covers every server-only SDK and picks up guides that were stale-omitted from the manual lists (effect, firebase, mastra). ## 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>
mydea
force-pushed
the
guide-cat/15-async-context
branch
from
September 21, 2026 08:30
e2b2c9f to
5d89ac8
Compare
mydea
added a commit
that referenced
this pull request
Sep 21, 2026
…19499) ## 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 - [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>
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>
mydea
force-pushed
the
guide-cat/15-async-context
branch
from
September 21, 2026 09:04
5d89ac8 to
c04ad8a
Compare
mydea
removed this pull request from stack #19507
September 22, 2026 13:07
mydea
added this pull request to stack #19556
September 22, 2026 13:08
mydea
added a commit
that referenced
this pull request
Sep 22, 2026
## DESCRIBE YOUR PR > **Stacked PR.** Part of the JavaScript guide-categories series (#19492 → #19506). Depends on #19501; review/merge bottom-up. Replace the enumerated server-framework `supported` lists on server-side integration pages (console, modules, onuncaughtexception, requestdata, trpc, unhandledrejection) with `supportedCategories: [server]`. - Keeps non-server platforms (electron) explicit; covers all server SDKs including meta-frameworks and picks up stale-omitted backends. - Only pages whose omissions were purely stale backends were converted; Node-only integrations are handled separately. ## 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>
Rework the 26 Node auto-instrumentation integration pages to use
`supportedCategories: [server]` with the specific edge runtimes the SDK does
not support listed under `notSupported`, verified against the actual exports
in getsentry/sentry-javascript (packages/{node,bun,deno,cloudflare}).
This fixes stale hand-maintained lists in two ways:
- Node frameworks omitted from the old lists (effect, elysia, firebase,
mastra, nitro, and some meta-frameworks) now correctly show these pages.
- Bun and Deno now export most Node integrations, so pages that excluded them
were wrong; support is corrected per the SDK:
- Exclude Cloudflare only (node+bun+deno): amqplib, dataloader, genericpool,
graphql, kafka, knex, lrumemoizer, mongo, mongoose, mysql, mysql2,
postgres, redis, tedious.
- Exclude Deno+Cloudflare (node+bun): anr, http, fs, localvariables,
nodecontext, pino.
- Exclude Bun+Deno+Cloudflare (node-only/native): nodefetch,
event-loop-block, nodeprofiling, noderuntimemetrics.
- Exclude Deno (node+bun+cloudflare): prisma, prisma__v8.x.
`javascript.electron` is preserved as an explicit `supported` entry where it
was already listed.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
mydea
added a commit
that referenced
this pull request
Sep 22, 2026
## DESCRIBE YOUR PR > **Stacked PR.** Part of the JavaScript guide-categories series (#19492 → #19506). Depends on #19502; review/merge bottom-up. The Supabase integration lives in `@sentry/core` and is re-exported by every SDK, so it is available on both browser and server platforms. - Set `supportedCategories: [browser, server]` (previously server only). - Drop the bogus `javascript.browser` key (not a real guide slug); the browser category now covers all browser SDKs. Keep `javascript.electron` explicit. ## 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>
…ages Replace the enumerated server-framework `supported` lists on crons/*, mcp-monitoring, and the queues/caches instrumentation pages with `supportedCategories: [server]`. These already listed all edge runtimes (cloudflare/deno/bun); the category form additionally covers the guides that were stale-omitted from the manual lists (effect, elysia, firebase, mastra, nitro, and some meta-frameworks). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Async context isolation applies to every server-side runtime, so replace the partial pure-Node backend `supported` list with `supportedCategories: [server]`. This adds the pages for the meta-frameworks, edge runtimes (bun/deno/cloudflare), and serverless targets that were previously missing. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
mydea
force-pushed
the
guide-cat/15-async-context
branch
from
September 22, 2026 13:22
c04ad8a to
c8d828c
Compare
coolguyzone
approved these changes
Sep 23, 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
Async context isolation applies to every server-side runtime, so replace the partial pure-Node backend
supportedlist withsupportedCategories: [server].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-DDwith the due date. You can update this information later by editing the PR description.SLA
Thanks in advance for your help!
PRE-MERGE CHECKLIST
Make sure you've checked the following before merging your changes:
🤖 Generated with Claude Code