http-request-manager 18.15.4 → 18.15.7

package/fesm2022/http-request-manager.mjs

@@ -6441,8 +6441,11 @@ class QueryParamsTrackerService {
         this.cleanupExpiredEntries();
         if (!normalized.hasQuery) {
             const pathState = this.ensurePathState(normalized.pathKey);
-            const pathSeenBefore = Object.keys(pathState.consumedValuesByKey).length > 0;
+            // Check if we've seen this path before by checking for a special marker
+            const pathSeenBefore = pathState.consumedValuesByKey.hasOwnProperty('__path_seen__');
             if (!pathSeenBefore) {
+                // Mark this path as seen
+                pathState.consumedValuesByKey['__path_seen__'] = ['true'];
                 this.persistState();
             }
             return !pathSeenBefore;
@@ -6811,6 +6814,7 @@ class HTTPManagerStateService extends ComponentStore {
             'if-none-match'
         ]);
         this.requestSignatureCache = {};
+        this.inFlightRequestSignatures = new Set();
         this._requestCachePaths = new Map();
         this.wsRetryAttempts = new BehaviorSubject(0);
         this.wsRetryAttempts$ = this.wsRetryAttempts.asObservable();
@@ -7204,6 +7208,18 @@ class HTTPManagerStateService extends ComponentStore {
             const fetchFromAPI = () => {
                 if (this.hasDatabase && this.databaseOptions?.table) {
                     this.setCachedRequestSignature(this.databaseOptions.table, 'GET', requestSignature);
+                    if (!this.tryBeginInFlightRequest(requestSignature)) {
+                        const currentStateData = this.get()?.data;
+                        if (Array.isArray(currentStateData) && currentStateData.length > 0) {
+                            return of(currentStateData);
+                        }
+                        if (currentStateData &&
+                            !Array.isArray(currentStateData) &&
+                            Object.keys(currentStateData).length > 0) {
+                            return of(currentStateData);
+                        }
+                        return of(this.dataType === DataType.ARRAY ? [] : {});
+                    }
                 }
                 return this.httpManagerService.getRequest(requestOptions, effectiveParams).pipe(tap((data) => {
                     // Extract array from paginated response if needed
@@ -7242,6 +7258,10 @@ class HTTPManagerStateService extends ComponentStore {
                         }));
                     }
                     return of(data);
+                }), finalize(() => {
+                    if (this.hasDatabase && this.databaseOptions?.table) {
+                        this.endInFlightRequest(requestSignature);
+                    }
                 }));
             };
             console.log('[DB STORAGE] Checking database storage:', {
@@ -7484,23 +7504,38 @@ class HTTPManagerStateService extends ComponentStore {
             // console.log('[DEBUG] Making streaming request:', requestOptions)
             if (this.hasDatabase && this.databaseOptions?.table) {
                 const requestSignature = this.buildRequestSignature('STREAM', requestOptions, effectiveParams);
+                const fetchStreamFromAPI = () => {
+                    if (!this.tryBeginInFlightRequest(requestSignature)) {
+                        const currentStateData = this.get()?.data;
+                        if (Array.isArray(currentStateData) && currentStateData.length > 0) {
+                            return of({ data: currentStateData, fromCache: true });
+                        }
+                        if (currentStateData &&
+                            !Array.isArray(currentStateData) &&
+                            Object.keys(currentStateData).length > 0) {
+                            return of({ data: currentStateData, fromCache: true });
+                        }
+                        return of({ data: this.dataType === DataType.ARRAY ? [] : {}, fromCache: true });
+                    }
+                    this.setCachedRequestSignature(this.databaseOptions.table, 'STREAM', requestSignature);
+                    return this.httpManagerService.getRequest(requestOptions, effectiveParams).pipe(map((apiData) => ({ data: apiData, fromCache: false })), finalize(() => this.endInFlightRequest(requestSignature)));
+                };
                 return this.localStorageManagerService.store$(this.databaseOptions.table).pipe(take(1), switchMap((storeData) => {
                     const forceRefresh = !!options?.forceRefresh;
                     const expires = storeData?.expires || 0;
                     const hasExpired = expires > 0 && this.utils.hasExpired(expires);
                     if (forceRefresh) {
-                        this.setCachedRequestSignature(this.databaseOptions.table, 'STREAM', requestSignature);
-                        return this.httpManagerService.getRequest(requestOptions, effectiveParams).pipe(map((apiData) => ({ data: apiData, fromCache: false })));
+                        return fetchStreamFromAPI();
                     }
                     if (hasExpired) {
-                        return this.dbManagerService.clearTable(this.databaseOptions.table).pipe(tap(() => this.setCachedRequestSignature(this.databaseOptions.table, 'STREAM', requestSignature)), switchMap(() => this.httpManagerService.getRequest(requestOptions, effectiveParams)), map((apiData) => ({ data: apiData, fromCache: false })));
+                        return this.dbManagerService.clearTable(this.databaseOptions.table).pipe(switchMap(() => fetchStreamFromAPI()));
                     }
                     const storedSchemaSignature = this.getStoredSchemaSignature(storeData);
                     const expectedSchema = this.buildSchemaFromAdapter();
                     const expectedSchemaSignature = this.buildSchemaSignature(expectedSchema);
                     if (storedSchemaSignature && storedSchemaSignature !== expectedSchemaSignature) {
                         const tableDef = TableSchemaDef.adapt({ table: this.databaseOptions.table, schema: expectedSchema });
-                        return this.dbManagerService.clearTable(this.databaseOptions.table).pipe(switchMap(() => this.dbManagerService.createDatabaseTable(tableDef)), tap(() => this.setCachedRequestSignature(this.databaseOptions.table, 'STREAM', requestSignature)), switchMap(() => this.httpManagerService.getRequest(requestOptions, effectiveParams)), map((apiData) => ({ data: apiData, fromCache: false })));
+                        return this.dbManagerService.clearTable(this.databaseOptions.table).pipe(switchMap(() => this.dbManagerService.createDatabaseTable(tableDef)), switchMap(() => fetchStreamFromAPI()));
                     }
                     const trackerAllowsRequest = this.checkTrackerAllowsRequest(this.databaseOptions.table, this.resolvePath(effectiveParams), options, storeData);
                     if (!trackerAllowsRequest) {
@@ -7520,8 +7555,7 @@ class HTTPManagerStateService extends ComponentStore {
                             return of({ data: this.dataType === DataType.ARRAY ? [] : {}, fromCache: true });
                         }));
                     }
-                    this.setCachedRequestSignature(this.databaseOptions.table, 'STREAM', requestSignature);
-                    return this.httpManagerService.getRequest(requestOptions, effectiveParams).pipe(map((apiData) => ({ data: apiData, fromCache: false })));
+                    return fetchStreamFromAPI();
                 })).pipe(tap((packet) => {
                     const res = packet?.data;
                     // console.log('[DEBUG] Streaming response received:', res)
@@ -7543,6 +7577,7 @@ class HTTPManagerStateService extends ComponentStore {
                     return of([]);
                 }), finalize(() => this.httpManagerService.isPending.next(false)));
             }
+            // If no database is configured, always call the API (do not check tracker)
             return this.httpManagerService.getRequest(requestOptions, effectiveParams)
                 .pipe(tap((res) => {
                 console.log('[DEBUG] Streaming response received:', res);
@@ -8281,6 +8316,16 @@ class HTTPManagerStateService extends ComponentStore {
             [type]: signature,
         };
     }
+    tryBeginInFlightRequest(signature) {
+        if (this.inFlightRequestSignatures.has(signature)) {
+            return false;
+        }
+        this.inFlightRequestSignatures.add(signature);
+        return true;
+    }
+    endInFlightRequest(signature) {
+        this.inFlightRequestSignatures.delete(signature);
+    }
     getRequestCacheMetadata(storeData, type) {
         return storeData?.requestCache?.[type] || null;
     }