feat: Ensure getRootSpan() does not rely on transaction

Instead, we keep the root span as a non-enumerable property on the span.

Author: mydea

packages/angular/src/tracing.ts

@@ -289,10 +289,6 @@ export class TraceService implements OnDestroy {
     }
 
     const rootSpan = getRootSpan(activeSpan);
-    if (!rootSpan) {
-      this._pageloadOngoing = false;
-      return false;
-    }
 
     this._pageloadOngoing = spanToJSON(rootSpan).op === 'pageload';
     return this._pageloadOngoing;

packages/astro/src/index.server.ts

@@ -59,6 +59,7 @@ export {
   linkedErrorsIntegration,
   setMeasurement,
   getActiveSpan,
+  getRootSpan,
   startSpan,
   startInactiveSpan,
   startSpanManual,

packages/astro/src/index.types.ts

@@ -32,6 +32,7 @@ export declare const startSpan: typeof clientSdk.startSpan;
 export declare const startInactiveSpan: typeof clientSdk.startInactiveSpan;
 export declare const startSpanManual: typeof clientSdk.startSpanManual;
 export declare const withActiveSpan: typeof clientSdk.withActiveSpan;
+export declare const getRootSpan: typeof clientSdk.getRootSpan;
 export declare const Span: clientSdk.Span;
 
 export declare const metrics: typeof clientSdk.metrics & typeof serverSdk.metrics;

packages/browser/src/index.bundle.tracing.replay.feedback.ts

@@ -8,6 +8,7 @@ addTracingExtensions();
 
 export {
   getActiveSpan,
+  getRootSpan,
   startSpan,
   startInactiveSpan,
   startSpanManual,

packages/browser/src/index.bundle.tracing.replay.ts

@@ -8,6 +8,7 @@ addTracingExtensions();
 
 export {
   getActiveSpan,
+  getRootSpan,
   startSpan,
   startInactiveSpan,
   startSpanManual,

packages/browser/src/index.bundle.tracing.ts

@@ -8,6 +8,7 @@ addTracingExtensions();
 
 export {
   getActiveSpan,
+  getRootSpan,
   startSpan,
   startInactiveSpan,
   startSpanManual,

packages/browser/src/index.ts

@@ -63,6 +63,7 @@ export type { RequestInstrumentationOptions } from '@sentry-internal/tracing';
 export {
   addTracingExtensions,
   getActiveSpan,
+  getRootSpan,
   startSpan,
   startInactiveSpan,
   startSpanManual,

packages/bun/src/index.ts

@@ -64,6 +64,7 @@ export {
   withMonitor,
   setMeasurement,
   getActiveSpan,
+  getRootSpan,
   startSpan,
   startInactiveSpan,
   startSpanManual,

packages/core/src/index.ts

@@ -91,8 +91,8 @@ export {
   spanIsSampled,
   spanToTraceContext,
   getSpanDescendants,
+  getRootSpan,
 } from './utils/spanUtils';
-export { getRootSpan } from './utils/getRootSpan';
 export { applySdkMetadata } from './utils/sdkMetadata';
 export { DEFAULT_ENVIRONMENT } from './constants';
 /* eslint-disable deprecation/deprecation */

packages/core/src/server-runtime-client.ts

@@ -24,8 +24,7 @@ import {
   getDynamicSamplingContextFromClient,
   getDynamicSamplingContextFromSpan,
 } from './tracing';
-import { getRootSpan } from './utils/getRootSpan';
-import { spanToTraceContext } from './utils/spanUtils';
+import { getRootSpan, spanToTraceContext } from './utils/spanUtils';
 
 export interface ServerRuntimeClientOptions extends ClientOptions<BaseTransportOptions> {
   platform?: string;
@@ -256,8 +255,9 @@ export class ServerRuntimeClient<
     // eslint-disable-next-line deprecation/deprecation
     const span = scope.getSpan();
     if (span) {
-      const samplingContext = getRootSpan(span) ? getDynamicSamplingContextFromSpan(span) : undefined;
-      return [samplingContext, spanToTraceContext(span)];
+      const rootSpan = getRootSpan(span);
+      const samplingContext = getDynamicSamplingContextFromSpan(rootSpan);
+      return [samplingContext, spanToTraceContext(rootSpan)];
     }
 
     const { traceId, spanId, parentSpanId, dsc } = scope.getPropagationContext();