feat(nitro): Nitro SDK by logaretm · Pull Request #19224 · getsentry/sentry-javascript

logaretm · 2026-02-06T19:47:42Z

This PR just isolates the mundane changes needed for a new SDK to keep the next stacked PRs clean, it adds the Nitro SDK to the monorepo.

This PR is a base of a stack, the stacked PRs will be merged into it. I thought this will be easier to review.

This PR is part of a stack:

github-actions · 2026-02-09T19:36:52Z

Codecov Results 📊

Generated by Codecov Action

github-actions · 2026-02-09T19:38:31Z

node-overhead report 🧳

Note: This is a synthetic benchmark with a minimal express app and does not necessarily reflect the real-world performance impact in an application.

Scenario	Requests/s	% of Baseline	Prev. Requests/s	Change %
GET Baseline	10,850	-	9,029	+20%
GET With Sentry	1,689	16%	1,578	+7%
GET With Sentry (error only)	7,026	65%	5,938	+18%
POST Baseline	1,044	-	1,193	-12%
POST With Sentry	490	47%	566	-13%
POST With Sentry (error only)	915	88%	1,035	-12%
MYSQL Baseline	3,653	-	3,285	+11%
MYSQL With Sentry	360	10%	405	-11%
MYSQL With Sentry (error only)	2,881	79%	2,652	+9%

View base workflow run

github-actions · 2026-02-19T15:49:17Z

size-limit report 📦

Path	Size	% Change	Change
@sentry/browser	25.88 kB	-	-
@sentry/browser - with treeshaking flags	24.35 kB	-	-
@sentry/browser (incl. Tracing)	43.8 kB	-	-
@sentry/browser (incl. Tracing + Span Streaming)	45.5 kB	-	-
@sentry/browser (incl. Tracing, Profiling)	48.73 kB	-	-
@sentry/browser (incl. Tracing, Replay)	82.98 kB	-	-
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags	72.5 kB	-	-
@sentry/browser (incl. Tracing, Replay with Canvas)	87.67 kB	-	-
@sentry/browser (incl. Tracing, Replay, Feedback)	99.93 kB	-	-
@sentry/browser (incl. Feedback)	42.7 kB	-	-
@sentry/browser (incl. sendFeedback)	30.55 kB	-	-
@sentry/browser (incl. FeedbackAsync)	35.55 kB	-	-
@sentry/browser (incl. Metrics)	27.16 kB	-	-
@sentry/browser (incl. Logs)	27.29 kB	-	-
@sentry/browser (incl. Metrics & Logs)	27.98 kB	-	-
@sentry/react	27.62 kB	-	-
@sentry/react (incl. Tracing)	46.05 kB	-	-
@sentry/vue	30.71 kB	-	-
@sentry/vue (incl. Tracing)	45.62 kB	-	-
@sentry/svelte	25.89 kB	-	-
CDN Bundle	28.57 kB	-	-
CDN Bundle (incl. Tracing)	46.08 kB	-	-
CDN Bundle (incl. Logs, Metrics)	29.95 kB	-	-
CDN Bundle (incl. Tracing, Logs, Metrics)	47.12 kB	-	-
CDN Bundle (incl. Replay, Logs, Metrics)	68.92 kB	-	-
CDN Bundle (incl. Tracing, Replay)	83.14 kB	-	-
CDN Bundle (incl. Tracing, Replay, Logs, Metrics)	84.17 kB	-	-
CDN Bundle (incl. Tracing, Replay, Feedback)	88.61 kB	-	-
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics)	89.69 kB	-	-
CDN Bundle - uncompressed	83.59 kB	-	-
CDN Bundle (incl. Tracing) - uncompressed	137.62 kB	-	-
CDN Bundle (incl. Logs, Metrics) - uncompressed	87.73 kB	-	-
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed	141.03 kB	-	-
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed	211.31 kB	-	-
CDN Bundle (incl. Tracing, Replay) - uncompressed	255.06 kB	-	-
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed	258.46 kB	-	-
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed	267.97 kB	-	-
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed	271.36 kB	-	-
@sentry/nextjs (client)	48.58 kB	-	-
@sentry/sveltekit (client)	44.22 kB	-	-
@sentry/node-core	58.35 kB	+0.02%	+9 B 🔺
@sentry/node	175.66 kB	+0.01%	+13 B 🔺
@sentry/node - without tracing	98.3 kB	+0.02%	+11 B 🔺
@sentry/aws-serverless	115.33 kB	+0.02%	+14 B 🔺

