mobx 6.13.6 → 6.15.0

package/CHANGELOG.md

@@ -1,5 +1,27 @@
 # mobx
 
+## 6.15.0
+
+### Minor Changes
+
+-   [`2e703388eda4ba3eefed2bf1f5ca3958980978c3`](https://github.com/mobxjs/mobx/commit/2e703388eda4ba3eefed2bf1f5ca3958980978c3) [#4584](https://github.com/mobxjs/mobx/pull/4584) Thanks [@mweststrate](https://github.com/mweststrate)! - Fix #4753: Wrong inferred type of observable.map in TS 5.9+, by @mbest in #578
+
+### Patch Changes
+
+-   [`61d6cf39764f28c6c2e0d2b3912364889739c619`](https://github.com/mobxjs/mobx/commit/61d6cf39764f28c6c2e0d2b3912364889739c619) [#4563](https://github.com/mobxjs/mobx/pull/4563) Thanks [@mweststrate](https://github.com/mweststrate)! - Fixed memory leak where makeAutoObservable would keep wrapping setters defined on the prototype. Fixes #4553
+
+## 6.14.0
+
+### Minor Changes
+
+-   [`fe1d3f45cfd0d21189e963579e5c9d764329fea9`](https://github.com/mobxjs/mobx/commit/fe1d3f45cfd0d21189e963579e5c9d764329fea9) [#4558](https://github.com/mobxjs/mobx/pull/4558) Thanks [@vkrol](https://github.com/vkrol)! - Add Explicit Resource Management support in reactions
+
+## 6.13.7
+
+### Patch Changes
+
+-   [`54e3f71ca02f09b3107290f18d8484b70a6e2f0b`](https://github.com/mobxjs/mobx/commit/54e3f71ca02f09b3107290f18d8484b70a6e2f0b) [#4528](https://github.com/mobxjs/mobx/pull/4528) Thanks [@k-g-a](https://github.com/k-g-a)! - Fix observable.set not respecting the new value from interceptors
+
 ## 6.13.6
 
 ### Patch Changes

package/README.md

@@ -57,6 +57,7 @@ MobX is made possible by the generosity of the sponsors below, and many other [i
 <a href="https://talentplot.com/"><img src="https://mobx.js.org/assets/talentplot.png" align="center" width="100" title="talentplot" alt="talentplot"></a>
 <a href="https://www.easeus.com/?utm_source=github_mobxjs_sponsorship&utm_medium=readme&utm_campaign=sponsorship"><img src="https://mobx.js.org/assets/easeus.png" align="center" width="100" title="EaseUS" alt="EaseUS"/></a>
 <a href="https://route4me.com/"><img src="https://mobx.js.org/assets/route4me.png" align="center" width="100" title="Route Planner and Route Optimizer" alt="Route Planner and Route Optimizer"/></a>
+<a href="https://handsontable.com/docs/react-data-grid/?utm_source=Mobx_homepage&utm_medium=sponsorship&utm_campaign=library_sponsorship"><img src="https://mobx.js.org/assets/handsontable.png" align="center" width="100" title="Handsontable" alt="Handsontable"/></a>
 
 ## Introduction
 

package/dist/api/observable.d.ts

@@ -1,4 +1,4 @@
-import { IEnhancer, IEqualsComparer, IObservableArray, IObservableMapInitialValues, IObservableSetInitialValues, IObservableValue, ObservableMap, ObservableSet, Annotation, AnnotationsMap } from "../internal";
+import { IEnhancer, IEqualsComparer, IObservableArray, IMapEntries, IReadonlyMapEntries, IKeyValueMap, IObservableSetInitialValues, IObservableValue, ObservableMap, ObservableSet, Annotation, AnnotationsMap } from "../internal";
 import type { ClassAccessorDecorator, ClassFieldDecorator } from "../types/decorator_fills";
 export declare const OBSERVABLE = "observable";
 export declare const OBSERVABLE_REF = "observable.ref";
@@ -21,6 +21,14 @@ export interface IObservableValueFactory {
     <T>(value: T, options?: CreateObservableOptions): IObservableValue<T>;
     <T>(value?: T, options?: CreateObservableOptions): IObservableValue<T | undefined>;
 }
+export interface IObservableMapFactory {
+    <K = any, V = any>(): ObservableMap<K, V>;
+    <K, V>(initialValues?: IMapEntries<K, V>, options?: CreateObservableOptions): ObservableMap<K, V>;
+    <K, V>(initialValues?: IReadonlyMapEntries<K, V>, options?: CreateObservableOptions): ObservableMap<K, V>;
+    <K, V>(initialValues?: IKeyValueMap<V>, options?: CreateObservableOptions): ObservableMap<K, V>;
+    <K, V>(initialValues?: Map<K, V>, options?: CreateObservableOptions): ObservableMap<K, V>;
+    <K = any, V = any>(initialValues: undefined, options?: CreateObservableOptions): ObservableMap<K, V>;
+}
 export interface IObservableFactory extends Annotation, PropertyDecorator, ClassAccessorDecorator, ClassFieldDecorator {
     <T = any>(value: T[], options?: CreateObservableOptions): IObservableArray<T>;
     <T = any>(value: Set<T>, options?: CreateObservableOptions): ObservableSet<T>;
@@ -29,7 +37,7 @@ export interface IObservableFactory extends Annotation, PropertyDecorator, Class
     box: IObservableValueFactory;
     array: <T = any>(initialValues?: T[], options?: CreateObservableOptions) => IObservableArray<T>;
     set: <T = any>(initialValues?: IObservableSetInitialValues<T>, options?: CreateObservableOptions) => ObservableSet<T>;
-    map: <K = any, V = any>(initialValues?: IObservableMapInitialValues<K, V>, options?: CreateObservableOptions) => ObservableMap<K, V>;
+    map: IObservableMapFactory;
     object: <T = any>(props: T, decorators?: AnnotationsMap<T, never>, options?: CreateObservableOptions) => T;
     /**
      * Decorator that creates an observable that only observes the references, but doesn't try to turn the assigned value into an observable.ts.

package/dist/mobx.cjs.development.js

@@ -1010,7 +1010,8 @@ function make_$5(adm, key, descriptor, source) {
   // lone setter -> action setter
   if (descriptor.set) {
     // TODO make action applicable to setter and delegate to action.make_
-    var set = createAction(key.toString(), descriptor.set);
+    var set = isAction(descriptor.set) ? descriptor.set // See #4553
+    : createAction(key.toString(), descriptor.set);
     // own
     if (source === adm.target_) {
       return adm.defineProperty_(key, {
@@ -1375,13 +1376,14 @@ var ObservableValue = /*#__PURE__*/function (_Atom) {
     _this.equals = equals;
     _this.value_ = enhancer(value, undefined, name_);
     if ( notifySpy && isSpyEnabled()) {
+      var _this$value_;
       // only notify spy if this is a stand-alone observable
       spyReport({
         type: CREATE,
         object: _this,
         observableKind: "value",
         debugObjectName: _this.name_,
-        newValue: "" + _this.value_
+        newValue: "" + ((_this$value_ = _this.value_) == null ? void 0 : _this$value_.toString())
       });
     }
     return _this;
@@ -2549,6 +2551,9 @@ var Reaction = /*#__PURE__*/function () {
     };
     abortSignal == null || abortSignal.addEventListener == null || abortSignal.addEventListener("abort", dispose);
     dispose[$mobx] = this;
+    if ("dispose" in Symbol && typeof Symbol.dispose === "symbol") {
+      dispose[Symbol.dispose] = dispose;
+    }
     return dispose;
   };
   _proto.toString = function toString() {
@@ -4682,8 +4687,8 @@ var ObservableSet = /*#__PURE__*/function () {
       if (!change) {
         return this;
       }
-      // ideally, value = change.value would be done here, so that values can be
-      // changed by interceptor. Same applies for other Set and Map api's.
+      // implemented reassignment same as it's done for ObservableMap
+      value = change.newValue;
     }
     if (!this.has(value)) {
       transaction(function () {
@@ -5738,6 +5743,8 @@ function deepEqual(a, b, depth) {
   return eq(a, b, depth);
 }
 // Copied from https://github.com/jashkenas/underscore/blob/5c237a7c682fb68fd5378203f0bf22dce1624854/underscore.js#L1186-L1289
+// Modified: "Deep compare objects" part to iterate over keys in forward order instead of reverse order.
+//
 // Internal recursive comparison function for `isEqual`.
 function eq(a, b, depth, aStack, bStack) {
   // Identical objects are equal. `0 === -0`, but they aren't identical.
@@ -5850,15 +5857,14 @@ function eq(a, b, depth, aStack, bStack) {
   } else {
     // Deep compare objects.
     var keys = Object.keys(a);
-    var key;
-    length = keys.length;
+    var _length = keys.length;
     // Ensure that both objects contain the same number of properties before comparing deep equality.
-    if (Object.keys(b).length !== length) {
+    if (Object.keys(b).length !== _length) {
       return false;
     }
-    while (length--) {
+    for (var i = 0; i < _length; i++) {
       // Deep compare each member
-      key = keys[length];
+      var key = keys[i];
       if (!(hasProp(b, key) && eq(a[key], b[key], depth - 1, aStack, bStack))) {
         return false;
       }