react-native-onyx 2.0.66 → 2.0.68

package/dist/Onyx.js

@@ -416,8 +416,15 @@ 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 = OnyxUtils_1.default.getCollectionKey(key);
-                    if (OnyxUtils_1.default.isCollectionKey(collectionKey)) {
+                    let collectionKey;
+                    try {
+                        collectionKey = OnyxUtils_1.default.getCollectionKey(key);
+                    }
+                    catch (e) {
+                        // If getCollectionKey() throws an error it means the key is not a collection key.
+                        collectionKey = undefined;
+                    }
+                    if (collectionKey) {
                         if (!keyValuesToResetAsCollection[collectionKey]) {
                             keyValuesToResetAsCollection[collectionKey] = {};
                         }

package/dist/OnyxConnectionManager.js

@@ -162,8 +162,13 @@ class OnyxConnectionManager {
      * */
     addToEvictionBlockList(connection) {
         var _a;
-        const connectionMetadata = this.connectionsMap.get(connection === null || connection === void 0 ? void 0 : connection.id);
+        if (!connection) {
+            Logger.logInfo(`[ConnectionManager] Attempted to add connection to eviction block list passing an undefined connection object.`);
+            return;
+        }
+        const connectionMetadata = this.connectionsMap.get(connection.id);
         if (!connectionMetadata) {
+            Logger.logInfo(`[ConnectionManager] Attempted to add connection to eviction block list but no connection was found.`);
             return;
         }
         const evictionBlocklist = OnyxUtils_1.default.getEvictionBlocklist();
@@ -178,8 +183,13 @@ class OnyxConnectionManager {
      */
     removeFromEvictionBlockList(connection) {
         var _a, _b, _c;
-        const connectionMetadata = this.connectionsMap.get(connection === null || connection === void 0 ? void 0 : connection.id);
+        if (!connection) {
+            Logger.logInfo(`[ConnectionManager] Attempted to remove connection from eviction block list passing an undefined connection object.`);
+            return;
+        }
+        const connectionMetadata = this.connectionsMap.get(connection.id);
         if (!connectionMetadata) {
+            Logger.logInfo(`[ConnectionManager] Attempted to remove connection from eviction block list but no connection was found.`);
             return;
         }
         const evictionBlocklist = OnyxUtils_1.default.getEvictionBlocklist();

package/dist/OnyxUtils.d.ts

@@ -90,11 +90,12 @@ declare function isSafeEvictionKey(testKey: OnyxKey): boolean;
  *
  * For example:
  * - `getCollectionKey("report_123")` would return "report_"
- * - `getCollectionKey("report")` would return "report"
  * - `getCollectionKey("report_")` would return "report_"
+ * - `getCollectionKey("report_-1_something")` would return "report_"
+ * - `getCollectionKey("sharedNVP_user_-1_something")` would return "sharedNVP_user_"
  *
  * @param {OnyxKey} key - The key to process.
- * @return {string} The pure key without any numeric
+ * @return {string} The plain collection key.
  */
 declare function getCollectionKey(key: OnyxKey): string;
 /**

package/dist/OnyxUtils.js

@@ -335,13 +335,8 @@ function isCollectionMemberKey(collectionKey, key, collectionKeyLength) {
  * @returns A tuple where the first element is the collection part and the second element is the ID part.
  */
 function splitCollectionMemberKey(key) {
-    const underscoreIndex = key.lastIndexOf('_');
-    if (underscoreIndex === -1) {
-        throw new Error(`Invalid ${key} key provided, only collection keys are allowed.`);
-    }
-    const collectionKey = key.substring(0, underscoreIndex + 1);
-    const memberKey = key.substring(underscoreIndex + 1);
-    return [collectionKey, memberKey];
+    const collectionKey = getCollectionKey(key);
+    return [collectionKey, key.slice(collectionKey.length)];
 }
 /**
  * Checks to see if a provided key is the exact configured key of our connected subscriber
@@ -359,18 +354,28 @@ function isSafeEvictionKey(testKey) {
  *
  * For example:
  * - `getCollectionKey("report_123")` would return "report_"
- * - `getCollectionKey("report")` would return "report"
  * - `getCollectionKey("report_")` would return "report_"
+ * - `getCollectionKey("report_-1_something")` would return "report_"
+ * - `getCollectionKey("sharedNVP_user_-1_something")` would return "sharedNVP_user_"
  *
  * @param {OnyxKey} key - The key to process.
- * @return {string} The pure key without any numeric
+ * @return {string} The plain collection key.
  */
 function getCollectionKey(key) {
-    const underscoreIndex = key.lastIndexOf('_');
-    if (underscoreIndex === -1) {
-        return key;
+    // Start by finding the position of the last underscore in the string
+    let lastUnderscoreIndex = key.lastIndexOf('_');
+    // Iterate backwards to find the longest key that ends with '_'
+    while (lastUnderscoreIndex > 0) {
+        const possibleKey = key.slice(0, lastUnderscoreIndex + 1);
+        // Check if the substring is a key in the Set
+        if (isCollectionKey(possibleKey)) {
+            // Return the matching key and the rest of the string
+            return possibleKey;
+        }
+        // Move to the next underscore to check smaller possible keys
+        lastUnderscoreIndex = key.lastIndexOf('_', lastUnderscoreIndex - 1);
     }
-    return key.substring(0, underscoreIndex + 1);
+    throw new Error(`Invalid '${key}' key provided, only collection keys are allowed.`);
 }
 /**
  * Tries to get a value from the cache. If the value is not present in cache it will return the default value or undefined.
@@ -639,11 +644,17 @@ function keyChanged(key, value, previousValue, canUpdateSubscriber = () => true,
     // do the same in keysChanged, because we only call that function when a collection key changes, and it doesn't happen that often.
     // For performance reason, we look for the given key and later if don't find it we look for the collection key, instead of checking if it is a collection key first.
     let stateMappingKeys = (_a = onyxKeyToSubscriptionIDs.get(key)) !== null && _a !== void 0 ? _a : [];
-    const collectionKey = getCollectionKey(key);
-    const plainCollectionKey = collectionKey.lastIndexOf('_') !== -1 ? collectionKey : undefined;
-    if (plainCollectionKey) {
+    let collectionKey;
+    try {
+        collectionKey = getCollectionKey(key);
+    }
+    catch (e) {
+        // If getCollectionKey() throws an error it means the key is not a collection key.
+        collectionKey = undefined;
+    }
+    if (collectionKey) {
         // Getting the collection key from the specific key because only collection keys were stored in the mapping.
-        stateMappingKeys = [...stateMappingKeys, ...((_b = onyxKeyToSubscriptionIDs.get(plainCollectionKey)) !== null && _b !== void 0 ? _b : [])];
+        stateMappingKeys = [...stateMappingKeys, ...((_b = onyxKeyToSubscriptionIDs.get(collectionKey)) !== null && _b !== void 0 ? _b : [])];
         if (stateMappingKeys.length === 0) {
             return;
         }

package/package.json

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