@data-weave/datamanager-mobx 0.6.3 → 0.7.0

package/dist/ObservableList.d.ts

@@ -1,4 +1,3 @@
 import { LiveList } from '@data-weave/datamanager';
-export type ObservableList<T> = LiveList<T>;
 export declare const ObservableList: <T>(sourceList: LiveList<T>) => LiveList<T>;
 //# sourceMappingURL=ObservableList.d.ts.map

package/dist/ObservableList.js

@@ -4,14 +4,14 @@ exports.ObservableList = void 0;
 const mobx_1 = require("mobx");
 const ObservableList = (sourceList) => {
     const atom = (0, mobx_1.createAtom)('ObservableList', () => sourceList.resolve(), () => sourceList.unsubscribe());
+    sourceList.registerOnChangeListener(() => {
+        atom.reportChanged();
+    });
     return new Proxy(sourceList, {
         get(target, prop, receiver) {
             if (prop === 'values' || prop === 'resolved' || prop === 'hasError' || prop === 'error') {
                 atom.reportObserved();
             }
-            if (prop === 'onValuesChange') {
-                atom.reportChanged();
-            }
             return Reflect.get(target, prop, receiver);
         },
     });

package/dist/ObservableOptimisticReference.d.ts

@@ -1,12 +1,3 @@
 import { LiveReference, OptimisticReference } from '@data-weave/datamanager';
-export type ObservableOptimisticReference<T extends object> = OptimisticReference<T>;
-/**
- * Wraps a `LiveReference<T>` with optimistic patch support and MobX
- * reactivity. Source-value reactivity is provided by `ObservableReference`;
- * a separate atom owned by this factory tracks `_patch` mutations made via
- * `applyOptimistic` / `clearOptimistic` so that MobX reactions observing
- * `value` (or `hasPendingOptimistic`) re-run when the optimistic overlay
- * changes.
- */
-export declare const ObservableOptimisticReference: <T extends object>(source: LiveReference<T>) => OptimisticReference<T>;
+export declare const ObservableOptimisticReference: <T>(sourceReference: LiveReference<T>) => OptimisticReference<T>;
 //# sourceMappingURL=ObservableOptimisticReference.d.ts.map

package/dist/ObservableOptimisticReference.js

@@ -3,33 +3,16 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.ObservableOptimisticReference = void 0;
 const datamanager_1 = require("@data-weave/datamanager");
 const mobx_1 = require("mobx");
-const ObservableReference_1 = require("./ObservableReference");
-/**
- * Wraps a `LiveReference<T>` with optimistic patch support and MobX
- * reactivity. Source-value reactivity is provided by `ObservableReference`;
- * a separate atom owned by this factory tracks `_patch` mutations made via
- * `applyOptimistic` / `clearOptimistic` so that MobX reactions observing
- * `value` (or `hasPendingOptimistic`) re-run when the optimistic overlay
- * changes.
- */
-const ObservableOptimisticReference = (source) => {
-    const observableSource = (0, ObservableReference_1.ObservableReference)(source);
-    const optimistic = new datamanager_1.OptimisticReference(observableSource);
-    const patchAtom = (0, mobx_1.createAtom)('ObservableOptimisticReference');
+const ObservableOptimisticReference = (sourceReference) => {
+    const optimistic = new datamanager_1.OptimisticReference(sourceReference);
+    const atom = (0, mobx_1.createAtom)('ObservableOptimisticLiveReference', () => sourceReference.resolve(), () => sourceReference.unsubscribe());
+    optimistic.registerOnChangeListener(() => atom.reportChanged());
     return new Proxy(optimistic, {
         get(target, prop, receiver) {
-            if (prop === 'value' || prop === 'hasPendingOptimistic') {
-                patchAtom.reportObserved();
+            if (prop === 'value' || prop === 'resolved' || prop === 'hasError' || prop === 'error') {
+                atom.reportObserved();
             }
-            const value = Reflect.get(target, prop, receiver);
-            if (prop === 'applyOptimistic' || prop === 'clearOptimistic') {
-                return (...args) => {
-                    const result = value.apply(target, args);
-                    patchAtom.reportChanged();
-                    return result;
-                };
-            }
-            return value;
+            return Reflect.get(target, prop, receiver);
         },
     });
 };

package/dist/ObservableReference.d.ts

@@ -1,4 +1,3 @@
 import { LiveReference } from '@data-weave/datamanager';
-export type ObservableReference<T> = LiveReference<T>;
 export declare const ObservableReference: <T>(sourceReference: LiveReference<T>) => LiveReference<T>;
 //# sourceMappingURL=ObservableReference.d.ts.map

package/dist/ObservableReference.js

@@ -2,23 +2,11 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ObservableReference = void 0;
 const mobx_1 = require("mobx");
-const PATCHED = Symbol('@data-weave/observableReference.patched');
 const ObservableReference = (sourceReference) => {
     const atom = (0, mobx_1.createAtom)('ObservableReference', () => sourceReference.resolve(), () => sourceReference.unsubscribe());
-    // FirestoreReference (and any LiveReference) calls `this.onUpdate(...)` on
-    // the bare instance from inside its snapshot callback, which then calls
-    // `this.onValueChange()` on the bare instance too. The Proxy's get trap
-    // never sees that call, so we patch the bare method to also notify the
-    // atom. Idempotent across multiple wraps via the PATCHED symbol guard.
-    const tagged = sourceReference;
-    if (!tagged[PATCHED]) {
-        tagged[PATCHED] = true;
-        const original = sourceReference.onValueChange.bind(sourceReference);
-        sourceReference.onValueChange = () => {
-            original();
-            atom.reportChanged();
-        };
-    }
+    sourceReference.registerOnChangeListener(() => {
+        atom.reportChanged();
+    });
     return new Proxy(sourceReference, {
         get(target, prop, receiver) {
             if (prop === 'value' || prop === 'resolved' || prop === 'hasError' || prop === 'error') {

package/dist/index.d.ts

@@ -1,6 +1,5 @@
 export * from '@data-weave/backend-firestore';
 export * from '@data-weave/datamanager';
 export * from './ObservableList';
-export * from './ObservableOptimisticReference';
 export * from './ObservableReference';
 //# sourceMappingURL=index.d.ts.map

package/dist/index.js

@@ -17,6 +17,5 @@ Object.defineProperty(exports, "__esModule", { value: true });
 __exportStar(require("@data-weave/backend-firestore"), exports);
 __exportStar(require("@data-weave/datamanager"), exports);
 __exportStar(require("./ObservableList"), exports);
-__exportStar(require("./ObservableOptimisticReference"), exports);
 __exportStar(require("./ObservableReference"), exports);
 //# sourceMappingURL=index.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@data-weave/datamanager-mobx",
-    "version": "0.6.3",
+    "version": "0.7.0",
     "author": "",
     "main": "dist/index.js",
     "types": "dist/index.d.ts",
@@ -23,15 +23,15 @@
         "registry": "https://registry.npmjs.org/"
     },
     "dependencies": {
-        "@data-weave/backend-firestore": "~0.6.3",
-        "@data-weave/datamanager": "~0.6.3"
+        "@data-weave/backend-firestore": "~0.7.0",
+        "@data-weave/datamanager": "~0.7.0"
     },
     "peerDependencies": {
         "mobx": ">=6.0.0"
     },
     "devDependencies": {
-        "@data-weave/backend-firestore": "~0.6.3",
-        "@data-weave/datamanager": "~0.6.3",
+        "@data-weave/backend-firestore": "~0.7.0",
+        "@data-weave/datamanager": "~0.7.0",
         "mobx": "6.15.0",
         "typescript": "5.9.3"
     },