feat: Bump to Sentry Javascript V9

CHANGELOG.md

@@ -8,6 +8,12 @@
 
 ## Unreleased
 
+### Upgrading from 6.x to 7.0
+
+Version 7 of the Sentry React Native SDK primarily introduces API cleanup and version support changes. This update contains behavioral changes that will not be caught by type checkers, linters, or tests, so we recommend carefully reading through the entire migration guide instead of relying on automatic tooling.
+
+Version 7 of the SDK is compatible with Sentry self-hosted versions 24.4.2 or higher (unchanged from v6). Lower versions may continue to work, but may not support all features.
+
 ### Features
 
 - Adds Sentry Android Gradle Plugin as an experimental Expo plugin feature ([#4440](https://github.com/getsentry/sentry-react-native/pull/4440))
@@ -38,6 +44,9 @@ Change `Cold/Warm App Start` span description to `Cold/Warm Start` ([#4636](http
 
 ### Dependencies
 
+- Bump JavaScript SDK from v8.54.0 to v9.1.0 ([#4568](https://github.com/getsentry/sentry-react-native/pull/4568))
+  - [changelog](https://github.com/getsentry/sentry-javascript/blob/9.1.0/CHANGELOG.md)
+  - [diff](https://github.com/getsentry/sentry-javascript/compare/8.54.0...9.1.0)
 - Bump Android SDK from v7.20.1 to v8.3.0 ([#4490](https://github.com/getsentry/sentry-react-native/pull/4490))
   - [changelog](https://github.com/getsentry/sentry-java/blob/main/CHANGELOG.md#830)
   - [diff](https://github.com/getsentry/sentry-java/compare/7.20.1...8.3.0)
@@ -48,6 +57,39 @@ Change `Cold/Warm App Start` span description to `Cold/Warm Start` ([#4636](http
   - [changelog](https://github.com/getsentry/sentry-cli/blob/master/CHANGELOG.md#2421)
   - [diff](https://github.com/getsentry/sentry-cli/compare/2.41.1...2.42.1)
 
+
+### Major Changes
+
+- `ip addresses` is only collected when `sendDefaultPii`: `true`
+- Exceptions from `captureConsoleIntegration` are now marked as handled: true by default
+- `shutdownTimeout` moved from `core` to `@sentry/react-native`
+- `hasTracingEnabled` was renamed to `hasSpansEnabled`
+- You can no longer drop spans or return null on `beforeSendSpan` hook
+
+### Removed types
+
+- TransactionNamingScheme
+- Request
+- Scope (prefer using the Scope class)
+
+### Other removed items.
+
+- `autoSessionTracking` from options.
+  To enable session tracking, ensure that `enableAutoSessionTracking` is enabled.
+
+- `enableTracing`. Instead, set `tracesSampleRate` to a value greater than `zero` to `enable tracing`, `0` to keep tracing integrations active without sampling, or `undefined` to disable the performance integration.
+
+- `getCurrentHub()`, `Hub`, and `getCurrentHubShim()`
+- `spanId` from propagation `context`
+- metrics API
+- `transactionContext` from `samplingContext`
+- `@sentry/utils` package, the exports were moved to `@sentry/core`
+- Standalone `Client` interface & deprecate `BaseClient`
+
+## Other Changes
+
+- Fork `scope` if custom scope is passed to `startSpanManual` or `startSpan`
+
 ## 6.7.0
 
 ### Features

dev-packages/type-check/ts3.8-test/index.ts

@@ -3,6 +3,8 @@ declare global {
   interface IDBObjectStore {}
   interface Window {
     fetch: any;
+    setTimeout: any;
+    document: any;
   }
   interface ShadowRoot {}
   interface BufferSource {}
@@ -19,6 +21,7 @@ declare global {
     redirectCount: number;
   }
   interface PerformanceEntry {}
+  interface Performance {}
 }
 import 'react-native';
 

packages/core/package.json

@@ -66,21 +66,20 @@
   },
   "dependencies": {
     "@sentry/babel-plugin-component-annotate": "3.1.2",
-    "@sentry/browser": "8.54.0",
+    "@sentry/browser": "9.1.0",
     "@sentry/cli": "2.42.1",
-    "@sentry/core": "8.54.0",
-    "@sentry/react": "8.54.0",
-    "@sentry/types": "8.54.0",
-    "@sentry/utils": "8.54.0"
+    "@sentry/core": "9.1.0",
+    "@sentry/react": "9.1.0",
+    "@sentry/types": "9.1.0"
   },
   "devDependencies": {
     "@babel/core": "^7.25.2",
     "@expo/metro-config": "0.19.5",
     "@mswjs/interceptors": "^0.25.15",
     "@react-native/babel-preset": "0.77.1",
-    "@sentry-internal/eslint-config-sdk": "8.54.0",
-    "@sentry-internal/eslint-plugin-sdk": "8.54.0",
-    "@sentry-internal/typescript": "8.54.0",
+    "@sentry-internal/eslint-config-sdk": "9.1.0",
+    "@sentry-internal/eslint-plugin-sdk": "9.1.0",
+    "@sentry-internal/typescript": "9.1.0",
     "@sentry/wizard": "3.42.0",
     "@testing-library/react-native": "^12.7.2",
     "@types/jest": "^29.5.13",
@@ -110,7 +109,7 @@
     "react-native": "0.77.1",
     "react-test-renderer": "^18.3.1",
     "rimraf": "^4.1.1",
-    "ts-jest": "^29.1.1",
+    "ts-jest": "^29.2.5",
     "typescript": "4.9.5",
     "uglify-js": "^3.17.4",
     "uuid": "^9.0.1",

packages/core/plugin/src/withSentry.ts

@@ -18,7 +18,7 @@ interface PluginProps {
 const withSentryPlugin: ConfigPlugin<PluginProps | void> = (config, props) => {
   const sentryProperties = getSentryProperties(props);
 
-  if (props && props.authToken) {
+  if (props?.authToken) {
     // If not removed, the plugin config with the authToken will be written to the application package
     delete props.authToken;
   }

packages/core/plugin/src/withSentryAndroidGradlePlugin.ts

@@ -35,7 +35,7 @@ export function withSentryAndroidGradlePlugin(
   const withSentryProjectBuildGradle = (config: any): any => {
     return withProjectBuildGradle(config, (projectBuildGradle: any) => {
       // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
-      if (!projectBuildGradle.modResults || !projectBuildGradle.modResults.contents) {
+      if (!projectBuildGradle.modResults?.contents) {
         warnOnce('android/build.gradle content is missing or undefined.');
         return config;
       }

packages/core/src/js/index.ts

@@ -1,6 +1,5 @@
 export type {
   Breadcrumb,
-  Request,
   SdkInfo,
   Event,
   Exception,
@@ -43,7 +42,6 @@ export {
   getClient,
   setCurrentClient,
   addEventProcessor,
-  metricsDefault as metrics,
   lastEventId,
 } from '@sentry/core';
 

packages/core/src/js/integrations/debugsymbolicator.ts

@@ -143,7 +143,7 @@ function replaceExceptionFramesInException(exception: Exception, frames: SentryS
  * @param frames StackFrame[]
  */
 function replaceThreadFramesInEvent(event: Event, frames: SentryStackFrame[]): void {
-  if (event.threads && event.threads.values && event.threads.values[0] && event.threads.values[0].stacktrace) {
+  if (event.threads?.values?.[0]?.stacktrace) {
     event.threads.values[0].stacktrace.frames = frames.reverse();
   }
 }

packages/core/src/js/integrations/default.ts

@@ -91,10 +91,7 @@ export function getDefaultIntegrations(options: ReactNativeClientOptions): Integ
   // hasTracingEnabled from `@sentry/core` only check if tracesSampler or tracesSampleRate keys are present
   // that's different from prev imp here and might lead misconfiguration
   // `tracesSampleRate: undefined` should not enable tracing
-  const hasTracingEnabled =
-    options.enableTracing ||
-    typeof options.tracesSampleRate === 'number' ||
-    typeof options.tracesSampler === 'function';
+  const hasTracingEnabled = typeof options.tracesSampleRate === 'number' || typeof options.tracesSampler === 'function';
   if (hasTracingEnabled && options.enableAppStartTracking) {
     integrations.push(appStartIntegration());
   }

packages/core/src/js/integrations/nativelinkederrors.ts

@@ -43,7 +43,7 @@ export const nativeLinkedErrorsIntegration = (options: Partial<LinkedErrorsOptio
 };
 
 function preprocessEvent(event: Event, hint: EventHint | undefined, client: Client, limit: number, key: string): void {
-  if (!event.exception || !event.exception.values || !hint || !isInstanceOf(hint.originalException, Error)) {
+  if (!event.exception?.values || !hint || !isInstanceOf(hint.originalException, Error)) {
     return;
   }
 
@@ -176,10 +176,10 @@ function exceptionFromAppleStackReturnAddresses(objCException: {
       type: objCException.name,
       value: objCException.message,
       stacktrace: {
-        frames: (nativeStackFrames && nativeStackFrames.frames.reverse()) || [],
+        frames: nativeStackFrames?.frames.reverse() || [],
       },
     },
-    appleDebugImages: (nativeStackFrames && (nativeStackFrames.debugMetaImages as DebugImage[])) || [],
+    appleDebugImages: (nativeStackFrames?.debugMetaImages as DebugImage[]) || [],
   };
 }
 

packages/core/src/js/integrations/reactnativeerrorhandlers.ts

@@ -1,6 +1,7 @@
 import type { EventHint, Integration, SeverityLevel } from '@sentry/core';
 import { addExceptionMechanism, captureException, getClient, getCurrentScope, logger } from '@sentry/core';
 
+import type { ReactNativeClientOptions } from '../options';
 import { createSyntheticError, isErrorLike } from '../utils/error';
 import { RN_GLOBAL_OBJ } from '../utils/worldwide';
 import { checkPromiseAndWarn, polyfillPromise, requireRejectionTracking } from './reactnativeerrorhandlersutils';
@@ -99,7 +100,7 @@ function setupErrorUtilsGlobalHandler(): void {
     return;
   }
 
-  const defaultHandler = errorUtils.getGlobalHandler && errorUtils.getGlobalHandler();
+  const defaultHandler = errorUtils.getGlobalHandler?.();
 
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
   errorUtils.setGlobalHandler(async (error: any, isFatal?: boolean) => {
@@ -155,7 +156,7 @@ function setupErrorUtilsGlobalHandler(): void {
       return;
     }
 
-    void client.flush(client.getOptions().shutdownTimeout || 2000).then(
+    void client.flush((client.getOptions() as ReactNativeClientOptions).shutdownTimeout || 2000).then(
       () => {
         defaultHandler(error, isFatal);
       },

packages/core/src/js/integrations/screenshot.ts

@@ -18,13 +18,13 @@ export const screenshotIntegration = (): Integration => {
 };
 
 async function processEvent(event: Event, hint: EventHint, client: ReactNativeClient): Promise<Event> {
-  const hasException = event.exception && event.exception.values && event.exception.values.length > 0;
+  const hasException = event.exception?.values?.length > 0;
   if (!hasException || client.getOptions().beforeScreenshot?.(event, hint) === false) {
     return event;
   }
 
   const screenshots: ScreenshotAttachment[] | null = await NATIVE.captureScreenshot();
-  if (screenshots && screenshots.length > 0) {
+  if (screenshots?.length > 0) {
     hint.attachments = [...screenshots, ...(hint?.attachments || [])];
   }
 

packages/core/src/js/integrations/spotlight.ts

@@ -103,7 +103,7 @@ function getHostnameFromString(urlString: string): string | null {
   const regex = /^(?:\w+:)?\/\/([^/:]+)(:\d+)?(.*)$/;
   const matches = urlString.match(regex);
 
-  if (matches && matches[1]) {
+  if (matches?.[1]) {
     return matches[1];
   } else {
     // Invalid URL format

packages/core/src/js/integrations/viewhierarchy.ts

@@ -21,7 +21,7 @@ export const viewHierarchyIntegration = (): Integration => {
 };
 
 async function processEvent(event: Event, hint: EventHint): Promise<Event> {
-  const hasException = event.exception && event.exception.values && event.exception.values.length > 0;
+  const hasException = event.exception?.values?.length > 0;
   if (!hasException) {
     return event;
   }

packages/core/src/js/options.ts

@@ -234,6 +234,14 @@ export interface BaseReactNativeOptions {
    */
   replaysOnErrorSampleRate?: number;
 
+  /**
+   * Controls how many milliseconds to wait before shutting down. The default is 2 seconds. Setting this too low can cause
+   * problems for sending events from command line applications. Setting it too
+   * high can cause the application to block for users with network connectivity
+   * problems.
+   */
+  shutdownTimeout?: number;
+
   /**
    * Options which are in beta, or otherwise not guaranteed to be stable.
    */

packages/core/src/js/profiling/convertHermesProfile.ts

@@ -163,7 +163,7 @@ function mapStacks(
     while (currentHermesFrameId !== undefined) {
       const sentryFrameId = hermesStackFrameIdToSentryFrameIdMap.get(currentHermesFrameId);
       sentryFrameId !== undefined && stack.push(sentryFrameId);
-      currentHermesFrameId = hermesStackFrames[currentHermesFrameId] && hermesStackFrames[currentHermesFrameId].parent;
+      currentHermesFrameId = hermesStackFrames[currentHermesFrameId]?.parent;
     }
     stacks.push(stack);
   }

packages/core/src/js/profiling/integration.ts

@@ -128,7 +128,7 @@ export const hermesProfilingIntegration = (initOptions: HermesProfilingOptions =
     }
 
     const client = getClient<ReactNativeClient>();
-    const options = client && client.getOptions();
+    const options = client?.getOptions?.();
 
     const profilesSampleRate =
       options && typeof options.profilesSampleRate === 'number' ? options.profilesSampleRate : undefined;

packages/core/src/js/profiling/utils.ts

@@ -75,12 +75,12 @@ export function enrichCombinedProfileWithEventContext(
     return null;
   }
 
-  const trace_id = (event.contexts && event.contexts.trace && event.contexts.trace.trace_id) || '';
+  const trace_id = event.contexts?.trace?.trace_id || '';
 
   // Log a warning if the profile has an invalid traceId (should be uuidv4).
   // All profiles and transactions are rejected if this is the case and we want to
   // warn users that this is happening if they enable debug flag
-  if (trace_id && trace_id.length !== 32) {
+  if (trace_id?.length !== 32) {
     if (__DEV__) {
       logger.log(`[Profiling] Invalid traceId: ${trace_id} on profiled event`);
     }
@@ -97,25 +97,25 @@ export function enrichCombinedProfileWithEventContext(
     release: event.release || '',
     environment: event.environment || getDefaultEnvironment(),
     os: {
-      name: (event.contexts && event.contexts.os && event.contexts.os.name) || '',
-      version: (event.contexts && event.contexts.os && event.contexts.os.version) || '',
-      build_number: (event.contexts && event.contexts.os && event.contexts.os.build) || '',
+      name: event.contexts?.os?.name || '',
+      version: event.contexts?.os?.version || '',
+      build_number: event.contexts?.os?.build || '',
     },
     device: {
-      locale: (event.contexts && event.contexts.device && (event.contexts.device.locale as string)) || '',
-      model: (event.contexts && event.contexts.device && event.contexts.device.model) || '',
-      manufacturer: (event.contexts && event.contexts.device && event.contexts.device.manufacturer) || '',
-      architecture: (event.contexts && event.contexts.device && event.contexts.device.arch) || '',
-      is_emulator: (event.contexts && event.contexts.device && event.contexts.device.simulator) || false,
+      locale: (event.contexts?.device && (event.contexts.device.locale as string)) || '',
+      model: event.contexts?.device?.model || '',
+      manufacturer: event.contexts?.device?.manufacturer || '',
+      architecture: event.contexts?.device?.arch || '',
+      is_emulator: event.contexts?.device?.simulator || false,
     },
     transaction: {
       name: event.transaction || '',
       id: event.event_id || '',
       trace_id,
-      active_thread_id: (profile.transaction && profile.transaction.active_thread_id) || '',
+      active_thread_id: profile.transaction?.active_thread_id || '',
     },
     debug_meta: {
-      images: [...getDebugMetadata(), ...((profile.debug_meta && profile.debug_meta.images) || [])],
+      images: [...getDebugMetadata(), ...(profile.debug_meta?.images || [])],
     },
   };
 }
@@ -136,19 +136,15 @@ export function enrichAndroidProfileWithEventContext(
     build_id: profile.build_id || '',
 
     device_cpu_frequencies: [],
-    device_is_emulator: (event.contexts && event.contexts.device && event.contexts.device.simulator) || false,
-    device_locale: (event.contexts && event.contexts.device && (event.contexts.device.locale as string)) || '',
-    device_manufacturer: (event.contexts && event.contexts.device && event.contexts.device.manufacturer) || '',
-    device_model: (event.contexts && event.contexts.device && event.contexts.device.model) || '',
-    device_os_name: (event.contexts && event.contexts.os && event.contexts.os.name) || '',
-    device_os_version: (event.contexts && event.contexts.os && event.contexts.os.version) || '',
+    device_is_emulator: event.contexts?.device?.simulator || false,
+    device_locale: (event.contexts?.device && (event.contexts.device.locale as string)) || '',
+    device_manufacturer: event.contexts?.device?.manufacturer || '',
+    device_model: event.contexts?.device?.model || '',
+    device_os_name: event.contexts?.os?.name || '',
+    device_os_version: event.contexts?.os?.version || '',
 
     device_physical_memory_bytes:
-      (event.contexts &&
-        event.contexts.device &&
-        event.contexts.device.memory_size &&
-        Number(event.contexts.device.memory_size).toString(10)) ||
-      '',
+      (event.contexts?.device?.memory_size && Number(event.contexts.device.memory_size).toString(10)) || '',
 
     environment: event.environment || getDefaultEnvironment(),
 
@@ -161,7 +157,7 @@ export function enrichAndroidProfileWithEventContext(
 
     transaction_id: event.event_id || '',
     transaction_name: event.transaction || '',
-    trace_id: (event.contexts && event.contexts.trace && event.contexts.trace.trace_id) || '',
+    trace_id: event.contexts?.trace?.trace_id || '',
 
     version_name: event.release || '',
     version_code: event.dist || '',

packages/core/src/js/replay/CustomMask.tsx

@@ -32,7 +32,7 @@ const UnmaskFallback = (viewProps: ViewProps): React.ReactElement => {
   return <View {...viewProps} />;
 };
 
-const hasViewManagerConfig = (nativeComponentName: string): boolean => UIManager.hasViewManagerConfig && UIManager.hasViewManagerConfig(nativeComponentName);
+const hasViewManagerConfig = (nativeComponentName: string): boolean => UIManager.hasViewManagerConfig?.(nativeComponentName);
 
 const Mask = ((): HostComponent<ViewProps> | React.ComponentType<ViewProps> => {
   if (!hasViewManagerConfig(MaskNativeComponentName)) {

packages/core/src/js/replay/mobilereplay.ts

@@ -76,7 +76,7 @@ export const mobileReplayIntegration = (initOptions: MobileReplayOptions = defau
   const options = { ...defaultOptions, ...initOptions };
 
   async function processEvent(event: Event): Promise<Event> {
-    const hasException = event.exception && event.exception.values && event.exception.values.length > 0;
+    const hasException = event.exception?.values?.length > 0;
     if (!hasException) {
       // Event is not an error, will not capture replay
       return event;

packages/core/src/js/sdk.tsx

@@ -157,6 +157,7 @@ export function wrap<P extends Record<string, unknown>>(
   const profilerProps = {
     ...(options?.profilerProps ?? {}),
     name: RootComponent.displayName ?? 'Root',
+    updateProps: {}
   };
 
   const RootApp: React.FC<P> = (appProps) => {