@sveltejs/kit 1.0.0-next.239 → 1.0.0-next.242

package/assets/server/index.js

@@ -1710,7 +1710,7 @@ async function respond(request, options, state = {}) {
 	let ssr = true;
 
 	try {
-		return await options.hooks.handle({
+		const response = await options.hooks.handle({
 			event,
 			resolve: async (event, opts) => {
 				if (opts && 'ssr' in opts) ssr = /** @type {boolean} */ (opts.ssr);
@@ -1806,6 +1806,13 @@ async function respond(request, options, state = {}) {
 				throw new Error('request in handle has been replaced with event' + details);
 			}
 		});
+
+		// TODO for 1.0, change the error message to point to docs rather than PR
+		if (response && !(response instanceof Response)) {
+			throw new Error('handle must return a Response object' + details);
+		}
+
+		return response;
 	} catch (/** @type {unknown} */ e) {
 		const error = coalesce_to_error(e);
 

package/dist/chunks/{http.js → build.js}

@@ -655,26 +655,4 @@ function sirv (dir, opts={}) {
 	};
 }
 
-/** @param {Partial<import('types/helper').ResponseHeaders> | undefined} object */
-function to_headers(object) {
-	const headers = new Headers();
-
-	if (object) {
-		for (const key in object) {
-			const value = object[key];
-			if (!value) continue;
-
-			if (typeof value === 'string') {
-				headers.set(key, value);
-			} else {
-				value.forEach((value) => {
-					headers.append(key, value);
-				});
-			}
-		}
-	}
-
-	return headers;
-}
-
-export { sirv as s, to_headers as t };
+export { sirv as s };

package/dist/chunks/index.js

@@ -5,11 +5,11 @@ import { c as create_manifest_data, a as create_app, d as deep_merge } from './i
 import { c as coalesce_to_error, S as SVELTE_KIT_ASSETS, r as resolve_entry, $, a as SVELTE_KIT, b as runtime, l as load_template, g as get_mime_lookup, d as copy_assets, e as get_aliases, p as print_config_conflicts } from '../cli.js';
 import fs__default from 'fs';
 import { URL as URL$1 } from 'url';
-import { t as to_headers, s as sirv } from './http.js';
+import { s as sirv } from './build.js';
 import { e as escape_html_attr, r as resolve, i as is_root_relative, a as escape_json_string_in_html } from './url.js';
 import { s } from './misc.js';
 import { __fetch_polyfill } from '../install-fetch.js';
-import { getRawBody, setResponse } from '../node.js';
+import { getRequest, setResponse } from '../node.js';
 import 'sade';
 import 'child_process';
 import 'net';
@@ -23,6 +23,28 @@ import 'node:util';
 import 'node:url';
 import 'stream';
 
+/** @param {Partial<import('types/helper').ResponseHeaders> | undefined} object */
+function to_headers(object) {
+	const headers = new Headers();
+
+	if (object) {
+		for (const key in object) {
+			const value = object[key];
+			if (!value) continue;
+
+			if (typeof value === 'string') {
+				headers.set(key, value);
+			} else {
+				value.forEach((value) => {
+					headers.append(key, value);
+				});
+			}
+		}
+	}
+
+	return headers;
+}
+
 /**
  * Hash using djb2
  * @param {import('types/hooks').StrictBody} value
@@ -1582,7 +1604,7 @@ async function respond(request, options, state = {}) {
 	let ssr = true;
 
 	try {
-		return await options.hooks.handle({
+		const response = await options.hooks.handle({
 			event,
 			resolve: async (event, opts) => {
 				if (opts && 'ssr' in opts) ssr = /** @type {boolean} */ (opts.ssr);
@@ -1678,6 +1700,13 @@ async function respond(request, options, state = {}) {
 				throw new Error('request in handle has been replaced with event' + details);
 			}
 		});
+
+		// TODO for 1.0, change the error message to point to docs rather than PR
+		if (response && !(response instanceof Response)) {
+			throw new Error('handle must return a Response object' + details);
+		}
+
+		return response;
 	} catch (/** @type {unknown} */ e) {
 		const error = coalesce_to_error(e);
 
@@ -1834,11 +1863,9 @@ async function create_plugin(config, cwd) {
 						if (!req.url || !req.method) throw new Error('Incomplete request');
 						if (req.url === '/favicon.ico') return not_found(res);
 
-						const url = new URL$1(
-							`${vite.config.server.https ? 'https' : 'http'}://${req.headers.host}${req.url}`
-						);
+						const base = `${vite.config.server.https ? 'https' : 'http'}://${req.headers.host}`;
 
-						const decoded = decodeURI(url.pathname);
+						const decoded = decodeURI(new URL$1(base + req.url).pathname);
 
 						if (decoded.startsWith(assets)) {
 							const pathname = decoded.slice(assets.length);
@@ -1898,75 +1925,70 @@ async function create_plugin(config, cwd) {
 							assets
 						});
 
-						let body;
+						let request;
 
 						try {
-							body = await getRawBody(req);
+							request = await getRequest(base, req);
 						} catch (/** @type {any} */ err) {
 							res.statusCode = err.status || 400;
 							return res.end(err.reason || 'Invalid request body');
 						}
 
-						const rendered = await respond(
-							new Request(url.href, {
-								headers: /** @type {Record<string, string>} */ (req.headers),
-								method: req.method,
-								body
-							}),
-							{
-								amp: config.kit.amp,
-								dev: true,
-								floc: config.kit.floc,
-								get_stack: (error) => {
-									vite.ssrFixStacktrace(error);
-									return error.stack;
-								},
-								handle_error: (error, event) => {
-									vite.ssrFixStacktrace(error);
-									hooks.handleError({
-										error,
-										event,
-
-										// TODO remove for 1.0
-										// @ts-expect-error
-										get request() {
-											throw new Error(
-												'request in handleError has been replaced with event. See https://github.com/sveltejs/kit/pull/3384 for details'
-											);
-										}
-									});
-								},
-								hooks,
-								hydrate: config.kit.hydrate,
-								manifest,
-								method_override: config.kit.methodOverride,
-								paths: {
-									base: config.kit.paths.base,
-									assets
-								},
-								prefix: '',
-								prerender: config.kit.prerender.enabled,
-								read: (file) => fs__default.readFileSync(path__default.join(config.kit.files.assets, file)),
-								root,
-								router: config.kit.router,
-								target: config.kit.target,
-								template: ({ head, body, assets }) => {
-									let rendered = load_template(cwd, config)
-										.replace('%svelte.head%', () => head)
-										.replace('%svelte.body%', () => body)
-										.replace(/%svelte\.assets%/g, assets);
-
-									if (amp) {
-										const result = amp.validateString(rendered);
-
-										if (result.status !== 'PASS') {
-											const lines = rendered.split('\n');
-
-											/** @param {string} str */
-											const escape = (str) =>
-												str.replace(/&/g, '&amp;').replace(/</g, '&lt;').replace(/>/g, '&gt;');
-
-											rendered = `<!doctype html>
+						const rendered = await respond(request, {
+							amp: config.kit.amp,
+							dev: true,
+							floc: config.kit.floc,
+							get_stack: (error) => {
+								vite.ssrFixStacktrace(error);
+								return error.stack;
+							},
+							handle_error: (error, event) => {
+								vite.ssrFixStacktrace(error);
+								hooks.handleError({
+									error,
+									event,
+
+									// TODO remove for 1.0
+									// @ts-expect-error
+									get request() {
+										throw new Error(
+											'request in handleError has been replaced with event. See https://github.com/sveltejs/kit/pull/3384 for details'
+										);
+									}
+								});
+							},
+							hooks,
+							hydrate: config.kit.hydrate,
+							manifest,
+							method_override: config.kit.methodOverride,
+							paths: {
+								base: config.kit.paths.base,
+								assets
+							},
+							prefix: '',
+							prerender: config.kit.prerender.enabled,
+							read: (file) => fs__default.readFileSync(path__default.join(config.kit.files.assets, file)),
+							root,
+							router: config.kit.router,
+							target: config.kit.target,
+							template: ({ head, body, assets }) => {
+								let rendered = load_template(cwd, config)
+									.replace(/%svelte\.assets%/g, assets)
+									// head and body must be replaced last, in case someone tries to sneak in %svelte.assets% etc
+									.replace('%svelte.head%', () => head)
+									.replace('%svelte.body%', () => body);
+
+								if (amp) {
+									const result = amp.validateString(rendered);
+
+									if (result.status !== 'PASS') {
+										const lines = rendered.split('\n');
+
+										/** @param {string} str */
+										const escape = (str) =>
+											str.replace(/&/g, '&amp;').replace(/</g, '&lt;').replace(/>/g, '&gt;');
+
+										rendered = `<!doctype html>
 										<head>
 											<meta charset="utf-8" />
 											<meta name="viewport" content="width=device-width, initial-scale=1" />
@@ -1997,14 +2019,13 @@ async function create_plugin(config, cwd) {
 											)
 											.join('\n\n')}
 									`;
-										}
 									}
+								}
 
-									return rendered;
-								},
-								trailing_slash: config.kit.trailingSlash
-							}
-						);
+								return rendered;
+							},
+							trailing_slash: config.kit.trailingSlash
+						});
 
 						if (rendered) {
 							setResponse(res, rendered);

package/dist/chunks/index5.js

@@ -414,8 +414,11 @@ async function prerender({ cwd, out, log, config, build_data, fallback, all }) {
 	async function visit(path, decoded_path, referrer) {
 		/** @type {Map<string, Response>} */
 		const dependencies = new Map();
+		const render_path = config.kit.paths?.base
+			? `http://sveltekit-prerender${config.kit.paths.base}${path === '/' ? '' : path}`
+			: `http://sveltekit-prerender${path}`;
 
-		const rendered = await app.render(new Request(`http://sveltekit-prerender${path}`), {
+		const rendered = await app.render(new Request(render_path), {
 			prerender: {
 				all,
 				dependencies

package/dist/chunks/index6.js

@@ -2,9 +2,9 @@ import fs__default from 'fs';
 import http from 'http';
 import https from 'https';
 import { resolve, join } from 'path';
-import { s as sirv, t as to_headers } from './http.js';
+import { s as sirv } from './build.js';
 import { pathToFileURL } from 'url';
-import { getRawBody } from '../node.js';
+import { getRequest, setResponse } from '../node.js';
 import { __fetch_polyfill } from '../install-fetch.js';
 import { a as SVELTE_KIT, S as SVELTE_KIT_ASSETS } from '../cli.js';
 import 'querystring';
@@ -91,32 +91,21 @@ async function preview({
 		const initial_url = req.url;
 
 		const render_handler = async () => {
-			if (!req.method) throw new Error('Incomplete request');
-
-			let body;
-
-			try {
-				body = await getRawBody(req);
-			} catch (/** @type {any} */ err) {
-				res.statusCode = err.status || 400;
-				return res.end(err.reason || 'Invalid request body');
-			}
-
 			if (initial_url.startsWith(config.kit.paths.base)) {
 				const protocol = use_https ? 'https' : 'http';
 				const host = req.headers['host'];
 
-				const rendered = await app.render(
-					new Request(`${protocol}://${host}${initial_url}`, {
-						method: req.method,
-						headers: to_headers(req.headers),
-						body
-					})
-				);
-
-				res.writeHead(rendered.status, Object.fromEntries(rendered.headers));
-				if (rendered.body) res.write(new Uint8Array(await rendered.arrayBuffer()));
-				res.end();
+				let request;
+
+				try {
+					req.url = initial_url;
+					request = await getRequest(`${protocol}://${host}`, req);
+				} catch (/** @type {any} */ err) {
+					res.statusCode = err.status || 400;
+					return res.end(err.reason || 'Invalid request body');
+				}
+
+				setResponse(res, await app.render(request));
 			} else {
 				res.statusCode = 404;
 				res.end('Not found');

package/dist/chunks/index7.js

@@ -1,7 +1,7 @@
 import * as fs from 'fs';
 import * as path from 'path';
 import { createRequire } from 'module';
-import { h as rimraf, w as walk$1, $, m as mkdirp } from '../cli.js';
+import { h as rimraf, m as mkdirp, w as walk$1, $ } from '../cli.js';
 import 'sade';
 import 'child_process';
 import 'net';
@@ -15257,6 +15257,7 @@ async function make_package(config, cwd = process.cwd()) {
 	const package_dir = config.kit.package.dir;
 	const abs_package_dir = path.isAbsolute(package_dir) ? package_dir : path.join(cwd, package_dir);
 	rimraf(abs_package_dir);
+	mkdirp(abs_package_dir); // TODO https://github.com/sveltejs/kit/issues/2333
 
 	if (config.kit.package.emitTypes) {
 		// Generate type definitions first so hand-written types can overwrite generated ones
@@ -15304,25 +15305,28 @@ async function make_package(config, cwd = process.cwd()) {
 		}
 
 		const filename = path.join(config.kit.files.lib, file);
-		const source = fs.readFileSync(filename, 'utf8');
+		const source = fs.readFileSync(filename);
 
 		/** @type {string} */
 		let out_file;
 
-		/** @type {string} */
+		/** @type {string | Buffer} */
 		let out_contents;
 
 		if (svelte_ext) {
 			// it's a Svelte component
+			contains_svelte_files = true;
 			out_file = file.slice(0, -svelte_ext.length) + '.svelte';
+			out_contents = source.toString('utf-8');
 			out_contents = config.preprocess
-				? strip_lang_tags((await preprocess(source, config.preprocess, { filename })).code)
-				: source;
-			contains_svelte_files = true;
+				? strip_lang_tags((await preprocess(out_contents, config.preprocess, { filename })).code)
+				: out_contents;
+			out_contents = resolve_$lib_alias(out_file, out_contents, config);
 		} else if (ext === '.ts' && file.endsWith('.d.ts')) {
 			// TypeScript's declaration emit won't copy over the d.ts files, so we do it here
 			out_file = file;
-			out_contents = source;
+			out_contents = source.toString('utf-8');
+			out_contents = resolve_$lib_alias(out_file, out_contents, config);
 			if (fs.existsSync(path.join(abs_package_dir, out_file))) {
 				console.warn(
 					'Found already existing file from d.ts generation for ' +
@@ -15332,12 +15336,12 @@ async function make_package(config, cwd = process.cwd()) {
 			}
 		} else if (ext === '.ts') {
 			out_file = file.slice(0, -'.ts'.length) + '.js';
-			out_contents = await transpile_ts(filename, source);
+			out_contents = await transpile_ts(filename, source.toString('utf-8'));
+			out_contents = resolve_$lib_alias(out_file, out_contents, config);
 		} else {
 			out_file = file;
 			out_contents = source;
 		}
-		out_contents = resolve_$lib_alias(out_file, out_contents, config);
 
 		write(path.join(abs_package_dir, out_file), out_contents);
 
@@ -15529,7 +15533,7 @@ function load_tsconfig(filename, ts) {
 
 /**
  * @param {string} file
- * @param {string} contents
+ * @param {Parameters<typeof fs.writeFileSync>[1]} contents
  */
 function write(file, contents) {
 	mkdirp(path.dirname(file));

package/dist/cli.js

@@ -933,7 +933,7 @@ async function launch(port, https) {
 	exec(`${cmd} ${https ? 'https' : 'http'}://localhost:${port}`);
 }
 
-const prog = sade('svelte-kit').version('1.0.0-next.239');
+const prog = sade('svelte-kit').version('1.0.0-next.242');
 
 prog
 	.command('dev')
@@ -1085,7 +1085,7 @@ async function check_port(port) {
 function welcome({ port, host, https, open, loose, allow, cwd }) {
 	if (open) launch(port, https);
 
-	console.log($.bold().cyan(`\n  SvelteKit v${'1.0.0-next.239'}\n`));
+	console.log($.bold().cyan(`\n  SvelteKit v${'1.0.0-next.242'}\n`));
 
 	const protocol = https ? 'https:' : 'http:';
 	const exposed = typeof host !== 'undefined' && host !== 'localhost' && host !== '127.0.0.1';

package/dist/node.js

@@ -1,7 +1,7 @@
 import { Readable } from 'stream';
 
 /** @type {import('@sveltejs/kit/node').GetRawBody} */
-function getRawBody(req) {
+function get_raw_body(req) {
 	return new Promise((fulfil, reject) => {
 		const h = req.headers;
 
@@ -55,13 +55,21 @@ async function getRequest(base, req) {
 	return new Request(base + req.url, {
 		method: req.method,
 		headers: /** @type {Record<string, string>} */ (req.headers),
-		body: await getRawBody(req)
+		body: await get_raw_body(req) // TODO stream rather than buffer
 	});
 }
 
 /** @type {import('@sveltejs/kit/node').SetResponse} */
 async function setResponse(res, response) {
-	res.writeHead(response.status, Object.fromEntries(response.headers));
+	/** @type {import('../types/helper').ResponseHeaders} */
+	const headers = Object.fromEntries(response.headers);
+
+	if (response.headers.has('set-cookie')) {
+		// @ts-expect-error (headers.raw() is non-standard)
+		headers['set-cookie'] = response.headers.raw()['set-cookie'];
+	}
+
+	res.writeHead(response.status, headers);
 
 	if (response.body instanceof Readable) {
 		response.body.pipe(res);
@@ -74,4 +82,4 @@ async function setResponse(res, response) {
 	}
 }
 
-export { getRawBody, getRequest, setResponse };
+export { getRequest, setResponse };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sveltejs/kit",
-  "version": "1.0.0-next.239",
+  "version": "1.0.0-next.242",
   "repository": {
     "type": "git",
     "url": "https://github.com/sveltejs/kit",

package/types/endpoint.d.ts

@@ -3,16 +3,18 @@ import { Either, JSONString, MaybePromise, ResponseHeaders } from './helper';
 
 type Body = JSONString | Uint8Array | ReadableStream | import('stream').Readable;
 
-export interface EndpointOutput {
+export interface EndpointOutput<Output extends Body = Body> {
 	status?: number;
 	headers?: Headers | Partial<ResponseHeaders>;
-	body?: Body;
+	body?: Output;
 }
 
 export interface Fallthrough {
 	fallthrough: true;
 }
 
-export interface RequestHandler<Locals = Record<string, any>> {
-	(event: RequestEvent<Locals>): MaybePromise<Either<Response | EndpointOutput, Fallthrough>>;
+export interface RequestHandler<Locals = Record<string, any>, Output extends Body = Body> {
+	(event: RequestEvent<Locals>): MaybePromise<
+		Either<Response | EndpointOutput<Output>, Fallthrough>
+	>;
 }