fix: bypass next tag cache when there are no tags to check

vicb · vicb · commit 67d86b05a3e4 · 2025-10-01T09:45:19.000+02:00
diff --git a/.changeset/rude-trains-know.md b/.changeset/rude-trains-know.md
@@ -0,0 +1,5 @@
+---
+"@opennextjs/aws": patch
+---
+
+fix: bypass next tag cache when there are no tags to check
diff --git a/packages/open-next/src/utils/cache.ts b/packages/open-next/src/utils/cache.ts
@@ -23,7 +23,9 @@ export async function hasBeenRevalidated(
   }
   const lastModified = cacheEntry.lastModified ?? Date.now();
   if (globalThis.tagCache.mode === "nextMode") {
-    return await globalThis.tagCache.hasBeenRevalidated(tags, lastModified);
+    return tags.length === 0
+      ? false
+      : await globalThis.tagCache.hasBeenRevalidated(tags, lastModified);
   }
   // TODO: refactor this, we should introduce a new method in the tagCache interface so that both implementations use hasBeenRevalidated
   const _lastModified = await globalThis.tagCache.getLastModified(

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +---
 +"@opennextjs/aws": patch
 +---
++
 +fix: bypass next tag cache when there are no tags to check
Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,9 @@ export async function hasBeenRevalidated(`
`23`	`23`	`}`
`24`	`24`	`const lastModified = cacheEntry.lastModified ?? Date.now();`
`25`	`25`	`if (globalThis.tagCache.mode === "nextMode") {`
`26`		`- return await globalThis.tagCache.hasBeenRevalidated(tags, lastModified);`
	`26`	`+ return tags.length === 0`
	`27`	`+ ? false`
	`28`	`+ : await globalThis.tagCache.hasBeenRevalidated(tags, lastModified);`
`27`	`29`	`}`
`28`	`30`	`// TODO: refactor this, we should introduce a new method in the tagCache interface so that both implementations use hasBeenRevalidated`
`29`	`31`	`const _lastModified = await globalThis.tagCache.getLastModified(`