mobx 6.11.0-pre → 6.12.0

package/dist/mobx.umd.development.js

@@ -422,7 +422,7 @@
     }
     function assert20223DecoratorType(context, types) {
       if ( !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");
       }
     }
 
@@ -442,14 +442,12 @@
         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;
@@ -478,13 +476,6 @@
        * 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();
@@ -920,24 +911,21 @@
     }
     function decorate_20223_$4(desc, context) {
       {
-        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_;
@@ -1240,6 +1228,9 @@
         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);
@@ -1987,7 +1978,6 @@
       this.runId = 0;
       this.mobxGuid = 0;
       this.inBatch = 0;
-      this.batchId = Number.MIN_SAFE_INTEGER;
       this.pendingUnobservations = [];
       this.pendingReactions = [];
       this.isRunningReactions = false;
@@ -2004,7 +1994,6 @@
       this.useProxies = true;
       this.verifyProxies = false;
       this.safeDescriptors = true;
-      this.stateVersion = Number.MIN_SAFE_INTEGER;
     };
     var canMergeGlobalState = true;
     var isolateCalled = false;
@@ -2127,9 +2116,6 @@
      * 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() {
@@ -3966,6 +3952,7 @@
      * 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);
@@ -3975,15 +3962,21 @@
     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);