fix(ios): clear purchasePayloadById on MainActor to match access pattern

hyochan · claude · hyochan · commit 9b4f52e3433d · 2026-03-04T20:22:42.000+09:00
Move purchasePayloadById cleanup from listenerLock to MainActor
dispatch, since all other accesses to this collection run on
MainActor. This eliminates a potential data race between cleanup
(background thread) and MainActor writes.

Co-Authored-By: Claude Opus 4.6 &lt;noreply@anthropic.com&gt;
diff --git a/ios/HybridRnIap.swift b/ios/HybridRnIap.swift
@@ -1089,7 +1089,10 @@ class HybridRnIap: HybridRnIapSpec {
             lastPurchaseErrorTimestamp = 0
             deliveredPurchaseEventKeys.removeAll()
             deliveredPurchaseEventOrder.removeAll()
-            purchasePayloadById.removeAll()
+        }
+        // Clear purchasePayloadById on MainActor to match its access pattern
+        Task { @MainActor in
+            self.purchasePayloadById.removeAll()
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -1089,7 +1089,10 @@ class HybridRnIap: HybridRnIapSpec {`
`1089`	`1089`	`lastPurchaseErrorTimestamp = 0`
`1090`	`1090`	`deliveredPurchaseEventKeys.removeAll()`
`1091`	`1091`	`deliveredPurchaseEventOrder.removeAll()`
`1092`		`- purchasePayloadById.removeAll()`
	`1092`	`+ }`
	`1093`	`+ // Clear purchasePayloadById on MainActor to match its access pattern`
	`1094`	`+ Task { @MainActor in`
	`1095`	`+ self.purchasePayloadById.removeAll()`
`1093`	`1096`	`}`
`1094`	`1097`	`}`
`1095`	`1098`