feat(v8): Remove @sentry/tracing (#10625)

Aside from deleting the tracing package and all of it's references, it
also moves some relevant tests into core.

Author: AbhiPrasad

.craft.yml

@@ -131,12 +131,7 @@ targets:
     id: '@sentry-internal/eslint-config-sdk'
     includeNames: /^sentry-internal-eslint-config-sdk-\d.*\.tgz$/
 
-  ## 8. Deprecated packages we still release (but no packages depend on them anymore)
-  - name: npm
-    id: '@sentry/tracing'
-    includeNames: /^sentry-tracing-\d.*\.tgz$/
-
-  ## 9. Experimental packages
+  ## 8. Experimental packages
   - name: npm
     id: '@sentry/node-experimental'
     includeNames: /^sentry-node-experimental-\d.*\.tgz$/

MIGRATION.md

@@ -88,7 +88,13 @@ to access and mutate the current scope.
 
 ## Deletion of `@sentry/hub` package (#10530)
 
-`@sentry/hub` has been removed. All exports from `@sentry.hub` should be available in `@sentry/core`.
+`@sentry/hub` has been removed. All exports from `@sentry/tracing` should be available in `@sentry/core` or in
+`@sentry/browser` and `@sentry/node`.
+
+## Deletion of `@sentry/tracing` package
+
+`@sentry/tracing` has been removed. All exports from `@sentry/tracing` should be available in `@sentry/core` or in
+`@sentry/browser` and `@sentry/node`.
 
 ## Removal of `makeXHRTransport` transport (#10703)
 

README.md

@@ -95,8 +95,6 @@ Besides the high-level SDKs, this repository contains shared packages, helpers a
 development. If you're thinking about contributing to or creating a JavaScript-based SDK, have a look at the resources
 below:
 
-- [`@sentry/tracing`](https://github.com/getsentry/sentry-javascript/tree/master/packages/tracing): Provides
-  integrations and extensions for Performance Monitoring / Tracing.
 - [`@sentry/replay`](https://github.com/getsentry/sentry-javascript/tree/master/packages/replay): Provides the
   integration for Session Replay.
 - [`@sentry/core`](https://github.com/getsentry/sentry-javascript/tree/master/packages/core): The base for all

dev-packages/browser-integration-tests/package.json

@@ -48,7 +48,6 @@
     "@playwright/test": "^1.40.1",
     "@sentry-internal/rrweb": "2.11.0",
     "@sentry/browser": "7.100.0",
-    "@sentry/tracing": "7.100.0",
     "axios": "1.6.7",
     "babel-loader": "^8.2.2",
     "html-webpack-plugin": "^5.5.0",

dev-packages/browser-integration-tests/suites/public-api/startSpan/init.js

@@ -1,10 +1,9 @@
-/* eslint-disable no-unused-vars */
 import * as Sentry from '@sentry/browser';
-// biome-ignore lint/nursery/noUnusedImports: Need to import tracing for side effect
-import * as _ from '@sentry/tracing';
 
 window.Sentry = Sentry;
 
+Sentry.addTracingExtensions();
+
 Sentry.init({
   dsn: 'https://public@dsn.ingest.sentry.io/1337',
   tracesSampleRate: 1.0,

dev-packages/browser-integration-tests/suites/public-api/startTransaction/init.js

@@ -1,10 +1,9 @@
-/* eslint-disable no-unused-vars */
 import * as Sentry from '@sentry/browser';
-// biome-ignore lint/nursery/noUnusedImports: Need to import tracing for side effect
-import * as _ from '@sentry/tracing';
 
 window.Sentry = Sentry;
 
+Sentry.addTracingExtensions();
+
 Sentry.init({
   dsn: 'https://public@dsn.ingest.sentry.io/1337',
   tracesSampleRate: 1.0,

dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/long-tasks-disabled/test.ts

@@ -1,6 +1,6 @@
 import type { Route } from '@playwright/test';
 import { expect } from '@playwright/test';
-import type { Event } from '@sentry/types';
+import type { SerializedEvent } from '@sentry/types';
 
 import { sentryTest } from '../../../../utils/fixtures';
 import { getFirstSentryEnvelopeRequest, shouldSkipTracingTest } from '../../../../utils/helpers';
@@ -15,8 +15,7 @@ sentryTest('should not capture long task when flag is disabled.', async ({ brows
 
   const url = await getLocalTestPath({ testDir: __dirname });
 
-  const eventData = await getFirstSentryEnvelopeRequest<Event>(page, url);
-  // eslint-disable-next-line deprecation/deprecation
+  const eventData = await getFirstSentryEnvelopeRequest<SerializedEvent>(page, url);
   const uiSpans = eventData.spans?.filter(({ op }) => op?.startsWith('ui'));
 
   expect(uiSpans?.length).toBe(0);

dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/long-tasks-enabled/test.ts

@@ -1,6 +1,6 @@
 import type { Route } from '@playwright/test';
 import { expect } from '@playwright/test';
-import type { Event } from '@sentry/types';
+import type { SerializedEvent } from '@sentry/types';
 
 import { sentryTest } from '../../../../utils/fixtures';
 import { getFirstSentryEnvelopeRequest, shouldSkipTracingTest } from '../../../../utils/helpers';
@@ -15,8 +15,7 @@ sentryTest('should capture long task.', async ({ browserName, getLocalTestPath,
 
   const url = await getLocalTestPath({ testDir: __dirname });
 
-  const eventData = await getFirstSentryEnvelopeRequest<Event>(page, url);
-  // eslint-disable-next-line deprecation/deprecation
+  const eventData = await getFirstSentryEnvelopeRequest<SerializedEvent>(page, url);
   const uiSpans = eventData.spans?.filter(({ op }) => op?.startsWith('ui'));
 
   expect(uiSpans?.length).toBeGreaterThan(0);
@@ -29,8 +28,8 @@ sentryTest('should capture long task.', async ({ browserName, getLocalTestPath,
       parent_span_id: eventData.contexts?.trace?.span_id,
     }),
   );
-  const start = (firstUISpan as Event)['start_timestamp'] ?? 0;
-  const end = (firstUISpan as Event)['timestamp'] ?? 0;
+  const start = firstUISpan.start_timestamp ?? 0;
+  const end = firstUISpan.timestamp ?? 0;
   const duration = end - start;
 
   expect(duration).toBeGreaterThanOrEqual(0.1);

dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegrationShim/init.js

@@ -5,7 +5,7 @@ window.Sentry = Sentry;
 Sentry.init({
   dsn: 'https://public@dsn.ingest.sentry.io/1337',
   sampleRate: 1,
-  integrations: [new Sentry.browserTracingIntegration()],
+  integrations: [Sentry.browserTracingIntegration()],
 });
 
 // This should not fail

dev-packages/browser-integration-tests/suites/tracing/browsertracing/backgroundtab-custom/init.js

@@ -1,10 +1,9 @@
 import * as Sentry from '@sentry/browser';
-import { Integrations } from '@sentry/tracing';
 
 window.Sentry = Sentry;
 
 Sentry.init({
   dsn: 'https://public@dsn.ingest.sentry.io/1337',
-  integrations: [new Integrations.BrowserTracing({ idleTimeout: 9000, startTransactionOnPageLoad: false })],
+  integrations: [new Sentry.BrowserTracing({ idleTimeout: 9000, startTransactionOnPageLoad: false })],
   tracesSampleRate: 1,
 });