@ethlete/query 6.0.0-next.13 → 6.0.0-next.14

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # @ethlete/query
 
+## 6.0.0-next.14
+
+### Patch Changes
+
+- [#3019](https://github.com/ethlete-io/ethdk/pull/3019) [`b5655a9`](https://github.com/ethlete-io/ethdk/commit/b5655a9124b2ffc4d6982ca20be360b227a37107) Thanks [@github-actions](https://github.com/apps/github-actions)! - Query cache: a cached request now correctly becomes stale once its freshness
+  window (from `cache-control` / `expires`) elapses, so `execute({ allowCache: true })`
+  refetches instead of serving the cached response forever.
+
 ## 6.0.0-next.13
 
 ### Patch Changes

package/fesm2022/ethlete-query.mjs

@@ -1576,11 +1576,14 @@ const createHttpRequest = (options) => {
     ...(ngDevMode ? [{ debugName: "lastExecuteTime" }] : /* istanbul ignore next */ []));
     const expiresIn = signal(null, /* @ts-ignore */
     ...(ngDevMode ? [{ debugName: "expiresIn" }] : /* istanbul ignore next */ []));
-    const isStale = computed(() => {
+    // NOTE: This must be a plain function, not a `computed`. The freshness check compares against
+    // `Date.now()`, which is not reactive, so a memoized computed would only ever recompute when
+    // `expiresIn` changes — once it evaluated to `false` (fresh) it would stay `false` forever, even
+    // after the window elapsed, turning every `allowCache` execute into a permanent cache hit.
+    const isStale = () => {
         const expiresInTs = expiresIn();
         return expiresInTs === null || expiresInTs < Date.now();
-    }, /* @ts-ignore */
-    ...(ngDevMode ? [{ debugName: "isStale" }] : /* istanbul ignore next */ []));
+    };
     const createStream = () => {
         const headers = typeof args?.headers === 'function' ? args.headers() : args?.headers;
         return dependencies.httpClient