@playfast/reform-query-browser 0.0.2 → 0.1.0

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@playfast/reform-query-browser",
   "playbook": "./playbook",
-  "version": "0.0.2",
+  "version": "0.1.0",
   "type": "module",
   "description": "Browser host layers for Reform's AsyncCalc — window focus/online auto-invalidation and localStorage persistence. Keeps reform core DOM-free: implements the optional QueryEvents and QueryStore seams.",
   "keywords": [

package/src/localStorage.store.ts

@@ -1,5 +1,5 @@
 import { QueryStore, type QueryStoreApi, noopQueryStore } from '@playfast/reform'
-import { Effect, Layer, Option } from 'effect'
+import { Effect, Layer, Option, Schema } from 'effect'
 
 // `QueryStore` implemented over `localStorage` (JSON values under a namespace
 // prefix). The driver hands already-`output`-encoded values across this seam, so
@@ -8,26 +8,24 @@ import { Effect, Layer, Option } from 'effect'
 
 const PREFIX = 'reform-query:'
 
-const parse = (raw: string): Option.Option<unknown> => {
-  try {
-    return Option.some(JSON.parse(raw))
-  } catch {
-    // A hand-edited or version-skewed entry: treat as a cache miss, fetch fresh.
-    return Option.none()
-  }
-}
+const JsonValue = Schema.parseJson(Schema.Unknown)
+// A hand-edited or version-skewed entry fails to decode → `None`: a cache miss, fetch fresh.
+const decodeJson = Schema.decodeUnknownOption(JsonValue)
+const encodeJson = Schema.encodeSync(JsonValue)
 
 const makeApi = (): QueryStoreApi => {
-  if (typeof window === 'undefined' || typeof localStorage === 'undefined') return noopQueryStore
+  if (typeof window === 'undefined' || typeof localStorage === 'undefined') {
+    return noopQueryStore
+  }
   return {
     get: (key) =>
       Effect.sync(() => {
         const raw = localStorage.getItem(PREFIX + key)
-        return raw === null ? Option.none() : parse(raw)
+        return raw === null ? Option.none() : decodeJson(raw)
       }),
-    set: (key, value) =>
+    set: (key, payload) =>
       Effect.sync(() => {
-        localStorage.setItem(PREFIX + key, JSON.stringify(value))
+        localStorage.setItem(PREFIX + key, encodeJson(payload))
       }),
     remove: (key) =>
       Effect.sync(() => {

package/src/queryEvents.browser.ts

@@ -11,17 +11,21 @@ import { Effect, Layer } from 'effect'
 export const QueryEventsBrowser: Layer.Layer<QueryEvents> = Layer.scoped(
   QueryEvents,
   Effect.gen(function* () {
-    if (typeof window === 'undefined') return noopQueryEvents
+    if (typeof window === 'undefined') {
+      return noopQueryEvents
+    }
 
     const focusListeners = new Set<() => void>()
     const onlineListeners = new Set<() => void>()
     const fire = (listeners: ReadonlySet<() => void>) => {
-      for (const listener of listeners) listener()
+      listeners.forEach((listener) => listener())
     }
     // Refocus and tab-becomes-visible both count as "focus"; ignore the
     // visibility event that fires on hide.
     const onFocus = () => {
-      if (document.visibilityState === 'visible') fire(focusListeners)
+      if (document.visibilityState === 'visible') {
+        fire(focusListeners)
+      }
     }
     const onOnline = () => fire(onlineListeners)
 

package/src/refetchManagers.ts

@@ -16,7 +16,7 @@ const onSignal = (
       const events = yield* QueryEvents
       const queries = yield* Queries
       const invalidateAll = () => {
-        for (const handle of queries.byName.values()) handle.invalidate()
+        queries.byName.forEach((handle) => handle.invalidate())
       }
       const unsubscribe = pick(events)(invalidateAll)
       yield* Effect.addFinalizer(() => Effect.sync(unsubscribe))