fix(compose): finish evicted root spans instead of leaving them open

isIdle spans only auto-finish when the whole transaction finishes, so
evicting a cached root span from RootSpans without finishing it left
it open on the transaction. If the same scopes was used again later, a
fresh root span was created alongside the still-open, now-untracked
original, showing up as a duplicate root span once the transaction
finished.

release now finishes the evicted composition/rendering spans before
dropping them from the cache.

Author: Tabishahmad

sentry-compose/src/androidMain/kotlin/io/sentry/compose/SentryComposeTracing.kt

@@ -91,8 +91,12 @@ private class RootSpans {
     val remaining = (refCounts[scopes] ?: 1) - 1
     if (remaining <= 0) {
       refCounts.remove(scopes)
-      compositionSpans.remove(scopes)
-      renderingSpans.remove(scopes)
+      // These are idle spans: they only auto-finish when the whole transaction finishes, so
+      // evicting them from the cache without finishing them here would leave them open on the
+      // transaction. If the same scopes is used again later, a fresh pair would be created
+      // alongside the still-open, now-untracked originals, showing up as duplicate root spans.
+      compositionSpans.remove(scopes)?.item?.finish()
+      renderingSpans.remove(scopes)?.item?.finish()
     } else {
       refCounts[scopes] = remaining
     }

sentry-compose/src/androidUnitTest/kotlin/io/sentry/compose/SentryTracedTest.kt

@@ -20,6 +20,7 @@ import io.sentry.SentryOptions
 import io.sentry.TransactionOptions
 import io.sentry.test.createTestScopes
 import kotlin.test.assertEquals
+import kotlin.test.assertTrue
 import org.junit.After
 import org.junit.Rule
 import org.junit.Test
@@ -147,4 +148,26 @@ class SentryTracedTest {
 
     assertEquals(1, tx.spans.count { it.operation == "ui.compose.composition" })
   }
+
+  @Test
+  fun `evicting a root span cache entry finishes the underlying span`() {
+    val scopes = newTracingScopes()
+    val tx = scopes.startBoundTransaction("custom-scopes-tx")
+    var mounted by mutableStateOf(true)
+
+    rule.setContent {
+      if (mounted) {
+        CompositionLocalProvider(LocalSentryScopes provides scopes) {
+          SentryTraced(tag = "first") { Box {} }
+        }
+      }
+    }
+    rule.waitForIdle()
+
+    mounted = false
+    rule.waitForIdle()
+
+    val compositionRootSpan = tx.spans.first { it.operation == "ui.compose.composition" }
+    assertTrue(compositionRootSpan.isFinished)
+  }
 }