mobx 6.11.0-pre → 6.11.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");
   }
 }
 
@@ -915,24 +915,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_;