refactor(extend-app-start): Inline the logger lookup instead of injecting it

The extension logs only two warnings, both on rare guard paths in extendAppStart().
Inline Sentry.getCurrentScopes().getOptions().getLogger() at those sites instead of
holding a logger field set at init, dropping the field, setLogger(), and the
AndroidOptionsInitializer wiring. extendAppStart() runs post-init, so the lookup
always yields the configured logger.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: buenaflor

sentry-android-core/src/main/java/io/sentry/android/core/AndroidOptionsInitializer.java

@@ -198,9 +198,7 @@ static void initializeIntegrationsAndProcessors(
     }
 
     final @NotNull AppStartMetrics appStartMetrics = AppStartMetrics.getInstance();
-    final @NotNull AppStartExtension appStartExtension = appStartMetrics.getAppStartExtension();
-    appStartExtension.setLogger(options.getLogger());
-    options.setAppStartExtender(appStartExtension);
+    options.setAppStartExtender(appStartMetrics.getAppStartExtension());
 
     if (options.getModulesLoader() instanceof NoOpModulesLoader) {
       options.setModulesLoader(new AssetsModulesLoader(context, options));

sentry-android-core/src/main/java/io/sentry/android/core/AppStartExtension.java

@@ -1,12 +1,11 @@
 package io.sentry.android.core;
 
 import io.sentry.IAppStartExtender;
-import io.sentry.ILogger;
 import io.sentry.ISentryLifecycleToken;
 import io.sentry.ISpan;
 import io.sentry.ITransaction;
-import io.sentry.NoOpLogger;
 import io.sentry.NoOpSpan;
+import io.sentry.Sentry;
 import io.sentry.SentryDate;
 import io.sentry.SentryLevel;
 import io.sentry.SpanStatus;
@@ -53,10 +52,6 @@ public interface ExtendAppStartListener {
   private final @NotNull AppStartMetrics metrics;
   private final @NotNull AutoClosableReentrantLock lock = new AutoClosableReentrantLock();
 
-  // Set once at SDK init via setLogger(), read later when an extension is requested. Defaults to a
-  // no-op because this component is created before SentryOptions (and its logger) exist.
-  private volatile @NotNull ILogger logger = NoOpLogger.getInstance();
-
   private @Nullable ExtendAppStartListener extendAppStartListener;
   private @Nullable ISpan extendedSpan;
   private @Nullable ITransaction extendedTransaction;
@@ -69,25 +64,27 @@ public void setExtendAppStartListener(final @Nullable ExtendAppStartListener lis
     this.extendAppStartListener = listener;
   }
 
-  void setLogger(final @NotNull ILogger logger) {
-    this.logger = logger;
-  }
-
   @Override
   public void extendAppStart() {
     try (final @NotNull ISentryLifecycleToken ignored = lock.acquire()) {
       if (extendedSpan != null) {
-        logger.log(SentryLevel.WARNING, "App start is already being extended.");
+        Sentry.getCurrentScopes()
+            .getOptions()
+            .getLogger()
+            .log(SentryLevel.WARNING, "App start is already being extended.");
         return;
       }
       // Ignore the foreground check: headless app starts (broadcast/service) run in a
       // non-foreground process but can still be extended. The window gate still rejects an
       // extension once an activity was created, the first frame was drawn, or measurements were
       // already sent.
       if (!metrics.isAppStartWindowOpen()) {
-        logger.log(
-            SentryLevel.WARNING,
-            "Cannot extend app start: the app start window has already passed.");
+        Sentry.getCurrentScopes()
+            .getOptions()
+            .getLogger()
+            .log(
+                SentryLevel.WARNING,
+                "Cannot extend app start: the app start window has already passed.");
         return;
       }
       final @Nullable ExtendAppStartListener listener = extendAppStartListener;