wrap Onyx.set for derived values in a method

src/libs/actions/OnyxDerived/configs/reportAttributes.ts

@@ -1,7 +1,7 @@
 import {generateIsEmptyReport, generateReportAttributes, generateReportName, isValidReport} from '@libs/ReportUtils';
 import SidebarUtils from '@libs/SidebarUtils';
 import createOnyxDerivedValueConfig from '@userActions/OnyxDerived/createOnyxDerivedValueConfig';
-import hasKeyTriggeredCompute from '@userActions/OnyxDerived/utils';
+import {hasKeyTriggeredCompute} from '@userActions/OnyxDerived/utils';
 import CONST from '@src/CONST';
 import ONYXKEYS from '@src/ONYXKEYS';
 import type {ReportAttributesDerivedValue} from '@src/types/onyx';

src/libs/actions/OnyxDerived/index.ts

@@ -13,6 +13,7 @@
 import ObjectUtils from '@src/types/utils/ObjectUtils';
 import ONYX_DERIVED_VALUES from './ONYX_DERIVED_VALUES';
 import type {DerivedValueContext} from './types';
+import {setDerivedValue} from './utils';
 
 /**
  * Initialize all Onyx derived values, store them in Onyx, and setup listeners to update them when dependencies change.
@@ -42,7 +43,7 @@
                     // @ts-expect-error TypeScript can't confirm the shape of dependencyValues matches the compute function's parameters
                     derivedValue = compute(dependencyValues, initialContext);
                     dependencyValues = values;
-                    Onyx.set(key, derivedValue ?? null);
+                    setDerivedValue(key, derivedValue ?? null);
                 });
             }
 
@@ -84,7 +85,7 @@
                 const newDerivedValue = compute(dependencyValues, context);
                 Log.info(`[OnyxDerived] updating value for ${key} in Onyx`);
                 derivedValue = newDerivedValue;
-                Onyx.set(key, derivedValue ?? null);
+                setDerivedValue(key, derivedValue);
             };
 
             for (let i = 0; i < dependencies.length; i++) {
@@ -92,7 +93,7 @@
                 const dependencyOnyxKey = dependencies[dependencyIndex];
 
                 if (OnyxUtils.isCollectionKey(dependencyOnyxKey)) {
                     Onyx.connect({
                         key: dependencyOnyxKey,
                         waitForCollectionCallback: true,
                         callback: (value, collectionKey, sourceValue) => {
@@ -103,7 +104,7 @@
                     });
                 } else if (dependencyOnyxKey === ONYXKEYS.NVP_PREFERRED_LOCALE) {
                     // Special case for locale, we want to recompute derived values when the locale change actually loads.
                     Onyx.connect({
                         key: ONYXKEYS.ARE_TRANSLATIONS_LOADING,
                         initWithStoredValues: false,
                         callback: (value) => {
@@ -123,7 +124,7 @@
                         },
                     });
                 } else {
                     Onyx.connect({
                         key: dependencyOnyxKey,
                         callback: (value) => {
                             Log.info(`[OnyxDerived] dependency ${dependencyOnyxKey} for derived key ${key} changed, recomputing`);

src/libs/actions/OnyxDerived/utils.ts

@@ -1,5 +1,7 @@
+import Onyx from 'react-native-onyx';
+import type {OnyxInput} from 'react-native-onyx';
 import type {NonEmptyTuple} from 'type-fest';
-import type {OnyxKey} from '@src/ONYXKEYS';
+import type {OnyxDerivedKey, OnyxKey} from '@src/ONYXKEYS';
 import type {DerivedValueContext} from './types';
 
 /**
@@ -12,4 +14,16 @@ const hasKeyTriggeredCompute = <K extends OnyxKey, Deps extends NonEmptyTuple<Ex
     return Object.keys(sourceValues).some((sourceKey) => sourceKey === key);
 };
 
-export default hasKeyTriggeredCompute;
+/**
+ * Set a derived value in Onyx
+ * As a performance optimization, it skips the cache check and null removal
+ * For derived values, we fully control their lifecycle and recompute them when any dependency changes - so we don’t need a deep comparison
+ * Also, null may be a legitimate result of the computation, so pruning it is unnecessary
+ */
+const setDerivedValue = (key: OnyxDerivedKey, value: OnyxInput<OnyxDerivedKey>) =>
+    Onyx.set(key, value, {
+        skipCacheCheck: true,
+        skipNullRemoval: true,
+    });
+
+export {hasKeyTriggeredCompute, setDerivedValue};