@jbrowse/mobx-state-tree 5.10.6 → 5.10.8

package/dist/mobx-state-tree.mjs

@@ -978,7 +978,7 @@ class BaseNode {
         if (this._escapedSubpath === undefined) {
             this._escapedSubpath = !this._subpath ? "" : escapeJsonPath(this._subpath);
         }
-        return (this.parent.getEscapedPath(reportObserved) + "/" + this._escapedSubpath);
+        return `${this.parent.getEscapedPath(reportObserved)}/${this._escapedSubpath}`;
     }
     get isRoot() {
         return this.parent === null;
@@ -1212,9 +1212,8 @@ class ObjectNode extends BaseNode {
         // uninitialized parent
         // this is done to avoid traversing the whole tree to the root when using
         // the same reference again
-        while (parent &&
-            parent._observableInstanceState ===
-                0 /* ObservableInstanceLifecycle.UNINITIALIZED */) {
+        while (parent?._observableInstanceState ===
+            0 /* ObservableInstanceLifecycle.UNINITIALIZED */) {
             parentChain.unshift(parent);
             parent = parent.parent;
         }
@@ -1402,12 +1401,9 @@ class ObjectNode extends BaseNode {
             actionContext = actionContext.parentActionEvent;
         }
         let actionFullPath = "";
-        if (actionContext && actionContext.name != null) {
+        if (actionContext?.name != null) {
             // try to use the context, and if it not available use the node one
-            const actionPath = (actionContext &&
-                actionContext.context &&
-                getPath(actionContext.context)) ||
-                escapedPath;
+            const actionPath = (actionContext.context && getPath(actionContext.context)) || escapedPath;
             actionFullPath = `${actionPath}.${actionContext.name}()`;
         }
         return `You are trying to read or write to an object that is no longer part of a state tree. (Object type: '${this.type.name}', Path upon death: '${escapedPath}', Subpath: '${subpath}', Action: '${actionFullPath}'). Either detach nodes first, or don't use objects after removing / replacing them in the tree.`;
@@ -1554,9 +1550,10 @@ class ObjectNode extends BaseNode {
     }
     emitPatch(basePatch, source) {
         if (this._internalEventsHasSubscribers("patch" /* InternalEvents.Patch */)) {
-            const localizedPatch = extend({}, basePatch, {
-                path: source.path.substr(this.path.length) + "/" + basePatch.path // calculate the relative path of the patch
-            });
+            const localizedPatch = {
+                ...basePatch,
+                path: `${source.path.substr(this.path.length)}/${basePatch.path}` // calculate the relative path of the patch
+            };
             const [patch, reversePatch] = splitPatch(localizedPatch);
             this._internalEventsEmit("patch" /* InternalEvents.Patch */, patch, reversePatch);
         }
@@ -1885,7 +1882,7 @@ class SimpleType extends BaseType {
  * @returns `true` if the value is a type.
  */
 function isType(value) {
-    return typeof value === "object" && value && value.isType === true;
+    return typeof value === "object" && value?.isType === true;
 }
 /**
  * @internal
@@ -2037,8 +2034,8 @@ function createActionTrackingMiddleware2(middlewareHooks) {
             const newCall = {
                 ...call,
                 // make a shallow copy of the parent action env
-                env: parentRunningAction && parentRunningAction.call.env,
-                parentCall: parentRunningAction && parentRunningAction.call
+                env: parentRunningAction?.call.env,
+                parentCall: parentRunningAction?.call
             };
             const passesFilter = !middlewareHooks.filter || middlewareHooks.filter(newCall);
             const hooks = passesFilter ? middlewareHooks : undefined;
@@ -2119,8 +2116,7 @@ function serializeArgument(node, actionName, index, arg) {
         return serializeTheUnserializable(`[function]`);
     }
     if (typeof arg === "object" && !isPlainObject(arg) && !isArray(arg)) {
-        return serializeTheUnserializable(`[object ${(arg && arg.constructor && arg.constructor.name) ||
-            "Complex Object"}]`);
+        return serializeTheUnserializable(`[object ${arg?.constructor?.name || "Complex Object"}]`);
     }
     try {
         // Check if serializable, cycle free etc...
@@ -2129,7 +2125,7 @@ function serializeArgument(node, actionName, index, arg) {
         return arg;
     }
     catch (e) {
-        return serializeTheUnserializable("" + e);
+        return serializeTheUnserializable(`${e}`);
     }
 }
 function deserializeArgument(adm, value) {
@@ -2489,7 +2485,7 @@ function runMiddleWares(node, baseCall, originalFn) {
     let result = null;
     function runNextMiddleware(call) {
         const middleware = middlewares.getNextMiddleware();
-        const handler = middleware && middleware.handler;
+        const handler = middleware?.handler;
         if (!handler) {
             return action(originalFn)(...call.args);
         }
@@ -2597,7 +2593,7 @@ function safeStringify(value) {
  */
 function prettyPrintValue(value) {
     return typeof value === "function"
-        ? `<function${value.name ? " " + value.name : ""}>`
+        ? `<function${value.name ? ` ${value.name}` : ""}>`
         : isStateTreeNode(value)
             ? `<${value}>`
             : `\`${safeStringify(value)}\``;
@@ -2621,16 +2617,13 @@ function toErrorString(error) {
             ? "value"
             : "snapshot";
     const isSnapshotCompatible = type && isStateTreeNode(value) && type.is(getStateTreeNode(value).snapshot);
-    return (`${pathPrefix}${currentTypename} ${shortenPrintValue(prettyPrintValue(value))} is not assignable ${type ? `to type: \`${type.name}\`` : ``}` +
-        (error.message ? ` (${error.message})` : "") +
-        (type
-            ? isPrimitiveType(type) || isPrimitive(value)
-                ? `.`
-                : `, expected an instance of \`${type.name}\` or a snapshot like \`${shortenPrintValue(type.describe())}\` instead.` +
-                    (isSnapshotCompatible
-                        ? " (Note that a snapshot of the provided value is compatible with the targeted type)"
-                        : "")
-            : `.`));
+    return `${pathPrefix}${currentTypename} ${shortenPrintValue(prettyPrintValue(value))} is not assignable ${type ? `to type: \`${type.name}\`` : ``}${error.message ? ` (${error.message})` : ""}${type
+        ? isPrimitiveType(type) || isPrimitive(value)
+            ? `.`
+            : `, expected an instance of \`${type.name}\` or a snapshot like \`${shortenPrintValue(type.describe())}\` instead.${isSnapshotCompatible
+                ? " (Note that a snapshot of the provided value is compatible with the targeted type)"
+                : ""}`
+        : `.`}`;
 }
 /**
  * @internal
@@ -2719,7 +2712,7 @@ function validationErrorsToString(type, value, errors) {
     if (errors.length === 0) {
         return undefined;
     }
-    return (`Error while converting ${shortenPrintValue(prettyPrintValue(value))} to \`${type.name}\`:\n\n    ` + formatValidationErrorLines(errors).join("\n    "));
+    return `Error while converting ${shortenPrintValue(prettyPrintValue(value))} to \`${type.name}\`:\n\n    ${formatValidationErrorLines(errors).join("\n    ")}`;
 }
 
 let identifierCacheId = 0;
@@ -2751,7 +2744,7 @@ class IdentifierCache {
                 this.cache.set(identifier, observable.array([], mobxShallow));
             }
             const set = this.cache.get(identifier);
-            if (set.indexOf(node) !== -1) {
+            if (set.includes(node)) {
                 throw fail(`Already registered`);
             }
             set.push(node);
@@ -2784,12 +2777,12 @@ class IdentifierCache {
         // The slash is added here so we only match children of the splitNode. In version 5.1.8 and
         // earlier there was no trailing slash, so non children that started with the same path string
         // were being matched incorrectly.
-        const basePath = splitNode.path + "/";
+        const basePath = `${splitNode.path}/`;
         entries(this.cache).forEach(([id, nodes]) => {
             let modified = false;
             for (let i = nodes.length - 1; i >= 0; i--) {
                 const node = nodes[i];
-                if (node === splitNode || node.path.indexOf(basePath) === 0) {
+                if (node === splitNode || node.path.startsWith(basePath)) {
                     newCache.addNodeToCache(node, false); // no need to update lastUpdated since it is a whole new cache
                     nodes.splice(i, 1);
                     // remove empty sets from cache
@@ -2887,7 +2880,7 @@ var NodeLifeCycle;
  * @returns true if the value is a state tree node.
  */
 function isStateTreeNode(value) {
-    return !!(value && value.$treenode);
+    return !!value?.$treenode;
 }
 /**
  * @internal
@@ -2912,7 +2905,7 @@ function getStateTreeNode(value) {
  * @hidden
  */
 function getStateTreeNodeSafe(value) {
-    return (value && value.$treenode) || null;
+    return value?.$treenode || null;
 }
 /**
  * @internal
@@ -3039,7 +3032,7 @@ const DEPRECATION_MESSAGE = "See https://github.com/mobxjs/mobx-state-tree/issue
  * @returns {Promise}
  */
 function process$1(asyncAction) {
-    deprecated("process", "`process()` has been renamed to `flow()`. " + DEPRECATION_MESSAGE);
+    deprecated("process", `\`process()\` has been renamed to \`flow()\`. ${DEPRECATION_MESSAGE}`);
     return flow(asyncAction);
 }
 
@@ -3067,7 +3060,7 @@ Object.freeze(mobxShallow);
  * @hidden
  */
 function fail(message = "Illegal state") {
-    return new Error("[mobx-state-tree] " + message);
+    return new Error(`[mobx-state-tree] ${message}`);
 }
 /**
  * @internal
@@ -3115,19 +3108,6 @@ function asArray(val) {
     }
     return [val];
 }
-/**
- * @internal
- * @hidden
- */
-function extend(a, ...b) {
-    for (let i = 0; i < b.length; i++) {
-        const current = b[i];
-        for (const key in current) {
-            a[key] = current[key];
-        }
-    }
-    return a;
-}
 /**
  * @internal
  * @hidden
@@ -3261,7 +3241,7 @@ class EventHandler {
         };
     }
     has(fn) {
-        return this.handlers.indexOf(fn) >= 0;
+        return this.handlers.includes(fn);
     }
     unregister(fn) {
         if (this.emitting) {
@@ -3310,8 +3290,7 @@ class EventHandler {
 class EventHandlers {
     eventHandlers;
     hasSubscribers(event) {
-        const handler = this.eventHandlers && this.eventHandlers[event];
-        return !!handler && handler.hasSubscribers;
+        return this.eventHandlers?.[event]?.hasSubscribers ?? false;
     }
     register(event, fn, atTheBeginning = false) {
         if (!this.eventHandlers) {
@@ -3324,11 +3303,10 @@ class EventHandlers {
         return handler.register(fn, atTheBeginning);
     }
     has(event, fn) {
-        const handler = this.eventHandlers && this.eventHandlers[event];
-        return !!handler && handler.has(fn);
+        return this.eventHandlers?.[event]?.has(fn) ?? false;
     }
     unregister(event, fn) {
-        const handler = this.eventHandlers && this.eventHandlers[event];
+        const handler = this.eventHandlers?.[event];
         if (handler) {
             handler.unregister(fn);
         }
@@ -3342,19 +3320,12 @@ class EventHandlers {
         this.eventHandlers = undefined;
     }
     emit(event, ...args) {
-        const handler = this.eventHandlers && this.eventHandlers[event];
+        const handler = this.eventHandlers?.[event];
         if (handler) {
             handler.emit(...args);
         }
     }
 }
-/**
- * @internal
- * @hidden
- */
-function stringStartsWith(str, beginning) {
-    return str.indexOf(beginning) === 0;
-}
 /**
  * @internal
  * @hidden
@@ -3366,7 +3337,7 @@ const deprecated = function (id, message) {
     }
     // warn if hasn't been warned before
     if (deprecated.ids && !deprecated.ids.hasOwnProperty(id)) {
-        warnError("Deprecation warning: " + message);
+        warnError(`Deprecation warning: ${message}`);
     }
     // mark as warned to avoid duplicate warn message
     if (deprecated.ids) {
@@ -3381,15 +3352,27 @@ deprecated.ids = {};
 function warnError(msg) {
     console.warn(new Error(`[mobx-state-tree] ${msg}`));
 }
+let _typeChecking;
+/**
+ * Forces full run-time type-checking on or off, overriding the dev-mode and
+ * `ENABLE_TYPE_CHECK` env-var defaults. Pass `true` to get better error messages
+ * in production builds without relying on an environment variable, `false` to
+ * force it off, or `undefined` to restore the default behavior.
+ */
+function setTypeChecking(enabled) {
+    _typeChecking = enabled;
+}
 /**
  * @internal
  * @hidden
  */
 function isTypeCheckingEnabled() {
-    return (devMode() ||
-        (typeof process !== "undefined" &&
-            process.env &&
-            process.env.ENABLE_TYPE_CHECK === "true"));
+    // an explicit setTypeChecking() override (incl. `false`) wins over the
+    // dev-mode / env-var default, hence ?? rather than ||
+    return (_typeChecking ??
+        (devMode() ||
+            (typeof process !== "undefined" &&
+                process.env?.ENABLE_TYPE_CHECK === "true")));
 }
 let _devMode = process.env.NODE_ENV !== "production";
 /**
@@ -3623,7 +3606,7 @@ function createFlowSpawner(name, generator) {
                 // TODO: support more type of values? See https://github.com/tj/co/blob/249bbdc72da24ae44076afd716349d2089b31c4c/index.js#L100
                 if (!ret.value || typeof ret.value.then !== "function") {
                     // istanbul ignore next
-                    throw fail("Only promises can be yielded to `async`, got: " + ret);
+                    throw fail(`Only promises can be yielded to \`async\`, got: ${ret}`);
                 }
                 return ret.value.then(onFulfilled, onRejected);
             }
@@ -3680,22 +3663,13 @@ function invertPatch(patch) {
             };
     }
 }
-/**
- * Simple simple check to check it is a number.
- */
-function isNumber(x) {
-    return typeof x === "number";
-}
 /**
  * Escape slashes and backslashes.
  *
  * http://tools.ietf.org/html/rfc6901
  */
 function escapeJsonPath(path) {
-    if (isNumber(path) === true) {
-        return "" + path;
-    }
-    if (path.indexOf("/") === -1 && path.indexOf("~") === -1) {
+    if (!path.includes("/") && !path.includes("~")) {
         return path;
     }
     return path.replace(/~/g, "~0").replace(/\//g, "~1");
@@ -3724,7 +3698,7 @@ function joinJsonPath(path) {
     }
     else {
         // absolute
-        return "/" + getPathStr(path);
+        return `/${getPathStr(path)}`;
     }
 }
 /**
@@ -3739,9 +3713,9 @@ function splitJsonPath(path) {
     const valid = path === "" ||
         path === "." ||
         path === ".." ||
-        stringStartsWith(path, "/") ||
-        stringStartsWith(path, "./") ||
-        stringStartsWith(path, "../");
+        path.startsWith("/") ||
+        path.startsWith("./") ||
+        path.startsWith("../");
     if (!valid) {
         throw fail(`a json path must be either rooted, empty or relative, but got '${path}'`);
     }
@@ -3949,16 +3923,16 @@ class MSTMap extends ObservableMap {
     }
     get(key) {
         // maybe this is over-enthousiastic? normalize numeric keys to strings
-        return super.get("" + key);
+        return super.get(`${key}`);
     }
     has(key) {
-        return super.has("" + key);
+        return super.has(`${key}`);
     }
     delete(key) {
-        return super.delete("" + key);
+        return super.delete(`${key}`);
     }
     set(key, value) {
-        return super.set("" + key, value);
+        return super.set(`${key}`, value);
     }
     put(value) {
         if (!value) {
@@ -4083,9 +4057,9 @@ class MapType extends ComplexType {
         return values(node.storedValue);
     }
     getChildNode(node, key) {
-        const childNode = node.storedValue.get("" + key);
+        const childNode = node.storedValue.get(`${key}`);
         if (!childNode) {
-            throw fail("Not a child " + key);
+            throw fail(`Not a child ${key}`);
         }
         return childNode;
     }
@@ -4193,7 +4167,7 @@ class MapType extends ComplexType {
             // Don't use target.replace, as it will throw away all existing items first
             for (const key in snapshot) {
                 target.set(key, snapshot[key]);
-                currentKeys["" + key] = true;
+                currentKeys[`${key}`] = true;
             }
         }
         Object.keys(currentKeys).forEach(key => {
@@ -4293,7 +4267,7 @@ class ArrayType extends ComplexType {
         const subType = objNode.type._subType;
         const result = {};
         snapshot.forEach((item, index) => {
-            const subpath = "" + index;
+            const subpath = `${index}`;
             result[subpath] = subType.instantiate(objNode, subpath, undefined, item);
         });
         return result;
@@ -4327,11 +4301,11 @@ class ArrayType extends ComplexType {
         if (index < node.storedValue.length) {
             return node.storedValue[index];
         }
-        throw fail("Not a child: " + key);
+        throw fail(`Not a child: ${key}`);
     }
     willChange(change) {
         const node = getStateTreeNode(change.object);
-        node.assertWritable({ subpath: "" + change.index });
+        node.assertWritable({ subpath: `${change.index}` });
         const subType = node.type._subType;
         const childNodes = node.getChildren();
         switch (change.type) {
@@ -4357,7 +4331,7 @@ class ArrayType extends ComplexType {
                     change.added = addedNodes;
                     // update paths of remaining items
                     for (let i = index + removedCount; i < childNodes.length; i++) {
-                        childNodes[i].setParent(node, "" + (i + added.length - removedCount));
+                        childNodes[i].setParent(node, `${i + added.length - removedCount}`);
                     }
                 }
                 break;
@@ -4380,7 +4354,7 @@ class ArrayType extends ComplexType {
             case "update":
                 return void node.emitPatch({
                     op: "replace",
-                    path: "" + change.index,
+                    path: `${change.index}`,
                     value: change.newValue.snapshot,
                     oldValue: change.oldValue ? change.oldValue.snapshot : undefined
                 }, node);
@@ -4388,15 +4362,15 @@ class ArrayType extends ComplexType {
                 for (let i = change.removedCount - 1; i >= 0; i--) {
                     node.emitPatch({
                         op: "remove",
-                        path: "" + (change.index + i),
+                        path: `${change.index + i}`,
                         oldValue: change.removed[i].snapshot
                     }, node);
                 }
                 for (let i = 0; i < change.addedCount; i++) {
                     node.emitPatch({
                         op: "add",
-                        path: "" + (change.index + i),
-                        value: node.getChildNode("" + (change.index + i)).snapshot,
+                        path: `${change.index + i}`,
+                        value: node.getChildNode(`${change.index + i}`).snapshot,
                         oldValue: undefined
                     }, node);
                 }
@@ -4431,7 +4405,7 @@ class ArrayType extends ComplexType {
             return typeCheckFailure(context, value, "Value is not an array");
         }
         for (let i = 0; i < value.length; i++) {
-            getContextForPath(context, "" + i, this._subType);
+            getContextForPath(context, `${i}`, this._subType);
             const errors = this._subType.validate(value[i], context);
             popContext(context);
             if (errors.length > 0) {
@@ -4482,7 +4456,7 @@ function reconcileArrayChildren(parent, childType, oldNodes, newValues, newPaths
         const hasNewNode = i <= newValues.length - 1;
         const oldNode = oldNodes[i];
         let newValue = hasNewNode ? newValues[i] : undefined;
-        const newPath = "" + newPaths[i];
+        const newPath = `${newPaths[i]}`;
         // for some reason, instead of newValue we got a node, fallback to the storedValue
         // TODO: https://github.com/mobxjs/mobx-state-tree/issues/340#issuecomment-325581681
         if (isNode(newValue)) {
@@ -4918,12 +4892,12 @@ class ModelType extends ComplexType {
     }
     getChildNode(node, key) {
         if (!(key in this.properties)) {
-            throw fail("Not a value property: " + key);
+            throw fail(`Not a value property: ${key}`);
         }
         const adm = _getAdministration(node.storedValue, key);
         const childNode = adm.raw?.();
         if (!childNode) {
-            throw fail("Node not available for property " + key);
+            throw fail(`Node not available for property ${key}`);
         }
         return childNode;
     }
@@ -5001,11 +4975,9 @@ class ModelType extends ComplexType {
     }
     describe() {
         // optimization: cache
-        return ("{ " +
-            this.propertyNames
-                .map(key => key + ": " + this.properties[key].describe())
-                .join("; ") +
-            " }");
+        return `{ ${this.propertyNames
+            .map(key => `${key}: ${this.properties[key].describe()}`)
+            .join("; ")} }`;
     }
     getDefaultSnapshot() {
         return EMPTY_OBJECT;
@@ -5050,7 +5022,7 @@ function compose(...args) {
     }
     return args
         .reduce((prev, cur) => prev.cloneAndEnhance({
-        name: prev.name + "_" + cur.name,
+        name: `${prev.name}_${cur.name}`,
         properties: cur.properties,
         initializers: cur.initializers,
         preProcessor: (snapshot) => cur.applySnapshotPreProcessor(prev.applySnapshotPreProcessor(snapshot)),
@@ -5217,7 +5189,7 @@ function getPrimitiveFactoryFromValue(value) {
                 return DatePrimitive;
             }
     }
-    throw fail("Cannot determine primitive type from value " + value);
+    throw fail(`Cannot determine primitive type from value ${value}`);
 }
 /**
  * Returns if a given value represents a primitive type.
@@ -5393,7 +5365,7 @@ function enumeration(name, options) {
             assertIsString(option, i + 1);
         });
     }
-    const type = union(...realOptions.map(option => literal("" + option)));
+    const type = union(...realOptions.map(option => literal(`${option}`)));
     if (typeof name === "string") {
         type.name = name;
     }
@@ -5451,7 +5423,7 @@ class Union extends BaseType {
         return this._types.some(subType => subType.isAssignableFrom(type));
     }
     describe() {
-        return ("(" + this._types.map(factory => factory.describe()).join(" | ") + ")");
+        return `(${this._types.map(factory => factory.describe()).join(" | ")})`;
     }
     instantiate(parent, subpath, environment, initialValue) {
         const type = this.determineType(initialValue, undefined);
@@ -5493,8 +5465,7 @@ class Union extends BaseType {
         if (errors.length === 0) {
             return baseWithDiscriminator;
         }
-        return (`${baseWithDiscriminator}:\n    ` +
-            formatValidationErrorLines(errors).join("\n    "));
+        return `${baseWithDiscriminator}:\n    ${formatValidationErrorLines(errors).join("\n    ")}`;
     }
     _findCandidateByTypeDiscriminator(discriminator) {
         let found;
@@ -5724,7 +5695,7 @@ function union(optionsOrType, ...otherTypes) {
     const types = isType(optionsOrType)
         ? [optionsOrType, ...otherTypes]
         : otherTypes;
-    const name = "(" + types.map(type => type.name).join(" | ") + ")";
+    const name = `(${types.map(type => type.name).join(" | ")})`;
     // check all options
     if (devMode()) {
         if (options) {
@@ -5791,17 +5762,17 @@ class OptionalValue extends BaseType {
         this.optionalValues = optionalValues;
     }
     describe() {
-        return this._subtype.describe() + "?";
+        return `${this._subtype.describe()}?`;
     }
     instantiate(parent, subpath, environment, initialValue) {
-        if (this.optionalValues.indexOf(initialValue) >= 0) {
+        if (this.optionalValues.includes(initialValue)) {
             const defaultInstanceOrSnapshot = this.getDefaultInstanceOrSnapshot();
             return this._subtype.instantiate(parent, subpath, environment, defaultInstanceOrSnapshot);
         }
         return this._subtype.instantiate(parent, subpath, environment, initialValue);
     }
     reconcile(current, newValue, parent, subpath) {
-        return this._subtype.reconcile(current, this.optionalValues.indexOf(newValue) < 0 && this._subtype.is(newValue)
+        return this._subtype.reconcile(current, !this.optionalValues.includes(newValue) && this._subtype.is(newValue)
             ? newValue
             : this.getDefaultInstanceOrSnapshot(), parent, subpath);
     }
@@ -5818,7 +5789,7 @@ class OptionalValue extends BaseType {
     }
     isValidSnapshot(value, context) {
         // defaulted values can be skipped
-        if (this.optionalValues.indexOf(value) >= 0) {
+        if (this.optionalValues.includes(value)) {
             return typeCheckSuccess();
         }
         // bounce validation to the sub-type
@@ -6019,8 +5990,7 @@ function late(nameOrType, maybeType) {
     // checks that the type is actually a late type
     if (devMode()) {
         if (!(typeof type === "function" && type.length === 0)) {
-            throw fail("Invalid late type, expected a function with zero arguments that returns a type, got: " +
-                type);
+            throw fail(`Invalid late type, expected a function with zero arguments that returns a type, got: ${type}`);
         }
     }
     return new Late(name, type);
@@ -6233,8 +6203,7 @@ class StoredReference {
         const normalizedId = normalizeIdentifier(this.identifier);
         const root = node.root;
         const lastCacheModification = root.identifierCache.getLastCacheModificationPerId(normalizedId);
-        if (!this.resolvedReference ||
-            this.resolvedReference.lastCacheModification !== lastCacheModification) {
+        if (this.resolvedReference?.lastCacheModification !== lastCacheModification) {
             const { targetType } = this;
             // reference was initialized with the identifier of the target
             const target = root.identifierCache.resolve(targetType, normalizedId);
@@ -6243,7 +6212,7 @@ class StoredReference {
             }
             this.resolvedReference = {
                 node: target,
-                lastCacheModification: lastCacheModification
+                lastCacheModification
             };
         }
     }
@@ -6291,7 +6260,7 @@ class BaseReferenceType extends SimpleType {
         // since it is a scalar value (immutable-ish) and we need to change it
         // from the parent
         const storedRefParentNode = storedRefNode.parent;
-        if (!storedRefParentNode || !storedRefParentNode.isAlive) {
+        if (!storedRefParentNode?.isAlive) {
             return;
         }
         const storedRefParentValue = storedRefParentNode.storedValue;
@@ -6363,7 +6332,7 @@ class BaseReferenceType extends SimpleType {
             }
             // make sure the target node is actually there and initialized
             const storedRefParentNode = storedRefNode.parent;
-            const storedRefParentValue = storedRefParentNode && storedRefParentNode.storedValue;
+            const storedRefParentValue = storedRefParentNode?.storedValue;
             if (storedRefParentNode &&
                 storedRefParentNode.isAlive &&
                 storedRefParentValue) {
@@ -6677,7 +6646,7 @@ function isIdentifierType(type) {
  * @hidden
  */
 function normalizeIdentifier(id) {
-    return "" + id;
+    return `${id}`;
 }
 /**
  * @internal
@@ -6773,7 +6742,7 @@ class CustomType extends SimpleType {
     instantiate(parent, subpath, environment, initialValue) {
         const valueToStore = this.options.isTargetType(initialValue)
             ? initialValue
-            : this.options.fromSnapshot(initialValue, parent && parent.root.environment);
+            : this.options.fromSnapshot(initialValue, parent?.root.environment);
         return createScalarNode(this, parent, subpath, environment, valueToStore);
     }
     reconcile(current, value, parent, subpath) {
@@ -6948,5 +6917,5 @@ const types = {
     resilient
 };
 
-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, getUnionSubtypes, 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, setDevMode, setLivelinessChecking, setLivelynessChecking, splitJsonPath, types as t, 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, getUnionSubtypes, 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, setDevMode, setLivelinessChecking, setLivelynessChecking, setTypeChecking, splitJsonPath, types as t, toGenerator, toGeneratorFunction, tryReference, tryResolve, typecheck, types, unescapeJsonPath, unprotect, walk };
 //# sourceMappingURL=mobx-state-tree.mjs.map