feat(core): Make setXXX methods set on isolation scope (#10678)

The hub already writes there, but this circumvents the hub now!

Author: mydea

packages/core/src/exports.ts

@@ -82,17 +82,15 @@ export function captureEvent(event: Event, hint?: EventHint): string {
  */
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 export function setContext(name: string, context: { [key: string]: any } | null): ReturnType<Hub['setContext']> {
-  // eslint-disable-next-line deprecation/deprecation
-  getCurrentHub().setContext(name, context);
+  getIsolationScope().setContext(name, context);
 }
 
 /**
  * Set an object that will be merged sent as extra data with the event.
  * @param extras Extras object to merge into current context.
  */
 export function setExtras(extras: Extras): ReturnType<Hub['setExtras']> {
-  // eslint-disable-next-line deprecation/deprecation
-  getCurrentHub().setExtras(extras);
+  getIsolationScope().setExtras(extras);
 }
 
 /**
@@ -101,17 +99,15 @@ export function setExtras(extras: Extras): ReturnType<Hub['setExtras']> {
  * @param extra Any kind of data. This data will be normalized.
  */
 export function setExtra(key: string, extra: Extra): ReturnType<Hub['setExtra']> {
-  // eslint-disable-next-line deprecation/deprecation
-  getCurrentHub().setExtra(key, extra);
+  getIsolationScope().setExtra(key, extra);
 }
 
 /**
  * Set an object that will be merged sent as tags data with the event.
  * @param tags Tags context object to merge into current context.
  */
 export function setTags(tags: { [key: string]: Primitive }): ReturnType<Hub['setTags']> {
-  // eslint-disable-next-line deprecation/deprecation
-  getCurrentHub().setTags(tags);
+  getIsolationScope().setTags(tags);
 }
 
 /**
@@ -123,8 +119,7 @@ export function setTags(tags: { [key: string]: Primitive }): ReturnType<Hub['set
  * @param value Value of tag
  */
 export function setTag(key: string, value: Primitive): ReturnType<Hub['setTag']> {
-  // eslint-disable-next-line deprecation/deprecation
-  getCurrentHub().setTag(key, value);
+  getIsolationScope().setTag(key, value);
 }
 
 /**
@@ -133,8 +128,7 @@ export function setTag(key: string, value: Primitive): ReturnType<Hub['setTag']>
  * @param user User context object to be set in the current context. Pass `null` to unset the user.
  */
 export function setUser(user: User | null): ReturnType<Hub['setUser']> {
-  // eslint-disable-next-line deprecation/deprecation
-  getCurrentHub().setUser(user);
+  getIsolationScope().setUser(user);
 }
 
 /**

packages/node-experimental/src/index.ts

@@ -32,13 +32,6 @@ export {
   captureException,
   captureEvent,
   captureMessage,
-  addEventProcessor,
-  setContext,
-  setExtra,
-  setExtras,
-  setTag,
-  setTags,
-  setUser,
   withScope,
   withIsolationScope,
   withActiveSpan,
@@ -84,6 +77,13 @@ export {
   functionToStringIntegration,
   inboundFiltersIntegration,
   linkedErrorsIntegration,
+  addEventProcessor,
+  setContext,
+  setExtra,
+  setExtras,
+  setTag,
+  setTags,
+  setUser,
   SEMANTIC_ATTRIBUTE_SENTRY_OP,
   SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN,
   SEMANTIC_ATTRIBUTE_SENTRY_SOURCE,

packages/node-experimental/src/sdk/api.ts

@@ -2,18 +2,7 @@
 
 import type { Span } from '@opentelemetry/api';
 import { context, trace } from '@opentelemetry/api';
-import type {
-  CaptureContext,
-  Event,
-  EventHint,
-  EventProcessor,
-  Extra,
-  Extras,
-  Primitive,
-  Scope,
-  SeverityLevel,
-  User,
-} from '@sentry/types';
+import type { CaptureContext, Event, EventHint, Scope, SeverityLevel } from '@sentry/types';
 import { getContextFromScope, getScopesFromContext, setScopesOnContext } from '../utils/contextData';
 
 import type { ExclusiveEventHintOrCaptureContext } from '../utils/prepareEvent';
@@ -112,44 +101,3 @@ export function captureMessage(message: string, captureContext?: CaptureContext
 export function captureEvent(event: Event, hint?: EventHint): string {
   return getCurrentScope().captureEvent(event, hint);
 }
-
-/**
- * Add an event processor to the current isolation scope.
- */
-export function addEventProcessor(eventProcessor: EventProcessor): void {
-  getIsolationScope().addEventProcessor(eventProcessor);
-}
-
-/** Set the user for the current isolation scope. */
-export function setUser(user: User | null): void {
-  getIsolationScope().setUser(user);
-}
-
-/** Set tags for the current isolation scope. */
-export function setTags(tags: { [key: string]: Primitive }): void {
-  getIsolationScope().setTags(tags);
-}
-
-/** Set a single tag user for the current isolation scope. */
-export function setTag(key: string, value: Primitive): void {
-  getIsolationScope().setTag(key, value);
-}
-
-/** Set extra data for the current isolation scope. */
-export function setExtra(key: string, extra: Extra): void {
-  getIsolationScope().setExtra(key, extra);
-}
-
-/** Set multiple extra data for the current isolation scope. */
-export function setExtras(extras: Extras): void {
-  getIsolationScope().setExtras(extras);
-}
-
-/** Set context data for the current isolation scope. */
-export function setContext(
-  name: string,
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  context: { [key: string]: any } | null,
-): void {
-  getIsolationScope().setContext(name, context);
-}

packages/node-experimental/src/sdk/hub.ts

@@ -10,19 +10,18 @@ import type {
   TransactionContext,
 } from '@sentry/types';
 
-import { addBreadcrumb, endSession, startSession } from '@sentry/core';
 import {
-  captureEvent,
-  getClient,
-  getCurrentScope,
+  addBreadcrumb,
+  endSession,
   setContext,
   setExtra,
   setExtras,
   setTag,
   setTags,
   setUser,
-  withScope,
-} from './api';
+  startSession,
+} from '@sentry/core';
+import { captureEvent, getClient, getCurrentScope, withScope } from './api';
 import { callExtensionMethod, getGlobalCarrier } from './globals';
 import { getIsolationScope } from './scope';
 import type { SentryCarrier } from './types';