feat(nuxt): Add _experimental.useDiagnosticsChannelInjection#22323
Open
s1gr1d wants to merge 1 commit into
Open
feat(nuxt): Add _experimental.useDiagnosticsChannelInjection#22323s1gr1d wants to merge 1 commit into
_experimental.useDiagnosticsChannelInjection#22323s1gr1d wants to merge 1 commit into
Conversation
s1gr1d
requested review from
JPeer264,
chargome,
mydea and
nicohrubec
and removed request for
a team
July 16, 2026 11:58
| "@sentry/node": "10.66.0", | ||
| "@sentry/node-core": "10.66.0", | ||
| "@sentry/rollup-plugin": "^5.3.0", | ||
| "@sentry/server-utils": "10.65.0", |
Contributor
There was a problem hiding this comment.
Bug: The @sentry/server-utils package is pinned to 10.65.0 in packages/nuxt/package.json, while new code depends on APIs from version 10.66.0, causing a build failure.
Severity: HIGH
Suggested Fix
In packages/nuxt/package.json, update the version of @sentry/server-utils from 10.65.0 to 10.66.0 to align with other Sentry packages and provide the necessary APIs for the new experimental feature.
Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent. Verify if this is a real issue. If it is, propose a fix; if not, explain why it's
not valid.
Location: packages/nuxt/package.json#L63
Potential issue: The `packages/nuxt/package.json` file incorrectly pins the
`@sentry/server-utils` dependency to version `10.65.0`, while all other Sentry packages
in the same file are at `10.66.0`. New experimental code for diagnostics channel
injection, enabled via `_experimental.useDiagnosticsChannelInjection`, imports APIs like
`INSTRUMENTED_MODULE_NAMES` from `@sentry/server-utils`. These APIs were introduced in
version `10.66.0`. Due to the version mismatch, attempting to use this new feature will
cause a build failure because the required exports are not present in version `10.65.0`.
Also affects:
packages/nuxt/src/vite/orchestrion.ts:1~3
Did we get this right? 👍 / 👎 to inform future reviews.
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
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.
Adds the
useDiagnosticsChannelInjectionoption and updates the E2E test.When using
_experimental.useDiagnosticsChannelInjection, users also need to do those steps:Sentry.experimentalUseDiagnosticsChannelInjection()just beforeSentry.init()--import ./.output/server/sentry.server.config.mjsfrom yourstartscriptsentry.autoInjectServerSentry: 'top-level-import'innuxt.config.tsso Sentry's server configuration is automatically imported