Adjusted the expiry time for the cache values (#17)

- Set the default expire time to null (no expire).
- Adjust expire of different components.

Author: FelixHertweck

src/hooks/api/useGetHistoryApi.ts

@@ -43,7 +43,7 @@ export default function useGetHistoryApi(): UseGetHistoryApiResponse {
         const response = await getHistory(options);
         if (response.data) {
           setError(null);
-          setCache("history" + sessionId, response.data, 120);
+          setCache("history" + sessionId, response.data);
           setLoading(false);
           setLastSessionId(sessionId);
           return response.data;

src/hooks/api/useGetKnowledgeApi.ts

@@ -64,7 +64,7 @@ export default function useGetKnowledgeApi({
             setCacheNotFound(cacheNotFoundKey, true, null);
             throw new Error("Knowledge not found");
           } else {
-            setCache(cacheKey, response.data, 120);
+            setCache(cacheKey, response.data);
           }
           setKnowledge(response.data);
         })

src/hooks/api/useGetModelsApi.ts

@@ -28,10 +28,10 @@ export default function useAvailableModels(): UseAvailableModelsResponse {
       getModels()
         .then((response) => {
           if (response.data) {
-            setCache("models", response.data, 120);
+            setCache("models", response.data);
             setModels(response.data);
           } else {
-            setCache("models", [], 120);
+            setCache("models", []);
             setModels([]);
           }
         })

src/services/CacheService.tsx

@@ -40,7 +40,7 @@ export default function CacheProvider<T>({
 
   // ttl in seconds
   // ttl = null means cache forever, so no expiry
-  function setCache(key: string, value: T, ttl: number | null = 10) {
+  function setCache(key: string, value: T, ttl: number | null = null) {
     if (ttl === null) {
       map.set(key, {
         expiry: null,

src/services/GetKnowledgeListService.tsx

@@ -48,7 +48,7 @@ export default function GetKnowledgeListProvider({
         const response = await getAllKnowledge();
         setError(null);
         if (response.data) {
-          setCache("allKnowledge", response.data, 120);
+          setCache("allKnowledge", response.data);
           setAllKnowledge(response.data);
         }
       } catch (error) {
@@ -72,7 +72,7 @@ export default function GetKnowledgeListProvider({
       (k) => k.id !== knowledge.id,
     );
     setAllKnowledge(updatedKnowledge);
-    setCache("allKnowledge", updatedKnowledge, 120);
+    setCache("allKnowledge", updatedKnowledge);
   };
 
   // add multiple knowledge to local state
@@ -92,7 +92,7 @@ export default function GetKnowledgeListProvider({
     }
     const updatedKnowledge = allKnowledgeRef.current.concat(knowledgeToAdd);
     setAllKnowledge(updatedKnowledge);
-    setCache("allKnowledge", updatedKnowledge, 120);
+    setCache("allKnowledge", updatedKnowledge);
   };
 
   // update Knowledge in local state
@@ -107,7 +107,7 @@ export default function GetKnowledgeListProvider({
     }
     const updatedKnowledges = filteredKnowledges.concat(newKnowledgeArray);
     setAllKnowledge(updatedKnowledges);
-    setCache("allKnowledge", updatedKnowledges, 120);
+    setCache("allKnowledge", updatedKnowledges);
   };
 
   const contextValue = useMemo(

src/services/GetSessionsService.tsx

@@ -65,7 +65,7 @@ export default function GetSessionsProvider({
           setError(null);
           if (response.data) {
             setSessions(response.data);
-            setCache("sessions", response.data, 120);
+            setCache("sessions", response.data);
           } else {
             setSessions([]);
           }

src/tests/services/CacheService.test.tsx

@@ -65,7 +65,7 @@ describe("CacheProvider", () => {
 
     act(() => {
       // Set cache with 10 seconds TTL (default)
-      result.current.setCache("shortLived", "will expire");
+      result.current.setCache("shortLived", "will expire", 10);
     });
 
     expect(result.current.getCache("shortLived")).toBe("will expire");

src/tests/services/GetKnowledgeListProvider.test.tsx

@@ -156,11 +156,7 @@ describe("GetKnowledgeListProvider", () => {
     // Wait for the API call to complete
     await waitFor(() => {
       expect(getAllKnowledge).toHaveBeenCalledTimes(1);
-      expect(mockSetCache).toHaveBeenCalledWith(
-        "allKnowledge",
-        mockKnowledge,
-        120,
-      );
+      expect(mockSetCache).toHaveBeenCalledWith("allKnowledge", mockKnowledge);
       expect(contextValue.allKnowledge).toEqual(mockKnowledge);
     });
   });
@@ -246,7 +242,6 @@ describe("GetKnowledgeListProvider", () => {
     expect(mockSetCache).toHaveBeenCalledWith(
       "allKnowledge",
       expectedKnowledge,
-      120,
     );
   });
 
@@ -317,7 +312,6 @@ describe("GetKnowledgeListProvider", () => {
     expect(mockSetCache).toHaveBeenCalledWith(
       "allKnowledge",
       expectedKnowledge,
-      120,
     );
   });
 
@@ -442,7 +436,6 @@ describe("GetKnowledgeListProvider", () => {
     expect(mockSetCache).toHaveBeenCalledWith(
       "allKnowledge",
       expectedKnowledge,
-      120,
     );
   });