chore: Remove unusued performance metrics system

Author: MrMuzyk

lib/GlobalSettings.ts

@@ -26,8 +26,6 @@ import OnyxUtils from './OnyxUtils';
 import logMessages from './logMessages';
 import type {Connection} from './OnyxConnectionManager';
 import connectionManager from './OnyxConnectionManager';
-import * as GlobalSettings from './GlobalSettings';
-import decorateWithMetrics from './metrics';
 import OnyxMerge from './OnyxMerge';
 
 /** Initialize the store with actions and listening for storage events */
@@ -37,17 +35,11 @@ function init({
     evictableKeys = [],
     maxCachedKeysCount = 1000,
     shouldSyncMultipleInstances = !!global.localStorage,
-    enablePerformanceMetrics = false,
     enableDevTools = true,
     skippableCollectionMemberIDs = [],
     ramOnlyKeys = [],
     snapshotMergeKeys = [],
 }: InitOptions): void {
-    if (enablePerformanceMetrics) {
-        GlobalSettings.setPerformanceMetricsEnabled(true);
-        applyDecorators();
-    }
-
     initDevTools(enableDevTools);
 
     Storage.init();
@@ -604,25 +596,5 @@ const Onyx = {
     registerLogger: Logger.registerLogger,
 };
 
-function applyDecorators() {
-    // We are reassigning the functions directly so that internal function calls are also decorated
-    // @ts-expect-error Reassign
-    connect = decorateWithMetrics(connect, 'Onyx.connect');
-    // @ts-expect-error Reassign
-    connectWithoutView = decorateWithMetrics(connectWithoutView, 'Onyx.connectWithoutView');
-    // @ts-expect-error Reassign
-    set = decorateWithMetrics(set, 'Onyx.set');
-    // @ts-expect-error Reassign
-    multiSet = decorateWithMetrics(multiSet, 'Onyx.multiSet');
-    // @ts-expect-error Reassign
-    merge = decorateWithMetrics(merge, 'Onyx.merge');
-    // @ts-expect-error Reassign
-    mergeCollection = decorateWithMetrics(mergeCollection, 'Onyx.mergeCollection');
-    // @ts-expect-error Reassign
-    update = decorateWithMetrics(update, 'Onyx.update');
-    // @ts-expect-error Reassign
-    clear = decorateWithMetrics(clear, 'Onyx.clear');
-}
-
 export default Onyx;
 export type {OnyxUpdate, ConnectOptions, SetOptions};

lib/Onyx.ts

@@ -36,8 +36,6 @@ import type {FastMergeOptions, FastMergeResult} from './utils';
 import utils from './utils';
 import type {DeferredTask} from './createDeferredTask';
 import createDeferredTask from './createDeferredTask';
-import * as GlobalSettings from './GlobalSettings';
-import decorateWithMetrics from './metrics';
 import type {StorageKeyValuePair} from './storage/providers/types';
 import logMessages from './logMessages';
 
@@ -1906,54 +1904,6 @@ const OnyxUtils = {
     isRamOnlyKey,
 };
 
-GlobalSettings.addGlobalSettingsChangeListener(({enablePerformanceMetrics}) => {
-    if (!enablePerformanceMetrics) {
-        return;
-    }
-    // We are reassigning the functions directly so that internal function calls are also decorated
-
-    // @ts-expect-error Reassign
-    initStoreValues = decorateWithMetrics(initStoreValues, 'OnyxUtils.initStoreValues');
-    // @ts-expect-error Complex type signature
-    get = decorateWithMetrics(get, 'OnyxUtils.get');
-    // @ts-expect-error Reassign
-    getAllKeys = decorateWithMetrics(getAllKeys, 'OnyxUtils.getAllKeys');
-    // @ts-expect-error Reassign
-    getCollectionKeys = decorateWithMetrics(getCollectionKeys, 'OnyxUtils.getCollectionKeys');
-    // @ts-expect-error Reassign
-    keysChanged = decorateWithMetrics(keysChanged, 'OnyxUtils.keysChanged');
-    // @ts-expect-error Reassign
-    keyChanged = decorateWithMetrics(keyChanged, 'OnyxUtils.keyChanged');
-    // @ts-expect-error Reassign
-    sendDataToConnection = decorateWithMetrics(sendDataToConnection, 'OnyxUtils.sendDataToConnection');
-    // @ts-expect-error Reassign
-    scheduleSubscriberUpdate = decorateWithMetrics(scheduleSubscriberUpdate, 'OnyxUtils.scheduleSubscriberUpdate');
-    // @ts-expect-error Reassign
-    scheduleNotifyCollectionSubscribers = decorateWithMetrics(scheduleNotifyCollectionSubscribers, 'OnyxUtils.scheduleNotifyCollectionSubscribers');
-    // @ts-expect-error Reassign
-    remove = decorateWithMetrics(remove, 'OnyxUtils.remove');
-    // @ts-expect-error Reassign
-    reportStorageQuota = decorateWithMetrics(reportStorageQuota, 'OnyxUtils.reportStorageQuota');
-    // @ts-expect-error Complex type signature
-    retryOperation = decorateWithMetrics(retryOperation, 'OnyxUtils.retryOperation');
-    // @ts-expect-error Reassign
-    broadcastUpdate = decorateWithMetrics(broadcastUpdate, 'OnyxUtils.broadcastUpdate');
-    // @ts-expect-error Reassign
-    initializeWithDefaultKeyStates = decorateWithMetrics(initializeWithDefaultKeyStates, 'OnyxUtils.initializeWithDefaultKeyStates');
-    // @ts-expect-error Complex type signature
-    multiGet = decorateWithMetrics(multiGet, 'OnyxUtils.multiGet');
-    // @ts-expect-error Reassign
-    tupleGet = decorateWithMetrics(tupleGet, 'OnyxUtils.tupleGet');
-    // @ts-expect-error Reassign
-    subscribeToKey = decorateWithMetrics(subscribeToKey, 'OnyxUtils.subscribeToKey');
-    // @ts-expect-error Reassign
-    setWithRetry = decorateWithMetrics(setWithRetry, 'OnyxUtils.setWithRetry');
-    // @ts-expect-error Reassign
-    multiSetWithRetry = decorateWithMetrics(multiSetWithRetry, 'OnyxUtils.multiSetWithRetry');
-    // @ts-expect-error Reassign
-    setCollectionWithRetry = decorateWithMetrics(setCollectionWithRetry, 'OnyxUtils.setCollectionWithRetry');
-});
-
 export type {OnyxMethod};
 export default OnyxUtils;
 export {clearOnyxUtilsInternals};

lib/OnyxUtils.ts

@@ -4,9 +4,6 @@ import PlatformStorage from './platforms';
 import InstanceSync from './InstanceSync';
 import MemoryOnlyProvider from './providers/MemoryOnlyProvider';
 import type StorageProvider from './providers/types';
-import * as GlobalSettings from '../GlobalSettings';
-import decorateWithMetrics from '../metrics';
-
 let provider = PlatformStorage as StorageProvider<unknown>;
 let shouldKeepInstancesSync = false;
 let finishInitalization: (value?: unknown) => void;
@@ -209,23 +206,4 @@ const storage: Storage = {
     },
 };
 
