feat(core): Deprecate the Hub constructor

Author: Luca Forstner

MIGRATION.md

@@ -285,6 +285,43 @@ If you are using the `Hub` right now, see the following table on how to migrate
 | endSession()           | `Sentry.endSession()`                                                                |
 | shouldSendDefaultPii() | REMOVED - The closest equivalent is `Sentry.getClient().getOptions().sendDefaultPii` |
 
+The `Hub` constructor is also gonna be deprecated. If you are creating Hubs for multi-client use like so:
+
+```
+// OLD
+const hub = new Hub();
+hub.bindClient(client);
+makeMain(hub)
+```
+
+instead initialize the client as follows:
+
+```
+// NEW
+Sentry.withIsolationScope(() => {
+   Sentry.setCurrentClient(client);
+   client.init();
+});
+```
+
+If you are using the Hub to capture events like so:
+
+```
+// OLD
+const client = new Client();
+const hub = new Hub(client);
+hub.captureException()
+```
+
+instead capture isolated events as follows:
+
+````
+// NEW
+const client = new Client();
+const scope = new Scope();
+scope.setClient(client);
+scope.captureException();
+
 ## Deprecate `client.setupIntegrations()`
 
 Instead, use the new `client.init()` method. You should probably not use this directly and instead use `Sentry.init()`,
@@ -351,7 +388,7 @@ app.get('/your-route', req => {
     );
   });
 });
-```
+````
 
 ## Deprecate `Sentry.lastEventId()` and `hub.lastEventId()`
 

packages/core/src/hub.ts

@@ -132,6 +132,46 @@ export class Hub implements HubInterface {
    * @param client bound to the hub.
    * @param scope bound to the hub.
    * @param version number, higher number means higher priority.
+   *
+   * @deprecated Instantiation of Hub objects is deprecated and the constructor will be removed in version 8 of the SDK.
+   *
+   * If you are currently using the Hub for multi-client use like so:
+   *
+   * ```
+   * // OLD
+   * const hub = new Hub();
+   * hub.bindClient(client);
+   * makeMain(hub)
+   * ```
+   *
+   * instead initialize the client as follows:
+   *
+   * ```
+   * // NEW
+   * Sentry.withIsolationScope(() => {
+   *    Sentry.setCurrentClient(client);
+   *    client.init();
+   * });
+   * ```
+   *
+   * If you are using the Hub to capture events like so:
+   *
+   * ```
+   * // OLD
+   * const client = new Client();
+   * const hub = new Hub(client);
+   * hub.captureException()
+   * ```
+   *
+   * instead capture isolated events as follows:
+   *
+   * ```
+   * // NEW
+   * const client = new Client();
+   * const scope = new Scope();
+   * scope.setClient(client);
+   * scope.captureException();
+   * ```
    */
   public constructor(
     client?: Client,

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

@@ -138,7 +138,7 @@ export function getCurrentHub(): Hub {
  */
 export function makeMain(hub: Hub): Hub {
   // eslint-disable-next-line no-console
-  console.warn('makeMain is a noop in @sentry/node-experimental. Use `setCurrentScope` instead.');
+  console.warn('makeMain is a noop in @sentry/node-experimental. Use `setCurrentClient` instead.');
   return hub;
 }