mobx 7.0.5 → 7.0.6

package/dist/mobx.umd.development.js

@@ -1197,14 +1197,13 @@
         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_
           });
         }
       }
@@ -2167,6 +2166,7 @@
         this.runId_ = 0;
         this.unboundDepsCount_ = 0;
         this.flags_ = 0b00000;
+        this.removeAbortListener_ = undefined;
         this.name_ = name_;
         this.onInvalidate_ = onInvalidate_;
         this.errorHandler_ = errorHandler_;
@@ -2303,7 +2303,10 @@
       }
       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();
@@ -2313,11 +2316,11 @@
         }
       }
       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;
@@ -3474,8 +3477,7 @@
         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
@@ -3520,7 +3522,7 @@
         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;
@@ -4302,7 +4304,8 @@
         }
         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();