feat(android): Match MemoryLimiter exits on the description namespace prefix

Match the "MemoryLimiter:" prefix instead of the full "MemoryLimiter:AnonSwap"
string. Per AOSP, AnonSwap is the only MemoryLimiter kill sub-reason on
Android 17, but the memory and swap limits it also tracks may start killing in a
future release; matching the namespace prefix keeps capturing those without a
code change, while the colon still anchors matching to the MemoryLimiter
namespace. The raw description is retained on the event mechanism.

Author: 0xadam-brown

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

@@ -72,7 +72,10 @@
 @ApiStatus.Internal
 public final class MemoryLimiterIntegration implements Integration, Closeable {
 
-  static final @NotNull String MEMORY_LIMITER_DESCRIPTION = "MemoryLimiter:AnonSwap";
+  static final @NotNull String MEMORY_LIMITER_DESCRIPTION_PREFIX = "MemoryLimiter:";
+  static final @NotNull String MEMORY_LIMITER_DESCRIPTION =
+      MEMORY_LIMITER_DESCRIPTION_PREFIX + "AnonSwap";
+
   static final @NotNull String MEMORY_LIMITER_MESSAGE = "Android process killed by MemoryLimiter";
 
   static final @NotNull String IMPORTANCE_DATA_KEY = "importance";
@@ -186,20 +189,26 @@ private MemoryLimiterPolicy(final @NotNull SentryAndroidOptions options) {
     }
 
     /**
-     * Matches Android exits that the system records as generic "other" deaths and annotates with
-     * the MemoryLimiter description.
+     * Returns true if the provided {@code exitInfo} looks like it comes from a
+     * MemoryLimiter-induced process death.
      *
      * <p>Criteria taken from <a
-     * href="https://developer.android.com/about/versions/17/behavior-changes-all?utm_source=android-studio-app&utm_medium=app#app-memory-limits">here</a>.
+     * href="https://developer.android.com/about/versions/17/behavior-changes-all#app-memory-limits">here</a>.
      */
     @Override
     @RequiresApi(api = Build.VERSION_CODES.R)
     public boolean matches(final @NotNull ApplicationExitInfo exitInfo) {
       if (exitInfo.getReason() != ApplicationExitInfo.REASON_OTHER) {
         return false;
       }
+
       final String description = exitInfo.getDescription();
-      return description != null && description.contains(MEMORY_LIMITER_DESCRIPTION);
+      // We match on the "MemoryLimiter:" prefix rather than the full "MemoryLimiter:AnonSwap"
+      // string mentioned in the Android 17 release notes because we want to capture any future
+      // MemoryLimiter kill reason without a code change. (MemoryLimiter source already tracks
+      // MemoryLimiter:Memory and MemoryLimiter:Swap reasons, but for now doesn't kill the process
+      // because of them.)
+      return description != null && description.contains(MEMORY_LIMITER_DESCRIPTION_PREFIX);
     }
 
     @Override
@@ -276,7 +285,7 @@ public void markReported(final long timestamp) {
      * <p>The <a
      * href="https://source.android.com/docs/core/perf/memory-limiter#process-monitoring">MemoryLimiter
      * docs</a> classify exact {@code PROCESS_STATE_*} values, but {@link ApplicationExitInfo} only
-     * exposes the coarser {@link RunningAppProcessInfo#Importance} bucket.
+     * exposes the coarser {@link RunningAppProcessInfo} importance bucket.
      */
     private @NotNull String getProcessVisibility(final int importance) {
       switch (importance) {

sentry-android-core/src/test/java/io/sentry/android/core/MemoryLimiterIntegrationTest.kt

@@ -365,6 +365,44 @@ class MemoryLimiterIntegrationTest {
     verify(fixture.scopes, never()).captureEvent(any(), anyOrNull<Hint>())
   }
 
+  @Test
+  fun `captures exit for any MemoryLimiter sub-reason, not just AnonSwap`() {
+    val integration = fixture.getSut(tmpDir, lastReportedTimestamp = oldTimestamp)
+    // A future MemoryLimiter kill sub-reason (e.g. the memory or swap limits) still lives in the
+    // "MemoryLimiter:" namespace and must be captured, with its raw sub-reason preserved.
+    fixture.addAppExitInfo(
+      reason = ApplicationExitInfo.REASON_OTHER,
+      description = "MemoryLimiter:Memory",
+      timestamp = newTimestamp,
+    )
+
+    integration.register(fixture.scopes, fixture.options)
+
+    verify(fixture.scopes)
+      .captureEvent(
+        check<SentryEvent> { event ->
+          assertEquals("MemoryLimiter:Memory", event.exceptions!!.single().mechanism!!.description)
+        },
+        anyOrNull<Hint>(),
+      )
+  }
+
+  @Test
+  fun `ignores exit when description mentions MemoryLimiter without the namespace delimiter`() {
+    val integration = fixture.getSut(tmpDir, lastReportedTimestamp = oldTimestamp)
+    // "MemoryLimiter" without the ":" delimiter is not a MemoryLimiter kill; matching requires the
+    // namespace prefix so we don't over-capture unrelated REASON_OTHER exits.
+    fixture.addAppExitInfo(
+      reason = ApplicationExitInfo.REASON_OTHER,
+      description = "NotAMemoryLimiterKill",
+      timestamp = newTimestamp,
+    )
+
+    integration.register(fixture.scopes, fixture.options)
+
+    verify(fixture.scopes, never()).captureEvent(any(), anyOrNull<Hint>())
+  }
+
   @Test
   fun `ignores exit when description is null`() {
     val integration = fixture.getSut(tmpDir, lastReportedTimestamp = oldTimestamp)