@data-weave/datamanager-mobx 0.6.4 → 0.7.1

package/dist/ObservableList.d.ts

@@ -1,15 +1,3 @@
 import { LiveList } from '@data-weave/datamanager';
-/**
- * Wraps a `LiveList<T>` so that MobX reactions observing `values`,
- * `resolved`, `hasError`, or `error` re-run whenever the underlying source
- * publishes a new snapshot. The atom drives `resolve` / `unsubscribe` via
- * `onBecomeObserved` / `onBecomeUnobserved`, and a one-time bridge installed
- * on the bare `onValuesChange` method propagates source updates to every
- * wrapping atom (since `FirestoreList` and other `LiveList` subclasses call
- * `this.onValuesChange()` on the bare instance, bypassing this proxy's get
- * trap). Multiple wraps of the same source are supported: each wrap registers
- * its own atom and all of them are notified on every snapshot.
- */
-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

@@ -2,26 +2,11 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ObservableList = void 0;
 const mobx_1 = require("mobx");
-const observingAtoms = Symbol('@data-weave/observableList.observingAtoms');
 const ObservableList = (sourceList) => {
     const atom = (0, mobx_1.createAtom)('ObservableList', () => sourceList.resolve(), () => sourceList.unsubscribe());
-    // FirestoreList (and any LiveList) calls `this.onValuesChange()` on the
-    // bare instance from inside its snapshot callback (via `onUpdate` /
-    // `onUpdateAll` / `onError`). The Proxy's get trap never sees that call,
-    // so we install (once per source) a bridge on the bare method that fans
-    // each call out to every registered atom.
-    const bridged = sourceList;
-    if (!bridged[observingAtoms]) {
-        const atoms = [];
-        bridged[observingAtoms] = atoms;
-        const original = sourceList.onValuesChange.bind(sourceList);
-        sourceList.onValuesChange = () => {
-            original();
-            for (const a of atoms)
-                a.reportChanged();
-        };
-    }
-    bridged[observingAtoms].push(atom);
+    sourceList.registerOnChangeListener(() => {
+        atom.reportChanged();
+    });
     return new Proxy(sourceList, {
         get(target, prop, receiver) {
             if (prop === 'values' || prop === 'resolved' || prop === 'hasError' || prop === 'error') {

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,16 +1,3 @@
 import { LiveReference } from '@data-weave/datamanager';
-/**
- * Wraps a `LiveReference<T>` so that MobX reactions observing `value`,
- * `resolved`, `hasError`, or `error` re-run whenever the underlying source
- * publishes a new value. The atom drives `resolve` / `unsubscribe` via
- * `onBecomeObserved` / `onBecomeUnobserved`, and a one-time bridge installed
- * on the bare `onValueChange` method propagates source updates to every
- * wrapping atom (since `FirestoreReference` and other `LiveReference`
- * subclasses call `this.onValueChange()` on the bare instance, bypassing
- * this proxy's get trap). Multiple wraps of the same source are supported:
- * each wrap registers its own atom and all of them are notified on every
- * publish.
- */
-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,27 +2,11 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ObservableReference = void 0;
 const mobx_1 = require("mobx");
-const observingAtoms = Symbol('@data-weave/observableReference.observingAtoms');
 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 install (once per source) a
-    // bridge on the bare method that fans each call out to every
-    // registered atom.
-    const bridged = sourceReference;
-    if (!bridged[observingAtoms]) {
-        const atoms = [];
-        bridged[observingAtoms] = atoms;
-        const original = sourceReference.onValueChange.bind(sourceReference);
-        sourceReference.onValueChange = () => {
-            original();
-            for (const a of atoms)
-                a.reportChanged();
-        };
-    }
-    bridged[observingAtoms].push(atom);
+    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.4",
+    "version": "0.7.1",
     "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.4",
-        "@data-weave/datamanager": "~0.6.4"
+        "@data-weave/backend-firestore": "~0.7.1",
+        "@data-weave/datamanager": "~0.7.1"
     },
     "peerDependencies": {
         "mobx": ">=6.0.0"
     },
     "devDependencies": {
-        "@data-weave/backend-firestore": "~0.6.4",
-        "@data-weave/datamanager": "~0.6.4",
+        "@data-weave/backend-firestore": "~0.7.1",
+        "@data-weave/datamanager": "~0.7.1",
         "mobx": "6.15.0",
         "typescript": "5.9.3"
     },