View base workflow run

github-actions · 2026-03-17T14:22:59Z

This pull request has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you apply the label PR: no-auto-close I will leave it alone ... forever!

Copilot

Pull request overview

Adds a new @sentry/nitro package skeleton to the monorepo and wires it into the repo’s release/testing/triage infrastructure as the base for a stacked series of Nitro SDK PRs.

Changes:

Introduces the new packages/nitro package with initial SDK/config scaffolding, build config, and a placeholder test.
Registers the package across monorepo plumbing (workspace list, root README, issue templates/labeling, release config).
Updates lockfile for the newly introduced dependency graph.

Reviewed changes

Copilot reviewed 23 out of 24 changed files in this pull request and generated 12 comments.

Show a summary per file

File	Description
yarn.lock	Lockfile updates due to dependency graph changes.
packages/nitro/package.json	New package manifest, exports, build/test scripts.
packages/nitro/rollup.npm.config.mjs	Rollup config for publishing builds (ESM-only).
packages/nitro/tsconfig.json	Package TS config (module/moduleResolution overrides).
packages/nitro/tsconfig.types.json	Declaration-only build config for emitted types.
packages/nitro/tsconfig.test.json	Test TS config for package tests.
packages/nitro/src/index.ts	Public entrypoint re-exporting config + `@sentry/node` + Nitro `init`.
packages/nitro/src/sdk.ts	Nitro SDK initialization wrapper around `@sentry/node`.
packages/nitro/src/config.ts	`withSentryConfig` / module setup helper for Nitro config.
packages/nitro/src/module.ts	Nitro module factory placeholder.
packages/nitro/src/common/debug-build.ts	Debug-build flag constant (pattern consistent with other packages).
packages/nitro/src/utils/resolver.ts	Path resolver helper utility.
packages/nitro/src/utils/plugin.ts	Helper to add Nitro plugins.
packages/nitro/test/index.test.ts	Placeholder Vitest test.
packages/nitro/test/tsconfig.json	Test tsconfig shim extending package test config.
packages/nitro/README.md	Initial package documentation.
packages/nitro/LICENSE	Package license file.
packages/nitro/.eslintrc.js	Package-local ESLint config extending repo base.
package.json	Adds `packages/nitro` to workspace list.
dev-packages/e2e-tests/verdaccio-config/config.yaml	Allows publishing `@sentry/nitro` in Verdaccio E2E setup.
README.md	Adds `@sentry/nitro` to root package list.
.github/workflows/issue-package-label.yml	Adds Nitro to package-to-label mapping.
.github/ISSUE_TEMPLATE/bug.yml	Adds `@sentry/nitro` to bug template SDK dropdown.
.craft.yml	Adds Nitro package to Craft release targets/metadata.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

logaretm · 2026-04-16T14:00:04Z

Thank you for all comments @s1gr1d very thorough review! all addressed 🙏

- Update LICENSE copyright year to 2026 - Expand peerDependencies range to cover nitro prereleases - Bump nitro devDependency to latest beta - Add beta notice to README - Improve README wording (section title, Vite capitalization, --import mention) - Point .craft.yml docs URL to package README until docs page exists - Include 'node' in applySdkMetadata packages list for consistency Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

