@t8/react-store 1.0.30 → 1.0.31

package/dist/index.js

@@ -34,7 +34,6 @@ import { useEffect, useMemo, useRef, useState } from "react";
 function useStore(store, shouldUpdate = true) {
   if (!isStore(store)) throw new Error("'store' is not an instance of Store");
   let [, setRevision] = useState(-1);
-  let initedRef = useRef(false);
   let state = store.getState();
   let setState = useMemo(() => store.setState.bind(store), [store]);
   let initialStoreRevision = useRef(store.revision);
@@ -44,14 +43,10 @@ function useStore(store, shouldUpdate = true) {
       if (typeof shouldUpdate !== "function" || shouldUpdate(nextState, prevState))
         setRevision(Math.random());
     });
-    if (!initedRef.current) {
-      initedRef.current = true;
-      if (store.revision !== initialStoreRevision.current)
-        setRevision(Math.random());
-    }
+    if (store.revision !== initialStoreRevision.current)
+      setRevision(Math.random());
     return () => {
       unsubscribe();
-      initedRef.current = false;
       initialStoreRevision.current = store.revision;
     };
   }, [store, shouldUpdate]);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@t8/react-store",
-  "version": "1.0.30",
+  "version": "1.0.31",
   "description": "Concise shared state management for React apps",
   "main": "dist/index.js",
   "type": "module",

package/src/useStore.ts

@@ -13,23 +13,24 @@ export function useStore<T>(
    *
    * @defaultValue `true`
    *
-   * Can be set to `false` when the component only requires the
-   * store state setter but not the state value itself, and so the
-   * component doesn't need to respond to updates in the store state.
+   * `shouldUpdate` can be set to `false` to prevent subscription
+   * to the store updates. Use case: when the component only requires
+   * the store state setter but not the store state value, the
+   * component may not need to respond to the store updates:
    *
    * ```ts
    * let [, setValue] = useStore(store, false);
    * ```
    *
-   * Can be set to a function `(nextState, prevState) => boolean` to
-   * make the component respond only to specific store state changes.
+   * `shouldUpdate` can be set to a function `(nextState, prevState) => boolean`
+   * to make the component respond only to specific store state changes,
+   * when this function returns `true`.
    */
   shouldUpdate: ShouldUpdate<T> = true,
 ): [T, SetStoreState<T>] {
   if (!isStore(store)) throw new Error("'store' is not an instance of Store");
 
   let [, setRevision] = useState(-1);
-  let initedRef = useRef(false);
 
   let state = store.getState();
   let setState = useMemo(() => store.setState.bind(store), [store]);
@@ -46,18 +47,11 @@ export function useStore<T>(
         setRevision(Math.random());
     });
 
-    // Trigger a rerender if the store state changed after the component
-    // last read it and before the component subscribed to its changes.
-    if (!initedRef.current) {
-      initedRef.current = true;
-
-      if (store.revision !== initialStoreRevision.current)
-        setRevision(Math.random());
-    }
+    if (store.revision !== initialStoreRevision.current)
+      setRevision(Math.random());
 
     return () => {
       unsubscribe();
-      initedRef.current = false;
       initialStoreRevision.current = store.revision;
     };
   }, [store, shouldUpdate]);