fix: allow logging $page.url when url.search is disabled (#10567)

fixes #10526

We were allow logging when url.hash is disabled but not when url.search is disabled. Just copied over the same thing for when url.search is disabled.

.changeset/small-mayflies-repeat.md

@@ -0,0 +1,5 @@
+---
+'@sveltejs/kit': patch
+---
+
+fix: allow logging `$page.url` during prerendering

packages/kit/src/utils/url.js

@@ -139,6 +139,8 @@ export function make_trackable(url, callback) {
  * @param {URL} url
  */
 export function disable_hash(url) {
+	allow_nodejs_console_log(url);
+
 	Object.defineProperty(url, 'hash', {
 		get() {
 			throw new Error(
@@ -153,6 +155,8 @@ export function disable_hash(url) {
  * @param {URL} url
  */
 export function disable_search(url) {
+	allow_nodejs_console_log(url);
+
 	for (const property of ['search', 'searchParams']) {
 		Object.defineProperty(url, property, {
 			get() {
@@ -162,6 +166,19 @@ export function disable_search(url) {
 	}
 }
 
+/**
+ * Allow URL to be console logged, bypassing disabled properties.
+ * @param {URL} url
+ */
+function allow_nodejs_console_log(url) {
+	if (!BROWSER) {
+		// @ts-ignore
+		url[Symbol.for('nodejs.util.inspect.custom')] = (depth, opts, inspect) => {
+			return inspect(new URL(url), opts);
+		};
+	}
+}
+
 const DATA_SUFFIX = '/__data.json';
 
 /** @param {string} pathname */

packages/kit/test/apps/basics/src/routes/prerendering/log-url/+page.js

@@ -0,0 +1 @@
+export const prerender = true;

packages/kit/test/apps/basics/src/routes/prerendering/log-url/+page.svelte

@@ -0,0 +1,13 @@
+<script>
+	import { page } from '$app/stores';
+
+	let error = false;
+
+	try {
+		console.log($page);
+	} catch (e) {
+		error = true;
+	}
+</script>
+
+<p>error: {error}</p>

packages/kit/test/apps/basics/test/test.js

@@ -509,6 +509,12 @@ test.describe('Load', () => {
 			'{"message":"Im prerendered and called from a non-prerendered +page.server.js"}'
 		);
 	});
+
+	test('Logging $page.url during prerendering works', async ({ page }) => {
+		await page.goto('/prerendering/log-url');
+
+		expect(await page.textContent('p')).toBe('error: false');
+	});
 });
 
 test.describe('Nested layouts', () => {