fix #648

.changeset/unlucky-olives-try.md

@@ -0,0 +1,5 @@
+---
+"@opennextjs/aws": patch
+---
+
+use correct version for applying stableIncrementalCache (It failed for next 14.0.4)

packages/open-next/src/build/createServerBundle.ts

@@ -179,7 +179,7 @@ async function generateBundle(
   const isBefore13413 =
     buildHelper.compareSemver(options.nextVersion, "13.4.13") <= 0;
   const isAfter141 =
-    buildHelper.compareSemver(options.nextVersion, "14.0.4") >= 0;
+    buildHelper.compareSemver(options.nextVersion, "14.1") >= 0;
 
   const disableRouting = isBefore13413 || config.middleware?.external;
 

packages/open-next/src/build/helper.ts

@@ -290,7 +290,7 @@ export function compareSemver(v1: string, v2: string): number {
 
   if (major1 !== major2) return major1 - major2;
   if (minor1 !== minor2) return minor1 - minor2;
-  return patch1 - patch2;
+  return (patch1 ?? 0) - (patch2 ?? 0);
 }
 
 export function copyOpenNextConfig(

packages/tests-unit/tests/build/helper.test.ts

@@ -0,0 +1,25 @@
+import { compareSemver } from "@opennextjs/aws/build/helper.js";
+
+// We don't need to test canary versions, they are stripped out
+describe("compareSemver", () => {
+  it("should return 0 when versions are equal", () => {
+    expect(compareSemver("1.0.0", "1.0.0")).toBe(0);
+  });
+
+  it("should return 1 when first version is greater", () => {
+    expect(compareSemver("1.0.1", "1.0.0")).toBe(1);
+  });
+
+  it("should return -1 when first version is smaller", () => {
+    expect(compareSemver("1.0.0", "1.0.1")).toBe(-1);
+  });
+
+  it("should work with latest", () => {
+    expect(compareSemver("latest", "1.0.0")).toBe(1);
+  });
+
+  it("should work with incomplete version", () => {
+    expect(compareSemver("14.1.0", "14.1")).toBe(0);
+    expect(compareSemver("14.1", "14.1.0")).toBe(0);
+  });
+});