ref: Deprecate extractTraceParentData from @sentry/core & downstream packages (#9158)

Instead, users should import this from `@sentry/utils`.

This was also re-exported from all downstream packages (e.g.
`@sentry/browser`, `@sentry/node`). I don't think this is something
users need to import from there...? If you need this, installing
`@sentry/utils` is probably fine.

Author: mydea

MIGRATION.md

@@ -8,6 +8,12 @@ npx @sentry/migr8@latest
 
 This will let you select which updates to run, and automatically update your code. Make sure to still review all code changes!
 
+## Deprecate `extractTraceParentData` export from `@sentry/core` & downstream packages
+
+Instead, import this directly from `@sentry/utils`.
+
+Generally, in most cases you should probably use `continueTrace` instead, which abstracts this away from you and handles scope propagation for you.
+
 ## Deprecate `timestampWithMs` export - #7878
 
 The `timestampWithMs` util is deprecated in favor of using `timestampInSeconds`.

packages/astro/src/index.server.ts

@@ -16,6 +16,7 @@ export {
   withMonitor,
   configureScope,
   createTransport,
+  // eslint-disable-next-line deprecation/deprecation
   extractTraceparentData,
   getActiveTransaction,
   getHubFromCarrier,

packages/browser/src/index.ts

@@ -42,6 +42,7 @@ export type { RequestInstrumentationOptions } from '@sentry-internal/tracing';
 export {
   addTracingExtensions,
   setMeasurement,
+  // eslint-disable-next-line deprecation/deprecation
   extractTraceparentData,
   getActiveTransaction,
   spanStatusfromHttpCode,

packages/bun/src/index.ts

@@ -33,6 +33,7 @@ export {
   close,
   configureScope,
   createTransport,
+  // eslint-disable-next-line deprecation/deprecation
   extractTraceparentData,
   flush,
   getActiveTransaction,

packages/core/src/tracing/index.ts

@@ -3,6 +3,7 @@ export { IdleTransaction, TRACING_DEFAULTS } from './idletransaction';
 export type { BeforeFinishCallback } from './idletransaction';
 export { Span, spanStatusfromHttpCode } from './span';
 export { Transaction } from './transaction';
+// eslint-disable-next-line deprecation/deprecation
 export { extractTraceparentData, getActiveTransaction } from './utils';
 // eslint-disable-next-line deprecation/deprecation
 export { SpanStatus } from './spanstatus';

packages/core/src/tracing/utils.ts

@@ -1,8 +1,19 @@
 import type { Transaction } from '@sentry/types';
+import { extractTraceparentData as _extractTraceparentData } from '@sentry/utils';
 
 import type { Hub } from '../hub';
 import { getCurrentHub } from '../hub';
 
+/** Grabs active transaction off scope, if any */
+export function getActiveTransaction<T extends Transaction>(maybeHub?: Hub): T | undefined {
+  const hub = maybeHub || getCurrentHub();
+  const scope = hub.getScope();
+  return scope.getTransaction() as T | undefined;
+}
+
+// so it can be used in manual instrumentation without necessitating a hard dependency on @sentry/utils
+export { stripUrlQueryAndFragment } from '@sentry/utils';
+
 /**
  * The `extractTraceparentData` function and `TRACEPARENT_REGEXP` constant used
  * to be declared in this file. It was later moved into `@sentry/utils` as part of a
@@ -11,18 +22,8 @@ import { getCurrentHub } from '../hub';
  *
  * These exports are kept here for backwards compatability's sake.
  *
- * TODO(v7): Reorganize these exports
- *
  * See https://github.com/getsentry/sentry-javascript/issues/4642 for more details.
+ *
+ * @deprecated Import this function from `@sentry/utils` instead
  */
-export { TRACEPARENT_REGEXP, extractTraceparentData } from '@sentry/utils';
-
-/** Grabs active transaction off scope, if any */
-export function getActiveTransaction<T extends Transaction>(maybeHub?: Hub): T | undefined {
-  const hub = maybeHub || getCurrentHub();
-  const scope = hub.getScope();
-  return scope.getTransaction() as T | undefined;
-}
-
-// so it can be used in manual instrumentation without necessitating a hard dependency on @sentry/utils
-export { stripUrlQueryAndFragment } from '@sentry/utils';
+export const extractTraceparentData = _extractTraceparentData;

packages/deno/src/index.ts

@@ -31,7 +31,9 @@ export {
   close,
   configureScope,
   createTransport,
+  // eslint-disable-next-line deprecation/deprecation
   extractTraceparentData,
+  continueTrace,
   flush,
   getActiveTransaction,
   getHubFromCarrier,

packages/node-experimental/src/index.ts

@@ -32,6 +32,7 @@ export {
   close,
   configureScope,
   createTransport,
+  // eslint-disable-next-line deprecation/deprecation
   extractTraceparentData,
   flush,
   getActiveTransaction,

packages/node/src/index.ts

@@ -33,6 +33,7 @@ export {
   close,
   configureScope,
   createTransport,
+  // eslint-disable-next-line deprecation/deprecation
   extractTraceparentData,
   flush,
   getActiveTransaction,

packages/remix/src/index.server.ts

@@ -19,6 +19,7 @@ export {
   captureMessage,
   configureScope,
   createTransport,
+  // eslint-disable-next-line deprecation/deprecation
   extractTraceparentData,
   getActiveTransaction,
   getHubFromCarrier,