@sveltejs/kit 2.38.0 → 2.38.1

package/package.json

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

package/src/runtime/app/server/remote/command.js

@@ -64,7 +64,13 @@ export function command(validate_or_fn, maybe_fn) {
 	const wrapper = (arg) => {
 		const { event, state } = get_request_store();
 
-		if (!event.isRemoteRequest) {
+		if (state.is_endpoint_request) {
+			if (!['POST', 'PUT', 'PATCH', 'DELETE'].includes(event.request.method)) {
+				throw new Error(
+					`Cannot call a command (\`${__.name}(${maybe_fn ? '...' : ''})\`) from a ${event.request.method} handler`
+				);
+			}
+		} else if (!event.isRemoteRequest) {
 			throw new Error(
 				`Cannot call a command (\`${__.name}(${maybe_fn ? '...' : ''})\`) during server-side rendering`
 			);

package/src/runtime/client/remote-functions/query.svelte.js

@@ -83,9 +83,7 @@ export function query_batch(id) {
 						}
 
 						if (result.type === 'redirect') {
-							// TODO double-check this
 							await goto(result.location);
-							await new Promise((r) => setTimeout(r, 100));
 							throw new Redirect(307, result.location);
 						}
 

package/src/runtime/client/remote-functions/shared.svelte.js

@@ -21,13 +21,7 @@ export async function remote_request(url) {
 	const result = /** @type {RemoteFunctionResponse} */ (await response.json());
 
 	if (result.type === 'redirect') {
-		// resource_cache.delete(cache_key);
-		// version++;
-		// await goto(result.location);
-		// /** @type {Query<any>} */ (resource).refresh();
-		// TODO double-check this
 		await goto(result.location);
-		await new Promise((r) => setTimeout(r, 100));
 		throw new Redirect(307, result.location);
 	}
 

package/src/runtime/server/endpoint.js

@@ -41,6 +41,8 @@ export async function render_endpoint(event, event_state, mod, state) {
 		}
 	}
 
+	event_state.is_endpoint_request = true;
+
 	try {
 		const response = await with_request_store({ event, state: event_state }, () =>
 			handler(/** @type {import('@sveltejs/kit').RequestEvent<Record<string, any>>} */ (event))

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

@@ -1,4 +1,5 @@
 import * as devalue from 'devalue';
+import { compact } from '../../../utils/array.js';
 import { create_async_iterator } from '../../../utils/streaming.js';
 import {
 	clarify_devalue_error,
@@ -96,7 +97,7 @@ export function server_data_serializer(event, event_state, options) {
 			const close = `</script>\n`;
 
 			return {
-				data: `[${strings.join(',')}]`,
+				data: `[${compact(strings).join(',')}]`,
 				chunks: promise_id > 1 ? iterator.iterate((str) => open + str + close) : null
 			};
 		}

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

@@ -195,7 +195,10 @@ export async function render_page(
 						}
 					});
 
-					data_serializer.add_node(i, server_data);
+					if (node) {
+						data_serializer.add_node(i, server_data);
+					}
+
 					data_serializer_json?.add_node(i, server_data);
 
 					return server_data;
@@ -348,7 +351,8 @@ export async function render_page(
 			branch: ssr === false ? [] : compact(branch),
 			action_result,
 			fetched,
-			data_serializer
+			data_serializer:
+				ssr === false ? server_data_serializer(event, event_state, options) : data_serializer
 		});
 	} catch (e) {
 		// if we end up here, it means the data loaded successfully

package/src/runtime/server/remote.js

@@ -117,8 +117,8 @@ async function handle_remote_call_internal(event, state, options, manifest, id)
 			}
 
 			const form_data = await event.request.formData();
-			form_client_refreshes = JSON.parse(
-				/** @type {string} */ (form_data.get('sveltekit:remote_refreshes')) ?? '[]'
+			form_client_refreshes = /** @type {string[]} */ (
+				JSON.parse(/** @type {string} */ (form_data.get('sveltekit:remote_refreshes')) ?? '[]')
 			);
 			form_data.delete('sveltekit:remote_refreshes');
 
@@ -129,7 +129,7 @@ async function handle_remote_call_internal(event, state, options, manifest, id)
 				/** @type {RemoteFunctionResponse} */ ({
 					type: 'result',
 					result: stringify(data, transport),
-					refreshes: await serialize_refreshes(/** @type {string[]} */ (form_client_refreshes))
+					refreshes: await serialize_refreshes(form_client_refreshes)
 				})
 			);
 		}
@@ -194,7 +194,7 @@ async function handle_remote_call_internal(event, state, options, manifest, id)
 	 * @param {string[]} client_refreshes
 	 */
 	async function serialize_refreshes(client_refreshes) {
-		const refreshes = /** @type {Record<string, Promise<any>>} */ (state.refreshes);
+		const refreshes = state.refreshes ?? {};
 
 		for (const key of client_refreshes) {
 			if (refreshes[key] !== undefined) continue;

package/src/types/internal.d.ts

@@ -598,6 +598,7 @@ export interface RequestState {
 	form_instances?: Map<any, any>;
 	remote_data?: Record<string, MaybePromise<any>>;
 	refreshes?: Record<string, Promise<any>>;
+	is_endpoint_request?: boolean;
 }
 
 export interface RequestStore {

package/src/version.js

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