r-state-tree 0.6.1 → 0.6.2

package/dist/observables/object.d.ts

@@ -7,7 +7,7 @@ export declare class ObjectAdministration<T extends object> extends Administrati
     hasMap: AtomMap<PropertyKey>;
     valuesMap: SignalMap<PropertyKey>;
     computedMap: Map<PropertyKey, ComputedNode<T[keyof T]>>;
-    types: Map<PropertyKey, PropertyType | null>;
+    types: Map<PropertyKey, PropertyType>;
     isWriting: boolean;
     static proxyTraps: ProxyHandler<object>;
     constructor(source?: T);

package/dist/r-state-tree.cjs

@@ -72,14 +72,7 @@ function getPropertyType(key, obj) {
   if (descriptor?.value && typeof descriptor.value === "function") {
     return "action";
   }
-  const isAccessor = descriptor && (typeof descriptor.get === "function" || typeof descriptor.set === "function");
   if (!hasMetadata) {
-    if (descriptor?.get) {
-      return "computed";
-    }
-    if (isAccessor) {
-      return null;
-    }
     return "observable";
   }
   const metadata = obj.constructor[Symbol.metadata];
@@ -96,7 +89,7 @@ function getPropertyType(key, obj) {
         return "observable";
     }
   }
-  return isAccessor ? null : "observable";
+  return "observable";
 }
 function getPropertyDescriptor$1(obj, key) {
   let node = obj;
@@ -358,55 +351,42 @@ class ObjectAdministration extends Administration {
   }
   read(key, receiver = this.proxy) {
     const type = this.getType(key);
-    if (type === null) {
+    if (type === "computed") {
+      if (receiver === this.proxy) {
+        return this.callComputed(key);
+      }
+      this.atom.reportObserved();
       return Reflect.get(this.source, key, receiver);
     }
-    switch (type) {
-      case "observable":
-      case "action": {
-        if (key in this.source) {
-          this.valuesMap.reportObserved(key, this.source[key]);
-        }
-        this.atom.reportObserved();
-        if (this.atom.observing) {
-          this.hasMap.reportObserved(key);
-        }
-        if (type === "observable") {
-          const value = Reflect.get(this.source, key, receiver);
-          const shouldWrap = this.explicitObservables.has(key) || isObservable(value);
-          if (shouldWrap && value && typeof value === "object" && !Object.isFrozen(value)) {
-            const desc = getPropertyDescriptor$1(this.source, key);
-            if (desc && !desc.configurable && !desc.writable) {
-              return value;
-            }
-            const existingAdm = getAdministration(value);
-            if (existingAdm) {
-              return existingAdm.proxy;
-            }
-          }
+    if (key in this.source) {
+      this.valuesMap.reportObserved(key, this.source[key]);
+    }
+    this.atom.reportObserved();
+    if (this.atom.observing) {
+      this.hasMap.reportObserved(key);
+    }
+    if (type === "observable") {
+      const value = Reflect.get(this.source, key, receiver);
+      const shouldWrap = this.explicitObservables.has(key) || isObservable(value);
+      if (shouldWrap && value && typeof value === "object" && !Object.isFrozen(value)) {
+        const desc = getPropertyDescriptor$1(this.source, key);
+        if (desc && !desc.configurable && !desc.writable) {
           return value;
         }
-        return getAction(
-          Reflect.get(this.source, key, receiver)
-        );
-      }
-      case "computed": {
-        if (receiver === this.proxy) {
-          return this.callComputed(key);
+        const existingAdm = getAdministration(value);
+        if (existingAdm) {
+          return existingAdm.proxy;
         }
-        this.atom.reportObserved();
-        return Reflect.get(this.source, key, receiver);
       }
-      default:
-        throw new Error(`unknown type passed to configure`);
+      return value;
     }
+    return getAction(
+      Reflect.get(this.source, key, receiver)
+    );
   }
   explicitObservables = /* @__PURE__ */ new Set();
   write(key, newValue) {
     const type = this.getType(key);
-    if (type === null) {
-      return this.set(key, newValue);
-    }
     if (type === "computed") {
       return signalsCore.batch(() => this.set(key, newValue));
     }
@@ -590,8 +570,24 @@ function createListener(callback) {
   };
   return listener;
 }
+function internalCreateComputed(fn) {
+  const version = signalsCore.signal(0);
+  const c = signalsCore.computed(
+    () => {
+      version.value;
+      return fn();
+    },
+    {
+      unwatched() {
+        version.value++;
+      }
+    }
+  );
+  return { c, version };
+}
 function createComputed(fn, context = null) {
-  const c = signalsCore.computed(context ? fn.bind(context) : fn);
+  const bound = context ? fn.bind(context) : fn;
+  const { c } = internalCreateComputed(bound);
   return {
     node: c,
     get() {
@@ -600,6 +596,7 @@ function createComputed(fn, context = null) {
     clear: () => {
       signalsCore.untracked(() => {
         c._value = void 0;
+        c._globalVersion = -1;
       });
     }
   };
@@ -615,7 +612,7 @@ function computed(value, context) {
     context.metadata[context.name] = { type: "computed" };
     return value;
   }
-  return signalsCore.computed(value);
+  return internalCreateComputed(value).c;
 }
 function source(obj) {
   return getSource(obj);

package/dist/r-state-tree.js

@@ -1,4 +1,4 @@
-import { batch, signal as signal$1, Signal, computed as computed$1, untracked, effect } from "@preact/signals-core";
+import { batch, signal as signal$1, Signal, untracked, effect, computed as computed$1 } from "@preact/signals-core";
 import { Signal as Signal2, batch as batch2, effect as effect2, untracked as untracked2 } from "@preact/signals-core";
 var lib = {};
 var hasRequiredLib;
@@ -71,14 +71,7 @@ function getPropertyType(key, obj) {
   if (descriptor?.value && typeof descriptor.value === "function") {
     return "action";
   }
-  const isAccessor = descriptor && (typeof descriptor.get === "function" || typeof descriptor.set === "function");
   if (!hasMetadata) {
-    if (descriptor?.get) {
-      return "computed";
-    }
-    if (isAccessor) {
-      return null;
-    }
     return "observable";
   }
   const metadata = obj.constructor[Symbol.metadata];
@@ -95,7 +88,7 @@ function getPropertyType(key, obj) {
         return "observable";
     }
   }
-  return isAccessor ? null : "observable";
+  return "observable";
 }
 function getPropertyDescriptor$1(obj, key) {
   let node = obj;
@@ -357,55 +350,42 @@ class ObjectAdministration extends Administration {
   }
   read(key, receiver = this.proxy) {
     const type = this.getType(key);
-    if (type === null) {
+    if (type === "computed") {
+      if (receiver === this.proxy) {
+        return this.callComputed(key);
+      }
+      this.atom.reportObserved();
       return Reflect.get(this.source, key, receiver);
     }
-    switch (type) {
-      case "observable":
-      case "action": {
-        if (key in this.source) {
-          this.valuesMap.reportObserved(key, this.source[key]);
-        }
-        this.atom.reportObserved();
-        if (this.atom.observing) {
-          this.hasMap.reportObserved(key);
-        }
-        if (type === "observable") {
-          const value = Reflect.get(this.source, key, receiver);
-          const shouldWrap = this.explicitObservables.has(key) || isObservable(value);
-          if (shouldWrap && value && typeof value === "object" && !Object.isFrozen(value)) {
-            const desc = getPropertyDescriptor$1(this.source, key);
-            if (desc && !desc.configurable && !desc.writable) {
-              return value;
-            }
-            const existingAdm = getAdministration(value);
-            if (existingAdm) {
-              return existingAdm.proxy;
-            }
-          }
+    if (key in this.source) {
+      this.valuesMap.reportObserved(key, this.source[key]);
+    }
+    this.atom.reportObserved();
+    if (this.atom.observing) {
+      this.hasMap.reportObserved(key);
+    }
+    if (type === "observable") {
+      const value = Reflect.get(this.source, key, receiver);
+      const shouldWrap = this.explicitObservables.has(key) || isObservable(value);
+      if (shouldWrap && value && typeof value === "object" && !Object.isFrozen(value)) {
+        const desc = getPropertyDescriptor$1(this.source, key);
+        if (desc && !desc.configurable && !desc.writable) {
           return value;
         }
-        return getAction(
-          Reflect.get(this.source, key, receiver)
-        );
-      }
-      case "computed": {
-        if (receiver === this.proxy) {
-          return this.callComputed(key);
+        const existingAdm = getAdministration(value);
+        if (existingAdm) {
+          return existingAdm.proxy;
         }
-        this.atom.reportObserved();
-        return Reflect.get(this.source, key, receiver);
       }
-      default:
-        throw new Error(`unknown type passed to configure`);
+      return value;
     }
+    return getAction(
+      Reflect.get(this.source, key, receiver)
+    );
   }
   explicitObservables = /* @__PURE__ */ new Set();
   write(key, newValue) {
     const type = this.getType(key);
-    if (type === null) {
-      return this.set(key, newValue);
-    }
     if (type === "computed") {
       return batch(() => this.set(key, newValue));
     }
@@ -589,8 +569,24 @@ function createListener(callback) {
   };
   return listener;
 }
+function internalCreateComputed(fn) {
+  const version = signal$1(0);
+  const c = computed$1(
+    () => {
+      version.value;
+      return fn();
+    },
+    {
+      unwatched() {
+        version.value++;
+      }
+    }
+  );
+  return { c, version };
+}
 function createComputed(fn, context = null) {
-  const c = computed$1(context ? fn.bind(context) : fn);
+  const bound = context ? fn.bind(context) : fn;
+  const { c } = internalCreateComputed(bound);
   return {
     node: c,
     get() {
@@ -599,6 +595,7 @@ function createComputed(fn, context = null) {
     clear: () => {
       untracked(() => {
         c._value = void 0;
+        c._globalVersion = -1;
       });
     }
   };
@@ -614,7 +611,7 @@ function computed(value, context) {
     context.metadata[context.name] = { type: "computed" };
     return value;
   }
-  return computed$1(value);
+  return internalCreateComputed(value).c;
 }
 function source(obj) {
   return getSource(obj);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "r-state-tree",
-  "version": "0.6.1",
+  "version": "0.6.2",
   "description": "reactive state management library",
   "main": "dist/r-state-tree.cjs",
   "module": "dist/r-state-tree.js",