ref(astro): Streamline how we add browser tracing

Author: mydea

packages/astro/src/client/sdk.ts

@@ -1,7 +1,7 @@
 import type { BrowserOptions } from '@sentry/browser';
-import { BrowserTracing, init as initBrowserSdk } from '@sentry/browser';
-import { getCurrentScope, hasTracingEnabled } from '@sentry/core';
-import { addOrUpdateIntegration } from '@sentry/utils';
+import { BrowserTracing, defaultIntegrations, init as initBrowserSdk, setTag } from '@sentry/browser';
+import { hasTracingEnabled } from '@sentry/core';
+import type { Integration } from '@sentry/types';
 
 import { applySdkMetadata } from '../common/metadata';
 
@@ -14,27 +14,26 @@ declare const __SENTRY_TRACING__: boolean;
  * @param options Configuration options for the SDK.
  */
 export function init(options: BrowserOptions): void {
-  applySdkMetadata(options, ['astro', 'browser']);
+  const opts = {
+    defaultIntegrations: getDefaultIntegrations(options),
+    ...options,
+  };
 
-  addClientIntegrations(options);
+  applySdkMetadata(opts, ['astro', 'browser']);
 
-  initBrowserSdk(options);
+  initBrowserSdk(opts);
 
-  getCurrentScope().setTag('runtime', 'browser');
+  setTag('runtime', 'browser');
 }
 
-function addClientIntegrations(options: BrowserOptions): void {
-  let integrations = options.integrations || [];
-
+function getDefaultIntegrations(options: BrowserOptions): Integration[] | undefined {
   // This evaluates to true unless __SENTRY_TRACING__ is text-replaced with "false",
   // in which case everything inside will get treeshaken away
   if (typeof __SENTRY_TRACING__ === 'undefined' || __SENTRY_TRACING__) {
     if (hasTracingEnabled(options)) {
-      const defaultBrowserTracingIntegration = new BrowserTracing({});
-
-      integrations = addOrUpdateIntegration(defaultBrowserTracingIntegration, integrations);
+      return [...defaultIntegrations, new BrowserTracing()];
     }
   }
 
-  options.integrations = integrations;
+  return undefined;
 }

packages/astro/src/server/sdk.ts

@@ -1,6 +1,5 @@
-import { getCurrentScope } from '@sentry/core';
 import type { NodeOptions } from '@sentry/node';
-import { init as initNodeSdk } from '@sentry/node';
+import { init as initNodeSdk, setTag } from '@sentry/node';
 
 import { applySdkMetadata } from '../common/metadata';
 
@@ -13,5 +12,5 @@ export function init(options: NodeOptions): void {
 
   initNodeSdk(options);
 
-  getCurrentScope().setTag('runtime', 'node');
+  setTag('runtime', 'node');
 }