mobx 6.2.0 → 6.3.3

package/dist/mobx.umd.development.js

@@ -9,6 +9,18 @@
       1: function _(annotationType, key) {
         return "Cannot apply '" + annotationType + "' to '" + key.toString() + "': Field not found.";
       },
+
+      /*
+      2(prop) {
+          return `invalid decorator for '${prop.toString()}'`
+      },
+      3(prop) {
+          return `Cannot decorate '${prop.toString()}': action can only be used on properties with a function value.`
+      },
+      4(prop) {
+          return `Cannot decorate '${prop.toString()}': computed can only be used on getter properties.`
+      },
+      */
       5: "'keys()' can only be used on observable objects, arrays, sets and maps",
       6: "'values()' can only be used on observable objects, arrays, sets and maps",
       7: "'entries()' can only be used on observable objects, arrays and maps",
@@ -65,7 +77,9 @@
       36: "isolateGlobalState should be called before MobX is running any reactions",
       37: function _(method) {
         return "[mobx] `observableArray." + method + "()` mutates the array in-place, which is not allowed inside a derivation. Use `array.slice()." + method + "()` instead";
-      }
+      },
+      38: "'ownKeys()' can only be used on observable objects",
+      39: "'defineProperty()' can only be used on observable objects"
     };
     var errors =  niceErrors ;
     function die(error) {
@@ -502,7 +516,8 @@
     }
 
     function defaultComparer(a, b) {
-      return Object.is(a, b);
+      if (Object.is) return Object.is(a, b);
+      return a === b ? a !== 0 || 1 / a === 1 / b : a !== a && b !== b;
     }
 
     var comparer = {
@@ -700,6 +715,8 @@
     }
 
     function make_$2(adm, key, descriptor, source) {
+      var _this$options_;
+
       // own
       if (source === adm.target_) {
         return this.extend_(adm, key, descriptor, false) === null ? 0
@@ -708,8 +725,15 @@
         /* Continue */
         ;
       } // prototype
+      // bound - must annotate protos to support super.flow()
 
 
+      if (((_this$options_ = this.options_) == null ? void 0 : _this$options_.bound) && !isFlow(adm.target_[key])) {
+        if (this.extend_(adm, key, descriptor, false) === null) return 0
+        /* Cancel */
+        ;
+      }
+
       if (isFlow(descriptor.value)) {
         // A prototype could have been annotated already by other constructor,
         // rest of the proto chain must be annotated already
@@ -718,7 +742,7 @@
         ;
       }
 
-      var flowDescriptor = createFlowDescriptor(adm, this, key, descriptor, false);
+      var flowDescriptor = createFlowDescriptor(adm, this, key, descriptor, false, false);
       defineProperty(source, key, flowDescriptor);
       return 2
       /* Continue */
@@ -726,7 +750,9 @@
     }
 
     function extend_$2(adm, key, descriptor, proxyTrap) {
-      var flowDescriptor = createFlowDescriptor(adm, this, key, descriptor);
+      var _this$options_2;
+
+      var flowDescriptor = createFlowDescriptor(adm, this, key, descriptor, (_this$options_2 = this.options_) == null ? void 0 : _this$options_2.bound);
       return adm.defineProperty_(key, flowDescriptor, proxyTrap);
     }
 
@@ -739,15 +765,23 @@
       }
     }
 
