Fix transaction request serialization

Restore transaction request services after deserialization and preserve method options needed by transaction invokers after copied or serialized delivery.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: ReubenBond

src/Orleans.Transactions/TransactionAttribute.cs

@@ -76,6 +76,7 @@ public enum TransactionOptionAlias
     }
 
     [GenerateSerializer]
+    [SerializationCallbacks(typeof(OnDeserializedCallbacks))]
     public abstract class TransactionRequestBase : RequestBase, IOutgoingGrainCallFilter, IOnDeserialized
     {
         [NonSerialized]
@@ -95,6 +96,9 @@ public abstract class TransactionRequestBase : RequestBase, IOutgoingGrainCallFi
         [Id(2)]
         public bool UseExclusiveLock { get; set; }
 
+        [Id(3)]
+        private InvokeMethodOptions _methodOptions;
+
         [GeneratedActivatorConstructor]
         protected TransactionRequestBase(Serializer<OrleansTransactionAbortedException> exceptionSerializer, IServiceProvider serviceProvider)
         {
@@ -126,6 +130,12 @@ protected void SetTransactionOptions(TransactionOption txOption)
             this.TransactionOption = txOption;
         }
 
+        protected new void AddInvokeMethodOptions(InvokeMethodOptions options)
+        {
+            base.AddInvokeMethodOptions(options);
+            _methodOptions |= options;
+        }
+
         protected void SetExclusiveLock(bool value)
         {
             this.UseExclusiveLock = value;
@@ -206,7 +216,8 @@ public override async ValueTask<Response> Invoke()
                     var transactionTimeout = Debugger.IsAttached ? TimeSpan.FromMinutes(30) : TimeSpan.FromSeconds(10);
 
                     // Start a new transaction
-                    var isReadOnly = this.Options.HasFlag(InvokeMethodOptions.ReadOnly);
+                    var isReadOnly = this.Options.HasFlag(InvokeMethodOptions.ReadOnly)
+                        || _methodOptions.HasFlag(InvokeMethodOptions.ReadOnly);
                     transactionInfo = await TransactionAgent.StartTransaction(isReadOnly, transactionTimeout);
                     startedNewTransaction = true;
                 }

src/api/Orleans.Transactions/Orleans.Transactions.cs

@@ -62,6 +62,7 @@ protected TransactionRequest(Serialization.Serializer<Transactions.OrleansTransa
     }
 
     [GenerateSerializer]
+    [SerializationCallbacks(typeof(Runtime.OnDeserializedCallbacks))]
     public abstract partial class TransactionRequestBase : Runtime.RequestBase, IOutgoingGrainCallFilter, Serialization.IOnDeserialized
     {
         [GeneratedActivatorConstructor]
@@ -2106,4 +2107,4 @@ public Copier_OperationState(global::Orleans.Serialization.Serializers.ICodecPro
 
         public global::Orleans.Transactions.TransactionCommitter<TService>.OperationState DeepCopy(global::Orleans.Transactions.TransactionCommitter<TService>.OperationState original, global::Orleans.Serialization.Cloning.CopyContext context) { throw null; }
     }
-}
+}