feat!: Remove getCurrentHub(), Hub, and getCurrentHubShim() (#1…

…5122)

docs/migration/v8-to-v9.md

@@ -188,6 +188,7 @@ Sentry.init({
 ```
 
 - The `DEFAULT_USER_INCLUDES` constant has been removed.
+- The `getCurrentHub()`, `Hub` and `getCurrentHubShim()` APIs have been removed. They were on compatibility life support since the release of v8 and have now been fully removed from the SDK.
 
 ### `@sentry/browser`
 

packages/astro/src/index.server.ts

@@ -45,8 +45,6 @@ export {
   getActiveSpan,
   getAutoPerformanceIntegrations,
   getClient,
-  // eslint-disable-next-line deprecation/deprecation
-  getCurrentHub,
   getCurrentScope,
   getDefaultIntegrations,
   getGlobalScope,

packages/astro/src/index.types.ts

@@ -24,9 +24,6 @@ export declare const defaultStackParser: StackParser;
 export declare function close(timeout?: number | undefined): PromiseLike<boolean>;
 export declare function flush(timeout?: number | undefined): PromiseLike<boolean>;
 
-// eslint-disable-next-line deprecation/deprecation
-export declare const getCurrentHub: typeof clientSdk.getCurrentHub;
-
 export declare const Span: clientSdk.Span;
 
 export default sentryAstro;

packages/aws-serverless/src/index.ts

@@ -12,8 +12,6 @@ export {
   endSession,
   withMonitor,
   createTransport,
-  // eslint-disable-next-line deprecation/deprecation
-  getCurrentHub,
   getClient,
   isInitialized,
   generateInstrumentOnce,

packages/browser/src/exports.ts

@@ -30,8 +30,6 @@ export {
   createTransport,
   lastEventId,
   flush,
-  // eslint-disable-next-line deprecation/deprecation
-  getCurrentHub,
   getClient,
   isInitialized,
   getCurrentScope,

packages/browser/test/sdk.test.ts

@@ -7,9 +7,9 @@ import type { Mock } from 'vitest';
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
 
 import * as SentryCore from '@sentry/core';
-import { Scope, createTransport } from '@sentry/core';
+import { createTransport } from '@sentry/core';
 import { resolvedSyncPromise } from '@sentry/core';
-import type { Client, Integration } from '@sentry/core';
+import type { Integration } from '@sentry/core';
 
 import type { BrowserOptions } from '../src';
 import { WINDOW } from '../src';
@@ -34,30 +34,6 @@ export class MockIntegration implements Integration {
   }
 }
 
-vi.mock('@sentry/core', async requireActual => {
-  return {
-    ...((await requireActual()) as any),
-    getCurrentHub(): {
-      bindClient(client: Client): boolean;
-      getClient(): boolean;
-      getScope(): Scope;
-    } {
-      return {
-        getClient(): boolean {
-          return false;
-        },
-        getScope(): Scope {
-          return new Scope();
-        },
-        bindClient(client: Client): boolean {
-          client.init!();
-          return true;
-        },
-      };
-    },
-  };
-});
-
 describe('init', () => {
   beforeEach(() => {
     vi.clearAllMocks();

packages/bun/src/index.ts

@@ -31,8 +31,6 @@ export {
   endSession,
   withMonitor,
   createTransport,
-  // eslint-disable-next-line deprecation/deprecation
-  getCurrentHub,
   getClient,
   isInitialized,
   generateInstrumentOnce,

packages/core/src/index.ts

@@ -115,9 +115,6 @@ export { trpcMiddleware } from './trpc';
 export { captureFeedback } from './feedback';
 export type { ReportDialogOptions } from './report-dialog';
 
-// eslint-disable-next-line deprecation/deprecation
-export { getCurrentHubShim, getCurrentHub } from './getCurrentHubShim';
-
 // TODO: Make this structure pretty again and don't do "export *"
 export * from './utils-hoist/index';
 // TODO: Make this structure pretty again and don't do "export *"