Conversation
JPeer264
added this pull request to stack #24600
September 22, 2026 16:42
Contributor
size-limit report 📦
|
JPeer264
force-pushed
the
jp/react-router-8-cloudflare-into-framework
branch
from
September 23, 2026 05:27
13b24d7 to
aa5e441
Compare
JPeer264
force-pushed
the
jp/react-router-8-cloudflare-into-framework
branch
from
September 23, 2026 07:57
aa5e441 to
5638fe5
Compare
JPeer264
force-pushed
the
jp/react-router-8-cloudflare-into-framework
branch
from
September 23, 2026 09:12
5638fe5 to
1289b2e
Compare
JPeer264
removed this pull request from stack #24600
September 23, 2026 09:14
JPeer264
added this pull request to stack #24613
September 23, 2026 09:15
JPeer264
removed this pull request from stack #24613
September 23, 2026 09:51
JPeer264
added this pull request to stack #24633
September 23, 2026 09:51
JPeer264
force-pushed
the
jp/react-router-8-cloudflare-into-framework
branch
from
September 23, 2026 10:04
1289b2e to
8128af7
Compare
JPeer264
force-pushed
the
jp/react-router-8-cloudflare-into-framework
branch
from
September 23, 2026 12:25
8128af7 to
cbc7810
Compare
JPeer264
force-pushed
the
jp/react-router-8-cloudflare-into-framework
branch
2 times, most recently
from
September 23, 2026 13:31
fc013ea to
d067bb5
Compare
JPeer264
force-pushed
the
jp/react-router-8-cloudflare-into-framework
branch
from
September 23, 2026 16:19
d067bb5 to
a8dfce1
Compare
JPeer264
removed this pull request from stack #24633
September 23, 2026 16:19
JPeer264
added this pull request to stack #24670
September 23, 2026 16:19
JPeer264
force-pushed
the
jp/react-router-8-cloudflare-into-framework
branch
from
September 23, 2026 16:40
a8dfce1 to
b8b902f
Compare
JPeer264
force-pushed
the
jp/react-router-8-cloudflare-into-framework
branch
from
September 24, 2026 14:19
b55e889 to
cb692c3
Compare
JPeer264
force-pushed
the
jp/react-router-8-cloudflare-into-framework
branch
2 times, most recently
from
September 25, 2026 15:31
7739767 to
b91a8ac
Compare
JPeer264
force-pushed
the
jp/react-router-8-cloudflare-into-framework
branch
from
September 25, 2026 15:41
b91a8ac to
29eab42
Compare
JPeer264
force-pushed
the
jp/react-router-8-cloudflare-into-framework
branch
3 times, most recently
from
September 27, 2026 13:47
dc84aab to
11257ca
Compare
JPeer264
force-pushed
the
jp/react-router-8-cloudflare-into-framework
branch
2 times, most recently
from
September 27, 2026 14:57
82884ff to
fb18e31
Compare
JPeer264
force-pushed
the
jp/react-router-8-cloudflare-into-framework
branch
2 times, most recently
from
September 28, 2026 08:07
6323645 to
e3b7e76
Compare
JPeer264
force-pushed
the
jp/react-router-8-cloudflare-into-framework
branch
3 times, most recently
from
September 28, 2026 09:38
910920b to
7479e87
Compare
JPeer264
force-pushed
the
jp/react-router-8-cloudflare-into-framework
branch
3 times, most recently
from
September 28, 2026 14:29
e781deb to
933683f
Compare
Under `bun run` the SDK cannot inject the diagnostics channels into
libraries (JS-3508), so Bun apps must be built with `@sentry/bun/plugin`
to get framework, database and AI spans. The new `node-suites-bun-build`
project tests that path: the runner builds each scenario with the plugin
right before it starts (`RUNTIME_BUILD_SCRIPT`), and runs the bundle
with `@sentry/bun`.
The Docker suites run as well, so the Bun CI job gets the disk space
step of the Node and Deno jobs. `pg-native` and the CommonJS
`postgresjs` tests are skipped on Bun: the `libpq` addon needs a Node
symbol that Bun does not have, and through the `bun` export condition
`require('postgres')` returns the ES module namespace.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…amework Runs the same Playwright suite on Bun, Deno and Cloudflare (local workerd) as optional variants of the existing app, instead of one app per runtime. A variant can set a `runtime` matrix key (`bun` or `deno`), and CI then installs that runtime for the job. The Cloudflare build uses its own Vite config with `@cloudflare/vite-plugin` and `sentryCloudflareVitePlugin`, and swaps in a workerd server entry with the new `runtimeEntryPlugin` from `@sentry-internal/test-utils/vite`. The Cloudflare variant needed these SDK fixes: - react-router: resolve the Cloudflare entry under the `workerd` condition, so `Sentry.flush` and `Sentry.startSpan` exist in a Worker. - react-router: export `lowQualityTransactionsFilterIntegration` from `/cloudflare`. - react-router: do not write trace meta tags into prerendered pages on Cloudflare, which otherwise pinned every visitor of the static page to one build-time trace. Under `bun run` Express is not instrumented, so on Bun the error transaction stays the request path. On Cloudflare the error has no transaction. On both, the meta tag names the http.server segment. The Bun variant relies on the diagnostics channel fixes of the Bun auto-instrumentation suites PR it is based on. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…er-8-framework The Cloudflare variant of react-router-8-framework now covers what the separate react-router-8-cloudflare app tested, so the MySQL route and tests move over and the old app is removed. The tests now also run on Node, Bun and Deno. The Worker is wrapped by `sentryCloudflareVitePlugin` instead of a manual `withSentry` call. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
JPeer264
force-pushed
the
jp/react-router-8-cloudflare-into-framework
branch
from
September 29, 2026 06:25
933683f to
492d9ef
Compare
This branch has not been 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
1 participant
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.
With the Cloudflare variant of
react-router-8-framework, the separatereact-router-8-cloudflareapp only added its MySQL test. This moves the MySQL route and both tests intoreact-router-8-frameworkand removes the old app, so one app covers React Router on Node, Deno and Cloudflare.The MySQL tests now run on every variant, not only on Cloudflare. The Worker is wrapped by
sentryCloudflareVitePlugininstead of a manualwithSentrycall; other Cloudflare apps still cover manualwithSentry.🤖 Generated with Claude Code