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