Fixed: Redis keys ttl do not respect revalidate configurations (#33)

* Package bumps

* Fixed revalidate (TTL) for full route cache

* Bump version

Author: AyronK

examples/redis-minimal/package-lock.json

@@ -13,20 +13,20 @@
     "lint": "next lint"
   },
   "dependencies": {
-    "@fortedigital/nextjs-cache-handler": "^2.0.0-canary12",
-    "next": "15.3.4",
-    "react": "^19.0.0",
-    "react-dom": "^19.0.0",
-    "redis": "^5.5.6"
+    "@fortedigital/nextjs-cache-handler": "^2.0.0",
+    "next": "15.4.1",
+    "react": "^19.1.0",
+    "react-dom": "^19.1.0",
+    "redis": "^5.6.0"
   },
   "devDependencies": {
     "@eslint/eslintrc": "^3",
     "@tailwindcss/postcss": "^4",
-    "@types/node": "^20",
+    "@types/node": "^24",
     "@types/react": "^19",
     "@types/react-dom": "^19",
     "eslint": "^9",
-    "eslint-config-next": "15.3.4",
+    "eslint-config-next": "15.4.1",
     "tailwindcss": "^4",
     "typescript": "^5"
   }

examples/redis-minimal/package.json

@@ -18,7 +18,7 @@
     "next",
     "redis"
   ],
-  "version": "2.0.0",
+  "version": "2.0.1",
   "type": "module",
   "license": "MIT",
   "description": "Next.js cache handlers",

package-lock.json

@@ -1 +1 @@
-export const REVALIDATED_TAGS_KEY = "__revalidated_tags__";
+export const REVALIDATED_TAGS_KEY = "__revalidated_tags__";

packages/nextjs-cache-handler/package-lock.json

@@ -16,12 +16,12 @@ import {
 } from "./cache-handler.types";
 import { PrerenderManifest } from "next/dist/build";
 import {
-  type CachedFetchValue,
   type IncrementalCachedPageValue,
   type GetIncrementalResponseCacheContext,
   type GetIncrementalFetchCacheContext,
   IncrementalCacheValue,
 } from "next/dist/server/response-cache/types";
+import { resolveRevalidateValue } from "../helpers/resolveRevalidateValue";
 
 const PRERENDER_MANIFEST_VERSION = 4;
 
@@ -651,7 +651,7 @@ export class CacheHandler implements NextCacheHandler {
 
     const { tags = [], internal_lastModified } = ctx ?? {};
 
-    const revalidate = (incrementalCacheValue as CachedFetchValue)?.revalidate;
+    const revalidate = resolveRevalidateValue(incrementalCacheValue, ctx);
 
     const lastModified = Math.round(internal_lastModified ?? Date.now());
 

packages/nextjs-cache-handler/package.json

@@ -0,0 +1,39 @@
+import {
+  type CachedFetchValue,
+  IncrementalCachedAppPageValue,
+  SetIncrementalResponseCacheContext,
+} from "next/dist/server/response-cache";
+import {
+  CacheHandlerParametersSet,
+  Revalidate,
+} from "../handlers/cache-handler.types";
+
+/**
+ * Resolved revalidate value based on type of the cached value.
+ *
+ * @param incrementalCacheValue - Todo
+ * @param ctx - todo
+ *
+ * @returns Cache revalidate value.
+ */
+export function resolveRevalidateValue(
+  incrementalCacheValue: CacheHandlerParametersSet[1],
+  ctx: CacheHandlerParametersSet[2] & {
+    revalidate?: Revalidate;
+  },
+) {
+  const cachedFetchValue = incrementalCacheValue as CachedFetchValue;
+  const cachedPageValue =
+    incrementalCacheValue as IncrementalCachedAppPageValue;
+  const responseCacheCtx = ctx as SetIncrementalResponseCacheContext;
+
+  let revalidate;
+
+  if (cachedFetchValue.kind === "FETCH") {
+    revalidate = cachedFetchValue.revalidate;
+  } else if (cachedPageValue.kind === "APP_PAGE") {
+    revalidate = responseCacheCtx.cacheControl?.revalidate;
+  }
+
+  return revalidate ?? ctx.revalidate;
+}

packages/nextjs-cache-handler/src/constants.ts

@@ -12,6 +12,7 @@ import {
 import type { OutgoingHttpHeaders } from "http";
 import { getTagsFromHeaders } from "../helpers/getTagsFromHeaders";
 import { Revalidate } from "../handlers/cache-handler.types";
+import { resolveRevalidateValue } from "../helpers/resolveRevalidateValue";
 
 type CacheHandlerType = typeof import("../handlers/cache-handler").CacheHandler;
 
@@ -392,9 +393,11 @@ export async function registerInitialCache(
       return;
     }
 
+    const revalidateValue = fetchCache.revalidate;
+
     // HACK: By default, Next.js sets the revalidate option to CACHE_ONE_YEAR if the revalidate option is set
     const revalidate =
-      fetchCache.revalidate === CACHE_ONE_YEAR ? false : fetchCache.revalidate;
+      revalidateValue === CACHE_ONE_YEAR ? false : revalidateValue;
 
     try {
       await cacheHandler.set(fetchCacheKey, fetchCache, {