@exodus/ui-config 3.13.1 → 3.13.2

package/CHANGELOG.md

@@ -3,6 +3,12 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [3.13.2](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/ui-config@3.13.1...@exodus/ui-config@3.13.2) (2026-01-13)
+
+### Bug Fixes
+
+- fix(ui-config): sync atoms in plugin instead of in atom definition (#14908)
+
 ## [3.13.1](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/ui-config@3.13.0...@exodus/ui-config@3.13.1) (2025-12-24)
 
 ### Bug Fixes

package/atoms/index.js

@@ -1,10 +1,10 @@
-import createSyncableUiConfigAtom from './syncable.js'
+import createSyncableRemoteAtom from './syncable.js'
 import createLocalUiConfigAtom from './local.js'
 
 const createUiConfigAtomDefinitions = ({ configValues }) => {
-  return configValues.map((configValue) =>
+  return configValues.flatMap((configValue) =>
     configValue.syncable
-      ? createSyncableUiConfigAtom(configValue)
+      ? [createLocalUiConfigAtom(configValue), createSyncableRemoteAtom(configValue)]
       : createLocalUiConfigAtom(configValue)
   )
 }

package/atoms/syncable.js

@@ -1,47 +1,19 @@
-/* eslint-disable @exodus/hydra/no-eternal-subscription */
-
-// eslint-disable-next-line no-restricted-imports -- TODO: Fix this the next time the file is edited.
-import lodash from 'lodash'
 import { createFusionAtom } from '@exodus/fusion-atoms'
-import { createStorageAtomFactory } from '@exodus/atoms'
-
-const { isEqual } = lodash
 
-const createSyncableUiConfigAtom = ({ id, atomId, encrypted, defaultValue }) => {
-  const factory = ({ fusion, storage }) => {
+const createSyncableAtom = ({ id, atomId, encrypted }) => {
+  const factory = ({ fusion }) => {
     const path = encrypted ? `private.${id}` : id
-
-    const localAtom = createStorageAtomFactory({ storage })({ key: id, isSoleWriter: true })
-    const fusionAtom = createFusionAtom({ fusion, path })
-
-    // Sync value up to fusion
-    localAtom.observe((value) => {
-      if (value === undefined) return // Avoid syncing down missing fusion values
-      fusionAtom.set((prevValue) => (isEqual(value, prevValue) ? prevValue : value))
-    })
-
-    // Sync value down from fusion
-    fusionAtom.observe((value) => {
-      if (value === undefined) return // Avoid syncing down missing fusion values
-      localAtom.set((prevValue) => (isEqual(value, prevValue) ? prevValue : value))
-    })
-
-    return localAtom
+    return createFusionAtom({ fusion, path })
   }
 
   return {
     definition: {
-      id: atomId,
+      id: `syncable${atomId}`,
       type: 'atom',
       factory,
-      dependencies: ['fusion', 'storage'],
-      public: true,
+      dependencies: ['fusion'],
     },
-    storage: { namespace: 'uiConfig' },
-    aliases: [
-      { implementationId: encrypted ? 'storage' : 'unsafeStorage', interfaceId: 'storage' },
-    ],
   }
 }
 
-export default createSyncableUiConfigAtom
+export default createSyncableAtom

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/ui-config",
-  "version": "3.13.1",
+  "version": "3.13.2",
   "description": "Helper for storing various UI-specific settings/config.",
   "author": "Exodus Movement, Inc.",
   "license": "MIT",
@@ -45,5 +45,5 @@
     "access": "public",
     "provenance": false
   },
-  "gitHead": "1306676a6484452bf215467d94d680d9311210bc"
+  "gitHead": "f23ac1572edd03edec89b37fae62cdb777f3f369"
 }

package/plugin/index.js

@@ -1,7 +1,12 @@
+// eslint-disable-next-line no-restricted-imports
+import lodash from 'lodash'
 import { createAtomObserver } from '@exodus/atoms'
 
+const { isEqual } = lodash
+
 const createUiConfigPluginDefinition = ({ configValues }) => {
   const atomIds = configValues.map((v) => v.atomId)
+  const syncableAtomIds = configValues.filter((v) => v.syncable).map((v) => `syncable${v.atomId}`)
 
   const factory = ({ port, ...atoms }) => {
     const items = configValues.map(({ id, atomId, ...config }) => ({
@@ -10,12 +15,43 @@ const createUiConfigPluginDefinition = ({ configValues }) => {
       config,
     }))
 
+    const syncAtoms = () => {
+      const subscriptions = configValues
+        .filter((config) => config.syncable)
+        .flatMap((config) => {
+          const localAtom = atoms[config.atomId]
+          const remoteAtom = atoms[`syncable${config.atomId}`]
+          return [
+            localAtom.observe((value) => {
+              if (value === undefined) return
+              remoteAtom.set((prevValue) => (isEqual(value, prevValue) ? prevValue : value))
+            }),
+
+            remoteAtom.observe((value) => {
+              if (value === undefined) return
+              localAtom.set((prevValue) => (isEqual(value, prevValue) ? prevValue : value))
+            }),
+          ]
+        })
+
+      return () => {
+        subscriptions.forEach((unsubscribe) => unsubscribe())
+      }
+    }
+
+    let unsubscribeSyncAtoms = () => {}
     return {
       onLoad: () => items.forEach(({ observer }) => observer.start()),
-      onStart: () => items.forEach(({ observer }) => observer.start()),
+      onStart: () => {
+        items.forEach(({ observer }) => observer.start())
+        unsubscribeSyncAtoms = syncAtoms()
+      },
       onClear: () =>
         Promise.all(items.map(({ atom, config }) => (config.persist ? null : atom.set(undefined)))),
-      onStop: () => items.forEach(({ observer }) => observer.unregister()),
+      onStop: () => {
+        items.forEach(({ observer }) => observer.unregister())
+        unsubscribeSyncAtoms()
+      },
     }
   }
 
@@ -24,7 +60,7 @@ const createUiConfigPluginDefinition = ({ configValues }) => {
       type: 'plugin',
       id: 'uiConfigPlugin',
       factory,
-      dependencies: ['port', ...atomIds],
+      dependencies: ['port', ...atomIds, ...syncableAtomIds],
       public: true,
     },
   }