feat(core): Pass SentryOptions to IScopesStorageFactory.create()

SPI-discovered factory implementations are instantiated via ServiceLoader
with no-arg constructors, so they need access to options like logger and
DSN at creation time. Change the interface method signature to accept
SentryOptions as a parameter.

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

Author: adinauer

sentry/api/sentry.api

@@ -1061,7 +1061,7 @@ public abstract interface class io/sentry/IScopesStorage {
 }
 
 public abstract interface class io/sentry/IScopesStorageFactory {
-	public abstract fun create ()Lio/sentry/IScopesStorage;
+	public abstract fun create (Lio/sentry/SentryOptions;)Lio/sentry/IScopesStorage;
 }
 
 public abstract interface class io/sentry/ISentryClient {

sentry/src/main/java/io/sentry/IScopesStorageFactory.java

@@ -7,5 +7,5 @@
 @ApiStatus.Experimental
 public interface IScopesStorageFactory {
   @NotNull
-  IScopesStorage create();
+  IScopesStorage create(@NotNull SentryOptions options);
 }

sentry/src/main/java/io/sentry/Sentry.java

@@ -440,7 +440,7 @@ private static void initFatalLogger(final @NotNull SentryOptions options) {
   private static void initScopesStorage(SentryOptions options) {
     getScopesStorage().close();
     if (options.getScopesStorageFactory() != null) {
-      scopesStorage = options.getScopesStorageFactory().create();
+      scopesStorage = options.getScopesStorageFactory().create(options);
       scopesStorage.init();
     } else if (SentryOpenTelemetryMode.OFF == options.getOpenTelemetryMode()) {
       scopesStorage = new DefaultScopesStorage();

sentry/src/test/java/io/sentry/SentryOptionsTest.kt

@@ -974,15 +974,15 @@ class SentryOptionsTest {
   @Test
   fun `scopesStorageFactory can be set and retrieved`() {
     val options = SentryOptions()
-    val factory = IScopesStorageFactory { DefaultScopesStorage() }
+    val factory = IScopesStorageFactory { _ -> DefaultScopesStorage() }
     options.scopesStorageFactory = factory
     assertSame(factory, options.scopesStorageFactory)
   }
 
   @Test
   fun `scopesStorageFactory can be set to null`() {
     val options = SentryOptions()
-    val factory = IScopesStorageFactory { DefaultScopesStorage() }
+    val factory = IScopesStorageFactory { _ -> DefaultScopesStorage() }
     options.scopesStorageFactory = factory
     options.scopesStorageFactory = null
     assertNull(options.scopesStorageFactory)

sentry/src/test/java/io/sentry/SentryTest.kt

@@ -1732,7 +1732,7 @@ class SentryTest {
 
     initForTest {
       it.dsn = dsn
-      it.scopesStorageFactory = IScopesStorageFactory { customStorage }
+      it.scopesStorageFactory = IScopesStorageFactory { _ -> customStorage }
     }
 
     verify(customStorage).init()
@@ -1758,7 +1758,7 @@ class SentryTest {
 
     initForTest {
       it.dsn = dsn
-      it.scopesStorageFactory = IScopesStorageFactory {
+      it.scopesStorageFactory = IScopesStorageFactory { _ ->
         factoryCalled.set(true)
         backingStorage
       }