@plq/use-persisted-state 1.4.1 → 1.4.2

package/lib/utils/get-persisted-value.js

@@ -2,29 +2,19 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.default = getPersistedValue;
 const is_1 = require("@plq/is");
-/**
- * Resolves the value a hook starts from: the persisted entry for `key` when the stored payload
- * carries one, the initial value otherwise.
- *
- * Presence is decided by the key rather than by the value, so a persisted `null` comes back as
- * the value the user set instead of being mistaken for an absence and replaced.
- */
+/** Resolves a hook's starting value by key presence, so a persisted `null` is not mistaken for an absence. */
 function getPersistedValue(key, initialValue, persist) {
     let initialPersist;
     try {
         initialPersist = persist ? JSON.parse(persist) : {};
     }
     catch (err) {
-        // A shared backend can hold a foreign or truncated entry, so a parse failure is expected here
-        // and must not stop the component mounting. It is reported rather than swallowed because the
-        // fallback discards whatever was persisted, and because the change path logs the same failure.
+        // A shared backend can hold a foreign or truncated entry, so a parse failure must not stop the mount.
         console.error("use-persisted-state: Can't parse value from storage", err);
         initialPersist = {};
     }
     let initialOrPersistedValue = (0, is_1.isFunction)(initialValue) ? initialValue() : initialValue;
-    // `JSON.parse` yields any JSON value, so a foreign entry can be a primitive,
-    // and `in` throws on one. This runs in the `useState` initializer, where a
-    // throw stops the component mounting instead of falling back.
+    // `in` throws on a primitive, and this runs in the `useState` initializer, where a throw stops the mount.
     if ((0, is_1.isObject)(initialPersist) && key in initialPersist) {
         initialOrPersistedValue = initialPersist[key];
     }

package/lib/utils/get-persisted-value.js.map

@@ -1 +1 @@
-{"version":3,"file":"get-persisted-value.js","sourceRoot":"","sources":["../../src/utils/get-persisted-value.ts"],"names":[],"mappings":";;AASA,oCAwBC;AAjCD,gCAA8C;AAE9C;;;;;;GAMG;AACH,SAAwB,iBAAiB,CAAI,GAAW,EAAE,YAA2B,EAAE,OAAgB;IACrG,IAAI,cAAuB,CAAA;IAE3B,IAAI,CAAC;QACH,cAAc,GAAG,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,EAAE,CAAA;IACrD,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,8FAA8F;QAC9F,6FAA6F;QAC7F,+FAA+F;QAC/F,OAAO,CAAC,KAAK,CAAC,qDAAqD,EAAE,GAAG,CAAC,CAAA;QAEzE,cAAc,GAAG,EAAE,CAAA;IACrB,CAAC;IAED,IAAI,uBAAuB,GAAG,IAAA,eAAU,EAAC,YAAY,CAAC,CAAC,CAAC,CAAC,YAAY,EAAE,CAAC,CAAC,CAAC,YAAY,CAAA;IAEtF,6EAA6E;IAC7E,2EAA2E;IAC3E,8DAA8D;IAC9D,IAAI,IAAA,aAAQ,EAAC,cAAc,CAAC,IAAI,GAAG,IAAI,cAAc,EAAE,CAAC;QACtD,uBAAuB,GAAG,cAAc,CAAC,GAAG,CAAM,CAAA;IACpD,CAAC;IAED,OAAO,uBAAuB,CAAA;AAChC,CAAC"}
+{"version":3,"file":"get-persisted-value.js","sourceRoot":"","sources":["../../src/utils/get-persisted-value.ts"],"names":[],"mappings":";;AAGA,oCAoBC;AAvBD,gCAA8C;AAE9C,8GAA8G;AAC9G,SAAwB,iBAAiB,CAAI,GAAW,EAAE,YAA2B,EAAE,OAAgB;IACrG,IAAI,cAAuB,CAAA;IAE3B,IAAI,CAAC;QACH,cAAc,GAAG,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,EAAE,CAAA;IACrD,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,sGAAsG;QACtG,OAAO,CAAC,KAAK,CAAC,qDAAqD,EAAE,GAAG,CAAC,CAAA;QAEzE,cAAc,GAAG,EAAE,CAAA;IACrB,CAAC;IAED,IAAI,uBAAuB,GAAG,IAAA,eAAU,EAAC,YAAY,CAAC,CAAC,CAAC,CAAC,YAAY,EAAE,CAAC,CAAC,CAAC,YAAY,CAAA;IAEtF,0GAA0G;IAC1G,IAAI,IAAA,aAAQ,EAAC,cAAc,CAAC,IAAI,GAAG,IAAI,cAAc,EAAE,CAAC;QACtD,uBAAuB,GAAG,cAAc,CAAC,GAAG,CAAM,CAAA;IACpD,CAAC;IAED,OAAO,uBAAuB,CAAA;AAChC,CAAC"}

package/lib/utils/is-async-storage.d.ts

@@ -1,10 +1,4 @@
 import type { AsyncStorage } from '../@types/storage';
-/**
- * Narrows a storage to {@link AsyncStorage}, deciding which hook the entry point builds.
- *
- * Inspection never writes: `set` and `remove` are only checked for shape, never invoked, so
- * `get` alone decides. A candidate pairing an async `get` with a sync `set` satisfies neither
- * storage interface, which is why proving the other two members is not worth a write.
- */
+/** Narrows a storage to {@link AsyncStorage}. Inspecting never writes: `set` and `remove` are never called. */
 export default function isAsyncStorage(storage: unknown): storage is AsyncStorage;
 //# sourceMappingURL=is-async-storage.d.ts.map

package/lib/utils/is-async-storage.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"is-async-storage.d.ts","sourceRoot":"","sources":["../../src/utils/is-async-storage.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,mBAAmB,CAAA;AAcrD;;;;;;GAMG;AACH,MAAM,CAAC,OAAO,UAAU,cAAc,CAAC,OAAO,EAAE,OAAO,GAAG,OAAO,IAAI,YAAY,CAoDhF"}
+{"version":3,"file":"is-async-storage.d.ts","sourceRoot":"","sources":["../../src/utils/is-async-storage.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,mBAAmB,CAAA;AAWrD,+GAA+G;AAC/G,MAAM,CAAC,OAAO,UAAU,cAAc,CAAC,OAAO,EAAE,OAAO,GAAG,OAAO,IAAI,YAAY,CAyChF"}

package/lib/utils/is-async-storage.js

@@ -2,17 +2,9 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.default = isAsyncStorage;
 const is_1 = require("@plq/is");
-// `AsyncStorage` is three methods, so every member has to be callable: a promise standing in for
-// one satisfies nothing the hook then does with it. `@plq/is` classifies async functions as their
-// own type, so `isFunction` rejects them and both checks are needed to admit a callable member.
+// `@plq/is` classifies async functions as their own type, so `isFunction` alone rejects them.
 const isCallableMember = (member) => (0, is_1.isFunction)(member) || (0, is_1.isAsyncFunction)(member);
-/**
- * Narrows a storage to {@link AsyncStorage}, deciding which hook the entry point builds.
- *
- * Inspection never writes: `set` and `remove` are only checked for shape, never invoked, so
- * `get` alone decides. A candidate pairing an async `get` with a sync `set` satisfies neither
- * storage interface, which is why proving the other two members is not worth a write.
- */
+/** Narrows a storage to {@link AsyncStorage}. Inspecting never writes: `set` and `remove` are never called. */
 function isAsyncStorage(storage) {
     const candidate = storage;
     if (!candidate) {
@@ -29,31 +21,20 @@ function isAsyncStorage(storage) {
     if (!(0, is_1.isFunction)(get)) {
         return false;
     }
-    // `AsyncStorage` admits a plain function returning a promise, and nothing short of a call tells
-    // that apart from a sync get. Any adapter written that way can only be recognised here, and a
-    // third-party one is free to be written that way, so this line stays necessary however the
-    // shipped adapters happen to declare themselves. A read is the one probe that leaves the
-    // inspected storage as it found it.
+    // A plain function returning a promise is a valid `AsyncStorage`, and only a call tells it from a sync get.
     let probe;
     try {
-        // Called as a method of the storage it belongs to: an adapter written as an
-        // object of methods reads its own state through `this`, and probing the
-        // extracted function would throw on a storage that is perfectly valid.
+        // Called as a method: an adapter reading its own state through `this` would throw on a torn-off function.
         probe = get.call(candidate, '');
     }
     catch {
-        // Not swallowed, deferred: consumers build the factory at module scope, where
-        // a throw takes the import down instead of reaching a component. A storage
-        // whose read fails is not provably async, and the sync path raises the same
-        // failure at the first read, where it can be handled.
+        // Deferred, not swallowed: factories are built at module scope, where a throw takes the import down.
         return false;
     }
     if (!(0, is_1.isPromise)(probe)) {
         return false;
     }
-    // Only the probe's shape answers the question, so its outcome is discarded by construction.
-    // The rejection still has to be claimed: unclaimed, it terminates the process on Node 15+,
-    // letting a storage that merely fails a read take the consuming application down with it.
+    // An unclaimed rejection terminates the process on Node 15+, so the discarded probe still has to be claimed.
     probe.then(undefined, () => undefined);
     return true;
 }

package/lib/utils/is-async-storage.js.map

@@ -1 +1 @@
-{"version":3,"file":"is-async-storage.js","sourceRoot":"","sources":["../../src/utils/is-async-storage.ts"],"names":[],"mappings":";;AAsBA,iCAoDC;AA1ED,gCAAgE;AAUhE,iGAAiG;AACjG,kGAAkG;AAClG,gGAAgG;AAChG,MAAM,gBAAgB,GAAG,CAAC,MAAe,EAAW,EAAE,CAAC,IAAA,eAAU,EAAC,MAAM,CAAC,IAAI,IAAA,oBAAe,EAAC,MAAM,CAAC,CAAA;AAEpG;;;;;;GAMG;AACH,SAAwB,cAAc,CAAC,OAAgB;IACrD,MAAM,SAAS,GAAG,OAA8C,CAAA;IAEhE,IAAI,CAAC,SAAS,EAAE,CAAC;QACf,OAAO,KAAK,CAAA;IACd,CAAC;IAED,MAAM,EAAE,GAAG,EAAE,GAAG,EAAE,MAAM,EAAE,GAAG,SAAS,CAAA;IAEtC,IAAI,CAAC,gBAAgB,CAAC,GAAG,CAAC,IAAI,CAAC,gBAAgB,CAAC,MAAM,CAAC,EAAE,CAAC;QACxD,OAAO,KAAK,CAAA;IACd,CAAC;IAED,8EAA8E;IAC9E,IAAI,IAAA,oBAAe,EAAC,GAAG,CAAC,EAAE,CAAC;QACzB,OAAO,IAAI,CAAA;IACb,CAAC;IAED,IAAI,CAAC,IAAA,eAAU,EAAC,GAAG,CAAC,EAAE,CAAC;QACrB,OAAO,KAAK,CAAA;IACd,CAAC;IAED,gGAAgG;IAChG,8FAA8F;IAC9F,2FAA2F;IAC3F,yFAAyF;IACzF,oCAAoC;IACpC,IAAI,KAAc,CAAA;IAElB,IAAI,CAAC;QACH,4EAA4E;QAC5E,wEAAwE;QACxE,uEAAuE;QACvE,KAAK,GAAG,GAAG,CAAC,IAAI,CAAC,SAAS,EAAE,EAAE,CAAC,CAAA;IACjC,CAAC;IAAC,MAAM,CAAC;QACP,8EAA8E;QAC9E,2EAA2E;QAC3E,4EAA4E;QAC5E,sDAAsD;QACtD,OAAO,KAAK,CAAA;IACd,CAAC;IAED,IAAI,CAAC,IAAA,cAAS,EAAC,KAAK,CAAC,EAAE,CAAC;QACtB,OAAO,KAAK,CAAA;IACd,CAAC;IAED,4FAA4F;IAC5F,2FAA2F;IAC3F,0FAA0F;IAC1F,KAAK,CAAC,IAAI,CAAC,SAAS,EAAE,GAAG,EAAE,CAAC,SAAS,CAAC,CAAA;IAEtC,OAAO,IAAI,CAAA;AACb,CAAC"}
+{"version":3,"file":"is-async-storage.js","sourceRoot":"","sources":["../../src/utils/is-async-storage.ts"],"names":[],"mappings":";;AAaA,iCAyCC;AAtDD,gCAAgE;AAShE,8FAA8F;AAC9F,MAAM,gBAAgB,GAAG,CAAC,MAAe,EAAW,EAAE,CAAC,IAAA,eAAU,EAAC,MAAM,CAAC,IAAI,IAAA,oBAAe,EAAC,MAAM,CAAC,CAAA;AAEpG,+GAA+G;AAC/G,SAAwB,cAAc,CAAC,OAAgB;IACrD,MAAM,SAAS,GAAG,OAA8C,CAAA;IAEhE,IAAI,CAAC,SAAS,EAAE,CAAC;QACf,OAAO,KAAK,CAAA;IACd,CAAC;IAED,MAAM,EAAE,GAAG,EAAE,GAAG,EAAE,MAAM,EAAE,GAAG,SAAS,CAAA;IAEtC,IAAI,CAAC,gBAAgB,CAAC,GAAG,CAAC,IAAI,CAAC,gBAAgB,CAAC,MAAM,CAAC,EAAE,CAAC;QACxD,OAAO,KAAK,CAAA;IACd,CAAC;IAED,8EAA8E;IAC9E,IAAI,IAAA,oBAAe,EAAC,GAAG,CAAC,EAAE,CAAC;QACzB,OAAO,IAAI,CAAA;IACb,CAAC;IAED,IAAI,CAAC,IAAA,eAAU,EAAC,GAAG,CAAC,EAAE,CAAC;QACrB,OAAO,KAAK,CAAA;IACd,CAAC;IAED,4GAA4G;IAC5G,IAAI,KAAc,CAAA;IAElB,IAAI,CAAC;QACH,0GAA0G;QAC1G,KAAK,GAAG,GAAG,CAAC,IAAI,CAAC,SAAS,EAAE,EAAE,CAAC,CAAA;IACjC,CAAC;IAAC,MAAM,CAAC;QACP,qGAAqG;QACrG,OAAO,KAAK,CAAA;IACd,CAAC;IAED,IAAI,CAAC,IAAA,cAAS,EAAC,KAAK,CAAC,EAAE,CAAC;QACtB,OAAO,KAAK,CAAA;IACd,CAAC;IAED,6GAA6G;IAC7G,KAAK,CAAC,IAAI,CAAC,SAAS,EAAE,GAAG,EAAE,CAAC,SAAS,CAAC,CAAA;IAEtC,OAAO,IAAI,CAAA;AACb,CAAC"}

package/lib/utils/storage-event-router.d.ts

@@ -6,11 +6,7 @@ export interface StorageEventRoute {
         [key: string]: StorageChange;
     }): void;
 }
-/**
- * Starts routing DOM storage events to `route`. Every adapter shares one
- * subscription on the global object, so creating adapters nobody listens to
- * costs nothing.
- */
+/** Starts routing DOM storage events to `route`. Every adapter shares one subscription on the global object. */
 export declare function addRoute(route: StorageEventRoute): void;
 /** Stops routing to `route`, releasing the shared subscription with the last one. */
 export declare function removeRoute(route: StorageEventRoute): void;

package/lib/utils/storage-event-router.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"storage-event-router.d.ts","sourceRoot":"","sources":["../../src/utils/storage-event-router.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,mBAAmB,CAAA;AAEtD,yEAAyE;AACzE,MAAM,WAAW,iBAAiB;IAChC,OAAO,EAAE,UAAU,CAAC,OAAO,CAAA;IAC3B,IAAI,CAAC,OAAO,EAAE;QAAE,CAAC,GAAG,EAAE,MAAM,GAAG,aAAa,CAAA;KAAE,GAAG,IAAI,CAAA;CACtD;AAoCD;;;;GAIG;AACH,wBAAgB,QAAQ,CAAC,KAAK,EAAE,iBAAiB,GAAG,IAAI,CAQvD;AAED,qFAAqF;AACrF,wBAAgB,WAAW,CAAC,KAAK,EAAE,iBAAiB,GAAG,IAAI,CAQ1D"}
+{"version":3,"file":"storage-event-router.d.ts","sourceRoot":"","sources":["../../src/utils/storage-event-router.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,mBAAmB,CAAA;AAEtD,yEAAyE;AACzE,MAAM,WAAW,iBAAiB;IAChC,OAAO,EAAE,UAAU,CAAC,OAAO,CAAA;IAC3B,IAAI,CAAC,OAAO,EAAE;QAAE,CAAC,GAAG,EAAE,MAAM,GAAG,aAAa,CAAA;KAAE,GAAG,IAAI,CAAA;CACtD;AA6BD,gHAAgH;AAChH,wBAAgB,QAAQ,CAAC,KAAK,EAAE,iBAAiB,GAAG,IAAI,CAQvD;AAED,qFAAqF;AACrF,wBAAgB,WAAW,CAAC,KAAK,EAAE,iBAAiB,GAAG,IAAI,CAQ1D"}

package/lib/utils/storage-event-router.js

@@ -14,28 +14,17 @@ function routeStorageEvent(event) {
         },
     };
     for (const route of routes) {
-        // A browser always names the area it changed, so real cross-tab events
-        // reach only the adapter of that area. An event a consumer synthesized —
-        // the sole way to test cross-tab sync under jsdom, and a common way to
-        // notify the writing tab in production — often cannot name one: the
-        // StorageEvent constructor rejects a mocked storage there. Those still go
-        // everywhere, exactly as they did before areas were told apart.
+        // An event a consumer synthesized often cannot name an area, so those still reach every route.
         if (event.storageArea == null || event.storageArea === route.storage) {
             route.fire(changes);
         }
     }
 }
-// A runtime can provide storage without the DOM event API, and can provide half
-// of that API: subscribing where nothing can unsubscribe strands a listener and
-// makes releasing it throw, so both halves are required before taking one.
+// A runtime can provide half the DOM event API, and subscribing where nothing can unsubscribe strands a listener.
 function canSubscribe() {
     return typeof globalThis.addEventListener === 'function' && typeof globalThis.removeEventListener === 'function';
 }
-/**
- * Starts routing DOM storage events to `route`. Every adapter shares one
- * subscription on the global object, so creating adapters nobody listens to
- * costs nothing.
- */
+/** Starts routing DOM storage events to `route`. Every adapter shares one subscription on the global object. */
 function addRoute(route) {
     routes.add(route);
     if (!subscription && canSubscribe()) {

package/lib/utils/storage-event-router.js.map

@@ -1 +1 @@
-{"version":3,"file":"storage-event-router.js","sourceRoot":"","sources":["../../src/utils/storage-event-router.ts"],"names":[],"mappings":";;AA+CA,4BAQC;AAGD,kCAQC;AA1DD,MAAM,MAAM,GAAG,IAAI,GAAG,EAAqB,CAAA;AAE3C,IAAI,YAAY,GAA2C,IAAI,CAAA;AAE/D,SAAS,iBAAiB,CAAC,KAAmB;IAC5C,IAAI,CAAC,KAAK,CAAC,GAAG;QAAE,OAAM;IAEtB,MAAM,OAAO,GAAG;QACd,CAAC,KAAK,CAAC,GAAG,CAAC,EAAE;YACX,QAAQ,EAAE,KAAK,CAAC,QAAQ;YACxB,QAAQ,EAAE,KAAK,CAAC,QAAQ;SACzB;KACF,CAAA;IAED,KAAK,MAAM,KAAK,IAAI,MAAM,EAAE,CAAC;QAC3B,uEAAuE;QACvE,yEAAyE;QACzE,uEAAuE;QACvE,oEAAoE;QACpE,0EAA0E;QAC1E,gEAAgE;QAChE,IAAI,KAAK,CAAC,WAAW,IAAI,IAAI,IAAI,KAAK,CAAC,WAAW,KAAK,KAAK,CAAC,OAAO,EAAE,CAAC;YACrE,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC,CAAA;QACrB,CAAC;IACH,CAAC;AACH,CAAC;AAED,gFAAgF;AAChF,gFAAgF;AAChF,2EAA2E;AAC3E,SAAS,YAAY;IACnB,OAAO,OAAO,UAAU,CAAC,gBAAgB,KAAK,UAAU,IAAI,OAAO,UAAU,CAAC,mBAAmB,KAAK,UAAU,CAAA;AAClH,CAAC;AAED;;;;GAIG;AACH,SAAgB,QAAQ,CAAC,KAAwB;IAC/C,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,CAAA;IAEjB,IAAI,CAAC,YAAY,IAAI,YAAY,EAAE,EAAE,CAAC;QACpC,YAAY,GAAG,iBAAiB,CAAA;QAEhC,UAAU,CAAC,gBAAgB,CAAC,SAAS,EAAE,YAAY,CAAC,CAAA;IACtD,CAAC;AACH,CAAC;AAED,qFAAqF;AACrF,SAAgB,WAAW,CAAC,KAAwB;IAClD,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAA;IAEpB,IAAI,MAAM,CAAC,IAAI,KAAK,CAAC,IAAI,YAAY,EAAE,CAAC;QACtC,UAAU,CAAC,mBAAmB,CAAC,SAAS,EAAE,YAAY,CAAC,CAAA;QAEvD,YAAY,GAAG,IAAI,CAAA;IACrB,CAAC;AACH,CAAC"}
+{"version":3,"file":"storage-event-router.js","sourceRoot":"","sources":["../../src/utils/storage-event-router.ts"],"names":[],"mappings":";;AAoCA,4BAQC;AAGD,kCAQC;AA/CD,MAAM,MAAM,GAAG,IAAI,GAAG,EAAqB,CAAA;AAE3C,IAAI,YAAY,GAA2C,IAAI,CAAA;AAE/D,SAAS,iBAAiB,CAAC,KAAmB;IAC5C,IAAI,CAAC,KAAK,CAAC,GAAG;QAAE,OAAM;IAEtB,MAAM,OAAO,GAAG;QACd,CAAC,KAAK,CAAC,GAAG,CAAC,EAAE;YACX,QAAQ,EAAE,KAAK,CAAC,QAAQ;YACxB,QAAQ,EAAE,KAAK,CAAC,QAAQ;SACzB;KACF,CAAA;IAED,KAAK,MAAM,KAAK,IAAI,MAAM,EAAE,CAAC;QAC3B,+FAA+F;QAC/F,IAAI,KAAK,CAAC,WAAW,IAAI,IAAI,IAAI,KAAK,CAAC,WAAW,KAAK,KAAK,CAAC,OAAO,EAAE,CAAC;YACrE,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC,CAAA;QACrB,CAAC;IACH,CAAC;AACH,CAAC;AAED,kHAAkH;AAClH,SAAS,YAAY;IACnB,OAAO,OAAO,UAAU,CAAC,gBAAgB,KAAK,UAAU,IAAI,OAAO,UAAU,CAAC,mBAAmB,KAAK,UAAU,CAAA;AAClH,CAAC;AAED,gHAAgH;AAChH,SAAgB,QAAQ,CAAC,KAAwB;IAC/C,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,CAAA;IAEjB,IAAI,CAAC,YAAY,IAAI,YAAY,EAAE,EAAE,CAAC;QACpC,YAAY,GAAG,iBAAiB,CAAA;QAEhC,UAAU,CAAC,gBAAgB,CAAC,SAAS,EAAE,YAAY,CAAC,CAAA;IACtD,CAAC;AACH,CAAC;AAED,qFAAqF;AACrF,SAAgB,WAAW,CAAC,KAAwB;IAClD,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAA;IAEpB,IAAI,MAAM,CAAC,IAAI,KAAK,CAAC,IAAI,YAAY,EAAE,CAAC;QACtC,UAAU,CAAC,mBAAmB,CAAC,SAAS,EAAE,YAAY,CAAC,CAAA;QAEvD,YAAY,GAAG,IAAI,CAAA;IACrB,CAAC;AACH,CAAC"}

package/lib/utils/use-storage-handler.d.ts

@@ -1,15 +1,6 @@
 import type React from 'react';
 import type { AsyncStorage, Storage } from '../@types/storage';
-/**
- * Remembers an entry the hook is about to write, so the change the backend
- * reports for it can be told apart from someone else's write.
- *
- * Recorded before the write, because a backend may report it before the call
- * settles. One slot was not enough: a backend free to report after its write
- * settles - chrome and browser storage both are - lets the next write overwrite
- * the record of a write still waiting to be reported, and that unsuppressed echo
- * then puts the earlier value back over the later one.
- */
+/** Records an entry before it is written, because a backend may report the change before the write settles. */
 export declare function recordOwnWrite(pendingOwnWrites: React.RefObject<string[]>, item: string): void;
 export default function useStorageHandler<T>(key: string, storageKey: string, applyValue: (value: T) => void, storage: AsyncStorage | Storage, initialValue: T | (() => T), pendingOwnWrites: React.RefObject<string[]>): void;
 //# sourceMappingURL=use-storage-handler.d.ts.map

package/lib/utils/use-storage-handler.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"use-storage-handler.d.ts","sourceRoot":"","sources":["../../src/utils/use-storage-handler.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,KAAK,MAAM,OAAO,CAAA;AAE9B,OAAO,KAAK,EAAE,YAAY,EAAE,OAAO,EAAiB,MAAM,mBAAmB,CAAA;AAkE7E;;;;;;;;;GASG;AACH,wBAAgB,cAAc,CAAC,gBAAgB,EAAE,KAAK,CAAC,SAAS,CAAC,MAAM,EAAE,CAAC,EAAE,IAAI,EAAE,MAAM,GAAG,IAAI,CAM9F;AA+DD,MAAM,CAAC,OAAO,UAAU,iBAAiB,CAAC,CAAC,EACzC,GAAG,EAAE,MAAM,EACX,UAAU,EAAE,MAAM,EAClB,UAAU,EAAE,CAAC,KAAK,EAAE,CAAC,KAAK,IAAI,EAC9B,OAAO,EAAE,YAAY,GAAG,OAAO,EAC/B,YAAY,EAAE,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC,EAC3B,gBAAgB,EAAE,KAAK,CAAC,SAAS,CAAC,MAAM,EAAE,CAAC,GAC1C,IAAI,CAsBN"}
+{"version":3,"file":"use-storage-handler.d.ts","sourceRoot":"","sources":["../../src/utils/use-storage-handler.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,KAAK,MAAM,OAAO,CAAA;AAE9B,OAAO,KAAK,EAAE,YAAY,EAAE,OAAO,EAAiB,MAAM,mBAAmB,CAAA;AA8C7E,+GAA+G;AAC/G,wBAAgB,cAAc,CAAC,gBAAgB,EAAE,KAAK,CAAC,SAAS,CAAC,MAAM,EAAE,CAAC,EAAE,IAAI,EAAE,MAAM,GAAG,IAAI,CAM9F;AAwCD,MAAM,CAAC,OAAO,UAAU,iBAAiB,CAAC,CAAC,EACzC,GAAG,EAAE,MAAM,EACX,UAAU,EAAE,MAAM,EAClB,UAAU,EAAE,CAAC,KAAK,EAAE,CAAC,KAAK,IAAI,EAC9B,OAAO,EAAE,YAAY,GAAG,OAAO,EAC/B,YAAY,EAAE,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC,EAC3B,gBAAgB,EAAE,KAAK,CAAC,SAAS,CAAC,MAAM,EAAE,CAAC,GAC1C,IAAI,CAiBN"}

package/lib/utils/use-storage-handler.js

@@ -13,90 +13,48 @@ function readStoredValue(key, entry) {
         console.error("use-persisted-state: Can't parse value from storage", err);
         return { status: 'unavailable' };
     }
-    // A foreign entry can be any JSON value, and `in` throws on a primitive. This
-    // runs inside the adapter's notify loop, where a throw also cuts off every
-    // listener queued behind this one.
+    // `in` throws on a primitive, and a throw here cuts off every listener queued behind this one.
     if (!(0, is_1.isObject)(parsed) || !(key in parsed))
         return { status: 'unavailable' };
     return { status: 'stored', value: parsed[key] };
 }
-// Restores the initial value when the whole entry is removed from the storage.
 function applyRemoval(change, itemKey, applyValue, latestInitialValue) {
     if (change.oldValue === null || change.oldValue === undefined)
         return;
     const oldValue = readStoredValue(itemKey, change.oldValue);
     const declaredInitialValue = latestInitialValue.current;
-    // Compared against the resolved value: a factory is never equal to what it
-    // produces, so comparing the declaration re-applies the initial value on every
-    // removal.
+    // Resolved before comparing: a factory is never equal to what it produces.
     const initialValue = (0, is_1.isFunction)(declaredInitialValue) ? declaredInitialValue() : declaredInitialValue;
-    // An entry with no readable value for the key cannot be shown to have held the
-    // initial value, so the fallback is applied rather than skipped. The entry is
-    // gone either way, and stale state left in place is worse than a redundant
-    // restore.
+    // Restore redundantly rather than leave stale state when the entry's old value cannot be read.
     if (oldValue.status === 'stored' && oldValue.value === initialValue)
         return;
     applyValue(initialValue);
 }
-// A backend that reports nothing back leaves every record unmatched, so the list
-// needs a ceiling or it grows by one string per write for as long as the hook is
-// mounted. A bound, not a tuning: a backend that does report drains the list on
-// every echo, and one that reports late is what the list exists for.
+// A list, not one slot: a write may still be awaiting its echo when the next one starts.
+// Capped, because a backend that never reports back leaves every record unmatched.
 const MAX_PENDING_OWN_WRITES = 8;
-/**
- * Remembers an entry the hook is about to write, so the change the backend
- * reports for it can be told apart from someone else's write.
- *
- * Recorded before the write, because a backend may report it before the call
- * settles. One slot was not enough: a backend free to report after its write
- * settles - chrome and browser storage both are - lets the next write overwrite
- * the record of a write still waiting to be reported, and that unsuppressed echo
- * then puts the earlier value back over the later one.
- */
+/** Records an entry before it is written, because a backend may report the change before the write settles. */
 function recordOwnWrite(pendingOwnWrites, item) {
     const pending = pendingOwnWrites.current;
     if (pending.length >= MAX_PENDING_OWN_WRITES)
         pending.shift();
     pending.push(item);
 }
-/**
- * Reports whether this change is a write the hook itself made, forgetting the
- * record when it is. A backend reports a write to every listener, the one that
- * made it included, and applying that echo is the storage-to-state re-sync this
- * hook was rebuilt without, arriving by another road: it decodes the entry again
- * and hands the caller an equal value with a new identity, plus a render for a
- * value it already holds.
- *
- * The price, accepted knowingly: for a value JSON cannot carry, the writer keeps
- * what it set - NaN - while every other component on the key decodes the null
- * that reached storage, so the two disagree until one of them remounts. Applying
- * the echo would settle that disagreement and bring back both the second
- * storage-to-state path and the render per write, so it is not the repair it
- * looks like.
- *
- * Forgetting on the first match keeps a later identical entry a change. Another
- * hook writing the same bytes is suppressed along with it, and nothing is lost:
- * it would have replaced a value with an equal one.
- */
+// Applying a hook's own echo would re-decode the entry and re-render for a value it already holds.
 function consumeOwnWriteEcho(entry, pendingOwnWrites) {
     const matched = pendingOwnWrites.current.indexOf(entry);
     if (matched === -1)
         return false;
-    // Everything recorded before the reported entry goes with it: a backend applies
-    // writes in the order it took them and reports them in that order too, so a
-    // record still unmatched by now has no echo left to wait for.
+    // A backend reports writes in the order it took them, so a record still unmatched has no echo left.
     pendingOwnWrites.current.splice(0, matched + 1);
     return true;
 }
-// Builds the change handler. Not a hook, despite living next to one.
 function createStorageHandler(itemKey, storageKey, applyValue, latestInitialValue, pendingOwnWrites) {
     return (changes) => {
         for (const [key, change] of Object.entries(changes)) {
             if (key !== storageKey)
                 continue;
-            // Ahead of the echo check, which needs an entry to match and a removal has
-            // none. A removal is never one of this hook's own writes anyway: only the
-            // entries it stores are recorded.
+            // Ahead of the echo check, which needs an entry to match and a removal has none.
             if (change.newValue === null || change.newValue === undefined) {
                 applyRemoval(change, itemKey, applyValue, latestInitialValue);
                 continue;
@@ -110,12 +68,7 @@ function createStorageHandler(itemKey, storageKey, applyValue, latestInitialValu
     };
 }
 function useStorageHandler(key, storageKey, applyValue, storage, initialValue, pendingOwnWrites) {
-    // A removal restores the initial value the hook holds now, the same one the
-    // key-change path reads, so a caller whose default travels with its data gets
-    // that record's default back rather than the mounted one. Tracked through a ref
-    // rather than a dependency because an inline object or factory has a new
-    // identity every render, which would tear the subscription down and rebuild it;
-    // written during render, as the key the hook is rendering for is.
+    // A ref, not a dependency: an inline initial value changes identity every render and would churn it.
     const latestInitialValue = (0, react_1.useRef)(initialValue);
     latestInitialValue.current = initialValue;
     (0, react_1.useEffect)(() => {

package/lib/utils/use-storage-handler.js.map

@@ -1 +1 @@
-{"version":3,"file":"use-storage-handler.js","sourceRoot":"","sources":["../../src/utils/use-storage-handler.ts"],"names":[],"mappings":";;AA8EA,wCAMC;AA+DD,oCA6BC;AA/KD,iCAAyC;AAEzC,gCAA8C;AAe9C,SAAS,eAAe,CAAI,GAAW,EAAE,KAAa;IACpD,IAAI,MAAe,CAAA;IAEnB,IAAI,CAAC;QACH,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,CAAA;IAC5B,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,OAAO,CAAC,KAAK,CAAC,qDAAqD,EAAE,GAAG,CAAC,CAAA;QAEzE,OAAO,EAAE,MAAM,EAAE,aAAa,EAAE,CAAA;IAClC,CAAC;IAED,8EAA8E;IAC9E,2EAA2E;IAC3E,mCAAmC;IACnC,IAAI,CAAC,IAAA,aAAQ,EAAC,MAAM,CAAC,IAAI,CAAC,CAAC,GAAG,IAAI,MAAM,CAAC;QAAE,OAAO,EAAE,MAAM,EAAE,aAAa,EAAE,CAAA;IAE3E,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,KAAK,EAAE,MAAM,CAAC,GAAG,CAAM,EAAE,CAAA;AACtD,CAAC;AAED,+EAA+E;AAC/E,SAAS,YAAY,CACnB,MAAqB,EACrB,OAAe,EACf,UAA8B,EAC9B,kBAAkD;IAElD,IAAI,MAAM,CAAC,QAAQ,KAAK,IAAI,IAAI,MAAM,CAAC,QAAQ,KAAK,SAAS;QAAE,OAAM;IAErE,MAAM,QAAQ,GAAG,eAAe,CAAI,OAAO,EAAE,MAAM,CAAC,QAAQ,CAAC,CAAA;IAC7D,MAAM,oBAAoB,GAAG,kBAAkB,CAAC,OAAO,CAAA;IACvD,2EAA2E;IAC3E,+EAA+E;IAC/E,WAAW;IACX,MAAM,YAAY,GAAG,IAAA,eAAU,EAAC,oBAAoB,CAAC,CAAC,CAAC,CAAC,oBAAoB,EAAE,CAAC,CAAC,CAAC,oBAAoB,CAAA;IAErG,+EAA+E;IAC/E,8EAA8E;IAC9E,2EAA2E;IAC3E,WAAW;IACX,IAAI,QAAQ,CAAC,MAAM,KAAK,QAAQ,IAAI,QAAQ,CAAC,KAAK,KAAK,YAAY;QAAE,OAAM;IAE3E,UAAU,CAAC,YAAY,CAAC,CAAA;AAC1B,CAAC;AAED,iFAAiF;AACjF,iFAAiF;AACjF,gFAAgF;AAChF,qEAAqE;AACrE,MAAM,sBAAsB,GAAG,CAAC,CAAA;AAEhC;;;;;;;;;GASG;AACH,SAAgB,cAAc,CAAC,gBAA2C,EAAE,IAAY;IACtF,MAAM,OAAO,GAAG,gBAAgB,CAAC,OAAO,CAAA;IAExC,IAAI,OAAO,CAAC,MAAM,IAAI,sBAAsB;QAAE,OAAO,CAAC,KAAK,EAAE,CAAA;IAE7D,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;AACpB,CAAC;AAED;;;;;;;;;;;;;;;;;;GAkBG;AACH,SAAS,mBAAmB,CAAC,KAAa,EAAE,gBAA2C;IACrF,MAAM,OAAO,GAAG,gBAAgB,CAAC,OAAO,CAAC,OAAO,CAAC,KAAK,CAAC,CAAA;IAEvD,IAAI,OAAO,KAAK,CAAC,CAAC;QAAE,OAAO,KAAK,CAAA;IAEhC,gFAAgF;IAChF,4EAA4E;IAC5E,8DAA8D;IAC9D,gBAAgB,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,EAAE,OAAO,GAAG,CAAC,CAAC,CAAA;IAE/C,OAAO,IAAI,CAAA;AACb,CAAC;AAED,qEAAqE;AACrE,SAAS,oBAAoB,CAC3B,OAAe,EACf,UAAkB,EAClB,UAA8B,EAC9B,kBAAkD,EAClD,gBAA2C;IAE3C,OAAO,CAAC,OAAyC,EAAQ,EAAE;QACzD,KAAK,MAAM,CAAC,GAAG,EAAE,MAAM,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,CAAC;YACpD,IAAI,GAAG,KAAK,UAAU;gBAAE,SAAQ;YAEhC,2EAA2E;YAC3E,0EAA0E;YAC1E,kCAAkC;YAClC,IAAI,MAAM,CAAC,QAAQ,KAAK,IAAI,IAAI,MAAM,CAAC,QAAQ,KAAK,SAAS,EAAE,CAAC;gBAC9D,YAAY,CAAI,MAAM,EAAE,OAAO,EAAE,UAAU,EAAE,kBAAkB,CAAC,CAAA;gBAChE,SAAQ;YACV,CAAC;YAED,IAAI,mBAAmB,CAAC,MAAM,CAAC,QAAQ,EAAE,gBAAgB,CAAC;gBAAE,SAAQ;YAEpE,MAAM,QAAQ,GAAG,eAAe,CAAI,OAAO,EAAE,MAAM,CAAC,QAAQ,CAAC,CAAA;YAE7D,IAAI,QAAQ,CAAC,MAAM,KAAK,QAAQ;gBAAE,UAAU,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAA;QAC9D,CAAC;IACH,CAAC,CAAA;AACH,CAAC;AAED,SAAwB,iBAAiB,CACvC,GAAW,EACX,UAAkB,EAClB,UAA8B,EAC9B,OAA+B,EAC/B,YAA2B,EAC3B,gBAA2C;IAE3C,4EAA4E;IAC5E,8EAA8E;IAC9E,gFAAgF;IAChF,yEAAyE;IACzE,gFAAgF;IAChF,kEAAkE;IAClE,MAAM,kBAAkB,GAAG,IAAA,cAAM,EAAC,YAAY,CAAC,CAAA;IAE/C,kBAAkB,CAAC,OAAO,GAAG,YAAY,CAAA;IAEzC,IAAA,iBAAS,EAAC,GAAG,EAAE;QACb,MAAM,aAAa,GAAG,oBAAoB,CAAI,GAAG,EAAE,UAAU,EAAE,UAAU,EAAE,kBAAkB,EAAE,gBAAgB,CAAC,CAAA;QAEhH,OAAO,CAAC,SAAS,CAAC,WAAW,CAAC,aAAa,CAAC,CAAA;QAE5C,OAAO,GAAG,EAAE;YACV,IAAI,OAAO,CAAC,SAAS,CAAC,WAAW,CAAC,aAAa,CAAC,EAAE,CAAC;gBACjD,OAAO,CAAC,SAAS,CAAC,cAAc,CAAC,aAAa,CAAC,CAAA;YACjD,CAAC;QACH,CAAC,CAAA;IACH,CAAC,EAAE,CAAC,GAAG,EAAE,OAAO,CAAC,SAAS,EAAE,UAAU,EAAE,UAAU,EAAE,gBAAgB,CAAC,CAAC,CAAA;AACxE,CAAC"}
+{"version":3,"file":"use-storage-handler.js","sourceRoot":"","sources":["../../src/utils/use-storage-handler.ts"],"names":[],"mappings":";;AAiDA,wCAMC;AAwCD,oCAwBC;AAtHD,iCAAyC;AAEzC,gCAA8C;AAK9C,SAAS,eAAe,CAAI,GAAW,EAAE,KAAa;IACpD,IAAI,MAAe,CAAA;IAEnB,IAAI,CAAC;QACH,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,CAAA;IAC5B,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,OAAO,CAAC,KAAK,CAAC,qDAAqD,EAAE,GAAG,CAAC,CAAA;QAEzE,OAAO,EAAE,MAAM,EAAE,aAAa,EAAE,CAAA;IAClC,CAAC;IAED,+FAA+F;IAC/F,IAAI,CAAC,IAAA,aAAQ,EAAC,MAAM,CAAC,IAAI,CAAC,CAAC,GAAG,IAAI,MAAM,CAAC;QAAE,OAAO,EAAE,MAAM,EAAE,aAAa,EAAE,CAAA;IAE3E,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,KAAK,EAAE,MAAM,CAAC,GAAG,CAAM,EAAE,CAAA;AACtD,CAAC;AAED,SAAS,YAAY,CACnB,MAAqB,EACrB,OAAe,EACf,UAA8B,EAC9B,kBAAkD;IAElD,IAAI,MAAM,CAAC,QAAQ,KAAK,IAAI,IAAI,MAAM,CAAC,QAAQ,KAAK,SAAS;QAAE,OAAM;IAErE,MAAM,QAAQ,GAAG,eAAe,CAAI,OAAO,EAAE,MAAM,CAAC,QAAQ,CAAC,CAAA;IAC7D,MAAM,oBAAoB,GAAG,kBAAkB,CAAC,OAAO,CAAA;IACvD,2EAA2E;IAC3E,MAAM,YAAY,GAAG,IAAA,eAAU,EAAC,oBAAoB,CAAC,CAAC,CAAC,CAAC,oBAAoB,EAAE,CAAC,CAAC,CAAC,oBAAoB,CAAA;IAErG,+FAA+F;IAC/F,IAAI,QAAQ,CAAC,MAAM,KAAK,QAAQ,IAAI,QAAQ,CAAC,KAAK,KAAK,YAAY;QAAE,OAAM;IAE3E,UAAU,CAAC,YAAY,CAAC,CAAA;AAC1B,CAAC;AAED,yFAAyF;AACzF,mFAAmF;AACnF,MAAM,sBAAsB,GAAG,CAAC,CAAA;AAEhC,+GAA+G;AAC/G,SAAgB,cAAc,CAAC,gBAA2C,EAAE,IAAY;IACtF,MAAM,OAAO,GAAG,gBAAgB,CAAC,OAAO,CAAA;IAExC,IAAI,OAAO,CAAC,MAAM,IAAI,sBAAsB;QAAE,OAAO,CAAC,KAAK,EAAE,CAAA;IAE7D,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;AACpB,CAAC;AAED,mGAAmG;AACnG,SAAS,mBAAmB,CAAC,KAAa,EAAE,gBAA2C;IACrF,MAAM,OAAO,GAAG,gBAAgB,CAAC,OAAO,CAAC,OAAO,CAAC,KAAK,CAAC,CAAA;IAEvD,IAAI,OAAO,KAAK,CAAC,CAAC;QAAE,OAAO,KAAK,CAAA;IAEhC,oGAAoG;IACpG,gBAAgB,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,EAAE,OAAO,GAAG,CAAC,CAAC,CAAA;IAE/C,OAAO,IAAI,CAAA;AACb,CAAC;AAED,SAAS,oBAAoB,CAC3B,OAAe,EACf,UAAkB,EAClB,UAA8B,EAC9B,kBAAkD,EAClD,gBAA2C;IAE3C,OAAO,CAAC,OAAyC,EAAQ,EAAE;QACzD,KAAK,MAAM,CAAC,GAAG,EAAE,MAAM,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,CAAC;YACpD,IAAI,GAAG,KAAK,UAAU;gBAAE,SAAQ;YAEhC,iFAAiF;YACjF,IAAI,MAAM,CAAC,QAAQ,KAAK,IAAI,IAAI,MAAM,CAAC,QAAQ,KAAK,SAAS,EAAE,CAAC;gBAC9D,YAAY,CAAI,MAAM,EAAE,OAAO,EAAE,UAAU,EAAE,kBAAkB,CAAC,CAAA;gBAChE,SAAQ;YACV,CAAC;YAED,IAAI,mBAAmB,CAAC,MAAM,CAAC,QAAQ,EAAE,gBAAgB,CAAC;gBAAE,SAAQ;YAEpE,MAAM,QAAQ,GAAG,eAAe,CAAI,OAAO,EAAE,MAAM,CAAC,QAAQ,CAAC,CAAA;YAE7D,IAAI,QAAQ,CAAC,MAAM,KAAK,QAAQ;gBAAE,UAAU,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAA;QAC9D,CAAC;IACH,CAAC,CAAA;AACH,CAAC;AAED,SAAwB,iBAAiB,CACvC,GAAW,EACX,UAAkB,EAClB,UAA8B,EAC9B,OAA+B,EAC/B,YAA2B,EAC3B,gBAA2C;IAE3C,qGAAqG;IACrG,MAAM,kBAAkB,GAAG,IAAA,cAAM,EAAC,YAAY,CAAC,CAAA;IAE/C,kBAAkB,CAAC,OAAO,GAAG,YAAY,CAAA;IAEzC,IAAA,iBAAS,EAAC,GAAG,EAAE;QACb,MAAM,aAAa,GAAG,oBAAoB,CAAI,GAAG,EAAE,UAAU,EAAE,UAAU,EAAE,kBAAkB,EAAE,gBAAgB,CAAC,CAAA;QAEhH,OAAO,CAAC,SAAS,CAAC,WAAW,CAAC,aAAa,CAAC,CAAA;QAE5C,OAAO,GAAG,EAAE;YACV,IAAI,OAAO,CAAC,SAAS,CAAC,WAAW,CAAC,aAAa,CAAC,EAAE,CAAC;gBACjD,OAAO,CAAC,SAAS,CAAC,cAAc,CAAC,aAAa,CAAC,CAAA;YACjD,CAAC;QACH,CAAC,CAAA;IACH,CAAC,EAAE,CAAC,GAAG,EAAE,OAAO,CAAC,SAAS,EAAE,UAAU,EAAE,UAAU,EAAE,gBAAgB,CAAC,CAAC,CAAA;AACxE,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@plq/use-persisted-state",
-  "version": "1.4.1",
+  "version": "1.4.2",
   "description": "useState hook with persistence in storage",
   "type": "commonjs",
   "main": "lib/index.js",

package/src/create-async-persisted-state.ts

@@ -15,20 +15,11 @@ export default function createAsyncPersistedState<S extends AsyncStorage>(
 ): [PersistedState, () => Promise<void>] {
   const safeStorageKey = `persisted_state_hook:${storageKey}`
 
-  // Every hook this factory makes lives in one entry, and storing a value means
-  // reading that entry, merging one key into it and writing all of it back, with
-  // a suspension point on either side of the merge. Left to overlap, a second
-  // writer merges into a snapshot taken before the first one landed and stores
-  // it: the first writer's key is gone from storage while its value is still on
-  // screen, and nothing reports the disagreement. It is the entry, not the hook,
-  // that has to be taken one at a time, so the chain belongs to the factory.
+  // Storing is read-merge-write on one shared entry, so overlapping writers would drop each other's keys.
   let entryWrites: Promise<unknown> = Promise.resolve()
 
   const clear = (): Promise<void> => {
-    // Removing the entry changes it as much as storing does, so it takes its turn
-    // in the same chain. Outside it, a write already queued lands after the
-    // removal and brings back what was cleared - and "clear this data" is the one
-    // request that cannot be allowed to half happen.
+    // In the same chain as writes, or a queued write lands after the removal and restores what was cleared.
     const removal = entryWrites.then(() => storage.remove(safeStorageKey))
 
     entryWrites = removal.catch(() => undefined)
@@ -44,23 +35,16 @@ export default function createAsyncPersistedState<S extends AsyncStorage>(
       try {
         newItem = getNewItem<T>(key, persistedItem[safeStorageKey], newValue)
       } catch {
-        // Refused, and reported where it was refused. A write replaces the whole
-        // entry, so an entry that cannot be read is one no write can be built on
-        // without dropping every other hook's key; skipping leaves the bytes for
-        // a repair to reach. The caller keeps what it set, unpersisted.
+        // A write replaces the whole entry, so an unreadable one is skipped rather than rebuilt without other keys.
         return
       }
 
-      // Recorded before the write, because the backend may report it before the
-      // promise settles.
       recordOwnWrite(pendingOwnWrites, newItem)
 
       await storage.set({ [safeStorageKey]: newItem })
     })
 
-    // The chain has to outlive a failed write, or one backend rejection stops
-    // every later write on this entry. The rejection itself is not swallowed: it
-    // stays on the promise handed back to the caller that asked for the write.
+    // The chain must outlive a failed write; the rejection still reaches the caller through `write`.
     entryWrites = write.catch(() => undefined)
 
     return write
@@ -69,14 +53,10 @@ export default function createAsyncPersistedState<S extends AsyncStorage>(
   const usePersistedState = <T>(key: string, initialValue: T | (() => T)): UsePersistedState<T> => {
     const [state, setState] = useState<T>(initialValue)
 
-    // The setter must resolve updater functions against the last value applied,
-    // not against the one captured by the render that created it. Reading the
-    // render closure collapses updates batched into a single event.
+    // Updater functions must resolve against the last applied value, not the render closure's.
     const latestValue = useRef(state)
 
-    // Whether anything has already put a value in place. A load that settles after
-    // the caller set one must not revert it, and no signal local to the load can
-    // answer this: the write it loses to happens outside the load entirely.
+    // A load that settles after the caller set a value must not revert it.
     const hasAppliedValue = useRef(false)
 
     const applyValue = useCallback((value: T): void => {
@@ -86,15 +66,12 @@ export default function createAsyncPersistedState<S extends AsyncStorage>(
       setState(value)
     }, [])
 
-    // The entries this hook has written and not yet seen reported back.
     const pendingOwnWrites = useRef<string[]>([])
 
     const setPersistedState = useCallback(
       async (newState: React.SetStateAction<T>): Promise<void> => {
         const newValue = getNewValue<T>(newState, latestValue.current)
 
-        // Applied before the write is even queued: the caller sees its value at
-        // once, and only the trip to storage waits its turn.
         applyValue(newValue)
 
         await commitEntry<T>(key, newValue, pendingOwnWrites)
@@ -102,26 +79,14 @@ export default function createAsyncPersistedState<S extends AsyncStorage>(
       [key, applyValue],
     )
 
-    // As in `useState`, the value the load falls back to is the one given on the
-    // first render. Later identities of an inline object must not reload, or the
-    // effect re-runs on every render and never settles.
+    // The first render's initial value: reloading on a later identity would re-run the effect forever.
     const mountInitialValue = useRef(initialValue)
 
-    // Subscribed before the load below, and the order is load-bearing: React runs
-    // effects in declaration order, so reading first would leave an interval
-    // between the read and the subscription in which a write belongs to neither
-    // and is lost. Reading last covers everything written before the subscription
-    // began, and the listener covers everything after.
+    // Subscribed before the load: effects run in declaration order, and reading first would lose writes in between.
     useStorageHandler<T>(key, safeStorageKey, applyValue, storage, initialValue, pendingOwnWrites)
 
     useEffect(() => {
-      // Two separate questions, and one cell cannot hold both: whether this load
-      // is still the current one, which only the closure it belongs to can answer,
-      // and whether a value has been applied meanwhile, which outlives it. A load
-      // abandoned by a key change would otherwise read the flag its successor had
-      // just set, apply the previous key's value and shut the successor out. The
-      // second question is also what makes subscribing first safe: a value the
-      // listener delivers while the load is in flight is not overwritten by it.
+      // Separate from `hasAppliedValue`: only this closure knows whether its own load is still current.
       let isCancelled = false
 
       hasAppliedValue.current = false
@@ -134,12 +99,7 @@ export default function createAsyncPersistedState<S extends AsyncStorage>(
 
           applyValue(getPersistedValue<T>(key, mountInitialValue.current, persist[safeStorageKey]))
         } catch (err) {
-          // An extension backend rejects on a quota error or an invalidated
-          // extension context. Nothing awaits this load, so an unclaimed
-          // rejection terminates the process on Node 15+ and surfaces as an
-          // uncaught error in the browser. The initial value is already in
-          // state, so the mount stands on it and the failure is reported rather
-          // than swallowed, as the synchronous path reports its own.
+          // Nothing awaits this load, so an unclaimed rejection would surface as an uncaught error.
           console.error("use-persisted-state: Can't read value from storage", err)
         }
       }

package/src/create-persisted-state.ts

@@ -19,14 +19,10 @@ export default function createPersistedState(storageKey: string, storage: Storag
     getPersistedValue<T>(key, initialValue, storage.get(safeStorageKey)[safeStorageKey])
 
   const usePersistedState = <T>(key: string, initialValue: T): UsePersistedState<T> => {
-    // Reading through the initializer keeps the storage out of the render path:
-    // the hook runs on every render of every consuming component, so a re-read is
-    // only warranted when the key it is asked for actually changes.
+    // Read through the initializer: the hook runs on every render, so storage stays out of the render path.
     const [state, setState] = useState<T>(() => readPersisted(key, initialValue))
 
-    // The setter must resolve updater functions against the last value applied,
-    // not against the one captured by the render that created it. Reading the
-    // render closure collapses updates batched into a single event.
+    // Updater functions must resolve against the last applied value, not the render closure's.
     const latestValue = useRef(state)
 
     const applyValue = useCallback((value: T): void => {
@@ -35,11 +31,7 @@ export default function createPersistedState(storageKey: string, storage: Storag
       setState(value)
     }, [])
 
-    // A mounted hook handed a different key has to show that key's value before
-    // anything can be written, or the next update stores the previous key's value
-    // under the new key and destroys what was there. Adjusting during render ties
-    // the re-read to the change itself, where an effect would cost a read and a
-    // second render on every mount as well.
+    // A new key must be read before anything is written, or the next update stores the old key's value under it.
     const renderedKey = useRef(key)
 
     if (renderedKey.current !== key) {
@@ -48,7 +40,6 @@ export default function createPersistedState(storageKey: string, storage: Storag
       applyValue(readPersisted(key, initialValue))
     }
 
-    // The entries this hook has written and not yet seen reported back.
     const pendingOwnWrites = useRef<string[]>([])
 
     const setPersistedState = useCallback(
@@ -63,10 +54,7 @@ export default function createPersistedState(storageKey: string, storage: Storag
         try {
           newItem = getNewItem<T>(key, persistedItem, newValue)
         } catch {
-          // Refused, and reported where it was refused. A write replaces the whole
-          // entry, so an entry that cannot be read is one no write can be built on
-          // without dropping every other hook's key; skipping leaves the bytes for
-          // a repair to reach. The caller keeps what it set, unpersisted.
+          // A write replaces the whole entry, so an unreadable one is skipped rather than rebuilt without other keys.
           return
         }
 

package/src/index.ts

@@ -20,9 +20,6 @@ export default function createPersistedState<S extends Storage | AsyncStorage>(
 export { default as createPersistedState } from './create-persisted-state'
 export { default as createAsyncPersistedState } from './create-async-persisted-state'
 
-// The contract a backend implements, and the shape the hook hands back. Both appear in the
-// signatures above, so a consumer writing an adapter or typing a wrapper has to be able to name
-// them from the supported entry point rather than from `lib/`, which the exports map keeps open
-// only as a frozen legacy path.
+// Both appear in the signatures above, so a consumer writing an adapter can name them from the entry point.
 export type { AsyncStorage, Storage, StorageChange, StorageChangeEvent, StorageChangeListener } from './@types/storage'
 export type { PersistedState, UsePersistedState } from './@types/hook'

package/src/storages/chrome-storage.ts

@@ -7,25 +7,13 @@ chrome.storage.onChanged.addListener((changes, area) => {
   registry.fire(toStorageChanges(changes), area)
 })
 
-// chrome.storage is callback-based, unlike its promise-based firefox counterpart.
+// Passing no callback selects the promise form; the callback form leaves failures in `runtime.lastError`.
+// Called through `storage`: a torn-off `StorageArea` method throws `Illegal invocation`, unlike firefox's.
+// `get` stays `async` so asking whether this storage is asynchronous costs no round trip to the extension.
 const createStorage = (storage: chrome.storage.StorageArea, area: Area): AsyncStorage => ({
-  // Declared async, like the firefox adapter, so that asking whether this
-  // storage is asynchronous can be answered from the function itself instead of
-  // by calling it — a call here costs a round trip to the extension process.
-  get: async keys =>
-    new Promise(resolve => {
-      storage.get(keys, items => {
-        resolve(toStoredItems(items))
-      })
-    }),
-  set: items =>
-    new Promise(resolve => {
-      storage.set(items, resolve)
-    }),
-  remove: keys =>
-    new Promise(resolve => {
-      storage.remove(keys, resolve)
-    }),
+  get: async keys => toStoredItems(await storage.get(keys)),
+  set: items => storage.set(items),
+  remove: keys => storage.remove(keys),
   onChanged: registry.createOnChanged(area),
 })
 

package/src/storages/local-storage.ts

@@ -1,5 +1,4 @@
 import createStorage from '../utils/create-web-storage'
 
-// The typeof guard keeps this import from throwing on server runtimes (SSR),
-// where the browser storage globals are not defined.
+// The typeof guard keeps this import from throwing on server runtimes, where the global is not defined.
 export default createStorage(typeof localStorage !== 'undefined' ? localStorage : undefined)

package/src/storages/session-storage.ts

@@ -1,5 +1,4 @@
 import createStorage from '../utils/create-web-storage'
 
-// The typeof guard keeps this import from throwing on server runtimes (SSR),
-// where the browser storage globals are not defined.
+// The typeof guard keeps this import from throwing on server runtimes, where the global is not defined.
 export default createStorage(typeof sessionStorage !== 'undefined' ? sessionStorage : undefined)