@sentry/react-native 6.0.0 → 6.2.0

package/README.md

@@ -1,12 +1,8 @@
-<p align="center">
-  <a href="https://sentry.io/?utm_source=github&utm_medium=logo" target="_blank">
-    <picture>
-      <source srcset="https://sentry-brand.storage.googleapis.com/sentry-logo-white.png" media="(prefers-color-scheme: dark)" />
-      <source srcset="https://sentry-brand.storage.googleapis.com/sentry-logo-black.png" media="(prefers-color-scheme: light), (prefers-color-scheme: no-preference)" />
-      <img src="https://sentry-brand.storage.googleapis.com/sentry-logo-black.png" alt="Sentry" width="280">
-    </picture>
-  </a>
-</p>
+<div align="center">
+    <a href="https://sentry.io/?utm_source=github&utm_medium=logo" target="_blank">
+        <img src="https://sentry-brand.storage.googleapis.com/github-banners/github-sdk-react.jpg" alt="Sentry for React Native">
+    </a>
+</div>
 
 _Bad software is everywhere, and we're tired of it. Sentry is on a mission to help developers write better software faster, so we can get back to enjoying technology. If you want to join us [<kbd>**Check out our open positions**</kbd>](https://sentry.io/careers/)_
 

package/RNSentry.podspec

@@ -37,7 +37,7 @@ Pod::Spec.new do |s|
 
   s.compiler_flags = other_cflags
 
-  s.dependency 'Sentry/HybridSDK', '8.37.0'
+  s.dependency 'Sentry/HybridSDK', '8.40.1'
 
   if defined? install_modules_dependencies
     # Default React Native dependencies for 0.71 and above (new and legacy architecture)

package/android/build.gradle

