@sveltejs/kit 2.69.1 → 2.69.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sveltejs/kit",
-  "version": "2.69.1",
+  "version": "2.69.3",
   "description": "SvelteKit is the fastest way to build Svelte apps",
   "keywords": [
     "framework",

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

@@ -636,6 +636,16 @@ export function tweak_types(content, is_server) {
 		const code = new MagicString(content);
 
 		const exports = new Map();
+		/** @param {import('typescript').BindingName} name */
+		function add_export(name) {
+			if (ts.isIdentifier(name)) {
+				if (names.has(name.text)) exports.set(name.text, name.text);
+			} else {
+				for (const element of name.elements) {
+					if (ts.isBindingElement(element)) add_export(element.name);
+				}
+			}
+		}
 
 		ast.forEachChild((node) => {
 			if (
@@ -662,9 +672,7 @@ export function tweak_types(content, is_server) {
 
 				if (ts.isVariableStatement(node)) {
 					node.declarationList.declarations.forEach((declaration) => {
-						if (ts.isIdentifier(declaration.name) && names.has(declaration.name.text)) {
-							exports.set(declaration.name.text, declaration.name.text);
-						}
+						add_export(declaration.name);
 					});
 				}
 			}

package/src/exports/vite/index.js

@@ -482,6 +482,23 @@ async function kit({ svelte_config }) {
 					];
 				}
 
+				// Vite's `define` is a compile-time text replacement, but Vitest strips
+				// user `define` from the server config and reinstalls the values only as
+				// `globalThis` properties inside test workers, so anything
+				// that runs outside of a test will freak out over
+				// them not being defined
+				if (process.env.VITEST === 'true') {
+					for (const key in new_config.define) {
+						const value = new_config.define[key];
+						try {
+							/** @type {Record<string, any>} */ (globalThis)[key] = JSON.parse(value);
+						} catch {
+							// `kit_global` isn't JSON, so don't try to parse it. We may one day
+							// need to define it in Vitest somehow but for now, ignore it
+						}
+					}
+				}
+
 				warn_overridden_config(config, new_config);
 
 				return new_config;
@@ -964,7 +981,7 @@ async function kit({ svelte_config }) {
 
 				if (is_build) {
 					const ssr = /** @type {boolean} */ (config.build?.ssr);
-					const prefix = `${kit.appDir}/immutable`;
+					const app_immutable = `${kit.appDir}/immutable`;
 
 					/** @type {Record<string, string>} */
 					const input = {};
@@ -1054,9 +1071,9 @@ async function kit({ svelte_config }) {
 
 					/** @type {string} */
 					const base = (kit.paths.assets || kit.paths.base) + '/';
-					const root_to_assets = prefix + '/assets/';
+					const root_to_assets = app_immutable + '/assets/';
 					const assets_to_root =
-						prefix
+						app_immutable
 							.split('/')
 							.map(() => '..')
 							.join('/') + '/../';
@@ -1082,9 +1099,11 @@ async function kit({ svelte_config }) {
 								output: {
 									format: inline ? 'iife' : 'esm',
 									name: `__sveltekit_${version_hash}.app`,
-									entryFileNames: ssr ? '[name].js' : `${prefix}/[name].[hash].${ext}`,
-									chunkFileNames: ssr ? 'chunks/[name].js' : `${prefix}/chunks/[hash].${ext}`,
-									assetFileNames: `${prefix}/assets/[name].[hash][extname]`,
+									entryFileNames: ssr ? '[name].js' : `${app_immutable}/[name].[hash].${ext}`,
+									chunkFileNames: ssr
+										? 'chunks/[name].js'
+										: `${app_immutable}/chunks/[hash].${ext}`,
+									assetFileNames: `${app_immutable}/assets/[name].[hash][extname]`,
 									hoistTransitiveImports: false,
 									sourcemapIgnoreList,
 									inlineDynamicImports: is_rolldown ? undefined : !split
@@ -1112,9 +1131,9 @@ async function kit({ svelte_config }) {
 						worker: {
 							rollupOptions: {
 								output: {
-									entryFileNames: `${prefix}/workers/[name]-[hash].js`,
-									chunkFileNames: `${prefix}/workers/chunks/[hash].js`,
-									assetFileNames: `${prefix}/workers/assets/[name]-[hash][extname]`,
+									entryFileNames: `${app_immutable}/workers/[name]-[hash].js`,
+									chunkFileNames: `${app_immutable}/workers/chunks/[hash].js`,
+									assetFileNames: `${app_immutable}/workers/assets/[name]-[hash][extname]`,
 									hoistTransitiveImports: false
 								}
 							}
@@ -1134,6 +1153,11 @@ async function kit({ svelte_config }) {
 									return { relative };
 								}
 
+								if (!relative) return;
+
+								// ensure assets loaded by CSS files are loaded relative to the
+								// CSS file rather than the default of relative to the root
+
 								// _app/immutable/assets files
 								if (filename.startsWith(root_to_assets)) {
 									return `./${filename.slice(root_to_assets.length)}`;

package/src/runtime/server/cookie.js

@@ -15,10 +15,11 @@ const INVALID_COOKIE_CHARACTER_REGEX = /[\x00-\x1F\x7F()<>@,;:"/[\]?={} \t]/;
 const cookie_paths = {};
 
 /**
- * Cookies that are larger than this size (including the name and other
- * attributes) are discarded by browsers
+ * Cookies whose name and value combined are larger than this size are
+ * discarded by browsers. This is the limit codified for the name/value pair
+ * in RFC 6265bis: https://datatracker.ietf.org/doc/html/draft-ietf-httpbis-rfc6265bis-20#section-5.6-7.5.1
  */
-const MAX_COOKIE_SIZE = 4129;
+const MAX_COOKIE_SIZE = 4096;
 
 // TODO 3.0 remove this check
 /** @param {import('./page/types.js').Cookie['options']} options */
@@ -254,8 +255,11 @@ export function get_cookies(request, url) {
 		new_cookies.set(cookie_key, cookie);
 
 		if (DEV) {
-			const serialized = serialize(name, value, cookie.options);
-			if (text_encoder.encode(serialized).byteLength > MAX_COOKIE_SIZE) {
+			// only the name/value pair counts towards MAX_COOKIE_SIZE, not the other attributes
+			const encoder = cookie.options.encode || encodeURIComponent;
+			const size =
+				text_encoder.encode(name).byteLength + text_encoder.encode(encoder(value)).byteLength;
+			if (size > MAX_COOKIE_SIZE) {
 				throw new Error(`Cookie "${name}" is too large, and will be discarded by the browser`);
 			}
 

package/src/version.js

@@ -1,4 +1,4 @@
 // generated during release, do not modify
 
 /** @type {string} */
-export const VERSION = '2.69.1';
+export const VERSION = '2.69.3';