refactor(distribution): Use interface pattern for distribution API (#4731)

* 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 <[email protected]>

* feat(distribution): Complete distribution module implementation

- Update AndroidOptionsInitializer to use new DistributionIntegration constructor
- Refactor DistributionIntegration to implement IDistributionApi methods directly
- Remove internal package structure and DistributionInternal dependency
- Update class names from Distribution to DistributionIntegration for clarity
- Convert data classes to regular classes to match API requirements
- Rename organizationSlug to orgSlug for consistency
- Implement downloadUpdate using Android Intent system
- Remove completed Distribution singleton approach

This completes the distribution module implementation to work with
the new interface-based API pattern from the previous commit.

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

Co-Authored-By: Claude <[email protected]>

* feat(distribution): Move UpdateStatus and UpdateInfo to core sentry module

- Move UpdateStatus and UpdateInfo from distribution module to core sentry module
- Update IDistributionApi to use proper types instead of Object
- Add UpdateCallback interface for type-safe async callbacks
- Rename UpdateStatus.Error to UpdateError to avoid java.lang.Error clash
- Update DistributionIntegration to implement IDistributionApi with proper types
- Remove duplicate classes from distribution module
- Regenerate API files with proper type signatures

This provides full type safety for the distribution API while keeping
the types accessible to all modules that might need them.

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

Co-Authored-By: Claude <[email protected]>

* fix(distribution): Update ProGuard rules for new DistributionIntegration class path

- Update ProGuard rules from `internal.DistributionIntegration` to `DistributionIntegration`
- Fixes R8 missing class error in release builds
- Ensures DistributionIntegration class is properly kept during code shrinking

This addresses the build failure in Android integration tests where R8 was
removing the DistributionIntegration class that is referenced by AndroidOptionsInitializer.

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

Co-Authored-By: Claude <[email protected]>

* chore(distribution): Update API dump to reflect interface refactor

- Update distribution module API file after refactoring to use interface pattern
- Class renamed from Distribution to DistributionIntegration
- Now implements IDistributionApi interface with proper method signatures
- Removed UpdateInfo and UpdateStatus classes (moved to core sentry module)
- Constructor now takes Context parameter
- Method signatures use proper types instead of Object parameters

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

Co-Authored-By: Claude <[email protected]>

* refactor(distribution): Move DistributionOptions to SentryOptions to resolve circular dependency

- Move DistributionOptions from SentryAndroidOptions to SentryOptions to resolve circular dependency between sentry-android-core and sentry-android-distribution modules
- Simplify DistributionIntegration.register() to work with SentryOptions directly instead of requiring SentryAndroidOptions
- Remove separate DistributionOptions.kt file
- Update API dumps

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

Co-Authored-By: Claude <[email protected]>

* fix(distribution): Correct DistributionIntegration class name reference in SentryAndroid

---------

Co-authored-by: Claude <[email protected]>

Author: runningcode

sentry-android-core/proguard-rules.pro

@@ -80,6 +80,6 @@
 ##---------------End: proguard configuration for sentry-android-replay  ----------
 
 ##---------------Begin: proguard configuration for sentry-android-distribution  ----------
--dontwarn io.sentry.android.distribution.internal.DistributionIntegration
--keepnames class io.sentry.android.distribution.internal.DistributionIntegration
+-dontwarn io.sentry.android.distribution.DistributionIntegration
+-keepnames class io.sentry.android.distribution.DistributionIntegration
 ##---------------End: proguard configuration for sentry-android-distribution  ----------

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

@@ -32,7 +32,7 @@
 import io.sentry.android.core.internal.util.AndroidThreadChecker;
 import io.sentry.android.core.internal.util.SentryFrameMetricsCollector;
 import io.sentry.android.core.performance.AppStartMetrics;
-import io.sentry.android.distribution.internal.DistributionIntegration;
+import io.sentry.android.distribution.DistributionIntegration;
 import io.sentry.android.fragment.FragmentLifecycleIntegration;
 import io.sentry.android.replay.DefaultReplayBreadcrumbConverter;
 import io.sentry.android.replay.ReplayIntegration;
@@ -394,7 +394,9 @@ static void installDefaultIntegrations(
       options.setReplayController(replay);
     }
     if (isDistributionAvailable) {
-      options.addIntegration(new DistributionIntegration());
+      final DistributionIntegration distribution = new DistributionIntegration((context));
+      options.setDistributionController(distribution);
+      options.addIntegration(distribution);
     }
     options
         .getFeedbackOptions()

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

@@ -42,7 +42,7 @@ public final class SentryAndroid {
       "io.sentry.android.replay.ReplayIntegration";
 
   static final String SENTRY_DISTRIBUTION_INTEGRATION_CLASS_NAME =
-      "io.sentry.android.distribution.internal.DistributionIntegration";
+      "io.sentry.android.distribution.DistributionIntegration";
 
   private static final String TIMBER_CLASS_NAME = "timber.log.Timber";
   private static final String FRAGMENT_CLASS_NAME =

sentry-android-distribution/api/sentry-android-distribution.api

@@ -1,79 +1,8 @@
-public final class io/sentry/android/distribution/Distribution {
-	public static final field INSTANCE Lio/sentry/android/distribution/Distribution;
-	public final fun checkForUpdate (Landroid/content/Context;Lkotlin/jvm/functions/Function1;)V
-	public final fun checkForUpdateBlocking (Landroid/content/Context;)Lio/sentry/android/distribution/UpdateStatus;
-	public final fun downloadUpdate (Landroid/content/Context;Lio/sentry/android/distribution/UpdateInfo;)V
-	public final fun init (Landroid/content/Context;)V
-	public final fun init (Landroid/content/Context;Lkotlin/jvm/functions/Function1;)V
-	public final fun isEnabled ()Z
-}
-
-public final class io/sentry/android/distribution/DistributionOptions {
-	public fun <init> ()V
-	public final fun getBuildConfiguration ()Ljava/lang/String;
-	public final fun getOrgAuthToken ()Ljava/lang/String;
-	public final fun getOrganizationSlug ()Ljava/lang/String;
-	public final fun getProjectSlug ()Ljava/lang/String;
-	public final fun getSentryBaseUrl ()Ljava/lang/String;
-	public final fun setBuildConfiguration (Ljava/lang/String;)V
-	public final fun setOrgAuthToken (Ljava/lang/String;)V
-	public final fun setOrganizationSlug (Ljava/lang/String;)V
-	public final fun setProjectSlug (Ljava/lang/String;)V
-	public final fun setSentryBaseUrl (Ljava/lang/String;)V
-}
-
-public final class io/sentry/android/distribution/UpdateInfo {
-	public fun <init> (Ljava/lang/String;Ljava/lang/String;ILjava/lang/String;Ljava/lang/String;Ljava/lang/String;)V
-	public final fun component1 ()Ljava/lang/String;
-	public final fun component2 ()Ljava/lang/String;
-	public final fun component3 ()I
-	public final fun component4 ()Ljava/lang/String;
-	public final fun component5 ()Ljava/lang/String;
-	public final fun component6 ()Ljava/lang/String;
-	public final fun copy (Ljava/lang/String;Ljava/lang/String;ILjava/lang/String;Ljava/lang/String;Ljava/lang/String;)Lio/sentry/android/distribution/UpdateInfo;
-	public static synthetic fun copy$default (Lio/sentry/android/distribution/UpdateInfo;Ljava/lang/String;Ljava/lang/String;ILjava/lang/String;Ljava/lang/String;Ljava/lang/String;ILjava/lang/Object;)Lio/sentry/android/distribution/UpdateInfo;
-	public fun equals (Ljava/lang/Object;)Z
-	public final fun getAppName ()Ljava/lang/String;
-	public final fun getBuildNumber ()I
-	public final fun getBuildVersion ()Ljava/lang/String;
-	public final fun getCreatedDate ()Ljava/lang/String;
-	public final fun getDownloadUrl ()Ljava/lang/String;
-	public final fun getId ()Ljava/lang/String;
-	public fun hashCode ()I
-	public fun toString ()Ljava/lang/String;
-}
-
-public abstract class io/sentry/android/distribution/UpdateStatus {
-}
-
-public final class io/sentry/android/distribution/UpdateStatus$Error : io/sentry/android/distribution/UpdateStatus {
-	public fun <init> (Ljava/lang/String;)V
-	public final fun component1 ()Ljava/lang/String;
-	public final fun copy (Ljava/lang/String;)Lio/sentry/android/distribution/UpdateStatus$Error;
-	public static synthetic fun copy$default (Lio/sentry/android/distribution/UpdateStatus$Error;Ljava/lang/String;ILjava/lang/Object;)Lio/sentry/android/distribution/UpdateStatus$Error;
-	public fun equals (Ljava/lang/Object;)Z
-	public final fun getMessage ()Ljava/lang/String;
-	public fun hashCode ()I
-	public fun toString ()Ljava/lang/String;
-}
-
-public final class io/sentry/android/distribution/UpdateStatus$NewRelease : io/sentry/android/distribution/UpdateStatus {
-	public fun <init> (Lio/sentry/android/distribution/UpdateInfo;)V
-	public final fun component1 ()Lio/sentry/android/distribution/UpdateInfo;
-	public final fun copy (Lio/sentry/android/distribution/UpdateInfo;)Lio/sentry/android/distribution/UpdateStatus$NewRelease;
-	public static synthetic fun copy$default (Lio/sentry/android/distribution/UpdateStatus$NewRelease;Lio/sentry/android/distribution/UpdateInfo;ILjava/lang/Object;)Lio/sentry/android/distribution/UpdateStatus$NewRelease;
-	public fun equals (Ljava/lang/Object;)Z
-	public final fun getInfo ()Lio/sentry/android/distribution/UpdateInfo;
-	public fun hashCode ()I
-	public fun toString ()Ljava/lang/String;
-}
-
-public final class io/sentry/android/distribution/UpdateStatus$UpToDate : io/sentry/android/distribution/UpdateStatus {
-	public static final field INSTANCE Lio/sentry/android/distribution/UpdateStatus$UpToDate;
-}
-
-public final class io/sentry/android/distribution/internal/DistributionIntegration : io/sentry/Integration {
-	public fun <init> ()V
+public final class io/sentry/android/distribution/DistributionIntegration : io/sentry/IDistributionApi, io/sentry/Integration {
+	public fun <init> (Landroid/content/Context;)V
+	public fun checkForUpdate (Lio/sentry/IDistributionApi$UpdateCallback;)V
+	public fun checkForUpdateBlocking ()Lio/sentry/UpdateStatus;
+	public fun downloadUpdate (Lio/sentry/UpdateInfo;)V
 	public fun register (Lio/sentry/IScopes;Lio/sentry/SentryOptions;)V
 }
 

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

@@ -0,0 +1,76 @@
+package io.sentry.android.distribution
+
+import android.content.Context
+import android.content.Intent
+import android.net.Uri
+import io.sentry.IDistributionApi
+import io.sentry.IScopes
+import io.sentry.Integration
+import io.sentry.SentryOptions
+import io.sentry.UpdateInfo
+import io.sentry.UpdateStatus
+
+/**
+ * The public Android SDK for Sentry Build Distribution.
+ *
+ * Provides functionality to check for app updates and download new versions from Sentry's preprod
+ * artifacts system.
+ */
+public class DistributionIntegration(context: Context) : Integration, IDistributionApi {
+
+  private lateinit var scopes: IScopes
+  private lateinit var sentryOptions: SentryOptions
+  private val context: Context = context.applicationContext
+
+  /**
+   * 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
+  }
+
+  /**
+   * 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
+   */
+  public override fun checkForUpdateBlocking(): UpdateStatus {
+    throw NotImplementedError()
+  }
+
+  /**
+   * Check for available updates asynchronously using a callback.
+   *
+   * @param onResult Callback that will be called with the UpdateStatus result
+   */
+  public override fun checkForUpdate(onResult: IDistributionApi.UpdateCallback) {
+    // TODO implement this in a async way
+    val result = checkForUpdateBlocking()
+    onResult.onResult(result)
+  }
+
+  /**
+   * 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
+   */
+  public override fun downloadUpdate(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
+    }
+  }
+}