Enhance transformation progress tracking logic

Author: emp0ry

CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 0.6.1 - 2026-08-21
+
+- Made transformation progress refresh immediately when one of multiple identical
+  transformation pedestals is collected in the same room. The displayed count now
+  combines the live inventory with Isaac's persisted PlayerForm state.
+
 ## 0.6.0 - 2026-08-20
 
 - Added a top-left pause-only inventory browser for collectibles, active items,

README.md

@@ -38,8 +38,8 @@ The overlay tracks the player's position and displays the nearest eligible objec
 - Untouched floor cards and runes remain hidden until the game marks them touched or a player holds them in a native pocket slot.
 - Pills remain hidden until Isaac's native ItemPool state identifies their effect.
 - Curse of the Blind suppresses every collectible description using the native level curse mask. The forced-blind field and question-mark sprite are retained as additional guards.
-- Transformation progress is read from the player's native persisted transformation state instead of nearby-item observations.
-- The 14 normal transformations use Isaac's persisted native PlayerForm counters, so replaced active items and saved/reloaded runs keep their real progress. Super Bum completion is read from its native merged familiar.
+- Transformation progress combines Isaac's persisted PlayerForm counters with the live owned-item table. It updates immediately after a pickup while still preserving progress across active-item replacement and save reloads.
+- The 14 normal transformations use the native counters and inventory state. Super Bum completion is read from its native merged familiar.
 - Native run identity and seed tracking reset learned cards and run-only knowledge when a new run begins.
 
 ### Artwork and presentation

include/EIDTransformationProgress.h

@@ -11,6 +11,7 @@ NS_ASSUME_NONNULL_BEGIN
 - (NSArray<NSNumber *> *)allTransformationIdentifiers;
 - (NSArray<NSNumber *> *)transformationsForCollectible:(NSInteger)collectible;
 - (NSInteger)progressForTransformation:(NSInteger)transformation;
+- (NSString *)progressSignatureForPickups:(NSArray *)pickups;
 - (NSString *)localizedNameForTransformation:(NSInteger)transformation;
 - (NSString *)englishNameForTransformation:(NSInteger)transformation;
 @end

src/EIDOverlayController.m

@@ -129,6 +129,7 @@ @interface EIDOverlayController ()
 @property(nonatomic) NSUInteger consecutiveMenuScans;
 @property(nonatomic, strong) EIDPickupIdentity *selectedInventoryItem;
 @property(nonatomic, copy) NSString *inventorySignature;
+@property(nonatomic, copy) NSString *transformationProgressSignature;
 @property(nonatomic) BOOL pauseUIActive;
 @end
 
@@ -454,7 +455,12 @@ - (void)tick:(NSTimer *)timer {
             BOOL paused = self.probe.gameplayActive && self.probe.pauseStateAvailable &&
                 self.probe.paused;
             [self updatePauseInventoryForPaused:paused];
-            if (![pickups isEqualToArray:self.lastPickups]) {
+            NSString *progressSignature = [[EIDTransformationProgress shared]
+                progressSignatureForPickups:pickups];
+            BOOL progressChanged = ![progressSignature
+                isEqualToString:self.transformationProgressSignature ?: @""];
+            self.transformationProgressSignature = progressSignature;
+            if (![pickups isEqualToArray:self.lastPickups] || progressChanged) {
                 self.lastPickups = pickups;
                 if (!self.menuMode && !paused && !self.selectedInventoryItem) {
                     [self renderPickups:pickups];

src/EIDTransformationProgress.m

@@ -162,9 +162,20 @@ - (NSInteger)progressForTransformation:(NSInteger)transformation {
     if (transformation <= 0) return 0;
     NSInteger nativeForm = EIDNativeFormIDForTransformation(transformation);
     if (nativeForm != NSNotFound) {
-        if (!self.probe.transformationStateAvailable) return 0;
-        NSInteger count = [self.probe nativeTransformationCounterForFormID:nativeForm];
-        return MIN(self.required, MAX(0, count));
+        NSInteger nativeCount = self.probe.transformationStateAvailable
+            ? [self.probe nativeTransformationCounterForFormID:nativeForm] : 0;
+        NSInteger liveCount = 0;
+        if (self.probe.ownedCollectibleStateAvailable) {
+            for (NSNumber *collectible in self.collectiblesByTransformation[@(transformation)]) {
+                liveCount += [self.probe
+                    transformationCollectibleCountForID:collectible.integerValue];
+            }
+        }
+        // PlayerForm is authoritative across reloads and after an active item is
+        // replaced, but Isaac can update it a few frames after the live inventory.
+        // Taking the maximum gives an immediate in-room update without losing the
+        // persisted history that the inventory alone cannot reconstruct.
+        return MIN(self.required, MAX(0, MAX(nativeCount, liveCount)));
     }
 
     // Super Bum (EID 11) is a familiar merge, not an entry in the native
@@ -179,6 +190,21 @@ - (NSInteger)progressForTransformation:(NSInteger)transformation {
     return MIN(self.required, MAX(0, count));
 }
 
+- (NSString *)progressSignatureForPickups:(NSArray<EIDPickupIdentity *> *)pickups {
+    EIDPickupIdentity *pickup = pickups.firstObject;
+    if (!pickup || pickup.variant != EIDPickupVariantCollectible) return @"";
+    NSArray<NSNumber *> *transformations = [self transformationsForCollectible:pickup.subtype];
+    if (!transformations.count) return @"";
+
+    NSMutableArray<NSString *> *parts = [NSMutableArray arrayWithCapacity:transformations.count];
+    for (NSNumber *transformation in transformations) {
+        [parts addObject:[NSString stringWithFormat:@"%@=%ld", transformation,
+                          (long)[self progressForTransformation:transformation.integerValue]]];
+    }
+    return [NSString stringWithFormat:@"%lu:%ld:%@", (unsigned long)self.probe.runCounter,
+            (long)pickup.subtype, [parts componentsJoinedByString:@","]];
+}
+
 - (void)logProgressForCollectible:(NSInteger)collectible
                    transformations:(NSArray<NSNumber *> *)transformations {
     if (!transformations.count) return;
@@ -194,7 +220,7 @@ - (void)logProgressForCollectible:(NSInteger)collectible
     static NSString *lastLoggedState;
     if ([lastLoggedState isEqualToString:state]) return;
     lastLoggedState = state;
-    EIDLog(@"transformation progress for collectible %ld: %@ (persisted native counters %@)",
+    EIDLog(@"transformation progress for collectible %ld: %@ (native/live counters %@)",
            (long)collectible, [parts componentsJoinedByString:@", "],
            self.probe.transformationStateAvailable ? @"ready" : @"unavailable");
 }