async-reactivity 2.0.20 → 2.0.22

package/lib/computed.js

@@ -9,7 +9,6 @@ var ComputedState;
     ComputedState[ComputedState["Uncertain"] = 2] = "Uncertain";
     ComputedState[ComputedState["Computing"] = 3] = "Computing";
 })(ComputedState || (ComputedState = {}));
-;
 class CircularDependencyError extends Error {
 }
 export default class Computed extends Tracker {
@@ -21,6 +20,7 @@ export default class Computed extends Tracker {
     computePromiseActions;
     lastComputeAttemptPromise;
     deferrer;
+    abortController;
     constructor(getter, isEqual = (defaultIsEqual), timeToLive) {
         super();
         this.getter = getter;
@@ -70,9 +70,13 @@ export default class Computed extends Tracker {
                 return this._value;
             }
         }
+        else if (this.state === ComputedState.Computing) {
+            this.abortController?.abort();
+        }
         this.state = ComputedState.Computing;
         this.clearDependencies(true);
-        const newValue = this.getter(this.trackDependency, this._value);
+        this.abortController = new AbortController();
+        const newValue = this.getter(this.trackDependency, this._value, this.abortController.signal);
         if (this.isEqual(newValue, this._value)) {
             this.handlePromiseThen(this.lastComputeAttemptPromise, this._value);
             this.validateDependents();
@@ -120,6 +124,7 @@ export default class Computed extends Tracker {
     finalizeComputing() {
         this.state = ComputedState.Valid;
         this.lastComputeAttemptPromise = undefined;
+        this.abortController = undefined;
         this.prepareComputePromise();
     }
     invalidate() {

package/lib/computed.test.js

@@ -186,6 +186,23 @@ describe('async computed', function () {
         a.value = 8;
         assert.strictEqual(await b.value, 13);
     });
+    it('dependency changed while computing - abortSignal', async function () {
+        const a = new Ref(5);
+        let gate = 0;
+        const b = new Computed(async (value, _previousValue, abortSignal) => {
+            const dep = value(a);
+            await new Promise(resolve => setTimeout(resolve, 10));
+            if (abortSignal.aborted) {
+                throw new Error();
+            }
+            gate++;
+            return dep + 5;
+        });
+        b.value; // trigger compute
+        a.value = 8;
+        await b.value;
+        assert.strictEqual(gate, 1);
+    });
     it('old dependency changed while computing', async function () {
         let gate = 0;
         const a = new Ref(5);

package/package.json

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

package/src/computed.test.ts

@@ -219,6 +219,24 @@ describe('async computed', function () {
         assert.strictEqual(await b.value, 13);
     });
 
+    it('dependency changed while computing - abortSignal', async function () {
+        const a = new Ref(5);
+        let gate = 0;
+        const b = new Computed(async (value, _previousValue, abortSignal) => {
+            const dep = value(a);
+            await new Promise(resolve => setTimeout(resolve, 10));
+            if (abortSignal.aborted) {
+                throw new Error();
+            }
+            gate++;
+            return dep + 5;
+        });
+        b.value;            // trigger compute
+        a.value = 8;
+        await b.value;
+        assert.strictEqual(gate, 1);
+    });
+
     it('old dependency changed while computing', async function () {
         let gate = 0;
         const a = new Ref(5);

package/src/computed.ts

@@ -6,29 +6,30 @@ import { Deferrer } from "./deferrer.js";
 import { debounce } from 'lodash-es';
 
 export declare type TrackValue = <T>(dependency: Dependency<T>) => T;
-export declare type ComputeFunc<T> = (value: TrackValue, previousValue?: T) => T;
-export declare type ComputeFuncScoped<T1, T2> = (value: TrackValue, scope: T1, previousValue?: T2) => T2;
+export declare type ComputeFunc<T> = (value: TrackValue, previousValue: T | undefined, abortSignal: AbortSignal) => T;
+export declare type ComputeFuncScoped<T1, T2> = (value: TrackValue, scope: T1, previousValue: T2 | undefined, abortSignal: AbortSignal) => T2;
 
 enum ComputedState {
     Invalid,
     Valid,
     Uncertain,
     Computing
-};
+}
 
 class CircularDependencyError extends Error { }
 
-export default class Computed<T extends TBase, TBase = T> extends Tracker<T> implements Dependent, Dependency<T> {
+export default class Computed<T> extends Tracker<T> implements Dependent, Dependency<T> {
     getter: ComputeFunc<T>;
-    isEqual: typeof defaultIsEqual<TBase>;
+    isEqual: typeof defaultIsEqual<T>;
     private state = ComputedState.Invalid;
     private dependencies = new Map<Dependency<any>, boolean>();
     private computePromise?: Promise<any>;
     private computePromiseActions?: { resolve: Function, reject: Function };
     private lastComputeAttemptPromise?: Promise<void>;
     private deferrer?: Deferrer;
+    private abortController?: AbortController;
 
-    constructor(getter: ComputeFunc<T>, isEqual = defaultIsEqual<TBase>, timeToLive?: number) {
+    constructor(getter: ComputeFunc<T>, isEqual = defaultIsEqual<T>, timeToLive?: number) {
         super();
         this.getter = getter;
         this.isEqual = isEqual;
@@ -80,12 +81,15 @@ export default class Computed<T extends TBase, TBase = T> extends Tracker<T> imp
                 this.validateDependents();
                 return this._value!;
             }
+        } else if (this.state === ComputedState.Computing) {
+            this.abortController?.abort();
         }
 
         this.state = ComputedState.Computing;
         this.clearDependencies(true);
+        this.abortController = new AbortController();
 
-        const newValue: T = this.getter(this.trackDependency, this._value);
+        const newValue: T = this.getter(this.trackDependency, this._value, this.abortController.signal);
         if (this.isEqual(newValue, this._value!)) {
             this.handlePromiseThen(this.lastComputeAttemptPromise!, this._value);
             this.validateDependents();
@@ -123,7 +127,7 @@ export default class Computed<T extends TBase, TBase = T> extends Tracker<T> imp
         }
     }
 
-    private innerTrackDependency(this: Computed<T, TBase>, dependency: Dependency<any>) {
+    private innerTrackDependency(this: Computed<T>, dependency: Dependency<any>) {
         if (this.dependents.has(dependency as any)) {
             throw new CircularDependencyError();
         }
@@ -137,6 +141,7 @@ export default class Computed<T extends TBase, TBase = T> extends Tracker<T> imp
     private finalizeComputing() {
         this.state = ComputedState.Valid;
         this.lastComputeAttemptPromise = undefined;
+        this.abortController = undefined;
         this.prepareComputePromise();
     }
 

package/src/listener.ts

@@ -3,13 +3,13 @@ import Ref from "./ref.js";
 import defaultIsEqual from "./defaultIsEqual.js";
 import { Deferrer } from "./deferrer.js";
 
-export default class Listener<T extends TBase, TBase = T> extends Ref<T> {
+export default class Listener<T> extends Ref<T> {
     private start: () => void;
     private stop: () => void;
     private listening = false;
     private deferrer: Deferrer;
 
-    constructor(_value: T, start: () => void, stop: () => void, isEqual = defaultIsEqual<TBase>) {
+    constructor(_value: T, start: () => void, stop: () => void, isEqual = defaultIsEqual<T>) {
         super(_value, isEqual);
 
         this.start = start;

package/src/ref.ts

@@ -2,10 +2,10 @@ import Dependency from "./dependency.js";
 import Tracker from "./tracker.js";
 import defaultIsEqual from "./defaultIsEqual.js";
 
-export default class Ref<T extends TBase, TBase = T> extends Tracker<T> implements Dependency<T> {
-    private isEqual: typeof defaultIsEqual<TBase>;
+export default class Ref<T> extends Tracker<T> implements Dependency<T> {
+    private isEqual: typeof defaultIsEqual<T>;
 
-    constructor(_value: T, isEqual = defaultIsEqual<TBase>) {
+    constructor(_value: T, isEqual = defaultIsEqual<T>) {
         super();
         this._value = _value;
         this.isEqual = isEqual;

package/types/computed.d.ts

@@ -3,18 +3,19 @@ import Dependent from "./dependent.js";
 import Tracker from "./tracker.js";
 import defaultIsEqual from "./defaultIsEqual.js";
 export declare type TrackValue = <T>(dependency: Dependency<T>) => T;
-export declare type ComputeFunc<T> = (value: TrackValue, previousValue?: T) => T;
-export declare type ComputeFuncScoped<T1, T2> = (value: TrackValue, scope: T1, previousValue?: T2) => T2;
-export default class Computed<T extends TBase, TBase = T> extends Tracker<T> implements Dependent, Dependency<T> {
+export declare type ComputeFunc<T> = (value: TrackValue, previousValue: T | undefined, abortSignal: AbortSignal) => T;
+export declare type ComputeFuncScoped<T1, T2> = (value: TrackValue, scope: T1, previousValue: T2 | undefined, abortSignal: AbortSignal) => T2;
+export default class Computed<T> extends Tracker<T> implements Dependent, Dependency<T> {
     getter: ComputeFunc<T>;
-    isEqual: typeof defaultIsEqual<TBase>;
+    isEqual: typeof defaultIsEqual<T>;
     private state;
     private dependencies;
     private computePromise?;
     private computePromiseActions?;
     private lastComputeAttemptPromise?;
     private deferrer?;
-    constructor(getter: ComputeFunc<T>, isEqual?: (v1: TBase, v2: TBase) => boolean, timeToLive?: number);
+    private abortController?;
+    constructor(getter: ComputeFunc<T>, isEqual?: (v1: T, v2: T) => boolean, timeToLive?: number);
     private prepareComputePromise;
     get value(): T;
     private compute;

package/types/computed.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"computed.d.ts","sourceRoot":"","sources":["../src/computed.ts"],"names":[],"mappings":"AAAA,OAAO,UAAU,MAAM,iBAAiB,CAAC;AACzC,OAAO,SAAS,MAAM,gBAAgB,CAAC;AACvC,OAAO,OAAO,MAAM,cAAc,CAAC;AACnC,OAAO,cAAc,MAAM,qBAAqB,CAAC;AAIjD,MAAM,CAAC,OAAO,MAAM,UAAU,GAAG,CAAC,CAAC,EAAE,UAAU,EAAE,UAAU,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC;AACrE,MAAM,CAAC,OAAO,MAAM,WAAW,CAAC,CAAC,IAAI,CAAC,KAAK,EAAE,UAAU,EAAE,aAAa,CAAC,EAAE,CAAC,KAAK,CAAC,CAAC;AACjF,MAAM,CAAC,OAAO,MAAM,iBAAiB,CAAC,EAAE,EAAE,EAAE,IAAI,CAAC,KAAK,EAAE,UAAU,EAAE,KAAK,EAAE,EAAE,EAAE,aAAa,CAAC,EAAE,EAAE,KAAK,EAAE,CAAC;AAWzG,MAAM,CAAC,OAAO,OAAO,QAAQ,CAAC,CAAC,SAAS,KAAK,EAAE,KAAK,GAAG,CAAC,CAAE,SAAQ,OAAO,CAAC,CAAC,CAAE,YAAW,SAAS,EAAE,UAAU,CAAC,CAAC,CAAC;IAC5G,MAAM,EAAE,WAAW,CAAC,CAAC,CAAC,CAAC;IACvB,OAAO,EAAE,OAAO,cAAc,CAAC,KAAK,CAAC,CAAC;IACtC,OAAO,CAAC,KAAK,CAAyB;IACtC,OAAO,CAAC,YAAY,CAAuC;IAC3D,OAAO,CAAC,cAAc,CAAC,CAAe;IACtC,OAAO,CAAC,qBAAqB,CAAC,CAA0C;IACxE,OAAO,CAAC,yBAAyB,CAAC,CAAgB;IAClD,OAAO,CAAC,QAAQ,CAAC,CAAW;gBAEhB,MAAM,EAAE,WAAW,CAAC,CAAC,CAAC,EAAE,OAAO,oCAAwB,EAAE,UAAU,CAAC,EAAE,MAAM;IAexF,OAAO,CAAC,qBAAqB;IAS7B,IAAW,KAAK,IAAI,CAAC,CAMpB;IAED,OAAO,CAAC,OAAO;IA0Cf,OAAO,CAAC,iBAAiB;IAOzB,OAAO,CAAC,iBAAiB;IAOzB,OAAO,CAAC,kBAAkB;IAO1B,OAAO,CAAC,oBAAoB;IAS5B,OAAO,CAAC,eAAe,CAAwC;IAE/D,OAAO,CAAC,iBAAiB;IAMlB,UAAU;IAUV,eAAe;IAKf,QAAQ,CAAC,UAAU,EAAE,UAAU,CAAC,GAAG,CAAC;IAI3C,OAAO,CAAC,kBAAkB;IAMnB,eAAe,CAAC,SAAS,EAAE,SAAS,EAAE,OAAO,gBAAoB,GAAG,IAAI;IAKxE,KAAK;IAQL,OAAO;CAGjB"}
+{"version":3,"file":"computed.d.ts","sourceRoot":"","sources":["../src/computed.ts"],"names":[],"mappings":"AAAA,OAAO,UAAU,MAAM,iBAAiB,CAAC;AACzC,OAAO,SAAS,MAAM,gBAAgB,CAAC;AACvC,OAAO,OAAO,MAAM,cAAc,CAAC;AACnC,OAAO,cAAc,MAAM,qBAAqB,CAAC;AAIjD,MAAM,CAAC,OAAO,MAAM,UAAU,GAAG,CAAC,CAAC,EAAE,UAAU,EAAE,UAAU,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC;AACrE,MAAM,CAAC,OAAO,MAAM,WAAW,CAAC,CAAC,IAAI,CAAC,KAAK,EAAE,UAAU,EAAE,aAAa,EAAE,CAAC,GAAG,SAAS,EAAE,WAAW,EAAE,WAAW,KAAK,CAAC,CAAC;AACtH,MAAM,CAAC,OAAO,MAAM,iBAAiB,CAAC,EAAE,EAAE,EAAE,IAAI,CAAC,KAAK,EAAE,UAAU,EAAE,KAAK,EAAE,EAAE,EAAE,aAAa,EAAE,EAAE,GAAG,SAAS,EAAE,WAAW,EAAE,WAAW,KAAK,EAAE,CAAC;AAW9I,MAAM,CAAC,OAAO,OAAO,QAAQ,CAAC,CAAC,CAAE,SAAQ,OAAO,CAAC,CAAC,CAAE,YAAW,SAAS,EAAE,UAAU,CAAC,CAAC,CAAC;IACnF,MAAM,EAAE,WAAW,CAAC,CAAC,CAAC,CAAC;IACvB,OAAO,EAAE,OAAO,cAAc,CAAC,CAAC,CAAC,CAAC;IAClC,OAAO,CAAC,KAAK,CAAyB;IACtC,OAAO,CAAC,YAAY,CAAuC;IAC3D,OAAO,CAAC,cAAc,CAAC,CAAe;IACtC,OAAO,CAAC,qBAAqB,CAAC,CAA0C;IACxE,OAAO,CAAC,yBAAyB,CAAC,CAAgB;IAClD,OAAO,CAAC,QAAQ,CAAC,CAAW;IAC5B,OAAO,CAAC,eAAe,CAAC,CAAkB;gBAE9B,MAAM,EAAE,WAAW,CAAC,CAAC,CAAC,EAAE,OAAO,4BAAoB,EAAE,UAAU,CAAC,EAAE,MAAM;IAepF,OAAO,CAAC,qBAAqB;IAS7B,IAAW,KAAK,IAAI,CAAC,CAMpB;IAED,OAAO,CAAC,OAAO;IA6Cf,OAAO,CAAC,iBAAiB;IAOzB,OAAO,CAAC,iBAAiB;IAOzB,OAAO,CAAC,kBAAkB;IAO1B,OAAO,CAAC,oBAAoB;IAS5B,OAAO,CAAC,eAAe,CAAwC;IAE/D,OAAO,CAAC,iBAAiB;IAOlB,UAAU;IAUV,eAAe;IAKf,QAAQ,CAAC,UAAU,EAAE,UAAU,CAAC,GAAG,CAAC;IAI3C,OAAO,CAAC,kBAAkB;IAMnB,eAAe,CAAC,SAAS,EAAE,SAAS,EAAE,OAAO,gBAAoB,GAAG,IAAI;IAKxE,KAAK;IAQL,OAAO;CAGjB"}

package/types/listener.d.ts

@@ -1,11 +1,11 @@
 import Dependent from "./dependent.js";
 import Ref from "./ref.js";
-export default class Listener<T extends TBase, TBase = T> extends Ref<T> {
+export default class Listener<T> extends Ref<T> {
     private start;
     private stop;
     private listening;
     private deferrer;
-    constructor(_value: T, start: () => void, stop: () => void, isEqual?: (v1: TBase, v2: TBase) => boolean);
+    constructor(_value: T, start: () => void, stop: () => void, isEqual?: (v1: T, v2: T) => boolean);
     addDependent(dependent: Dependent): void;
     removeDependent(dependent: Dependent, promise?: Promise<void>): void;
 }

package/types/listener.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"listener.d.ts","sourceRoot":"","sources":["../src/listener.ts"],"names":[],"mappings":"AAAA,OAAO,SAAS,MAAM,gBAAgB,CAAC;AACvC,OAAO,GAAG,MAAM,UAAU,CAAC;AAI3B,MAAM,CAAC,OAAO,OAAO,QAAQ,CAAC,CAAC,SAAS,KAAK,EAAE,KAAK,GAAG,CAAC,CAAE,SAAQ,GAAG,CAAC,CAAC,CAAC;IACpE,OAAO,CAAC,KAAK,CAAa;IAC1B,OAAO,CAAC,IAAI,CAAa;IACzB,OAAO,CAAC,SAAS,CAAS;IAC1B,OAAO,CAAC,QAAQ,CAAW;gBAEf,MAAM,EAAE,CAAC,EAAE,KAAK,EAAE,MAAM,IAAI,EAAE,IAAI,EAAE,MAAM,IAAI,EAAE,OAAO,oCAAwB;IAcpF,YAAY,CAAC,SAAS,EAAE,SAAS,GAAG,IAAI;IAQxC,eAAe,CAAC,SAAS,EAAE,SAAS,EAAE,OAAO,gBAAoB,GAAG,IAAI;CAIlF"}
+{"version":3,"file":"listener.d.ts","sourceRoot":"","sources":["../src/listener.ts"],"names":[],"mappings":"AAAA,OAAO,SAAS,MAAM,gBAAgB,CAAC;AACvC,OAAO,GAAG,MAAM,UAAU,CAAC;AAI3B,MAAM,CAAC,OAAO,OAAO,QAAQ,CAAC,CAAC,CAAE,SAAQ,GAAG,CAAC,CAAC,CAAC;IAC3C,OAAO,CAAC,KAAK,CAAa;IAC1B,OAAO,CAAC,IAAI,CAAa;IACzB,OAAO,CAAC,SAAS,CAAS;IAC1B,OAAO,CAAC,QAAQ,CAAW;gBAEf,MAAM,EAAE,CAAC,EAAE,KAAK,EAAE,MAAM,IAAI,EAAE,IAAI,EAAE,MAAM,IAAI,EAAE,OAAO,4BAAoB;IAchF,YAAY,CAAC,SAAS,EAAE,SAAS,GAAG,IAAI;IAQxC,eAAe,CAAC,SAAS,EAAE,SAAS,EAAE,OAAO,gBAAoB,GAAG,IAAI;CAIlF"}

package/types/ref.d.ts

@@ -1,8 +1,8 @@
 import Dependency from "./dependency.js";
 import Tracker from "./tracker.js";
-export default class Ref<T extends TBase, TBase = T> extends Tracker<T> implements Dependency<T> {
+export default class Ref<T> extends Tracker<T> implements Dependency<T> {
     private isEqual;
-    constructor(_value: T, isEqual?: (v1: TBase, v2: TBase) => boolean);
+    constructor(_value: T, isEqual?: (v1: T, v2: T) => boolean);
     set value(_value: T);
     get value(): T;
 }

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;AACzC,OAAO,OAAO,MAAM,cAAc,CAAC;AAGnC,MAAM,CAAC,OAAO,OAAO,GAAG,CAAC,CAAC,SAAS,KAAK,EAAE,KAAK,GAAG,CAAC,CAAE,SAAQ,OAAO,CAAC,CAAC,CAAE,YAAW,UAAU,CAAC,CAAC,CAAC;IAC5F,OAAO,CAAC,OAAO,CAA+B;gBAElC,MAAM,EAAE,CAAC,EAAE,OAAO,oCAAwB;IAMtD,IAAW,KAAK,CAAC,MAAM,EAAE,CAAC,EAMzB;IAED,IAAW,KAAK,IAAI,CAAC,CAEpB;CACJ"}
+{"version":3,"file":"ref.d.ts","sourceRoot":"","sources":["../src/ref.ts"],"names":[],"mappings":"AAAA,OAAO,UAAU,MAAM,iBAAiB,CAAC;AACzC,OAAO,OAAO,MAAM,cAAc,CAAC;AAGnC,MAAM,CAAC,OAAO,OAAO,GAAG,CAAC,CAAC,CAAE,SAAQ,OAAO,CAAC,CAAC,CAAE,YAAW,UAAU,CAAC,CAAC,CAAC;IACnE,OAAO,CAAC,OAAO,CAA2B;gBAE9B,MAAM,EAAE,CAAC,EAAE,OAAO,4BAAoB;IAMlD,IAAW,KAAK,CAAC,MAAM,EAAE,CAAC,EAMzB;IAED,IAAW,KAAK,IAAI,CAAC,CAEpB;CACJ"}