Add NoNetwork status and improve error messages

- Add UpdateStatus.NoNetwork subclass for network-specific errors
- Update DistributionIntegration to use NoNetwork for UnknownHostException and SocketTimeoutException
- Improve UpdateResponseParser error messages to specify which required fields are missing
- Add comprehensive tests for specific missing field error messages

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

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

Author: runningcode

sentry-android-distribution/src/main/java/io/sentry/android/distribution/DistributionIntegration.kt

@@ -72,11 +72,11 @@ public class DistributionIntegration(context: Context) : Integration, IDistribut
         e,
         "DNS lookup failed - check internet connection",
       )
-      UpdateStatus.UpdateError("No internet connection or invalid server URL")
+      UpdateStatus.NoNetwork("No internet connection or invalid server URL")
     } catch (e: SocketTimeoutException) {
       // SocketTimeoutException could indicate either slow network or server issues
       sentryOptions.logger.log(SentryLevel.ERROR, e, "Network request timed out")
-      UpdateStatus.UpdateError("Request timed out - check network connection")
+      UpdateStatus.NoNetwork("Request timed out - check network connection")
     } catch (e: Exception) {
       sentryOptions.logger.log(SentryLevel.ERROR, e, "Unexpected error checking for updates")
       UpdateStatus.UpdateError("Unexpected error: ${e.message}")
@@ -130,7 +130,8 @@ public class DistributionIntegration(context: Context) : Integration, IDistribut
         mainBinaryIdentifier = appId,
         appId = appId,
         platform = "android",
-        version = versionName,
+        versionName = versionName,
+        versionCode = 5,
       )
     } catch (e: PackageManager.NameNotFoundException) {
       sentryOptions.logger.log(SentryLevel.ERROR, e, "Failed to get package info")

sentry-android-distribution/src/main/java/io/sentry/android/distribution/UpdateResponseParser.kt

@@ -59,15 +59,22 @@ internal class UpdateResponseParser(private val options: SentryOptions) {
     val createdDate = json.optString("createdDate", "")
 
     // Validate required fields (optString returns "null" for null values)
-    if (
-      id.isEmpty() ||
-        id == "null" ||
-        buildVersion.isEmpty() ||
-        buildVersion == "null" ||
-        downloadUrl.isEmpty() ||
-        downloadUrl == "null"
-    ) {
-      throw IllegalArgumentException("Missing required update information in API response")
+    val missingFields = mutableListOf<String>()
+
+    if (id.isEmpty() || id == "null") {
+      missingFields.add("id")
+    }
+    if (buildVersion.isEmpty() || buildVersion == "null") {
+      missingFields.add("buildVersion")
+    }
+    if (downloadUrl.isEmpty() || downloadUrl == "null") {
+      missingFields.add("downloadUrl")
+    }
+
+    if (missingFields.isNotEmpty()) {
+      throw IllegalArgumentException(
+        "Missing required fields in API response: ${missingFields.joinToString(", ")}"
+      )
     }
 
     return UpdateInfo(id, buildVersion, buildNumber, downloadUrl, appName, createdDate)

sentry-android-distribution/src/test/java/io/sentry/android/distribution/UpdateResponseParserTest.kt

@@ -198,4 +198,116 @@ class UpdateResponseParserTest {
       error.message.startsWith("Failed to parse response:"),
     )
   }
+
+  @Test
+  fun `parseResponse returns specific error message when id is missing`() {
+    val responseBody =
+      """
+      {
+        "updateAvailable": true,
+        "buildVersion": "2.0.0",
+        "downloadUrl": "https://example.com/download"
+      }
+    """
+        .trimIndent()
+
+    val result = parser.parseResponse(200, responseBody)
+
+    assertTrue("Should return UpdateError", result is UpdateStatus.UpdateError)
+    val error = result as UpdateStatus.UpdateError
+    assertTrue(
+      "Error message should mention missing id field",
+      error.message.contains("Missing required fields in API response: id"),
+    )
+  }
+
+  @Test
+  fun `parseResponse returns specific error message when buildVersion is missing`() {
+    val responseBody =
+      """
+      {
+        "updateAvailable": true,
+        "id": "update-123",
+        "downloadUrl": "https://example.com/download"
+      }
+    """
+        .trimIndent()
+
+    val result = parser.parseResponse(200, responseBody)
+
+    assertTrue("Should return UpdateError", result is UpdateStatus.UpdateError)
+    val error = result as UpdateStatus.UpdateError
+    assertTrue(
+      "Error message should mention missing buildVersion field",
+      error.message.contains("Missing required fields in API response: buildVersion"),
+    )
+  }
+
+  @Test
+  fun `parseResponse returns specific error message when downloadUrl is missing`() {
+    val responseBody =
+      """
+      {
+        "updateAvailable": true,
+        "id": "update-123",
+        "buildVersion": "2.0.0"
+      }
+    """
+        .trimIndent()
+
+    val result = parser.parseResponse(200, responseBody)
+
+    assertTrue("Should return UpdateError", result is UpdateStatus.UpdateError)
+    val error = result as UpdateStatus.UpdateError
+    assertTrue(
+      "Error message should mention missing downloadUrl field",
+      error.message.contains("Missing required fields in API response: downloadUrl"),
+    )
+  }
+
+  @Test
+  fun `parseResponse returns specific error message when multiple fields are missing`() {
+    val responseBody =
+      """
+      {
+        "updateAvailable": true,
+        "buildNumber": 42
+      }
+    """
+        .trimIndent()
+
+    val result = parser.parseResponse(200, responseBody)
+
+    assertTrue("Should return UpdateError", result is UpdateStatus.UpdateError)
+    val error = result as UpdateStatus.UpdateError
+    assertTrue(
+      "Error message should mention all missing required fields",
+      error.message.contains(
+        "Missing required fields in API response: id, buildVersion, downloadUrl"
+      ),
+    )
+  }
+
+  @Test
+  fun `parseResponse returns specific error message when field is null string`() {
+    val responseBody =
+      """
+      {
+        "updateAvailable": true,
+        "id": "null",
+        "buildVersion": "2.0.0",
+        "downloadUrl": "https://example.com/download"
+      }
+    """
+        .trimIndent()
+
+    val result = parser.parseResponse(200, responseBody)
+
+    assertTrue("Should return UpdateError", result is UpdateStatus.UpdateError)
+    val error = result as UpdateStatus.UpdateError
+    assertTrue(
+      "Error message should mention missing id field when value is 'null' string",
+      error.message.contains("Missing required fields in API response: id"),
+    )
+  }
 }

sentry/api/sentry.api

@@ -4344,6 +4344,11 @@ public final class io/sentry/UpdateStatus$NewRelease : io/sentry/UpdateStatus {
 	public fun getInfo ()Lio/sentry/UpdateInfo;
 }
 
+public final class io/sentry/UpdateStatus$NoNetwork : io/sentry/UpdateStatus {
+	public fun <init> (Ljava/lang/String;)V
+	public fun getMessage ()Ljava/lang/String;
+}
+
 public final class io/sentry/UpdateStatus$UpToDate : io/sentry/UpdateStatus {
 	public static fun getInstance ()Lio/sentry/UpdateStatus$UpToDate;
 }

sentry/src/main/java/io/sentry/UpdateStatus.java

@@ -43,4 +43,17 @@ public UpdateError(final @NotNull String message) {
       return message;
     }
   }
+
+  /** No network connection is available to check for updates. */
+  public static final class NoNetwork extends UpdateStatus {
+    private final @NotNull String message;
+
+    public NoNetwork(final @NotNull String message) {
+      this.message = message;
+    }
+
+    public @NotNull String getMessage() {
+      return message;
+    }
+  }
 }