BIT-7817: Skeleton crash reports for native crashes on API level 30

CHANGELOG.md

@@ -29,6 +29,8 @@
 
 **Fixed**
 
+- Fixed an issue where native crashes on API level 30 were silently discarded due to a missing tombstone trace. Skeleton crash reports with basic metadata but no stack traces are now generated in this case.
+
 - Fixed an issue where 3rd-party dependencies like `androidx` were included in the released Javadoc artifacts. Also updated the style of `capture-timber` and `capture-apollo` javadocs to match. 
 
 ### iOS

platform/jvm/capture/src/main/kotlin/io/bitdrift/capture/reports/IssueReporter.kt

@@ -165,13 +165,14 @@ internal class IssueReporter(
         val lastReasonResult = latestAppExitInfoProvider.get()
         if (lastReasonResult is LatestAppExitReasonResult.Valid) {
             val lastReason = lastReasonResult.applicationExitInfo
-            lastReason.traceInputStream?.use {
-                latestAppExitInfoProvider.convertExitReasonToFbsReportType(lastReason.reason)?.let { fatalIssueType ->
+            latestAppExitInfoProvider.convertExitReasonToFbsReportType(lastReason.reason)?.let { fatalIssueType ->
+                lastReason.traceInputStream.use { traceInputStream ->
                     issueReporterProcessor?.processAppExitReport(
                         fatalIssueType = fatalIssueType,
                         timestamp = lastReason.timestamp,
                         description = lastReason.description,
-                        traceInputStream = it,
+                        traceInputStream = traceInputStream,
+                        signalNumber = lastReason.status,
                     )
                 }
             }

platform/jvm/capture/src/main/kotlin/io/bitdrift/capture/reports/processor/IIssueReporterProcessor.kt

@@ -24,13 +24,18 @@ interface IIssueReporterProcessor {
      * (e.g. [ReportType.AppNotResponding] or [ReportType.NativeCrash]).
      * @param timestamp The timestamp when the issue occurred.
      * @param description Optional description of the issue.
-     * @param traceInputStream Input stream containing the fatal issue trace data.
+     * @param traceInputStream Input stream containing the fatal issue trace data. May be null for
+     * native crashes on API level 30 where tombstone data is unavailable; a skeleton report will
+     * be generated in that case.
+     * @param signalNumber The signal number from [android.app.ApplicationExitInfo.getStatus] for
+     * native crashes. Used to populate signal name and description in skeleton reports.
      */
     fun processAppExitReport(
         fatalIssueType: Byte,
         timestamp: Long,
         description: String? = null,
-        traceInputStream: InputStream,
+        traceInputStream: InputStream? = null,
+        signalNumber: Int = 0,
     )
 
     /**

platform/jvm/capture/src/main/kotlin/io/bitdrift/capture/reports/processor/IssueReporterProcessor.kt

@@ -100,18 +100,23 @@ internal class IssueReporterProcessor(
      * (e.g. [ReportType.AppNotResponding] or [ReportType.NativeCrash])
      * @param timestamp The timestamp when the issue occurred
      * @param description Optional description of the issue
-     * @param traceInputStream Input stream containing the fatal issue trace data
+     * @param traceInputStream Input stream containing the fatal issue trace data. May be null for
+     * native crashes on API level 30 where tombstone data is unavailable; a skeleton report will
+     * be generated in that case.
+     * @param signalNumber The signal number from [android.app.ApplicationExitInfo.getStatus] for
+     * native crashes. Used to populate signal name and description in skeleton reports.
      */
     override fun processAppExitReport(
         fatalIssueType: Byte,
         timestamp: Long,
         description: String?,
-        traceInputStream: InputStream,
+        traceInputStream: InputStream?,
+        signalNumber: Int,
     ) {
         runCatching {
             if (fatalIssueType == ReportType.AppNotResponding) {
                 streamingReportsProcessor.processAndPersistANR(
-                    traceInputStream,
+                    traceInputStream!!,
                     timestamp,
                     reporterIssueStore.generateFatalIssueFilePath(),
                     clientAttributes,
@@ -129,6 +134,7 @@ internal class IssueReporterProcessor(
                         deviceMetrics,
                         description,
                         traceInputStream,
+                        signalNumber,
                     )
                 builder.finish(report)
 

platform/jvm/capture/src/main/kotlin/io/bitdrift/capture/reports/processor/NativeCrashProcessor.kt

@@ -35,14 +35,29 @@ internal object NativeCrashProcessor {
             "SIGTRAP" to "Trace/breakpoint trap",
         )
 
+    private val signalNumberToName =
+        mapOf(
+            4 to "SIGILL",
+            5 to "SIGTRAP",
+            6 to "SIGABRT",
+            7 to "SIGBUS",
+            8 to "SIGFPE",
+            11 to "SIGSEGV",
+        )
+
     fun process(
         builder: FlatBufferBuilder,
         sdk: Int,
         appMetrics: Int,
         deviceMetrics: Int,
         description: String?,
-        traceInputStream: InputStream,
+        traceInputStream: InputStream?,
+        signalNumber: Int = 0,
     ): Int {
+        if (traceInputStream == null) {
+            return createSkeletonReport(builder, sdk, appMetrics, deviceMetrics, description, signalNumber)
+        }
+
         val tombstone = Tombstone.parseFrom(traceInputStream)
         val nativeErrors = mutableListOf<Int>()
         val threadOffsets = mutableListOf<Int>()
@@ -139,6 +154,55 @@ internal object NativeCrashProcessor {
         )
     }
 
+    /**
+     * Creates a minimal native crash report for cases where tombstone data is unavailable
+     * (e.g. native crashes on API level 30 where [android.app.ApplicationExitInfo.getTraceInputStream]
+     * returns null). The report includes the crash description and uses empty stack traces,
+     * empty thread details, and empty binary images. When a [signalNumber] is available
+     * (from [android.app.ApplicationExitInfo.getStatus]), the signal name and description
+     * are used for the error name and reason fields.
+     */
+    private fun createSkeletonReport(
+        builder: FlatBufferBuilder,
+        sdk: Int,
+        appMetrics: Int,
+        deviceMetrics: Int,
+        description: String?,
+        signalNumber: Int = 0,
+    ): Int {
+        val signalName = signalNumberToName[signalNumber]
+        val name = builder.createString(signalName ?: description ?: "Native crash")
+        val reason = builder.createString(
+            signalName?.let { signalDescriptions[it] } ?: "Native crash",
+        )
+        val errorOffset =
+            Error.createError(
+                builder,
+                name,
+                reason,
+                Error.createStackTraceVector(builder, intArrayOf()),
+                ErrorRelation.CausedBy,
+            )
+        val threadDetailsOffset =
+            ThreadDetails.createThreadDetails(
+                builder,
+                0.toUShort(),
+                ThreadDetails.createThreadsVector(builder, intArrayOf()),
+            )
+        return Report.createReport(
+            builder,
+            sdk,
+            ReportType.NativeCrash,
+            appMetrics,
+            deviceMetrics,
+            Report.createErrorsVector(builder, intArrayOf(errorOffset)),
+            threadDetailsOffset,
+            Report.createBinaryImagesVector(builder, intArrayOf()),
+            stateOffset = 0,
+            featureFlagsOffset = 0,
+        )
+    }
+
     private fun createErrorOffset(
         builder: FlatBufferBuilder,
         description: String?,

platform/jvm/capture/src/test/kotlin/io/bitdrift/capture/reports/processor/IssueReporterProcessorTest.kt

@@ -275,6 +275,60 @@ class IssueReporterProcessorTest {
         assertThat(deviceMetrics?.cpuAbis(0)).isEqualTo("armeabi-v7a")
     }
 
+    @Test
+    fun processAppExitReport_whenNativeCrashWithNullTrace_shouldCreateSkeletonNativeReport() {
+        val description = "Segmentation fault"
+
+        processor.processAppExitReport(
+            ReportType.NativeCrash,
+            FAKE_TIME_STAMP,
+            description,
+            traceInputStream = null,
+            signalNumber = 11, // SIGSEGV
+        )
+
+        verify(issueReporterStorage).persistFatalIssue(
+            eq(FAKE_TIME_STAMP),
+            issueReportCaptor.capture(),
+            reportTypeCaptor.capture(),
+        )
+        val buffer = ByteBuffer.wrap(issueReportCaptor.firstValue)
+        val report = Report.getRootAsReport(buffer)
+        assertThat(report.errorsLength).isEqualTo(1)
+        assertThat(reportTypeCaptor.firstValue).isEqualTo(ReportType.NativeCrash)
+
+        val capturedError = report.errors(0)!!
+        assertThat(capturedError.name).isEqualTo("SIGSEGV")
+        assertThat(capturedError.reason).isEqualTo("Segmentation violation (invalid memory reference)")
+        assertThat(capturedError.stackTraceLength).isEqualTo(0)
+        assertThat(report.threadDetails?.threadsLength).isEqualTo(0)
+        assertThat(report.binaryImagesLength).isEqualTo(0)
+    }
+
+    @Test
+    fun processAppExitReport_whenNativeCrashWithNullTraceAndUnknownSignal_shouldFallBackToDescription() {
+        val description = "Segmentation fault"
+
+        processor.processAppExitReport(
+            ReportType.NativeCrash,
+            FAKE_TIME_STAMP,
+            description,
+            traceInputStream = null,
+            signalNumber = 0,
+        )
+
+        verify(issueReporterStorage).persistFatalIssue(
+            eq(FAKE_TIME_STAMP),
+            issueReportCaptor.capture(),
+            reportTypeCaptor.capture(),
+        )
+        val buffer = ByteBuffer.wrap(issueReportCaptor.firstValue)
+        val report = Report.getRootAsReport(buffer)
+        val capturedError = report.errors(0)!!
+        assertThat(capturedError.name).isEqualTo(description)
+        assertThat(capturedError.reason).isEqualTo("Native crash")
+    }
+
     @Test
     fun processAppExitReport_withInvalidReason_shouldNotInteractWithStorage() {
         val description = null