Treat cache as promise

launchdarkly · Jan 29, 2025 · e8a7400 · e8a7400
1 parent 63daf60
commit e8a7400
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/packages/shared/akamai-edgeworker-sdk/src/api/LDClient.ts b/packages/shared/akamai-edgeworker-sdk/src/api/LDClient.ts
@@ -34,6 +34,10 @@ class LDClient extends LDClientImpl {
     this._cacheableStoreProvider = storeProvider;
   }
 
+  override initialized(): boolean {
+    return true;
+  }
+
   override waitForInitialization(): Promise<LDClientType> {
     // we need to resolve the promise immediately because Akamai's runtime doesnt
     // have a setimeout so everything executes synchronously.

diff --git a/packages/shared/akamai-edgeworker-sdk/src/featureStore/cacheableStoreProvider.ts b/packages/shared/akamai-edgeworker-sdk/src/featureStore/cacheableStoreProvider.ts
@@ -22,7 +22,7 @@ import { EdgeProvider } from '.';
  * - 10 for enterprise
  */
 export default class CacheableStoreProvider implements EdgeProvider {
-  cache: string | null | undefined;
+  cache: Promise<string | null | undefined> | null | undefined;
   cachedAt: number | undefined;
 
   constructor(
@@ -38,10 +38,7 @@ export default class CacheableStoreProvider implements EdgeProvider {
    */
   async get(rootKey: string): Promise<string | null | undefined> {
     if (!this._isCacheValid()) {
-      const updatedResults = await this._edgeProvider.get(rootKey);
-      if (updatedResults !== undefined) {
-        this.cache = updatedResults;
-      }
+      this.cache = this._edgeProvider.get(rootKey);
       this.cachedAt = Date.now();
     }