fix: ensure cache key is consistent between client/server (#14563)

fixes #14562

Use same function to generate cache keys for remote query data during ssr and hydration so the logic doesn't need to be duplicated.

Author: lettucebowler

.changeset/dry-ideas-ring.md

@@ -0,0 +1,5 @@
+---
+'@sveltejs/kit': patch
+---
+
+fix: ensure cache key is consistent between client/server

packages/kit/src/runtime/app/server/remote/query.js

@@ -90,11 +90,10 @@ export function query(validate_or_fn, maybe_fn) {
 				);
 			}
 
-			const cache = get_cache(__, state);
-			const key = stringify_remote_arg(arg, state.transport);
-
 			if (__.id) {
-				refreshes[__.id + '/' + key] = cache[key] = Promise.resolve(value);
+				const cache = get_cache(__, state);
+				const key = stringify_remote_arg(arg, state.transport);
+				refreshes[create_remote_cache_key(__.id, key)] = cache[key] = Promise.resolve(value);
 			}
 		};
 

packages/kit/src/runtime/server/page/render.js

@@ -16,6 +16,7 @@ import { add_resolution_suffix } from '../../pathname.js';
 import { with_request_store } from '@sveltejs/kit/internal/server';
 import { text_encoder } from '../../utils.js';
 import { get_global_name } from '../utils.js';
+import { create_remote_cache_key } from '../../shared.js';
 
 // TODO rename this function/module
 
@@ -493,7 +494,7 @@ export async function render_response({
 				if (!info.id) continue;
 
 				for (const key in cache) {
-					remote[key ? info.id + '/' + key : info.id] = await cache[key];
+					remote[create_remote_cache_key(info.id, key)] = await cache[key];
 				}
 			}