feat(core): Remove transactionContext from samplingContext

Author: s1gr1d

docs/migration/v8-to-v9.md

@@ -220,6 +220,7 @@ Since v9, the types have been merged into `@sentry/core`, which removed some of
 - The `TransactionNamingScheme` type has been removed. There is no replacement.
 - The `Request` type has been removed. Use `RequestEventData` type instead.
 - The `IntegrationClass` type is no longer exported - it was not used anymore. Instead, use `Integration` or `IntegrationFn`.
+- The `samplingContext.transactionContext` object in `tracesSampler` has been removed. All object attributes are available in the top-level of `samplingContext`.
 
 # No Version Support Timeline
 

packages/core/src/tracing/trace.ts

@@ -381,10 +381,6 @@ function _startRootSpan(spanArguments: SentrySpanArguments, scope: Scope, parent
         name,
         parentSampled,
         attributes,
-        transactionContext: {
-          name,
-          parentSampled,
-        },
       });
 
   const rootSpan = new SentrySpan({

packages/core/src/types-hoist/samplingcontext.ts

@@ -14,15 +14,6 @@ export interface CustomSamplingContext {
  * Adds default data to data provided by the user. See {@link Hub.startTransaction}
  */
 export interface SamplingContext extends CustomSamplingContext {
-  /**
-   * Context data with which transaction being sampled was created.
-   * @deprecated This is duplicate data and will be removed eventually.
-   */
-  transactionContext: {
-    name: string;
-    parentSampled?: boolean | undefined;
-  };
-
   /**
    * Sampling decision from the parent transaction, if any.
    */

packages/core/test/lib/tracing/trace.test.ts

@@ -496,7 +496,6 @@ describe('startSpan', () => {
         test2: 'aa',
         test3: 'bb',
       },
-      transactionContext: expect.objectContaining({ name: 'outer', parentSampled: undefined }),
     });
   });
 

packages/opentelemetry/src/sampler.ts

@@ -99,10 +99,6 @@ export class SentrySampler implements Sampler {
     const [sampled, sampleRate] = sampleSpan(options, {
       name: inferredSpanName,
       attributes: mergedAttributes,
-      transactionContext: {
-        name: inferredSpanName,
-        parentSampled,
-      },
       parentSampled,
     });
 

packages/opentelemetry/test/trace.test.ts

@@ -1322,7 +1322,6 @@ describe('trace (sampling)', () => {
       parentSampled: undefined,
       name: 'outer',
       attributes: {},
-      transactionContext: { name: 'outer', parentSampled: undefined },
     });
 
     // Now return `false`, it should not sample
@@ -1341,7 +1340,6 @@ describe('trace (sampling)', () => {
       parentSampled: false,
       name: 'inner2',
       attributes: {},
-      transactionContext: { name: 'inner2', parentSampled: false },
     });
   });
 
@@ -1376,7 +1374,6 @@ describe('trace (sampling)', () => {
         attr2: 1,
         'sentry.op': 'test.op',
       },
-      transactionContext: { name: 'outer', parentSampled: undefined },
     });
 
     // Now return `0`, it should not sample
@@ -1395,7 +1392,6 @@ describe('trace (sampling)', () => {
       parentSampled: false,
       name: 'inner2',
       attributes: {},
-      transactionContext: { name: 'inner2', parentSampled: false },
     });
 
     // Now return `0.4`, it should not sample
@@ -1410,7 +1406,6 @@ describe('trace (sampling)', () => {
       parentSampled: undefined,
       name: 'outer3',
       attributes: {},
-      transactionContext: { name: 'outer3', parentSampled: undefined },
     });
   });
 
@@ -1444,10 +1439,6 @@ describe('trace (sampling)', () => {
       parentSampled: true,
       name: 'outer',
       attributes: {},
-      transactionContext: {
-        name: 'outer',
-        parentSampled: true,
-      },
     });
   });
 

packages/profiling-node/src/spanProfileUtils.ts

@@ -48,10 +48,6 @@ export function maybeProfileSpan(
     profilesSampleRate = profilesSampler({
       name: spanName,
       attributes: data,
-      transactionContext: {
-        name: spanName,
-        parentSampled,
-      },
       parentSampled,
       ...customSamplingContext,
     });