react-native-onyx 1.0.75 → 1.0.77

package/lib/Onyx.js

@@ -8,7 +8,6 @@ import createDeferredTask from './createDeferredTask';
 import fastMerge from './fastMerge';
 import * as PerformanceUtils from './metrics/PerformanceUtils';
 import Storage from './storage';
-import DevTools from './DevTools';
 
 // Method constants
 const METHOD = {
@@ -38,11 +37,6 @@ let recentlyAccessedKeys = [];
 // whatever appears in this list it will NEVER be a candidate for eviction.
 let evictionAllowList = [];
 
-let devTools = {
-    registerAction: () => {},
-    clearState: () => {},
-};
-
 // Holds a map of keys and connectionID arrays whose keys will never be automatically evicted as
 // long as we have at least one subscriber that returns false for the canEvict property.
 const evictionBlocklist = {};
@@ -848,10 +842,7 @@ function notifyCollectionSubscribersOnNextTick(key, value) {
 function remove(key) {
     cache.drop(key);
     notifySubscribersOnNextTick(key, null);
-    return Storage.removeItem(key).then((result) => {
-        devTools.registerAction(`REMOVE/${key.toUpperCase()}`, undefined, {[key]: undefined});
-        return result;
-    });
+    return Storage.removeItem(key);
 }
 
 /**
@@ -975,7 +966,6 @@ function set(key, value) {
     const hasChanged = cache.hasValueChanged(key, valueWithNullRemoved);
 
     // This approach prioritizes fast UI changes without waiting for data to be stored in device storage.
-    broadcastUpdate(key, value, 'set');
     broadcastUpdate(key, valueWithNullRemoved, hasChanged, 'set');
 
     // If the value has not changed, calling Storage.setItem() would be redundant and a waste of performance, so return early instead.
@@ -984,10 +974,6 @@ function set(key, value) {
     }
 
     return Storage.setItem(key, valueWithNullRemoved)
-        .then((result) => {
-            devTools.registerAction(`SET/${key.toUpperCase()}`, valueWithNullRemoved, {[key]: valueWithNullRemoved});
-            return result;
-        })
         .catch(error => evictStorageAndRetry(error, set, key, valueWithNullRemoved));
 }
 
@@ -1018,11 +1004,6 @@ function multiSet(data) {
         // Update cache and optimistically inform subscribers on the next tick
         cache.set(key, val);
         notifySubscribersOnNextTick(key, val);
-        if (_.isNull(val)) {
-            devTools.registerAction(`REMOVE/${key.toUpperCase()}`, val, {[key]: undefined});
-        } else {
-            devTools.registerAction(`SET/${key.toUpperCase()}`, val, {[key]: val});
-        }
     });
 
     return Storage.multiSet(keyValuePairs)
@@ -1117,10 +1098,7 @@ function merge(key, changes) {
                     return Promise.resolve();
                 }
 
-                return Storage.mergeItem(key, batchedChanges, modifiedData).then((results) => {
-                    devTools.registerAction(`MERGE/${key.toUpperCase()}`, modifiedData, {[key]: modifiedData});
-                    return results;
-                });
+                return Storage.mergeItem(key, batchedChanges, modifiedData);
             } catch (error) {
                 Logger.logAlert(`An error occurred while applying merge for key: ${key}, Error: ${error}`);
             }
@@ -1132,10 +1110,9 @@ function merge(key, changes) {
 /**
  * Merge user provided default key value pairs.
  * @private
- * @param {boolean} enableDevTools
  * @returns {Promise}
  */
-function initializeWithDefaultKeyStates(enableDevTools = false) {
+function initializeWithDefaultKeyStates() {
     return Storage.multiGet(_.keys(defaultKeyStates))
         .then((pairs) => {
             const asObject = _.object(pairs);
@@ -1143,9 +1120,6 @@ function initializeWithDefaultKeyStates(enableDevTools = false) {
             const merged = fastMerge(asObject, defaultKeyStates);
             cache.merge(merged);
             _.each(merged, (val, key) => keyChanged(key, val));
-            if (enableDevTools) {
-                devTools = new DevTools(merged);
-            }
         });
 }
 
@@ -1193,7 +1167,7 @@ function clear(keysToPreserve = []) {
                 //      since collection key subscribers need to be updated differently
                 if (!isKeyToPreserve) {
                     const oldValue = cache.getValue(key);
-                    const newValue = _.get(defaultKeyStates, key, undefined);
+                    const newValue = _.get(defaultKeyStates, key, null);
                     if (newValue !== oldValue) {
                         cache.set(key, newValue);
                         const collectionKey = key.substring(0, key.indexOf('_') + 1);
@@ -1224,15 +1198,11 @@ function clear(keysToPreserve = []) {
                 notifyCollectionSubscribersOnNextTick(key, value);
             });
 
-            const defaultKeyValueState = _.omit(defaultKeyStates, keysToPreserve);
-            const defaultKeyValuePairs = _.pairs(defaultKeyValueState);
+            const defaultKeyValuePairs = _.pairs(_.omit(defaultKeyStates, keysToPreserve));
 
             // Remove only the items that we want cleared from storage, and reset others to default
             _.each(keysToBeClearedFromStorage, key => cache.drop(key));
-            return Storage.removeItems(keysToBeClearedFromStorage).then(() => {
-                devTools.clearState(keysToBeClearedFromStorage);
-                return Storage.multiSet(defaultKeyValuePairs);
-            });
+            return Storage.removeItems(keysToBeClearedFromStorage).then(() => Storage.multiSet(defaultKeyValuePairs));
         });
 }
 
@@ -1306,11 +1276,6 @@ function mergeCollection(collectionKey, collection) {
             Promise.all(_.map(existingKeys, get)).then(() => {
                 cache.merge(collection);
                 keysChanged(collectionKey, collection);
-                if (_.isNull(collection)) {
-                    devTools.registerAction(`REMOVE/${collectionKey.toUpperCase()}`, collection, {[collectionKey]: undefined});
-                } else {
-                    devTools.registerAction(`SET/${collectionKey.toUpperCase()}`, collection, {[collectionKey]: collection});
-                }
             });
 
             return Promise.all(promises)
@@ -1412,7 +1377,6 @@ function init({
     captureMetrics = false,
     shouldSyncMultipleInstances = Boolean(global.localStorage),
     debugSetState = false,
-    enableDevTools = false,
 } = {}) {
     if (captureMetrics) {
         // The code here is only bundled and applied when the captureMetrics is set
@@ -1445,7 +1409,7 @@ function init({
     // Initialize all of our keys with data provided then give green light to any pending connections
     Promise.all([
         addAllSafeEvictionKeysToRecentlyAccessedList(),
-        initializeWithDefaultKeyStates(enableDevTools),
+        initializeWithDefaultKeyStates(),
     ])
         .then(deferredInitTask.resolve);
 
@@ -1453,11 +1417,6 @@ function init({
         Storage.keepInstancesSync((key, value) => {
             cache.set(key, value);
             keyChanged(key, value);
-            if (_.isNull(value)) {
-                devTools.registerAction(`REMOVE/${key.toUpperCase()}`, value, {[key]: undefined});
-            } else {
-                devTools.registerAction(`SET/${key.toUpperCase()}`, value, {[key]: value});
-            }
         });
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-native-onyx",
-  "version": "1.0.75",
+  "version": "1.0.77",
   "author": "Expensify, Inc.",
   "homepage": "https://expensify.com",
   "description": "State management for React Native",
@@ -71,7 +71,6 @@
     "react-native-performance": "^2.0.0",
     "react-native-quick-sqlite": "^8.0.0-beta.2",
     "react-test-renderer": "18.1.0",
-    "remotedev": "^0.2.9",
     "type-fest": "^3.12.0",
     "webpack": "^5.72.1",
     "webpack-cli": "^4.9.2",
@@ -80,7 +79,7 @@
   "peerDependencies": {
     "idb-keyval": "^6.2.1",
     "react": ">=18.1.0",
-    "react-native-performance": "^4.0.0",
+    "react-native-performance": "^5.1.0",
     "react-native-quick-sqlite": "^8.0.0-beta.2",
     "react-native-device-info": "^10.3.0"
   },

package/dist/web.min.js.LICENSE.txt

@@ -1 +0,0 @@
-/*! http://mths.be/base64 v0.1.0 by @mathias | MIT license */

package/lib/DevTools.js

@@ -1,55 +0,0 @@
-import _ from 'underscore';
-
-const {connectViaExtension} = process.env.NODE_ENV === 'production' ? require('remotedev') : {
-    connectViaExtension: () => ({
-        init: () => {},
-        send: () => {},
-    }),
-};
-
-class DevTools {
-    /**
-     * @callback onStateChange
-     * @param {object} state
-     */
-    /**
-     * Creates an instance of DevTools, with an internal state that mirrors the storage.
-     *
-     * @param {object} initialState - initial state of the storage
-     * @param {onStateChange} onStateChange - callback which is triggered when we timetravel to a different registered action
-     */
-    constructor(initialState = {}) {
-        this.state = initialState;
-        this.remotedev = connectViaExtension();
-        this.remotedev.init(this.state);
-    }
-
-    /**
-     * Registers an action that updated the current state of the storage
-     *
-     * @param {string} type - name of the action
-     * @param {any} payload - data written to the storage
-     * @param {object} stateChanges - partial state that got updated after the changes
-     */
-    registerAction(type, payload = undefined, stateChanges = {}) {
-        const newState = {
-            ...this.state,
-            ...stateChanges,
-        };
-
-        this.remotedev.send({type, payload}, newState);
-        this.state = newState;
-    }
-
-    /**
-     * This clears the internal state of the DevTools, preserving the keys not included in `keyToBeRemoved`
-     *
-     * @param {string[]} keysToBeRemoved
-     */
-    clearState(keysToBeRemoved = []) {
-        const pairs = _.map(keysToBeRemoved, key => [key, undefined]);
-        this.registerAction('CLEAR', undefined, _.object(pairs));
-    }
-}
-
-export default DevTools;