@exodus/atoms 1.1.0 → 2.1.0

package/index.js

@@ -73,19 +73,48 @@ export const createFusionAtomFactory =
     })
   }
 
+const createSimpleObserver = ({ enable = true } = {}) => {
+  let listeners = []
+  const notify = async (value) => Promise.all(listeners.map((listener) => listener(value)))
+
+  const observe = (listener) => {
+    if (!enable) {
+      throw new Error('observe method is not supported')
+    }
+    listeners.push(listener)
+    return () => {
+      listeners = listeners.filter((fn) => fn !== listener)
+    }
+  }
+
+  return {
+    observe,
+    notify,
+  }
+}
+
 export const createStorageAtomFactory =
   ({ storage }) =>
-  ({ key, defaultValue }) => {
-    const set = async (value) => storage.set(key, value)
+  ({ key, defaultValue, isSoleWriter }) => {
+    const { notify, observe } = createSimpleObserver({ enable: isSoleWriter })
+
+    const set = async (value) => {
+      if (typeof value === 'undefined') {
+        await storage.delete(key)
+      } else {
+        await storage.set(key, value)
+      }
+      if (isSoleWriter) {
+        await notify(value)
+      }
+    }
 
     const get = () => storage.get(key)
 
     return enforceObservableRules({
       get,
       set,
-      observe: async () => {
-        throw new Error('storage atom does not support observe')
-      },
+      observe,
       defaultValue,
     })
   }
@@ -93,32 +122,14 @@ export const createStorageAtomFactory =
 export const createRemoteConfigAtomFactory =
   ({ remoteConfig }) =>
   ({ path, defaultValue }) => {
-    const getValue = (remoteConfigJSON) => getValueAtPath(remoteConfigJSON, path)
-
-    let listeners = []
-
-    const notify = async (remoteConfigJSON) => {
-      const value = getValue(remoteConfigJSON)
-      return Promise.all(listeners.map((listener) => listener(value)))
-    }
-
-    const get = async () => {
-      const remoteConfigJSON = await remoteConfig.get()
-      return getValue(remoteConfigJSON)
-    }
+    const { notify, observe } = createSimpleObserver()
 
+    const get = () => remoteConfig.get(path)
     const set = async () => {
       throw new Error('remoteConfig is read-only')
     }
 
-    const observe = (listener) => {
-      listeners.push(listener)
-      return () => {
-        listeners = listeners.filter((fn) => fn !== listener)
-      }
-    }
-
-    remoteConfig.on('remote-config', notify)
+    remoteConfig.on('sync', ({ current }) => notify(getValueAtPath(current, path)))
 
     return enforceObservableRules({
       get,
@@ -131,8 +142,8 @@ export const createRemoteConfigAtomFactory =
 export const createLocalConfigAtomFactory =
   ({ localConfig }) =>
   ({ key, defaultValue }) => {
-    const observe = (listener) =>
-      localConfig.on(`update:${key}`, ({ current }) => listener(current))
+    const { notify, observe } = createSimpleObserver()
+    localConfig.on(`update:${key}`, ({ current }) => notify(current))
 
     return enforceObservableRules({
       get: () => localConfig.get(key),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/atoms",
-  "version": "1.1.0",
+  "version": "2.1.0",
   "main": "index.js",
   "author": "Exodus Movement Inc.",
   "scripts": {
@@ -29,5 +29,5 @@
     "@exodus/storage-memory": "^1.0.0",
     "delay": "^5.0.0"
   },
-  "gitHead": "b7d8b3cefb9c5aff0a390c2745266bffa9bc5e52"
+  "gitHead": "231a041f31a91990024a1282067b737a3c9dfe41"
 }