FFM-9574 Upgrade lru-cache to version 7.18.0 (#88)

* FFM-9574 RC for customer PR

* FFM-9574 RC for customer PR

* FFM-9574 Upgrade lru-cache to 7.0.0

* FFM-9574 Use as * syntax for LRU

* FFM-9574 Use latest minor version 7 of lib. Re-add deleted example code

* FFM-9574 Package-lock of example code

* FFM-9574 1.3.4 release prep

Author: erdirowlands

examples/getting_started/package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@harnessio/ff-nodejs-server-sdk",
-  "version": "1.3.3",
+  "version": "1.3.4",
   "description": "Feature flags SDK for NodeJS environments",
   "main": "dist/cjs/index.js",
   "module": "dist/esm/index.mjs",
@@ -61,13 +61,12 @@
     "url": "https://github.com/harness/ff-nodejs-server-sdk"
   },
   "dependencies": {
-    "@types/lru-cache": "^7.10.10",
     "axios": "^0.22.0",
     "axios-retry": "^3.2.0",
     "jwt-decode": "^3.1.2",
     "keyv": "^4.0.3",
     "keyv-file": "^0.2.0",
-    "lru-cache": "^6.0.0",
+    "lru-cache": "^7.18.0",
     "murmurhash": "^2.0.0",
     "tslib": "~2.3.0"
   },

package-lock.json

@@ -11,11 +11,13 @@ export class SimpleCache implements KeyValueStore {
     return this.cache[key];
   }
 
-  del(key: string): void {
+  delete(key: string): void {
     delete this.cache[key];
   }
 
-  keys(): string[] {
-    return Object.keys(this.cache);
+  *keys(): Generator<string, void, void> {
+    for (const key of Object.keys(this.cache)) {
+      yield key;
+    }
   }
 }

package.json

@@ -1,5 +1,5 @@
 import Client, { Event } from './client';
-import LRU from 'lru-cache';
+import * as LRU from 'lru-cache';
 import { Options, Target } from './types';
 import { Logger } from './log';
 import { AsyncKeyValueStore, KeyValueStore } from './types';

src/cache.ts

@@ -45,7 +45,7 @@ export class StorageRepository implements Repository {
     const flagKey = this.formatFlagKey(identifier);
     if (this.store) {
       await this.store.set(flagKey, fc);
-      this.cache.del(flagKey);
+      this.cache.delete(flagKey);
     } else {
       this.cache.set(flagKey, fc);
     }
@@ -61,7 +61,7 @@ export class StorageRepository implements Repository {
     const segmentKey = this.formatSegmentKey(identifier);
     if (this.store) {
       await this.store.set(segmentKey, segment);
-      this.cache.del(segmentKey);
+      this.cache.delete(segmentKey);
     } else {
       this.cache.set(segmentKey, segment);
     }
@@ -75,7 +75,7 @@ export class StorageRepository implements Repository {
     if (this.store) {
       await this.store.del(flagKey);
     }
-    this.cache.del(flagKey);
+    this.cache.delete(flagKey);
     if (this.eventBus) {
       this.eventBus.emit(RepositoryEvent.FLAG_DELETED, identifier);
     }
@@ -86,7 +86,7 @@ export class StorageRepository implements Repository {
     if (this.store) {
       await this.store.del(segmentKey);
     }
-    this.cache.del(segmentKey);
+    this.cache.delete(segmentKey);
     if (this.eventBus) {
       this.eventBus.emit(RepositoryEvent.SEGMENT_DELETED, identifier);
     }
@@ -172,4 +172,6 @@ export class StorageRepository implements Repository {
   private formatSegmentKey(key: string): string {
     return `segments/${key}`;
   }
+
+
 }

src/index.ts

@@ -25,7 +25,11 @@ export class FileStore implements AsyncKeyValueStore {
     return this.keyv.delete(key);
   }
 
-  keys(): Promise<string[]> {
-    return Promise.resolve(this.keyvFile.keys());
+  keys(): Promise<Generator<string, void, void>> {
+    return Promise.resolve((function* (keys: string[]) {
+      for (const key of keys) {
+        yield key;
+      }
+    })(this.keyvFile.keys()));
   }
 }

src/repository.ts

@@ -65,15 +65,15 @@ export interface Query {
 export interface KeyValueStore {
   set(key: string, value: unknown): void;
   get(key: string): unknown;
-  del(key: string): void;
-  keys(): string[];
+  delete(key: string): void;
+  keys(): Generator<string, void, void>;
 }
 
 export interface AsyncKeyValueStore {
   set(key: string, value: unknown): Promise<true>;
   get<T>(key: string): Promise<T>;
   del(key: string): Promise<boolean>;
-  keys(): Promise<string[]>;
+  keys(): Promise<Generator<string, void, void>>;
 }
 
 export interface Target {

src/store.ts

@@ -1 +1 @@
-export const VERSION = '1.3.3';
+export const VERSION = "1.3.4";