fix(thread) trimming thread size

Author: lemnik

CHANGELOG.md

@@ -6,6 +6,8 @@
 
 * Native crashes will now include their signal code (where applicable) in the error message
   [#2135](https://github.com/bugsnag/bugsnag-android/pull/2135)
+* Bumped the version of `protobuf-javalite` used by [bugsnag-plugin-android-exitinfo](bugsnag-plugin-android-exitinfo) to 3.25.5
+  [#2144](https://github.com/bugsnag/bugsnag-android/pull/2144)
 
 ## 6.11.0 (2025-01-22)
 

bugsnag-android-core/src/main/java/com/bugsnag/android/EventPayload.kt

@@ -80,6 +80,10 @@ class EventPayload @JvmOverloads internal constructor(
             dataTrimmed
         )
 
+        if (event.threads.size > config.maxReportedThreads) {
+            event.threads.subList(config.maxReportedThreads, event.threads.size).clear()
+        }
+
         json = rebuildPayloadCache()
         if (json.size <= maxSizeBytes) {
             return this

bugsnag-plugin-android-exitinfo/build.gradle.kts

@@ -58,12 +58,12 @@ android {
 dependencies {
     addCommonModuleDependencies()
     api(project(":bugsnag-android-core"))
-    implementation("com.google.protobuf:protobuf-javalite:3.24.2")
+    implementation("com.google.protobuf:protobuf-javalite:3.25.5")
 }
 
 protobuf {
     protoc {
-        artifact = "com.google.protobuf:protoc:3.24.2"
+        artifact = "com.google.protobuf:protoc:3.25.5"
     }
     generateProtoTasks {
         all().configureEach {

bugsnag-plugin-android-exitinfo/detekt-baseline.xml

@@ -5,7 +5,6 @@
     <ID>CyclomaticComplexMethod:CodeStrings.kt$@RequiresApi(Build.VERSION_CODES.R) @SuppressLint("SwitchIntDef") @Suppress("DEPRECATION") internal fun importanceDescriptionOf(exitInfo: ApplicationExitInfo)</ID>
     <ID>CyclomaticComplexMethod:CodeStrings.kt$@RequiresApi(Build.VERSION_CODES.R) internal fun exitReasonOf(exitInfo: ApplicationExitInfo)</ID>
     <ID>LongParameterList:TombstoneParser.kt$TombstoneParser$( exitInfo: ApplicationExitInfo, listOpenFds: Boolean, includeLogcat: Boolean, threadConsumer: (BugsnagThread) -> Unit, fileDescriptorConsumer: (Int, String, String) -> Unit, logcatConsumer: (String) -> Unit )</ID>
-    <ID>MagicNumber:BugsnagExitInfoPlugin.kt$BugsnagExitInfoPlugin$100</ID>
     <ID>MagicNumber:TraceParser.kt$TraceParser$16</ID>
     <ID>MagicNumber:TraceParser.kt$TraceParser$3</ID>
     <ID>MaxLineLength:ExitInfoCallbackTest.kt$ExitInfoCallbackTest$exitInfoCallback = ExitInfoCallback(context, nativeEnhancer, anrEventEnhancer, null, ApplicationExitInfoMatcher(context, 100))</ID>

features/full_tests/startup_anr.feature

@@ -7,9 +7,10 @@ Feature: onCreate ANR
 # and then terminate the "misbehaving" app with a KILL (9) signal (almost like the ANR code doesn't
 # fire at all). Since we can't cover a KILL signal in a test, we skip Android 10.
   @skip_android_10
-# Android 13 Note: we no longer have permission to inject BACK button events, which are used to
+# Android 13+ Note: we no longer have permission to inject BACK button events, which are used to
 # trigger the ANR - so the test is not valid on Android 13 either
   @skip_android_13
+  @skip_android_14
   Scenario: onCreate ANR is reported
     When I clear any error dialogue
     And I run "ConfigureStartupAnrScenario"