@exodus/atoms 7.2.0 → 7.3.1

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.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
+
+- don't proxyFreeze ReadonlySet and ReadonlyMap ([#6539](https://github.com/ExodusMovement/exodus-hydra/issues/6539)) ([ed7fa30](https://github.com/ExodusMovement/exodus-hydra/commit/ed7fa30b715df0ce3a1c551f3482988dbec3ec48))
+
 ## [7.2.0](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/atoms@7.1.0...@exodus/atoms@7.2.0) (2024-04-16)
 
 ### Features

package/lib/enforce-rules.js

@@ -1,6 +1,8 @@
 import makeConcurrent from 'make-concurrent';
 import proxyFreeze from 'proxy-freeze';
 import { isSetter } from './utils/guards.js';
+const isReadonlySetOrMap = (value) => ['[object ReadonlySet]', '[object ReadonlyMap]'].includes(Object.prototype.toString.call(value));
+const freeze = (value) => (isReadonlySetOrMap(value) ? value : proxyFreeze(value));
 const withChangeDetection = (listener) => {
     let currentValue;
     let called = false;
@@ -14,7 +16,7 @@ const withChangeDetection = (listener) => {
 };
 const enforceObservableRules = ({ defaultValue, getInitialized = () => true, ...atom }) => {
     const enqueue = makeConcurrent((fn) => fn(), { concurrency: 1 });
-    const postProcessValue = (value = defaultValue) => value && typeof value === 'object' ? proxyFreeze(value) : value;
+    const postProcessValue = (value = defaultValue) => value && typeof value === 'object' ? freeze(value) : value;
     const get = () => atom.get().then(postProcessValue);
     const observe = (listener) => {
         let called = false;

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;
         };

package/package.json

@@ -1,11 +1,11 @@
 {
   "name": "@exodus/atoms",
-  "version": "7.2.0",
+  "version": "7.3.1",
   "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": "fa602a1a0c09f5e0afe7218d05a85eb162ae84c8"
+  "gitHead": "2792d27a3830bea53f036b542b34a3ddce0eee6d"
 }