only inline data if hydrate=true (#837)

* only inline data if hydrate=true

* changeset

.changeset/wild-beans-happen.md

@@ -0,0 +1,5 @@
+---
+'@sveltejs/kit': patch
+---
+
+Only inline data if hydrate=true

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

@@ -150,7 +150,7 @@ async function get_response({ request, options, $session, route, status = 200, e
 			}
 		}
 
-		if (response) {
+		if (response && page_config.hydrate) {
 			const proxy = new Proxy(response, {
 				get(response, key, receiver) {
 					async function text() {
@@ -190,9 +190,12 @@ async function get_response({ request, options, $session, route, status = 200, e
 			return proxy;
 		}
 
-		return new Response('Not found', {
-			status: 404
-		});
+		return (
+			response ||
+			new Response('Not found', {
+				status: 404
+			})
+		);
 	};
 
 	const component_promises = error

packages/kit/test/apps/basics/src/routes/hydrate/__tests__.js

@@ -12,6 +12,12 @@ export default function (test, is_dev) {
 
 			await page.click('button');
 			assert.equal(await page.textContent('button'), 'clicks: 1');
+		} else {
+			// ensure data wasn't inlined
+			assert.equal(
+				await page.evaluate(() => document.querySelectorAll('script[type="svelte-data"]').length),
+				0
+			);
 		}
 	});
 }

packages/kit/test/apps/basics/src/routes/hydrate/index.json.js

@@ -0,0 +1,7 @@
+export function get() {
+	return {
+		body: {
+			type: 'hydrate'
+		}
+	};
+}

packages/kit/test/apps/basics/src/routes/hydrate/index.svelte

@@ -1,13 +1,28 @@
 <script context="module">
 	export const hydrate = false;
+
+	/** @type {import('../../../../../../types').Load} */
+	export async function load({ fetch }) {
+		const res = await fetch('/hydrate.json');
+
+		/** @type {any} */
+		const { type } = await res.json();
+
+		return {
+			props: { type }
+		}
+	}
 </script>
 
 <script>
+	/** @type {string} */
+	export let type;
+
 	let n = 0;
 </script>
 
 <button on:click={() => n += 1}>
 	clicks: {n}
 </button>
 
-<a href="/hydrate/other">other</a>
+<a href="/{type}/other">other</a>