@astral/mobx-query 0.1.2 → 0.1.4

package/DataStorage/DataStorage.js

@@ -4,6 +4,10 @@ import { makeAutoObservable } from 'mobx';
  */
 export class DataStorage {
     constructor() {
+        /**
+         * @description поле, отвечающее за непосредственное хранение данных
+         */
+        this.internalData = undefined;
         /**
          * @description метод для установки данных
          */

package/InfiniteQuery/InfiniteQuery.js

@@ -1,4 +1,4 @@
-import { makeAutoObservable } from 'mobx';
+import { makeAutoObservable, when } from 'mobx';
 import { AuxiliaryQuery } from '../AuxiliaryQuery';
 export const DEFAULT_INFINITE_ITEMS_COUNT = 30;
 /**
@@ -151,7 +151,9 @@ export class InfiniteQuery {
     get data() {
         const shouldSync = this.enabledAutoFetch && !this.isSuccess && !this.isLoading;
         if (this.isInvalid || shouldSync) {
-            this.proceedSync();
+            // т.к. при вызове апдейта, изменяются флаги, на которые подписан data,
+            // нужно вызывать этот экшн асинхронно
+            when(() => true, this.proceedSync);
         }
         // возвращаем имеющиеся данные
         return this.storage.data;

package/MobxQuery/MobxQuery.js

@@ -2,6 +2,10 @@ import { Query } from '../Query';
 import { InfiniteQuery, } from '../InfiniteQuery';
 import { Mutation } from '../Mutation';
 import { DataStorageFactory } from '../DataStorage';
+/**
+ * @description время, спустя которое, запись о query c network-only будет удалена
+ */
+const DEFAULT_TIME_TO_CLEAN = 100;
 /**
  * @description Сервис, позволяющий кэшировать данные.
  */
@@ -46,16 +50,28 @@ export class MobxQuery {
          * и если нет, создает новый, добавляет его к себе в память, и возвращает его пользователю
          */
         this.getCachedQuery = (key, createStore, fetchPolicy) => {
-            if (fetchPolicy === 'network-only') {
-                return createStore();
-            }
-            const keyHash = this.serialize(key);
+            // создаем хэш ключа с добавляем к ключу значения fetchPolicy,
+            // чтобы query c одинаковым ключом, но разным fetchPolicy не пересекались бы
+            const keyHash = this.serialize([...key, fetchPolicy]);
             if (this.cacheableStores.has(keyHash)) {
                 return this.cacheableStores.get(keyHash);
             }
             const store = createStore();
             this.cacheableStores.set(keyHash, store);
             this.keys[keyHash] = key;
+            // Ожидается, что network-only квери не должны кешироваться,
+            // но, с введением StrictMode в реакт 18, проявилась проблема,
+            // что network-only квери, созданные в одном реакт компоненте,
+            // создаются дважды (т.к. все хуки вызываются дважды)
+            // и т.к. мы не ограничиваем момент запроса,
+            // то можем получить эффект, что оба network-only квери
+            // делают по отдельному запросу к данным единомоментно,
+            if (fetchPolicy === 'network-only') {
+                setTimeout(() => {
+                    this.cacheableStores.delete(keyHash);
+                    delete this.keys[keyHash];
+                }, DEFAULT_TIME_TO_CLEAN);
+            }
             return store;
         };
         /**

package/Query/Query.js

@@ -1,4 +1,4 @@
-import { makeAutoObservable } from 'mobx';
+import { makeAutoObservable, when } from 'mobx';
 import { AuxiliaryQuery } from '../AuxiliaryQuery';
 /**
  * @description стор для работы с запросами,
@@ -90,7 +90,9 @@ export class Query {
     get data() {
         const shouldSync = this.enabledAutoFetch && !this.isSuccess && !this.isLoading;
         if (this.isInvalid || shouldSync) {
-            this.proceedSync();
+            // т.к. при вызове апдейта, изменяются флаги, на которые подписан data,
+            // нужно вызывать этот экшн асинхронно
+            when(() => true, this.proceedSync);
         }
         // возвращаем имеющиеся данные
         return this.storage.data;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@astral/mobx-query",
-  "version": "0.1.2",
+  "version": "0.1.4",
   "browser": "./index.js",
   "main": "./index.js",
   "dependencies": {