@astral/mobx-query 1.6.1 → 1.7.0

package/AuxiliaryQuery/AuxiliaryQuery.d.ts

@@ -30,7 +30,7 @@ export declare class AuxiliaryQuery<TResult, TError = void> {
      * Метод ответственный за создание единого промиса,
      * для устранения гонки запросов
      */
-    getUnifiedPromise: (executor: Executor<TResult>) => Promise<TResult>;
+    getUnifiedPromise: (executor: Executor<TResult>, onSuccess?: ((data: TResult) => void) | undefined) => Promise<TResult>;
     private setSuccess;
     private checkBackgroundAndSet;
     /**

package/AuxiliaryQuery/AuxiliaryQuery.js

@@ -22,13 +22,14 @@ export class AuxiliaryQuery {
          * Метод ответственный за создание единого промиса,
          * для устранения гонки запросов
          */
-        this.getUnifiedPromise = (executor) => {
+        this.getUnifiedPromise = (executor, onSuccess) => {
             // проверяем, если синглтона нет, то надо создать
             if (!Boolean(this.unifiedPromise)) {
                 this.startLoading();
                 this.unifiedPromise = executor()
                     .then((resData) => {
                     runInAction(this.submitSuccess);
+                    onSuccess === null || onSuccess === void 0 ? void 0 : onSuccess(resData);
                     return resData;
                 })
                     .catch((e) => {

package/InfiniteQuery/InfiniteQuery.js

@@ -67,10 +67,10 @@ export class InfiniteQuery extends QueryContainer {
                 this.offset += this.incrementCount;
                 // запускаем запрос с последними параметрами, и флагом необходимости инкремента
                 this.auxiliary
-                    .getUnifiedPromise(this.infiniteExecutor)
-                    .then((resData) => {
+                    .getUnifiedPromise(this.infiniteExecutor, (resData) => {
                     this.submitSuccess(resData, undefined, true);
-                });
+                })
+                    .catch((e) => { var _a; return (_a = this.defaultOnError) === null || _a === void 0 ? void 0 : _a.call(this, e); });
             }
         };
         /**
@@ -89,8 +89,7 @@ export class InfiniteQuery extends QueryContainer {
             this.offset = 0;
             this.isEndReached = false;
             this.auxiliary
-                .getUnifiedPromise(this.infiniteExecutor)
-                .then((resData) => {
+                .getUnifiedPromise(this.infiniteExecutor, (resData) => {
                 this.submitSuccess(resData, onSuccess);
             })
                 .catch((e) => {
@@ -115,16 +114,7 @@ export class InfiniteQuery extends QueryContainer {
             }
             this.offset = 0;
             this.isEndReached = false;
-            return this.auxiliary
-                .getUnifiedPromise(this.infiniteExecutor)
-                .then((resData) => {
-                this.submitSuccess(resData);
-                return resData;
-            })
-                .catch((e) => {
-                this.auxiliary.submitError(e);
-                throw e;
-            });
+            return this.auxiliary.getUnifiedPromise(this.infiniteExecutor, this.submitSuccess);
         };
         this.storage = dataStorage;
         this.incrementCount = incrementCount;

package/MobxQuery/MobxQuery.js

@@ -4,10 +4,6 @@ import { Mutation, } from '../Mutation';
 import { DataStorageFactory } from '../DataStorage';
 import { StatusStorageFactory } from '../StatusStorage';
 import { AdaptableMap } from '../AdaptableMap';
-/**
- * Время, спустя которое, запись о query c network-only будет удалена
- */
-const DEFAULT_TIME_TO_CLEAN = 100;
 /**
  * Сервис, позволяющий кэшировать данные.
  */
@@ -35,7 +31,7 @@ export class MobxQuery {
          * предполагается использование из домена
          */
         this.invalidate = (keysParts) => {
-            // Сет сериализовонных ключей
+            // Сет сериализованных ключей
             const keysSet = new Set(keysParts.map(this.serialize));
             [...this.keys.keys()].forEach((keyHash) => {
                 const key = this.keys.get(keyHash);
@@ -91,30 +87,36 @@ export class MobxQuery {
                 dataStorage: this.queryDataStorageFactory.getStorage(keys.dataKeyHash),
                 statusStorage: this.statusStorageFactory.getStorage(keys.statusKeyHash),
                 backgroundStatusStorage: this.getBackgroundStatusStorage(keys.backgroundStatusKeyHash, Boolean(createParams === null || createParams === void 0 ? void 0 : createParams.isBackground)),
-                submitValidity: () => this.submitValidity(keys.queryKeyHash),
+                submitValidity: 
+                // 'network-only' квери не будут подтверждать свою валидность,
+                // следовательно, они всегда будут храниться как "слабые",
+                // что позволит сборщику мусора удалять их из памяти при отсутствии ссылок
+                fetchPolicy !== 'network-only'
+                    ? () => this.submitValidity(keys.queryKeyHash)
+                    : undefined,
             });
             this.queriesMap.set(keys.queryKeyHash, query);
             this.keys.set(keys.queryKeyHash, keys.queryKey);
-            // Ожидается, что network-only квери не должны кешироваться,
-            // но, с введением StrictMode в реакт 18, проявилась проблема,
-            // что network-only квери, созданные в одном реакт компоненте,
-            // создаются дважды (т.к. все хуки вызываются дважды)
-            // и т.к. мы не ограничиваем момент запроса,
-            // то можем получить эффект, что оба network-only квери
-            // делают по отдельному запросу к данным единомоментно,
-            if (fetchPolicy === 'network-only') {
-                setTimeout(() => {
-                    this.queriesMap.delete(keys.queryKeyHash);
-                    this.keys.delete(keys.queryKeyHash);
-                }, DEFAULT_TIME_TO_CLEAN);
-            }
             return query;
         };
         /**
          * Метод для создания ключей к внутренним хранилищам
          */
         this.makeKeys = (rootKey, fetchPolicy, isBackground, type) => {
-            const queryKey = [...rootKey, { fetchPolicy, isBackground, type }];
+            // C введением StrictMode в реакт 18, проявилась проблема,
+            // что network-only квери, созданные в одном реакт компоненте,
+            // создаются дважды (т.к. все хуки вызываются дважды)
+            // и т.к. мы не ограничиваем момент запроса,
+            // то можем получить эффект, что оба network-only квери
+            // делают по отдельному запросу к данным единомоментно.
+            // Для решения этой проблемы, добавляем уникальный ключ даты, с обнуляемыми милисекундами,
+            // что позволит создавать одинаковые network-only квери в течении одной секунды.
+            // Остается крайне редкий корнер кейс, когда разница между созданиями квери крайне мала,
+            // но все таки первый был создан в одной секунде, а последующее создание уже попало в следующую секунду.
+            // Этот кейс крайне маловероятен, и будет проявляться только в дев режиме с включенным StrictMode,
+            // поэтому мы пренебрегаем решением этой проблемы, в угоду упрощения.
+            const date = fetchPolicy === 'network-only' ? new Date().setMilliseconds(0) : null;
+            const queryKey = [...rootKey, { fetchPolicy, date, isBackground, type }];
             const queryKeyHash = this.serialize(queryKey);
             const dataKeyHash = this.serialize([...rootKey, { type }]);
             const statusKeyHash = this.serialize([...rootKey, { type }]);

package/Mutation/Mutation.js

@@ -17,8 +17,7 @@ export class Mutation extends QueryContainer {
         this.sync = (options) => {
             const { onSuccess, onError, params } = options || {};
             this.auxiliary
-                .getUnifiedPromise(() => this.executor(params))
-                .then((resData) => {
+                .getUnifiedPromise(() => this.executor(params), (resData) => {
                 onSuccess === null || onSuccess === void 0 ? void 0 : onSuccess(resData);
             })
                 .catch((e) => {

package/Query/Query.js

@@ -47,8 +47,7 @@ export class Query extends QueryContainer {
         this.proceedSync = (options) => {
             const { onSuccess, onError } = options || {};
             this.auxiliary
-                .getUnifiedPromise(this.executor)
-                .then((res) => {
+                .getUnifiedPromise(this.executor, (res) => {
                 this.submitSuccess(res);
                 onSuccess === null || onSuccess === void 0 ? void 0 : onSuccess(res);
             })
@@ -70,9 +69,7 @@ export class Query extends QueryContainer {
             if (!this.isNetworkOnly && this.isSuccess && !this.auxiliary.isInvalid) {
                 return Promise.resolve(this.storage.data);
             }
-            return this.auxiliary
-                .getUnifiedPromise(this.executor)
-                .then(this.submitSuccess);
+            return this.auxiliary.getUnifiedPromise(this.executor, this.submitSuccess);
         };
         this.defaultOnError = onError;
         this.enabledAutoFetch = enabledAutoFetch;

package/package.json

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