chore(open-next): add server-side sentry to open-next site by dario-piotrowicz · Pull Request #8830 · nodejs/nodejs.org

dario-piotrowicz · 2026-04-19T22:59:59Z

Description

This PR adds server-side sentry to the open-next site.

Validation

To be done.

Check List

I have read the Contributing Guidelines and made commit messages that follow the guideline.
I have run pnpm format to ensure the code follows the style guide.
I have run pnpm test to check if all tests are passing.
I have run pnpm build to check if the website builds without errors.
I've covered new added functionality with unit tests if necessary.

vercel · 2026-04-19T23:00:05Z

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

Project	Deployment	Actions	Updated (UTC)
nodejs-org	Ready	Preview	Apr 21, 2026 4:37pm

cursor · 2026-04-19T23:00:09Z

PR Summary

Medium Risk
Changes the Cloudflare Worker entrypoint to wrap requests with Sentry (including sendDefaultPii) and adjusts deployment config; mistakes here could impact production request handling and data captured in telemetry.

Overview
Adds a custom Cloudflare Worker entrypoint (cloudflare/worker-entrypoint.ts) that wraps the OpenNext-generated worker with @sentry/cloudflare for server-side error/performance monitoring, and re-exports DOQueueHandler so Durable Object bindings keep working.

Updates wrangler.jsonc to use the new entrypoint and adds version_metadata (CF_VERSION_METADATA) for Sentry release tagging. Tooling is adjusted to avoid a TypeScript compiler crash by excluding the entrypoint from tsconfig.json and ESLint type-aware linting, and new dev deps (@sentry/cloudflare, @cloudflare/workers-types) plus lockfile updates are added.

^{Reviewed by Cursor Bugbot for commit 3d76a43. Bugbot is set up for automated code reviews on this repo. Configure here.}

github-actions · 2026-04-19T23:00:12Z

👋 Codeowner Review Request

The following codeowners have been identified for the changed files:

Team reviewers: @nodejs/nodejs-website @nodejs/web-infra

Please review the changes when you have a chance. Thank you! 🙏

codecov · 2026-04-19T23:02:52Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.88%. Comparing base (333d70f) to head (3d76a43).
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #8830   +/-   ##
=======================================
  Coverage   73.88%   73.88%           
=======================================
  Files         105      105           
  Lines        8889     8889           
  Branches      326      326           
=======================================
  Hits         6568     6568           
  Misses       2320     2320           
  Partials        1        1

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Dario Piotrowicz <dario.piotrowicz@gmail.com>

cursor

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

There are 2 total unresolved issues (including 1 from previous review).

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

^{Reviewed by Cursor Bugbot for commit 0f161be. Configure here.}

ovflowd

Why are we doing this? Where the need of adding Sentry comes from?

ovflowd · 2026-04-20T14:45:55Z

@dario-piotrowicz please don't use Copilot. We're using Cursor on this repository for now :)

dario-piotrowicz · 2026-04-20T15:15:51Z

@dario-piotrowicz please don't use Copilot. We're using Cursor on this repository for now :)

oh... I actually didn't do anything (at least I don't think so!)... Copilot just automatically reviewed the PR... 😓

ovflowd · 2026-04-20T15:18:04Z

@dario-piotrowicz please don't use Copilot. We're using Cursor on this repository for now :)

oh... I actually didn't do anything (at least I don't think so!)... Copilot just automatically reviewed the PR... 😓

No? You explicitly requested a review from it

ovflowd · 2026-04-20T15:18:27Z

If that's not the case, I'd recommend you to review your Copilot settings, IDK 😅

dario-piotrowicz · 2026-04-20T15:21:56Z

no, I didn't... 😕 😓 🤷

ovflowd · 2026-04-20T15:23:53Z

no, I didn't... 😕 😓 🤷

dario-piotrowicz · 2026-04-20T15:41:34Z

If that's not the case, I'd recommend you to review your Copilot settings, IDK 😅

Maybe this was the problem 🤷

I've disabled it now 😅👍

ovflowd · 2026-04-20T16:01:24Z

+    dsn: 'https://examplePublicKey@o0.ingest.sentry.io/0',
+    // Adds request headers and IP for users, for more info visit:
+    // https://docs.sentry.io/platforms/javascript/guides/cloudflare/configuration/options/#sendDefaultPii
+    sendDefaultPii: true,


@dario-piotrowicz disable this please

Why do we need to disable this? This is pretty important for allowing Sentry to properly tell us the scope of an issue in terms of user impact etc.

mh... shall we disable it just to get sentry integrated and possibly re-enable it later when/if we want? 🤔

ovflowd · 2026-04-20T16:02:25Z

@dario-piotrowicz we had Sentry in the past with good chunk of customization, could you please go through GitHub history and check it out (just to compare what we had at the time)

We should replicate the settings that we use on the release worker, as we know those settings work for us. I don't think historical settings from the Next.js site make much sense as a reference point, given this is Worker instrumentation.

