mobx 3.2.2 → 3.3.0

package/lib/mobx.js

@@ -111,7 +111,7 @@ var Atom = (function (_super) {
 var isAtom = createInstanceofPredicate("Atom", BaseAtom);
 
 function hasInterceptors(interceptable) {
-    return (interceptable.interceptors && interceptable.interceptors.length > 0);
+    return interceptable.interceptors && interceptable.interceptors.length > 0;
 }
 function registerInterceptor(interceptable, handler) {
     var interceptors = interceptable.interceptors || (interceptable.interceptors = []);
@@ -223,7 +223,11 @@ var MAX_SPLICE_SIZE = 10000; // See e.g. https://github.com/mobxjs/mobx/issues/8
 var safariPrototypeSetterInheritanceBug = (function () {
     var v = false;
     var p = {};
-    Object.defineProperty(p, "0", { set: function () { v = true; } });
+    Object.defineProperty(p, "0", {
+        set: function () {
+            v = true;
+        }
+    });
     Object.create(p)["0"] = 1;
     return v === false;
 })();
@@ -251,6 +255,32 @@ function inherit(ctor, proto) {
     }
 }
 inherit(StubArray, Array.prototype);
+// Weex freeze Array.prototype
+// Make them writeable and configurable in prototype chain
+// https://github.com/alibaba/weex/pull/1529
+if (Object.isFrozen(Array)) {
+    
+    [
+        "constructor",
+        "push",
+        "shift",
+        "concat",
+        "pop",
+        "unshift",
+        "replace",
+        "find",
+        "findIndex",
+        "splice",
+        "reverse",
+        "sort"
+    ].forEach(function (key) {
+        Object.defineProperty(StubArray.prototype, key, {
+            configurable: true,
+            writable: true,
+            value: Array.prototype[key]
+        });
+    });
+}
 var ObservableArrayAdministration = (function () {
     function ObservableArrayAdministration(name, enhancer, array, owned) {
         this.array = array;
@@ -259,7 +289,7 @@ var ObservableArrayAdministration = (function () {
         this.lastKnownLength = 0;
         this.interceptors = null;
         this.changeListeners = null;
-        this.atom = new BaseAtom(name || ("ObservableArray@" + getNextId()));
+        this.atom = new BaseAtom(name || "ObservableArray@" + getNextId());
         this.enhancer = function (newV, oldV) { return enhancer(newV, oldV, name + "[..]"); };
     }
     ObservableArrayAdministration.prototype.dehanceValue = function (value) {
@@ -362,7 +392,9 @@ var ObservableArrayAdministration = (function () {
         }
         else {
             var res = this.values.slice(index, index + deleteCount);
-            this.values = this.values.slice(0, index).concat(newItems, this.values.slice(index + deleteCount));
+            this.values = this.values
+                .slice(0, index)
+                .concat(newItems, this.values.slice(index + deleteCount));
             return res;
         }
         var _a;
@@ -370,11 +402,15 @@ var ObservableArrayAdministration = (function () {
     ObservableArrayAdministration.prototype.notifyArrayChildUpdate = function (index, newValue, oldValue) {
         var notifySpy = !this.owned && isSpyEnabled();
         var notify = hasListeners(this);
-        var change = notify || notifySpy ? {
-            object: this.array,
-            type: "update",
-            index: index, newValue: newValue, oldValue: oldValue
-        } : null;
+        var change = notify || notifySpy
+            ? {
+                object: this.array,
+                type: "update",
+                index: index,
+                newValue: newValue,
+                oldValue: oldValue
+            }
+            : null;
         if (notifySpy)
             spyReportStart(change);
         this.atom.reportChanged();
@@ -386,13 +422,17 @@ var ObservableArrayAdministration = (function () {
     ObservableArrayAdministration.prototype.notifyArraySplice = function (index, added, removed) {
         var notifySpy = !this.owned && isSpyEnabled();
         var notify = hasListeners(this);
-        var change = notify || notifySpy ? {
-            object: this.array,
-            type: "splice",
-            index: index, removed: removed, added: added,
-            removedCount: removed.length,
-            addedCount: added.length
-        } : null;
+        var change = notify || notifySpy
+            ? {
+                object: this.array,
+                type: "splice",
+                index: index,
+                removed: removed,
+                added: added,
+                removedCount: removed.length,
+                addedCount: added.length
+            }
+            : null;
         if (notifySpy)
             spyReportStart(change);
         this.atom.reportChanged();
@@ -438,7 +478,7 @@ var ObservableArray = (function (_super) {
             arrays[_i] = arguments[_i];
         }
         this.$mobx.atom.reportObserved();
-        return Array.prototype.concat.apply(this.peek(), arrays.map(function (a) { return isObservableArray(a) ? a.peek() : a; }));
+        return Array.prototype.concat.apply(this.peek(), arrays.map(function (a) { return (isObservableArray(a) ? a.peek() : a); }));
     };
     ObservableArray.prototype.replace = function (newItems) {
         return this.$mobx.spliceWithArray(0, this.$mobx.values.length, newItems);
@@ -474,11 +514,11 @@ var ObservableArray = (function (_super) {
         return -1;
     };
     /*
-        functions that do alter the internal structure of the array, (based on lib.es6.d.ts)
-        since these functions alter the inner structure of the array, the have side effects.
-        Because the have side effects, they should not be used in computed function,
-        and for that reason the do not call dependencyState.notifyObserved
-        */
+     * functions that do alter the internal structure of the array, (based on lib.es6.d.ts)
+     * since these functions alter the inner structure of the array, the have side effects.
+     * Because the have side effects, they should not be used in computed function,
+     * and for that reason the do not call dependencyState.notifyObserved
+     */
     ObservableArray.prototype.splice = function (index, deleteCount) {
         var newItems = [];
         for (var _i = 2; _i < arguments.length; _i++) {
@@ -560,10 +600,15 @@ var ObservableArray = (function (_super) {
         var oldItems = this.$mobx.values;
         var newItems;
         if (fromIndex < toIndex) {
-            newItems = oldItems.slice(0, fromIndex).concat(oldItems.slice(fromIndex + 1, toIndex + 1), [oldItems[fromIndex]], oldItems.slice(toIndex + 1));
+            newItems = oldItems.slice(0, fromIndex).concat(oldItems.slice(fromIndex + 1, toIndex + 1), [
+                oldItems[fromIndex]
+            ], oldItems.slice(toIndex + 1));
         }
         else {
-            newItems = oldItems.slice(0, toIndex).concat([oldItems[fromIndex]], oldItems.slice(toIndex, fromIndex), oldItems.slice(fromIndex + 1));
+            // toIndex < fromIndex
+            newItems = oldItems.slice(0, toIndex).concat([
+                oldItems[fromIndex]
+            ], oldItems.slice(toIndex, fromIndex), oldItems.slice(fromIndex + 1));
         }
         this.replace(newItems);
     };
@@ -575,7 +620,9 @@ var ObservableArray = (function (_super) {
                 impl.atom.reportObserved();
                 return impl.dehanceValue(impl.values[index]);
             }
-            console.warn("[mobx.array] Attempt to read an array index (" + index + ") that is out of bounds (" + impl.values.length + "). Please check length first. Out of bound indices will not be tracked by MobX");
+            console.warn("[mobx.array] Attempt to read an array index (" + index + ") that is out of bounds (" + impl
+                .values
+                .length + "). Please check length first. Out of bound indices will not be tracked by MobX");
         }
         return undefined;
     };
@@ -591,7 +638,8 @@ var ObservableArray = (function (_super) {
                 var change = interceptChange(adm, {
                     type: "update",
                     object: this,
-                    index: index, newValue: newValue
+                    index: index,
+                    newValue: newValue
                 });
                 if (!change)
                     return;
@@ -628,9 +676,6 @@ Object.defineProperty(ObservableArray.prototype, "length", {
         this.$mobx.setArrayLength(newLength);
     }
 });
-/**
- * Wrap function from prototype
- */
 [
     "every",
     "filter",
@@ -742,7 +787,8 @@ var ObservableValue = (function (_super) {
                 spyReportStart({
                     type: "update",
                     object: this,
-                    newValue: newValue, oldValue: oldValue
+                    newValue: newValue,
+                    oldValue: oldValue
                 });
             }
             this.setNewValue(newValue);
@@ -753,16 +799,18 @@ var ObservableValue = (function (_super) {
     ObservableValue.prototype.prepareNewValue = function (newValue) {
         checkIfStateModificationsAreAllowed(this);
         if (hasInterceptors(this)) {
-            var change = interceptChange(this, { object: this, type: "update", newValue: newValue });
+            var change = interceptChange(this, {
+                object: this,
+                type: "update",
+                newValue: newValue
+            });
             if (!change)
                 return UNCHANGED;
             newValue = change.newValue;
         }
         // apply modifier
         newValue = this.enhancer(newValue, this.value, this.name);
-        return this.value !== newValue
-            ? newValue
-            : UNCHANGED;
+        return this.value !== newValue ? newValue : UNCHANGED;
     };
     ObservableValue.prototype.setNewValue = function (newValue) {
         var oldValue = this.value;
@@ -772,7 +820,8 @@ var ObservableValue = (function (_super) {
             notifyListeners(this, {
                 type: "update",
                 object: this,
-                newValue: newValue, oldValue: oldValue
+                newValue: newValue,
+                oldValue: oldValue
             });
         }
     };
@@ -808,44 +857,43 @@ ObservableValue.prototype[primitiveSymbol()] = ObservableValue.prototype.valueOf
 var isObservableValue = createInstanceofPredicate("ObservableValue", ObservableValue);
 
 var messages = {
-    "m001": "It is not allowed to assign new values to @action fields",
-    "m002": "`runInAction` expects a function",
-    "m003": "`runInAction` expects a function without arguments",
-    "m004": "autorun expects a function",
-    "m005": "Warning: attempted to pass an action to autorun. Actions are untracked and will not trigger on state changes. Use `reaction` or wrap only your state modification code in an action.",
-    "m006": "Warning: attempted to pass an action to autorunAsync. Actions are untracked and will not trigger on state changes. Use `reaction` or wrap only your state modification code in an action.",
-    "m007": "reaction only accepts 2 or 3 arguments. If migrating from MobX 2, please provide an options object",
-    "m008": "wrapping reaction expression in `asReference` is no longer supported, use options object instead",
-    "m009": "@computed can only be used on getter functions, like: '@computed get myProps() { return ...; }'. It looks like it was used on a property.",
-    "m010": "@computed can only be used on getter functions, like: '@computed get myProps() { return ...; }'",
-    "m011": "First argument to `computed` should be an expression. If using computed as decorator, don't pass it arguments",
-    "m012": "computed takes one or two arguments if used as function",
-    "m013": "[mobx.expr] 'expr' should only be used inside other reactive functions.",
-    "m014": "extendObservable expected 2 or more arguments",
-    "m015": "extendObservable expects an object as first argument",
-    "m016": "extendObservable should not be used on maps, use map.merge instead",
-    "m017": "all arguments of extendObservable should be objects",
-    "m018": "extending an object with another observable (object) is not supported. Please construct an explicit propertymap, using `toJS` if need. See issue #540",
-    "m019": "[mobx.isObservable] isObservable(object, propertyName) is not supported for arrays and maps. Use map.has or array.length instead.",
-    "m020": "modifiers can only be used for individual object properties",
-    "m021": "observable expects zero or one arguments",
-    "m022": "@observable can not be used on getters, use @computed instead",
-    "m023": "Using `transaction` is deprecated, use `runInAction` or `(@)action` instead.",
-    "m024": "whyRun() can only be used if a derivation is active, or by passing an computed value / reaction explicitly. If you invoked whyRun from inside a computation; the computation is currently suspended but re-evaluating because somebody requested its value.",
-    "m025": "whyRun can only be used on reactions and computed values",
-    "m026": "`action` can only be invoked on functions",
-    "m028": "It is not allowed to set `useStrict` when a derivation is running",
-    "m029": "INTERNAL ERROR only onBecomeUnobserved shouldn't be called twice in a row",
-    "m030a": "Since strict-mode is enabled, changing observed observable values outside actions is not allowed. Please wrap the code in an `action` if this change is intended. Tried to modify: ",
-    "m030b": "Side effects like changing state are not allowed at this point. Are you trying to modify state from, for example, the render function of a React component? Tried to modify: ",
-    "m031": "Computed values are not allowed to cause side effects by changing observables that are already being observed. Tried to modify: ",
-    "m032": "* This computation is suspended (not in use by any reaction) and won't run automatically.\n	Didn't expect this computation to be suspended at this point?\n	  1. Make sure this computation is used by a reaction (reaction, autorun, observer).\n	  2. Check whether you are using this computation synchronously (in the same stack as they reaction that needs it).",
-    "m033": "`observe` doesn't support the fire immediately property for observable maps.",
-    "m034": "`mobx.map` is deprecated, use `new ObservableMap` or `mobx.observable.map` instead",
-    "m035": "Cannot make the designated object observable; it is not extensible",
-    "m036": "It is not possible to get index atoms from arrays",
-    "m037": "Hi there! I'm sorry you have just run into an exception.\nIf your debugger ends up here, know that some reaction (like the render() of an observer component, autorun or reaction)\nthrew an exception and that mobx caught it, to avoid that it brings the rest of your application down.\nThe original cause of the exception (the code that caused this reaction to run (again)), is still in the stack.\n\nHowever, more interesting is the actual stack trace of the error itself.\nHopefully the error is an instanceof Error, because in that case you can inspect the original stack of the error from where it was thrown.\nSee `error.stack` property, or press the very subtle \"(...)\" link you see near the console.error message that probably brought you here.\nThat stack is more interesting than the stack of this console.error itself.\n\nIf the exception you see is an exception you created yourself, make sure to use `throw new Error(\"Oops\")` instead of `throw \"Oops\"`,\nbecause the javascript environment will only preserve the original stack trace in the first form.\n\nYou can also make sure the debugger pauses the next time this very same exception is thrown by enabling \"Pause on caught exception\".\n(Note that it might pause on many other, unrelated exception as well).\n\nIf that all doesn't help you out, feel free to open an issue https://github.com/mobxjs/mobx/issues!\n",
-    "m038": "Missing items in this list?\n    1. Check whether all used values are properly marked as observable (use isObservable to verify)\n    2. Make sure you didn't dereference values too early. MobX observes props, not primitives. E.g: use 'person.name' instead of 'name' in your computation.\n"
+    m001: "It is not allowed to assign new values to @action fields",
+    m002: "`runInAction` expects a function",
+    m003: "`runInAction` expects a function without arguments",
+    m004: "autorun expects a function",
+    m005: "Warning: attempted to pass an action to autorun. Actions are untracked and will not trigger on state changes. Use `reaction` or wrap only your state modification code in an action.",
+    m006: "Warning: attempted to pass an action to autorunAsync. Actions are untracked and will not trigger on state changes. Use `reaction` or wrap only your state modification code in an action.",
+    m007: "reaction only accepts 2 or 3 arguments. If migrating from MobX 2, please provide an options object",
+    m008: "wrapping reaction expression in `asReference` is no longer supported, use options object instead",
+    m009: "@computed can only be used on getter functions, like: '@computed get myProps() { return ...; }'. It looks like it was used on a property.",
+    m010: "@computed can only be used on getter functions, like: '@computed get myProps() { return ...; }'",
+    m011: "First argument to `computed` should be an expression. If using computed as decorator, don't pass it arguments",
+    m012: "computed takes one or two arguments if used as function",
+    m013: "[mobx.expr] 'expr' should only be used inside other reactive functions.",
+    m014: "extendObservable expected 2 or more arguments",
+    m015: "extendObservable expects an object as first argument",
+    m016: "extendObservable should not be used on maps, use map.merge instead",
+    m017: "all arguments of extendObservable should be objects",
+    m018: "extending an object with another observable (object) is not supported. Please construct an explicit propertymap, using `toJS` if need. See issue #540",
+    m019: "[mobx.isObservable] isObservable(object, propertyName) is not supported for arrays and maps. Use map.has or array.length instead.",
+    m020: "modifiers can only be used for individual object properties",
+    m021: "observable expects zero or one arguments",
+    m022: "@observable can not be used on getters, use @computed instead",
+    m024: "whyRun() can only be used if a derivation is active, or by passing an computed value / reaction explicitly. If you invoked whyRun from inside a computation; the computation is currently suspended but re-evaluating because somebody requested its value.",
+    m025: "whyRun can only be used on reactions and computed values",
+    m026: "`action` can only be invoked on functions",
+    m028: "It is not allowed to set `useStrict` when a derivation is running",
+    m029: "INTERNAL ERROR only onBecomeUnobserved shouldn't be called twice in a row",
+    m030a: "Since strict-mode is enabled, changing observed observable values outside actions is not allowed. Please wrap the code in an `action` if this change is intended. Tried to modify: ",
+    m030b: "Side effects like changing state are not allowed at this point. Are you trying to modify state from, for example, the render function of a React component? Tried to modify: ",
+    m031: "Computed values are not allowed to cause side effects by changing observables that are already being observed. Tried to modify: ",
+    m032: "* This computation is suspended (not in use by any reaction) and won't run automatically.\n	Didn't expect this computation to be suspended at this point?\n	  1. Make sure this computation is used by a reaction (reaction, autorun, observer).\n	  2. Check whether you are using this computation synchronously (in the same stack as they reaction that needs it).",
+    m033: "`observe` doesn't support the fire immediately property for observable maps.",
+    m034: "`mobx.map` is deprecated, use `new ObservableMap` or `mobx.observable.map` instead",
+    m035: "Cannot make the designated object observable; it is not extensible",
+    m036: "It is not possible to get index atoms from arrays",
+    m037: "Hi there! I'm sorry you have just run into an exception.\nIf your debugger ends up here, know that some reaction (like the render() of an observer component, autorun or reaction)\nthrew an exception and that mobx caught it, to avoid that it brings the rest of your application down.\nThe original cause of the exception (the code that caused this reaction to run (again)), is still in the stack.\n\nHowever, more interesting is the actual stack trace of the error itself.\nHopefully the error is an instanceof Error, because in that case you can inspect the original stack of the error from where it was thrown.\nSee `error.stack` property, or press the very subtle \"(...)\" link you see near the console.error message that probably brought you here.\nThat stack is more interesting than the stack of this console.error itself.\n\nIf the exception you see is an exception you created yourself, make sure to use `throw new Error(\"Oops\")` instead of `throw \"Oops\"`,\nbecause the javascript environment will only preserve the original stack trace in the first form.\n\nYou can also make sure the debugger pauses the next time this very same exception is thrown by enabling \"Pause on caught exception\".\n(Note that it might pause on many other, unrelated exception as well).\n\nIf that all doesn't help you out, feel free to open an issue https://github.com/mobxjs/mobx/issues!\n",
+    m038: "Missing items in this list?\n    1. Check whether all used values are properly marked as observable (use isObservable to verify)\n    2. Make sure you didn't dereference values too early. MobX observes props, not primitives. E.g: use 'person.name' instead of 'name' in your computation.\n"
 };
 function getMessage(id) {
     return messages[id];
@@ -984,7 +1032,7 @@ function createClassPropertyDecorator(
                     }
                 }
             };
-            if (arguments.length < 3 || arguments.length === 5 && argLen < 3) {
+            if (arguments.length < 3 || (arguments.length === 5 && argLen < 3)) {
                 // Typescript target is ES3, so it won't define property for us
                 // or using Reflect.decorate polyfill, which will return no descriptor
                 // (see https://github.com/mobxjs/mobx/issues/333)
@@ -1000,10 +1048,11 @@ function createClassPropertyDecorator(
             }
             var value_1 = descriptor.value, initializer_1 = descriptor.initializer;
             target.__mobxLazyInitializers.push(function (instance) {
-                onInitialize(instance, key, (initializer_1 ? initializer_1.call(instance) : value_1), customArgs, descriptor);
+                onInitialize(instance, key, initializer_1 ? initializer_1.call(instance) : value_1, customArgs, descriptor);
             });
             return {
-                enumerable: enumerable, configurable: true,
+                enumerable: enumerable,
+                configurable: true,
                 get: function () {
                     if (this.__mobxDidRunLazyInitializers !== true)
                         runLazyInitializers(this);
@@ -1026,7 +1075,9 @@ function createClassPropertyDecorator(
             /** Indirect invocation: @decorator(args) bla */
             var outerArgs = arguments;
             var argLen = arguments.length;
-            return function (target, key, descriptor) { return classPropertyDecorator(target, key, descriptor, outerArgs, argLen); };
+            return function (target, key, descriptor) {
+                return classPropertyDecorator(target, key, descriptor, outerArgs, argLen);
+            };
         };
     }
     return classPropertyDecorator;
@@ -1042,7 +1093,8 @@ function runLazyInitializers(instance) {
         return;
     if (instance.__mobxLazyInitializers) {
         addHiddenProp(instance, "__mobxDidRunLazyInitializers", true);
-        instance.__mobxDidRunLazyInitializers && instance.__mobxLazyInitializers.forEach(function (initializer) { return initializer(instance); });
+        instance.__mobxDidRunLazyInitializers &&
+            instance.__mobxLazyInitializers.forEach(function (initializer) { return initializer(instance); });
     }
 }
 function quacksLikeADecorator(args) {
@@ -1050,7 +1102,7 @@ function quacksLikeADecorator(args) {
 }
 
 var actionFieldDecorator = createClassPropertyDecorator(function (target, key, value, args, originalDescriptor) {
-    var actionName = (args && args.length === 1) ? args[0] : (value.name || key || "<unnamed action>");
+    var actionName = args && args.length === 1 ? args[0] : value.name || key || "<unnamed action>";
     var wrappedAction = action(actionName, value);
     addHiddenProp(target, key, wrappedAction);
 }, function (key) {
@@ -1092,6 +1144,9 @@ function namedActionDecorator(name) {
             descriptor.configurable = true;
             return descriptor;
         }
+        if (descriptor !== undefined && descriptor.get !== undefined) {
+            throw new Error("[mobx] action is not expected to be used with getters");
+        }
         // bound instance methods
         return actionFieldDecorator(name).apply(this, arguments);
     };
@@ -1120,13 +1175,13 @@ function identityComparer(a, b) {
     return a === b;
 }
 function structuralComparer(a, b) {
-    if (typeof a === 'number' && typeof b === 'number' && isNaN(a) && isNaN(b)) {
+    if (typeof a === "number" && typeof b === "number" && isNaN(a) && isNaN(b)) {
         return true;
     }
     return deepEqual(a, b);
 }
 function defaultComparer(a, b) {
-    if (typeof a === 'number' && typeof b === 'number' && isNaN(a) && isNaN(b)) {
+    if (typeof a === "number" && typeof b === "number" && isNaN(a) && isNaN(b)) {
         return true;
     }
     return identityComparer(a, b);
@@ -1145,7 +1200,7 @@ function autorun(arg1, arg2, arg3) {
         scope = arg3;
     }
     else {
-        name = arg1.name || ("Autorun@" + getNextId());
+        name = arg1.name || "Autorun@" + getNextId();
         view = arg1;
         scope = arg2;
     }
@@ -1171,7 +1226,7 @@ function when(arg1, arg2, arg3, arg4) {
         scope = arg4;
     }
     else {
-        name = ("When@" + getNextId());
+        name = "When@" + getNextId();
         predicate = arg1;
         effect = arg2;
         scope = arg3;
@@ -1195,7 +1250,7 @@ function autorunAsync(arg1, arg2, arg3, arg4) {
         scope = arg4;
     }
     else {
-        name = arg1.name || ("AutorunAsync@" + getNextId());
+        name = arg1.name || "AutorunAsync@" + getNextId();
         func = arg1;
         delay = arg2;
         scope = arg3;
@@ -1216,7 +1271,9 @@ function autorunAsync(arg1, arg2, arg3, arg4) {
             }, delay);
         }
     });
-    function reactionRunner() { func(r); }
+    function reactionRunner() {
+        func(r);
+    }
     r.schedule();
     return r.getDisposer();
 }
@@ -1234,7 +1291,8 @@ function reaction(expression, effect, arg3) {
     else {
         opts = {};
     }
-    opts.name = opts.name || expression.name || effect.name || ("Reaction@" + getNextId());
+    opts.name =
+        opts.name || expression.name || effect.name || "Reaction@" + getNextId();
     opts.fireImmediately = arg3 === true || opts.fireImmediately === true;
     opts.delay = opts.delay || 0;
     opts.compareStructural = opts.compareStructural || opts.struct || false;
@@ -1248,9 +1306,7 @@ function reaction(expression, effect, arg3) {
     var value;
     var equals = opts.equals
         ? opts.equals
-        : (opts.compareStructural || opts.struct)
-            ? comparer.structural
-            : comparer.default;
+        : opts.compareStructural || opts.struct ? comparer.structural : comparer.default;
     var r = new Reaction(opts.name, function () {
         if (firstTime || opts.delay < 1) {
             reactionRunner();
@@ -1286,7 +1342,9 @@ function reaction(expression, effect, arg3) {
 /**
  * A node in the state dependency root that observes other nodes, and can be observed itself.
  *
- * ComputedValue will remember result of the computation for duration of a batch, or being observed
+ * ComputedValue will remember the result of the computation for the duration of the batch, or
+ * while being observed.
+ *
  * During this time it will recompute only when one of its direct dependencies changed,
  * but only when it is being accessed with `ComputedValue.get()`.
  *
@@ -1377,7 +1435,8 @@ var ComputedValue = (function () {
     };
     ComputedValue.prototype.set = function (value) {
         if (this.setter) {
-            invariant(!this.isRunningSetter, "The setter of computed value '" + this.name + "' is trying to update itself. Did you intend to update an _observable_ value, instead of the computed property?");
+            invariant(!this.isRunningSetter, "The setter of computed value '" + this
+                .name + "' is trying to update itself. Did you intend to update an _observable_ value, instead of the computed property?");
             this.isRunningSetter = true;
             try {
                 this.setter.call(this.scope, value);
@@ -1387,7 +1446,8 @@ var ComputedValue = (function () {
             }
         }
         else
-            invariant(false, "[ComputedValue '" + this.name + "'] It is not possible to assign a new value to a computed value.");
+            invariant(false, "[ComputedValue '" + this
+                .name + "'] It is not possible to assign a new value to a computed value.");
     };
     ComputedValue.prototype.trackAndCompute = function () {
         if (isSpyEnabled()) {
@@ -1398,7 +1458,7 @@ var ComputedValue = (function () {
             });
         }
         var oldValue = this.value;
-        var newValue = this.value = this.computeValue(true);
+        var newValue = (this.value = this.computeValue(true));
         return (isCaughtException(oldValue) ||
             isCaughtException(newValue) ||
             !this.equals(oldValue, newValue));
@@ -1422,7 +1482,6 @@ var ComputedValue = (function () {
         this.isComputing = false;
         return res;
     };
-    
     ComputedValue.prototype.observe = function (listener, fireImmediately) {
         var _this = this;
         var firstTime = true;
@@ -1452,14 +1511,20 @@ var ComputedValue = (function () {
     ComputedValue.prototype.valueOf = function () {
         return toPrimitive(this.get());
     };
-    
     ComputedValue.prototype.whyRun = function () {
         var isTracking = Boolean(globalState.trackingDerivation);
         var observing = unique(this.isComputing ? this.newObserving : this.observing).map(function (dep) { return dep.name; });
         var observers = unique(getObservers(this).map(function (dep) { return dep.name; }));
-        return ("\nWhyRun? computation '" + this.name + "':\n * Running because: " + (isTracking ? "[active] the value of this computation is needed by a reaction" : this.isComputing ? "[get] The value of this computed was requested outside a reaction" : "[idle] not running at the moment") + "\n" +
-            (this.dependenciesState === exports.IDerivationState.NOT_TRACKING ? getMessage("m032") :
-                " * This computation will re-run if any of the following observables changes:\n    " + joinStrings(observing) + "\n    " + ((this.isComputing && isTracking) ? " (... or any observable accessed during the remainder of the current run)" : "") + "\n\t" + getMessage("m038") + "\n\n  * If the outcome of this computation changes, the following observers will be re-run:\n    " + joinStrings(observers) + "\n"));
+        return ("\nWhyRun? computation '" + this.name + "':\n * Running because: " + (isTracking
+            ? "[active] the value of this computation is needed by a reaction"
+            : this.isComputing
+                ? "[get] The value of this computed was requested outside a reaction"
+                : "[idle] not running at the moment") + "\n" +
+            (this.dependenciesState === exports.IDerivationState.NOT_TRACKING
+                ? getMessage("m032")
+                : " * This computation will re-run if any of the following observables changes:\n    " + joinStrings(observing) + "\n    " + (this.isComputing && isTracking
+                    ? " (... or any observable accessed during the remainder of the current run)"
+                    : "") + "\n\t" + getMessage("m038") + "\n\n  * If the outcome of this computation changes, the following observers will be re-run:\n    " + joinStrings(observers) + "\n"));
     };
     return ComputedValue;
 }());
@@ -1475,10 +1540,10 @@ var ObservableObjectAdministration = (function () {
         this.interceptors = null;
     }
     /**
-        * Observes this object. Triggers for the events 'add', 'update' and 'delete'.
-        * See: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/observe
-        * for callback details
-        */
+     * Observes this object. Triggers for the events 'add', 'update' and 'delete'.
+     * See: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/observe
+     * for callback details
+     */
     ObservableObjectAdministration.prototype.observe = function (callback, fireImmediately) {
         invariant(fireImmediately !== true, "`observe` doesn't support the fire immediately property for observable objects.");
         return registerListener(this, callback);
@@ -1489,7 +1554,7 @@ var ObservableObjectAdministration = (function () {
     return ObservableObjectAdministration;
 }());
 function asObservableObject(target, name) {
-    if (isObservableObject(target) && target.hasOwnProperty('$mobx'))
+    if (isObservableObject(target) && target.hasOwnProperty("$mobx"))
         return target.$mobx;
     invariant(Object.isExtensible(target), getMessage("m035"));
     if (!isPlainObject(target))
@@ -1501,7 +1566,7 @@ function asObservableObject(target, name) {
     return adm;
 }
 function defineObservablePropertyFromDescriptor(adm, propName, descriptor, defaultEnhancer) {
-    if (adm.values[propName]) {
+    if (adm.values[propName] && !isComputedValue(adm.values[propName])) {
         // already observable property
         invariant("value" in descriptor, "The property " + propName + " in " + adm.name + " is already observable, cannot redefine it as computed property");
         adm.target[propName] = descriptor.value; // the property setter will make 'value' reactive if needed.
@@ -1545,7 +1610,7 @@ function defineObservableProperty(adm, propName, newValue, enhancer) {
             return;
         newValue = change.newValue;
     }
-    var observable = adm.values[propName] = new ObservableValue(newValue, enhancer, adm.name + "." + propName, false);
+    var observable = (adm.values[propName] = new ObservableValue(newValue, enhancer, adm.name + "." + propName, false));
     newValue = observable.value; // observableValue might have changed it
     Object.defineProperty(adm.target, propName, generateObservablePropConfig(propName));
     notifyPropertyAddition(adm, adm.target, propName, newValue);
@@ -1569,28 +1634,30 @@ function defineComputedPropertyFromComputedValue(adm, propName, computedValue) {
 var observablePropertyConfigs = {};
 var computedPropertyConfigs = {};
 function generateObservablePropConfig(propName) {
-    return observablePropertyConfigs[propName] || (observablePropertyConfigs[propName] = {
-        configurable: true,
-        enumerable: true,
-        get: function () {
-            return this.$mobx.values[propName].get();
-        },
-        set: function (v) {
-            setPropertyValue(this, propName, v);
-        }
-    });
+    return (observablePropertyConfigs[propName] ||
+        (observablePropertyConfigs[propName] = {
+            configurable: true,
+            enumerable: true,
+            get: function () {
+                return this.$mobx.values[propName].get();
+            },
+            set: function (v) {
+                setPropertyValue(this, propName, v);
+            }
+        }));
 }
 function generateComputedPropConfig(propName) {
-    return computedPropertyConfigs[propName] || (computedPropertyConfigs[propName] = {
-        configurable: true,
-        enumerable: false,
-        get: function () {
-            return this.$mobx.values[propName].get();
-        },
-        set: function (v) {
-            return this.$mobx.values[propName].set(v);
-        }
-    });
+    return (computedPropertyConfigs[propName] ||
+        (computedPropertyConfigs[propName] = {
+            configurable: true,
+            enumerable: false,
+            get: function () {
+                return this.$mobx.values[propName].get();
+            },
+            set: function (v) {
+                return this.$mobx.values[propName].set(v);
+            }
+        }));
 }
 function setPropertyValue(instance, name, newValue) {
     var adm = instance.$mobx;
@@ -1600,7 +1667,8 @@ function setPropertyValue(instance, name, newValue) {
         var change = interceptChange(adm, {
             type: "update",
             object: instance,
-            name: name, newValue: newValue
+            name: name,
+            newValue: newValue
         });
         if (!change)
             return;
@@ -1611,12 +1679,15 @@ function setPropertyValue(instance, name, newValue) {
     if (newValue !== UNCHANGED) {
         var notify = hasListeners(adm);
         var notifySpy = isSpyEnabled();
-        var change = notify || notifySpy ? {
-            type: "update",
-            object: instance,
-            oldValue: observable.value,
-            name: name, newValue: newValue
-        } : null;
+        var change = notify || notifySpy
+            ? {
+                type: "update",
+                object: instance,
+                oldValue: observable.value,
+                name: name,
+                newValue: newValue
+            }
+            : null;
         if (notifySpy)
             spyReportStart(change);
         observable.setNewValue(newValue);
@@ -1629,10 +1700,14 @@ function setPropertyValue(instance, name, newValue) {
 function notifyPropertyAddition(adm, object, name, newValue) {
     var notify = hasListeners(adm);
     var notifySpy = isSpyEnabled();
-    var change = notify || notifySpy ? {
-        type: "add",
-        object: object, name: name, newValue: newValue
-    } : null;
+    var change = notify || notifySpy
+        ? {
+            type: "add",
+            object: object,
+            name: name,
+            newValue: newValue
+        }
+        : null;
     if (notifySpy)
         spyReportStart(change);
     if (notify)
@@ -1651,10 +1726,10 @@ function isObservableObject(thing) {
 }
 
 /**
-    * Returns true if the provided value is reactive.
-    * @param value object, function or array
-    * @param property if property is specified, checks whether value.property is reactive.
-    */
+ * Returns true if the provided value is reactive.
+ * @param value object, function or array
+ * @param property if property is specified, checks whether value.property is reactive.
+ */
 function isObservable(value, property) {
     if (value === null || value === undefined)
         return false;
@@ -1668,7 +1743,11 @@ function isObservable(value, property) {
         return false;
     }
     // For first check, see #701
-    return isObservableObject(value) || !!value.$mobx || isAtom(value) || isReaction(value) || isComputedValue(value);
+    return (isObservableObject(value) ||
+        !!value.$mobx ||
+        isAtom(value) ||
+        isReaction(value) ||
+        isComputedValue(value));
 }
 
 function createDecoratorForEnhancer(enhancer) {
@@ -1680,7 +1759,8 @@ function createDecoratorForEnhancer(enhancer) {
         defineObservableProperty(adm, name, baseValue, enhancer);
     }, function (name) {
         var observable = this.$mobx.values[name];
-        if (observable === undefined)
+        if (observable === undefined // See #505
+        )
             return undefined;
         return observable.get();
     }, function (name, value) {
@@ -1705,7 +1785,7 @@ function extendShallowObservable(target) {
 function extendObservableHelper(target, defaultEnhancer, properties) {
     invariant(arguments.length >= 2, getMessage("m014"));
     invariant(typeof target === "object", getMessage("m015"));
-    invariant(!(isObservableMap(target)), getMessage("m016"));
+    invariant(!isObservableMap(target), getMessage("m016"));
     properties.forEach(function (propSet) {
         invariant(typeof propSet === "object", getMessage("m017"));
         invariant(!isObservable(propSet), getMessage("m018"));
@@ -1852,7 +1932,7 @@ var observable = createObservable;
 // ES6 class methods aren't enumerable, can't use Object.keys
 Object.getOwnPropertyNames(IObservableFactories.prototype)
     .filter(function (name) { return name !== "constructor"; })
-    .forEach(function (name) { return observable[name] = IObservableFactories.prototype[name]; });
+    .forEach(function (name) { return (observable[name] = IObservableFactories.prototype[name]); });
 observable.deep.struct = observable.struct;
 observable.ref.struct = function () {
     if (arguments.length < 2) {
@@ -1950,7 +2030,6 @@ function refStructEnhancer(v, oldValue, name) {
  */
 function transaction(action, thisArg) {
     if (thisArg === void 0) { thisArg = undefined; }
-    deprecated(getMessage("m023"));
     return runInTransaction.apply(undefined, arguments);
 }
 function runInTransaction(action, thisArg) {
@@ -2024,12 +2103,14 @@ var ObservableMap = (function () {
         if (this._has(key)) {
             var notifySpy = isSpyEnabled();
             var notify = hasListeners(this);
-            var change = notify || notifySpy ? {
-                type: "delete",
-                object: this,
-                oldValue: this._data[key].value,
-                name: key
-            } : null;
+            var change = notify || notifySpy
+                ? {
+                    type: "delete",
+                    object: this,
+                    oldValue: this._data[key].value,
+                    name: key
+                }
+                : null;
             if (notifySpy)
                 spyReportStart(change);
             runInTransaction(function () {
@@ -2064,12 +2145,15 @@ var ObservableMap = (function () {
         if (newValue !== UNCHANGED) {
             var notifySpy = isSpyEnabled();
             var notify = hasListeners(this);
-            var change = notify || notifySpy ? {
-                type: "update",
-                object: this,
-                oldValue: observable$$1.value,
-                name: name, newValue: newValue
-            } : null;
+            var change = notify || notifySpy
+                ? {
+                    type: "update",
+                    object: this,
+                    oldValue: observable$$1.value,
+                    name: name,
+                    newValue: newValue
+                }
+                : null;
             if (notifySpy)
                 spyReportStart(change);
             observable$$1.setNewValue(newValue);
@@ -2082,19 +2166,21 @@ var ObservableMap = (function () {
     ObservableMap.prototype._addValue = function (name, newValue) {
         var _this = this;
         runInTransaction(function () {
-            var observable$$1 = _this._data[name] = new ObservableValue(newValue, _this.enhancer, _this.name + "." + name, false);
+            var observable$$1 = (_this._data[name] = new ObservableValue(newValue, _this.enhancer, _this.name + "." + name, false));
             newValue = observable$$1.value; // value might have been changed
             _this._updateHasMapEntry(name, true);
             _this._keys.push(name);
         });
         var notifySpy = isSpyEnabled();
         var notify = hasListeners(this);
-        var change = notify || notifySpy ? {
-            type: "add",
-            object: this,
-            name: name,
-            newValue: newValue
-        } : null;
+        var change = notify || notifySpy
+            ? {
+                type: "add",
+                object: this,
+                name: name,
+                newValue: newValue
+            }
+            : null;
         if (notifySpy)
             spyReportStart(change);
         if (notify)
@@ -2179,7 +2265,7 @@ var ObservableMap = (function () {
     ObservableMap.prototype.toJS = function () {
         var _this = this;
         var res = {};
-        this.keys().forEach(function (key) { return res[key] = _this.get(key); });
+        this.keys().forEach(function (key) { return (res[key] = _this.get(key)); });
         return res;
     };
     ObservableMap.prototype.toJSON = function () {
@@ -2199,7 +2285,10 @@ var ObservableMap = (function () {
     };
     ObservableMap.prototype.toString = function () {
         var _this = this;
-        return this.name + "[{ " + this.keys().map(function (key) { return key + ": " + ("" + _this.get(key)); }).join(", ") + " }]";
+        return (this.name +
+            "[{ " +
+            this.keys().map(function (key) { return key + ": " + ("" + _this.get(key)); }).join(", ") +
+            " }]");
     };
     /**
      * Observes this object. Triggers for the events 'add', 'update' and 'delete'.
@@ -2228,7 +2317,7 @@ var isObservableMap = createInstanceofPredicate("ObservableMap", ObservableMap);
 var EMPTY_ARRAY = [];
 Object.freeze(EMPTY_ARRAY);
 function getGlobal() {
-    return typeof window !== 'undefined' ? window : global;
+    return typeof window !== "undefined" ? window : global;
 }
 function getNextId() {
     return ++globalState.mobxGuid;
@@ -2280,7 +2369,9 @@ function joinStrings(things, limit, separator) {
     if (!things)
         return "";
     var sliced = things.slice(0, limit);
-    return "" + sliced.join(separator) + (things.length > limit ? " (... and " + (things.length - limit) + "more)" : "");
+    return "" + sliced.join(separator) + (things.length > limit
+        ? " (... and " + (things.length - limit) + "more)"
+        : "");
 }
 function isObject(value) {
     return value !== null && typeof value === "object";
@@ -2422,7 +2513,7 @@ function primitiveSymbol() {
     return (typeof Symbol === "function" && Symbol.toPrimitive) || "@@toPrimitive";
 }
 function toPrimitive(value) {
-    return value === null ? null : typeof value === "object" ? ("" + value) : value;
+    return value === null ? null : typeof value === "object" ? "" + value : value;
 }
 
 /**
@@ -2567,6 +2658,7 @@ function addObserver(observable, node) {
     // invariantObservers(observable);
     var l = observable.observers.length;
     if (l) {
+        // because object assignment is relatively expensive, let's not store data about index 0.
         observable.observersIndexes[node.__mapid] = l;
     }
     observable.observers[l] = node;
@@ -2590,8 +2682,10 @@ function removeObserver(observable, node) {
         var map = observable.observersIndexes;
         var filler = list.pop(); // get last element, which should fill the place of `node`, so the array doesn't have holes
         if (filler !== node) {
+            // otherwise node was the last element, which already got removed from array
             var index = map[node.__mapid] || 0; // getting index of `node`. this is the only place we actually use map.
             if (index) {
+                // map store all indexes but 0, see comment in `addObserver`
                 map[filler.__mapid] = index;
             }
             else {
@@ -2688,7 +2782,8 @@ function propagateChangeConfirmed(observable) {
         var d = observers[i];
         if (d.dependenciesState === exports.IDerivationState.POSSIBLY_STALE)
             d.dependenciesState = exports.IDerivationState.STALE;
-        else if (d.dependenciesState === exports.IDerivationState.UP_TO_DATE)
+        else if (d.dependenciesState === exports.IDerivationState.UP_TO_DATE // this happens during computing of `d`, just keep lowestObserverState up to date.
+        )
             observable.lowestObserverState = exports.IDerivationState.UP_TO_DATE;
     }
     // invariantLOS(observable, "confirmed end");
@@ -2753,9 +2848,11 @@ function isCaughtException(e) {
  */
 function shouldCompute(derivation) {
     switch (derivation.dependenciesState) {
-        case exports.IDerivationState.UP_TO_DATE: return false;
+        case exports.IDerivationState.UP_TO_DATE:
+            return false;
         case exports.IDerivationState.NOT_TRACKING:
-        case exports.IDerivationState.STALE: return true;
+        case exports.IDerivationState.STALE:
+            return true;
         case exports.IDerivationState.POSSIBLY_STALE: {
             var prevUntracked = untrackedStart(); // no need for those computeds to be reported, they will be picked up in trackDerivedFunction.
             var obs = derivation.observing, l = obs.length;
@@ -2829,9 +2926,8 @@ function trackDerivedFunction(derivation, f, context) {
 function bindDependencies(derivation) {
     // invariant(derivation.dependenciesState !== IDerivationState.NOT_TRACKING, "INTERNAL ERROR bindDependencies expects derivation.dependenciesState !== -1");
     var prevObserving = derivation.observing;
-    var observing = derivation.observing = derivation.newObserving;
+    var observing = (derivation.observing = derivation.newObserving);
     var lowestNewObservingDerivationState = exports.IDerivationState.UP_TO_DATE;
-    derivation.newObserving = null; // newObserving shouldn't be needed outside tracking
     // Go through all new observables and check diffValue: (this list can contain duplicates):
     //   0: first occurrence, change to 1 and keep it
     //   1: extra occurrence, drop it
@@ -2851,6 +2947,7 @@ function bindDependencies(derivation) {
         }
     }
     observing.length = i0;
+    derivation.newObserving = null; // newObserving shouldn't be needed outside tracking (statement moved down to work around FF bug, see #614)
     // Go through all old observables and check diffValue: (it is unique after last bindDependencies)
     //   0: it's not in new observables, unobserve it
     //   1: it keeps being observed, don't want to notify it. change to 0
@@ -2872,8 +2969,8 @@ function bindDependencies(derivation) {
             addObserver(dep, derivation);
         }
     }
-    // Some new observed derivations might become stale during this derivation computation
-    // so say had no chance to propagate staleness (#916)
+    // Some new observed derivations may become stale during this derivation computation
+    // so they have had no chance to propagate staleness (#916)
     if (lowestNewObservingDerivationState !== exports.IDerivationState.UP_TO_DATE) {
         derivation.dependenciesState = lowestNewObservingDerivationState;
         derivation.onBecomeStale();
@@ -3038,7 +3135,11 @@ var Reaction = (function () {
     };
     Reaction.prototype.whyRun = function () {
         var observing = unique(this._isRunning ? this.newObserving : this.observing).map(function (dep) { return dep.name; });
-        return ("\nWhyRun? reaction '" + this.name + "':\n * Status: [" + (this.isDisposed ? "stopped" : this._isRunning ? "running" : this.isScheduled() ? "scheduled" : "idle") + "]\n * This reaction will re-run if any of the following observables changes:\n    " + joinStrings(observing) + "\n    " + ((this._isRunning) ? " (... or any observable accessed during the remainder of the current run)" : "") + "\n\t" + getMessage("m038") + "\n");
+        return "\nWhyRun? reaction '" + this.name + "':\n * Status: [" + (this.isDisposed
+            ? "stopped"
+            : this._isRunning ? "running" : this.isScheduled() ? "scheduled" : "idle") + "]\n * This reaction will re-run if any of the following observables changes:\n    " + joinStrings(observing) + "\n    " + (this._isRunning
+            ? " (... or any observable accessed during the remainder of the current run)"
+            : "") + "\n\t" + getMessage("m038") + "\n";
     };
     return Reaction;
 }());
@@ -3077,8 +3178,8 @@ function runReactionsHelper() {
     // we converge to no remaining reactions after a while.
     while (allReactions.length > 0) {
         if (++iterations === MAX_REACTION_ITERATIONS) {
-            console.error("Reaction doesn't converge to a stable state after " + MAX_REACTION_ITERATIONS + " iterations."
-                + (" Probably there is a cycle in the reactive function: " + allReactions[0]));
+            console.error("Reaction doesn't converge to a stable state after " + MAX_REACTION_ITERATIONS + " iterations." +
+                (" Probably there is a cycle in the reactive function: " + allReactions[0]));
             allReactions.splice(0); // clear reactions
         }
         var remainingReactions = allReactions.splice(0);
@@ -3118,7 +3219,8 @@ function createComputedDecorator(equals) {
         defineComputedProperty(adm, name, originalDescriptor.get, originalDescriptor.set, equals, false);
     }, function (name) {
         var observable = this.$mobx.values[name];
-        if (observable === undefined)
+        if (observable === undefined // See #505
+        )
             return undefined;
         return observable.get();
     }, function (name, value) {
@@ -3131,7 +3233,7 @@ var computedStructDecorator = createComputedDecorator(comparer.structural);
  * Decorator for class properties: @computed get value() { return expr; }.
  * For legacy purposes also invokable as ES5 observable created: `computed(() => expr)`;
  */
-var computed = (function computed(arg1, arg2, arg3) {
+var computed = function computed(arg1, arg2, arg3) {
     if (typeof arg2 === "string") {
         return computedDecorator.apply(null, arguments);
     }
@@ -3141,11 +3243,9 @@ var computed = (function computed(arg1, arg2, arg3) {
     opts.setter = typeof arg2 === "function" ? arg2 : opts.setter;
     var equals = opts.equals
         ? opts.equals
-        : (opts.compareStructural || opts.struct)
-            ? comparer.structural
-            : comparer.default;
+        : opts.compareStructural || opts.struct ? comparer.structural : comparer.default;
     return new ComputedValue(arg1, opts.context, equals, opts.name || arg1.name || "", opts.setter);
-});
+};
 computed.struct = computedStructDecorator;
 computed.equals = createComputedDecorator;
 
@@ -3250,19 +3350,19 @@ function interceptProperty(thing, property, handler) {
 }
 
 /**
-    * expr can be used to create temporarily views inside views.
-    * This can be improved to improve performance if a value changes often, but usually doesn't affect the outcome of an expression.
-    *
-    * In the following example the expression prevents that a component is rerender _each time_ the selection changes;
-    * instead it will only rerenders when the current todo is (de)selected.
-    *
-    * reactiveComponent((props) => {
-    *     const todo = props.todo;
-    *     const isSelected = mobx.expr(() => props.viewState.selection === todo);
-    *     return <div className={isSelected ? "todo todo-selected" : "todo"}>{todo.title}</div>
-    * });
-    *
-    */
+ * expr can be used to create temporarily views inside views.
+ * This can be improved to improve performance if a value changes often, but usually doesn't affect the outcome of an expression.
+ *
+ * In the following example the expression prevents that a component is rerender _each time_ the selection changes;
+ * instead it will only rerenders when the current todo is (de)selected.
+ *
+ * reactiveComponent((props) => {
+ *     const todo = props.todo;
+ *     const isSelected = mobx.expr(() => props.viewState.selection === todo);
+ *     return <div className={isSelected ? "todo todo-selected" : "todo"}>{todo.title}</div>
+ * });
+ *
+ */
 function expr(expr, scope) {
     if (!isComputingDerivation())
         console.warn(getMessage("m013"));
@@ -3304,7 +3404,7 @@ function toJS(source, detectCycles, __alreadySeen) {
         }
         if (isObservableMap(source)) {
             var res_1 = cache({});
-            source.forEach(function (value, key) { return res_1[key] = toJS(value, detectCycles, __alreadySeen); });
+            source.forEach(function (value, key) { return (res_1[key] = toJS(value, detectCycles, __alreadySeen)); });
             return res_1;
         }
         if (isObservableValue(source))
@@ -3353,7 +3453,7 @@ function createTransformer(transformer, onCleanup) {
     };
 }
 function getMemoizationId(object) {
-    if (typeof object === 'string' || typeof object === 'number')
+    if (typeof object === "string" || typeof object === "number")
         return object;
     if (object === null || typeof object !== "object")
         throw new Error("[mobx] transform expected some kind of object or primitive value, got: " + object);
@@ -3474,32 +3574,45 @@ var extras = {
 var everything = {
     Reaction: Reaction,
     untracked: untracked,
-    Atom: Atom, BaseAtom: BaseAtom,
-    useStrict: useStrict, isStrictModeEnabled: isStrictModeEnabled,
+    Atom: Atom,
+    BaseAtom: BaseAtom,
+    useStrict: useStrict,
+    isStrictModeEnabled: isStrictModeEnabled,
     spy: spy,
     comparer: comparer,
-    asReference: asReference, asFlat: asFlat, asStructure: asStructure, asMap: asMap,
+    asReference: asReference,
+    asFlat: asFlat,
+    asStructure: asStructure,
+    asMap: asMap,
     isModifierDescriptor: isModifierDescriptor,
     isObservableObject: isObservableObject,
     isBoxedObservable: isObservableValue,
     isObservableArray: isObservableArray,
-    ObservableMap: ObservableMap, isObservableMap: isObservableMap, map: map,
+    ObservableMap: ObservableMap,
+    isObservableMap: isObservableMap,
+    map: map,
     transaction: transaction,
     observable: observable,
     computed: computed,
     isObservable: isObservable,
     isComputed: isComputed,
-    extendObservable: extendObservable, extendShallowObservable: extendShallowObservable,
+    extendObservable: extendObservable,
+    extendShallowObservable: extendShallowObservable,
     observe: observe,
     intercept: intercept,
-    autorun: autorun, autorunAsync: autorunAsync, when: when, reaction: reaction,
-    action: action, isAction: isAction, runInAction: runInAction,
+    autorun: autorun,
+    autorunAsync: autorunAsync,
+    when: when,
+    reaction: reaction,
+    action: action,
+    isAction: isAction,
+    runInAction: runInAction,
     expr: expr,
     toJS: toJS,
     createTransformer: createTransformer,
     whyRun: whyRun,
     isArrayLike: isArrayLike,
-    extras: extras,
+    extras: extras
 };
 var warnedAboutDefaultExport = false;
 var _loop_1 = function (p) {
@@ -3508,9 +3621,9 @@ var _loop_1 = function (p) {
         get: function () {
             if (!warnedAboutDefaultExport) {
                 warnedAboutDefaultExport = true;
-                console.warn('Using default export (`import mobx from \'mobx\'`) is deprecated ' +
-                    'and won’t work in mobx@4.0.0\n' +
-                    'Use `import * as mobx from \'mobx\'` instead');
+                console.warn("Using default export (`import mobx from 'mobx'`) is deprecated " +
+                    "and won’t work in mobx@4.0.0\n" +
+                    "Use `import * as mobx from 'mobx'` instead");
             }
             return val;
         }