Fix LocalForageConfigStore storing stringified data (#385)

configu · Mar 11, 2024 · b6f4dd1 · b6f4dd1
1 parent 68d3270
commit b6f4dd1
Showing 1 changed file with 17 additions and 2 deletions.
diff --git a/ts/packages/browser/src/stores/LocalForage.ts b/ts/packages/browser/src/stores/LocalForage.ts
@@ -16,12 +16,27 @@ export class LocalForageConfigStore extends KeyValueConfigStore {
     if (typeof data === 'string') {
       return data ?? '';
     }
-    return JSON.stringify(data) ?? '';
+    return data === null ? '' : JSON.stringify(data) ?? '';
   }
 
   // * https://localforage.github.io/localForage/#data-api-setitem
   async upsert(key: string, value: string): Promise<void> {
-    await this.client.setItem(key, value);
+    let parsedValue;
+    try {
+      parsedValue = JSON.parse(value, (__, jsonValue) => {
+        if (typeof jsonValue === 'string') {
+          try {
+            return JSON.parse(jsonValue);
+          } catch (e) {
+            return jsonValue;
+          }
+        }
+        return jsonValue;
+      });
+    } catch {
+      parsedValue = value;
+    }
+    await this.client.setItem(key, parsedValue);
   }
 
   // * https://localforage.github.io/localForage/#data-api-removeitem