fix: setting the right tag values for fetch cache (#304)

* fix: get tag values for fetch cache

Author: mathisobadia

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

@@ -69,6 +69,14 @@ type IncrementalCacheValue =
   | CachedFetchValue
   | CachedRouteValue;
 
+type IncrementalCacheContext = {
+  revalidate?: number | false | undefined;
+  fetchCache?: boolean | undefined;
+  fetchUrl?: string | undefined;
+  fetchIdx?: number | undefined;
+  tags?: string[] | undefined;
+};
+
 interface CacheHandlerContext {
   fs?: never;
   dev?: boolean;
@@ -243,7 +251,11 @@ export default class S3Cache {
     }
   }
 
-  async set(key: string, data?: IncrementalCacheValue): Promise<void> {
+  async set(
+    key: string,
+    data?: IncrementalCacheValue,
+    ctx?: IncrementalCacheContext,
+  ): Promise<void> {
     if (globalThis.disableIncrementalCache) {
       return;
     }
@@ -294,7 +306,7 @@ export default class S3Cache {
     // If we use an in house version of getDerivedTags in build we should use it here instead of next's one
     const derivedTags: string[] =
       data?.kind === "FETCH"
-        ? data.data.tags ?? []
+        ? ctx?.tags ?? data?.data?.tags ?? [] // before version 14 next.js used data?.data?.tags so we keep it for backward compatibility
         : data?.kind === "PAGE"
         ? data.headers?.["x-next-cache-tags"]?.split(",") ?? []
         : [];