refactor(android): Pass logger to RNSentryBreadcrumb.fromMap instead of fetching from scopes

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: antonis

packages/core/RNSentryAndroidTester/app/src/test/java/io/sentry/rnsentryandroidtester/RNSentryBreadcrumbTest.kt

@@ -1,16 +1,20 @@
 package io.sentry.rnsentryandroidtester
 
 import com.facebook.react.bridge.JavaOnlyMap
+import io.sentry.ILogger
 import io.sentry.SentryLevel
 import io.sentry.react.RNSentryBreadcrumb
 import junit.framework.TestCase.assertEquals
 import junit.framework.TestCase.assertNotNull
 import org.junit.Test
 import org.junit.runner.RunWith
 import org.junit.runners.JUnit4
+import org.mockito.Mockito
 
 @RunWith(JUnit4::class)
 class RNSentryBreadcrumbTest {
+    private val logger = Mockito.mock(ILogger::class.java)
+
     @Test
     fun generatesSentryBreadcrumbFromMap() {
         val testData =
@@ -33,7 +37,7 @@ class RNSentryBreadcrumbTest {
                 "data",
                 testData,
             )
-        val actual = RNSentryBreadcrumb.fromMap(map)
+        val actual = RNSentryBreadcrumb.fromMap(map, logger)
         assertEquals(SentryLevel.ERROR, actual.level)
         assertEquals("testCategory", actual.category)
         assertEquals("testOrigin", actual.origin)
@@ -49,7 +53,7 @@ class RNSentryBreadcrumbTest {
                 "message",
                 "testMessage",
             )
-        val actual = RNSentryBreadcrumb.fromMap(map)
+        val actual = RNSentryBreadcrumb.fromMap(map, logger)
         assertEquals(SentryLevel.INFO, actual.level)
     }
 
@@ -60,7 +64,7 @@ class RNSentryBreadcrumbTest {
                 "message",
                 "testMessage",
             )
-        val actual = RNSentryBreadcrumb.fromMap(map)
+        val actual = RNSentryBreadcrumb.fromMap(map, logger)
         assertNotNull(actual.timestamp)
     }
 
@@ -71,7 +75,7 @@ class RNSentryBreadcrumbTest {
                 "message",
                 "testMessage",
             )
-        val actual = RNSentryBreadcrumb.fromMap(map)
+        val actual = RNSentryBreadcrumb.fromMap(map, logger)
         assertEquals("testMessage", actual.message)
         assertEquals("react-native", actual.origin)
     }

packages/core/android/src/main/java/io/sentry/react/RNSentryBreadcrumb.java

@@ -5,7 +5,6 @@
 import com.facebook.react.bridge.ReadableType;
 import io.sentry.Breadcrumb;
 import io.sentry.ILogger;
-import io.sentry.Sentry;
 import io.sentry.SentryLevel;
 import io.sentry.util.MapObjectReader;
 import java.util.Map;
@@ -41,8 +40,7 @@ public static String getCurrentScreenFrom(ReadableMap from) {
   }
 
   @NotNull
-  public static Breadcrumb fromMap(ReadableMap from) {
-    final @NotNull ILogger logger = Sentry.getCurrentScopes().getOptions().getLogger();
+  public static Breadcrumb fromMap(ReadableMap from, @NotNull ILogger logger) {
     try {
       final @NotNull MapObjectReader reader = new MapObjectReader(toDeepHashMap(from));
       final @NotNull Breadcrumb breadcrumb =

packages/core/android/src/main/java/io/sentry/react/RNSentryModuleImpl.java

@@ -611,7 +611,7 @@ public void setUser(final ReadableMap userKeys, final ReadableMap userDataKeys)
   public void addBreadcrumb(final ReadableMap breadcrumb) {
     Sentry.configureScope(
         scope -> {
-          scope.addBreadcrumb(RNSentryBreadcrumb.fromMap(breadcrumb));
+          scope.addBreadcrumb(RNSentryBreadcrumb.fromMap(breadcrumb, logger));
 
           final @Nullable String screen = RNSentryBreadcrumb.getCurrentScreenFrom(breadcrumb);
           if (screen != null) {