mobx 6.11.0-pre → 6.12.0

package/dist/mobx.esm.js

@@ -417,7 +417,7 @@ function is20223Decorator(context) {
 }
 function assert20223DecoratorType(context, types) {
   if (process.env.NODE_ENV !== "production" && !types.includes(context.kind)) {
-    die("Decorator may not be used like this");
+    die("The decorator applied to '" + String(context.name) + "' cannot be used on a " + context.kind + " element");
   }
 }
 
@@ -437,14 +437,12 @@ var Atom = /*#__PURE__*/function () {
     this.isPendingUnobservation_ = false;
     this.isBeingObserved_ = false;
     this.observers_ = new Set();
-    this.batchId_ = void 0;
     this.diffValue_ = 0;
     this.lastAccessedBy_ = 0;
     this.lowestObserverState_ = IDerivationState_.NOT_TRACKING_;
     this.onBOL = void 0;
     this.onBUOL = void 0;
     this.name_ = name_;
-    this.batchId_ = globalState.inBatch ? globalState.batchId : NaN;
   }
   // onBecomeObservedListeners
   var _proto = Atom.prototype;
@@ -473,13 +471,6 @@ var Atom = /*#__PURE__*/function () {
    * Invoke this method _after_ this method has changed to signal mobx that all its observers should invalidate.
    */;
   _proto.reportChanged = function reportChanged() {
-    if (!globalState.inBatch || this.batchId_ !== globalState.batchId) {
-      // We could update state version only at the end of batch,
-      // but we would still have to switch some global flag here to signal a change.
-      globalState.stateVersion = globalState.stateVersion < Number.MAX_SAFE_INTEGER ? globalState.stateVersion + 1 : Number.MIN_SAFE_INTEGER;
-      // Avoids the possibility of hitting the same globalState.batchId when it cycled through all integers (necessary?)
-      this.batchId_ = NaN;
-    }
     startBatch();
     propagateChanged(this);
     endBatch();
@@ -915,24 +906,21 @@ function extend_$4(adm, key, descriptor, proxyTrap) {
 }
 function decorate_20223_$4(desc, context) {
   if (process.env.NODE_ENV !== "production") {
-    assert20223DecoratorType(context, ["accessor", "field"]);
+    if (context.kind === "field") {
+      throw die("Please use `@observable accessor " + String(context.name) + "` instead of `@observable " + String(context.name) + "`");
+    }
+    assert20223DecoratorType(context, ["accessor"]);
   }
   var ann = this;
   var kind = context.kind,
-    name = context.name,
-    addInitializer = context.addInitializer;
-  // Backwards/Legacy behavior, expects makeObservable(this)
-  if (kind == "field") {
-    addInitializer(function () {
-      storeAnnotation(this, name, ann);
-    });
-    return;
-  }
+    name = context.name;
   // The laziness here is not ideal... It's a workaround to how 2022.3 Decorators are implemented:
   //   `addInitializer` callbacks are executed _before_ any accessors are defined (instead of the ideal-for-us right after each).
   //   This means that, if we were to do our stuff in an `addInitializer`, we'd attempt to read a private slot
   //   before it has been initialized. The runtime doesn't like that and throws a `Cannot read private member
   //   from an object whose class did not declare it` error.
+  // TODO: it seems that this will not be required anymore in the final version of the spec
+  // See TODO: link
   var initializedObjects = new WeakSet();
   function initializeObservable(target, value) {
     var _ann$options_$enhance, _ann$options_;
@@ -1235,6 +1223,9 @@ function createAction(actionName, fn, autoAction, ref) {
     return executeAction(actionName, autoAction, fn, ref || this, arguments);
   }
   res.isMobxAction = true;
+  res.toString = function () {
+    return fn.toString();
+  };
   if (isFunctionNameConfigurable) {
     tmpNameDescriptor.value = actionName;
     defineProperty(res, "name", tmpNameDescriptor);
@@ -1991,7 +1982,6 @@ var MobXGlobals = function MobXGlobals() {
   this.runId = 0;
   this.mobxGuid = 0;
   this.inBatch = 0;
-  this.batchId = Number.MIN_SAFE_INTEGER;
   this.pendingUnobservations = [];
   this.pendingReactions = [];
   this.isRunningReactions = false;
@@ -2008,7 +1998,6 @@ var MobXGlobals = function MobXGlobals() {
   this.useProxies = true;
   this.verifyProxies = false;
   this.safeDescriptors = true;
-  this.stateVersion = Number.MIN_SAFE_INTEGER;
 };
 var canMergeGlobalState = true;
 var isolateCalled = false;
@@ -2131,9 +2120,6 @@ function queueForUnobservation(observable) {
  * Avoids unnecessary recalculations.
  */
 function startBatch() {
-  if (globalState.inBatch === 0) {
-    globalState.batchId = globalState.batchId < Number.MAX_SAFE_INTEGER ? globalState.batchId + 1 : Number.MIN_SAFE_INTEGER;
-  }
   globalState.inBatch++;
 }
 function endBatch() {
@@ -3985,6 +3971,7 @@ var arrayExtensions = {
  * Without this, everything works as well, but this works
  * faster as everything works on unproxied values
  */
+addArrayExtension("at", simpleFunc);
 addArrayExtension("concat", simpleFunc);
 addArrayExtension("flat", simpleFunc);
 addArrayExtension("includes", simpleFunc);
@@ -3994,15 +3981,21 @@ addArrayExtension("lastIndexOf", simpleFunc);
 addArrayExtension("slice", simpleFunc);
 addArrayExtension("toString", simpleFunc);
 addArrayExtension("toLocaleString", simpleFunc);
+addArrayExtension("toSorted", simpleFunc);
+addArrayExtension("toSpliced", simpleFunc);
+addArrayExtension("with", simpleFunc);
 // map
 addArrayExtension("every", mapLikeFunc);
 addArrayExtension("filter", mapLikeFunc);
 addArrayExtension("find", mapLikeFunc);
 addArrayExtension("findIndex", mapLikeFunc);
+addArrayExtension("findLast", mapLikeFunc);
+addArrayExtension("findLastIndex", mapLikeFunc);
 addArrayExtension("flatMap", mapLikeFunc);
 addArrayExtension("forEach", mapLikeFunc);
 addArrayExtension("map", mapLikeFunc);
 addArrayExtension("some", mapLikeFunc);
+addArrayExtension("toReversed", mapLikeFunc);
 // reduce
 addArrayExtension("reduce", reduceLikeFunc);
 addArrayExtension("reduceRight", reduceLikeFunc);