react-native-onyx 2.0.137 → 2.0.138

package/dist/OnyxCache.d.ts

@@ -19,8 +19,8 @@ declare class OnyxCache {
     private recentKeys;
     /** A map of cached values */
     private storageMap;
-    /** Index mapping collection keys to their member keys for O(1) lookup */
-    private collectionIndex;
+    /** Cache of complete collection data objects for O(1) retrieval */
+    private collectionData;
     /**
      * Captured pending tasks for already running storage methods
      * Using a map yields better performance on operations such a delete
@@ -165,10 +165,6 @@ declare class OnyxCache {
      * Get all data for a collection key
      */
     getCollectionData(collectionKey: OnyxKey): Record<OnyxKey, OnyxValue<OnyxKey>> | undefined;
-    /**
-     * Get all member keys for a collection key
-     */
-    getCollectionMemberKeys(collectionKey: OnyxKey): Set<OnyxKey> | undefined;
 }
 declare const instance: OnyxCache;
 export default instance;

package/dist/OnyxCache.js

@@ -68,10 +68,10 @@ class OnyxCache {
         this.nullishStorageKeys = new Set();
         this.recentKeys = new Set();
         this.storageMap = {};
-        this.collectionIndex = {};
+        this.collectionData = {};
         this.pendingPromises = new Map();
         // bind all public methods to prevent problems with `this`
-        (0, bindAll_1.default)(this, 'getAllKeys', 'get', 'hasCacheForKey', 'addKey', 'addNullishStorageKey', 'hasNullishStorageKey', 'clearNullishStorageKeys', 'set', 'drop', 'merge', 'hasPendingTask', 'getTaskPromise', 'captureTask', 'addToAccessedKeys', 'removeLeastRecentlyUsedKeys', 'setRecentKeysLimit', 'setAllKeys', 'setEvictionAllowList', 'getEvictionBlocklist', 'isEvictableKey', 'removeLastAccessedKey', 'addLastAccessedKey', 'addEvictableKeysToRecentlyAccessedList', 'getKeyForEviction', 'setCollectionKeys', 'isCollectionKey', 'getCollectionKey', 'getCollectionData', 'getCollectionMemberKeys');
+        (0, bindAll_1.default)(this, 'getAllKeys', 'get', 'hasCacheForKey', 'addKey', 'addNullishStorageKey', 'hasNullishStorageKey', 'clearNullishStorageKeys', 'set', 'drop', 'merge', 'hasPendingTask', 'getTaskPromise', 'captureTask', 'addToAccessedKeys', 'removeLeastRecentlyUsedKeys', 'setRecentKeysLimit', 'setAllKeys', 'setEvictionAllowList', 'getEvictionBlocklist', 'isEvictableKey', 'removeLastAccessedKey', 'addLastAccessedKey', 'addEvictableKeysToRecentlyAccessedList', 'getKeyForEviction', 'setCollectionKeys', 'isCollectionKey', 'getCollectionKey', 'getCollectionData');
     }
     /** Get all the storage keys */
     getAllKeys() {
@@ -136,33 +136,33 @@ class OnyxCache {
         const collectionKey = this.getCollectionKey(key);
         if (value === null || value === undefined) {
             delete this.storageMap[key];
-            // Remove from collection index if it's a collection member
-            if (collectionKey && this.getCollectionMemberKeys(collectionKey)) {
-                this.collectionIndex[collectionKey].delete(key);
+            // Remove from collection data cache if it's a collection member
+            if (collectionKey && this.collectionData[collectionKey]) {
+                delete this.collectionData[collectionKey][key];
             }
             return undefined;
         }
         this.storageMap[key] = value;
-        // Update collection index if this is a collection member
+        // Update collection data cache if this is a collection member
         if (collectionKey) {
-            if (!this.getCollectionMemberKeys(collectionKey)) {
-                this.collectionIndex[collectionKey] = new Set();
+            if (!this.collectionData[collectionKey]) {
+                this.collectionData[collectionKey] = {};
             }
-            this.collectionIndex[collectionKey].add(key);
+            this.collectionData[collectionKey][key] = value;
         }
         return value;
     }
     /** Forget the cached value for the given key */
     drop(key) {
         delete this.storageMap[key];
-        // Update collection index if this is a collection member
+        // Remove from collection data cache if this is a collection member
         const collectionKey = this.getCollectionKey(key);
-        if (collectionKey && this.getCollectionMemberKeys(collectionKey)) {
-            this.collectionIndex[collectionKey].delete(key);
+        if (collectionKey && this.collectionData[collectionKey]) {
+            delete this.collectionData[collectionKey][key];
         }
-        // If this is a collection key, clear its index
+        // If this is a collection key, clear its data
         if (this.isCollectionKey(key)) {
-            delete this.collectionIndex[key];
+            delete this.collectionData[key];
         }
         this.storageKeys.delete(key);
         this.recentKeys.delete(key);
@@ -185,19 +185,19 @@ class OnyxCache {
             const collectionKey = this.getCollectionKey(key);
             if (value === null || value === undefined) {
                 this.addNullishStorageKey(key);
-                // Remove from collection index if it's a collection member
-                if (collectionKey && this.getCollectionMemberKeys(collectionKey)) {
-                    this.collectionIndex[collectionKey].delete(key);
+                // Remove from collection data cache if it's a collection member
+                if (collectionKey && this.collectionData[collectionKey]) {
+                    delete this.collectionData[collectionKey][key];
                 }
             }
             else {
                 this.nullishStorageKeys.delete(key);
-                // Update collection index if this is a collection member
+                // Update collection data cache if this is a collection member
                 if (collectionKey) {
-                    if (!this.getCollectionMemberKeys(collectionKey)) {
-                        this.collectionIndex[collectionKey] = new Set();
+                    if (!this.collectionData[collectionKey]) {
+                        this.collectionData[collectionKey] = {};
                     }
-                    this.collectionIndex[collectionKey].add(key);
+                    this.collectionData[collectionKey][key] = this.storageMap[key];
                 }
             }
         });
@@ -257,10 +257,10 @@ class OnyxCache {
         }
         for (const key of keysToRemove) {
             delete this.storageMap[key];
-            // Update collection index if this is a collection member
+            // Remove from collection data cache if this is a collection member
             const collectionKey = this.getCollectionKey(key);
-            if (collectionKey && this.getCollectionMemberKeys(collectionKey)) {
-                this.collectionIndex[collectionKey].delete(key);
+            if (collectionKey && this.collectionData[collectionKey]) {
+                delete this.collectionData[collectionKey][key];
             }
             this.recentKeys.delete(key);
         }
@@ -358,12 +358,12 @@ class OnyxCache {
      */
     setCollectionKeys(collectionKeys) {
         this.collectionKeys = collectionKeys;
-        // Initialize collection indexes for existing collection keys
+        // Initialize collection data for existing collection keys
         collectionKeys.forEach((collectionKey) => {
-            if (this.getCollectionMemberKeys(collectionKey)) {
+            if (this.collectionData[collectionKey]) {
                 return;
             }
-            this.collectionIndex[collectionKey] = new Set();
+            this.collectionData[collectionKey] = {};
         });
     }
     /**
@@ -387,24 +387,12 @@ class OnyxCache {
      * Get all data for a collection key
      */
     getCollectionData(collectionKey) {
-        const memberKeys = this.getCollectionMemberKeys(collectionKey);
-        if (!memberKeys || memberKeys.size === 0) {
+        const cachedCollection = this.collectionData[collectionKey];
+        if (!cachedCollection || Object.keys(cachedCollection).length === 0) {
             return undefined;
         }
-        const collectionData = {};
-        memberKeys.forEach((memberKey) => {
-            const value = this.storageMap[memberKey];
-            if (value !== undefined) {
-                collectionData[memberKey] = value;
-            }
-        });
-        return collectionData;
-    }
-    /**
-     * Get all member keys for a collection key
-     */
-    getCollectionMemberKeys(collectionKey) {
-        return this.collectionIndex[collectionKey];
+        // Return a shallow copy to ensure React detects changes when items are added/removed
+        return Object.assign({}, cachedCollection);
     }
 }
 const instance = new OnyxCache();

package/dist/OnyxUtils.js

@@ -440,25 +440,16 @@ function tryGetCachedValue(key, mapping) {
     let val = OnyxCache_1.default.get(key);
     if (isCollectionKey(key)) {
         const collectionData = OnyxCache_1.default.getCollectionData(key);
-        const allCacheKeys = OnyxCache_1.default.getAllKeys();
-        if (collectionData !== undefined && allCacheKeys.size > 0) {
+        if (collectionData !== undefined) {
             val = collectionData;
         }
         else {
-            // Fallback to original logic
-            // It is possible we haven't loaded all keys yet so we do not know if the
-            // collection actually exists.
-            if (allCacheKeys.size === 0) {
+            // If we haven't loaded all keys yet, we can't determine if the collection exists
+            if (OnyxCache_1.default.getAllKeys().size === 0) {
                 return;
             }
-            const values = {};
-            allCacheKeys.forEach((cacheKey) => {
-                if (!cacheKey.startsWith(key)) {
-                    return;
-                }
-                values[cacheKey] = OnyxCache_1.default.get(cacheKey);
-            });
-            val = values;
+            // Set an empty collection object for collections that exist but have no data
+            val = {};
         }
     }
     if (mapping === null || mapping === void 0 ? void 0 : mapping.selector) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-native-onyx",
-  "version": "2.0.137",
+  "version": "2.0.138",
   "author": "Expensify, Inc.",
   "homepage": "https://expensify.com",
   "description": "State management for React Native",