The repo uses Oxlint, not ESLint. Remove the legacy .eslintrc.js file. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Implements HTTP server instrumentation for both `h3` and `srvx` by listening to their tracing channel events. - `h3` TC PR: h3js/h3#1251 - `srvx` TC PR: h3js/srvx#141Closes#18123 --- **This PR is part of a stack:** - #20358 - #19224 - #19225 👈 - #19304 --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Adds automatic sourcemap handling to the Nitro SDK, using `@sentry/bundler-plugin-core` for builder-agnostic post-build upload. Nitro uses rollup or rolldown, so it made sense to make it as agnostic as possible. Closes#17992 --- **This PR is part of a stack:** - #20358 - #19224 - #19225 - #19304 👈 --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Nitro 3 is ESM-only, so the cjs output was shipping dead code. Also removes a broken typesVersions entry pointing at build/types/plugins.d.ts (file never existed, no consumer imports @sentry/nitro/plugins). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Align with Bun/Deno/Cloudflare SDKs by guarding the default integrations computation behind an undefined check instead of relying on spread order. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

The URL constructor throws on relative paths like `/api/users`, causing the transaction name and root span updates to be silently skipped. Use parseUrl from @sentry/core, which handles relative URLs correctly. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

@sentry

E2E tests now install @sentry packages from packed tarballs under dev-packages/e2e-tests/packed/ rather than a local verdaccio registry, so the .npmrc and "latest || *" version specifiers are no longer needed. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

CI's ciPnpmOverrides script already pins @sentry/* deps to the packed tarballs, so the "latest || *" specifiers are sufficient. Keep only the .npmrc removal from the previous commit. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

- Add `keywords`, `lint:fix` script, and `nx.targets` build config to package.json - Link to the nitro-3 e2e app from the README - Point `.craft.yml` `mainDocsUrl` to the docs page - Add a CHANGELOG unreleased entry for the new SDK Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Remove the `exclude` and "built separately" comment referencing `src/setup.ts` and `tsconfig.setup-types.json`, both of which no longer exist in the package. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

cursor

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

There are 4 total unresolved issues (including 3 from previous reviews).

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

^{Reviewed by Cursor Bugbot for commit 000b7b0. Configure here.}

There is no stable stream of Nitro 3 canary builds yet, so no point in having this. I will add it back once we have a stable stream of builds, likely after the full v3 release. I probably had it at one point then didn't remove it properly before merging #19224Closes#20472

logaretm mentioned this pull request Feb 6, 2026
feat(nitro): Instrument HTTP Server #19225
Merged

logaretm force-pushed the awad/create-nitro-sdk-metadata branch from 81b555e to 161862dCompare February 10, 2026 19:31

logaretm mentioned this pull request Feb 12, 2026
feat(nitro): Handle sourcemap preparation and upload #19304
Merged

logaretm force-pushed the awad/create-nitro-sdk-metadata branch 2 times, most recently from 1317257 to 2a86160Compare February 23, 2026 14:51

github-actionsBot added the PR: stale label Mar 17, 2026

logaretm added PR: no-auto-close and removed PR: stale labels Mar 17, 2026

logaretm force-pushed the awad/create-nitro-sdk-metadata branch from 99ea611 to 9ded4c9Compare April 15, 2026 13:10

logaretm changed the title ~~feat(nitro): Nitro SDK entry and metadata~~feat(nitro): Nitro SDKApr 15, 2026

semgrep-code-getsentryBot reviewed Apr 15, 2026
View reviewed changes

Comment threadyarn.lock Outdated

logaretm force-pushed the awad/create-nitro-sdk-metadata branch from 53bf486 to 5f43f63Compare April 15, 2026 13:15

logaretm marked this pull request as ready for review April 15, 2026 17:16

logaretm requested review from JPeer264, Copilot and s1gr1d April 15, 2026 17:16

Copilotstarted reviewing on behalf of logaretm April 15, 2026 17:17 View session

