refactor(distribution): Use interface pattern for distribution API

Address PR feedback from #4712:
- Refactor Sentry.distribution() to follow existing replay() pattern
- Create IDistributionApi interface with core methods (checkForUpdateBlocking, checkForUpdate, downloadUpdate)
- Add distributionController to SentryOptions with NoOp default
- Use getCurrentScopes().getScope().getOptions().getDistributionController() access pattern
- Remove reflection-based implementation for type safety
- Provide NoOpDistributionApi fallback when module not available

This follows the established architecture pattern used by replay API,
allowing distribution integrations to register real implementations
via options.setDistributionController() during integration registration.

🤖 Generated with [Claude Code](https://claude.ai/code)

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

Author: runningcode

sentry-android-distribution/src/main/AndroidManifest.xml

@@ -1,8 +1,9 @@
 package io.sentry.android.distribution
 
 import android.content.Context
-import android.content.Intent
-import android.net.Uri
+import io.sentry.IScopes
+import io.sentry.Integration
+import io.sentry.SentryOptions
 import io.sentry.android.distribution.internal.DistributionInternal
 
 /**
@@ -11,7 +12,26 @@ import io.sentry.android.distribution.internal.DistributionInternal
  * Provides functionality to check for app updates and download new versions from Sentry's preprod
  * artifacts system.
  */
-public object Distribution {
+public class Distribution(context: Context) : Integration {
+  private var scopes: IScopes? = null
+  private var sentryOptions: SentryOptions? = null
+
+  /**
+   * Registers the Distribution integration with Sentry.
+   *
+   * @param scopes the Scopes
+   * @param options the options
+   */
+  public override fun register(scopes: IScopes, options: SentryOptions) {
+    // Store scopes and options for use by distribution functionality
+    this.scopes = scopes
+    this.sentryOptions = options
+    // Distribution integration is registered but initialization still requires manual call to
+    // init()
+    // This allows the integration to be discovered by Sentry's auto-discovery mechanism
+    // while maintaining explicit control over when distribution functionality is enabled
+  }
+
   /**
    * Initialize build distribution with default options. This should be called once per process,
    * typically in Application.onCreate().
@@ -32,7 +52,7 @@ public object Distribution {
   public fun init(context: Context, configuration: (DistributionOptions) -> Unit) {
     val options = DistributionOptions()
     configuration(options)
-    DistributionInternal.init(context, options)
+    DistributionInternal(context, options)
   }
 
   /**
@@ -57,15 +77,28 @@ public object Distribution {
   }
 
   /**
-   * Check for available updates asynchronously using a callback.
+   * Check for available updates asynchronously using a Kotlin lambda callback.
    *
    * @param context Android context
-   * @param onResult Callback that will be called with the UpdateStatus result
+   * @param onResult Lambda that will be called with the UpdateStatus result
    */
   public fun checkForUpdate(context: Context, onResult: (UpdateStatus) -> Unit) {
     DistributionInternal.checkForUpdateAsync(context, onResult)
   }
 
+  /**
+   * Download and install the provided update by opening the download URL in the default browser or
+   * appropriate application.
+   *
+   * @param context Android context
+   * @param updateInfo Information about the update to download
+   */
+  public fun downloadUpdate(context: Context, updateInfo: Any) {
+    if (updateInfo is UpdateInfo) {
+      DistributionInternal.downloadUpdate(context, updateInfo)
+    }
+  }
+
   /**
    * Download and install the provided update by opening the download URL in the default browser or
    * appropriate application.
@@ -74,14 +107,6 @@ public object Distribution {
    * @param info Information about the update to download
    */
   public fun downloadUpdate(context: Context, info: UpdateInfo) {
-    val browserIntent = Intent(Intent.ACTION_VIEW, Uri.parse(info.downloadUrl))
-    browserIntent.addFlags(Intent.FLAG_ACTIVITY_NEW_TASK)
-
-    try {
-      context.startActivity(browserIntent)
-    } catch (e: android.content.ActivityNotFoundException) {
-      // No application can handle the HTTP/HTTPS URL, typically no browser installed
-      // Silently fail as this is expected behavior in some environments
-    }
+    DistributionInternal.downloadUpdate(context, info)
   }
 }

…try/android/distribution/Distribution.kt …/distribution/DistributionIntegration.ktsentry-android-distribution/src/main/java/io/sentry/android/distribution/Distribution.kt renamed to sentry-android-distribution/src/main/java/io/sentry/android/distribution/DistributionIntegration.kt sentry-android-distribution/src/main/java/io/sentry/android/distribution/Distribution.kt renamed to sentry-android-distribution/src/main/java/io/sentry/android/distribution/DistributionIntegration.kt

@@ -767,6 +767,12 @@ public abstract interface class io/sentry/IContinuousProfiler {
 	public abstract fun stopProfiler (Lio/sentry/ProfileLifecycle;)V
 }
 
+public abstract interface class io/sentry/IDistributionApi {
+	public abstract fun checkForUpdate (Ljava/lang/Object;)V
+	public abstract fun checkForUpdateBlocking ()Ljava/lang/Object;
+	public abstract fun downloadUpdate (Ljava/lang/Object;)V
+}
+
 public abstract interface class io/sentry/IEnvelopeReader {
 	public abstract fun read (Ljava/io/InputStream;)Lio/sentry/SentryEnvelope;
 }
@@ -1469,6 +1475,13 @@ public final class io/sentry/NoOpContinuousProfiler : io/sentry/IContinuousProfi
 	public fun stopProfiler (Lio/sentry/ProfileLifecycle;)V
 }
 
+public final class io/sentry/NoOpDistributionApi : io/sentry/IDistributionApi {
+	public fun checkForUpdate (Ljava/lang/Object;)V
+	public fun checkForUpdateBlocking ()Ljava/lang/Object;
+	public fun downloadUpdate (Ljava/lang/Object;)V
+	public static fun getInstance ()Lio/sentry/NoOpDistributionApi;
+}
+
 public final class io/sentry/NoOpEnvelopeReader : io/sentry/IEnvelopeReader {
 	public static fun getInstance ()Lio/sentry/NoOpEnvelopeReader;
 	public fun read (Ljava/io/InputStream;)Lio/sentry/SentryEnvelope;
@@ -2551,7 +2564,7 @@ public final class io/sentry/Sentry {
 	public static fun configureScope (Lio/sentry/ScopeCallback;)V
 	public static fun configureScope (Lio/sentry/ScopeType;Lio/sentry/ScopeCallback;)V
 	public static fun continueTrace (Ljava/lang/String;Ljava/util/List;)Lio/sentry/TransactionContext;
-	public static fun distribution ()Ljava/lang/Object;
+	public static fun distribution ()Lio/sentry/IDistributionApi;
 	public static fun endSession ()V
 	public static fun flush (J)V
 	public static fun forkedCurrentScope (Ljava/lang/String;)Lio/sentry/IScopes;
@@ -3306,6 +3319,7 @@ public class io/sentry/SentryOptions {
 	public fun getDiagnosticLevel ()Lio/sentry/SentryLevel;
 	public fun getDist ()Ljava/lang/String;
 	public fun getDistinctId ()Ljava/lang/String;
+	public fun getDistributionController ()Lio/sentry/IDistributionApi;
 	public fun getDsn ()Ljava/lang/String;
 	public fun getEnvelopeDiskCache ()Lio/sentry/cache/IEnvelopeCache;
 	public fun getEnvelopeReader ()Lio/sentry/IEnvelopeReader;
@@ -3440,6 +3454,7 @@ public class io/sentry/SentryOptions {
 	public fun setDiagnosticLevel (Lio/sentry/SentryLevel;)V
 	public fun setDist (Ljava/lang/String;)V
 	public fun setDistinctId (Ljava/lang/String;)V
+	public fun setDistributionController (Lio/sentry/IDistributionApi;)V
 	public fun setDsn (Ljava/lang/String;)V
 	public fun setEnableAppStartProfiling (Z)V
 	public fun setEnableAutoSessionTracking (Z)V

sentry-android-distribution/src/main/java/io/sentry/android/distribution/internal/DistributionInternal.kt

@@ -0,0 +1,37 @@
+package io.sentry;
+
+import org.jetbrains.annotations.NotNull;
+
+/**
+ * API for Sentry Build Distribution functionality.
+ *
+ * <p>Provides methods to check for app updates and download new versions from Sentry's preprod
+ * artifacts system.
+ */
+public interface IDistributionApi {
+
+  /**
+   * Check for available updates synchronously (blocking call). This method will block the calling
+   * thread while making the network request. Consider using checkForUpdate with callback for
+   * non-blocking behavior.
+   *
+   * @return UpdateStatus indicating if an update is available, up to date, or error
+   */
+  @NotNull
+  Object checkForUpdateBlocking();
+
+  /**
+   * Check for available updates asynchronously using a callback.
+   *
+   * @param onResult Callback that will be called with the UpdateStatus result
+   */
+  void checkForUpdate(@NotNull Object onResult);
+
+  /**
+   * Download and install the provided update by opening the download URL in the default browser or
+   * appropriate application.
+   *
+   * @param info Information about the update to download
+   */
+  void downloadUpdate(@NotNull Object info);
+}

sentry/api/sentry.api

@@ -0,0 +1,31 @@
+package io.sentry;
+
+import org.jetbrains.annotations.NotNull;
+
+/** No-op implementation of IDistributionApi. Used when distribution module is not available. */
+public final class NoOpDistributionApi implements IDistributionApi {
+
+  private static final NoOpDistributionApi instance = new NoOpDistributionApi();
+
+  private NoOpDistributionApi() {}
+
+  public static NoOpDistributionApi getInstance() {
+    return instance;
+  }
+
+  @Override
+  public @NotNull Object checkForUpdateBlocking() {
+    // Return a no-op result - could be null or an error status
+    return new Object(); // This will need to be properly typed when the actual types are available
+  }
+
+  @Override
+  public void checkForUpdate(@NotNull Object onResult) {
+    // No-op implementation - do nothing
+  }
+
+  @Override
+  public void downloadUpdate(@NotNull Object info) {
+    // No-op implementation - do nothing
+  }
+}

sentry/src/main/java/io/sentry/IDistributionApi.java

@@ -1307,15 +1307,9 @@ public static IReplayApi replay() {
    *
    * @return The distribution API object that provides update checking functionality
    */
-  public static @Nullable Object distribution() {
-    try {
-      // Try to get the Distribution object via reflection
-      Class<?> distributionClass = Class.forName("io.sentry.android.distribution.Distribution");
-      return distributionClass.getField("INSTANCE").get(null);
-    } catch (Exception e) {
-      // Distribution module not available, return null
-      return null;
-    }
+  @NotNull
+  public static IDistributionApi distribution() {
+    return getCurrentScopes().getScope().getOptions().getDistributionController();
   }
 
   public static void showUserFeedbackDialog() {

sentry/src/main/java/io/sentry/NoOpDistributionApi.java

@@ -531,6 +531,8 @@ public class SentryOptions {
 
   private @NotNull ReplayController replayController = NoOpReplayController.getInstance();
 
+  private @NotNull IDistributionApi distributionController = NoOpDistributionApi.getInstance();
+
   /**
    * Controls whether to enable screen tracking. When enabled, the SDK will automatically capture
    * screen transitions as context for events.
@@ -2822,6 +2824,15 @@ public void setReplayController(final @Nullable ReplayController replayControlle
         replayController != null ? replayController : NoOpReplayController.getInstance();
   }
 
+  public @NotNull IDistributionApi getDistributionController() {
+    return distributionController;
+  }
+
+  public void setDistributionController(final @Nullable IDistributionApi distributionController) {
+    this.distributionController =
+        distributionController != null ? distributionController : NoOpDistributionApi.getInstance();
+  }
+
   @ApiStatus.Experimental
   public boolean isEnableScreenTracking() {
     return enableScreenTracking;