fix dirtyAttributes to look at persisted values rather than latest and remove observability from data property on store

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@artemisag/mobx-async-store",
-  "version": "7.1.0",
+  "version": "7.1.1",
   "license": "MIT",
   "module": "dist/mobx-async-store.esm.js",
   "main": "dist/mobx-async-store.cjs.js",

spec/Model.spec.ts

@@ -943,8 +943,15 @@ describe('Model', () => {
         description: 'Example description'
       })
 
+      const note2 = store.add('notes', {
+        id: '11',
+        description: 'Example description'
+      })
+
       todo.notes.add(note)
       note.description = 'something different'
+      todo.notes.replace([note, note2])
+
       expect(todo.dirtyAttributes).toEqual(blankSet)
       expect(note.dirtyAttributes.size).toEqual(1)
       expect(note.dirtyAttributes).toContain('description')

spec/Store.spec.ts

@@ -275,10 +275,6 @@ describe('Store', () => {
     })
   })
 
-  it('has observable data property', () => {
-    expect(isObservable(store.data)).toBe(true)
-  })
-
   it('sets network configuration properties', () => {
     expect(store.baseUrl).toEqual(mockBaseUrl)
     expect(store.defaultFetchOptions).toEqual(mockFetchOptions)
@@ -300,11 +296,11 @@ describe('Store', () => {
 
   it('initializes data observable', () => {
     const map = new Map()
-    expect(toJS(store.data)).toEqual({
-      todos: { cache: map, meta: map, records: map },
-      notes: { cache: map, meta: map, records: map },
-      categories: { cache: map, meta: map, records: map },
-      tags: { cache: map, meta: map, records: map }
+    expect(store.data).toEqual({
+      todos: { cache: new Map(), meta: new Map(), records: new Map() },
+      notes: { cache: new Map(), meta: new Map(), records: new Map() },
+      categories: { cache: new Map(), meta: new Map(), records: new Map() },
+      tags: { cache: new Map(), meta: new Map(), records: new Map() }
     })
   })
 

src/Model.ts

@@ -231,7 +231,7 @@ class Model {
 
     return Object.keys(this.attributes).reduce((dirtyAccumulator, attr) => {
       const currentValue = this.attributes[attr]
-      const previousValue = this.previousSnapshot.attributes[attr]
+      const previousValue = this.persistedOrFirstSnapshot.attributes[attr]
 
       if (isObject(currentValue)) {
         const currentToPreviousDiff = diff(currentValue, previousValue)

src/Store.ts

@@ -15,7 +15,6 @@ import cloneDeep from 'lodash/cloneDeep'
  */
 
 const mobxAnnotations = {
-  data: observable,
   lastResponseHeaders: observable,
   loadingStates: observable,
   loadedStates: observable,
@@ -69,9 +68,9 @@ class Store {
    * Stores data by type.
    * {
    *   todos: {
-   *     records: observable.map(), // records by id
-   *     cache: observable.map(), // cached ids by url
-   *     meta: observable.map() // meta information by url
+   *     records: new Map(), // records by id
+   *     cache: new Map(), // cached ids by url
+   *     meta: new Map() // meta information by url
    *   }
    * }
    *
@@ -680,9 +679,9 @@ class Store {
     const types = type ? [type] : this.models.map(({ type }) => type)
     types.forEach((type) => {
       this.data[type] = {
-        records: observable.map(),
-        cache: observable.map(),
-        meta: observable.map()
+        records: new Map(),
+        cache: new Map(),
+        meta: new Map()
       }
     })
   }