@exodus/atoms 7.3.0 → 7.3.2

package/CHANGELOG.md

@@ -3,6 +3,18 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [7.3.2](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/atoms@7.3.1...@exodus/atoms@7.3.2) (2024-05-09)
+
+### Bug Fixes
+
+- prevent storage atoms of flooding reads on boot ([#6877](https://github.com/ExodusMovement/exodus-hydra/issues/6877)) ([23c1e79](https://github.com/ExodusMovement/exodus-hydra/commit/23c1e798808ac7417e151e627cb8fced5ef55454))
+
+## [7.3.1](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/atoms@7.3.0...@exodus/atoms@7.3.1) (2024-05-09)
+
+### Bug Fixes
+
+- atoms should not hit storage after first read or write ([#6834](https://github.com/ExodusMovement/exodus-hydra/issues/6834)) ([63e23f1](https://github.com/ExodusMovement/exodus-hydra/commit/63e23f1f0bf33732f67b70feb9e95e854a8fd90e))
+
 ## [7.3.0](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/atoms@7.2.0...@exodus/atoms@7.3.0) (2024-04-17)
 
 ### Features

package/lib/enforce-rules.d.ts

@@ -1,8 +1,9 @@
 import { Atom } from './utils/types.js';
 type Params<T> = {
+    makeGetConcurrent?: boolean;
     getInitialized?: () => boolean;
     defaultValue?: T;
     set: (value: T) => Promise<void>;
 } & Omit<Atom<T>, 'set' | 'reset'>;
-declare const enforceObservableRules: <T>({ defaultValue, getInitialized, ...atom }: Params<T>) => Atom<T>;
+declare const enforceObservableRules: <T>({ defaultValue, makeGetConcurrent, getInitialized, ...atom }: Params<T>) => Atom<T>;
 export default enforceObservableRules;

package/lib/enforce-rules.js

@@ -14,10 +14,11 @@ const withChangeDetection = (listener) => {
         await listener(currentValue);
     };
 };
-const enforceObservableRules = ({ defaultValue, getInitialized = () => true, ...atom }) => {
+const enforceObservableRules = ({ defaultValue, makeGetConcurrent = false, getInitialized = () => true, ...atom }) => {
     const enqueue = makeConcurrent((fn) => fn(), { concurrency: 1 });
     const postProcessValue = (value = defaultValue) => value && typeof value === 'object' ? freeze(value) : value;
-    const get = () => atom.get().then(postProcessValue);
+    const _get = makeGetConcurrent ? makeConcurrent(atom.get) : atom.get;
+    const get = () => _get().then(postProcessValue);
     const observe = (listener) => {
         let called = false;
         let valueEmittedFromGet;

package/lib/factories/storage.js

@@ -4,6 +4,7 @@ import pDefer from 'p-defer';
 const createStorageAtomFactory = ({ storage }) => {
     function createStorageAtom({ key, defaultValue, isSoleWriter, }) {
         const { notify, observe } = createSimpleObserver({ enable: isSoleWriter });
+        let canUseCached = false;
         let cached;
         let writePromiseDefer;
         const set = async (value) => {
@@ -18,22 +19,24 @@ const createStorageAtomFactory = ({ storage }) => {
             writePromiseDefer.resolve();
             if (isSoleWriter) {
                 cached = value;
+                canUseCached = value !== undefined;
                 await notify(value);
             }
         };
         const get = async () => {
-            if (cached) {
+            if (canUseCached) {
                 return cached;
             }
             if (writePromiseDefer) {
                 await writePromiseDefer.promise;
             }
-            if (cached) {
+            if (canUseCached) {
                 return cached;
             }
             const value = await storage.get(key);
             if (isSoleWriter) {
                 cached = value;
+                canUseCached = true;
             }
             return value;
         };
@@ -42,6 +45,7 @@ const createStorageAtomFactory = ({ storage }) => {
             set,
             observe,
             defaultValue,
+            makeGetConcurrent: isSoleWriter,
         });
     }
     return createStorageAtom;

package/package.json

@@ -1,11 +1,11 @@
 {
   "name": "@exodus/atoms",
-  "version": "7.3.0",
+  "version": "7.3.2",
   "description": "Abstraction for encapsulating a piece of data behind a simple unified interface: get, set, observe",
   "type": "module",
   "main": "lib/index.js",
   "exports": "./lib/index.js",
-  "author": "Exodus Movement Inc.",
+  "author": "Exodus Movement, Inc.",
   "scripts": {
     "build": "run -T tsc --build tsconfig.build.json",
     "clean": "run -T tsc --build --clean",
@@ -39,11 +39,11 @@
   },
   "devDependencies": {
     "@exodus/atom-tests": "^1.0.0",
-    "@exodus/remote-config": "^2.3.0",
+    "@exodus/remote-config": "^2.4.0",
     "@exodus/storage-memory": "^2.1.1",
     "@types/jest": "^29.5.11",
     "@types/lodash": "^4.14.200",
     "@types/minimalistic-assert": "^1.0.2"
   },
-  "gitHead": "1a06c9388e1cb50ddcaa6dd4c92c924a48996d57"
+  "gitHead": "d826264da11cdb907877d8796d882f8e488abcea"
 }