ref(sentry): Extract JsonObjectReader recovery helper

Move the repeated recovery logging flow behind a dedicated helper
so the collection readers stay focused on parsing and keep the guarded
recovery behavior aligned across all three paths.

This does not change behavior. It only removes duplicated error
handling around failed value recovery.

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

Author: adinauer

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

@@ -116,14 +116,13 @@ public void nextUnknown(ILogger logger, Map<String, Object> unknown, String name
         try {
           list.add(deserializer.deserialize(this, logger));
         } catch (Exception e) {
-          logger.log(SentryLevel.WARNING, "Failed to deserialize object in list.", e);
-          try {
-            recoverValue(startDepth, startToken);
-          } catch (Exception recoveryException) {
-            logger.log(
-                SentryLevel.ERROR,
-                "Stream unrecoverable, aborting list deserialization.",
-                recoveryException);
+          if (!recoverAfterValueFailure(
+              logger,
+              e,
+              "Failed to deserialize object in list.",
+              "Stream unrecoverable, aborting list deserialization.",
+              startDepth,
+              startToken)) {
             break;
           }
         }
@@ -150,14 +149,13 @@ public void nextUnknown(ILogger logger, Map<String, Object> unknown, String name
         try {
           map.put(key, deserializer.deserialize(this, logger));
         } catch (Exception e) {
-          logger.log(SentryLevel.WARNING, "Failed to deserialize object in map.", e);
-          try {
-            recoverValue(startDepth, startToken);
-          } catch (Exception recoveryException) {
-            logger.log(
-                SentryLevel.ERROR,
-                "Stream unrecoverable, aborting map deserialization.",
-                recoveryException);
+          if (!recoverAfterValueFailure(
+              logger,
+              e,
+              "Failed to deserialize object in map.",
+              "Stream unrecoverable, aborting map deserialization.",
+              startDepth,
+              startToken)) {
             break;
           }
         }
@@ -190,14 +188,13 @@ public void nextUnknown(ILogger logger, Map<String, Object> unknown, String name
             result.put(key, list);
           }
         } catch (Exception e) {
-          logger.log(SentryLevel.WARNING, "Failed to deserialize list in map.", e);
-          try {
-            recoverValue(startDepth, startToken);
-          } catch (Exception recoveryException) {
-            logger.log(
-                SentryLevel.ERROR,
-                "Stream unrecoverable, aborting map-of-lists deserialization.",
-                recoveryException);
+          if (!recoverAfterValueFailure(
+              logger,
+              e,
+              "Failed to deserialize list in map.",
+              "Stream unrecoverable, aborting map-of-lists deserialization.",
+              startDepth,
+              startToken)) {
             break;
           }
         }
@@ -331,6 +328,23 @@ public void skipValue() throws IOException {
     jsonReader.skipValue();
   }
 
+  private boolean recoverAfterValueFailure(
+      final @NotNull ILogger logger,
+      final @NotNull Exception error,
+      final @NotNull String warningMessage,
+      final @NotNull String unrecoverableMessage,
+      final int startDepth,
+      final @NotNull JsonToken startToken) {
+    logger.log(SentryLevel.WARNING, warningMessage, error);
+    try {
+      recoverValue(startDepth, startToken);
+      return true;
+    } catch (Exception recoveryException) {
+      logger.log(SentryLevel.ERROR, unrecoverableMessage, recoveryException);
+      return false;
+    }
+  }
+
   private void recoverValue(final int startDepth, final @NotNull JsonToken startToken)
       throws IOException {
     final boolean enteredNestedValue = depth > startDepth;