react-native-onyx 2.0.62 → 2.0.64

package/dist/Onyx.js

@@ -510,8 +510,8 @@ function clear(keysToPreserve = []) {
                 const newValue = (_a = defaultKeyStates[key]) !== null && _a !== void 0 ? _a : null;
                 if (newValue !== oldValue) {
                     OnyxCache_1.default.set(key, newValue);
-                    const collectionKey = key.substring(0, key.indexOf('_') + 1);
-                    if (collectionKey) {
+                    const collectionKey = OnyxUtils_1.default.getCollectionKey(key);
+                    if (OnyxUtils_1.default.isCollectionKey(collectionKey)) {
                         if (!keyValuesToResetAsCollection[collectionKey]) {
                             keyValuesToResetAsCollection[collectionKey] = {};
                         }
@@ -560,6 +560,7 @@ function updateSnapshots(data) {
         return;
     const promises = [];
     const snapshotCollection = OnyxUtils_1.default.getCachedCollection(snapshotCollectionKey);
+    const snapshotCollectionKeyLength = snapshotCollectionKey.length;
     Object.entries(snapshotCollection).forEach(([snapshotKey, snapshotValue]) => {
         // Snapshots may not be present in cache. We don't know how to update them so we skip.
         if (!snapshotValue) {
@@ -568,7 +569,7 @@ function updateSnapshots(data) {
         let updatedData = {};
         data.forEach(({ key, value }) => {
             // snapshots are normal keys so we want to skip update if they are written to Onyx
-            if (OnyxUtils_1.default.isCollectionMemberKey(snapshotCollectionKey, key)) {
+            if (OnyxUtils_1.default.isCollectionMemberKey(snapshotCollectionKey, key, snapshotCollectionKeyLength)) {
                 return;
             }
             if (typeof snapshotValue !== 'object' || !('data' in snapshotValue)) {

package/dist/OnyxUtils.d.ts

@@ -79,13 +79,13 @@ declare function getCollectionKeys(): Set<OnyxKey>;
  * is associated with a collection of keys.
  */
 declare function isCollectionKey(key: OnyxKey): key is CollectionKeyBase;
-declare function isCollectionMemberKey<TCollectionKey extends CollectionKeyBase>(collectionKey: TCollectionKey, key: string): key is `${TCollectionKey}${string}`;
+declare function isCollectionMemberKey<TCollectionKey extends CollectionKeyBase>(collectionKey: TCollectionKey, key: string, collectionKeyLength: number): key is `${TCollectionKey}${string}`;
 /**
  * Splits a collection member key into the collection key part and the ID part.
  * @param key - The collection member key to split.
  * @returns A tuple where the first element is the collection part and the second element is the ID part.
  */
-declare function splitCollectionMemberKey<TKey extends CollectionKey>(key: TKey): [TKey extends `${infer Prefix}_${string}` ? `${Prefix}_` : never, string];
+declare function splitCollectionMemberKey<TKey extends CollectionKey, CollectionKeyType = TKey extends `${infer Prefix}_${string}` ? `${Prefix}_` : never>(key: TKey): [CollectionKeyType, string];
 /**
  * Checks to see if a provided key is the exact configured key of our connected subscriber
  * or if the provided key is a collection member key (in case our configured key is a "collection key")

package/dist/OnyxUtils.js

@@ -315,8 +315,8 @@ function getCollectionKeys() {
 function isCollectionKey(key) {
     return onyxCollectionKeySet.has(key);
 }
-function isCollectionMemberKey(collectionKey, key) {
-    return Str.startsWith(key, collectionKey) && key.length > collectionKey.length;
+function isCollectionMemberKey(collectionKey, key, collectionKeyLength) {
+    return key.startsWith(collectionKey) && key.length > collectionKeyLength;
 }
 /**
  * Splits a collection member key into the collection key part and the ID part.
@@ -328,7 +328,9 @@ function splitCollectionMemberKey(key) {
     if (underscoreIndex === -1) {
         throw new Error(`Invalid ${key} key provided, only collection keys are allowed.`);
     }
-    return [key.substring(0, underscoreIndex + 1), key.substring(underscoreIndex + 1)];
+    const collectionKey = key.substring(0, underscoreIndex + 1);
+    const memberKey = key.substring(underscoreIndex + 1);
+    return [collectionKey, memberKey];
 }
 /**
  * Checks to see if a provided key is the exact configured key of our connected subscriber
@@ -450,12 +452,13 @@ function addAllSafeEvictionKeysToRecentlyAccessedList() {
 function getCachedCollection(collectionKey, collectionMemberKeys) {
     const allKeys = collectionMemberKeys || OnyxCache_1.default.getAllKeys();
     const collection = {};
+    const collectionKeyLength = collectionKey.length;
     // forEach exists on both Set and Array
     allKeys.forEach((key) => {
         // If we don't have collectionMemberKeys array then we have to check whether a key is a collection member key.
         // Because in that case the keys will be coming from `cache.getAllKeys()` and we need to filter out the keys that
         // are not part of the collection.
-        if (!collectionMemberKeys && !isCollectionMemberKey(collectionKey, key)) {
+        if (!collectionMemberKeys && !isCollectionMemberKey(collectionKey, key, collectionKeyLength)) {
             return;
         }
         const cachedValue = OnyxCache_1.default.get(key);
@@ -478,6 +481,7 @@ function keysChanged(collectionKey, partialCollection, partialPreviousCollection
     // individual collection key member for the collection that is being updated. It is important to note that the collection parameter cane be a PARTIAL collection
     // and does not represent all of the combined keys and values for a collection key. It is just the "new" data that was merged in via mergeCollection().
     const stateMappingKeys = Object.keys(callbackToStateMapping);
+    const collectionKeyLength = collectionKey.length;
     for (let i = 0; i < stateMappingKeys.length; i++) {
         const subscriber = callbackToStateMapping[stateMappingKeys[i]];
         if (!subscriber) {
@@ -494,7 +498,7 @@ function keysChanged(collectionKey, partialCollection, partialPreviousCollection
         /**
          * e.g. Onyx.connect({key: `${ONYXKEYS.COLLECTION.REPORT}{reportID}`, callback: ...});
          */
-        const isSubscribedToCollectionMemberKey = isCollectionMemberKey(collectionKey, subscriber.key);
+        const isSubscribedToCollectionMemberKey = isCollectionMemberKey(collectionKey, subscriber.key, collectionKeyLength);
         // Regular Onyx.connect() subscriber found.
         if (typeof subscriber.callback === 'function') {
             if (!notifyRegularSubscibers) {

package/dist/useOnyx.js

@@ -46,7 +46,9 @@ function useOnyx(key, options) {
         try {
             const previousCollectionKey = OnyxUtils_1.default.splitCollectionMemberKey(previousKey)[0];
             const collectionKey = OnyxUtils_1.default.splitCollectionMemberKey(key)[0];
-            if (OnyxUtils_1.default.isCollectionMemberKey(previousCollectionKey, previousKey) && OnyxUtils_1.default.isCollectionMemberKey(collectionKey, key) && previousCollectionKey === collectionKey) {
+            if (OnyxUtils_1.default.isCollectionMemberKey(previousCollectionKey, previousKey, previousCollectionKey.length) &&
+                OnyxUtils_1.default.isCollectionMemberKey(collectionKey, key, collectionKey.length) &&
+                previousCollectionKey === collectionKey) {
                 return;
             }
         }

package/package.json

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