@subiz/widget-v4 4.1.210 → 4.1.211

package/src/store/kv.js

@@ -31,29 +31,19 @@ export default class {
 			inmemItemM[item.key] = item.value
 		})
 		this.cache = inmemItemM
-		// If the cache is full. Randomly remove half of the cache. credit to thanos
-		// this is so simple since we don't need to keep track of last modified time
-		// for each key. Implementing a LRU cache won't add much value.
-		let allkeys = Object.keys(this.cache)
-		if (allkeys.length > KvMaxSize) {
-			let liveChance = KvMaxSize / 2 / allkeys.lenth
-
-			let removekeys = []
-			for (var i = 0; i < KvMaxSize; i++) {
-				let live = Math.random() <= liveChance
-				if (live) continue
-
-				let key = allkeys[i]
-				delete this.cache[key]
-				removekeys.push(key)
-			}
-
-			this.db.removes('items', [removekeys])
-		}
+		this.trimCache()
 
 		this.initialized = true
 	}
 
+	trimCache() {
+		const keys = Object.keys(this.cache)
+		if (keys.length <= KvMaxSize) return
+		const removed = keys.slice(0, keys.length - Math.floor(KvMaxSize / 2))
+		for (const key of removed) delete this.cache[key]
+		this.db.removes('items', removed)
+	}
+
 	all() {
 		if (this.dead) return []
 		if (!this.initialized) return []
@@ -77,6 +67,7 @@ export default class {
 		// if multiple key
 		this.cache[key] = value
 		this.db.put('items', key, {key, value})
+		this.trimCache()
 	}
 
 	// delete an record in a schema

package/src/store/lsdb.js

@@ -75,5 +75,5 @@ export default class {
 
 function toString(obj) {
 	if (obj === undefined) return 'null'
-	return JSON.stringify(obj)
+	return sb.stringify(obj)
 }