Workaround stale Scope.Transaction issue

Author: jamescrosswell

src/Sentry.Maui.CommunityToolkit.Mvvm/MauiCommunityToolkitMvvmEventsBinder.cs

@@ -147,6 +147,11 @@ private void RelayCommandOnPropertyChanged(object? sender, PropertyChangedEventA
             // results would be that we would store the transaction on the scope, but it would never be cleared again,
             // since the next call to OnPropertyChanged for this RelayCommand will (likely) be from a different thread.
             var span = hub.StartSpan(SpanName, SpanOp);
+            if (span is ITransactionTracer transaction)
+            {
+                hub.ConfigureScope(scope => scope.Transaction = transaction);
+            }
+
             relay.SetFused(span);
         }
         else if (relay.GetFused<ISpan>() is { } span)

src/Sentry/Scope.cs

@@ -212,7 +212,9 @@ public ITransactionTracer? Transaction
             _transactionLock.EnterReadLock();
             try
             {
-                return _transaction.Value;
+                // Workaround for https://github.com/getsentry/sentry-dotnet/pull/4125#discussion_r2087994417
+                // TODO: We should really find the root cause of this issue and address in a seprate PR
+                return _transaction.Value is { IsFinished: false } transaction ? transaction : null;
             }
             finally
             {