@sveltejs/kit 2.5.17 → 2.5.19

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sveltejs/kit",
-  "version": "2.5.17",
+  "version": "2.5.19",
   "description": "SvelteKit is the fastest way to build Svelte apps",
   "keywords": [
     "framework",
@@ -36,15 +36,14 @@
     "@sveltejs/vite-plugin-svelte": "^3.0.1",
     "@types/connect": "^3.4.38",
     "@types/node": "^18.19.3",
-    "@types/sade": "^1.7.8",
     "@types/set-cookie-parser": "^2.4.7",
     "dts-buddy": "0.4.6",
     "rollup": "^4.14.2",
     "svelte": "^4.2.10",
     "svelte-preprocess": "^6.0.0",
     "typescript": "^5.3.3",
-    "vite": "^5.2.8",
-    "vitest": "^1.6.0"
+    "vite": "^5.3.2",
+    "vitest": "^2.0.1"
   },
   "peerDependencies": {
     "@sveltejs/vite-plugin-svelte": "^3.0.0",

package/src/core/sync/write_root.js

@@ -21,15 +21,20 @@ export function write_root(manifest_data, output) {
 
 	let l = max_depth;
 
-	let pyramid = `<svelte:component this={constructors[${l}]} bind:this={components[${l}]} data={data_${l}} {form} />`;
+	let pyramid = dedent`
+	${isSvelte5Plus() ? '<!-- svelte-ignore binding_property_non_reactive -->' : ''}
+	<svelte:component this={constructors[${l}]} bind:this={components[${l}]} data={data_${l}} {form} />
+	`;
 
 	while (l--) {
 		pyramid = dedent`
 			{#if constructors[${l + 1}]}
+				${isSvelte5Plus() ? '<!-- svelte-ignore binding_property_non_reactive -->' : ''}
 				<svelte:component this={constructors[${l}]} bind:this={components[${l}]} data={data_${l}}>
 					${pyramid}
 				</svelte:component>
 			{:else}
+				${isSvelte5Plus() ? '<!-- svelte-ignore binding_property_non_reactive -->' : ''}
 				<svelte:component this={constructors[${l}]} bind:this={components[${l}]} data={data_${l}} {form} />
 			{/if}
 		`;

package/src/runtime/app/forms.js

@@ -56,7 +56,7 @@ function clone(element) {
  * If nothing is returned, the fallback will be used.
  *
  * If this function or its return value isn't set, it
- * - falls back to updating the `form` prop with the returned data if the action is one same page as the form
+ * - falls back to updating the `form` prop with the returned data if the action is on the same page as the form
  * - updates `$page.status`
  * - resets the `<form>` element and invalidates all data in case of successful submission with no redirect response
  * - redirects in case of a redirect response
@@ -124,9 +124,13 @@ export function enhance(form_element, submit = () => {}) {
 				: clone(form_element).action
 		);
 
+		const enctype = event.submitter?.hasAttribute('formenctype')
+			? /** @type {HTMLButtonElement | HTMLInputElement} */ (event.submitter).formEnctype
+			: clone(form_element).enctype;
+
 		const form_data = new FormData(form_element);
 
-		if (DEV && clone(form_element).enctype !== 'multipart/form-data') {
+		if (DEV && enctype !== 'multipart/form-data') {
 			for (const value of form_data.values()) {
 				if (value instanceof File) {
 					throw new Error(
@@ -161,14 +165,31 @@ export function enhance(form_element, submit = () => {}) {
 		let result;
 
 		try {
+			const headers = new Headers({
+				accept: 'application/json',
+				'x-sveltekit-action': 'true'
+			});
+
+			// do not explicitly set the `Content-Type` header when sending `FormData`
+			// or else it will interfere with the browser's header setting
+			// see https://developer.mozilla.org/en-US/docs/Web/API/XMLHttpRequest_API/Using_FormData_Objects#sect4
+			if (enctype !== 'multipart/form-data') {
+				headers.set(
+					'Content-Type',
+					/^(:?application\/x-www-form-urlencoded|text\/plain)$/.test(enctype)
+						? enctype
+						: 'application/x-www-form-urlencoded'
+				);
+			}
+
+			// @ts-expect-error `URLSearchParams(form_data)` is kosher, but typescript doesn't know that
+			const body = enctype === 'multipart/form-data' ? form_data : new URLSearchParams(form_data);
+
 			const response = await fetch(action, {
 				method: 'POST',
-				headers: {
-					accept: 'application/json',
-					'x-sveltekit-action': 'true'
-				},
+				headers,
 				cache: 'no-store',
-				body: form_data,
+				body,
 				signal: controller.signal
 			});
 

package/src/utils/filesystem.js

@@ -181,7 +181,7 @@ export function resolve_entry(entry) {
 			const base = path.basename(entry);
 			const files = fs.readdirSync(dir);
 
-			const found = files.find((file) => file.replace(/\.[^.]+$/, '') === base);
+			const found = files.find((file) => file.replace(/\.(js|ts)$/, '') === base);
 
 			if (found) return path.join(dir, found);
 		}

package/src/version.js

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

package/types/index.d.ts

@@ -1999,7 +1999,7 @@ declare module '$app/forms' {
 	 * If nothing is returned, the fallback will be used.
 	 *
 	 * If this function or its return value isn't set, it
-	 * - falls back to updating the `form` prop with the returned data if the action is one same page as the form
+	 * - falls back to updating the `form` prop with the returned data if the action is on the same page as the form
 	 * - updates `$page.status`
 	 * - resets the `<form>` element and invalidates all data in case of successful submission with no redirect response
 	 * - redirects in case of a redirect response