@sveltejs/kit 1.0.0-next.587 → 1.0.0-next.589

package/package.json

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

package/src/cli.js

@@ -27,20 +27,6 @@ prog
 	.describe('Synchronise generated files')
 	.option('--mode', 'Specify a mode for loading environment variables', 'development')
 	.action(async ({ mode }) => {
-		const event = process.env.npm_lifecycle_event;
-
-		// TODO remove for 1.0
-		if (event === 'prepare') {
-			const pkg = JSON.parse(fs.readFileSync('package.json', 'utf8'));
-			const message =
-				pkg.scripts.prepare === 'svelte-kit sync'
-					? `\`svelte-kit sync\` now runs on "postinstall" — please remove the "prepare" script from your package.json\n`
-					: `\`svelte-kit sync\` now runs on "postinstall" — please remove it from your "prepare" script\n`;
-
-			console.error(colors.bold().red(message));
-			return;
-		}
-
 		if (!fs.existsSync('svelte.config.js')) {
 			console.warn(`Missing ${path.resolve('svelte.config.js')} — skipping`);
 			return;
@@ -55,54 +41,4 @@ prog
 		}
 	});
 
-// TODO remove for 1.0
-replace('dev');
-replace('build');
-replace('preview');
-prog
-	.command('package')
-	.describe('No longer available - use @sveltejs/package instead')
-	.action(() => {
-		console.error(
-			'svelte-kit package has been removed. It now lives in its own npm package. See the PR on how to migrate: https://github.com/sveltejs/kit/pull/5730'
-		);
-	});
-
 prog.parse(process.argv, { unknown: (arg) => `Unknown option: ${arg}` });
-
-/** @param {string} command */
-function replace(command) {
-	prog
-		.command(command)
-		.describe(`No longer available — use vite ${command} instead`)
-		.action(async () => {
-			const message = `\n> svelte-kit ${command} is no longer available — use vite ${command} instead`;
-			console.error(colors.bold().red(message));
-
-			const steps = [
-				'Install vite as a devDependency with npm/pnpm/etc',
-				'Create a vite.config.js with the @sveltejs/kit/vite plugin (see below)',
-				`Update your package.json scripts to reference \`vite ${command}\` instead of \`svelte-kit ${command}\``
-			];
-
-			steps.forEach((step, i) => {
-				console.error(`  ${i + 1}. ${colors.cyan(step)}`);
-			});
-
-			console.error(
-				`
-				${colors.grey('// vite.config.js')}
-				import { sveltekit } from '@sveltejs/kit/vite';
-
-				/** @type {import('vite').UserConfig} */
-				const config = {
-					plugins: [sveltekit()]
-				};
-
-				export default config;
-
-				`.replace(/^\t{4}/gm, '')
-			);
-			process.exit(1);
-		});
-}

package/src/core/adapt/builder.js

@@ -33,6 +33,23 @@ export function create_builder({ config, build_data, routes, prerendered, log })
 		config,
 		prerendered,
 
