react-native-onyx 3.0.74 → 3.0.75

package/dist/OnyxUtils.d.ts

@@ -254,10 +254,6 @@ declare function mergeCollectionWithPatches<TKey extends CollectionKeyBase>({ co
 declare function partialSetCollection<TKey extends CollectionKeyBase>({ collectionKey, collection }: SetCollectionParams<TKey>, retryAttempt?: number): Promise<void>;
 declare function logKeyChanged(onyxMethod: Extract<OnyxMethod, 'set' | 'merge'>, key: OnyxKey, value: unknown, hasChanged: boolean): void;
 declare function logKeyRemoved(onyxMethod: Extract<OnyxMethod, 'set' | 'merge'>, key: OnyxKey): void;
-/**
- * Getter - returns the callback to state mapping, useful in test environments.
- */
-declare function getCallbackToStateMapping(): Record<number, CallbackToStateMapping<OnyxKey>>;
 /**
  * Clear internal variables used in this file, useful in test environments.
  */
@@ -317,7 +313,6 @@ declare const OnyxUtils: {
     setWithRetry: typeof setWithRetry;
     multiSetWithRetry: typeof multiSetWithRetry;
     setCollectionWithRetry: typeof setCollectionWithRetry;
-    getCallbackToStateMapping: typeof getCallbackToStateMapping;
 };
 export type { OnyxMethod };
 export default OnyxUtils;

package/dist/OnyxUtils.js

@@ -882,24 +882,6 @@ function subscribeToKey(connectOptions) {
     const subscriptionID = lastSubscriptionID++;
     callbackToStateMapping[subscriptionID] = mapping;
     callbackToStateMapping[subscriptionID].subscriptionID = subscriptionID;
-    // If the subscriber is attempting to connect to a collection member whose ID is skippable (e.g. "undefined", "null", etc.)
-    // we suppress wiring the subscription fully to avoid unnecessary callback emissions such as for "report_undefined".
-    // We still return a valid subscriptionID so callers can disconnect safely.
-    try {
-        const skippableIDs = getSkippableCollectionMemberIDs();
-        if (skippableIDs.size) {
-            const [, collectionMemberID] = OnyxKeys_1.default.splitCollectionMemberKey(mapping.key);
-            if (skippableIDs.has(collectionMemberID)) {
-                // Clean up the provisional mapping to avoid retaining unused subscribers.
-                OnyxCache_1.default.addNullishStorageKey(mapping.key);
-                delete callbackToStateMapping[subscriptionID];
-                return subscriptionID;
-            }
-        }
-    }
-    catch (e) {
-        // Not a collection member key, proceed as usual.
-    }
     // When keyChanged is called, a key is passed and the method looks through all the Subscribers in callbackToStateMapping for the matching key to get the subscriptionID
     // to avoid having to loop through all the Subscribers all the time (even when just one connection belongs to one key),
     // We create a mapping from key to lists of subscriptionIDs to access the specific list of subscriptionIDs.
@@ -1433,12 +1415,6 @@ function logKeyChanged(onyxMethod, key, value, hasChanged) {
 function logKeyRemoved(onyxMethod, key) {
     Logger.logInfo(`${onyxMethod} called for key: ${key} => null passed, so key was removed`);
 }
-/**
- * Getter - returns the callback to state mapping, useful in test environments.
- */
-function getCallbackToStateMapping() {
-    return callbackToStateMapping;
-}
 /**
  * Clear internal variables used in this file, useful in test environments.
  */
@@ -1497,6 +1473,5 @@ const OnyxUtils = {
     setWithRetry,
     multiSetWithRetry,
     setCollectionWithRetry,
-    getCallbackToStateMapping,
 };
 exports.default = OnyxUtils;

package/dist/useOnyx.js

@@ -210,12 +210,8 @@ function useOnyx(key, options, dependencies = []) {
             newValueRef.current = null;
             sourceValueRef.current = undefined;
             resultRef.current = [undefined, { status: 'loading' }];
+            shouldGetCachedValueRef.current = true;
         }
-        // Force a cache re-read on every (re)subscription so any side effects from
-        // subscribeToKey (e.g. addNullishStorageKey for skippable collection member ids)
-        // are reflected in the next getSnapshot. Resetting this flag does not change
-        // resultRef by itself, so it doesn't cause an extra mount render.
-        shouldGetCachedValueRef.current = true;
         hasMountedRef.current = true;
         isConnectingRef.current = true;
         onStoreChangeFnRef.current = onStoreChange;

package/package.json

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