Merge pull request #31 from dwhiffing/fix-collection

Fixes collection issue
magiclabs · Aug 29, 2024 · db0eef1 · db0eef1
2 parents 873e749 + 2747793
commit db0eef1
Showing 1 changed file with 6 additions and 7 deletions.
diff --git a/utils/kvCollection.ts b/utils/kvCollection.ts
@@ -24,20 +24,19 @@ export class KVCollection<T> {
     const autoInc = await this.getAutoInc();
     const members = await this.get();
 
-    const valueToSet = [...members, { key: autoInc, ...item }];
+    const valueToSet = [...members, { key: autoInc, ...item }].filter(
+      (m) => m.key > -1,
+    );
 
     await kv.set(this.getStorageKey(), valueToSet);
 
     await this.onAutoInc();
   }
 
-  public async delete(key: string): Promise<void> {
+  public async delete(key: number): Promise<void> {
     const members = await this.get();
-    await kv.set(
-      this.getStorageKey(),
-      // @ts-ignore
-      members.filter((m) => m.key !== key),
-    );
+    const valueToSet = members.filter((m) => m.key !== key && m.key > -1);
+    await kv.set(this.getStorageKey(), valueToSet);
   }
 
   public async get(): Promise<(T & { key: number })[]> {