Conversation
…egration v11 has no lightweight mode. `@sentry/node-core` was merged back into `@sentry/node`, so `@sentry/node-core/light` no longer exists and the whole page described a setup that cannot be followed. The OTLP half of that page survives in a different form. The integration moved to the main export of every server SDK and was renamed to `openTelemetryIntegration()`, because it sends nothing over OTLP. Give it its own integration page covering what it does, the exporter wiring through `getOtlpTracesEndpoint()`, and the `enableOpenTelemetrySetup` caveat for `@sentry/nextjs` and `@sentry/sveltekit`. The removed `setupOtlpTracesExporter` and `collectorUrl` options are not carried over: you configure the exporter yourself now. - Drop the lightweight entry from the installation methods page. - Point the "simpler way to bridge your existing setup" line on the custom OpenTelemetry setup page at the new integration. - Point the Node.js card on the OTLP concepts page there too. - Redirect `/install/lightweight/` to the new page for the seven guides that served it. Refs SDK-1352 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Every page in the OpenTelemetry section documented APIs that v11 removed: - `skipOpenTelemetrySetup` was replaced by `enableOpenTelemetrySetup`. - `SentryContextManager`, `SentrySampler`, `SentrySpanProcessor` and `validateOpenTelemetrySetup` were removed, so wiring Sentry's components into a provider you own is no longer possible. - The `openTelemetryInstrumentations` and `openTelemetrySpanProcessors` options were removed. Snapshot all three pages as `__v10.x` so readers on v10 keep working instructions, since every one of those APIs is valid there, and rewrite the current pages around what v11 ships. - The landing page claimed OpenTelemetry instrumentation is picked up "without any further configuration". It isn't: the SDK registers no tracer provider by default, so spans from `@opentelemetry/api` are not captured. Describe the two ways to connect the pipelines instead. - "Using Your Existing OpenTelemetry Setup" was built end to end on the removed components. It now covers the two setups that exist, and says what happened to the components it used to document. - "Using OpenTelemetry APIs" loses the two sections built on the removed options. Adding instrumentation goes through OpenTelemetry's own `registerInstrumentations()`, and the page states that it assumes `enableOpenTelemetrySetup: true`. The `performance/opentelemetry-setup` includes are left alone: only the v10 page renders them now, and they document v10 correctly. Refs SDK-1351 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
JPeer264
added this pull request to stack #19517
September 18, 2026 12:03
andreiborza
removed this pull request from stack #19517
September 22, 2026 12:15
andreiborza
changed the base branch from
janpeer/sdk-1352-prepare-pr-to-remove-lightweight-mode-page
to
js-v11-docs
September 22, 2026 12:15
andreiborza
marked this pull request as ready for review
September 22, 2026 13:16
Contributor
|
Please update the IS YOUR CHANGE URGENT? section of the PR description:
This information helps the Docs team prioritize your review. |
| You can add your own `@opentelemetry/instrumentation-http` instance in your OpenTelemetry setup. However, in this case, you need to disable span creation in Sentry's `httpIntegration`. You can also set `tracePropagation: false` to prevent Sentry from injecting trace headers, letting your OpenTelemetry setup handle propagation instead: | ||
|
|
||
| <PlatformSection notSupported={["javascript.bun"]}> | ||
| ```javascript |
Contributor
There was a problem hiding this comment.
Bug: The v10 documentation snapshot custom-setup__v10.x.mdx contains an empty <Alert> block, which will render a confusing, empty box for users viewing v10-specific documentation.
Severity: LOW
Suggested Fix
Restore the original content about 'lightweight mode' inside the <Alert> block within the custom-setup__v10.x.mdx file. This will ensure the v10 documentation snapshot is complete and provides the necessary context for users.
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: docs/platforms/javascript/common/opentelemetry/custom-setup__v10.x.mdx#L119
Potential issue: The documentation snapshot for v10, `custom-setup__v10.x.mdx`, contains
an empty `<Alert>` block. This block was intended to preserve content about 'lightweight
mode' for users of the v10 SDK. Instead, it will render a visually empty and confusing
alert box for users viewing the v10 documentation for specific platforms like Node,
Express, and others. While this does not cause a functional failure in the SDK, it
creates a regression in documentation quality and leaves v10 users without important
guidance that was supposed to be preserved.
Did we get this right? 👍 / 👎 to inform future reviews.
Member
|
superseded by #19458 |
This branch was successfully 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
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.
DESCRIBE YOUR PR
closes SDK-1351
Every page in the OpenTelemetry section documented APIs that v11 removed:
skipOpenTelemetrySetupwas replaced byenableOpenTelemetrySetup.SentryContextManager,SentrySampler,SentrySpanProcessorandvalidateOpenTelemetrySetupwere removed, so wiring Sentry'scomponents into a provider you own is no longer possible.
openTelemetryInstrumentationsandopenTelemetrySpanProcessorsoptions were removed.
Snapshot all three pages as
__v10.xso readers on v10 keep workinginstructions, since every one of those APIs is valid there, and rewrite
the current pages around what v11 ships.
"without any further configuration". It isn't: the SDK registers no
tracer provider by default, so spans from
@opentelemetry/apiare notcaptured. Describe the two ways to connect the pipelines instead.
removed components. It now covers the two setups that exist, and says
what happened to the components it used to document.
options. Adding instrumentation goes through OpenTelemetry's own
registerInstrumentations(), and the page states that it assumesenableOpenTelemetrySetup: true.The
performance/opentelemetry-setupincludes are left alone: only the v10 page renders them now, and they document v10 correctly.IS YOUR CHANGE URGENT?
Help us prioritize incoming PRs by letting us know when the change needs to go live.
Select exactly one option. For deadlines, replace
YYYY-MM-DDwith the due date. You can update this information later by editing the PR description.SLA
Thanks in advance for your help!
PRE-MERGE CHECKLIST
Make sure you've checked the following before merging your changes:
LEGAL BOILERPLATE
Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. and is gonna need some rights from me in order to utilize my contributions in this here PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.
EXTRA RESOURCES