feat(#169): cache expiry for remote places

package.json

@@ -30,6 +30,7 @@
     "liquidjs": "^10.9.2",
     "lodash": "^4.17.21",
     "luxon": "^3.4.4",
+    "node-cache": "^5.1.2",
     "pino-pretty": "^10.2.3",
     "typescript": "^5.2.2",
     "uuid": "^9.0.1"

src/lib/remote-place-cache.ts

@@ -1,5 +1,8 @@
+import NodeCache from 'node-cache';
 import Place from '../services/place';
-import { ChtApi, RemotePlace } from './cht-api';
+import {ChtApi, RemotePlace} from './cht-api';
+
+const CACHE_KEY = 'CACHE_REMOTE_PLACE';
 
 type RemotePlacesByType = {
   [key: string]: RemotePlace[];
@@ -10,12 +13,14 @@ type RemotePlaceDatastore = {
 };
 
 export default class RemotePlaceCache {
-  private static cache: RemotePlaceDatastore = {};
+  // 60 min cache
+  private static cache = new NodeCache({
+    stdTTL: 60 * 60
+  });
 
   public static async getPlacesWithType(chtApi: ChtApi, placeType: string)
     : Promise<RemotePlace[]> {
-    const domainStore = await RemotePlaceCache.getDomainStore(chtApi, placeType);
-    return domainStore;
+    return await RemotePlaceCache.getDomainStore(chtApi, placeType);
   }
 
   public static async add(place: Place, chtApi: ChtApi): Promise<void> {
@@ -25,19 +30,27 @@ export default class RemotePlaceCache {
 
   public static clear(chtApi: ChtApi, contactTypeName?: string): void {
     const domain = chtApi?.chtSession?.authInfo?.domain;
+    let placeCache = RemotePlaceCache.cache.get<RemotePlaceDatastore>(CACHE_KEY);
+    if (!placeCache) {
+      return;
+    }
     if (!domain) {
-      RemotePlaceCache.cache = {};
+      placeCache = {};
     } else if (!contactTypeName) {
-      delete RemotePlaceCache.cache[domain];
+      delete placeCache[domain];
     } else {
-      delete RemotePlaceCache.cache[domain][contactTypeName];
+      delete placeCache[domain][contactTypeName];
     }
+    RemotePlaceCache.cache.set<RemotePlaceDatastore>(CACHE_KEY, placeCache);
   }
 
   private static async getDomainStore(chtApi: ChtApi, placeType: string)
     : Promise<RemotePlace[]> {
     const { domain } = chtApi.chtSession.authInfo;
-    const { cache: domainCache } = RemotePlaceCache;
+    let domainCache = RemotePlaceCache.cache.get<RemotePlaceDatastore>(CACHE_KEY);
+    if (!domainCache) {
+      domainCache = {};
+    }
 
     const places = domainCache[domain]?.[placeType];
     if (!places) {
@@ -46,6 +59,7 @@ export default class RemotePlaceCache {
         ...domainCache[domain],
         [placeType]: await fetchPlacesWithType,
       };
+      RemotePlaceCache.cache.set<RemotePlaceDatastore>(CACHE_KEY, domainCache);
     }
 
     return domainCache[domain][placeType];