fix(tracer): ensure callbacks are always invoked when processing ended spans

Author: yousif-bugsnag

CHANGELOG.md

@@ -5,6 +5,11 @@
 * System metrics are now reported for first-class `ViewLoad` spans
   [#475](https://github.com/bugsnag/bugsnag-android-performance/pull/475)
 
+### Bug fixes
+
+* Fixed an issue where `onSpanEnd` callbacks were not invoked for sampled spans
+  [#486](https://github.com/bugsnag/bugsnag-android-performance/pull/486)
+
 ## 2.1.1 (2025-11-20)
 
 ### Bug fixes

bugsnag-android-performance-impl/src/main/kotlin/com/bugsnag/android/performance/internal/processing/Tracer.kt

@@ -55,7 +55,7 @@ public class Tracer(
     override fun onEnd(span: Span) {
         if (span !is SpanImpl) return
 
-        if (sampler.shouldKeepSpan(span) && callbacksKeepSpan(span)) {
+        if (callbacksKeepSpan(span) && sampler.shouldKeepSpan(span)) {
             span.isSealed = true
             val batchSize = addToBatch(span)
             if (batchSize >= InternalDebug.spanBatchSizeSendTriggerPoint) {