mobx-state-tree 5.3.1-alpha.1 → 5.4.0

package/dist/mobx-state-tree.module.js

@@ -1458,6 +1458,12 @@ var ObjectNode = /** @class */ (function (_super) {
             writable: true,
             value: void 0
         });
+        Object.defineProperty(_this, "hasSnapshotPostProcessor", {
+            enumerable: true,
+            configurable: true,
+            writable: true,
+            value: false
+        });
         Object.defineProperty(_this, "_applyPatches", {
             enumerable: true,
             configurable: true,
@@ -1788,6 +1794,9 @@ var ObjectNode = /** @class */ (function (_super) {
     Object.defineProperty(ObjectNode.prototype, "snapshot", {
         // advantage of using computed for a snapshot is that nicely respects transactions etc.
         get: function () {
+            if (this.hasSnapshotPostProcessor) {
+                this.createObservableInstanceIfNeeded();
+            }
             return this._snapshotComputed.get();
         },
         enumerable: false,
@@ -4700,9 +4709,9 @@ var SnapshotProcessor = /** @class */ (function (_super) {
         enumerable: false,
         configurable: true,
         writable: true,
-        value: function (sn) {
+        value: function (sn, node) {
             if (this._processors.postProcessor) {
-                return this._processors.postProcessor.call(null, sn);
+                return this._processors.postProcessor.call(null, sn, node.storedValue);
             }
             return sn;
         }
@@ -4715,10 +4724,11 @@ var SnapshotProcessor = /** @class */ (function (_super) {
             var _this = this;
             // the node has to use these methods rather than the original type ones
             proxyNodeTypeMethods(node.type, this, "create");
+            if (node instanceof ObjectNode) {
+                node.hasSnapshotPostProcessor = !!this._processors.postProcessor;
+            }
             var oldGetSnapshot = node.getSnapshot;
-            node.getSnapshot = function () {
-                return _this.postProcessSnapshot(oldGetSnapshot.call(node));
-            };
+            node.getSnapshot = function () { return _this.postProcessSnapshot(oldGetSnapshot.call(node), node); };
             if (!isUnionType(this._subtype)) {
                 node.getReconciliationType = function () {
                     return _this;
@@ -4758,7 +4768,7 @@ var SnapshotProcessor = /** @class */ (function (_super) {
         value: function (node, applyPostProcess) {
             if (applyPostProcess === void 0) { applyPostProcess = true; }
             var sn = this._subtype.getSnapshot(node);
-            return applyPostProcess ? this.postProcessSnapshot(sn) : sn;
+            return applyPostProcess ? this.postProcessSnapshot(sn, node) : sn;
         }
     });
     Object.defineProperty(SnapshotProcessor.prototype, "isValidSnapshot", {
@@ -4849,15 +4859,17 @@ function proxyNodeTypeMethods(nodeType, snapshotProcessorType) {
  * interface BackendTodo {
  *     text: string | null
  * }
+ *
  * const Todo2 = types.snapshotProcessor(Todo1, {
  *     // from snapshot to instance
- *     preProcessor(sn: BackendTodo) {
+ *     preProcessor(snapshot: BackendTodo) {
  *         return {
  *             text: sn.text || "";
  *         }
  *     },
+ *
  *     // from instance to snapshot
- *     postProcessor(sn): BackendTodo {
+ *     postProcessor(snapshot, node): BackendTodo {
  *         return {
  *             text: !sn.text ? null : sn.text
  *         }
@@ -5855,27 +5867,34 @@ function toPropertiesObject(declaredProps) {
         if (key in Hook) {
             throw fail$1("Hook '" + key + "' was defined as property. Hooks should be defined as part of the actions");
         }
+        // the user intended to use a view
         var descriptor = Object.getOwnPropertyDescriptor(declaredProps, key);
         if ("get" in descriptor) {
             throw fail$1("Getters are not supported as properties. Please use views instead");
         }
+        // undefined and null are not valid
         var value = descriptor.value;
         if (value === null || value === undefined) {
             throw fail$1("The default value of an attribute cannot be null or undefined as the type cannot be inferred. Did you mean `types.maybe(someType)`?");
         }
+        // its a primitive, convert to its type
         else if (isPrimitive(value)) {
             props[key] = optional(getPrimitiveFactoryFromValue(value), value);
         }
+        // map defaults to empty object automatically for models
         else if (value instanceof MapType) {
             props[key] = optional(value, {});
         }
         else if (value instanceof ArrayType) {
             props[key] = optional(value, []);
         }
+        // its already a type
         else if (isType(value)) ;
+        // its a function, maybe the user wanted a view?
         else if (devMode() && typeof value === "function") {
             throw fail$1("Invalid type definition for property '" + key + "', it looks like you passed a function. Did you forget to invoke it, or did you intend to declare a view / action?");
         }
+        // no other complex values
         else if (devMode() && typeof value === "object") {
             throw fail$1("Invalid type definition for property '" + key + "', it looks like you passed an object. Try passing another model type or a types.frozen.");
         }
@@ -8524,4 +8543,4 @@ var types = {
     snapshotProcessor: snapshotProcessor
 };
 
-export { addDisposer, addMiddleware, applyAction, applyPatch, applySnapshot, cast, castFlowReturn, castToReferenceSnapshot, castToSnapshot, clone, createActionTrackingMiddleware, createActionTrackingMiddleware2, decorate, destroy, detach, escapeJsonPath, flow, getChildType, getEnv, getIdentifier, getLivelinessChecking, getMembers, getNodeId, getParent, getParentOfType, getPath, getPathParts, getPropertyMembers, getRelativePath, getRoot, getRunningActionContext, getSnapshot, getType, hasParent, hasParentOfType, isActionContextChildOf, isActionContextThisOrChildOf, isAlive, isArrayType, isFrozenType, isIdentifierType, isLateType, isLiteralType, isMapType, isModelType, isOptionalType, isPrimitiveType, isProtected, isReferenceType, isRefinementType, isRoot, isStateTreeNode, isType, isUnionType, isValidReference, joinJsonPath, onAction, onPatch, onSnapshot, process$1 as process, protect, recordActions, recordPatches, resolveIdentifier, resolvePath, setLivelinessChecking, setLivelynessChecking, splitJsonPath, toGenerator, toGeneratorFunction, tryReference, tryResolve, typecheck, types, unescapeJsonPath, unprotect, walk };
+export { addDisposer, addMiddleware, applyAction, applyPatch, applySnapshot, cast, castFlowReturn, castToReferenceSnapshot, castToSnapshot, clone, createActionTrackingMiddleware, createActionTrackingMiddleware2, decorate, destroy, detach, escapeJsonPath, flow, getChildType, getEnv, getIdentifier, getLivelinessChecking, getMembers, getNodeId, getParent, getParentOfType, getPath, getPathParts, getPropertyMembers, getRelativePath, getRoot, getRunningActionContext, getSnapshot, getType, hasParent, hasParentOfType, isActionContextChildOf, isActionContextThisOrChildOf, isAlive, isArrayType, isFrozenType, isIdentifierType, isLateType, isLiteralType, isMapType, isModelType, isOptionalType, isPrimitiveType, isProtected, isReferenceType, isRefinementType, isRoot, isStateTreeNode, isType, isUnionType, isValidReference, joinJsonPath, onAction, onPatch, onSnapshot, process$1 as process, protect, recordActions, recordPatches, resolveIdentifier, resolvePath, setLivelinessChecking, setLivelynessChecking, splitJsonPath, types as t, toGenerator, toGeneratorFunction, tryReference, tryResolve, typecheck, types, unescapeJsonPath, unprotect, walk };

package/dist/mobx-state-tree.umd.js

@@ -1462,6 +1462,12 @@
                 writable: true,
                 value: void 0
             });
+            Object.defineProperty(_this, "hasSnapshotPostProcessor", {
+                enumerable: true,
+                configurable: true,
+                writable: true,
+                value: false
+            });
             Object.defineProperty(_this, "_applyPatches", {
                 enumerable: true,
                 configurable: true,
@@ -1792,6 +1798,9 @@
         Object.defineProperty(ObjectNode.prototype, "snapshot", {
             // advantage of using computed for a snapshot is that nicely respects transactions etc.
             get: function () {
+                if (this.hasSnapshotPostProcessor) {
+                    this.createObservableInstanceIfNeeded();
+                }
                 return this._snapshotComputed.get();
             },
             enumerable: false,
@@ -4682,9 +4691,9 @@
             enumerable: false,
             configurable: true,
             writable: true,
-            value: function (sn) {
+            value: function (sn, node) {
                 if (this._processors.postProcessor) {
-                    return this._processors.postProcessor.call(null, sn);
+                    return this._processors.postProcessor.call(null, sn, node.storedValue);
                 }
                 return sn;
             }
@@ -4697,10 +4706,11 @@
                 var _this = this;
                 // the node has to use these methods rather than the original type ones
                 proxyNodeTypeMethods(node.type, this, "create");
+                if (node instanceof ObjectNode) {
+                    node.hasSnapshotPostProcessor = !!this._processors.postProcessor;
+                }
                 var oldGetSnapshot = node.getSnapshot;
-                node.getSnapshot = function () {
-                    return _this.postProcessSnapshot(oldGetSnapshot.call(node));
-                };
+                node.getSnapshot = function () { return _this.postProcessSnapshot(oldGetSnapshot.call(node), node); };
                 if (!isUnionType(this._subtype)) {
                     node.getReconciliationType = function () {
                         return _this;
@@ -4740,7 +4750,7 @@
             value: function (node, applyPostProcess) {
                 if (applyPostProcess === void 0) { applyPostProcess = true; }
                 var sn = this._subtype.getSnapshot(node);
-                return applyPostProcess ? this.postProcessSnapshot(sn) : sn;
+                return applyPostProcess ? this.postProcessSnapshot(sn, node) : sn;
             }
         });
         Object.defineProperty(SnapshotProcessor.prototype, "isValidSnapshot", {
@@ -4831,15 +4841,17 @@
      * interface BackendTodo {
      *     text: string | null
      * }
+     *
      * const Todo2 = types.snapshotProcessor(Todo1, {
      *     // from snapshot to instance
-     *     preProcessor(sn: BackendTodo) {
+     *     preProcessor(snapshot: BackendTodo) {
      *         return {
      *             text: sn.text || "";
      *         }
      *     },
+     *
      *     // from instance to snapshot
-     *     postProcessor(sn): BackendTodo {
+     *     postProcessor(snapshot, node): BackendTodo {
      *         return {
      *             text: !sn.text ? null : sn.text
      *         }
@@ -5837,27 +5849,34 @@
             if (key in Hook) {
                 throw fail$1("Hook '" + key + "' was defined as property. Hooks should be defined as part of the actions");
             }
+            // the user intended to use a view
             var descriptor = Object.getOwnPropertyDescriptor(declaredProps, key);
             if ("get" in descriptor) {
                 throw fail$1("Getters are not supported as properties. Please use views instead");
             }
+            // undefined and null are not valid
             var value = descriptor.value;
             if (value === null || value === undefined) {
                 throw fail$1("The default value of an attribute cannot be null or undefined as the type cannot be inferred. Did you mean `types.maybe(someType)`?");
             }
+            // its a primitive, convert to its type
             else if (isPrimitive(value)) {
                 props[key] = optional(getPrimitiveFactoryFromValue(value), value);
             }
+            // map defaults to empty object automatically for models
             else if (value instanceof MapType) {
                 props[key] = optional(value, {});
             }
             else if (value instanceof ArrayType) {
                 props[key] = optional(value, []);
             }
+            // its already a type
             else if (isType(value)) ;
+            // its a function, maybe the user wanted a view?
             else if (typeof value === "function") {
                 throw fail$1("Invalid type definition for property '" + key + "', it looks like you passed a function. Did you forget to invoke it, or did you intend to declare a view / action?");
             }
+            // no other complex values
             else if (typeof value === "object") {
                 throw fail$1("Invalid type definition for property '" + key + "', it looks like you passed an object. Try passing another model type or a types.frozen.");
             }
@@ -8574,6 +8593,7 @@
     exports.setLivelinessChecking = setLivelinessChecking;
     exports.setLivelynessChecking = setLivelynessChecking;
     exports.splitJsonPath = splitJsonPath;
+    exports.t = types;
     exports.toGenerator = toGenerator;
     exports.toGeneratorFunction = toGeneratorFunction;
     exports.tryReference = tryReference;