mobx 7.0.5 → 7.0.6

package/dist/mobx.esm.development.js

@@ -1191,14 +1191,13 @@ class ObservableValue extends Atom {
     this.equals_ = equals_;
     this.value_ = enhancer_(value, undefined, name_);
     if (notifySpy && isSpyEnabled()) {
-      var _this$value_;
       // only notify spy if this is a stand-alone observable
       spyReport({
         type: CREATE,
         object: this,
         observableKind: "value",
         debugObjectName: this.name_,
-        newValue: "" + ((_this$value_ = this.value_) == null ? void 0 : _this$value_.toString())
+        newValue: this.value_
       });
     }
   }
@@ -2161,6 +2160,7 @@ class Reaction {
     this.runId_ = 0;
     this.unboundDepsCount_ = 0;
     this.flags_ = 0b00000;
+    this.removeAbortListener_ = undefined;
     this.name_ = name_;
     this.onInvalidate_ = onInvalidate_;
     this.errorHandler_ = errorHandler_;
@@ -2297,7 +2297,10 @@ class Reaction {
   }
   dispose() {
     if (!this.isDisposed) {
+      var _this$removeAbortList;
       this.isDisposed = true;
+      // also covers self-disposal (`r.dispose()`, `when`), which bypasses the disposer
+      (_this$removeAbortList = this.removeAbortListener_) == null || _this$removeAbortList.call(this);
       if (!this.isRunning) {
         // if disposed while running, clean up later. Maybe not optimal, but rare case
         startBatch();
@@ -2307,11 +2310,11 @@ class Reaction {
     }
   }
   getDisposer_(abortSignal) {
-    const dispose = () => {
-      this.dispose();
-      abortSignal == null || abortSignal.removeEventListener == null || abortSignal.removeEventListener("abort", dispose);
-    };
-    abortSignal == null || abortSignal.addEventListener == null || abortSignal.addEventListener("abort", dispose);
+    const dispose = () => this.dispose();
+    if (abortSignal != null && abortSignal.addEventListener && !this.isDisposed) {
+      abortSignal.addEventListener("abort", dispose);
+      this.removeAbortListener_ = () => abortSignal.removeEventListener == null ? void 0 : abortSignal.removeEventListener("abort", dispose);
+    }
     dispose[$mobx] = this;
     if ("dispose" in Symbol && typeof Symbol.dispose === "symbol") {
       dispose[Symbol.dispose] = dispose;
@@ -3468,8 +3471,7 @@ const arrayTraps = {
     const adm = target[$mobx];
     if (name === "length") {
       adm.setArrayLength_(value);
-    }
-    if (typeof name === "symbol" || isNaN(name)) {
+    } else if (typeof name === "symbol" || isNaN(name)) {
       target[name] = value;
     } else {
       // numeric string
@@ -3514,7 +3516,7 @@ class ObservableArrayAdministration {
     return this.values_.length;
   }
   setArrayLength_(newLength) {
-    if (typeof newLength !== "number" || isNaN(newLength) || newLength < 0) {
+    if (!Number.isInteger(newLength) || newLength < 0) {
       die(40, newLength);
     }
     let currentLength = this.values_.length;
@@ -4296,7 +4298,8 @@ class ObservableSet {
     }
     if (!this.has(value)) {
       transaction(() => {
-        this.data_.add(this.enhancer_(value, undefined));
+        value = this.enhancer_(value, undefined); // report the stored value, like map and array
+        this.data_.add(value);
         this.atom_.reportChanged();
       });
       const notifySpy = isSpyEnabled();