Update adapters to use new app.render API

.changeset/honest-beers-sing.md

@@ -0,0 +1,9 @@
+---
+'@sveltejs/adapter-cloudflare': patch
+'@sveltejs/adapter-cloudflare-workers': patch
+'@sveltejs/adapter-netlify': patch
+'@sveltejs/adapter-node': patch
+'@sveltejs/adapter-vercel': patch
+---
+
+Update adapters to provide app.render with a url

.changeset/rude-balloons-return.md

@@ -0,0 +1,5 @@
+---
+'@sveltejs/kit': patch
+---
+
+Error if adapter provides wrong input to app.render

.changeset/shy-oranges-sin.md

@@ -0,0 +1,5 @@
+---
+'@sveltejs/kit': patch
+---
+
+Replace [request|page].[origin|path|query] with url object

documentation/docs/05-modules.md

@@ -48,7 +48,7 @@ import { base, assets } from '$app/paths';
 import { getStores, navigating, page, session } from '$app/stores';
 ```
 
-Stores are _contextual_ — they are added to the [context](https://svelte.dev/tutorial/context-api) of your root component. This means that `session` and `page` are unique to each request on the server, rather than shared between multiple requests handled by the same server simultaneously, which is what makes it safe to include user-specific data in `session`.
+Stores are _contextual_ — they are added to the [context](https://svelte.dev/tutorial/context-api) of your root component. This means that `session` and `route` are unique to each request on the server, rather than shared between multiple requests handled by the same server simultaneously, which is what makes it safe to include user-specific data in `session`.
 
 Because of that, the stores are not free-floating objects: they must be accessed during component initialisation, like anything else that would be accessed with `getContext`.
 

documentation/migrating/04-pages.md

@@ -35,7 +35,7 @@ const { preloading, page, session } = stores();
 
 The `page` and `session` stores still exist; `preloading` has been replaced with a `navigating` store that contains `from` and `to` properties. `page` now has `url` and `params` properties, but no `path` or `query`.
 
-You access them differently in SvelteKit. `stores` is now `getStores`, but in most cases it is unnecessary since you can import `navigating`, `page` and `session` directly from [`$app/stores`](/docs#modules-$app-stores).
+You access them differently in SvelteKit. `stores` is now `getStores`, but in most cases it is unnecessary since you can import `navigating`, `route` and `session` directly from [`$app/stores`](/docs#modules-$app-stores).
 
 ### Routing
 

packages/adapter-cloudflare-workers/files/entry.js

@@ -26,12 +26,10 @@ async function handle(event) {
 
 	// fall back to an app route
 	const request = event.request;
-	const request_url = new URL(request.url);
 
 	try {
 		const rendered = await app.render({
-			path: request_url.pathname,
-			query: request_url.searchParams,
+			url: request.url,
 			rawBody: await read(request),
 			headers: Object.fromEntries(request.headers),
 			method: request.method

packages/adapter-cloudflare/files/worker.js

@@ -33,8 +33,7 @@ export default {
 		// dynamically-generated pages
 		try {
 			const rendered = await app.render({
-				path: url.pathname,
-				query: url.searchParams,
+				url,
 				rawBody: new Uint8Array(await req.arrayBuffer()),
 				headers: Object.fromEntries(req.headers),
 				method: req.method

packages/adapter-netlify/src/handler.js

@@ -7,18 +7,15 @@ export function init(manifest) {
 	const app = new App(manifest);
 
 	return async (event) => {
-		const { path, httpMethod, headers, rawQuery, body, isBase64Encoded } = event;
-
-		const query = new URLSearchParams(rawQuery);
+		const { httpMethod, headers, rawUrl, body, isBase64Encoded } = event;
 
 		const encoding = isBase64Encoded ? 'base64' : headers['content-encoding'] || 'utf-8';
 		const rawBody = typeof body === 'string' ? Buffer.from(body, encoding) : body;
 
 		const rendered = await app.render({
+			url: rawUrl,
 			method: httpMethod,
 			headers,
-			path,
-			query,
 			rawBody
 		});
 

packages/adapter-node/src/handler.js

@@ -57,10 +57,9 @@ const ssr = async (req, res) => {
 	}
 
 	const rendered = await app.render({
+		url: req.url,
 		method: req.method,
 		headers: req.headers, // TODO: what about repeated headers, i.e. string[]
-		path: parsed.pathname,
-		query: parsed.searchParams,
 		rawBody: body
 	});
 

packages/adapter-vercel/files/entry.js

@@ -8,8 +8,6 @@ __fetch_polyfill();
 const app = new App(manifest);
 
 export default async (req, res) => {
-	const { pathname, searchParams } = new URL(req.url || '', 'http://localhost');
-
 	let body;
 
 	try {
@@ -20,10 +18,9 @@ export default async (req, res) => {
 	}
 
 	const rendered = await app.render({
+		url: req.url,
 		method: req.method,
 		headers: req.headers,
-		path: pathname,
-		query: searchParams,
 		rawBody: body
 	});
 

packages/kit/src/core/build/build_server.js

@@ -88,6 +88,11 @@ export class App {
 	render(request, {
 		prerender
 	} = {}) {
+		// TODO remove this for 1.0
+		if (Object.keys(request).sort().join() !== 'headers,method,rawBody,url') {
+			throw new Error('Adapters should call app.render({ url, method, headers, rawBody })');
+		}
+
 		const host = ${
 			config.kit.host
 				? s(config.kit.host)