@@ -54,5 +54,5 @@ android {
 
 dependencies {
     implementation 'com.facebook.react:react-native:+'
-    api 'io.sentry:sentry-android:7.15.0'
+    api 'io.sentry:sentry-android:7.17.0'
 }

package/android/src/main/java/io/sentry/react/RNSentryBreadcrumb.java

@@ -7,13 +7,17 @@ import java.util.Map;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
-public class RNSentryBreadcrumb {
+public final class RNSentryBreadcrumb {
+
+  private RNSentryBreadcrumb() {
+    throw new AssertionError("Utility class should not be instantiated");
+  }
 
   @Nullable
   public static String getCurrentScreenFrom(ReadableMap from) {
     final @Nullable String maybeCategory =
         from.hasKey("category") ? from.getString("category") : null;
-    if (maybeCategory == null || !maybeCategory.equals("navigation")) {
+    if (maybeCategory == null || !"navigation".equals(maybeCategory)) {
       return null;
     }
 
@@ -26,7 +30,7 @@ public class RNSentryBreadcrumb {
       // getString might throw if cast to string fails (data.to is not enforced by TS to be a
       // string)
       return maybeData.hasKey("to") ? maybeData.getString("to") : null;
-    } catch (Throwable exception) {
+    } catch (Throwable exception) { // NOPMD - We don't want to crash in any case
       return null;
     }
   }

package/android/src/main/java/io/sentry/react/RNSentryMapConverter.java

@@ -14,11 +14,15 @@ import java.util.List;
 import java.util.Map;
 import org.jetbrains.annotations.Nullable;
 
-public class RNSentryMapConverter {
+public final class RNSentryMapConverter {
   public static final String NAME = "RNSentry.MapConverter";
 
   private static final ILogger logger = new AndroidLogger(NAME);
 
+  private RNSentryMapConverter() {
+    throw new AssertionError("Utility class should not be instantiated");
+  }
+
   public static Object convertToWritable(@Nullable Object serialized) {
     if (serialized instanceof List) {
       WritableArray writable = Arguments.createArray();

package/android/src/main/java/io/sentry/react/RNSentryModuleImpl.java

@@ -20,11 +20,13 @@ import com.facebook.react.bridge.Promise;
 import com.facebook.react.bridge.ReactApplicationContext;
 import com.facebook.react.bridge.ReadableMap;
 import com.facebook.react.bridge.ReadableMapKeySetIterator;
+import com.facebook.react.bridge.ReadableType;
 import com.facebook.react.bridge.UiThreadUtil;
 import com.facebook.react.bridge.WritableArray;
 import com.facebook.react.bridge.WritableMap;
 import com.facebook.react.bridge.WritableNativeArray;
 import com.facebook.react.bridge.WritableNativeMap;
+import com.facebook.react.common.JavascriptException;
 import com.facebook.react.modules.core.DeviceEventManagerModule;
 import io.sentry.HubAdapter;
 import io.sentry.ILogger;
@@ -57,7 +59,6 @@ import io.sentry.android.core.internal.debugmeta.AssetsDebugMetaLoader;
 import io.sentry.android.core.internal.util.SentryFrameMetricsCollector;
 import io.sentry.android.core.performance.AppStartMetrics;
 import io.sentry.protocol.SdkVersion;
-import io.sentry.protocol.SentryException;
 import io.sentry.protocol.SentryId;
 import io.sentry.protocol.SentryPackage;
 import io.sentry.protocol.User;
@@ -91,7 +92,7 @@ public class RNSentryModuleImpl {
   private static final ILogger logger = new AndroidLogger(NAME);
   private static final BuildInfoProvider buildInfo = new BuildInfoProvider(logger);
   private static final String modulesPath = "modules.json";
-  private static final Charset UTF_8 = Charset.forName("UTF-8");
+  private static final Charset UTF_8 = Charset.forName("UTF-8"); // NOPMD - Allow using UTF-8
 
   private final ReactApplicationContext reactApplicationContext;
   private final PackageInfo packageInfo;
@@ -126,10 +127,13 @@ public class RNSentryModuleImpl {
   /** Max trace file size in bytes. */
   private long maxTraceFileSize = 5 * 1024 * 1024;
 
+  private final @NotNull SentryDateProvider dateProvider;
+
   public RNSentryModuleImpl(ReactApplicationContext reactApplicationContext) {
     packageInfo = getPackageInfo(reactApplicationContext);
     this.reactApplicationContext = reactApplicationContext;
     this.emitNewFrameEvent = createEmitNewFrameEvent();
+    this.dateProvider = new SentryAndroidDateProvider();
   }
 
   private ReactApplicationContext getReactApplicationContext() {
@@ -141,8 +145,6 @@ public class RNSentryModuleImpl {
   }
 
   private @NotNull Runnable createEmitNewFrameEvent() {
-    final @NotNull SentryDateProvider dateProvider = new SentryAndroidDateProvider();
-
     return () -> {
       final SentryDate endDate = dateProvider.now();
       WritableMap event = Arguments.createMap();
@@ -174,135 +176,136 @@ public class RNSentryModuleImpl {
   public void initNativeSdk(final ReadableMap rnOptions, Promise promise) {
     SentryAndroid.init(
         this.getReactApplicationContext(),
-        options -> {
-          @Nullable SdkVersion sdkVersion = options.getSdkVersion();
-          if (sdkVersion == null) {
-            sdkVersion = new SdkVersion(ANDROID_SDK_NAME, BuildConfig.VERSION_NAME);
-          } else {
-            sdkVersion.setName(ANDROID_SDK_NAME);
-          }
+        options -> getSentryAndroidOptions(options, rnOptions, logger));
 
-          options.setSentryClientName(sdkVersion.getName() + "/" + sdkVersion.getVersion());
-          options.setNativeSdkName(NATIVE_SDK_NAME);
-          options.setSdkVersion(sdkVersion);
+    promise.resolve(true);
+  }
 
-          if (rnOptions.hasKey("debug") && rnOptions.getBoolean("debug")) {
-            options.setDebug(true);
-          }
-          if (rnOptions.hasKey("dsn") && rnOptions.getString("dsn") != null) {
-            String dsn = rnOptions.getString("dsn");
-            logger.log(SentryLevel.INFO, String.format("Starting with DSN: '%s'", dsn));
-            options.setDsn(dsn);
-          } else {
-            // SentryAndroid needs an empty string fallback for the dsn.
-            options.setDsn("");
-          }
-          if (rnOptions.hasKey("sampleRate")) {
-            options.setSampleRate(rnOptions.getDouble("sampleRate"));
-          }
-          if (rnOptions.hasKey("sendClientReports")) {
-            options.setSendClientReports(rnOptions.getBoolean("sendClientReports"));
-          }
-          if (rnOptions.hasKey("maxBreadcrumbs")) {
-            options.setMaxBreadcrumbs(rnOptions.getInt("maxBreadcrumbs"));
-          }
-          if (rnOptions.hasKey("maxCacheItems")) {
-            options.setMaxCacheItems(rnOptions.getInt("maxCacheItems"));
-          }
-          if (rnOptions.hasKey("environment") && rnOptions.getString("environment") != null) {
-            options.setEnvironment(rnOptions.getString("environment"));
-          }
-          if (rnOptions.hasKey("release") && rnOptions.getString("release") != null) {
-            options.setRelease(rnOptions.getString("release"));
-          }
-          if (rnOptions.hasKey("dist") && rnOptions.getString("dist") != null) {
-            options.setDist(rnOptions.getString("dist"));
-          }
-          if (rnOptions.hasKey("enableAutoSessionTracking")) {
-            options.setEnableAutoSessionTracking(rnOptions.getBoolean("enableAutoSessionTracking"));
-          }
-          if (rnOptions.hasKey("sessionTrackingIntervalMillis")) {
-            options.setSessionTrackingIntervalMillis(
-                rnOptions.getInt("sessionTrackingIntervalMillis"));
-          }
-          if (rnOptions.hasKey("shutdownTimeout")) {
-            options.setShutdownTimeoutMillis(rnOptions.getInt("shutdownTimeout"));
-          }
-          if (rnOptions.hasKey("enableNdkScopeSync")) {
-            options.setEnableScopeSync(rnOptions.getBoolean("enableNdkScopeSync"));
-          }
-          if (rnOptions.hasKey("attachStacktrace")) {
-            options.setAttachStacktrace(rnOptions.getBoolean("attachStacktrace"));
-          }
-          if (rnOptions.hasKey("attachThreads")) {
-            // JS use top level stacktrace and android attaches Threads which hides them so
-            // by default we hide.
-            options.setAttachThreads(rnOptions.getBoolean("attachThreads"));
-          }
-          if (rnOptions.hasKey("attachScreenshot")) {
-            options.setAttachScreenshot(rnOptions.getBoolean("attachScreenshot"));
-          }
-          if (rnOptions.hasKey("attachViewHierarchy")) {
-            options.setAttachViewHierarchy(rnOptions.getBoolean("attachViewHierarchy"));
-          }
-          if (rnOptions.hasKey("sendDefaultPii")) {
-            options.setSendDefaultPii(rnOptions.getBoolean("sendDefaultPii"));
-          }
-          if (rnOptions.hasKey("maxQueueSize")) {
-            options.setMaxQueueSize(rnOptions.getInt("maxQueueSize"));
-          }
-          if (rnOptions.hasKey("enableNdk")) {
-            options.setEnableNdk(rnOptions.getBoolean("enableNdk"));
-          }
-          if (rnOptions.hasKey("_experiments")) {
-            options.getExperimental().setSessionReplay(getReplayOptions(rnOptions));
-            options
-                .getReplayController()
-                .setBreadcrumbConverter(new RNSentryReplayBreadcrumbConverter());
-          }
-          options.setBeforeSend(
-              (event, hint) -> {
-                // React native internally throws a JavascriptException
-                // Since we catch it before that, we don't want to send this one
-                // because we would send it twice
-                try {
-                  SentryException ex = event.getExceptions().get(0);
-                  if (null != ex && ex.getType().contains("JavascriptException")) {
-                    return null;
-                  }
-                } catch (Throwable ignored) {
-                  // We do nothing
-                }
+  protected void getSentryAndroidOptions(
+      @NotNull SentryAndroidOptions options, @NotNull ReadableMap rnOptions, ILogger logger) {
+    @Nullable SdkVersion sdkVersion = options.getSdkVersion();
+    if (sdkVersion == null) {
+      sdkVersion = new SdkVersion(ANDROID_SDK_NAME, BuildConfig.VERSION_NAME);
+    } else {
+      sdkVersion.setName(ANDROID_SDK_NAME);
+    }
+
+    options.setSentryClientName(sdkVersion.getName() + "/" + sdkVersion.getVersion());
+    options.setNativeSdkName(NATIVE_SDK_NAME);
+    options.setSdkVersion(sdkVersion);
+
+    if (rnOptions.hasKey("debug") && rnOptions.getBoolean("debug")) {
+      options.setDebug(true);
+    }
+    if (rnOptions.hasKey("dsn") && rnOptions.getString("dsn") != null) {
+      String dsn = rnOptions.getString("dsn");
+      logger.log(SentryLevel.INFO, String.format("Starting with DSN: '%s'", dsn));
+      options.setDsn(dsn);
+    } else {
+      // SentryAndroid needs an empty string fallback for the dsn.
+      options.setDsn("");
+    }
+    if (rnOptions.hasKey("sampleRate")) {
+      options.setSampleRate(rnOptions.getDouble("sampleRate"));
+    }
+    if (rnOptions.hasKey("sendClientReports")) {
+      options.setSendClientReports(rnOptions.getBoolean("sendClientReports"));
+    }
+    if (rnOptions.hasKey("maxBreadcrumbs")) {
+      options.setMaxBreadcrumbs(rnOptions.getInt("maxBreadcrumbs"));
+    }
+    if (rnOptions.hasKey("maxCacheItems")) {
+      options.setMaxCacheItems(rnOptions.getInt("maxCacheItems"));
+    }
+    if (rnOptions.hasKey("environment") && rnOptions.getString("environment") != null) {
+      options.setEnvironment(rnOptions.getString("environment"));
+    }
+    if (rnOptions.hasKey("release") && rnOptions.getString("release") != null) {
+      options.setRelease(rnOptions.getString("release"));
+    }
+    if (rnOptions.hasKey("dist") && rnOptions.getString("dist") != null) {
+      options.setDist(rnOptions.getString("dist"));
+    }
+    if (rnOptions.hasKey("enableAutoSessionTracking")) {
+      options.setEnableAutoSessionTracking(rnOptions.getBoolean("enableAutoSessionTracking"));
+    }
+    if (rnOptions.hasKey("sessionTrackingIntervalMillis")) {
+      options.setSessionTrackingIntervalMillis(rnOptions.getInt("sessionTrackingIntervalMillis"));
+    }
+    if (rnOptions.hasKey("shutdownTimeout")) {
+      options.setShutdownTimeoutMillis(rnOptions.getInt("shutdownTimeout"));
+    }
+    if (rnOptions.hasKey("enableNdkScopeSync")) {
+      options.setEnableScopeSync(rnOptions.getBoolean("enableNdkScopeSync"));
+    }
+    if (rnOptions.hasKey("attachStacktrace")) {
+      options.setAttachStacktrace(rnOptions.getBoolean("attachStacktrace"));
+    }
+    if (rnOptions.hasKey("attachThreads")) {
+      // JS use top level stacktrace and android attaches Threads which hides them so
+      // by default we hide.
+      options.setAttachThreads(rnOptions.getBoolean("attachThreads"));
+    }
+    if (rnOptions.hasKey("attachScreenshot")) {
+      options.setAttachScreenshot(rnOptions.getBoolean("attachScreenshot"));
+    }
+    if (rnOptions.hasKey("attachViewHierarchy")) {
+      options.setAttachViewHierarchy(rnOptions.getBoolean("attachViewHierarchy"));
+    }
+    if (rnOptions.hasKey("sendDefaultPii")) {
+      options.setSendDefaultPii(rnOptions.getBoolean("sendDefaultPii"));
+    }
+    if (rnOptions.hasKey("maxQueueSize")) {
+      options.setMaxQueueSize(rnOptions.getInt("maxQueueSize"));
+    }
+    if (rnOptions.hasKey("enableNdk")) {
+      options.setEnableNdk(rnOptions.getBoolean("enableNdk"));
+    }
+    if (rnOptions.hasKey("spotlight")) {
+      if (rnOptions.getType("spotlight") == ReadableType.Boolean) {
+        options.setEnableSpotlight(rnOptions.getBoolean("spotlight"));
+        options.setSpotlightConnectionUrl(rnOptions.getString("defaultSidecarUrl"));
+      } else if (rnOptions.getType("spotlight") == ReadableType.String) {
+        options.setEnableSpotlight(true);
+        options.setSpotlightConnectionUrl(rnOptions.getString("spotlight"));
+      }
+    }
+    if (rnOptions.hasKey("_experiments")) {
+      options.getExperimental().setSessionReplay(getReplayOptions(rnOptions));
+      options.getReplayController().setBreadcrumbConverter(new RNSentryReplayBreadcrumbConverter());
+    }
 
-                setEventOriginTag(event);
-                addPackages(event, options.getSdkVersion());
+    // React native internally throws a JavascriptException.
+    // we want to ignore it on the native side to avoid sending it twice.
+    options.addIgnoredExceptionForType(JavascriptException.class);
 
-                return event;
-              });
+    options.setBeforeSend(
+        (event, hint) -> {
+          setEventOriginTag(event);
+          addPackages(event, options.getSdkVersion());
 
-          if (rnOptions.hasKey("enableNativeCrashHandling")
-              && !rnOptions.getBoolean("enableNativeCrashHandling")) {
-            final List<Integration> integrations = options.getIntegrations();
-            for (final Integration integration : integrations) {
-              if (integration instanceof UncaughtExceptionHandlerIntegration
-                  || integration instanceof AnrIntegration
-                  || integration instanceof NdkIntegration) {
-                integrations.remove(integration);
-              }
-            }
-          }
-          logger.log(
-              SentryLevel.INFO,
-              String.format("Native Integrations '%s'", options.getIntegrations()));
-
-          final CurrentActivityHolder currentActivityHolder = CurrentActivityHolder.getInstance();
-          final Activity currentActivity = getCurrentActivity();
-          if (currentActivity != null) {
-            currentActivityHolder.setActivity(currentActivity);
-          }
+          return event;
         });
 
-    promise.resolve(true);
+    if (rnOptions.hasKey("enableNativeCrashHandling")
+        && !rnOptions.getBoolean("enableNativeCrashHandling")) {
+      final List<Integration> integrations = options.getIntegrations();
+      for (final Integration integration : integrations) {
+        if (integration instanceof UncaughtExceptionHandlerIntegration
+            || integration instanceof AnrIntegration
+            || integration instanceof NdkIntegration) {
+          integrations.remove(integration);
+        }
+      }
+    }
+    logger.log(
+        SentryLevel.INFO, String.format("Native Integrations '%s'", options.getIntegrations()));
+
+    final CurrentActivityHolder currentActivityHolder = CurrentActivityHolder.getInstance();
+    final Activity currentActivity = getCurrentActivity();
+    if (currentActivity != null) {
+      currentActivityHolder.setActivity(currentActivity);
+    }
   }
 
   private SentryReplayOptions getReplayOptions(@NotNull ReadableMap rnOptions) {
@@ -356,12 +359,12 @@ public class RNSentryModuleImpl {
     throw new RuntimeException("TEST - Sentry Client Crash (only works in release mode)");
   }
 
-  public void addListener(String _eventType) {
+  public void addListener(String eventType) {
     // Is must be defined otherwise the generated interface from TS won't be fulfilled
     logger.log(SentryLevel.ERROR, "addListener of NativeEventEmitter can't be used on Android!");
   }
 
-  public void removeListeners(double _id) {
+  public void removeListeners(double id) {
     // Is must be defined otherwise the generated interface from TS won't be fulfilled
     logger.log(
         SentryLevel.ERROR, "removeListeners of NativeEventEmitter can't be used on Android!");
@@ -369,17 +372,16 @@ public class RNSentryModuleImpl {
 
   public void fetchModules(Promise promise) {
     final AssetManager assets = this.getReactApplicationContext().getResources().getAssets();
-    try (final InputStream stream =
-        new BufferedInputStream(assets.open(RNSentryModuleImpl.modulesPath))) {
+    try (InputStream stream = new BufferedInputStream(assets.open(modulesPath))) {
       int size = stream.available();
       byte[] buffer = new byte[size];
       stream.read(buffer);
       stream.close();
-      String modulesJson = new String(buffer, RNSentryModuleImpl.UTF_8);
+      String modulesJson = new String(buffer, UTF_8);
       promise.resolve(modulesJson);
     } catch (FileNotFoundException e) {
       promise.resolve(null);
-    } catch (Throwable e) {
+    } catch (Throwable e) { // NOPMD - We don't want to crash in any case
       logger.log(SentryLevel.WARNING, "Fetching JS Modules failed.");
       promise.resolve(null);
     }
@@ -462,7 +464,7 @@ public class RNSentryModuleImpl {
         map.putInt("frozenFrames", frozenFrames);
 
         promise.resolve(map);
-      } catch (Throwable ignored) {
+      } catch (Throwable ignored) { // NOPMD - We don't want to crash in any case
         logger.log(SentryLevel.WARNING, "Error fetching native frames.");
         promise.resolve(null);
       }
@@ -493,7 +495,7 @@ public class RNSentryModuleImpl {
     try {
       InternalSentrySdk.captureEnvelope(
           bytes, !options.hasKey("hardCrashed") || !options.getBoolean("hardCrashed"));
-    } catch (Throwable e) {
+    } catch (Throwable e) { // NOPMD - We don't want to crash in any case
       logger.log(SentryLevel.ERROR, "Error while capturing envelope");
       promise.resolve(false);
     }
@@ -511,7 +513,7 @@ public class RNSentryModuleImpl {
 
     final byte[] raw = takeScreenshotOnUiThread(activity);
 
-    if (raw == null) {
+    if (raw == null || raw.length == 0) {
       logger.log(SentryLevel.WARNING, "Screenshot is null, screen was not captured.");
       promise.resolve(null);
       return;
@@ -550,7 +552,7 @@ public class RNSentryModuleImpl {
       doneSignal.await(SCREENSHOT_TIMEOUT_SECONDS, SECONDS);
     } catch (InterruptedException e) {
       logger.log(SentryLevel.ERROR, "Screenshot process was interrupted.");
-      return null;
+      return new byte[0];
     }
 
     return bytesWrapper[0];
@@ -627,7 +629,7 @@ public class RNSentryModuleImpl {
             }
 
             if (userDataKeys != null) {
-              HashMap<String, String> userDataMap = new HashMap<>();
+              Map<String, String> userDataMap = new HashMap<>();
               ReadableMapKeySetIterator it = userDataKeys.keySetIterator();
               while (it.hasNextKey()) {
                 String key = it.nextKey();
@@ -694,7 +696,7 @@ public class RNSentryModuleImpl {
             return;
           }
 
-          final HashMap<String, Object> contextHashMap = context.toHashMap();
+          final Map<String, Object> contextHashMap = context.toHashMap();
           scope.setContexts(key, contextHashMap);
         });
   }
@@ -726,7 +728,7 @@ public class RNSentryModuleImpl {
           frameMetricsAggregator.add(currentActivity);
 
           logger.log(SentryLevel.INFO, "FrameMetricsAggregator installed.");
-        } catch (Throwable ignored) {
+        } catch (Throwable ignored) { // NOPMD - We don't want to crash in any case
           // throws ConcurrentModification when calling addOnFrameMetricsAvailableListener
           // this is a best effort since we can't reproduce it
           logger.log(SentryLevel.ERROR, "Error adding Activity to frameMetricsAggregator.");
@@ -746,6 +748,10 @@ public class RNSentryModuleImpl {
     }
   }
 
+  public void getNewScreenTimeToDisplay(Promise promise) {
+    RNSentryTimeToDisplay.getTimeToDisplay(promise, dateProvider);
+  }
+
   private String getProfilingTracesDirPath() {
     if (cacheDirPath == null) {
       cacheDirPath =
@@ -785,7 +791,7 @@ public class RNSentryModuleImpl {
       }
 
       result.putBoolean("started", true);
-    } catch (Throwable e) {
+    } catch (Throwable e) { // NOPMD - We don't want to crash in any case
       result.putBoolean("started", false);
       result.putString("error", e.toString());
     }
@@ -825,7 +831,7 @@ public class RNSentryModuleImpl {
         androidProfile.putString("build_id", getProguardUuid());
         result.putMap("androidProfile", androidProfile);
       }
-    } catch (Throwable e) {
+    } catch (Throwable e) { // NOPMD - We don't want to crash in any case
       result.putString("error", e.toString());
     } finally {
       if (output != null) {
@@ -834,7 +840,7 @@ public class RNSentryModuleImpl {
           if (!wasProfileSuccessfullyDeleted) {
             logger.log(SentryLevel.WARNING, "Profile not deleted from:" + output.getAbsolutePath());
           }
-        } catch (Throwable e) {
+        } catch (Throwable e) { // NOPMD - We don't want to crash in any case
           logger.log(SentryLevel.WARNING, "Profile not deleted from:" + output.getAbsolutePath());
         }
       }
@@ -848,7 +854,7 @@ public class RNSentryModuleImpl {
     }
     isProguardDebugMetaLoaded = true;
     final @Nullable List<Properties> debugMetaList =
-        (new AssetsDebugMetaLoader(this.getReactApplicationContext(), logger)).loadDebugMeta();
+        new AssetsDebugMetaLoader(this.getReactApplicationContext(), logger).loadDebugMeta();
     if (debugMetaList == null) {
       return null;
     }
@@ -866,7 +872,7 @@ public class RNSentryModuleImpl {
   }
 
   private String readStringFromFile(File path) throws IOException {
-    try (final BufferedReader br = new BufferedReader(new FileReader(path)); ) {
+    try (BufferedReader br = new BufferedReader(new FileReader(path)); ) {
 
       final StringBuilder text = new StringBuilder();
       String line;
@@ -943,7 +949,7 @@ public class RNSentryModuleImpl {
   private void addPackages(SentryEvent event, SdkVersion sdk) {
     SdkVersion eventSdk = event.getSdk();
     if (eventSdk != null
-        && eventSdk.getName().equals("sentry.javascript.react-native")
+        && "sentry.javascript.react-native".equals(eventSdk.getName())
         && sdk != null) {
       List<SentryPackage> sentryPackages = sdk.getPackages();
       if (sentryPackages != null) {

package/android/src/main/java/io/sentry/react/RNSentryReactFragmentLifecycleTracer.java

@@ -40,7 +40,7 @@ public class RNSentryReactFragmentLifecycleTracer extends FragmentLifecycleCallb
       @NotNull Fragment f,
       @NotNull View v,
       @Nullable Bundle savedInstanceState) {
-    if (!("com.swmansion.rnscreens.ScreenStackFragment".equals(f.getClass().getCanonicalName()))) {
+    if (!"com.swmansion.rnscreens.ScreenStackFragment".equals(f.getClass().getCanonicalName())) {
       logger.log(
           SentryLevel.DEBUG,
           "Fragment is not a ScreenStackFragment, won't listen for the first draw.");

package/android/src/main/java/io/sentry/react/RNSentryReplayBreadcrumbConverter.java

@@ -13,8 +13,6 @@ import org.jetbrains.annotations.Nullable;
 import org.jetbrains.annotations.TestOnly;
 
 public final class RNSentryReplayBreadcrumbConverter extends DefaultReplayBreadcrumbConverter {
-  public RNSentryReplayBreadcrumbConverter() {}
-
   @Override
   public @Nullable RRWebEvent convert(final @NotNull Breadcrumb breadcrumb) {
     if (breadcrumb.getCategory() == null) {
@@ -22,22 +20,22 @@ public final class RNSentryReplayBreadcrumbConverter extends DefaultReplayBreadc
     }
 
     // Do not add Sentry Event breadcrumbs to replay
-    if (breadcrumb.getCategory().equals("sentry.event")
-        || breadcrumb.getCategory().equals("sentry.transaction")) {
+    if ("sentry.event".equals(breadcrumb.getCategory())
+        || "sentry.transaction".equals(breadcrumb.getCategory())) {
       return null;
     }
-    if (breadcrumb.getCategory().equals("http")) {
+    if ("http".equals(breadcrumb.getCategory())) {
       // Drop native http breadcrumbs to avoid duplicates
       return null;
     }
 
-    if (breadcrumb.getCategory().equals("touch")) {
+    if ("touch".equals(breadcrumb.getCategory())) {
       return convertTouchBreadcrumb(breadcrumb);
     }
-    if (breadcrumb.getCategory().equals("navigation")) {
+    if ("navigation".equals(breadcrumb.getCategory())) {
       return convertNavigationBreadcrumb(breadcrumb);
     }
-    if (breadcrumb.getCategory().equals("xhr")) {
+    if ("xhr".equals(breadcrumb.getCategory())) {
       return convertNetworkBreadcrumb(breadcrumb);
     }
 
@@ -46,8 +44,7 @@ public final class RNSentryReplayBreadcrumbConverter extends DefaultReplayBreadc
     // ignore native navigation breadcrumbs
     if (nativeBreadcrumb instanceof RRWebBreadcrumbEvent) {
       final RRWebBreadcrumbEvent rrWebBreadcrumb = (RRWebBreadcrumbEvent) nativeBreadcrumb;
-      if (rrWebBreadcrumb.getCategory() != null
-          && rrWebBreadcrumb.getCategory().equals("navigation")) {
+      if ("navigation".equals(rrWebBreadcrumb.getCategory())) {
         return null;
       }
     }
@@ -69,8 +66,7 @@ public final class RNSentryReplayBreadcrumbConverter extends DefaultReplayBreadc
 
     rrWebBreadcrumb.setCategory("ui.tap");
 
-    rrWebBreadcrumb.setMessage(
-        RNSentryReplayBreadcrumbConverter.getTouchPathMessage(breadcrumb.getData("path")));
+    rrWebBreadcrumb.setMessage(getTouchPathMessage(breadcrumb.getData("path")));
 
     setRRWebEventDefaultsFrom(rrWebBreadcrumb, breadcrumb);
     return rrWebBreadcrumb;
@@ -83,7 +79,7 @@ public final class RNSentryReplayBreadcrumbConverter extends DefaultReplayBreadc
     }
 
     final @NotNull List path = (List) maybePath;
-    if (path.size() == 0) {
+    if (path.isEmpty()) {
       return null;
     }
 
@@ -145,7 +141,7 @@ public final class RNSentryReplayBreadcrumbConverter extends DefaultReplayBreadc
       return null;
     }
 
-    final HashMap<String, Object> data = new HashMap<>();
+    final Map<String, Object> data = new HashMap<>();
     if (breadcrumb.getData("method") instanceof String) {
       data.put("method", breadcrumb.getData("method"));
     }