@uniflowed/state 0.2.0 → 0.4.0

package/internal/atom.js

@@ -79,8 +79,13 @@ export type AtomSetter = <V, A>(target: AtomRecord<V, A>, arg: A) => void;
  * of `AtomRecord<T, empty>`: a writable atom is accepted anywhere a readable
  * one is, and the argument type stays exact at the call site rather than
  * widening to `mixed` the moment an atom is passed somewhere general.
+ *
+ * `in A` says so to the checker. `A` appears only as the write's argument, so
+ * it is contravariant, but Flow compares two applications of an alias by the
+ * variance written on its parameters. Without the annotation, `get(value)` on
+ * a primitive was "SetAction<T> is incompatible with empty".
  */
-export type AtomRecord<T, A> = {
+export type AtomRecord<T, in A> = {
   readonly kind: "primitive" | "derived" | "async",
   /** For diagnostics only. Never load-bearing. */
   readonly label: string,

package/internal/composed.js

@@ -530,7 +530,14 @@ export function createJSONStorage<T>(
         return;
       }
       try {
-        store.setItem(key, JSON.stringify(value));
+        const serialized = JSON.stringify(value);
+        // `undefined` has no JSON. Nothing stored reads back as the initial
+        // value, which is what a stored "undefined" failing to parse did.
+        if (serialized === undefined) {
+          store.removeItem(key);
+        } else {
+          store.setItem(key, serialized);
+        }
       } catch {
         // Out of quota, or site data blocked. The value is still the atom's;
         // it simply will not outlive the tab.
@@ -558,7 +565,7 @@ export function createJSONStorage<T>(
         if (event.key != null && event.key !== key) {
           return;
         }
-        onChange(parse(event.key == null ? null : event.newValue, initial));
+        onChange(parse(event.key == null ? null : (event.newValue ?? null), initial));
       };
       host.addEventListener("storage", listener);
       return () => {
@@ -816,7 +823,7 @@ export function unwrap<T>(
   return defineSelector((get) => {
     const settled = get(target);
     return match (settled) {
-      {state: "hasData", data: const data, ...} => data,
+      {state: "hasData", data: const data} => data,
       _ => fallback,
     };
   }, null);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uniflowed/state",
-  "version": "0.2.0",
+  "version": "0.4.0",
   "description": "Atoms and the React binding for @uniflowed/state, part of the Unified Toolchain for Flow.",
   "type": "module",
   "license": "MIT",
@@ -19,7 +19,7 @@
     "!*.test.js"
   ],
   "dependencies": {
-    "@uniflowed/cell": "0.2.0",
-    "@uniflowed/react": "0.2.0"
+    "@uniflowed/cell": "0.4.0",
+    "@uniflowed/react": "0.4.0"
   }
 }