You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Commit a4762fe
Browse filesBrowse the repository at this point in the historyBrowse files
fix(orchestrion): Stop force-bundling instrumented deps in dev servers (#23835)
The Vite plugin's `ssr.noExternal` and the Nuxt module's Nitro
`externals.inline` force-bundle instrumented packages unconditionally.
That's right for a build, but in a dev server it breaks the drivers
(`exports is not defined` on Vite, `(0, lodash_1.defaults) is not a
function` on Nitro) and takes them off Node's module loader, where the
runtime hook `Sentry.init()` registers would have instrumented them
instead.
Gate both on the build. Dev falls through to the runtime hook and emits
the same spans.
astro-7 and sveltekit-2 gain a dev pass over their DB tests; the
react-router-7 dev skip is removed.
Nuxt still records no DB spans in dev — Nitro hoists `ioredis`/`mysql`
above `Sentry.init()` in the generated `--import` entry. Tracked
separately; this only removes the crash there.
Fixes#22857
Co-authored-by: Claude Opus 5 (1M context) <[email protected]>
Copy file name to clipboardExpand all lines: dev-packages/e2e-tests/test-applications/react-router-7-framework-instrumentation/tests/performance/db.server.test.ts
+3-8Lines changed: 3 additions & 8 deletions
Original file line number
Diff line number
Diff line change
@@ -2,14 +2,9 @@ import { expect, test } from '@playwright/test';
0 commit comments