async-reactivity 2.0.28 → 2.0.29

package/lib/ref.js

@@ -3,14 +3,14 @@ export default class Ref {
     isEqual;
     dependents = new Set();
     _value;
-    constructor(_value, isEqual = (defaultIsEqual)) {
-        this._value = _value;
+    constructor(value, isEqual = (defaultIsEqual)) {
+        this._value = value;
         this.isEqual = isEqual;
     }
-    set value(_value) {
+    set value(value) {
         const oldValue = this._value;
-        this._value = _value;
-        if (!this.isEqual(_value, oldValue)) {
+        this._value = value;
+        if (this.dependents.size > 0 && !this.isEqual(value, oldValue)) {
             this.invalidate();
         }
     }

package/lib/ref.test.js

@@ -11,11 +11,15 @@ describe('ref', function () {
         a.value = 4;
         assert.strictEqual(a.value, 4);
     });
-    it('setter isEqual', function () {
+    it('should not invalidate when isEqual returns true', function () {
         const a = new Ref(5, () => true);
         new Watcher(a, () => {
             assert.fail();
         }, false);
         a.value = 4;
     });
+    it('should not call isEqual when there are no dependents', function () {
+        const a = new Ref(5, () => assert.fail());
+        a.value = 4;
+    });
 });

package/lib/tracker.js

@@ -0,0 +1,18 @@
+export default class Tracker {
+    dependents = new Set();
+    _value;
+    addDependent(dependent) {
+        this.dependents.add(dependent);
+    }
+    removeDependent(dependent) {
+        this.dependents.delete(dependent);
+    }
+    invalidate() {
+        for (const dependent of [...this.dependents.keys()]) {
+            dependent.invalidate(this);
+        }
+    }
+    get value() {
+        return this._value;
+    }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "async-reactivity",
-  "version": "2.0.28",
+  "version": "2.0.29",
   "description": "",
   "main": "lib/index.js",
   "types": "types/index.d.ts",

package/src/ref.test.ts

@@ -14,11 +14,16 @@ describe('ref', function () {
         assert.strictEqual(a.value, 4);
     });
 
-    it('setter isEqual', function () {
+    it('should not invalidate when isEqual returns true', function () {
         const a = new Ref(5, () => true);
         new Watcher(a, () => {
             assert.fail();
         }, false);
         a.value = 4;
     });
+
+    it('should not call isEqual when there are no dependents', function () {
+        const a = new Ref(5, () => assert.fail());
+        a.value = 4;
+    });
 });

package/src/ref.ts

@@ -7,15 +7,15 @@ export default class Ref<T> implements Dependency<T> {
     protected dependents = new Set<Dependent>();
     protected _value?: T;
 
-    constructor(_value: T, isEqual = defaultIsEqual<T>) {
-        this._value = _value;
+    constructor(value: T, isEqual = defaultIsEqual<T>) {
+        this._value = value;
         this.isEqual = isEqual;
     }
 
-    public set value(_value: T) {
+    public set value(value: T) {
         const oldValue = this._value!;
-        this._value = _value;
-        if (!this.isEqual(_value, oldValue)) {
+        this._value = value;
+        if (this.dependents.size > 0 && !this.isEqual(value, oldValue)) {
             this.invalidate();
         }
     }

package/types/ref.d.ts

@@ -4,8 +4,8 @@ export default class Ref<T> implements Dependency<T> {
     private isEqual;
     protected dependents: Set<Dependent>;
     protected _value?: T;
-    constructor(_value: T, isEqual?: (v1: T, v2: T) => boolean);
-    set value(_value: T);
+    constructor(value: T, isEqual?: (v1: T, v2: T) => boolean);
+    set value(value: T);
     get value(): T;
     addDependent(dependent: Dependent): void;
     removeDependent(dependent: Dependent): void;

package/types/ref.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ref.d.ts","sourceRoot":"","sources":["../src/ref.ts"],"names":[],"mappings":"AAAA,OAAO,UAAU,MAAM,iBAAiB,CAAC;AAEzC,OAAO,EAAE,SAAS,EAAE,MAAM,YAAY,CAAC;AAEvC,MAAM,CAAC,OAAO,OAAO,GAAG,CAAC,CAAC,CAAE,YAAW,UAAU,CAAC,CAAC,CAAC;IAChD,OAAO,CAAC,OAAO,CAA2B;IAC1C,SAAS,CAAC,UAAU,iBAAwB;IAC5C,SAAS,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;gBAET,MAAM,EAAE,CAAC,EAAE,OAAO,4BAAoB;IAKlD,IAAW,KAAK,CAAC,MAAM,EAAE,CAAC,EAMzB;IAED,IAAW,KAAK,IAAI,CAAC,CAEpB;IAEM,YAAY,CAAC,SAAS,EAAE,SAAS;IAIjC,eAAe,CAAC,SAAS,EAAE,SAAS;IAIpC,UAAU,IAAI,IAAI;CAK5B"}
+{"version":3,"file":"ref.d.ts","sourceRoot":"","sources":["../src/ref.ts"],"names":[],"mappings":"AAAA,OAAO,UAAU,MAAM,iBAAiB,CAAC;AAEzC,OAAO,EAAE,SAAS,EAAE,MAAM,YAAY,CAAC;AAEvC,MAAM,CAAC,OAAO,OAAO,GAAG,CAAC,CAAC,CAAE,YAAW,UAAU,CAAC,CAAC,CAAC;IAChD,OAAO,CAAC,OAAO,CAA2B;IAC1C,SAAS,CAAC,UAAU,iBAAwB;IAC5C,SAAS,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;gBAET,KAAK,EAAE,CAAC,EAAE,OAAO,4BAAoB;IAKjD,IAAW,KAAK,CAAC,KAAK,EAAE,CAAC,EAMxB;IAED,IAAW,KAAK,IAAI,CAAC,CAEpB;IAEM,YAAY,CAAC,SAAS,EAAE,SAAS;IAIjC,eAAe,CAAC,SAAS,EAAE,SAAS;IAIpC,UAAU,IAAI,IAAI;CAK5B"}

package/types/tracker.d.ts

@@ -0,0 +1,10 @@
+import Dependent from "./dependent.js";
+export default class Tracker<T> {
+    protected dependents: Set<Dependent>;
+    protected _value?: T;
+    addDependent(dependent: Dependent): void;
+    removeDependent(dependent: Dependent): void;
+    invalidate(): void;
+    get value(): T | undefined;
+}
+//# sourceMappingURL=tracker.d.ts.map

package/types/tracker.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"tracker.d.ts","sourceRoot":"","sources":["../src/tracker.ts"],"names":[],"mappings":"AAAA,OAAO,SAAS,MAAM,gBAAgB,CAAC;AAEvC,MAAM,CAAC,OAAO,OAAO,OAAO,CAAC,CAAC;IAC1B,SAAS,CAAC,UAAU,iBAAwB;IAC5C,SAAS,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;IAEd,YAAY,CAAC,SAAS,EAAE,SAAS;IAIjC,eAAe,CAAC,SAAS,EAAE,SAAS;IAIpC,UAAU,IAAI,IAAI;IAMzB,IAAW,KAAK,kBAEf;CACJ"}