fix: merge array

__tests__/index.spec.ts

@@ -216,4 +216,69 @@ describe('Persist', () => {
       done()
     })
   })
+
+  it('no strict patch', (done) => {
+    window.localStorage.clear()
+    window.localStorage.setItem('pinia_foo', JSON.stringify({
+      test: [
+        { id: 1, name: 'foo' },
+      ],
+    }))
+
+    const testStore = defineStore('foo', {
+      state: () => ({
+        test: [
+          { id: 1, name: 'foo' },
+        ],
+      }),
+      actions: {
+        add(item: { id: number; name: string }) {
+          this.test.push(item)
+        },
+        remove() {
+          this.test = this.test.filter(item => item.id !== 1)
+        },
+      },
+    })
+    const pinia = createPinia()
+    pinia.use(persist())
+
+    mount({ template: 'none' }, { global: { plugins: [pinia] } })
+    const store = testStore(pinia)
+    expect(store.test).toStrictEqual([
+      { id: 1, name: 'foo' },
+    ])
+
+    Promise.resolve().then(() => {
+      expect(window.localStorage.getItem('pinia_foo')).toBe(JSON.stringify({
+        test: [
+          { id: 1, name: 'foo' },
+        ],
+      }))
+      store.add({ id: 2, name: 'bar' })
+    }).then(() => {
+      expect(window.localStorage.getItem('pinia_foo')).toBe(JSON.stringify({
+        test: [
+          { id: 1, name: 'foo' },
+          { id: 2, name: 'bar' },
+        ],
+      }))
+      store.remove()
+    }).then(() => {
+      expect(window.localStorage.getItem('pinia_foo')).toBe(JSON.stringify({
+        test: [
+          { id: 2, name: 'bar' },
+        ],
+      }))
+      store.add({ id: 2, name: 'bar' })
+    }).then(() => {
+      expect(window.localStorage.getItem('pinia_foo')).toBe(JSON.stringify({
+        test: [
+          { id: 2, name: 'bar' },
+          { id: 2, name: 'bar' },
+        ],
+      }))
+      done()
+    })
+  })
 })

dist/index.d.ts

@@ -30,14 +30,17 @@ interface Storage {
 interface PersistOptions {
     /**
      * it is the prefix of the key to store the persisted state under, such as the store id is "test", that store key is "pinia_test" by default.
+     * @default pinia
      */
     prefix?: string;
     /**
      * it is the storage for state persist,  you can choose localStorage, sessionStorage or your custom storage.
+     * @default window.localStorage
      */
     storage?: Storage;
     /**
      * when app first loader or refresh, whether to overwrite the existing state with the output from state directly, instead of merging the two objects with deep merge.
+     * @default false
      */
     overwrite?: boolean;
 }

src/helper.ts

@@ -1,3 +1,22 @@
 export const isObject = (val: unknown) => val && typeof val === 'object'
 
-export const mergeArrayWithDedupe = (a: any[], b: any[]) => Array.from(new Set([...a, ...b]))
+export const mergeArrayWithDedupe = (a: any[], b: any[]) => {
+  const result: any[] = [...a]
+
+  for (const item of b) {
+    if (!isObject(item)) {
+      if (!result.includes(item)) result.push(item)
+    }
+    else {
+      const itemStr = JSON.stringify(item)
+      if (!a.some((i) => {
+        if (!isObject(i)) return false
+        else
+          return itemStr === JSON.stringify(i)
+      }))
+        result.push(item)
+    }
+  }
+
+  return result
+}