fix(replay): Do not capture replay for cached events

Author: romtsn

sentry/src/main/java/io/sentry/SentryClient.java

@@ -3,7 +3,9 @@
 import io.sentry.clientreport.DiscardReason;
 import io.sentry.exception.SentryEnvelopeException;
 import io.sentry.hints.AbnormalExit;
+import io.sentry.hints.ApplyScopeData;
 import io.sentry.hints.Backfillable;
+import io.sentry.hints.Cached;
 import io.sentry.hints.DiskFlushNotification;
 import io.sentry.hints.TransactionEnd;
 import io.sentry.logger.ILoggerBatchProcessor;
@@ -210,9 +212,10 @@ private boolean shouldApplyScopeData(final @NotNull CheckIn event, final @NotNul
     }
 
     final boolean isBackfillable = HintUtils.hasType(hint, Backfillable.class);
-    // if event is backfillable we don't wanna trigger capture replay, because it's an event from
-    // the past
-    if (event != null && !isBackfillable && (event.isErrored() || event.isCrashed())) {
+    final boolean isCached = HintUtils.hasType(hint, Cached.class);
+    // if event is backfillable or cached we don't wanna trigger capture replay, because it's
+    // an event from the past
+    if (event != null && !isBackfillable && !isCached && (event.isErrored() || event.isCrashed())) {
       options.getReplayController().captureReplay(event.isCrashed());
     }
 

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

@@ -2991,6 +2991,29 @@ class SentryClientTest {
         assertFalse(called)
     }
 
+    @Test
+    fun `does not captureReplay for cached events`() {
+        var called = false
+        fixture.sentryOptions.setReplayController(object : ReplayController by NoOpReplayController.getInstance() {
+            override fun captureReplay(isTerminating: Boolean?) {
+                called = true
+            }
+        })
+        val sut = fixture.getSut()
+
+        sut.captureEvent(
+            SentryEvent().apply {
+                exceptions = listOf(
+                    SentryException().apply {
+                        mechanism = Mechanism().apply { isHandled = false }
+                    }
+                )
+            },
+            HintUtils.createWithTypeCheckHint(CachedHint())
+        )
+        assertFalse(called)
+    }
+
     @Test
     fun `when beforeSendReplay is set, callback is invoked`() {
         var invoked = false
@@ -3502,6 +3525,8 @@ class SentryClientTest {
     private class BackfillableHint : Backfillable {
         override fun shouldEnrich(): Boolean = false
     }
+
+    private class CachedHint : Cached {}
 }
 
 class DropEverythingEventProcessor : EventProcessor {