@adventurelabs/scout-core 1.4.23 → 1.4.24

package/dist/hooks/useInfiniteQuery.js

@@ -419,6 +419,9 @@ export const useInfiniteFeedByHerd = (herdId, options) => {
     const [currentCursor, setCurrentCursor] = useState(null);
     const prevHerdIdRef = useRef();
     const lastAddedCursorRef = useRef(undefined);
+    // Store hasMore/nextCursor from the last merged response (like dummy's currentResult) so loadMore
+    // always has a usable nextCursor even when currentQuery has switched to the next request.
+    const lastResultRef = useRef(null);
     const currentQuery = useGetFeedInfiniteByHerdQuery({
         herdId,
         limit: options.limit || 20,
@@ -433,6 +436,7 @@ export const useInfiniteFeedByHerd = (herdId, options) => {
             setPages([]);
             setCurrentCursor(null);
             lastAddedCursorRef.current = undefined;
+            lastResultRef.current = null;
         }
         prevHerdIdRef.current = herdId;
     }, [herdId, options.enabled]);
@@ -447,14 +451,18 @@ export const useInfiniteFeedByHerd = (herdId, options) => {
         if (pages.length > 0 &&
             feedCursorEq(lastAddedCursorRef.current ?? null, currentCursor))
             return;
+        const items = Array.isArray(currentQuery.data?.items)
+            ? currentQuery.data.items
+            : [];
+        const limit = options.limit || 20;
+        // Store hasMore/nextCursor from this response so loadMore can use it (like dummy's currentResult)
+        lastResultRef.current = {
+            hasMore: currentQuery.data.hasMore ?? false,
+            nextCursor: currentQuery.data.nextCursor ?? null,
+        };
         setPages((prev) => {
             const existingPage = prev.find((p) => feedCursorEq(p.cursor, currentCursor));
             if (!existingPage) {
-                const items = Array.isArray(currentQuery.data?.items)
-                    ? currentQuery.data.items
-                    : [];
-                const limit = options.limit || 20;
-                // Only mark "last added" for full pages so partial/empty responses don't block a later merge
                 if (items.length >= limit) {
                     lastAddedCursorRef.current = currentCursor;
                 }
@@ -467,16 +475,18 @@ export const useInfiniteFeedByHerd = (herdId, options) => {
         });
     }, [currentQuery.data, currentQuery.isLoading, currentCursor, pages.length, options.limit]);
     const loadMore = useCallback(() => {
-        if (currentQuery.data?.hasMore &&
-            currentQuery.data.nextCursor &&
+        const result = lastResultRef.current;
+        if (result?.hasMore &&
+            result.nextCursor != null &&
             !currentQuery.isLoading) {
-            setCurrentCursor(currentQuery.data.nextCursor);
+            setCurrentCursor(result.nextCursor);
         }
-    }, [currentQuery.data, currentQuery.isLoading]);
+    }, [currentQuery.isLoading]);
     const refetch = useCallback(() => {
         setPages([]);
         setCurrentCursor(null);
         lastAddedCursorRef.current = undefined;
+        lastResultRef.current = null;
         currentQuery.refetch();
     }, [currentQuery]);
     const allItems = useMemo(() => {
@@ -493,7 +503,7 @@ export const useInfiniteFeedByHerd = (herdId, options) => {
         items: allItems,
         isLoading: currentQuery.isLoading && pages.length === 0,
         isLoadingMore: currentQuery.isLoading && pages.length > 0,
-        hasMore: (currentQuery.data?.hasMore ??
+        hasMore: (lastResultRef.current?.hasMore ??
             (currentCursor !== null && pages.length > 0)) ??
             false,
         loadMore,
@@ -506,6 +516,7 @@ export const useInfiniteFeedByDevice = (deviceId, options) => {
     const [currentCursor, setCurrentCursor] = useState(null);
     const prevDeviceIdRef = useRef();
     const lastAddedCursorRef = useRef(undefined);
+    const lastResultRef = useRef(null);
     const currentQuery = useGetFeedInfiniteByDeviceQuery({
         deviceId,
         limit: options.limit || 20,
@@ -520,6 +531,7 @@ export const useInfiniteFeedByDevice = (deviceId, options) => {
             setPages([]);
             setCurrentCursor(null);
             lastAddedCursorRef.current = undefined;
+            lastResultRef.current = null;
         }
         prevDeviceIdRef.current = deviceId;
     }, [deviceId, options.enabled]);
@@ -543,6 +555,10 @@ export const useInfiniteFeedByDevice = (deviceId, options) => {
                 if (items.length >= limit) {
                     lastAddedCursorRef.current = currentCursor;
                 }
+                lastResultRef.current = {
+                    hasMore: currentQuery.data?.hasMore ?? false,
+                    nextCursor: currentQuery.data?.nextCursor ?? null,
+                };
                 return [
                     ...prev,
                     { cursor: currentCursor, data: items },
@@ -552,16 +568,18 @@ export const useInfiniteFeedByDevice = (deviceId, options) => {
         });
     }, [currentQuery.data, currentQuery.isLoading, currentCursor, pages.length, options.limit]);
     const loadMore = useCallback(() => {
-        if (currentQuery.data?.hasMore &&
-            currentQuery.data.nextCursor &&
+        const result = lastResultRef.current;
+        if (result?.hasMore &&
+            result.nextCursor != null &&
             !currentQuery.isLoading) {
-            setCurrentCursor(currentQuery.data.nextCursor);
+            setCurrentCursor(result.nextCursor);
         }
-    }, [currentQuery.data, currentQuery.isLoading]);
+    }, [currentQuery.isLoading]);
     const refetch = useCallback(() => {
         setPages([]);
         setCurrentCursor(null);
         lastAddedCursorRef.current = undefined;
+        lastResultRef.current = null;
         currentQuery.refetch();
     }, [currentQuery]);
     const allItems = useMemo(() => {
@@ -578,7 +596,7 @@ export const useInfiniteFeedByDevice = (deviceId, options) => {
         items: allItems,
         isLoading: currentQuery.isLoading && pages.length === 0,
         isLoadingMore: currentQuery.isLoading && pages.length > 0,
-        hasMore: (currentQuery.data?.hasMore ??
+        hasMore: (lastResultRef.current?.hasMore ??
             (currentCursor !== null && pages.length > 0)) ??
             false,
         loadMore,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adventurelabs/scout-core",
-  "version": "1.4.23",
+  "version": "1.4.24",
   "description": "Core utilities and helpers for Adventure Labs Scout applications",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",