cursorBot reviewed Apr 15, 2026
View reviewed changes

Comment threadpackages/nitro/src/sdk.ts Outdated
Comment threadpackages/nitro/test/index.test.ts
Comment threadpackages/nitro/package.json Outdated
Comment threadpackages/nitro/tsconfig.types.json Outdated

CopilotAI reviewed Apr 15, 2026
View reviewed changes

s1gr1d reviewed Apr 16, 2026
View reviewed changes

logaretm requested a review from s1gr1d April 16, 2026 13:59

cursorBot reviewed Apr 16, 2026
View reviewed changes

Comment threadpackages/nitro/src/index.ts

logaretm force-pushed the awad/create-nitro-sdk-metadata branch from 5f3baf0 to 1dad524Compare April 16, 2026 14:53

logaretm changed the base branch from develop to awad/tracing-channel-otel-vendoringApril 16, 2026 14:53

logaretm force-pushed the awad/create-nitro-sdk-metadata branch from 1dad524 to 120ecbaCompare April 16, 2026 16:19

logaretmand others added 16 commits April 23, 2026 09:56

fix: satisfy the test runner tests
bc55f26

fix: lint
4ae2b1d

fix: dedup
24a438a

chore: bump
6cf301b

chore: dedup deps
c6d4918

chore: update other metadata
0454a21

chore(nitro): remove leftover eslintrc config
bc68aff
The repo uses Oxlint, not ESLint. Remove the legacy .eslintrc.js file. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

chore: update packages
5ac385e

chore(nitro): switch lint scripts to oxlint
5fdd0df
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

fix: updated the readme with the correct min version
bb4b1e7

fix(nitro): only compute default integrations when not provided
2d31add
Align with Bun/Deno/Cloudflare SDKs by guarding the default integrations computation behind an undefined check instead of relying on spread order. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

chore: update package versions
a62ff3c

logaretm force-pushed the awad/create-nitro-sdk-metadata branch from 830ac6e to a62ff3cCompare April 23, 2026 13:57

logaretmand others added 2 commits April 23, 2026 10:47

cursorBot reviewed Apr 23, 2026
View reviewed changes

Comment threadpackages/nitro/src/runtime/hooks/captureErrorHook.ts

logaretmand others added 4 commits April 23, 2026 11:11

chore(nitro): clean up stale tsconfig.types.json references
2e68a40
Remove the `exclude` and "built separately" comment referencing `src/setup.ts` and `tsconfig.setup-types.json`, both of which no longer exist in the package. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

fix: format
000b7b0

cursorBot reviewed Apr 23, 2026
View reviewed changes

Comment threadpackages/nitro/src/runtime/hooks/captureErrorHook.ts

logaretm merged commit 8f32e18 into developApr 23, 2026
484 of 489 checks passed

logaretm deleted the awad/create-nitro-sdk-metadata branch April 23, 2026 16:45

logaretm mentioned this pull request Apr 24, 2026
fix(tests): Remove nitro canary test job #20473
Merged

sentry-release-botBot mentioned this pull request Apr 29, 2026
publish: getsentry/sentry-javascript@10.51.0 getsentry/publish#7981
Closed
47 tasks

Uh oh!

Uh oh!

Conversation

logaretm commented Feb 6, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actionsBot commented Feb 9, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Results 📊

Uh oh!

github-actionsBot commented Feb 9, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

node-overhead report 🧳

Uh oh!

github-actionsBot commented Feb 19, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

size-limit report 📦

Uh oh!

github-actionsBot commented Mar 17, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

logaretm commented Apr 16, 2026

Uh oh!

Uh oh!

Uh oh!

cursorBot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

logaretm commented Feb 6, 2026•
edited
Loading

github-actionsBot commented Feb 9, 2026•
edited
Loading

github-actionsBot commented Feb 9, 2026•
edited
Loading

github-actionsBot commented Feb 19, 2026•
edited
Loading