@sveltejs/kit 1.2.1 → 1.2.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sveltejs/kit",
-  "version": "1.2.1",
+  "version": "1.2.3",
   "repository": {
     "type": "git",
     "url": "https://github.com/sveltejs/kit",
@@ -22,7 +22,7 @@
     "set-cookie-parser": "^2.5.1",
     "sirv": "^2.0.2",
     "tiny-glob": "^0.2.9",
-    "undici": "5.15.1"
+    "undici": "5.15.2"
   },
   "devDependencies": {
     "@playwright/test": "^1.29.2",
@@ -86,7 +86,7 @@
     "format": "pnpm lint --write",
     "test": "pnpm test:unit && pnpm test:integration",
     "test:integration": "pnpm -r --workspace-concurrency 1 --filter=\"./test/**\" test",
-    "test:cross-browser": "pnpm -r --workspace-concurrency 1 --filter=\"./test/**\" test:cross-browser",
+    "test:cross-platform": "pnpm test:unit && pnpm -r --workspace-concurrency 1 --filter=\"./test/**\" test:cross-platform",
     "test:unit": "uvu src \"(spec\\.js|test[\\\\/]index\\.js)\"",
     "postinstall": "node postinstall.js"
   }

package/src/core/env.js

@@ -44,7 +44,7 @@ export function create_dynamic_module(type, dev_values) {
 		);
 		return `export const env = {\n${keys.join(',\n')}\n}`;
 	}
-	return `export { env } from '${runtime_base}/env-${type}.js';`;
+	return `export { ${type}_env as env } from '${runtime_base}/shared.js';`;
 }
 
 /**

package/src/core/postbuild/index.js

@@ -10,7 +10,8 @@ import {
 import { load_config } from '../config/index.js';
 import { prerender } from './prerender.js';
 
-const [, , client_out_dir, manifest_path, results_path, verbose, env] = process.argv;
+const [, , client_out_dir, manifest_path, results_path, verbose, env_json] = process.argv;
+const env = JSON.parse(env_json);
 
 /** @type {import('types').SSRManifest} */
 const manifest = (await import(pathToFileURL(manifest_path).href)).manifest;
