@legendapp/state 0.23.2 → 1.0.0-rc.1

package/persist.js

@@ -7,13 +7,14 @@ function configureObservablePersistence(options) {
     Object.assign(observablePersistConfiguration, options);
 }
 
-function transformPath(path, map, passThroughKeys, ignoreKeys) {
+let validateMap;
+function transformPath(path, map, passThroughKeys) {
     const data = {};
     let d = data;
     for (let i = 0; i < path.length; i++) {
         d = d[path[i]] = i === path.length - 1 ? null : {};
     }
-    let value = transformObject(data, map, passThroughKeys, ignoreKeys);
+    let value = transformObject(data, map, passThroughKeys);
     const pathOut = [];
     for (let i = 0; i < path.length; i++) {
         const key = Object.keys(value)[0];
@@ -23,11 +24,20 @@ function transformPath(path, map, passThroughKeys, ignoreKeys) {
     return pathOut;
 }
 function transformObject(dataIn, map, passThroughKeys, ignoreKeys) {
+    if (process.env.NODE_ENV === 'development') {
+        validateMap(map);
+    }
     // Note: If changing this, change it in IndexedDB preloader
     let ret = dataIn;
     if (dataIn) {
+        if (dataIn === state.symbolDelete)
+            return dataIn;
         ret = {};
         const dict = Object.keys(map).length === 1 && map['_dict'];
+        let dateModified = dataIn[state.symbolDateModified];
+        if (dateModified) {
+            ret[state.symbolDateModified] = dateModified;
+        }
         Object.keys(dataIn).forEach((key) => {
             if (ret[key] !== undefined || (ignoreKeys === null || ignoreKeys === void 0 ? void 0 : ignoreKeys.includes(key)))
                 return;
@@ -49,14 +59,24 @@ function transformObject(dataIn, map, passThroughKeys, ignoreKeys) {
                 }
                 else if (mapped !== null) {
                     if (v !== undefined && v !== null) {
-                        if (map[key + '_obj']) {
+                        if (map[key + '_val']) {
+                            const valMap = map[key + '_val'];
+                            v = valMap[key];
+                        }
+                        else if (map[key + '_obj']) {
                             v = transformObject(v, map[key + '_obj'], passThroughKeys, ignoreKeys);
                         }
                         else if (map[key + '_dict']) {
                             const mapChild = map[key + '_dict'];
+                            let out = {};
+                            let dateModifiedChild = dataIn[state.symbolDateModified];
+                            if (dateModifiedChild) {
+                                out[state.symbolDateModified] = dateModifiedChild;
+                            }
                             Object.keys(v).forEach((keyChild) => {
-                                v[keyChild] = transformObject(v[keyChild], mapChild, passThroughKeys, ignoreKeys);
+                                out[keyChild] = transformObject(v[keyChild], mapChild, passThroughKeys, ignoreKeys);
                             });
+                            v = out;
                         }
                         else if (map[key + '_arr']) {
                             const mapChild = map[key + '_arr'];
@@ -74,8 +94,14 @@ function transformObject(dataIn, map, passThroughKeys, ignoreKeys) {
         debugger;
     return ret;
 }
+function transformObjectWithPath(obj, path, pathTypes, fieldTransforms) {
+    let constructed = state.constructObjectWithPath(path, obj, pathTypes);
+    const transformed = transformObject(constructed, fieldTransforms, [state.dateModifiedKey]);
+    const transformedPath = transformPath(path, fieldTransforms, [state.dateModifiedKey]);
+    return { path: transformedPath, obj: state.deconstructObjectWithPath(transformedPath, transformed) };
+}
 const invertedMaps = new WeakMap();
-function invertMap(obj) {
+function invertFieldMap(obj) {
     // Note: If changing this, change it in IndexedDB preloader
     const existing = invertedMaps.get(obj);
     if (existing)
@@ -86,12 +112,12 @@ function invertMap(obj) {
         if (process.env.NODE_ENV === 'development' && target[val])
             debugger;
         if (key === '_dict') {
-            target[key] = invertMap(val);
+            target[key] = invertFieldMap(val);
         }
         else if (key.endsWith('_obj') || key.endsWith('_dict') || key.endsWith('_arr')) {
             const keyMapped = obj[key.replace(/_obj|_dict|_arr$/, '')];
             const suffix = key.match(/_obj|_dict|_arr$/)[0];
-            target[keyMapped + suffix] = invertMap(val);
+            target[keyMapped + suffix] = invertFieldMap(val);
         }
         else if (typeof val === 'string') {
             target[val] = key;
@@ -102,21 +128,25 @@ function invertMap(obj) {
     invertedMaps.set(obj, target);
     return target;
 }
-
-function removeNullUndefined(val) {
-    if (val === undefined)
-        return null;
-    Object.keys(val).forEach((key) => {
-        const v = val[key];
-        if (v === null || v === undefined) {
-            delete val[key];
-        }
-        else if (state.isObject(v)) {
-            removeNullUndefined(v);
+if (process.env.NODE_ENV === 'development') {
+    validateMap = function (record) {
+        const values = Object.values(record).filter((value) => {
+            if (state.isObject(value)) {
+                validateMap(value);
+            }
+            else {
+                return state.isString(value);
+            }
+        });
+        const uniques = Array.from(new Set(values));
+        if (values.length !== uniques.length) {
+            console.error('Field transform map has duplicate values', record, values.length, uniques.length);
+            debugger;
         }
-    });
-    return val;
+        return record;
+    };
 }
+
 function replaceKeyInObject(obj, keySource, keyTarget, clone) {
     if (state.isObject(obj)) {
         const target = clone ? {} : obj;
@@ -124,6 +154,9 @@ function replaceKeyInObject(obj, keySource, keyTarget, clone) {
             target[keyTarget] = obj[keySource];
             delete target[keySource];
         }
+        if (keySource !== state.symbolDateModified && obj[state.symbolDateModified]) {
+            target[state.symbolDateModified] = obj[state.symbolDateModified];
+        }
         Object.keys(obj).forEach((key) => {
             if (key !== keySource) {
                 target[key] = replaceKeyInObject(obj[key], keySource, keyTarget, clone);
@@ -138,59 +171,134 @@ function replaceKeyInObject(obj, keySource, keyTarget, clone) {
 function getDateModifiedKey(dateModifiedKey) {
     return dateModifiedKey || observablePersistConfiguration.dateModifiedKey || '@';
 }
+function mergeDateModified(obs, source) {
+    const isArr = state.isArray(source);
+    const isObj = !isArr && state.isObject(source);
+    let dateModified = isObj && source[state.symbolDateModified];
+    if (dateModified) {
+        delete source[state.symbolDateModified];
+    }
+    if (isArr || isObj) {
+        const keys = isArr ? source : Object.keys(source);
+        for (let i = 0; i < keys.length; i++) {
+            const key = isArr ? i : keys[i];
+            dateModified = Math.max(dateModified || 0, mergeDateModified(obs[key], source[key]));
+        }
+    }
+    if (dateModified) {
+        obs[state.symbolDateModified].set(dateModified);
+    }
+    return dateModified || 0;
+}
 
 const mapPersistences = new WeakMap();
 const persistState = state.observable({ inRemoteSync: false });
 function parseLocalConfig(config) {
     return state.isString(config) ? { table: config, config: { name: config } } : { table: config.name, config };
 }
-async function onObsChange(obs, obsState, localState, persistOptions, value, getPrevious, changes) {
+function adjustSaveData(value, path, pathTypes, { adjustData, fieldTransforms, }, replaceKey) {
+    let cloned = replaceKey ? replaceKeyInObject(value, state.symbolDateModified, state.dateModifiedKey, /*clone*/ true) : value;
+    const transform = () => {
+        if (fieldTransforms) {
+            const { obj, path: pathTransformed } = transformObjectWithPath(cloned, path, pathTypes, fieldTransforms);
+            cloned = obj;
+            path = pathTransformed;
+        }
+        return { value: cloned, path };
+    };
+    let promise;
+    if (adjustData === null || adjustData === void 0 ? void 0 : adjustData.save) {
+        const constructed = state.constructObjectWithPath(path, cloned, pathTypes);
+        promise = adjustData.save(constructed);
+    }
+    return state.isPromise(promise)
+        ? promise.then((adjusted) => {
+            cloned = state.deconstructObjectWithPath(path, adjusted);
+            return transform();
+        })
+        : transform();
+}
+function adjustLoadData(value, { adjustData, fieldTransforms, }, replaceKey) {
+    let cloned = replaceKey ? replaceKeyInObject(value, state.dateModifiedKey, state.symbolDateModified, /*clone*/ true) : value;
+    if (fieldTransforms) {
+        const inverted = invertFieldMap(fieldTransforms);
+        cloned = transformObject(cloned, inverted, [state.dateModifiedKey]);
+    }
+    if (adjustData === null || adjustData === void 0 ? void 0 : adjustData.load) {
+        cloned = adjustData.load(cloned);
+    }
+    return cloned;
+}
+async function onObsChange(obs, obsState, localState, persistOptions, { value, changes }) {
+    var _a;
     const { persistenceLocal, persistenceRemote } = localState;
     const local = persistOptions.local;
     const { table, config } = parseLocalConfig(local);
+    const configRemote = persistOptions.remote;
     const inRemoteChange = state.tracking.inRemoteChange;
-    const saveRemote = !inRemoteChange && persistOptions.remote && !persistOptions.remote.readonly && obsState.isEnabledRemote.peek();
-    if (local && obsState.isEnabledLocal.peek()) {
+    const saveRemote = !inRemoteChange && configRemote && !configRemote.readonly && obsState.isEnabledRemote.peek();
+    const isQueryingModified = !!((_a = configRemote === null || configRemote === void 0 ? void 0 : configRemote.firebase) === null || _a === void 0 ? void 0 : _a.queryByModified);
+    if (local && !config.readonly && obsState.isEnabledLocal.peek()) {
         if (!obsState.isLoadedLocal.peek()) {
             console.error('[legend-state] WARNING: An observable was changed before being loaded from persistence', local);
             return;
         }
-        // If saving remotely convert symbolDateModified to dateModifiedKey before saving locally
-        // as persisting may not include symbols correctly
-        let localValue = value;
-        if (persistOptions.remote) {
-            if (saveRemote) {
-                for (let i = 0; i < changes.length; i++) {
-                    const { path, valueAtPath, prevAtPath } = changes[i];
-                    if (path[path.length - 1] === state.symbolDateModified)
-                        continue;
-                    const pathStr = path.join('/');
-                    if (!localState.pendingChanges) {
-                        localState.pendingChanges = {};
-                    }
-                    // The value saved in pending should be the previous state before changes,
-                    // so don't overwrite it if it already exists
-                    if (!localState.pendingChanges[pathStr]) {
-                        localState.pendingChanges[pathStr] = { p: prevAtPath !== null && prevAtPath !== void 0 ? prevAtPath : null };
-                    }
-                    localState.pendingChanges[pathStr].v = valueAtPath;
+        // Prepare pending changes
+        if (saveRemote) {
+            for (let i = 0; i < changes.length; i++) {
+                const { path, valueAtPath, prevAtPath, pathTypes } = changes[i];
+                if (path[path.length - 1] === state.symbolDateModified)
+                    continue;
+                const pathStr = path.join('/');
+                if (!localState.pendingChanges) {
+                    localState.pendingChanges = {};
                 }
+                // The value saved in pending should be the previous state before changes,
+                // so don't overwrite it if it already exists
+                if (!localState.pendingChanges[pathStr]) {
+                    localState.pendingChanges[pathStr] = { p: prevAtPath !== null && prevAtPath !== void 0 ? prevAtPath : null, t: pathTypes };
+                }
+                localState.pendingChanges[pathStr].v = valueAtPath;
             }
         }
-        let changesLocal = changes;
-        if (config.fieldTransforms) {
-            localValue = transformObject(localValue, config.fieldTransforms, [state.dateModifiedKey]);
-            changesLocal = changesLocal.map(({ path, prevAtPath, valueAtPath }) => {
-                let transformed = state.constructObject(path, state.clone(valueAtPath));
-                transformed = transformObject(transformed, config.fieldTransforms, [state.dateModifiedKey]);
-                const transformedPath = transformPath(path, config.fieldTransforms, [state.dateModifiedKey]);
-                const toSave = state.deconstructObject(transformedPath, transformed);
-                return { path, prevAtPath, valueAtPath: toSave };
-            });
+        // Save changes locally
+        const changesLocal = [];
+        const changesPaths = new Set();
+        const promises = [];
+        changes.forEach((_, i) => {
+            // Reverse order
+            let { path: pathOriginal, prevAtPath, valueAtPath, pathTypes } = changes[changes.length - 1 - i];
+            if (state.isSymbol(pathOriginal[pathOriginal.length - 1])) {
+                return;
+            }
+            if (isQueryingModified) {
+                pathOriginal = pathOriginal.map((p) => (p === state.symbolDateModified ? state.dateModifiedKey : p));
+            }
+            const pathStr = pathOriginal.join('/');
+            // Optimization to only save the latest update at each path. We might have multiple changes at the same path
+            // and we only need the latest value, so it starts from the end of the array, skipping any earlier changes
+            // already processed.
+            if (!changesPaths.has(pathStr)) {
+                changesPaths.add(pathStr);
+                let promise = adjustSaveData(valueAtPath, pathOriginal, pathTypes, config, isQueryingModified);
+                const push = ({ path, value }) => {
+                    changesLocal.push({ path, pathTypes, prevAtPath, valueAtPath: value });
+                };
+                if (state.isPromise(promise)) {
+                    promises.push(promise.then(push));
+                }
+                else {
+                    push(promise);
+                }
+            }
+        });
+        if (promises.length > 0) {
+            await Promise.all(promises);
         }
-        localValue = replaceKeyInObject(localValue, state.symbolDateModified, state.dateModifiedKey, 
-        /*clone*/ true);
-        persistenceLocal.set(table, localValue, changesLocal, config);
+        if (changesLocal.length > 0) {
+            persistenceLocal.set(table, changesLocal, config);
+        }
+        // Save metadata
         const metadata = {};
         if (inRemoteChange) {
             const dateModified = value[state.symbolDateModified];
@@ -206,49 +314,55 @@ async function onObsChange(obs, obsState, localState, persistOptions, value, get
         }
     }
     if (saveRemote) {
-        await state.when(obsState.isLoadedRemote);
-        for (let i = 0; i < changes.length; i++) {
-            const { path, valueAtPath, prevAtPath } = changes[i];
+        await state.when(() => obsState.isLoadedRemote.get() || (configRemote.allowSaveIfError && obsState.remoteError.get()));
+        const fieldTransforms = configRemote.fieldTransforms;
+        changes.forEach(async (change) => {
+            const { path, valueAtPath, prevAtPath, pathTypes } = change;
             if (path[path.length - 1] === state.symbolDateModified)
-                continue;
+                return;
             const pathStr = path.join('/');
+            const { path: pathSave, value: valueSave } = await adjustSaveData(valueAtPath, path, pathTypes, configRemote, isQueryingModified);
             // Save to remote persistence and get the remote value from it. Some providers (like Firebase) will return a
-            // server value different than the saved value (like Firebase has server timestamps for dateModified)
-            persistenceRemote.save(persistOptions, value, path, valueAtPath, prevAtPath).then((saved) => {
+            // server value with server timestamps for dateModified.
+            persistenceRemote
+                .save({
+                obs,
+                state: obsState,
+                options: persistOptions,
+                path: pathSave,
+                pathTypes,
+                valueAtPath: valueSave,
+                prevAtPath,
+            })
+                .then((saved) => {
                 var _a;
                 if (local) {
-                    let toSave = persistenceLocal.getTable(table, config);
                     const pending = (_a = persistenceLocal.getMetadata(table, config)) === null || _a === void 0 ? void 0 : _a.pending;
-                    let dateModified;
-                    let didDelete = false;
+                    // Clear pending for this path
                     if (pending === null || pending === void 0 ? void 0 : pending[pathStr]) {
-                        didDelete = true;
                         // Remove pending from the saved object
                         delete pending[pathStr];
                         // Remove pending from local state
                         delete localState.pendingChanges[pathStr];
+                        persistenceLocal.updateMetadata(table, { pending }, config);
                     }
                     // Only the latest save will return a value so that it saves back to local persistence once
-                    if (saved !== undefined) {
+                    // It needs to get the dateModified from the save and update that through the observable
+                    // which will fire onObsChange and save it locally.
+                    if (saved !== undefined && isQueryingModified) {
+                        // Note: Don't need to adjust data because we're just merging dateModified
+                        const invertedMap = fieldTransforms && invertFieldMap(fieldTransforms);
+                        if (invertedMap) {
+                            saved = transformObject(saved, invertedMap, [state.dateModifiedKey]);
+                        }
                         onChangeRemote(() => {
-                            state.mergeIntoObservable(obs, saved);
+                            mergeDateModified(obs, saved);
                         });
-                        dateModified = saved[state.symbolDateModified];
-                        // Replace the dateModifiedKey and remove null/undefined before saving
-                        if (config.fieldTransforms) {
-                            saved = transformObject(saved, config.fieldTransforms, [state.dateModifiedKey]);
-                        }
-                        saved = replaceKeyInObject(removeNullUndefined(saved), state.symbolDateModified, state.dateModifiedKey, 
-                        /*clone*/ false);
-                        toSave = toSave ? state.mergeIntoObservable(toSave, saved) : saved;
-                    }
-                    if (saved !== undefined || didDelete) {
-                        persistenceLocal.set(table, toSave, [changes[i]], config);
-                        persistenceLocal.updateMetadata(table, { pending, modified: dateModified }, config);
                     }
                 }
+                localState.onSaveRemoteListeners.forEach((cb) => cb());
             });
-        }
+        });
     }
 }
 function onChangeRemote(cb) {
@@ -266,23 +380,33 @@ function onChangeRemote(cb) {
     }
 }
 async function loadLocal(obs, persistOptions, obsState, localState) {
-    var _a;
+    var _a, _b, _c, _d;
     const { local, remote } = persistOptions;
     const localPersistence = persistOptions.persistLocal || observablePersistConfiguration.persistLocal;
     if (local) {
         const { table, config } = parseLocalConfig(local);
+        const isQueryingModified = !!((_b = (_a = persistOptions.remote) === null || _a === void 0 ? void 0 : _a.firebase) === null || _b === void 0 ? void 0 : _b.queryByModified);
         if (!localPersistence) {
             throw new Error('Local persistence is not configured');
         }
         // Ensure there's only one instance of the persistence plugin
         if (!mapPersistences.has(localPersistence)) {
             const persistenceLocal = new localPersistence();
+            const mapValue = { persist: persistenceLocal, initialized: state.observable(false) };
+            mapPersistences.set(localPersistence, mapValue);
             if (persistenceLocal.initialize) {
-                await ((_a = persistenceLocal.initialize) === null || _a === void 0 ? void 0 : _a.call(persistenceLocal, observablePersistConfiguration.persistLocalOptions));
+                const initializePromise = (_c = persistenceLocal.initialize) === null || _c === void 0 ? void 0 : _c.call(persistenceLocal, observablePersistConfiguration.persistLocalOptions);
+                if (state.isPromise(initializePromise)) {
+                    await initializePromise;
+                }
             }
-            mapPersistences.set(localPersistence, persistenceLocal);
+            mapValue.initialized.set(true);
+        }
+        const { persist: persistenceLocal, initialized } = mapPersistences.get(localPersistence);
+        localState.persistenceLocal = persistenceLocal;
+        if (!initialized.get()) {
+            await state.when(initialized);
         }
-        const persistenceLocal = (localState.persistenceLocal = mapPersistences.get(localPersistence));
         // If persistence has an asynchronous load, wait for it
         if (persistenceLocal.loadTable) {
             const promise = persistenceLocal.loadTable(table, config);
@@ -293,36 +417,37 @@ async function loadLocal(obs, persistOptions, obsState, localState) {
         // Get the value from state
         let value = persistenceLocal.getTable(table, config);
         const metadata = persistenceLocal.getMetadata(table, config);
-        if (config.fieldTransforms) {
-            // Get preloaded translated if available
-            let valueLoaded = persistenceLocal.getTableTransformed(table, config);
-            if (valueLoaded) {
-                value = valueLoaded;
-            }
-            else {
-                const inverted = invertMap(config.fieldTransforms);
-                value = transformObject(value, inverted, [state.dateModifiedKey]);
-            }
-        }
         if (metadata) {
             const pending = metadata.pending;
             localState.pendingChanges = pending;
         }
         // Merge the data from local persistence into the default state
         if (value !== null && value !== undefined) {
-            if (remote) {
-                replaceKeyInObject(value, state.dateModifiedKey, state.symbolDateModified, /*clone*/ false);
+            let { adjustData, fieldTransforms } = config;
+            if (fieldTransforms) {
+                let valueLoaded = (_d = persistenceLocal.getTableTransformed) === null || _d === void 0 ? void 0 : _d.call(persistenceLocal, table, config);
+                if (valueLoaded) {
+                    value = valueLoaded;
+                    fieldTransforms = undefined;
+                }
             }
-            if (metadata === null || metadata === void 0 ? void 0 : metadata.modified) {
-                value[state.symbolDateModified] = metadata.modified;
+            value = adjustLoadData(value, { adjustData, fieldTransforms }, !!remote && isQueryingModified);
+            if (state.isPromise(value)) {
+                value = await value;
             }
-            state.batch(() => state.mergeIntoObservable(obs, value));
+            state.batch(() => {
+                state.mergeIntoObservable(obs, value);
+            });
         }
         obsState.peek().clearLocal = () => persistenceLocal.deleteTable(table, config);
     }
     obsState.isLoadedLocal.set(true);
 }
 function persistObservable(obs, persistOptions) {
+    const { remote, local } = persistOptions;
+    const remotePersistence = persistOptions.persistRemote || (observablePersistConfiguration === null || observablePersistConfiguration === void 0 ? void 0 : observablePersistConfiguration.persistRemote);
+    const onSaveRemoteListeners = [];
+    const localState = { onSaveRemoteListeners };
     const obsState = state.observable({
         isLoadedLocal: false,
         isLoadedRemote: false,
@@ -330,10 +455,8 @@ function persistObservable(obs, persistOptions) {
         isEnabledRemote: true,
         clearLocal: undefined,
         sync: () => Promise.resolve(),
+        getPendingChanges: () => localState.pendingChanges,
     });
-    const { remote, local } = persistOptions;
-    const remotePersistence = persistOptions.persistRemote || (observablePersistConfiguration === null || observablePersistConfiguration === void 0 ? void 0 : observablePersistConfiguration.persistRemote);
-    const localState = {};
     if (local) {
         loadLocal(obs, persistOptions, obsState, localState);
     }
@@ -343,24 +466,62 @@ function persistObservable(obs, persistOptions) {
         }
         // Ensure there's only one instance of the persistence plugin
         if (!mapPersistences.has(remotePersistence)) {
-            mapPersistences.set(remotePersistence, new remotePersistence());
+            mapPersistences.set(remotePersistence, { persist: new remotePersistence() });
         }
-        localState.persistenceRemote = mapPersistences.get(remotePersistence);
+        localState.persistenceRemote = mapPersistences.get(remotePersistence).persist;
         let isSynced = false;
         const sync = async () => {
+            var _a;
             if (!isSynced) {
                 isSynced = true;
-                localState.persistenceRemote.listen(obs, persistOptions, () => {
-                    obsState.isLoadedRemote.set(true);
-                }, onChangeRemote);
-                await state.when(obsState.isLoadedRemote);
+                const onSaveRemote = (_a = persistOptions.remote) === null || _a === void 0 ? void 0 : _a.onSaveRemote;
+                if (onSaveRemote) {
+                    onSaveRemoteListeners.push(onSaveRemote);
+                }
+                localState.persistenceRemote.listen({
+                    state: obsState,
+                    obs,
+                    options: persistOptions,
+                    onLoad: () => {
+                        obsState.isLoadedRemote.set(true);
+                    },
+                    onChange: async ({ value, path, mode }) => {
+                        if (value !== undefined) {
+                            value = adjustLoadData(value, remote, true);
+                            if (state.isPromise(value)) {
+                                value = await value;
+                            }
+                            const pending = localState.pendingChanges;
+                            if (pending) {
+                                Object.keys(pending).forEach((key) => {
+                                    const p = key.split('/').filter((p) => p !== '');
+                                    const { v, t } = pending[key];
+                                    const constructed = state.constructObjectWithPath(p, v, t);
+                                    value = state.mergeIntoObservable(value, constructed);
+                                });
+                            }
+                            const invertedMap = remote.fieldTransforms && invertFieldMap(remote.fieldTransforms);
+                            if (path.length && invertedMap) {
+                                path = transformPath(path, invertedMap);
+                            }
+                            onChangeRemote(() => {
+                                state.setAtPath(obs, path, value, mode);
+                            });
+                        }
+                    },
+                });
+                await state.when(() => obsState.isLoadedRemote.get() || (remote.allowSaveIfError && obsState.remoteError.get()));
                 const pending = localState.pendingChanges;
                 if (pending) {
                     Object.keys(pending).forEach((key) => {
                         const path = key.split('/').filter((p) => p !== '');
-                        const { p, v } = pending[key];
+                        const { p, v, t } = pending[key];
                         // TODO getPrevious if any remote persistence layers need it
-                        onObsChange(obs, obsState, localState, persistOptions, obs.peek(), () => undefined, [{ path, valueAtPath: v, prevAtPath: p }]);
+                        onObsChange(obs, obsState, localState, persistOptions, {
+                            value: obs.peek(),
+                            getPrevious: () => undefined,
+                            changes: [{ path, valueAtPath: v, prevAtPath: p, pathTypes: t }],
+                        });
                     });
                 }
             }
@@ -389,10 +550,11 @@ function isInRemoteChange() {
 
 exports.configureObservablePersistence = configureObservablePersistence;
 exports.getDateModifiedKey = getDateModifiedKey;
-exports.invertMap = invertMap;
+exports.invertFieldMap = invertFieldMap;
 exports.isInRemoteChange = isInRemoteChange;
 exports.mapPersistences = mapPersistences;
 exports.observablePersistConfiguration = observablePersistConfiguration;
+exports.onChangeRemote = onChangeRemote;
 exports.persistObservable = persistObservable;
 exports.persistState = persistState;
 exports.transformObject = transformObject;