@data-weave/datamanager-mobx 0.6.3 → 0.6.4

package/dist/ObservableList.d.ts

@@ -1,4 +1,15 @@
 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,16 +2,31 @@
 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);
     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/ObservableReference.d.ts

@@ -1,4 +1,16 @@
 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,23 +2,27 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ObservableReference = void 0;
 const mobx_1 = require("mobx");
-const PATCHED = Symbol('@data-weave/observableReference.patched');
+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 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;
+    // 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();
-            atom.reportChanged();
+            for (const a of atoms)
+                a.reportChanged();
         };
     }
+    bridged[observingAtoms].push(atom);
     return new Proxy(sourceReference, {
         get(target, prop, receiver) {
             if (prop === 'value' || prop === 'resolved' || prop === 'hasError' || prop === 'error') {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@data-weave/datamanager-mobx",
-    "version": "0.6.3",
+    "version": "0.6.4",
     "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.6.4",
+        "@data-weave/datamanager": "~0.6.4"
     },
     "peerDependencies": {
         "mobx": ">=6.0.0"
     },
     "devDependencies": {
-        "@data-weave/backend-firestore": "~0.6.3",
-        "@data-weave/datamanager": "~0.6.3",
+        "@data-weave/backend-firestore": "~0.6.4",
+        "@data-weave/datamanager": "~0.6.4",
         "mobx": "6.15.0",
         "typescript": "5.9.3"
     },