ref: Use addBreadcrumb directly & allow to pass hint

Instead of using `hub.addBreadcrumb()`

Author: mydea

packages/browser/src/integrations/breadcrumbs.ts

@@ -1,5 +1,5 @@
 /* eslint-disable max-lines */
-import { getClient, getCurrentHub } from '@sentry/core';
+import { addBreadcrumb, getClient } from '@sentry/core';
 import type {
   Event as SentryEvent,
   HandlerDataConsole,
@@ -123,7 +123,7 @@ export class Breadcrumbs implements Integration {
  * Adds a breadcrumb for Sentry events or transactions if this option is enabled.
  */
 function addSentryBreadcrumb(event: SentryEvent): void {
-  getCurrentHub().addBreadcrumb(
+  addBreadcrumb(
     {
       category: `sentry.${event.type === 'transaction' ? 'transaction' : 'event'}`,
       event_id: event.event_id,
@@ -173,7 +173,7 @@ function _domBreadcrumb(dom: BreadcrumbsOptions['dom']): (handlerData: HandlerDa
       return;
     }
 
-    getCurrentHub().addBreadcrumb(
+    addBreadcrumb(
       {
         category: `ui.${handlerData.name}`,
         message: target,
@@ -213,7 +213,7 @@ function _consoleBreadcrumb(handlerData: HandlerDataConsole): void {
     }
   }
 
-  getCurrentHub().addBreadcrumb(breadcrumb, {
+  addBreadcrumb(breadcrumb, {
     input: handlerData.args,
     level: handlerData.level,
   });
@@ -247,7 +247,7 @@ function _xhrBreadcrumb(handlerData: HandlerDataXhr): void {
     endTimestamp,
   };
 
-  getCurrentHub().addBreadcrumb(
+  addBreadcrumb(
     {
       category: 'xhr',
       data,
@@ -282,7 +282,7 @@ function _fetchBreadcrumb(handlerData: HandlerDataFetch): void {
       endTimestamp,
     };
 
-    getCurrentHub().addBreadcrumb(
+    addBreadcrumb(
       {
         category: 'fetch',
         data,
@@ -303,7 +303,7 @@ function _fetchBreadcrumb(handlerData: HandlerDataFetch): void {
       startTimestamp,
       endTimestamp,
     };
-    getCurrentHub().addBreadcrumb(
+    addBreadcrumb(
       {
         category: 'fetch',
         data,
@@ -338,7 +338,7 @@ function _historyBreadcrumb(handlerData: HandlerDataHistory): void {
     from = parsedFrom.relative;
   }
 
-  getCurrentHub().addBreadcrumb({
+  addBreadcrumb({
     category: 'navigation',
     data: {
       from,

packages/browser/test/unit/integrations/breadcrumbs.test.ts

@@ -1,4 +1,4 @@
-import { getCurrentHub } from '@sentry/core';
+import * as SentryCore from '@sentry/core';
 import type { Client } from '@sentry/types';
 
 import { Breadcrumbs, BrowserClient, Hub, flush } from '../../../src';
@@ -18,21 +18,20 @@ jest.mock('@sentry/core', () => {
 
 describe('Breadcrumbs', () => {
   it('Should add sentry breadcrumb', async () => {
-    const addBreadcrumb = jest.fn();
-    hub.addBreadcrumb = addBreadcrumb;
-
     client = new BrowserClient({
       ...getDefaultBrowserClientOptions(),
       dsn: 'https://username@domain/123',
       integrations: [new Breadcrumbs()],
     });
 
-    getCurrentHub().bindClient(client);
+    SentryCore.getCurrentHub().bindClient(client);
+
+    const addBreadcrumbSpy = jest.spyOn(SentryCore, 'addBreadcrumb').mockImplementation(() => {});
 
     client.captureMessage('test');
     await flush(2000);
 
-    expect(addBreadcrumb.mock.calls[0][0].category).toEqual('sentry.event');
-    expect(addBreadcrumb.mock.calls[0][0].message).toEqual('test');
+    expect(addBreadcrumbSpy.mock.calls[0][0].category).toEqual('sentry.event');
+    expect(addBreadcrumbSpy.mock.calls[0][0].message).toEqual('test');
   });
 });

packages/core/src/exports.ts

@@ -1,5 +1,6 @@
 import type {
   Breadcrumb,
+  BreadcrumbHint,
   CaptureContext,
   CheckIn,
   Client,
@@ -90,8 +91,8 @@ export function configureScope(callback: (scope: Scope) => void): ReturnType<Hub
  *
  * @param breadcrumb The breadcrumb to record.
  */
-export function addBreadcrumb(breadcrumb: Breadcrumb): ReturnType<Hub['addBreadcrumb']> {
-  getCurrentHub().addBreadcrumb(breadcrumb);
+export function addBreadcrumb(breadcrumb: Breadcrumb, hint?: BreadcrumbHint): ReturnType<Hub['addBreadcrumb']> {
+  getCurrentHub().addBreadcrumb(breadcrumb, hint);
 }
 
 /**

packages/node-experimental/src/integrations/http.ts

@@ -3,7 +3,7 @@ import type { Span } from '@opentelemetry/api';
 import { SpanKind } from '@opentelemetry/api';
 import { registerInstrumentations } from '@opentelemetry/instrumentation';
 import { HttpInstrumentation } from '@opentelemetry/instrumentation-http';
-import { hasTracingEnabled, isSentryRequestUrl } from '@sentry/core';
+import { addBreadcrumb, hasTracingEnabled, isSentryRequestUrl } from '@sentry/core';
 import { _INTERNAL, getClient, getCurrentHub, getSpanKind, setSpanMetadata } from '@sentry/opentelemetry';
 import type { EventProcessor, Hub, Integration } from '@sentry/types';
 import { stringMatchesSomePattern } from '@sentry/utils';
@@ -159,7 +159,7 @@ export class Http implements Integration {
     }
 
     const data = _INTERNAL.getRequestSpanData(span);
-    getCurrentHub().addBreadcrumb(
+    addBreadcrumb(
       {
         category: 'http',
         data: {

packages/node-experimental/src/integrations/node-fetch.ts

@@ -1,8 +1,8 @@
 import type { Span } from '@opentelemetry/api';
 import { SpanKind } from '@opentelemetry/api';
 import type { Instrumentation } from '@opentelemetry/instrumentation';
-import { hasTracingEnabled } from '@sentry/core';
-import { _INTERNAL, getClient, getCurrentHub, getSpanKind } from '@sentry/opentelemetry';
+import { addBreadcrumb, hasTracingEnabled } from '@sentry/core';
+import { _INTERNAL, getClient, getSpanKind } from '@sentry/opentelemetry';
 import type { Integration } from '@sentry/types';
 
 import type { NodeExperimentalClient } from '../types';
@@ -114,7 +114,7 @@ export class NodeFetch extends NodePerformanceIntegration<NodeFetchOptions> impl
     }
 
     const data = _INTERNAL.getRequestSpanData(span);
-    getCurrentHub().addBreadcrumb({
+    addBreadcrumb({
       category: 'http',
       data: {
         ...data,

packages/node/src/integrations/console.ts

@@ -1,5 +1,5 @@
 import * as util from 'util';
-import { getCurrentHub } from '@sentry/core';
+import { addBreadcrumb, getCurrentHub } from '@sentry/core';
 import type { Integration } from '@sentry/types';
 import { addConsoleInstrumentationHandler, severityLevelFromString } from '@sentry/utils';
 
@@ -26,7 +26,7 @@ export class Console implements Integration {
         return;
       }
 
-      hub.addBreadcrumb(
+      addBreadcrumb(
         {
           category: 'console',
           level: severityLevelFromString(level),

packages/node/src/integrations/http.ts

@@ -1,7 +1,7 @@
 import type * as http from 'http';
 import type * as https from 'https';
 import type { Hub } from '@sentry/core';
-import { getClient, getCurrentScope } from '@sentry/core';
+import { addBreadcrumb, getClient, getCurrentScope } from '@sentry/core';
 import { getCurrentHub, getDynamicSamplingContextFromClient, isSentryRequestUrl } from '@sentry/core';
 import type {
   DynamicSamplingContext,
@@ -214,7 +214,7 @@ function _createWrappedRequestMethodFactory(
       return;
     }
 
-    getCurrentHub().addBreadcrumb(
+    addBreadcrumb(
       {
         category: 'http',
         data: {

packages/node/src/integrations/undici/index.ts

@@ -1,4 +1,10 @@
-import { getCurrentHub, getCurrentScope, getDynamicSamplingContextFromClient, isSentryRequestUrl } from '@sentry/core';
+import {
+  addBreadcrumb,
+  getCurrentHub,
+  getCurrentScope,
+  getDynamicSamplingContextFromClient,
+  isSentryRequestUrl,
+} from '@sentry/core';
 import type { EventProcessor, Integration, Span } from '@sentry/types';
 import {
   LRUMap,
@@ -208,7 +214,7 @@ export class Undici implements Integration {
     }
 
     if (this._options.breadcrumbs) {
-      hub.addBreadcrumb(
+      addBreadcrumb(
         {
           category: 'http',
           data: {
@@ -248,7 +254,7 @@ export class Undici implements Integration {
     }
 
     if (this._options.breadcrumbs) {
-      hub.addBreadcrumb(
+      addBreadcrumb(
         {
           category: 'http',
           data: {

packages/node/test/integrations/http.test.ts

@@ -592,6 +592,7 @@ describe('default protocols', () => {
   function captureBreadcrumb(key: string): Promise<Breadcrumb> {
     const hub = new Hub();
     jest.spyOn(sentryCore, 'getCurrentHub').mockReturnValue(hub);
+    jest.spyOn(sentryCore, 'addBreadcrumb').mockImplementation((...rest) => hub.addBreadcrumb(...rest));
 
     let resolve: (value: Breadcrumb | PromiseLike<Breadcrumb>) => void;
     const p = new Promise<Breadcrumb>(r => {

packages/replay/src/util/log.ts

@@ -1,4 +1,4 @@
-import { getCurrentHub } from '@sentry/core';
+import { addBreadcrumb } from '@sentry/core';
 import { logger } from '@sentry/utils';
 
 import { DEBUG_BUILD } from '../debug-build';
@@ -14,7 +14,7 @@ export function logInfo(message: string, shouldAddBreadcrumb?: boolean): void {
   logger.info(message);
 
   if (shouldAddBreadcrumb) {
-    addBreadcrumb(message);
+    addLogBreadcrumb(message);
   }
 }
 
@@ -33,14 +33,13 @@ export function logInfoNextTick(message: string, shouldAddBreadcrumb?: boolean):
     // Wait a tick here to avoid race conditions for some initial logs
     // which may be added before replay is initialized
     setTimeout(() => {
-      addBreadcrumb(message);
+      addLogBreadcrumb(message);
     }, 0);
   }
 }
 
-function addBreadcrumb(message: string): void {
-  const hub = getCurrentHub();
-  hub.addBreadcrumb(
+function addLogBreadcrumb(message: string): void {
+  addBreadcrumb(
     {
       category: 'console',
       data: {