@@ -33,6 +34,12 @@ const { Server } = await import(pathToFileURL(`${server_root}/server/index.js`).
 // essential we do this before analysing the code
 internal.set_building(true);
 
+// set env, in case it's used in initialisation
+const entries = Object.entries(env);
+const prefix = config.env.publicPrefix;
+internal.set_private_env(Object.fromEntries(entries.filter(([k]) => !k.startsWith(prefix))));
+internal.set_public_env(Object.fromEntries(entries.filter(([k]) => k.startsWith(prefix))));
+
 // analyse routes
 for (const route of manifest._.routes) {
 	if (route.endpoint) {

package/src/core/postbuild/prerender.js

@@ -50,7 +50,7 @@ const REDIRECT = 3;
  *   prerender_map: import('types').PrerenderMap;
  *   client_out_dir: string;
  *   verbose: string;
- *   env: string;
+ *   env: Record<string, string>;
  * }} opts
  * @returns
  */
@@ -92,7 +92,7 @@ export async function prerender({
 	internal.set_paths(config.paths);
 
 	const server = new Server(manifest);
-	await server.init({ env: JSON.parse(env) });
+	await server.init({ env });
 
 	const handle_http_error = normalise_error_handler(
 		log,

package/src/core/sync/write_server.js

@@ -24,7 +24,7 @@ const server_template = ({
 	error_page
 }) => `
 import root from './root.svelte';
-import { set_building, set_paths, set_version } from '${runtime_directory}/shared.js';
+import { set_building, set_paths, set_private_env, set_public_env, set_version } from '${runtime_directory}/shared.js';
 
 set_paths(${s(config.kit.paths)});
 set_version(${s(config.kit.version.name)});
@@ -57,7 +57,7 @@ export function get_hooks() {
 	return ${hooks ? `import(${s(hooks)})` : '{}'};
 }
 
-export { set_building, set_paths };
+export { set_building, set_paths, set_private_env, set_public_env };
 `;
 
 // TODO need to re-run this whenever src/app.html or src/error.html are

package/src/runtime/client/client.js

@@ -326,13 +326,22 @@ export function create_client({ target, base }) {
 		// opts must be passed if we're navigating
 		if (opts) {
 			const { scroll, keepfocus } = opts;
+			const { activeElement } = document;
 
-			// reset focus first, so that manual focus management can override it
-			if (!keepfocus) reset_focus();
-
-			// need to render the DOM before we can scroll to the rendered elements
+			// need to render the DOM before we can scroll to the rendered elements and do focus management
 			await tick();
 
+			const changed_focus =
+				// reset focus only if any manual focus management didn't override it
+				document.activeElement !== activeElement &&
+				// also refocus when activeElement is body already because the
+				// focus event might not have been fired on it yet
+				document.activeElement !== document.body;
+
+			if (!keepfocus && !changed_focus) {
+				await reset_focus();
+			}
+
 			if (autoscroll) {
 				const deep_linked = url.hash && document.getElementById(url.hash.slice(1));
 				if (scroll) {
@@ -1718,16 +1727,19 @@ function reset_focus() {
 		root.tabIndex = -1;
 		root.focus({ preventScroll: true });
 
-		setTimeout(() => {
-			getSelection()?.removeAllRanges();
-		});
-
 		// restore `tabindex` as to prevent `root` from stealing input from elements
 		if (tabindex !== null) {
 			root.setAttribute('tabindex', tabindex);
 		} else {
 			root.removeAttribute('tabindex');
 		}
+
+		return new Promise((resolve) => {
+			setTimeout(() => {
+				// fixes https://github.com/sveltejs/kit/issues/8439
+				resolve(getSelection()?.removeAllRanges());
+			});
+		});
 	}
 }
 

package/src/runtime/client/start.js

@@ -1,8 +1,7 @@
 import { DEV } from 'esm-env';
 import { create_client } from './client.js';
 import { init } from './singletons.js';
-import { set_paths, set_version } from '../shared.js';
-import { set_public_env } from '../env-public.js';
+import { set_paths, set_version, set_public_env } from '../shared.js';
 
 /**
  * @param {{

package/src/runtime/env/dynamic/private.js

@@ -1 +1 @@
-export { env } from '../../env-private.js';
+export { private_env as env } from '../../shared.js';

package/src/runtime/env/dynamic/public.js

@@ -1 +1 @@
-export { env } from '../../env-public.js';
+export { public_env as env } from '../../shared.js';

package/src/runtime/server/index.js

@@ -1,6 +1,5 @@
 import { respond } from './respond.js';
-import { set_private_env } from '../env-private.js';
-import { set_public_env } from '../env-public.js';
+import { set_private_env, set_public_env } from '../shared.js';
 import { options, get_hooks } from '__GENERATED__/server-internal.js';
 
 export class Server {

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

@@ -7,8 +7,7 @@ import { s } from '../../../utils/misc.js';
 import { Csp } from './csp.js';
 import { uneval_action_response } from './actions.js';
 import { clarify_devalue_error } from '../utils.js';
-import { assets, base, version } from '../../shared.js';
-import { env } from '../../env-public.js';
+import { assets, base, version, public_env } from '../../shared.js';
 import { text } from '../../../exports/index.js';
 
 // TODO rename this function/module
@@ -258,7 +257,7 @@ export async function render_response({
 
 	if (page_config.csr) {
 		const opts = [
-			`env: ${s(env)}`,
+			`env: ${s(public_env)}`,
 			`paths: ${s({ assets, base })}`,
 			`target: document.querySelector('[data-sveltekit-hydrate="${target}"]').parentNode`,
 			`version: ${s(version)}`
@@ -368,7 +367,7 @@ export async function render_response({
 		body,
 		assets: resolved_assets,
 		nonce: /** @type {string} */ (csp.nonce),
-		env
+		env: public_env
 	});
 
 	// TODO flush chunks as early as we can

package/src/runtime/shared.js

@@ -3,6 +3,12 @@ export let base = '';
 export let building = false;
 export let version = '';
 
+/** @type {Record<string, string>} */
+export let private_env = {};
+
+/** @type {Record<string, string>} */
+export let public_env = {};
+
 /** @param {string} stack */
 export let fix_stack_trace = (stack) => stack;
 
@@ -17,6 +23,16 @@ export function set_building(value) {
 	building = value;
 }
 
+/** @type {(environment: Record<string, string>) => void} */
+export function set_private_env(environment) {
+	private_env = environment;
+}
+
+/** @type {(environment: Record<string, string>) => void} */
+export function set_public_env(environment) {
+	public_env = environment;
+}
+
 /** @param {string} value */
 export function set_version(value) {
 	version = value;

package/types/internal.d.ts

@@ -32,6 +32,8 @@ export interface ServerModule {
 export interface ServerInternalModule {
 	set_building(building: boolean): void;
 	set_paths(paths: { base: string; assets: string }): void;
+	set_private_env(environment: Record<string, string>): void;
+	set_public_env(environment: Record<string, string>): void;
 	set_version(version: string): void;
 	set_fix_stack_trace(fix_stack_trace: (stack: string) => string): void;
 }

package/src/runtime/env-private.js

@@ -1,6 +0,0 @@
-export let env = {};
-
-/** @type {(environment: Record<string, string>) => void} */
-export function set_private_env(environment) {
-	env = environment;
-}

package/src/runtime/env-public.js

@@ -1,7 +0,0 @@
-/** @type {Record<string, string>} */
-export let env = {};
-
-/** @type {(environment: Record<string, string>) => void} */
-export function set_public_env(environment) {
-	env = environment;
-}