Ensure props are loaded from matching endpoint during client-side navigation (#4203)

* add failing test from #4139

* fix #4038

* changeset

* add comment

Author: Rich-Harris

.changeset/nasty-buttons-invent.md

@@ -0,0 +1,5 @@
+---
+'@sveltejs/kit': patch
+---
+
+Ensure props are loaded from matching endpoint during client-side navigation

packages/kit/src/core/dev/plugin.js

@@ -107,6 +107,7 @@ export async function create_plugin(config, cwd) {
 							if (route.type === 'page') {
 								return {
 									type: 'page',
+									key: route.key,
 									pattern: route.pattern,
 									params: get_params(route.params),
 									shadow: route.shadow

packages/kit/src/core/generate_manifest/index.js

@@ -69,6 +69,7 @@ export function generate_manifest({ build_data, relative_path, routes, format =
 					if (route.type === 'page') {
 						return `{
 							type: 'page',
+							key: ${s(route.key)},
 							pattern: ${route.pattern},
 							params: ${get_params(route.params)},
 							path: ${route.path ? s(route.path) : null},

packages/kit/src/core/sync/create_manifest_data/index.js

@@ -266,20 +266,19 @@ export default function create_manifest_data({
 
 	walk(config.kit.files.routes, [], [], [], [layout], [error]);
 
-	// merge matching page/endpoint pairs into shadowed pages
+	const lookup = new Map();
+	for (const route of routes) {
+		if (route.type === 'page') {
+			lookup.set(route.key, route);
+		}
+	}
+
 	let i = routes.length;
 	while (i--) {
 		const route = routes[i];
-		const prev = routes[i - 1];
-
-		if (prev && prev.key === route.key) {
-			if (prev.type !== 'endpoint' || route.type !== 'page') {
-				const relative = path.relative(cwd, path.resolve(config.kit.files.routes, prev.key));
-				throw new Error(`Duplicate route files: ${relative}`);
-			}
-
-			route.shadow = prev.file;
-			routes.splice(--i, 1);
+		if (route.type === 'endpoint' && lookup.has(route.key)) {
+			lookup.get(route.key).shadow = route.file;
+			routes.splice(i, 1);
 		}
 	}
 

packages/kit/src/core/sync/write_manifest.js

@@ -48,7 +48,7 @@ export function write_manifest(manifest_data, base, output) {
 
 					// optional items
 					if (params || route.shadow) tuple.push(params || 'null');
-					if (route.shadow) tuple.push('1');
+					if (route.shadow) tuple.push(`'${route.key}'`);
 
 					return `// ${route.a[route.a.length - 1]}\n\t\t[${tuple.join(', ')}]`;
 				}

packages/kit/src/runtime/client/client.js

@@ -567,7 +567,7 @@ export function create_client({ target, session, base, trailing_slash }) {
 			if (cached) return cached;
 		}
 
-		const [pattern, a, b, get_params, has_shadow] = route;
+		const [pattern, a, b, get_params, shadow_key] = route;
 		const params = get_params
 			? // the pattern is for the route which we've already matched to this path
 			  get_params(/** @type {RegExpExecArray}  */ (pattern.exec(path)))
@@ -618,18 +618,23 @@ export function create_client({ target, session, base, trailing_slash }) {
 					/** @type {Record<string, any>} */
 					let props = {};
 
-					const is_shadow_page = has_shadow && i === a.length - 1;
+					const is_shadow_page = shadow_key !== undefined && i === a.length - 1;
 
 					if (is_shadow_page) {
 						const res = await fetch(
 							`${url.pathname}${url.pathname.endsWith('/') ? '' : '/'}__data.json${url.search}`,
 							{
 								headers: {
-									'x-sveltekit-load': 'true'
+									'x-sveltekit-load': /** @type {string} */ (shadow_key)
 								}
 							}
 						);
 
+						if (res.status === 204) {
+							// fallthrough
+							return;
+						}
+
 						if (res.ok) {
 							const redirect = res.headers.get('x-sveltekit-location');
 

packages/kit/src/runtime/server/index.js

@@ -150,7 +150,17 @@ export async function respond(request, options, state = {}) {
 					event.url = new URL(event.url.origin + normalized + event.url.search);
 				}
 
+				// `key` will be set if this request came from a client-side navigation
+				// to a page with a matching endpoint
+				const key = request.headers.get('x-sveltekit-load');
+
 				for (const route of options.manifest._.routes) {
+					if (key) {
+						// client is requesting data for a specific endpoint
+						if (route.type !== 'page') continue;
+						if (route.key !== key) continue;
+					}
+
 					const match = route.pattern.exec(decoded);
 					if (!match) continue;
 
@@ -163,7 +173,7 @@ export async function respond(request, options, state = {}) {
 						response = await render_endpoint(event, await route.shadow());
 
 						// loading data for a client-side transition is a special case
-						if (request.headers.get('x-sveltekit-load') === 'true') {
+						if (key) {
 							if (response) {
 								// since redirects are opaque to the browser, we need to repackage
 								// 3xx responses as 200s with a custom header
@@ -180,9 +190,9 @@ export async function respond(request, options, state = {}) {
 									}
 								}
 							} else {
-								// TODO ideally, the client wouldn't request this data
-								// in the first place (at least in production)
-								response = new Response('{}', {
+								// fallthrough
+								response = new Response(undefined, {
+									status: 204,
 									headers: {
 										'content-type': 'application/json'
 									}

packages/kit/test/apps/basics/src/routes/shadowed/fallthrough/[a].js

@@ -0,0 +1,15 @@
+/** @type {import('./[a]').RequestHandler} */
+export async function get({ params }) {
+	const param = params.a;
+
+	if (param !== 'a') {
+		return {
+			fallthrough: true
+		};
+	}
+
+	return {
+		status: 200,
+		body: { param }
+	};
+}

packages/kit/test/apps/basics/src/routes/shadowed/fallthrough/[a].svelte

@@ -0,0 +1,6 @@
+<script>
+	/** @type {string} */
+	export let param;
+</script>
+
+<h2>a-{param}</h2>

packages/kit/test/apps/basics/src/routes/shadowed/fallthrough/[b].js

@@ -0,0 +1,15 @@
+/** @type {import('./[b]').RequestHandler} */
+export async function get({ params }) {
+	const param = params.b;
+
+	if (param !== 'b') {
+		return {
+			fallthrough: true
+		};
+	}
+
+	return {
+		status: 200,
+		body: { param }
+	};
+}