mobx 6.3.3 → 6.3.7

package/dist/mobx.esm.js

@@ -673,7 +673,7 @@ function assertActionDescriptor(adm, _ref, key, _ref2) {
 
 function createActionDescriptor(adm, annotation, key, descriptor, // provides ability to disable safeDescriptors for prototypes
 safeDescriptors) {
-  var _annotation$options_, _annotation$options_$, _annotation$options_2, _annotation$options_$2, _annotation$options_3;
+  var _annotation$options_, _annotation$options_$, _annotation$options_2, _annotation$options_$2, _annotation$options_3, _annotation$options_4, _adm$proxy_2;
 
   if (safeDescriptors === void 0) {
     safeDescriptors = globalState.safeDescriptors;
@@ -689,7 +689,8 @@ safeDescriptors) {
   }
 
   return {
-    value: createAction((_annotation$options_$ = (_annotation$options_2 = annotation.options_) == null ? void 0 : _annotation$options_2.name) != null ? _annotation$options_$ : key.toString(), value, (_annotation$options_$2 = (_annotation$options_3 = annotation.options_) == null ? void 0 : _annotation$options_3.autoAction) != null ? _annotation$options_$2 : false),
+    value: createAction((_annotation$options_$ = (_annotation$options_2 = annotation.options_) == null ? void 0 : _annotation$options_2.name) != null ? _annotation$options_$ : key.toString(), value, (_annotation$options_$2 = (_annotation$options_3 = annotation.options_) == null ? void 0 : _annotation$options_3.autoAction) != null ? _annotation$options_$2 : false, // https://github.com/mobxjs/mobx/discussions/3140
+    ((_annotation$options_4 = annotation.options_) == null ? void 0 : _annotation$options_4.bound) ? (_adm$proxy_2 = adm.proxy_) != null ? _adm$proxy_2 : adm.target_ : undefined),
     // Non-configurable for classes
     // prevents accidental field redefinition in subclass
     configurable: safeDescriptors ? adm.isPlainObject_ : true,
@@ -1536,22 +1537,21 @@ var ComputedValue = /*#__PURE__*/function () {
     /* see #1208 */
     this.dependenciesState_ === IDerivationState_.NOT_TRACKING_;
     var newValue = this.computeValue_(true);
-
-    if (process.env.NODE_ENV !== "production" && isSpyEnabled()) {
-      spyReport({
-        observableKind: "computed",
-        debugObjectName: this.name_,
-        object: this.scope_,
-        type: "update",
-        oldValue: this.value_,
-        newValue: newValue
-      });
-    }
-
     var changed = wasSuspended || isCaughtException(oldValue) || isCaughtException(newValue) || !this.equals_(oldValue, newValue);
 
     if (changed) {
       this.value_ = newValue;
+
+      if (process.env.NODE_ENV !== "production" && isSpyEnabled()) {
+        spyReport({
+          observableKind: "computed",
+          debugObjectName: this.name_,
+          object: this.scope_,
+          type: "update",
+          oldValue: oldValue,
+          newValue: newValue
+        });
+      }
     }
 
     return changed;
@@ -1975,6 +1975,8 @@ var globalState = /*#__PURE__*/function () {
   if (global.__mobxGlobals && global.__mobxGlobals.version !== new MobXGlobals().version) canMergeGlobalState = false;
 
   if (!canMergeGlobalState) {
+    // Because this is a IIFE we need to let isolateCalled a chance to change
+    // so we run it after the event loop completed at least 1 iteration
     setTimeout(function () {
       if (!isolateCalled) {
         die(35);
@@ -2222,8 +2224,7 @@ function printDepTree(tree, lines, depth) {
     return;
   }
 
-  lines.push("" + new Array(depth).join("\t") + tree.name); // MWE: not the fastest, but the easiest way :)
-
+  lines.push("" + "\t".repeat(depth - 1) + tree.name);
   if (tree.dependencies) tree.dependencies.forEach(function (child) {
     return printDepTree(child, lines, depth + 1);
   });
@@ -2660,8 +2661,7 @@ function reaction(expression, effect, opts) {
   var firstTime = true;
   var isScheduled = false;
   var value;
-  var oldValue = undefined; // only an issue with fireImmediately
-
+  var oldValue;
   var equals = opts.compareStructural ? comparer.structural : opts.equals || comparer["default"];
   var r = new Reaction(name, function () {
     if (firstTime || runSync) {
@@ -2777,7 +2777,7 @@ function extendObservable(target, properties, annotations, options) {
     if (arguments.length > 4) die("'extendObservable' expected 2-4 arguments");
     if (typeof target !== "object") die("'extendObservable' expects an object as first argument");
     if (isObservableMap(target)) die("'extendObservable' should not be used on maps, use map.merge instead");
-    if (!isPlainObject(properties)) die("'extendObservabe' only accepts plain objects as second argument");
+    if (!isPlainObject(properties)) die("'extendObservable' only accepts plain objects as second argument");
     if (isObservable(properties) || isObservable(annotations)) die("Extending an object with another observable (object) is not supported");
   } // Pull descriptors first, so we don't have to deal with props added by administration ($mobx)
 
@@ -2968,14 +2968,14 @@ function interceptProperty(thing, property, handler) {
 }
 
 function _isComputed(value, property) {
-  if (property !== undefined) {
-    if (isObservableObject(value) === false) return false;
-    if (!value[$mobx].values_.has(property)) return false;
-    var atom = getAtom(value, property);
-    return isComputedValue(atom);
+  if (property === undefined) {
+    return isComputedValue(value);
   }
 
-  return isComputedValue(value);
+  if (isObservableObject(value) === false) return false;
+  if (!value[$mobx].values_.has(property)) return false;
+  var atom = getAtom(value, property);
+  return isComputedValue(atom);
 }
 function isComputed(value) {
   if (process.env.NODE_ENV !== "production" && arguments.length > 1) return die("isComputed expects only 1 argument. Use isComputedProp to inspect the observability of a property");
@@ -3335,7 +3335,7 @@ function whenPromise(predicate, opts) {
 
     cancel = function cancel() {
       disposer();
-      reject("WHEN_CANCELLED");
+      reject(new Error("WHEN_CANCELLED"));
     };
   });
   res.cancel = cancel;
@@ -3466,7 +3466,11 @@ function makeObservable(target, annotations, options) {
   try {
     var _annotations;
 
-    // Default to decorators
+    if (process.env.NODE_ENV !== "production" && annotations && target[storedAnnotationsSymbol]) {
+      die("makeObservable second arg must be nullish when using decorators. Mixing @decorator syntax with annotations is not supported.");
+    } // Default to decorators
+
+
     (_annotations = annotations) != null ? _annotations : annotations = collectStoredAnnotations(target); // Annotate
 
     ownKeys(annotations).forEach(function (key) {
@@ -3627,7 +3631,7 @@ var ObservableArrayAdministration = /*#__PURE__*/function () {
   };
 
   _proto.setArrayLength_ = function setArrayLength_(newLength) {
-    if (typeof newLength !== "number" || newLength < 0) die("Out of range: " + newLength);
+    if (typeof newLength !== "number" || isNaN(newLength) || newLength < 0) die("Out of range: " + newLength);
     var currentLength = this.values_.length;
     if (newLength === currentLength) return;else if (newLength > currentLength) {
       var newItems = new Array(newLength - currentLength);
@@ -4116,9 +4120,11 @@ var ObservableMap = /*#__PURE__*/function () {
 
       if (process.env.NODE_ENV !== "production" && notifySpy) spyReportStart(_change);
       transaction(function () {
+        var _this2$hasMap_$get;
+
         _this2.keysAtom_.reportChanged();
 
-        _this2.updateHasMapEntry_(key, false);
+        (_this2$hasMap_$get = _this2.hasMap_.get(key)) == null ? void 0 : _this2$hasMap_$get.setNewValue_(false);
 
         var observable = _this2.data_.get(key);
 
@@ -4134,14 +4140,6 @@ var ObservableMap = /*#__PURE__*/function () {
     return false;
   };
 
-  _proto.updateHasMapEntry_ = function updateHasMapEntry_(key, value) {
-    var entry = this.hasMap_.get(key);
-
-    if (entry) {
-      entry.setNewValue_(value);
-    }
-  };
-
   _proto.updateValue_ = function updateValue_(key, newValue) {
     var observable = this.data_.get(key);
     newValue = observable.prepareNewValue_(newValue);
@@ -4170,13 +4168,15 @@ var ObservableMap = /*#__PURE__*/function () {
 
     checkIfStateModificationsAreAllowed(this.keysAtom_);
     transaction(function () {
+      var _this3$hasMap_$get;
+
       var observable = new ObservableValue(newValue, _this3.enhancer_, process.env.NODE_ENV !== "production" ? _this3.name_ + "." + stringifyKey(key) : "ObservableMap.key", false);
 
       _this3.data_.set(key, observable);
 
       newValue = observable.value_; // value might have been changed
 
-      _this3.updateHasMapEntry_(key, true);
+      (_this3$hasMap_$get = _this3.hasMap_.get(key)) == null ? void 0 : _this3$hasMap_$get.setNewValue_(true);
 
       _this3.keysAtom_.reportChanged();
     });