-    function createFlowDescriptor(adm, annotation, key, descriptor, // provides ability to disable safeDescriptors for prototypes
+    function createFlowDescriptor(adm, annotation, key, descriptor, bound, // provides ability to disable safeDescriptors for prototypes
     safeDescriptors) {
       if (safeDescriptors === void 0) {
         safeDescriptors = globalState.safeDescriptors;
       }
 
       assertFlowDescriptor(adm, annotation, key, descriptor);
+      var value = descriptor.value;
+
+      if (bound) {
+        var _adm$proxy_;
+
+        value = value.bind((_adm$proxy_ = adm.proxy_) != null ? _adm$proxy_ : adm.target_);
+      }
+
       return {
-        value: flow(descriptor.value),
+        value: flow(value),
         // Non-configurable for classes
         // prevents accidental field redefinition in subclass
         configurable: safeDescriptors ? adm.isPlainObject_ : true,
@@ -837,7 +871,7 @@
     }
 
     function make_$5(adm, key, descriptor, source) {
-      var _this$options_2, _this$options_3;
+      var _this$options_3, _this$options_4;
 
       // getter -> computed
       if (descriptor.get) {
@@ -872,22 +906,25 @@
 
 
       if (source !== adm.target_ && typeof descriptor.value === "function") {
-        var _this$options_;
+        var _this$options_2;
 
         if (isGenerator(descriptor.value)) {
-          return flow.make_(adm, key, descriptor, source);
+          var _this$options_;
+
+          var flowAnnotation = ((_this$options_ = this.options_) == null ? void 0 : _this$options_.autoBind) ? flow.bound : flow;
+          return flowAnnotation.make_(adm, key, descriptor, source);
         }
 
-        var actionAnnotation = ((_this$options_ = this.options_) == null ? void 0 : _this$options_.autoBind) ? autoAction.bound : autoAction;
+        var actionAnnotation = ((_this$options_2 = this.options_) == null ? void 0 : _this$options_2.autoBind) ? autoAction.bound : autoAction;
         return actionAnnotation.make_(adm, key, descriptor, source);
       } // other -> observable
       // Copy props from proto as well, see test:
       // "decorate should work with Object.create"
 
 
-      var observableAnnotation = ((_this$options_2 = this.options_) == null ? void 0 : _this$options_2.deep) === false ? observable.ref : observable; // if function respect autoBind option
+      var observableAnnotation = ((_this$options_3 = this.options_) == null ? void 0 : _this$options_3.deep) === false ? observable.ref : observable; // if function respect autoBind option
 
-      if (typeof descriptor.value === "function" && ((_this$options_3 = this.options_) == null ? void 0 : _this$options_3.autoBind)) {
+      if (typeof descriptor.value === "function" && ((_this$options_4 = this.options_) == null ? void 0 : _this$options_4.autoBind)) {
         var _adm$proxy_;
 
         descriptor.value = descriptor.value.bind((_adm$proxy_ = adm.proxy_) != null ? _adm$proxy_ : adm.target_);
@@ -897,7 +934,7 @@
     }
 
     function extend_$5(adm, key, descriptor, proxyTrap) {
-      var _this$options_4, _this$options_5;
+      var _this$options_5, _this$options_6;
 
       // getter -> computed
       if (descriptor.get) {
@@ -915,16 +952,20 @@
       // if function respect autoBind option
 
 
-      if (typeof descriptor.value === "function" && ((_this$options_4 = this.options_) == null ? void 0 : _this$options_4.autoBind)) {
+      if (typeof descriptor.value === "function" && ((_this$options_5 = this.options_) == null ? void 0 : _this$options_5.autoBind)) {
         var _adm$proxy_2;
 
         descriptor.value = descriptor.value.bind((_adm$proxy_2 = adm.proxy_) != null ? _adm$proxy_2 : adm.target_);
       }
 
-      var observableAnnotation = ((_this$options_5 = this.options_) == null ? void 0 : _this$options_5.deep) === false ? observable.ref : observable;
+      var observableAnnotation = ((_this$options_6 = this.options_) == null ? void 0 : _this$options_6.deep) === false ? observable.ref : observable;
       return observableAnnotation.extend_(adm, key, descriptor, proxyTrap);
     }
 
+    var OBSERVABLE = "observable";
+    var OBSERVABLE_REF = "observable.ref";
+    var OBSERVABLE_SHALLOW = "observable.shallow";
+    var OBSERVABLE_STRUCT = "observable.struct"; // Predefined bags of create observable options, to avoid allocating temporarily option objects
     // in the majority of cases
 
     var defaultCreateObservableOptions = {
@@ -937,14 +978,14 @@
     function asCreateObservableOptions(thing) {
       return thing || defaultCreateObservableOptions;
     }
-    var observableAnnotation = /*#__PURE__*/createObservableAnnotation("observable");
-    var observableRefAnnotation = /*#__PURE__*/createObservableAnnotation("observable.ref", {
+    var observableAnnotation = /*#__PURE__*/createObservableAnnotation(OBSERVABLE);
+    var observableRefAnnotation = /*#__PURE__*/createObservableAnnotation(OBSERVABLE_REF, {
       enhancer: referenceEnhancer
     });
-    var observableShallowAnnotation = /*#__PURE__*/createObservableAnnotation("observable.shallow", {
+    var observableShallowAnnotation = /*#__PURE__*/createObservableAnnotation(OBSERVABLE_SHALLOW, {
       enhancer: shallowEnhancer
     });
-    var observableStructAnnotation = /*#__PURE__*/createObservableAnnotation("observable.struct", {
+    var observableStructAnnotation = /*#__PURE__*/createObservableAnnotation(OBSERVABLE_STRUCT, {
       enhancer: refStructEnhancer
     });
     var observableDecoratorAnnotation = /*#__PURE__*/createDecoratorAnnotation(observableAnnotation);
@@ -1549,6 +1590,10 @@
         if (!this.keepAlive_) {
           clearObserving(this);
           this.value_ = undefined; // don't hold on to computed value!
+
+          if ( this.isTracing_ !== TraceMode.NONE) {
+            console.log("[mobx.trace] Computed value '" + this.name_ + "' was suspended and it will recompute on the next access.");
+          }
         }
       };
 
@@ -1581,16 +1626,12 @@
 
       _proto.warnAboutUntrackedRead_ = function warnAboutUntrackedRead_() {
 
-        if (this.requiresReaction_ === true) {
-          die("[mobx] Computed value " + this.name_ + " is read outside a reactive context");
-        }
-
         if (this.isTracing_ !== TraceMode.NONE) {
-          console.log("[mobx.trace] '" + this.name_ + "' is being read outside a reactive context. Doing a full recompute");
+          console.log("[mobx.trace] Computed value '" + this.name_ + "' is being read outside a reactive context. Doing a full recompute.");
         }
 
-        if (globalState.computedRequiresReaction) {
-          console.warn("[mobx] Computed value " + this.name_ + " is being read outside a reactive context. Doing a full recompute");
+        if (globalState.computedRequiresReaction || this.requiresReaction_) {
+          console.warn("[mobx] Computed value '" + this.name_ + "' is being read outside a reactive context. Doing a full recompute.");
         }
       };
 
@@ -1723,7 +1764,7 @@
     }
     function checkIfStateReadsAreAllowed(observable) {
       if ( !globalState.allowStateReads && globalState.observableRequiresReaction) {
-        console.warn("[mobx] Observable " + observable.name_ + " being read outside a reactive context");
+        console.warn("[mobx] Observable '" + observable.name_ + "' being read outside a reactive context.");
       }
     }
     /**
@@ -1767,7 +1808,7 @@
       if (derivation.observing_.length !== 0) return;
 
       if (globalState.reactionRequiresObservable || derivation.requiresObservable_) {
-        console.warn("[mobx] Derivation " + derivation.name_ + " is created/updated without reading any observable value");
+        console.warn("[mobx] Derivation '" + derivation.name_ + "' is created/updated without reading any observable value.");
       }
     }
     /**
@@ -2787,6 +2828,9 @@
       return error instanceof FlowCancellationError;
     }
     var flowAnnotation = /*#__PURE__*/createFlowAnnotation("flow");
+    var flowBoundAnnotation = /*#__PURE__*/createFlowAnnotation("flow.bound", {
+      bound: true
+    });
     var flow = /*#__PURE__*/Object.assign(function flow(arg1, arg2) {
       // @flow
       if (isStringish(arg2)) {
@@ -2872,6 +2916,7 @@
       res.isMobXFlow = true;
       return res;
     }, flowAnnotation);
+    flow.bound = /*#__PURE__*/createDecoratorAnnotation(flowBoundAnnotation);
 
     function cancelPromise(promise) {
       if (isFunction(promise.cancel)) promise.cancel();
@@ -3096,6 +3141,20 @@
 
       die(11);
     }
+    function apiDefineProperty(obj, key, descriptor) {
+      if (isObservableObject(obj)) {
+        return obj[$mobx].defineProperty_(key, descriptor);
+      }
+
+      die(39);
+    }
+    function apiOwnKeys(obj) {
+      if (isObservableObject(obj)) {
+        return obj[$mobx].ownKeys_();
+      }
+
+      die(38);
+    }
 
     function observe(thing, propOrCb, cbOrFire, fireImmediately) {
       if (isFunction(cbOrFire)) return observeObservableProperty(thing, propOrCb, cbOrFire, fireImmediately);else return observeObservable(thing, propOrCb, cbOrFire);
@@ -3116,7 +3175,7 @@
 
     function toJSHelper(source, __alreadySeen) {
       if (source == null || typeof source !== "object" || source instanceof Date || !isObservable(source)) return source;
-      if (isObservableValue(source)) return toJSHelper(source.get(), __alreadySeen);
+      if (isObservableValue(source) || isComputedValue(source)) return toJSHelper(source.get(), __alreadySeen);
 
       if (__alreadySeen.has(source)) {
         return __alreadySeen.get(source);
@@ -3150,7 +3209,7 @@
         // must be observable object
         var _res3 = cache(__alreadySeen, source, {});
 
-        source[$mobx].ownKeys_().forEach(function (key) {
+        apiOwnKeys(source).forEach(function (key) {
           if (objectPrototype.propertyIsEnumerable.call(source, key)) {
             _res3[key] = toJSHelper(source[key], __alreadySeen);
           }
@@ -3233,10 +3292,10 @@
       var timeoutHandle;
 
       if (typeof opts.timeout === "number") {
+        var error = new Error("WHEN_TIMEOUT");
         timeoutHandle = setTimeout(function () {
           if (!disposer[$mobx].isDisposed_) {
             disposer();
-            var error = new Error("WHEN_TIMEOUT");
             if (opts.onError) opts.onError(error);else throw error;
           }
         }, opts.timeout);
@@ -3323,7 +3382,7 @@
         return (_getAdm$definePropert = getAdm(target).defineProperty_(name, descriptor)) != null ? _getAdm$definePropert : true;
       },
       ownKeys: function ownKeys(target) {
-        if ( globalState.trackingDerivation) warnAboutProxyRequirement("iterate keys to detect added / removed properties. Use `keys` from 'mobx' instead.");
+        if ( globalState.trackingDerivation) warnAboutProxyRequirement("iterate keys to detect added / removed properties. Use 'keys' from 'mobx' instead.");
         return getAdm(target).ownKeys_();
       },
       preventExtensions: function preventExtensions(target) {
@@ -5686,7 +5745,7 @@
      * - utils/   Utility stuff.
      *
      */
-    ["Symbol", "Map", "Set", "Symbol"].forEach(function (m) {
+    ["Symbol", "Map", "Set"].forEach(function (m) {
       var g = getGlobal();
 
       if (typeof g[m] === "undefined") {
@@ -5728,6 +5787,7 @@
     exports.computed = computed;
     exports.configure = configure;
     exports.createAtom = createAtom;
+    exports.defineProperty = apiDefineProperty;
     exports.entries = entries;
     exports.extendObservable = extendObservable;
     exports.flow = flow;
@@ -5760,6 +5820,7 @@
     exports.onBecomeUnobserved = onBecomeUnobserved;
     exports.onReactionError = onReactionError;
     exports.override = override;
+    exports.ownKeys = apiOwnKeys;
     exports.reaction = reaction;
     exports.remove = remove;
     exports.runInAction = runInAction;