don't check tag cache for page router

Author: conico974

packages/open-next/src/adapters/cache.ts

@@ -66,7 +66,7 @@ export default class Cache {
       const _hasBeenRevalidated = await hasBeenRevalidated(
         key,
         _tags,
-        cachedEntry?.lastModified,
+        cachedEntry,
       );
 
       if (_hasBeenRevalidated) return null;
@@ -85,7 +85,7 @@ export default class Cache {
           const hasPathBeenUpdated = await hasBeenRevalidated(
             path.replace("_N_T_/", ""),
             [],
-            cachedEntry.lastModified,
+            cachedEntry,
           );
           if (hasPathBeenUpdated) {
             // In case the path has been revalidated, we don't want to use the fetch cache
@@ -121,7 +121,7 @@ export default class Cache {
       const _hasBeenRevalidated = await hasBeenRevalidated(
         key,
         tags,
-        _lastModified,
+        cachedEntry,
       );
       if (cacheData === undefined || _hasBeenRevalidated) return null;
 

packages/open-next/src/core/routing/cacheInterceptor.ts

@@ -168,7 +168,7 @@ export async function cacheInterceptor(
         const _hasBeenRevalidated = await hasBeenRevalidated(
           localizedPath,
           tags,
-          cachedData.lastModified,
+          cachedData,
         );
         if (_hasBeenRevalidated) {
           return event;

packages/open-next/src/types/cache.ts

@@ -97,6 +97,7 @@ export type TagCacheMetaFile = {
   path: { S: string };
   revalidatedAt: { N: string };
 };
+
 export type IncrementalCacheContext = {
   revalidate?: number | false | undefined;
   fetchCache?: boolean | undefined;

packages/open-next/src/utils/cache.ts

@@ -1,13 +1,22 @@
-import type { CacheValue } from "types/overrides";
+import type { CacheValue, WithLastModified } from "types/overrides";
 
 export async function hasBeenRevalidated(
   key: string,
   tags: string[],
-  lastModified?: number,
+  cacheEntry: WithLastModified<CacheValue<any>>,
 ): Promise<boolean> {
   if (globalThis.openNextConfig.dangerous?.disableTagCache) {
     return false;
   }
+  const value = cacheEntry.value;
+  if (!value) {
+    // We should never reach this point
+    return true;
+  }
+  if ("type" in cacheEntry && cacheEntry.type === "page") {
+    return false;
+  }
+  const lastModified = cacheEntry.lastModified ?? Date.now();
   if (globalThis.tagCache.mode === "nextMode") {
     return await globalThis.tagCache.hasBeenRevalidated(tags, lastModified);
   }