mobx 4.15.3 → 4.15.4

package/CHANGELOG.md

@@ -1,3 +1,7 @@
+# 5.15.4 / 4.15.4
+
+-   Fix process.env replacement in build [#2267](https://github.com/mobxjs/mobx/pull/2267) by [@fredyc](https://github.com/fredyc)
+
 # 5.15.3 / 4.15.3
 
 -   Define action name to be as the function name [#2262](https://github.com/mobxjs/mobx/pull/2262) by [@nadavkaner](https://github.com/nadavkaner)

package/lib/index.js

@@ -0,0 +1,7 @@
+
+if (typeof process !== 'undefined' && process.env.NODE_ENV === 'production') {
+    module.exports = require('./mobx.min.js');
+} else {
+    module.exports = require('./mobx.js');
+}
+        

package/lib/mobx.es6.js

@@ -34,6 +34,8 @@ function invariant(check, message) {
  */
 const deprecatedMessages = [];
 function deprecated(msg, thing) {
+    if (process.env.NODE_ENV === "production")
+        return false;
     if (thing) {
         return deprecated(`'${msg}', use '${thing}' instead.`);
     }
@@ -113,7 +115,7 @@ function isPropertyConfigurable(object, prop) {
     return !descriptor || (descriptor.configurable !== false && descriptor.writable !== false);
 }
 function assertPropertyConfigurable(object, prop) {
-    if (!isPropertyConfigurable(object, prop))
+    if (process.env.NODE_ENV !== "production" && !isPropertyConfigurable(object, prop))
         fail(`Cannot make property '${prop}' observable, it is not configurable and writable in the target object`);
 }
 function createInstanceofPredicate(name, clazz) {
@@ -290,7 +292,7 @@ function createPropDecorator(propertyInitiallyEnumerable, propertyCreator) {
                 propertyCreator(target, prop, descriptor, target, decoratorArguments);
                 return null;
             }
-            if (!quacksLikeADecorator(arguments))
+            if (process.env.NODE_ENV !== "production" && !quacksLikeADecorator(arguments))
                 fail("This function is a decorator, but it wasn't invoked like a decorator");
             if (!Object.prototype.hasOwnProperty.call(target, "__mobxDecorators")) {
                 const inheritedDecorators = target.__mobxDecorators;
@@ -350,14 +352,15 @@ function shallowEnhancer(v, _, name) {
         return observable.map(v, { name, deep: false });
     if (isES6Set(v))
         return observable.set(v, { name, deep: false });
-    return fail("The shallow modifier / decorator can only used in combination with arrays, objects, maps and sets");
+    return fail(process.env.NODE_ENV !== "production" &&
+        "The shallow modifier / decorator can only used in combination with arrays, objects, maps and sets");
 }
 function referenceEnhancer(newValue) {
     // never turn into an observable
     return newValue;
 }
 function refStructEnhancer(v, oldValue, name) {
-    if (isObservable(v))
+    if (process.env.NODE_ENV !== "production" && isObservable(v))
         throw `observable.struct should not be used with observable values`;
     if (deepEqual(v, oldValue))
         return oldValue;
@@ -367,7 +370,7 @@ function refStructEnhancer(v, oldValue, name) {
 function createDecoratorForEnhancer(enhancer) {
     invariant(enhancer);
     const decorator = createPropDecorator(true, (target, propertyName, descriptor, _decoratorTarget, decoratorArgs) => {
-        {
+        if (process.env.NODE_ENV !== "production") {
             invariant(!descriptor || !descriptor.get, `@observable cannot be used on getter (property "${propertyName}"), use @computed instead.`);
         }
         const initialValue = descriptor
@@ -379,7 +382,7 @@ function createDecoratorForEnhancer(enhancer) {
     });
     const res = 
     // Extra process checks, as this happens during module initialization
-    typeof process !== "undefined" && process.env && "development" !== "production"
+    typeof process !== "undefined" && process.env && process.env.NODE_ENV !== "production"
         ? function observableDecorator() {
             // This wrapper function is just to detect illegal decorator invocations, deprecate in a next version
             // and simply return the created prop decorator
@@ -415,7 +418,7 @@ function asCreateObservableOptions(thing) {
         return defaultCreateObservableOptions;
     if (typeof thing === "string")
         return { name: thing, deep: true };
-    {
+    if (process.env.NODE_ENV !== "production") {
         if (typeof thing !== "object")
             return fail("expected options object");
         Object.keys(thing).forEach(assertValidOption);
@@ -459,7 +462,8 @@ function createObservable(v, arg2, arg3) {
     if (res !== v)
         return res;
     // otherwise, just box it
-    fail(`The provided value could not be converted into an observable. If you want just create an observable reference to the object use 'observable.box(value)'`);
+    fail(process.env.NODE_ENV !== "production" &&
+        `The provided value could not be converted into an observable. If you want just create an observable reference to the object use 'observable.box(value)'`);
 }
 const observableFactories = {
     box(value, options) {
@@ -526,7 +530,7 @@ const observable = createObservable;
 Object.keys(observableFactories).forEach(name => (observable[name] = observableFactories[name]));
 function incorrectlyUsedAsDecorator(methodName) {
     fail(
-    // "development" !== "production" &&
+    // process.env.NODE_ENV !== "production" &&
     `Expected one or two arguments to observable.${methodName}. Did you accidentally try to use observable.${methodName} as decorator?`);
 }
 
@@ -553,7 +557,7 @@ const computed = function computed(arg1, arg2, arg3) {
         return computedDecorator.apply(null, arguments);
     }
     // computed(expr, options?)
-    {
+    if (process.env.NODE_ENV !== "production") {
         invariant(typeof arg1 === "function", "First argument to `computed` should be an expression.");
         invariant(arguments.length < 3, "Computed takes one or two arguments if used as function");
     }
@@ -660,7 +664,7 @@ function shouldCompute(derivation) {
 // function invariantShouldCompute(derivation: IDerivation) {
 //     const newDepState = (derivation as any).dependenciesState
 //     if (
-//         "development" === "production" &&
+//         process.env.NODE_ENV === "production" &&
 //         (newDepState === IDerivationState.POSSIBLY_STALE ||
 //             newDepState === IDerivationState.NOT_TRACKING)
 //     )
@@ -673,16 +677,19 @@ function checkIfStateModificationsAreAllowed(atom) {
     const hasObservers = atom.observers.length > 0;
     // Should never be possible to change an observed observable from inside computed, see #798
     if (globalState.computationDepth > 0 && hasObservers)
-        fail(`Computed values are not allowed to cause side effects by changing observables that are already being observed. Tried to modify: ${atom.name}`);
+        fail(process.env.NODE_ENV !== "production" &&
+            `Computed values are not allowed to cause side effects by changing observables that are already being observed. Tried to modify: ${atom.name}`);
     // Should not be possible to change observed state outside strict mode, except during initialization, see #563
     if (!globalState.allowStateChanges && (hasObservers || globalState.enforceActions === "strict"))
-        fail((globalState.enforceActions
+        fail(process.env.NODE_ENV !== "production" &&
+            (globalState.enforceActions
                 ? "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: "
                 : "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: ") +
                 atom.name);
 }
 function checkIfStateReadsAreAllowed(observable) {
-    if (!globalState.allowStateReads &&
+    if (process.env.NODE_ENV !== "production" &&
+        !globalState.allowStateReads &&
         globalState.observableRequiresReaction) {
         console.warn(`[mobx] Observable ${observable.name} being read outside a reactive context`);
     }
@@ -723,6 +730,8 @@ function trackDerivedFunction(derivation, f, context) {
     return result;
 }
 function warnAboutDerivationWithoutDependencies(derivation) {
+    if (process.env.NODE_ENV === "production")
+        return;
     if (globalState.reactionRequiresObservable || derivation.requiresObservable) {
         console.warn(`[mobx] Derivation ${derivation.name} is created/updated without reading any observable value`);
     }
@@ -837,7 +846,7 @@ let nextActionId = 1;
 const functionNameDescriptor = Object.getOwnPropertyDescriptor(() => { }, "name");
 const isFunctionNameConfigurable = functionNameDescriptor && functionNameDescriptor.configurable;
 function createAction(actionName, fn) {
-    {
+    if (process.env.NODE_ENV !== "production") {
         invariant(typeof fn === "function", "`action` can only be invoked on functions");
         if (typeof actionName !== "string" || !actionName)
             fail(`actions should have valid names, got: '${actionName}'`);
@@ -845,7 +854,7 @@ function createAction(actionName, fn) {
     const res = function () {
         return executeAction(actionName, fn, this, arguments);
     };
-    {
+    if (process.env.NODE_ENV !== "production") {
         if (isFunctionNameConfigurable) {
             Object.defineProperty(res, "name", { value: actionName });
         }
@@ -1156,7 +1165,8 @@ class ComputedValue {
             }
         }
         else
-            invariant(false, `[ComputedValue '${this.name}'] It is not possible to assign a new value to a computed value.`);
+            invariant(false, process.env.NODE_ENV !== "production" &&
+                `[ComputedValue '${this.name}'] It is not possible to assign a new value to a computed value.`);
     }
     trackAndCompute() {
         if (isSpyEnabled()) {
@@ -1229,6 +1239,8 @@ class ComputedValue {
         });
     }
     warnAboutUntrackedRead() {
+        if (process.env.NODE_ENV === "production")
+            return;
         if (this.requiresReaction === true) {
             fail(`[mobx] Computed value ${this.name} is read outside a reactive context`);
         }
@@ -1877,12 +1889,12 @@ function spy(listener) {
 }
 
 function dontReassignFields() {
-    fail("@action fields are not reassignable");
+    fail(process.env.NODE_ENV !== "production" && "@action fields are not reassignable");
 }
 function namedActionDecorator(name) {
     return function (target, prop, descriptor) {
         if (descriptor) {
-            if (descriptor.get !== undefined) {
+            if (process.env.NODE_ENV !== "production" && descriptor.get !== undefined) {
                 return fail("@action cannot be used with getters");
             }
             // babel / typescript
@@ -1983,7 +1995,7 @@ function runInAction(arg1, arg2) {
     // TODO: deprecate?
     const actionName = typeof arg1 === "string" ? arg1 : arg1.name || "<unnamed action>";
     const fn = typeof arg1 === "function" ? arg1 : arg2;
-    {
+    if (process.env.NODE_ENV !== "production") {
         invariant(typeof fn === "function" && fn.length === 0, "`runInAction` expects a function without arguments");
         if (typeof actionName !== "string" || !actionName)
             fail(`actions should have valid names, got: '${actionName}'`);
@@ -2004,7 +2016,7 @@ function defineBoundAction(target, propertyName, fn) {
  * @returns disposer function, which can be used to stop the view from being updated in the future.
  */
 function autorun(view, opts = EMPTY_OBJECT) {
-    {
+    if (process.env.NODE_ENV !== "production") {
         invariant(typeof view === "function", "Autorun expects a function as first argument");
         invariant(isAction(view) === false, "Autorun does not accept actions since actions are untrackable");
     }
@@ -2051,7 +2063,7 @@ function reaction(expression, effect, opts = EMPTY_OBJECT) {
         opts = { fireImmediately: opts };
         deprecated(`Using fireImmediately as argument is deprecated. Use '{ fireImmediately: true }' instead`);
     }
-    {
+    if (process.env.NODE_ENV !== "production") {
         invariant(typeof expression === "function", "First argument to reaction should be a function");
         invariant(typeof opts === "object", "Third argument of reactions should be an object");
     }
@@ -2116,7 +2128,7 @@ function interceptHook(hook, thing, arg2, arg3) {
     const cb = typeof arg3 === "function" ? arg3 : arg2;
     const orig = atom[hook];
     if (typeof orig !== "function")
-        return fail("Not an atom that can be (un)observed");
+        return fail(process.env.NODE_ENV !== "production" && "Not an atom that can be (un)observed");
     atom[hook] = function () {
         orig.call(this);
         cb.call(this);
@@ -2181,7 +2193,7 @@ function configure(options) {
 }
 
 function decorate(thing, decorators) {
-    if (!isPlainObject(decorators))
+    if (process.env.NODE_ENV !== "production" && !isPlainObject(decorators))
         fail("Decorators should be a key value map");
     const target = typeof thing === "function" ? thing.prototype : thing;
     for (let prop in decorators) {
@@ -2190,7 +2202,7 @@ function decorate(thing, decorators) {
             propertyDecorators = [propertyDecorators];
         }
         // prettier-ignore
-        if (!propertyDecorators.every(decorator => typeof decorator === "function"))
+        if (process.env.NODE_ENV !== "production" && !propertyDecorators.every(decorator => typeof decorator === "function"))
             fail(`Decorate: expected a decorator function or array of decorator functions for '${prop}'`);
         const descriptor = Object.getOwnPropertyDescriptor(target, prop);
         const newDescriptor = propertyDecorators.reduce((accDescriptor, decorator) => decorator(target, prop, accDescriptor), descriptor);
@@ -2205,7 +2217,7 @@ function extendShallowObservable(target, properties, decorators) {
     return extendObservable(target, properties, decorators, shallowCreateObservableOptions);
 }
 function extendObservable(target, properties, decorators, options) {
-    {
+    if (process.env.NODE_ENV !== "production") {
         invariant(arguments.length >= 2 && arguments.length <= 4, "'extendObservable' expected 2-4 arguments");
         invariant(typeof target === "object", "'extendObservable' expects an object as first argument");
         invariant(!isObservableMap(target), "'extendObservable' should not be used on maps, use map.merge instead");
@@ -2223,7 +2235,7 @@ function extendObservable(target, properties, decorators, options) {
     try {
         for (let key in properties) {
             const descriptor = Object.getOwnPropertyDescriptor(properties, key);
-            {
+            if (process.env.NODE_ENV !== "production") {
                 if (isComputed(descriptor.value))
                     fail(`Passing a 'computed' as initial property value is no longer supported by extendObservable. Use a getter or decorator instead`);
             }
@@ -2232,7 +2244,7 @@ function extendObservable(target, properties, decorators, options) {
                 : descriptor.get
                     ? computedDecorator
                     : defaultDecorator;
-            if (typeof decorator !== "function")
+            if (process.env.NODE_ENV !== "production" && typeof decorator !== "function")
                 return fail(`Not a valid decorator for '${key}', got: ${decorator}`);
             const resultDescriptor = decorator(target, key, descriptor, true);
             if (resultDescriptor // otherwise, assume already applied, due to `applyToInstance`
@@ -2279,7 +2291,7 @@ function isFlowCancellationError(error) {
 }
 function flow(generator) {
     if (arguments.length !== 1)
-        fail(`Flow expects one 1 argument and cannot be used as decorator`);
+        fail(!!process.env.NODE_ENV && `Flow expects one 1 argument and cannot be used as decorator`);
     const name = generator.name || "<unnamed flow>";
     // Implementation based on https://github.com/tj/co/blob/master/index.js
     return function () {
@@ -2359,14 +2371,16 @@ function interceptReads(thing, propOrHandler, handler) {
     }
     else if (isObservableObject(thing)) {
         if (typeof propOrHandler !== "string")
-            return fail(`InterceptReads can only be used with a specific property, not with an object in general`);
+            return fail(process.env.NODE_ENV !== "production" &&
+                `InterceptReads can only be used with a specific property, not with an object in general`);
         target = getAdministration(thing, propOrHandler);
     }
     else {
-        return fail(`Expected observable map, object or array as first array`);
+        return fail(process.env.NODE_ENV !== "production" &&
+            `Expected observable map, object or array as first array`);
     }
     if (target.dehancer !== undefined)
-        return fail(`An intercept reader was already established`);
+        return fail(process.env.NODE_ENV !== "production" && `An intercept reader was already established`);
     target.dehancer = typeof propOrHandler === "function" ? propOrHandler : handler;
     return () => {
         target.dehancer = undefined;
@@ -2401,12 +2415,14 @@ function _isComputed(value, property) {
 }
 function isComputed(value) {
     if (arguments.length > 1)
-        return fail(`isComputed expects only 1 argument. Use isObservableProp to inspect the observability of a property`);
+        return fail(process.env.NODE_ENV !== "production" &&
+            `isComputed expects only 1 argument. Use isObservableProp to inspect the observability of a property`);
     return _isComputed(value);
 }
 function isComputedProp(value, propName) {
     if (typeof propName !== "string")
-        return fail(`isComputed expected a property name as second argument`);
+        return fail(process.env.NODE_ENV !== "production" &&
+            `isComputed expected a property name as second argument`);
     return _isComputed(value, propName);
 }
 
@@ -2414,7 +2430,8 @@ function _isObservable(value, property) {
     if (value === null || value === undefined)
         return false;
     if (property !== undefined) {
-        if (isObservableMap(value) || isObservableArray(value))
+        if (process.env.NODE_ENV !== "production" &&
+            (isObservableMap(value) || isObservableArray(value)))
             return fail("isObservable(object, propertyName) is not supported for arrays and maps. Use map.has or array.length instead.");
         if (isObservableObject(value)) {
             const o = value.$mobx;
@@ -2431,12 +2448,13 @@ function _isObservable(value, property) {
 }
 function isObservable(value) {
     if (arguments.length !== 1)
-        fail(`isObservable expects only 1 argument. Use isObservableProp to inspect the observability of a property`);
+        fail(process.env.NODE_ENV !== "production" &&
+            `isObservable expects only 1 argument. Use isObservableProp to inspect the observability of a property`);
     return _isObservable(value);
 }
 function isObservableProp(value, propName) {
     if (typeof propName !== "string")
-        return fail(`expected a property name as second argument`);
+        return fail(process.env.NODE_ENV !== "production" && `expected a property name as second argument`);
     return _isObservable(value, propName);
 }
 
@@ -2453,7 +2471,8 @@ function keys(obj) {
     if (isObservableArray(obj)) {
         return obj.map((_, index) => index);
     }
-    return fail("'keys()' can only be used on observable objects, arrays, sets and maps");
+    return fail(process.env.NODE_ENV !== "production" &&
+        "'keys()' can only be used on observable objects, arrays, sets and maps");
 }
 function values(obj) {
     if (isObservableObject(obj)) {
@@ -2468,7 +2487,8 @@ function values(obj) {
     if (isObservableArray(obj)) {
         return obj.slice();
     }
-    return fail("'values()' can only be used on observable objects, arrays, sets and maps");
+    return fail(process.env.NODE_ENV !== "production" &&
+        "'values()' can only be used on observable objects, arrays, sets and maps");
 }
 function entries(obj) {
     if (isObservableObject(obj)) {
@@ -2483,7 +2503,8 @@ function entries(obj) {
     if (isObservableArray(obj)) {
         return obj.map((key, index) => [index, key]);
     }
-    return fail("'entries()' can only be used on observable objects, arrays and maps");
+    return fail(process.env.NODE_ENV !== "production" &&
+        "'entries()' can only be used on observable objects, arrays and maps");
 }
 function set(obj, key, value) {
     if (arguments.length === 2 && !isObservableSet(obj)) {
@@ -2525,7 +2546,8 @@ function set(obj, key, value) {
         endBatch();
     }
     else {
-        return fail("'set()' can only be used on observable objects, arrays and maps");
+        return fail(process.env.NODE_ENV !== "production" &&
+            "'set()' can only be used on observable objects, arrays and maps");
     }
 }
 function remove(obj, key) {
@@ -2545,7 +2567,8 @@ function remove(obj, key) {
         obj.splice(key, 1);
     }
     else {
-        return fail("'remove()' can only be used on observable objects, arrays and maps");
+        return fail(process.env.NODE_ENV !== "production" &&
+            "'remove()' can only be used on observable objects, arrays and maps");
     }
 }
 function has(obj, key) {
@@ -2565,7 +2588,8 @@ function has(obj, key) {
         return key >= 0 && key < obj.length;
     }
     else {
-        return fail("'has()' can only be used on observable objects, arrays and maps");
+        return fail(process.env.NODE_ENV !== "production" &&
+            "'has()' can only be used on observable objects, arrays and maps");
     }
 }
 function get(obj, key) {
@@ -2581,7 +2605,8 @@ function get(obj, key) {
         return obj[key];
     }
     else {
-        return fail("'get()' can only be used on observable objects, arrays and maps");
+        return fail(process.env.NODE_ENV !== "production" &&
+            "'get()' can only be used on observable objects, arrays and maps");
     }
 }
 
@@ -2697,7 +2722,8 @@ function trace(...args) {
         enterBreakPoint = args.pop();
     const derivation = getAtomFromArgs(args);
     if (!derivation) {
-        return fail(`'trace(break?)' can only be used inside a tracked computed value or a Reaction. Consider passing in the computed value or reaction explicitly`);
+        return fail(process.env.NODE_ENV !== "production" &&
+            `'trace(break?)' can only be used inside a tracked computed value or a Reaction. Consider passing in the computed value or reaction explicitly`);
     }
     if (derivation.isTracing === TraceMode.NONE) {
         console.log(`[mobx.trace] '${derivation.name}' tracing enabled`);
@@ -2764,7 +2790,7 @@ function _when(predicate, effect, opts) {
     return disposer;
 }
 function whenPromise(predicate, opts) {
-    if (opts && opts.onError)
+    if (process.env.NODE_ENV !== "production" && opts && opts.onError)
         return fail(`the options 'onError' and 'promise' cannot be combined`);
     let cancel;
     const res = new Promise((resolve, reject) => {
@@ -3641,7 +3667,8 @@ class ObservableMap {
      * for callback details
      */
     observe(listener, fireImmediately) {
-        invariant(fireImmediately !== true, "`observe` doesn't support fireImmediately=true in combination with maps.");
+        process.env.NODE_ENV !== "production" &&
+            invariant(fireImmediately !== true, "`observe` doesn't support fireImmediately=true in combination with maps.");
         return registerListener(this, listener);
     }
     intercept(handler) {
@@ -3727,11 +3754,11 @@ class ObservableSet {
                     newValue: value
                 }
                 : null;
-            if (notifySpy && "development" !== "production")
+            if (notifySpy && process.env.NODE_ENV !== "production")
                 spyReportStart(change);
             if (notify)
                 notifyListeners(this, change);
-            if (notifySpy && "development" !== "production")
+            if (notifySpy && process.env.NODE_ENV !== "production")
                 spyReportEnd();
         }
         return this;
@@ -3756,7 +3783,7 @@ class ObservableSet {
                     oldValue: value
                 }
                 : null;
-            if (notifySpy && "development" !== "production")
+            if (notifySpy && process.env.NODE_ENV !== "production")
                 spyReportStart(Object.assign(Object.assign({}, change), { name: this.name }));
             transaction(() => {
                 this._atom.reportChanged();
@@ -3764,7 +3791,7 @@ class ObservableSet {
             });
             if (notify)
                 notifyListeners(this, change);
-            if (notifySpy && "development" !== "production")
+            if (notifySpy && process.env.NODE_ENV !== "production")
                 spyReportEnd();
             return true;
         }
@@ -3833,7 +3860,8 @@ class ObservableSet {
     }
     observe(listener, fireImmediately) {
         // TODO 'fireImmediately' can be true?
-        invariant(fireImmediately !== true, "`observe` doesn't support fireImmediately=true in combination with sets.");
+        process.env.NODE_ENV !== "production" &&
+            invariant(fireImmediately !== true, "`observe` doesn't support fireImmediately=true in combination with sets.");
         return registerListener(this, listener);
     }
     intercept(handler) {
@@ -3860,10 +3888,18 @@ class ObservableObjectAdministration {
         this.values = {};
     }
     read(owner, key) {
+        if (process.env.NODE_ENV === "production" && this.target !== owner) {
+            this.illegalAccess(owner, key);
+            if (!this.values[key])
+                return undefined;
+        }
         return this.values[key].get();
     }
     write(owner, key, newValue) {
         const instance = this.target;
+        if (process.env.NODE_ENV === "production" && instance !== owner) {
+            this.illegalAccess(owner, key);
+        }
         const observable = this.values[key];
         if (observable instanceof ComputedValue) {
             observable.set(newValue);
@@ -3973,7 +4009,8 @@ class ObservableObjectAdministration {
      * for callback details
      */
     observe(callback, fireImmediately) {
-        invariant(fireImmediately !== true, "`observe` doesn't support the fire immediately property for observable objects.");
+        process.env.NODE_ENV !== "production" &&
+            invariant(fireImmediately !== true, "`observe` doesn't support the fire immediately property for observable objects.");
         return registerListener(this, callback);
     }
     intercept(handler) {
@@ -3990,7 +4027,8 @@ function asObservableObject(target, name = "", defaultEnhancer = deepEnhancer) {
     let adm = target.$mobx;
     if (adm)
         return adm;
-    invariant(Object.isExtensible(target), "Cannot make the designated object observable; it is not extensible");
+    process.env.NODE_ENV !== "production" &&
+        invariant(Object.isExtensible(target), "Cannot make the designated object observable; it is not extensible");
     if (!isPlainObject(target))
         name = (target.constructor.name || "ObservableObject") + "@" + getNextId();
     if (!name)
@@ -4098,7 +4136,8 @@ function getAtom(thing, property) {
     if (typeof thing === "object" && thing !== null) {
         if (isObservableArray(thing)) {
             if (property !== undefined)
-                fail("It is not possible to get index atoms from arrays");
+                fail(process.env.NODE_ENV !== "production" &&
+                    "It is not possible to get index atoms from arrays");
             return thing.$mobx.atom;
         }
         if (isObservableSet(thing)) {
@@ -4110,7 +4149,8 @@ function getAtom(thing, property) {
                 return getAtom(anyThing._keys);
             const observable = anyThing._data.get(property) || anyThing._hasMap.get(property);
             if (!observable)
-                fail(`the entry '${property}' does not exist in the observable map '${getDebugName(thing)}'`);
+                fail(process.env.NODE_ENV !== "production" &&
+                    `the entry '${property}' does not exist in the observable map '${getDebugName(thing)}'`);
             return observable;
         }
         // Initializers run lazily when transpiling to babel, so make sure they are run...
@@ -4119,10 +4159,11 @@ function getAtom(thing, property) {
             thing[property]; // See #1072
         if (isObservableObject(thing)) {
             if (!property)
-                return fail(`please specify a property`);
+                return fail(process.env.NODE_ENV !== "production" && `please specify a property`);
             const observable = thing.$mobx.values[property];
             if (!observable)
-                fail(`no observable property '${property}' found on the observable object '${getDebugName(thing)}'`);
+                fail(process.env.NODE_ENV !== "production" &&
+                    `no observable property '${property}' found on the observable object '${getDebugName(thing)}'`);
             return observable;
         }
         if (isAtom(thing) || isComputedValue(thing) || isReaction(thing)) {
@@ -4135,7 +4176,7 @@ function getAtom(thing, property) {
             return thing.$mobx;
         }
     }
-    return fail("Cannot obtain atom from " + thing);
+    return fail(process.env.NODE_ENV !== "production" && "Cannot obtain atom from " + thing);
 }
 function getAdministration(thing, property) {
     if (!thing)
@@ -4150,7 +4191,7 @@ function getAdministration(thing, property) {
     initializeInstance(thing);
     if (thing.$mobx)
         return thing.$mobx;
-    fail("Cannot obtain administration from " + thing);
+    fail(process.env.NODE_ENV !== "production" && "Cannot obtain administration from " + thing);
 }
 function getDebugName(thing, property) {
     let named;
@@ -4326,6 +4367,11 @@ but at least in this file we can magically reorder the imports with trial and er
  *
  */
 try {
+    // define process.env if needed
+    // if this is not a production build in the first place
+    // (in which case the expression below would be substituted with 'production')
+    // tslint:disable-next-line
+    process.env.NODE_ENV;
 }
 catch (e) {
     const g = getGlobal();
@@ -4336,8 +4382,8 @@ catch (e) {
 (() => {
     function testCodeMinification() { }
     if (testCodeMinification.name !== "testCodeMinification" &&
-        "development" !== "production" &&
-        process.env.IGNORE_MOBX_MINIFY_WARNING !== "true") {
+        process.env.NODE_ENV !== "production" &&
+        typeof process !== 'undefined' && process.env.IGNORE_MOBX_MINIFY_WARNING !== "true") {
         // trick so it doesn't get replaced
         const varName = ["process", "env", "NODE_ENV"].join(".");
         console.warn(`[mobx] you are running a minified build, but '${varName}' was not set to 'production' in your bundler. This results in an unnecessarily large and slow bundle`);
@@ -4356,7 +4402,8 @@ if (typeof __MOBX_DEVTOOLS_GLOBAL_HOOK__ === "object") {
     });
 }
 // TODO: remove in some future build
-if (typeof module !== "undefined" &&
+if (process.env.NODE_ENV !== "production" &&
+    typeof module !== "undefined" &&
     typeof module.exports !== "undefined") {
     let warnedAboutDefaultExport = false;
     Object.defineProperty(module.exports, "default", {

package/lib/mobx.js

@@ -4480,7 +4480,7 @@ catch (e) {
     function testCodeMinification() { }
     if (testCodeMinification.name !== "testCodeMinification" &&
         "development" !== "production" &&
-        process.env.IGNORE_MOBX_MINIFY_WARNING !== "true") {
+        typeof process !== 'undefined' && process.env.IGNORE_MOBX_MINIFY_WARNING !== "true") {
         // trick so it doesn't get replaced
         var varName = ["process", "env", "NODE_ENV"].join(".");
         console.warn("[mobx] you are running a minified build, but '" + varName + "' was not set to 'production' in your bundler. This results in an unnecessarily large and slow bundle");