I don't see settings set for the release worker: https://github.com/nodejs/release-cloudflare-worker/blob/main/src/worker.ts 👀 am I looking in the wrong place? or do you mean those Sentry.setTags?

ovflowd · 2026-04-20T16:02:46Z

+    tracesSampleRate: 1.0,
+  }),
+  {
+    async fetch(


What is this fetch override, ooc? 👀

This fetch calls the actual open-next worker

flakey5 · 2026-04-21T16:44:49Z

+     */
+    SENTRY_DSN?: string;
+  }) => ({
+    dsn: env.SENTRY_DSN,


Will this also need a SENTRY_DSN secret be added to the repo +

nodejs.org/.github/workflows/tmp-cloudflare-open-next-deploy.yml

Lines 63 to 69 in 333d70f

- name: Deploy open-next site

working-directory: apps/site

run: node --run cloudflare:deploy

env:

CF_WORKERS_SCRIPTS_API_TOKEN: ${{ secrets.CLOUDFLARE_API_TOKEN }}

CLOUDFLARE_API_TOKEN: ${{ secrets.CLOUDFLARE_API_TOKEN }}

CLOUDFLARE_ACCOUNT_ID: fb4a2d0f103c6ff38854ac69eb709272

?

I needs to set it in the Cloudflare dashboard, I've done that 🙂

chore(open-next): add server-side sentry to open-next site

b7b2263

Copilot AI review requested due to automatic review settings April 19, 2026 23:00

dario-piotrowicz requested review from a team as code owners April 19, 2026 23:00

Copilot started reviewing on behalf of dario-piotrowicz April 19, 2026 23:00 View session

vercel bot deployed to Preview April 19, 2026 23:01 View deployment

dario-piotrowicz commented Apr 19, 2026

View reviewed changes

Comment thread apps/site/cloudflare/worker-entrypoint.ts Outdated

cursor bot reviewed Apr 19, 2026

View reviewed changes

Comment thread apps/site/cloudflare/worker-entrypoint.ts Outdated

This comment was marked as low quality.

Sign in to view

Update apps/site/cloudflare/worker-entrypoint.ts

0f161be

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Dario Piotrowicz <dario.piotrowicz@gmail.com>

vercel bot deployed to Preview April 19, 2026 23:17 View deployment

cursor bot reviewed Apr 19, 2026

View reviewed changes

Comment thread apps/site/cloudflare/worker-entrypoint.ts Outdated

ovflowd requested changes Apr 20, 2026

View reviewed changes

ovflowd reviewed Apr 20, 2026

View reviewed changes

Comment thread apps/site/cloudflare/worker-entrypoint.ts Outdated

remove sentry * import

3c40dba

vercel bot deployed to Preview April 20, 2026 15:53 View deployment

ovflowd reviewed Apr 20, 2026

View reviewed changes

reduce tracesSampleRate to 0.05

16eb479

vercel bot deployed to Preview April 21, 2026 16:29 View deployment

use SENTRY_DSN from env

3d76a43

vercel bot deployed to Preview April 21, 2026 16:37 View deployment

flakey5 reviewed Apr 21, 2026

View reviewed changes

dario-piotrowicz requested a review from ovflowd April 21, 2026 17:58

	- name: Deploy open-next site
	working-directory: apps/site
	run: node --run cloudflare:deploy
	env:
	CF_WORKERS_SCRIPTS_API_TOKEN: ${{ secrets.CLOUDFLARE_API_TOKEN }}
	CLOUDFLARE_API_TOKEN: ${{ secrets.CLOUDFLARE_API_TOKEN }}
	CLOUDFLARE_ACCOUNT_ID: fb4a2d0f103c6ff38854ac69eb709272

Conversation

dario-piotrowicz commented Apr 19, 2026

Description

Validation

Check List

Uh oh!

vercel bot commented Apr 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

cursor bot commented Apr 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Summary

Uh oh!

github-actions bot commented Apr 19, 2026

👋 Codeowner Review Request

Uh oh!

Uh oh!

codecov bot commented Apr 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Uh oh!

This comment was marked as low quality.

Uh oh!

cursor bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

ovflowd left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

ovflowd commented Apr 20, 2026

Uh oh!

dario-piotrowicz commented Apr 20, 2026

Uh oh!

ovflowd commented Apr 20, 2026

Uh oh!

ovflowd commented Apr 20, 2026

Uh oh!

dario-piotrowicz commented Apr 20, 2026

Uh oh!

ovflowd commented Apr 20, 2026

Uh oh!

dario-piotrowicz commented Apr 20, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

MattIPv4 Apr 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

vercel bot commented Apr 19, 2026 •

edited

Loading

cursor bot commented Apr 19, 2026 •

edited

Loading

codecov bot commented Apr 19, 2026 •

edited

Loading

MattIPv4 Apr 20, 2026 •

edited

Loading