fix(sentry): Continue list recovery until array end

Continue JsonObjectReader list deserialization until the array boundary
instead of stopping when the next token is not BEGIN_OBJECT.

After recovering from a failed object element, the next unread token can be a
primitive. The previous loop exited early in that case and then aborted when
endArray() encountered unread content. Iterating with hasNext() keeps recovery
aligned with the actual array boundary.

Add a regression test covering a failing object followed by a primitive and a
later valid object.

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

Author: adinauer

sentry/src/main/java/io/sentry/JsonObjectReader.java

@@ -120,24 +120,22 @@ public void nextUnknown(ILogger logger, Map<String, Object> unknown, String name
     }
     beginArray();
     List<T> list = new ArrayList<>();
-    if (jsonReader.hasNext()) {
-      do {
-        final RecoveryState recoveryState = beginRecovery(peek());
-        try {
-          list.add(deserializer.deserialize(this, logger));
-        } catch (Exception e) {
-          if (!recoverAfterValueFailure(
-              logger,
-              e,
-              "Failed to deserialize object in list.",
-              "Stream unrecoverable, aborting list deserialization.",
-              recoveryState)) {
-            break;
-          }
-        } finally {
-          endRecovery(recoveryState);
+    while (jsonReader.hasNext()) {
+      final RecoveryState recoveryState = beginRecovery(peek());
+      try {
+        list.add(deserializer.deserialize(this, logger));
+      } catch (Exception e) {
+        if (!recoverAfterValueFailure(
+            logger,
+            e,
+            "Failed to deserialize object in list.",
+            "Stream unrecoverable, aborting list deserialization.",
+            recoveryState)) {
+          break;
         }
-      } while (jsonReader.peek() == JsonToken.BEGIN_OBJECT);
+      } finally {
+        endRecovery(recoveryState);
+      }
     }
     endArray();
     return list;

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

@@ -298,6 +298,31 @@ class JsonObjectReaderTest {
     assertEquals(listOf("two"), actual)
   }
 
+  @Test
+  fun `nextListOrNull keeps elements after a failing object followed by a primitive`() {
+    val reader =
+      getValuesReader(
+        "[{\"kind\": \"fail\", \"value\": \"bad\"}, \"oops\", {\"kind\": \"ok\", \"value\": \"two\"}]"
+      )
+    val deserializer =
+      JsonDeserializer<String> { objectReader, _ ->
+        objectReader.beginObject()
+        objectReader.nextName()
+        if (objectReader.nextString() == "fail") {
+          throw IllegalStateException("intentional")
+        }
+        objectReader.nextName()
+        val value = objectReader.nextString()
+        objectReader.endObject()
+        value
+      }
+
+    val actual = reader.nextListOrNull(fixture.logger, deserializer)
+
+    assertEquals(listOf("two"), actual)
+    assertEquals(io.sentry.vendor.gson.stream.JsonToken.END_OBJECT, reader.peek())
+  }
+
   @Test
   fun `nextListOrNull keeps elements after skipValue consumes a failing element`() {
     var callCount = 0