Skip to content

docs(js): correct runtime support for Node integrations via categories - #19504

Draft
mydea wants to merge 2 commits into
masterfrom
guide-cat/13-node-integrations
Draft

mydea wants to merge 2 commits into
masterfrom
guide-cat/13-node-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 #19503; review/merge bottom-up.

IMPORTANT: This is for v11 and should only be merged when v11 is done.

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.

  • Fixes stale hand-maintained lists: Node frameworks omitted from the old lists now show correctly, and Bun/Deno (which now export most Node integrations) are 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.

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 22, 2026 1:35pm UTC
1 Skipped Deployment
Project Deployment Actions Updated
develop-docs Ignored Ignored Preview Sep 22, 2026 1:35pm UTC

Request Review

@mydea
mydea added this pull request to stack #19507 September 18, 2026 10:24
@github-actions github-actions Bot added the Priority: Needs Triage The PR description is missing valid priority information label Sep 18, 2026
@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/13-node-integrations branch from bce869c to 245df10 Compare September 18, 2026 10:50
@mydea
mydea force-pushed the guide-cat/13-node-integrations branch from 245df10 to 7215407 Compare September 21, 2026 08:07
@mydea
mydea force-pushed the guide-cat/13-node-integrations branch from 7215407 to 3d7584c Compare September 21, 2026 08:18
@mydea
mydea force-pushed the guide-cat/13-node-integrations branch from 3d7584c to ed84948 Compare September 21, 2026 08:30
@mydea
mydea force-pushed the guide-cat/13-node-integrations branch from ed84948 to 40d4f62 Compare September 21, 2026 09:04

@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 40d4f62. Configure here.

@mydea
mydea removed this pull request from stack #19507 September 22, 2026 13:07
@mydea
mydea added this pull request to stack #19556 September 22, 2026 13:08
@andreiborza
andreiborza marked this pull request as ready for review September 22, 2026 13:16
@codeowner-assignment
codeowner-assignment Bot requested a review from a team September 22, 2026 13:17
Base automatically changed from guide-cat/12-supabase to master September 22, 2026 13:22
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
mydea force-pushed the guide-cat/13-node-integrations branch from 40d4f62 to 9bc5ad5 Compare September 22, 2026 13:22
@mydea
mydea marked this pull request as draft September 22, 2026 13:24
Elysia runs on the Bun runtime (fallbackGuide: javascript.bun), so any
integration not supported on Bun is likewise unsupported on Elysia. Add
javascript.elysia alongside javascript.bun in notSupported for the
node-only native integrations.

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 95495f23 Deployed Sep 22, 2026 by vercel[bot]
Preview – develop-docs bce869c6 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

js-v11-docs Priority: Normal Docs review has no urgent deadline

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant