feat(opentelemetry): Remove unused exports#22581
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 58b21e2. Configure here.
| startSpan, | ||
| startSpanManual, | ||
| startInactiveSpan, | ||
| withActiveSpan, |
There was a problem hiding this comment.
Public API removals without deprecation
High Severity
Flagged because the PR Review Guidelines call out removal of publicly exported functions without deprecation as a critical breaking change. @sentry/opentelemetry drops public exports such as startSpan, getActiveSpan, getClient, continueTrace, shouldPropagateTraceForUrl, and the spanHas* helpers with no deprecation aliases and no MIGRATION.md note, so external importers can break on upgrade even if this repo no longer uses them.
Additional Locations (2)
Triggered by project rule: PR Review Guidelines for Cursor Bot
Reviewed by Cursor Bugbot for commit 58b21e2. Configure here.
size-limit report 📦
|
4d8f7bd to
3f21c81
Compare
3f21c81 to
348851a
Compare
Lms24
left a comment
There was a problem hiding this comment.
Do we need deprecations for these on the v10 branch?
Also should this PR have a ! in the title?


Removes exports that are not used anywhere from the opentelemetry package.