Skip to content

chore!: Remove SEMANTIC_ATTRIBUTE_SENTRY_OP in favour of @sentry/conventions#22565

Draft
Lms24 wants to merge 1 commit into
developfrom
lms/chore-use-sentry.op-conventions
Draft

chore!: Remove SEMANTIC_ATTRIBUTE_SENTRY_OP in favour of @sentry/conventions#22565
Lms24 wants to merge 1 commit into
developfrom
lms/chore-use-sentry.op-conventions

Conversation

@Lms24

@Lms24 Lms24 commented Jul 23, 2026

Copy link
Copy Markdown
Member

No description provided.

@Lms24 Lms24 changed the title chore: Remove SEMANTIC_ATTRIBUTE_SENTRY_OP in favour of @sentry/conventions chore!: Remove SEMANTIC_ATTRIBUTE_SENTRY_OP in favour of @sentry/conventions Jul 23, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit ef7ed2b. Configure here.

import { waitForTransaction } from '@sentry-internal/test-utils';
// Cannot use @sentry/angular here due to build stuff
import { SEMANTIC_ATTRIBUTE_SENTRY_OP, SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN } from '@sentry/core';
import { SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN } from '@sentry/core';

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Public OP export removed without deprecation

High Severity

Flagged because it was mentioned in the PR review rules file: this removes the publicly exported SEMANTIC_ATTRIBUTE_SENTRY_OP from SDK entry points (@sentry/core and re-exports such as @sentry/nitro / @sentry/nuxt) without a deprecation notice. Sibling migrations kept deprecated aliases (for example SEMANTIC_ATTRIBUTE_SENTRY_SEGMENT_NAME), so external imports of SEMANTIC_ATTRIBUTE_SENTRY_OP will break at compile time.

Additional Locations (2)
Fix in Cursor Fix in Web

Triggered by project rule: PR Review Guidelines for Cursor Bot

Reviewed by Cursor Bugbot for commit ef7ed2b. Configure here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant