fix(cache): add null check for globalThis.openNextConfig

Adds optional chaining when accessing globalThis.openNextConfig to prevent
TypeError during Next.js 16 build phase when using the Adapters API. The cache
handler can be instantiated during SSG/prerendering before openNextConfig is
initialized by the runtime handlers.

Author: DBosley

.changeset/fix-opennextconfig-null-check.md

@@ -0,0 +1,10 @@
+---
+"@opennextjs/aws": patch
+---
+
+fix: add null check for globalThis.openNextConfig in cache handler
+
+Adds optional chaining when accessing globalThis.openNextConfig to prevent
+TypeError during Next.js 16 build phase when using the Adapters API. The cache
+handler can be instantiated during SSG/prerendering before openNextConfig is
+initialized by the runtime handlers.

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

@@ -45,7 +45,7 @@ export default class Cache {
           kind?: "FETCH";
         },
   ) {
-    if (globalThis.openNextConfig.dangerous?.disableIncrementalCache) {
+    if (globalThis.openNextConfig?.dangerous?.disableIncrementalCache) {
       return null;
     }
 
@@ -204,7 +204,7 @@ export default class Cache {
     data?: IncrementalCacheValue,
     ctx?: IncrementalCacheContext,
   ): Promise<void> {
-    if (globalThis.openNextConfig.dangerous?.disableIncrementalCache) {
+    if (globalThis.openNextConfig?.dangerous?.disableIncrementalCache) {
       return;
     }
     // This one might not even be necessary anymore
@@ -322,7 +322,7 @@ export default class Cache {
   }
 
   public async revalidateTag(tags: string | string[]) {
-    const config = globalThis.openNextConfig.dangerous;
+    const config = globalThis.openNextConfig?.dangerous;
     if (config?.disableTagCache || config?.disableIncrementalCache) {
       return;
     }
@@ -430,7 +430,7 @@ export default class Cache {
     ctx?: IncrementalCacheContext,
   ) {
     if (
-      globalThis.openNextConfig.dangerous?.disableTagCache ||
+      globalThis.openNextConfig?.dangerous?.disableTagCache ||
       globalThis.tagCache.mode === "nextMode" ||
       // Here it means it's a delete
       !data

packages/tests-unit/tests/adapters/cache.test.ts

@@ -85,6 +85,34 @@ describe("CacheHandler", () => {
       expect(result).toBeNull();
     });
 
+    describe("undefined openNextConfig", () => {
+      it("Should not throw when globalThis.openNextConfig is undefined", async () => {
+        // @ts-expect-error - Testing undefined config scenario
+        globalThis.openNextConfig = undefined;
+
+        // Should not throw TypeError: Cannot read properties of undefined (reading 'dangerous')
+        const result = await cache.get("key");
+
+        expect(result).not.toBeUndefined();
+      });
+
+      it("Should not throw on set when globalThis.openNextConfig is undefined", async () => {
+        // @ts-expect-error - Testing undefined config scenario
+        globalThis.openNextConfig = undefined;
+
+        await expect(
+          cache.set("key", { kind: "REDIRECT", props: {} }),
+        ).resolves.not.toThrow();
+      });
+
+      it("Should not throw on revalidateTag when globalThis.openNextConfig is undefined", async () => {
+        // @ts-expect-error - Testing undefined config scenario
+        globalThis.openNextConfig = undefined;
+
+        await expect(cache.revalidateTag("tag")).resolves.not.toThrow();
+      });
+    });
+
     describe("disableIncrementalCache", () => {
       beforeEach(() => {
         globalThis.openNextConfig.dangerous.disableIncrementalCache = true;