+		async compress(directory) {
+			if (!existsSync(directory)) {
+				return;
+			}
+
+			const files = await glob('**/*.{html,js,json,css,svg,xml,wasm}', {
+				cwd: directory,
+				dot: true,
+				absolute: true,
+				filesOnly: true
+			});
+
+			await Promise.all(
+				files.map((file) => Promise.all([compress_file(file, 'gz'), compress_file(file, 'br')]))
+			);
+		},
+
 		async createEntries(fn) {
 			/** @type {import('types').RouteDefinition[]} */
 			const facades = routes.map((route) => {
@@ -177,39 +194,6 @@ export function create_builder({ config, build_data, routes, prerendered, log })
 
 		writeServer(dest) {
 			return copy(`${config.kit.outDir}/output/server`, dest);
-		},
-
-		// TODO remove these methods for 1.0
-		// @ts-expect-error
-		writeStatic() {
-			throw new Error(
-				`writeStatic has been removed. Please ensure you are using the latest version of ${
-					config.kit.adapter.name || 'your adapter'
-				}`
-			);
-		},
-
-		async compress(directory) {
-			if (!existsSync(directory)) {
-				return;
-			}
-
-			const files = await glob('**/*.{html,js,json,css,svg,xml,wasm}', {
-				cwd: directory,
-				dot: true,
-				absolute: true,
-				filesOnly: true
-			});
-
-			await Promise.all(
-				files.map((file) => Promise.all([compress_file(file, 'gz'), compress_file(file, 'br')]))
-			);
-		},
-
-		async prerender() {
-			throw new Error(
-				'builder.prerender() has been removed. Prerendering now takes place in the build phase — see builder.prerender and builder.writePrerendered'
-			);
 		}
 	};
 }

package/src/core/config/index.js

@@ -16,14 +16,6 @@ export function load_template(cwd, config) {
 	if (fs.existsSync(appTemplate)) {
 		const contents = fs.readFileSync(appTemplate, 'utf8');
 
-		// TODO remove this for 1.0
-		const match = /%svelte\.([a-z]+)%/.exec(contents);
-		if (match) {
-			throw new Error(
-				`%svelte.${match[1]}% in ${relative} should be replaced with %sveltekit.${match[1]}%`
-			);
-		}
-
 		const expected_tags = ['%sveltekit.head%', '%sveltekit.body%'];
 		expected_tags.forEach((tag) => {
 			if (contents.indexOf(tag) === -1) {
@@ -81,9 +73,6 @@ function process_config(config, { cwd = process.cwd() } = {}) {
 	validated.kit.outDir = path.resolve(cwd, validated.kit.outDir);
 
 	for (const key in validated.kit.files) {
-		// TODO remove for 1.0
-		if (key === 'template') continue;
-
 		if (key === 'hooks') {
 			validated.kit.files.hooks.client = path.resolve(cwd, validated.kit.files.hooks.client);
 			validated.kit.files.hooks.server = path.resolve(cwd, validated.kit.files.hooks.server);

package/src/core/config/options.js

@@ -85,12 +85,6 @@ const options = object(
 				return input;
 			}),
 
-			// TODO: remove this for the 1.0 release
-			amp: error(
-				(keypath) =>
-					`${keypath} has been removed. See https://kit.svelte.dev/docs/seo#manual-setup-amp for details on how to support AMP`
-			),
-
 			appDir: validate('_app', (input, keypath) => {
 				assert_string(input, keypath);
 
@@ -107,18 +101,6 @@ const options = object(
 				return input;
 			}),
 
-			// TODO: remove this for the 1.0 release
-			browser: object({
-				hydrate: error(
-					(keypath) =>
-						`${keypath} has been removed. You can set \`export const csr = false\` inside the top level +layout.js instead. See the PR for more information: https://github.com/sveltejs/kit/pull/6197`
-				),
-				router: error(
-					(keypath) =>
-						`${keypath} has been removed. You can set \`export const csr = false\` inside the top level +layout.js instead. See the PR for more information: https://github.com/sveltejs/kit/pull/6197`
-				)
-			}),
-
 			csp: object({
 				mode: list(['auto', 'hash', 'nonce']),
 				directives,
@@ -131,11 +113,6 @@ const options = object(
 
 			embedded: boolean(false),
 
-			// TODO: remove this for the 1.0 release
-			endpointExtensions: error(
-				(keypath) => `${keypath} has been renamed to config.kit.moduleExtensions`
-			),
-
 			env: object({
 				dir: string(process.cwd()),
 				publicPrefix: string('PUBLIC_')
@@ -143,59 +120,24 @@ const options = object(
 
 			files: object({
 				assets: string('static'),
-				hooks: (input, keypath) => {
-					// TODO remove this for the 1.0 release
-					if (typeof input === 'string') {
-						throw new Error(
-							`${keypath} is an object with { server: string, client: string } now. See the PR for more information: https://github.com/sveltejs/kit/pull/6586`
-						);
-					}
-
-					return object({
-						client: string(join('src', 'hooks.client')),
-						server: string(join('src', 'hooks.server'))
-					})(input, keypath);
-				},
+				hooks: object({
+					client: string(join('src', 'hooks.client')),
+					server: string(join('src', 'hooks.server'))
+				}),
 				lib: string(join('src', 'lib')),
 				params: string(join('src', 'params')),
 				routes: string(join('src', 'routes')),
 				serviceWorker: string(join('src', 'service-worker')),
 				appTemplate: string(join('src', 'app.html')),
-				errorTemplate: string(join('src', 'error.html')),
-				// TODO: remove this for the 1.0 release
-				template: error(
-					() => 'config.kit.files.template has been renamed to config.kit.files.appTemplate'
-				)
+				errorTemplate: string(join('src', 'error.html'))
 			}),
 
-			// TODO: remove this for the 1.0 release
-			headers: error(
-				(keypath) =>
-					`${keypath} has been removed. See https://github.com/sveltejs/kit/pull/3384 for details`
-			),
-
-			// TODO: remove this for the 1.0 release
-			host: error(
-				(keypath) =>
-					`${keypath} has been removed. See https://github.com/sveltejs/kit/pull/3384 for details`
-			),
-
-			// TODO remove for 1.0
-			hydrate: error((keypath) => `${keypath} has been moved to config.kit.browser.hydrate`),
-
 			inlineStyleThreshold: number(0),
 
-			methodOverride: error(
-				() =>
-					'Method overrides have been removed in favor of actions. See the PR for more information: https://github.com/sveltejs/kit/pull/6469'
-			),
-
 			moduleExtensions: string_array(['.js', '.ts']),
 
 			outDir: string('.svelte-kit'),
 
-			package: error((keypath) => `${keypath} has been removed — use @sveltejs/package instead`),
-
 			paths: object({
 				base: validate('', (input, keypath) => {
 					assert_string(input, keypath);
@@ -232,18 +174,6 @@ const options = object(
 			prerender: object({
 				concurrency: number(1),
 				crawl: boolean(true),
-				createIndexFiles: error(
-					(keypath) =>
-						`${keypath} has been removed — it is now controlled by the trailingSlash option. See https://kit.svelte.dev/docs/page-options#trailingslash`
-				),
-				default: error(
-					(keypath) =>
-						`${keypath} has been removed. You can set it inside the top level +layout.js instead. See the PR for more information: https://github.com/sveltejs/kit/pull/6197`
-				),
-				enabled: error(
-					(keypath) =>
-						`${keypath} has been removed. You can wrap any code that should not be executed during build in a \`if (!building) {...}\` block. See the discussion for more information: https://github.com/sveltejs/kit/discussions/7716`
-				),
 				entries: validate(['*'], (input, keypath) => {
 					if (!Array.isArray(input) || !input.every((page) => typeof page === 'string')) {
 						throw new Error(`${keypath} must be an array of strings`);
@@ -260,17 +190,6 @@ const options = object(
 					return input;
 				}),
 
-				// TODO: remove this for the 1.0 release
-				force: validate(undefined, (input, keypath) => {
-					const new_input = input ? 'warn' : 'fail';
-					const needs_option = new_input === 'warn';
-					throw new Error(
-						`${keypath} has been removed in favor of \`handleHttpError\`. In your case, set \`handleHttpError\` to "${new_input}"${
-							needs_option ? '' : ' (or leave it undefined)'
-						} to get the same behavior as you would with \`force: ${JSON.stringify(input)}\``
-					);
-				}),
-
 				handleHttpError: validate('fail', (input, keypath) => {
 					if (typeof input === 'function') return input;
 					if (['fail', 'warn', 'ignore'].includes(input)) return input;
@@ -283,17 +202,6 @@ const options = object(
 					throw new Error(`${keypath} should be "fail", "warn", "ignore" or a custom function`);
 				}),
 
-				// TODO: remove this for the 1.0 release
-				onError: validate(undefined, (input, keypath) => {
-					let message = `${keypath} has been renamed to \`handleHttpError\``;
-
-					if (input === 'continue') {
-						message += ', and "continue" has been renamed to "warn"';
-					}
-
-					throw new Error(message);
-				}),
-
 				origin: validate('http://sveltekit-prerender', (input, keypath) => {
 					assert_string(input, keypath);
 
@@ -310,56 +218,18 @@ const options = object(
 					}
 
 					return origin;
-				}),
-
-				// TODO: remove this for the 1.0 release
-				pages: error((keypath) => `${keypath} has been renamed to \`entries\`.`)
+				})
 			}),
 
-			// TODO: remove this for the 1.0 release
-			protocol: error(
-				(keypath) =>
-					`${keypath} has been removed. See https://github.com/sveltejs/kit/pull/3384 for details`
-			),
-
-			// TODO remove for 1.0
-			router: error(
-				(keypath) =>
-					`${keypath} has been removed. You can set \`export const csr = false\` inside the top level +layout.js (or +layout.server.js) instead. See the PR for more information: https://github.com/sveltejs/kit/pull/6197`
-			),
-
-			// TODO remove for 1.0
-			routes: error(
-				(keypath) =>
-					`${keypath} has been removed. See https://github.com/sveltejs/kit/discussions/5774 for details`
-			),
-
 			serviceWorker: object({
 				register: boolean(true),
 				files: fun((filename) => !/\.DS_Store/.test(filename))
 			}),
 
-			// TODO remove this for 1.0
-			ssr: error(
-				(keypath) =>
-					`${keypath} has been removed — use the handle hook instead: https://kit.svelte.dev/docs/hooks#server-hooks-handle`
-			),
-
-			// TODO remove this for 1.0
-			target: error((keypath) => `${keypath} is no longer required, and should be removed`),
-
-			trailingSlash: error(
-				(keypath, input) =>
-					`${keypath} has been removed. You can set \`export const trailingSlash = '${input}'\` inside a top level +layout.js (or +layout.server.js) instead. See the PR for more information: https://github.com/sveltejs/kit/pull/7719`
-			),
-
 			version: object({
 				name: string(Date.now().toString()),
 				pollInterval: number(0)
-			}),
-
-			// TODO remove this for 1.0
-			vite: error((keypath) => `${keypath} has been removed — use vite.config.js instead`)
+			})
 		})
 	},
 	true
@@ -514,11 +384,4 @@ function assert_string(input, keypath) {
 	}
 }
 
-/** @param {(keypath?: string, input?: any) => string} fn */
-function error(fn) {
-	return validate(undefined, (input, keypath) => {
-		throw new Error(fn(keypath, input));
-	});
-}
-
 export default options;

package/src/core/prerender/prerender.js

@@ -91,19 +91,6 @@ export async function prerender() {
 
 	installPolyfills();
 
-	// TODO remove this for 1.0
-	const { fetch } = globalThis;
-	globalThis.fetch = async (info, init) => {
-		const url = info instanceof Request ? info.url : info.toString();
-
-		if (url.startsWith(config.prerender.origin + '/')) {
-			const sliced = url.slice(config.prerender.origin.length);
-			throw new Error(`Use \`event.fetch('${sliced}')\` instead of the global \`fetch('${url}')\``);
-		}
-
-		return fetch(info, init);
-	};
-
 	const server_root = join(config.outDir, 'output');
 
 	/** @type {import('types').ServerModule} */

package/src/core/sync/create_manifest_data/index.js

@@ -266,9 +266,7 @@ function create_routes_and_nodes(cwd, config, fallback) {
 			const root = routes[0];
 			if (!root.leaf && !root.error && !root.layout && !root.endpoint) {
 				throw new Error(
-					// TODO adjust this error message for 1.0
-					// 'No routes found. If you are using a custom src/routes directory, make sure it is specified in svelte.config.js'
-					'The filesystem router API has changed, see https://github.com/sveltejs/kit/discussions/5774 for details'
+					'No routes found. If you are using a custom src/routes directory, make sure it is specified in svelte.config.js'
 				);
 			}
 		}
@@ -386,13 +384,6 @@ function analyze(project_relative, file, component_extensions, module_extensions
 		const pattern = /^\+(?:(page(?:@(.*))?)|(layout(?:@(.*))?)|(error))$/;
 		const match = pattern.exec(name);
 		if (!match) {
-			// TODO remove for 1.0
-			if (/^\+layout-/.test(name)) {
-				throw new Error(
-					`${project_relative} should be reimplemented with layout groups: https://kit.svelte.dev/docs/advanced-routing#advanced-layouts`
-				);
-			}
-
 			throw new Error(`Files prefixed with + are reserved (saw ${project_relative})`);
 		}
 

package/src/core/sync/sync.js

@@ -1,4 +1,3 @@
-import fs from 'fs';
 import path from 'path';
 import create_manifest_data from './create_manifest_data/index.js';
 import { write_client_manifest } from './write_client_manifest.js';
@@ -14,24 +13,6 @@ import { write_ambient } from './write_ambient.js';
  * @param {string} mode
  */
 export function init(config, mode) {
-	// TODO remove for 1.0
-	if (fs.existsSync('src/app.d.ts')) {
-		const content = fs.readFileSync('src/app.d.ts', 'utf-8');
-		if (content.includes('PageError')) {
-			if (content.includes('// interface PageError')) {
-				fs.writeFileSync(
-					'src/app.d.ts',
-					content.replace(/\/\/ interface PageError/g, '// interface Error')
-				);
-				console.warn('App.PageError has been renamed to App.Error — we updated your src/app.d.ts');
-			} else {
-				throw new Error(
-					'App.PageError has been renamed to App.Error — please update your src/app.d.ts'
-				);
-			}
-		}
-	}
-
 	write_tsconfig(config.kit);
 	write_ambient(config.kit, mode);
 }

package/src/core/sync/write_types/index.js

@@ -49,7 +49,7 @@ export async function write_all_types(config, manifest_data) {
 		for (const file of walk(types_dir)) {
 			const dir = path.dirname(file);
 			if (!expected_directories.has(dir)) {
-				rimraf(file);
+				rimraf(path.join(types_dir, file));
 			}
 		}
 	}

package/src/exports/vite/build/build_server.js

@@ -68,16 +68,9 @@ export class Server {
 			dev: false,
 			embedded: ${config.kit.embedded},
 			handle_error: (error, event) => {
-				return this.options.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');
-					}
-				}) ?? { message: event.route.id != null ? 'Internal Error' : 'Not Found' };
+				return this.options.hooks.handleError({ error, event }) ?? {
+					message: event.route.id != null ? 'Internal Error' : 'Not Found'
+				};
 			},
 			hooks: null,
 			manifest,
@@ -117,11 +110,6 @@ export class Server {
 		if (!this.options.hooks) {
 			const module = await import(${s(hooks)});
 
-			// TODO remove this for 1.0
-			if (module.externalFetch) {
-				throw new Error('externalFetch has been removed — use handleFetch instead. See https://github.com/sveltejs/kit/pull/6565 for details');
-			}
-
 			this.options.hooks = {
 				handle: module.handle || (({ event, resolve }) => resolve(event)),
 				handleError: module.handleError || (({ error }) => console.error(error.stack)),
@@ -167,22 +155,6 @@ export async function build_server(options, client) {
 
 	let hooks_file = resolve_entry(config.kit.files.hooks.server);
 
-	// TODO remove for 1.0
-	if (!hooks_file) {
-		const old_file = resolve_entry(path.join(process.cwd(), 'src', 'hooks'));
-		if (old_file && fs.existsSync(old_file)) {
-			throw new Error(
-				`Rename your server hook file from ${posixify(
-					path.relative(process.cwd(), old_file)
-				)} to ${posixify(
-					path.relative(process.cwd(), config.kit.files.hooks.server)
-				)}${path.extname(
-					old_file
-				)} (because there's also client hooks now). See the PR for more information: https://github.com/sveltejs/kit/pull/6586`
-			);
-		}
-	}
-
 	if (!hooks_file || !fs.existsSync(hooks_file)) {
 		hooks_file = path.join(config.kit.outDir, 'build/hooks.js');
 		fs.writeFileSync(hooks_file, '');

package/src/exports/vite/build/build_service_worker.js

@@ -34,13 +34,6 @@ export async function build_service_worker(
 	fs.writeFileSync(
 		service_worker,
 		`
-			// TODO remove for 1.0
-			export const timestamp = {
-				toString: () => {
-					throw new Error('\`timestamp\` has been removed from $service-worker. Use \`version\` instead');
-				}
-			};
-
 			export const build = [
 				${Array.from(build)
 					.map((file) => `${s(`${config.kit.paths.base}/${file}`)}`)

package/src/exports/vite/dev/index.js

@@ -377,32 +377,8 @@ export async function dev(vite, vite_config, svelte_config) {
 					? await vite.ssrLoadModule(`/${hooks_file}`)
 					: {};
 
-				// TODO remove for 1.0
-				if (!resolve_entry(hooks_file)) {
-					const old_file = resolve_entry(path.join(process.cwd(), 'src', 'hooks'));
-					if (old_file && fs.existsSync(old_file)) {
-						throw new Error(
-							`Rename your server hook file from ${posixify(
-								path.relative(process.cwd(), old_file)
-							)} to ${posixify(
-								path.relative(process.cwd(), svelte_config.kit.files.hooks.server)
-							)}${path.extname(
-								old_file
-							)} (because there's also client hooks now). See the PR for more information: https://github.com/sveltejs/kit/pull/6586`
-						);
-					}
-				}
-
 				const handle = user_hooks.handle || (({ event, resolve }) => resolve(event));
 
-				// TODO remove for 1.0
-				// @ts-expect-error
-				if (user_hooks.externalFetch) {
-					throw new Error(
-						'externalFetch has been removed — use handleFetch instead. See https://github.com/sveltejs/kit/pull/6565 for details'
-					);
-				}
-
 				/** @type {import('types').ServerHooks} */
 				const hooks = {
 					handle,
@@ -421,18 +397,6 @@ export async function dev(vite, vite_config, svelte_config) {
 					handleFetch: user_hooks.handleFetch || (({ request, fetch }) => fetch(request))
 				};
 
-				if (/** @type {any} */ (hooks).getContext) {
-					// TODO remove this for 1.0
-					throw new Error(
-						'The getContext hook has been removed. See https://kit.svelte.dev/docs/hooks'
-					);
-				}
-
-				if (/** @type {any} */ (hooks).serverFetch) {
-					// TODO remove this for 1.0
-					throw new Error('The serverFetch hook has been renamed to externalFetch.');
-				}
-
 				// TODO the / prefix will probably fail if outDir is outside the cwd (which
 				// could be the case in a monorepo setup), but without it these modules
 				// can get loaded twice via different URLs, which causes failures. Might
@@ -497,15 +461,7 @@ export async function dev(vite, vite_config, svelte_config) {
 										return Reflect.get(target, property, target);
 									}
 								}),
-								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'
-									);
-								}
+								event
 							});
 							return (
 								error_object ?? { message: event.route.id != null ? 'Internal Error' : 'Not Found' }

package/src/exports/vite/index.js

@@ -199,9 +199,6 @@ function kit({ svelte_config }) {
 		};
 	}
 
-	// TODO remove this for 1.0
-	check_vite_version();
-
 	/** @type {import('vite').Plugin} */
 	const plugin_setup = {
 		name: 'vite-plugin-sveltekit-setup',
@@ -631,19 +628,6 @@ function kit({ svelte_config }) {
 	return [plugin_setup, plugin_compile];
 }
 
-function check_vite_version() {
-	// TODO parse from kit peer deps and maybe do a full semver compare if we ever require feature releases a min
-	const min_required_vite_major = 4;
-	const vite_version = vite.version ?? '2.x'; // vite started exporting it's version in 3.0
-	const current_vite_major = parseInt(vite_version.split('.')[0], 10);
-
-	if (current_vite_major < min_required_vite_major) {
-		throw new Error(
-			`Vite version ${current_vite_major} is no longer supported. Please upgrade to version ${min_required_vite_major}`
-		);
-	}
-}
-
 /** @param {import('rollup').OutputBundle} bundle */
 function collect_output(bundle) {
 	/** @type {import('rollup').OutputChunk[]} */

package/src/runtime/app/forms.js

@@ -119,17 +119,7 @@ export function enhance(form, submit = () => {}) {
 			form,
 			update: (opts) => fallback_callback({ action, result, reset: opts?.reset }),
 			// @ts-expect-error generic constraints stuff we don't care about
-			result,
-			// TODO remove for 1.0
-			get type() {
-				throw new Error('(result) => {...} has changed to ({ result }) => {...}');
-			},
-			get location() {
-				throw new Error('(result) => {...} has changed to ({ result }) => {...}');
-			},
-			get error() {
-				throw new Error('(result) => {...} has changed to ({ result }) => {...}');
-			}
+			result
 		});
 	}
 

package/src/runtime/app/navigation.js

@@ -20,13 +20,3 @@ export const preloadData = BROWSER ? client.preload_data : guard('preloadData');
 export const preloadCode = BROWSER ? client.preload_code : guard('preloadCode');
 export const beforeNavigate = BROWSER ? client.before_navigate : () => {};
 export const afterNavigate = BROWSER ? client.after_navigate : () => {};
-
-// TODO remove for 1.0 release
-/** @param {any} _args */
-export const prefetch = (..._args) => {
-	throw new Error('prefetch has been renamed to preloadData');
-};
-/** @param {any} _args */
-export const prefetchRoutes = (..._args) => {
-	throw new Error('prefetchRoutes has been renamed to preloadCode');
-};

package/src/runtime/app/stores.js

@@ -2,23 +2,13 @@ import { getContext } from 'svelte';
 import { browser } from './environment.js';
 import { stores as browser_stores } from '../client/singletons.js';
 
-// TODO remove this (for 1.0? after 1.0?)
-let warned = false;
-export function stores() {
-	if (!warned) {
-		console.error('stores() is deprecated; use getStores() instead');
-		warned = true;
-	}
-	return getStores();
-}
-
 /**
  * @type {import('$app/stores').getStores}
  */
 export const getStores = () => {
 	const stores = browser ? browser_stores : getContext('__svelte__');
 
-	const readonly_stores = {
+	return {
 		page: {
 			subscribe: stores.page.subscribe
 		},
@@ -27,28 +17,6 @@ export const getStores = () => {
 		},
 		updated: stores.updated
 	};
-
-	// TODO remove this for 1.0
-	Object.defineProperties(readonly_stores, {
-		preloading: {
-			get() {
-				console.error('stores.preloading is deprecated; use stores.navigating instead');
-				return {
-					subscribe: stores.navigating.subscribe
-				};
-			},
-			enumerable: false
-		},
-		session: {
-			get() {
-				removed_session();
-				return {};
-			},
-			enumerable: false
-		}
-	});
-
-	return readonly_stores;
 };
 
 /** @type {typeof import('$app/stores').page} */
@@ -68,19 +36,6 @@ export const navigating = {
 	}
 };
 
-function removed_session() {
-	// TODO remove for 1.0
-	throw new Error(
-		'stores.session is no longer available. See https://github.com/sveltejs/kit/discussions/5883'
-	);
-}
-
-export const session = {
-	subscribe: removed_session,
-	set: removed_session,
-	update: removed_session
-};
-
 /** @type {typeof import('$app/stores').updated} */
 export const updated = {
 	subscribe(fn) {

package/src/runtime/client/client.js

@@ -62,26 +62,6 @@ function update_scroll_positions(index) {
 	scroll_positions[index] = scroll_state();
 }
 
-// TODO remove for 1.0
-/** @type {Record<string, true>} */
-let warned_about_attributes = {};
-
-function check_for_removed_attributes() {
-	const attrs = ['prefetch', 'noscroll', 'reload'];
-	for (const attr of attrs) {
-		if (document.querySelector(`[sveltekit\\:${attr}]`)) {
-			if (!warned_about_attributes[attr]) {
-				warned_about_attributes[attr] = true;
-				console.error(
-					`The sveltekit:${attr} attribute has been replaced with data-sveltekit-${
-						attr === 'prefetch' ? 'preload-data' : attr
-					}`
-				);
-			}
-		}
-	}
-}
-
 /**
  * @param {{
  *   target: HTMLElement;
@@ -337,9 +317,7 @@ export function create_client({ target, base }) {
 				navigation_result.props.page.url = url;
 			}
 
-			const post_update = pre_update();
 			root.$set(navigation_result.props);
-			post_update();
 		} else {
 			initialize(navigation_result);
 		}
@@ -393,22 +371,20 @@ export function create_client({ target, base }) {
 
 		page = result.props.page;
 
-		const post_update = pre_update();
 		root = new Root({
 			target,
 			props: { ...result.props, stores },
 			hydrate: true
 		});
-		post_update();
 
 		/** @type {import('types').AfterNavigate} */
 		const navigation = {
 			from: null,
-			to: add_url_properties('to', {
+			to: {
 				params: current.params,
 				route: { id: current.route?.id ?? null },
 				url: new URL(location.href)
-			}),
+			},
 			willUnload: false,
 			type: 'enter'
 		};
@@ -504,29 +480,6 @@ export function create_client({ target, base }) {
 				// The whole page store is updated, but this way the object reference stays the same
 				data: data_changed ? data : page.data
 			};
-
-			// TODO remove this for 1.0
-			Object.defineProperty(result.props.page, 'routeId', {
-				get() {
-					throw new Error('$page.routeId has been replaced by $page.route.id');
-				},
-				enumerable: false
-			});
-			/**
-			 * @param {string} property
-			 * @param {string} replacement
-			 */
-			const print_error = (property, replacement) => {
-				Object.defineProperty(result.props.page, property, {
-					get: () => {
-						throw new Error(`$page.${property} has been replaced by $page.url.${replacement}`);
-					}
-				});
-			};
-
-			print_error('origin', 'origin');
-			print_error('path', 'pathname');
-			print_error('query', 'searchParams');
 		}
 
 		return result;
@@ -642,40 +595,6 @@ export function create_client({ target, base }) {
 				}
 			};
 
-			// TODO remove this for 1.0
-			Object.defineProperties(load_input, {
-				props: {
-					get() {
-						throw new Error(
-							'@migration task: Replace `props` with `data` stuff https://github.com/sveltejs/kit/discussions/5774#discussioncomment-3292693'
-						);
-					},
-					enumerable: false
-				},
-				session: {
-					get() {
-						throw new Error(
-							'session is no longer available. See https://github.com/sveltejs/kit/discussions/5883'
-						);
-					},
-					enumerable: false
-				},
-				stuff: {
-					get() {
-						throw new Error(
-							'@migration task: Remove stuff https://github.com/sveltejs/kit/discussions/5774#discussioncomment-3292693'
-						);
-					},
-					enumerable: false
-				},
-				routeId: {
-					get() {
-						throw new Error('routeId has been replaced by route.id');
-					},
-					enumerable: false
-				}
-			});
-
 			if (DEV) {
 				try {
 					lock_fetch();
@@ -1072,16 +991,16 @@ export function create_client({ target, base }) {
 
 		/** @type {import('types').Navigation} */
 		const navigation = {
-			from: add_url_properties('from', {
+			from: {
 				params: current.params,
 				route: { id: current.route?.id ?? null },
 				url: current.url
-			}),
-			to: add_url_properties('to', {
+			},
+			to: {
 				params: intent?.params ?? null,
 				route: { id: intent?.route?.id ?? null },
 				url
-			}),
+			},
 			willUnload: !intent,
 			type
 		};
@@ -1324,30 +1243,10 @@ export function create_client({ target, base }) {
 		},
 
 		goto: (href, opts = {}) => {
-			// TODO remove for 1.0
-			if ('keepfocus' in opts && !('keepFocus' in opts)) {
-				throw new Error(
-					'`keepfocus` has been renamed to `keepFocus` (note the difference in casing)'
-				);
-			}
-
-			if ('noscroll' in opts && !('noScroll' in opts)) {
-				throw new Error(
-					'`noscroll` has been renamed to `noScroll` (note the difference in casing)'
-				);
-			}
-
 			return goto(href, opts, []);
 		},
 
 		invalidate: (resource) => {
-			if (resource === undefined) {
-				// TODO remove for 1.0
-				throw new Error(
-					'`invalidate()` (with no arguments) has been replaced by `invalidateAll()`'
-				);
-			}
-
 			if (typeof resource === 'function') {
 				invalidated.push(resource);
 			} else {
@@ -1394,9 +1293,7 @@ export function create_client({ target, base }) {
 
 					current = navigation_result.state;
 
-					const post_update = pre_update();
 					root.$set(navigation_result.props);
-					post_update();
 
 					tick().then(reset_focus);
 				}
@@ -1408,9 +1305,7 @@ export function create_client({ target, base }) {
 					form: result.data,
 					page: { ...page, form: result.data, status: result.status }
 				};
-				const post_update = pre_update();
 				root.$set(props);
-				post_update();
 
 				if (result.type === 'success') {
 					tick().then(reset_focus);
@@ -1433,11 +1328,11 @@ export function create_client({ target, base }) {
 					// it's due to an external or full-page-reload link, for which we don't want to call the hook again.
 					/** @type {import('types').BeforeNavigate} */
 					const navigation = {
-						from: add_url_properties('from', {
+						from: {
 							params: current.params,
 							route: { id: current.route?.id ?? null },
 							url: current.url
-						}),
+						},
 						to: null,
 						willUnload: true,
 						type: 'leave',
@@ -1783,60 +1678,6 @@ function handle_error(error, event) {
 	);
 }
 
-// TODO remove for 1.0
-const properties = [
-	'hash',
-	'href',
-	'host',
-	'hostname',
-	'origin',
-	'pathname',
-	'port',
-	'protocol',
-	'search',
-	'searchParams',
-	'toString',
-	'toJSON'
-];
-
-/**
- * @param {'from' | 'to'} type
- * @param {import('types').NavigationTarget} target
- */
-function add_url_properties(type, target) {
-	for (const prop of properties) {
-		Object.defineProperty(target, prop, {
-			get() {
-				throw new Error(
-					`The navigation shape changed - ${type}.${prop} should now be ${type}.url.${prop}`
-				);
-			},
-			enumerable: false
-		});
-	}
-
-	Object.defineProperty(target, 'routeId', {
-		get() {
-			throw new Error(
-				`The navigation shape changed - ${type}.routeId should now be ${type}.route.id`
-			);
-		},
-		enumerable: false
-	});
-
-	return target;
-}
-
-function pre_update() {
-	if (DEV) {
-		return () => {
-			check_for_removed_attributes();
-		};
-	}
-
-	return () => {};
-}
-
 function reset_focus() {
 	const autofocus = document.querySelector('[autofocus]');
 	if (autofocus) {

package/src/runtime/server/endpoint.js

@@ -1,6 +1,6 @@
 import { negotiate } from '../../utils/http.js';
 import { Redirect } from '../control.js';
-import { check_method_names, method_not_allowed } from './utils.js';
+import { method_not_allowed } from './utils.js';
 
 /**
  * @param {import('types').RequestEvent} event
@@ -11,9 +11,6 @@ import { check_method_names, method_not_allowed } from './utils.js';
 export async function render_endpoint(event, mod, state) {
 	const method = /** @type {import('types').HttpMethod} */ (event.request.method);
 
-	// TODO: Remove for 1.0
-	check_method_names(mod);
-
 	let handler = mod[method];
 
 	if (!handler && method === 'HEAD') {

package/src/runtime/server/index.js

@@ -151,41 +151,6 @@ export async function respond(request, options, state) {
 		isDataRequest: is_data_request
 	};
 
-	// TODO remove this for 1.0
-	/**
-	 * @param {string} property
-	 * @param {string} replacement
-	 * @param {string} suffix
-	 */
-	const removed = (property, replacement, suffix = '') => ({
-		get: () => {
-			throw new Error(`event.${property} has been replaced by event.${replacement}` + suffix);
-		}
-	});
-
-	const details = '. See https://github.com/sveltejs/kit/pull/3384 for details';
-
-	const body_getter = {
-		get: () => {
-			throw new Error(
-				'To access the request body use the text/json/arrayBuffer/formData methods, e.g. `body = await request.json()`' +
-					details
-			);
-		}
-	};
-
-	Object.defineProperties(event, {
-		clientAddress: removed('clientAddress', 'getClientAddress'),
-		method: removed('method', 'request.method', details),
-		headers: removed('headers', 'request.headers', details),
-		origin: removed('origin', 'url.origin'),
-		path: removed('path', 'url.pathname'),
-		query: removed('query', 'url.searchParams'),
-		body: body_getter,
-		rawBody: body_getter,
-		routeId: removed('routeId', 'route.id')
-	});
-
 	/** @type {import('types').RequiredResolveOptions} */
 	let resolve_opts = {
 		transformPageChunk: default_transform,
@@ -279,12 +244,7 @@ export async function respond(request, options, state) {
 					}
 
 					return response;
-				}),
-			// TODO remove for 1.0
-			// @ts-expect-error
-			get request() {
-				throw new Error('request in handle has been replaced with event' + details);
-			}
+				})
 		});
 
 		// respond with 304 if etag matches
@@ -350,13 +310,6 @@ export async function respond(request, options, state) {
 	async function resolve(event, opts) {
 		try {
 			if (opts) {
-				// TODO remove for 1.0
-				if ('transformPage' in opts) {
-					throw new Error(
-						'transformPage has been replaced by transformPageChunk — see https://github.com/sveltejs/kit/pull/5657 for more information'
-					);
-				}
-
 				if ('ssr' in opts) {
 					throw new Error(
 						'ssr has been removed, set it in the appropriate +layout.js instead. See the PR for more information: https://github.com/sveltejs/kit/pull/6197'

package/src/runtime/server/page/load_data.js

@@ -102,8 +102,7 @@ export async function load_data({
 		return server_data_node?.data ?? null;
 	}
 
-	/** @type {import('types').LoadEvent} */
-	const load_event = {
+	const result = await node.universal.load.call(null, {
 		url: event.url,
 		params: event.params,
 		data: server_data_node?.data ?? null,
@@ -226,21 +225,8 @@ export async function load_data({
 		setHeaders: event.setHeaders,
 		depends: () => {},
 		parent
-	};
-
-	// TODO remove this for 1.0
-	Object.defineProperties(load_event, {
-		session: {
-			get() {
-				throw new Error(
-					'session is no longer available. See https://github.com/sveltejs/kit/discussions/5883'
-				);
-			},
-			enumerable: false
-		}
 	});
 
-	const result = await node.universal.load.call(null, load_event);
 	const data = result ? await unwrap_promises(result) : null;
 	validate_load_response(data, /** @type {string} */ (event.route.id));
 	return data;

package/src/runtime/server/page/render.js

@@ -102,23 +102,6 @@ export async function render_response({
 			form: form_value
 		};
 
-		// TODO remove this for 1.0
-		/**
-		 * @param {string} property
-		 * @param {string} replacement
-		 */
-		const print_error = (property, replacement) => {
-			Object.defineProperty(props.page, property, {
-				get: () => {
-					throw new Error(`$page.${property} has been replaced by $page.url.${replacement}`);
-				}
-			});
-		};
-
-		print_error('origin', 'origin');
-		print_error('path', 'pathname');
-		print_error('query', 'searchParams');
-
 		rendered = options.root.render(props);
 
 		for (const { node } of branch) {

package/src/runtime/server/utils.js

@@ -11,30 +11,11 @@ export function is_pojo(body) {
 	if (body) {
 		if (body instanceof Uint8Array) return false;
 		if (body instanceof ReadableStream) return false;
-
-		// if body is a node Readable, throw an error
-		// TODO remove this for 1.0
-		if (body._readableState && typeof body.pipe === 'function') {
-			throw new Error('Node streams are no longer supported — use a ReadableStream instead');
-		}
 	}
 
 	return true;
 }
 
-// TODO: Remove for 1.0
-/** @param {Record<string, any>} mod */
-export function check_method_names(mod) {
-	['get', 'post', 'put', 'patch', 'del'].forEach((m) => {
-		if (m in mod) {
-			const replacement = m === 'del' ? 'DELETE' : m.toUpperCase();
-			throw Error(
-				`Endpoint method "${m}" has changed to "${replacement}". See https://github.com/sveltejs/kit/discussions/5359 for more information.`
-			);
-		}
-	});
-}
-
 /** @type {import('types').SSRErrorPage} */
 export const GENERIC_ERROR = {
 	id: '__error'
@@ -79,15 +60,6 @@ export function allowed_methods(mod) {
  */
 export function get_option(nodes, option) {
 	return nodes.reduce((value, node) => {
-		// TODO remove for 1.0
-		for (const thing of [node?.server, node?.universal]) {
-			if (thing && ('router' in thing || 'hydrate' in thing)) {
-				throw new Error(
-					'`export const hydrate` and `export const router` have been replaced with `export const csr`. See https://github.com/sveltejs/kit/pull/6446'
-				);
-			}
-		}
-
 		return /** @type {any} TypeScript's too dumb to understand this */ (
 			node?.universal?.[option] ?? node?.server?.[option] ?? value
 		);

package/src/utils/filesystem.js

@@ -83,6 +83,7 @@ export function copy(source, target, opts = {}) {
  * Get a list of all files in a directory
  * @param {string} cwd - the directory to walk
  * @param {boolean} [dirs] - whether to include directories in the result
+ * @returns {string[]} a list of all found files (and possibly directories) relative to `cwd`
  */
 export function walk(cwd, dirs = false) {
 	/** @type {string[]} */