fix: correctly compare routes (#8952)

sveltejs · Feb 9, 2023 · 15c2777 · 15c2777
1 parent 51cd6e6
commit 15c2777
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/.changeset/rich-chefs-refuse.md b/.changeset/rich-chefs-refuse.md
@@ -0,0 +1,5 @@
+---
+'@sveltejs/adapter-netlify': patch
+---
+
+fix: correctly compare routes when generating split functions
diff --git a/packages/adapter-netlify/index.js b/packages/adapter-netlify/index.js
@@ -213,10 +213,11 @@ async function generate_lambda_functions({ builder, publish, split }) {
 
 			// figure out which lower priority routes should be considered fallbacks
 			for (let j = i + 1; j < builder.routes.length; j += 1) {
-				if (routes[j].prerender === true) continue;
+				const other = builder.routes[j];
+				if (other.prerender === true) continue;
 
-				if (matches(route.segments, routes[j].segments)) {
-					routes.push(builder.routes[j]);
+				if (matches(route.segments, other.segments)) {
+					routes.push(other);
 				}
 			}