Transaction commit now correctly clones the transaction when scheduling processing on the Unity main thread

IPA.Loader/Loader/PluginManager.cs

@@ -85,7 +85,11 @@ namespace IPA.Loader
             if (!transaction.HasStateChanged) return TaskEx.WhenAll();
 
             if (!UnityGame.OnMainThread)
-                return UnityMainThreadTaskScheduler.Factory.StartNew(() => CommitTransaction(transaction)).Unwrap();
+            {
+                var transactionCopy = transaction.Clone();
+                transaction.Dispose();
+                return UnityMainThreadTaskScheduler.Factory.StartNew(() => CommitTransaction(transactionCopy)).Unwrap();
+            }
 
             lock (commitTransactionLockObject)
             {
@@ -94,6 +98,7 @@ namespace IPA.Loader
                  || transaction.CurrentlyDisabled.Except(DisabledPlugins)
                                .Concat(DisabledPlugins.Except(transaction.CurrentlyDisabled)).Any())
                 { // ensure that the transaction's base state reflects the current state, otherwise throw
+                    transaction.Dispose();
                     throw new InvalidOperationException("Transaction no longer resembles the current state of plugins");
                 }
 

IPA.Loader/Loader/StateTransitionTransaction.cs

@@ -237,6 +237,23 @@ namespace IPA.Loader
         /// <exception cref="InvalidOperationException">if the plugins' state no longer matches this transaction's original state</exception>
         public Task Commit() => ThrowIfDisposed<Task>() ?? PluginManager.CommitTransaction(this);
 
+        /// <summary>
+        /// Clones this transaction to be identical, but with unrelated underlying sets.
+        /// </summary>
+        /// <returns>the new <see cref="StateTransitionTransaction"/></returns>
+        /// <exception cref="ObjectDisposedException">if this object has been disposed</exception>
+        public StateTransitionTransaction Clone()
+        {
+            ThrowIfDisposed();
+            var copy = new StateTransitionTransaction(CurrentlyEnabled, CurrentlyDisabled);
+            foreach (var toEnable in ToEnable)
+                copy.toEnable.Add(toEnable);
+            foreach (var toDisable in ToDisable)
+                copy.toDisable.Add(toDisable);
+            copy.stateChanged = stateChanged;
+            return copy;
+        }
+
         private void ThrowIfDisposed() => ThrowIfDisposed<byte>();
         private T ThrowIfDisposed<T>()
         {