-GlobalSettings.addGlobalSettingsChangeListener(({enablePerformanceMetrics}) => {
-    if (!enablePerformanceMetrics) {
-        return;
-    }
-
-    // Apply decorators
-    storage.getItem = decorateWithMetrics(storage.getItem, 'Storage.getItem');
-    storage.multiGet = decorateWithMetrics(storage.multiGet, 'Storage.multiGet');
-    storage.setItem = decorateWithMetrics(storage.setItem, 'Storage.setItem');
-    storage.multiSet = decorateWithMetrics(storage.multiSet, 'Storage.multiSet');
-    storage.mergeItem = decorateWithMetrics(storage.mergeItem, 'Storage.mergeItem');
-    storage.multiMerge = decorateWithMetrics(storage.multiMerge, 'Storage.multiMerge');
-    storage.removeItem = decorateWithMetrics(storage.removeItem, 'Storage.removeItem');
-    storage.removeItems = decorateWithMetrics(storage.removeItems, 'Storage.removeItems');
-    storage.clear = decorateWithMetrics(storage.clear, 'Storage.clear');
-    storage.getAllKeys = decorateWithMetrics(storage.getAllKeys, 'Storage.getAllKeys');
-    storage.getAll = decorateWithMetrics(storage.getAll, 'Storage.getAll');
-});
-
 export default storage;

lib/dependencies/ModuleProxy.ts

@@ -407,12 +407,6 @@ type InitOptions = {
      */
     shouldSyncMultipleInstances?: boolean;
 
-    /**
-     * If enabled it will use the performance API to measure the time taken by Onyx operations.
-     * @default false
-     */
-    enablePerformanceMetrics?: boolean;
-
     /**
      * If enabled, it will connect to Redux DevTools Extension for debugging.
      * This allows you to see all Onyx state changes in the Redux DevTools.

lib/dependencies/PerformanceProxy/index.native.ts

@@ -5,10 +5,8 @@ import OnyxCache, {TASK} from './OnyxCache';
 import type {Connection} from './OnyxConnectionManager';
 import connectionManager from './OnyxConnectionManager';
 import OnyxUtils from './OnyxUtils';
-import * as GlobalSettings from './GlobalSettings';
 import type {CollectionKeyBase, OnyxKey, OnyxValue} from './types';
 import usePrevious from './usePrevious';
-import decorateWithMetrics from './metrics';
 import onyxSnapshotCache from './OnyxSnapshotCache';
 import useLiveRef from './useLiveRef';
 
@@ -329,19 +327,11 @@ function useOnyx<TKey extends OnyxKey, TReturnValue = OnyxValue<TKey>>(
         [key, options?.initWithStoredValues, options?.reuseConnection, checkEvictableKey],
     );
 
-    const getSnapshotDecorated = useMemo(() => {
-        if (!GlobalSettings.isPerformanceMetricsEnabled()) {
-            return getSnapshot;
-        }
-
-        return decorateWithMetrics(getSnapshot, 'useOnyx.getSnapshot');
-    }, [getSnapshot]);
-
     useEffect(() => {
         checkEvictableKey();
     }, [checkEvictableKey]);
 
-    const result = useSyncExternalStore<UseOnyxResult<TReturnValue>>(subscribe, getSnapshotDecorated);
+    const result = useSyncExternalStore<UseOnyxResult<TReturnValue>>(subscribe, getSnapshot);
 
     return result;
 }