@sveltejs/kit 1.0.0-next.397 → 1.0.0-next.398

package/assets/client/start.js

@@ -1595,10 +1595,12 @@ function create_client({ target, session, base, trailing_slash }) {
 				const is_svg_a_element = a instanceof SVGAElement;
 				const url = get_href(a);
 
-				// Ignore if url does not have origin (e.g. `mailto:`, `tel:`.)
+				// Ignore non-HTTP URL protocols (e.g. `mailto:`, `tel:`, `myapp:`, etc.)
 				// MEMO: Without this condition, firefox will open mailer twice.
-				// See: https://github.com/sveltejs/kit/issues/4045
-				if (!is_svg_a_element && url.origin === 'null') return;
+				// See:
+				// - https://github.com/sveltejs/kit/issues/4045
+				// - https://github.com/sveltejs/kit/issues/5725
+				if (!is_svg_a_element && !(url.protocol === 'https:' || url.protocol === 'http:')) return;
 
 				// Ignore if tag has
 				// 1. 'download' attribute

package/dist/chunks/sync.js

@@ -5,7 +5,7 @@ import { p as posixify, c as copy, r as rimraf } from './filesystem.js';
 import { fileURLToPath } from 'url';
 import { w as write_if_changed, t as trim, a as write, v as valid_identifier, r as reserved, b as write_tsconfig } from './write_tsconfig.js';
 import { $ } from './index.js';
-import { loadEnv, loadConfigFromFile } from 'vite';
+import { loadEnv, normalizePath, loadConfigFromFile } from 'vite';
 
 /**
  * @param typeMap [Object] Map of MIME type -> Array[extensions]
@@ -1104,7 +1104,7 @@ function get_env(mode, prefix) {
 /**
  * @param {(id: string) => import('rollup').ModuleInfo | null} node_getter
  * @param {import('rollup').ModuleInfo} node
- * @param {Set<string>} illegal_imports
+ * @param {Set<string>} illegal_imports Illegal module IDs -- be sure to call vite.normalizePath!
  * @param {string} out_dir The directory specified by config.kit.outDir
  */
 function prevent_illegal_rollup_imports(node_getter, node, illegal_imports, out_dir) {
@@ -1116,9 +1116,9 @@ function prevent_illegal_rollup_imports(node_getter, node, illegal_imports, out_
  * @param {(id: string) => import('rollup').ModuleInfo | null} node_getter
  * @param {import('rollup').ModuleInfo} node
  * @param {boolean} dynamic
- * @param {Set<string>} illegal_imports
+ * @param {Set<string>} illegal_imports Illegal module IDs -- be sure to call vite.normalizePath!
  * @param {Set<string>} seen
- * @returns {Array<import('types').ImportNode> | undefined}
+ * @returns {Array<import('types').ImportNode> | null}
  */
 const find_illegal_rollup_imports = (
 	node_getter,
@@ -1127,92 +1127,126 @@ const find_illegal_rollup_imports = (
 	illegal_imports,
 	seen = new Set()
 ) => {
-	if (seen.has(node.id)) return;
-	seen.add(node.id);
+	const name = normalizePath(node.id);
+	if (seen.has(name)) return null;
+	seen.add(name);
 
-	if (illegal_imports.has(node.id)) {
-		return [{ name: node.id, dynamic }];
+	if (illegal_imports.has(name)) {
+		return [{ name, dynamic }];
 	}
 
 	for (const id of node.importedIds) {
 		const child = node_getter(id);
 		const chain =
 			child && find_illegal_rollup_imports(node_getter, child, false, illegal_imports, seen);
-		if (chain) return [{ name: node.id, dynamic }, ...chain];
+		if (chain) return [{ name, dynamic }, ...chain];
 	}
 
 	for (const id of node.dynamicallyImportedIds) {
 		const child = node_getter(id);
 		const chain =
 			child && find_illegal_rollup_imports(node_getter, child, true, illegal_imports, seen);
-		if (chain) return [{ name: node.id, dynamic }, ...chain];
+		if (chain) return [{ name, dynamic }, ...chain];
 	}
+
+	seen.delete(name);
+	return null;
+};
+
+/**
+ * Vite does some weird things with import trees in dev
+ * for example, a Tailwind app.css will appear to import
+ * every file in the project. This isn't a problem for
+ * Rollup during build.
+ * @param {Iterable<string>} config_module_types
+ */
+const get_module_types = (config_module_types) => {
+	return new Set([
+		'.ts',
+		'.js',
+		'.svelte',
+		'.mts',
+		'.mjs',
+		'.cts',
+		'.cjs',
+		'.svelte.md',
+		'.svx',
+		'.md',
+		...config_module_types
+	]);
 };
 
 /**
  * Throw an error if a private module is imported from a client-side node.
  * @param {import('vite').ModuleNode} node
- * @param {Set<string>} illegal_imports
+ * @param {Set<string>} illegal_imports Illegal module IDs -- be sure to call vite.normalizePath!
+ * @param {Iterable<string>} module_types File extensions to analyze in addition to the defaults: `.ts`, `.js`, etc.
  * @param {string} out_dir The directory specified by config.kit.outDir
  */
-function prevent_illegal_vite_imports(node, illegal_imports, out_dir) {
-	const chain = find_illegal_vite_imports(node, illegal_imports);
+function prevent_illegal_vite_imports(node, illegal_imports, module_types, out_dir) {
+	const chain = find_illegal_vite_imports(node, illegal_imports, get_module_types(module_types));
 	if (chain) throw new Error(format_illegal_import_chain(chain, out_dir));
 }
 
 /**
  * @param {import('vite').ModuleNode} node
- * @param {Set<string>} illegal_imports
+ * @param {Set<string>} illegal_imports Illegal module IDs -- be sure to call vite.normalizePath!
+ * @param {Set<string>} module_types File extensions to analyze: `.ts`, `.js`, etc.
  * @param {Set<string>} seen
  * @returns {Array<import('types').ImportNode> | null}
  */
-function find_illegal_vite_imports(node, illegal_imports, seen = new Set()) {
+function find_illegal_vite_imports(node, illegal_imports, module_types, seen = new Set()) {
 	if (!node.id) return null; // TODO when does this happen?
+	const name = normalizePath(node.id);
 
-	if (seen.has(node.id)) return null;
-	seen.add(node.id);
+	if (seen.has(name) || !module_types.has(path__default.extname(name))) return null;
+	seen.add(name);
 
-	if (node.id && illegal_imports.has(node.id)) {
-		return [{ name: node.id, dynamic: false }];
+	if (name && illegal_imports.has(name)) {
+		return [{ name, dynamic: false }];
 	}
 
 	for (const child of node.importedModules) {
-		const chain = child && find_illegal_vite_imports(child, illegal_imports, seen);
-		if (chain) return [{ name: node.id, dynamic: false }, ...chain];
+		const chain = child && find_illegal_vite_imports(child, illegal_imports, module_types, seen);
+		if (chain) return [{ name, dynamic: false }, ...chain];
 	}
 
+	seen.delete(name);
 	return null;
 }
 
 const autogen_comment = '// this file is generated — do not edit it\n';
+const types_reference = '/// <reference types="@sveltejs/kit" />\n\n';
 
 /**
- * Writes the existing environment variables in process.env to
+ * Writes ambient declarations including types reference to @sveltejs/kit,
+ * and the existing environment variables in process.env to
  * $env/static/private and $env/static/public
  * @param {import('types').ValidatedKitConfig} config
  * @param {string} mode The Vite mode
  */
-function write_env(config, mode) {
+function write_ambient(config, mode) {
 	const env = get_env(mode, config.env.publicPrefix);
 
 	// TODO when testing src, `$app` points at `src/runtime/app`... will
 	// probably need to fiddle with aliases
 	write_if_changed(
 		path__default.join(config.outDir, 'runtime/env/static/public.js'),
-		create_module('$env/static/public', env.public)
+		create_env_module('$env/static/public', env.public)
 	);
 
 	write_if_changed(
 		path__default.join(config.outDir, 'runtime/env/static/private.js'),
-		create_module('$env/static/private', env.private)
+		create_env_module('$env/static/private', env.private)
 	);
 
 	write_if_changed(
 		path__default.join(config.outDir, 'ambient.d.ts'),
 		autogen_comment +
-			create_types('$env/static/public', env.public) +
+			types_reference +
+			create_env_types('$env/static/public', env.public) +
 			'\n\n' +
-			create_types('$env/static/private', env.private)
+			create_env_types('$env/static/private', env.private)
 	);
 }
 
@@ -1221,7 +1255,7 @@ function write_env(config, mode) {
  * @param {Record<string, string>} env
  * @returns {string}
  */
-function create_module(id, env) {
+function create_env_module(id, env) {
 	/** @type {string[]} */
 	const declarations = [];
 
@@ -1255,7 +1289,7 @@ function create_module(id, env) {
  * @param {Record<string, string>} env
  * @returns {string}
  */
-function create_types(id, env) {
+function create_env_types(id, env) {
 	const declarations = Object.keys(env)
 		.filter((k) => valid_identifier.test(k))
 		.map((k) => `\texport const ${k}: string;`)
@@ -1273,7 +1307,7 @@ function init(config, mode) {
 	copy_assets(path__default.join(config.kit.outDir, 'runtime'));
 
 	write_tsconfig(config.kit);
-	write_env(config.kit, mode);
+	write_ambient(config.kit, mode);
 }
 
 /**

package/dist/cli.js

@@ -19,7 +19,7 @@ function handle_error(e) {
 	process.exit(1);
 }
 
-const prog = sade('svelte-kit').version('1.0.0-next.397');
+const prog = sade('svelte-kit').version('1.0.0-next.398');
 
 prog
 	.command('package')

package/dist/vite.js

@@ -1297,7 +1297,12 @@ async function dev(vite, vite_config, svelte_config, illegal_imports) {
 						const node = await vite.moduleGraph.getModuleByUrl(url);
 						if (!node) throw new Error(`Could not find node for ${url}`);
 
-						prevent_illegal_vite_imports(node, illegal_imports, svelte_config.kit.outDir);
+						prevent_illegal_vite_imports(
+							node,
+							illegal_imports,
+							[...svelte_config.extensions, ...svelte_config.kit.moduleExtensions],
+							svelte_config.kit.outDir
+						);
 
 						return {
 							module,
@@ -2191,8 +2196,8 @@ function kit() {
 			};
 
 			illegal_imports = new Set([
-				`${svelte_config.kit.outDir}/runtime/env/dynamic/private.js`,
-				`${svelte_config.kit.outDir}/runtime/env/static/private.js`
+				vite.normalizePath(`${svelte_config.kit.outDir}/runtime/env/dynamic/private.js`),
+				vite.normalizePath(`${svelte_config.kit.outDir}/runtime/env/static/private.js`)
 			]);
 
 			if (is_build) {
@@ -2283,7 +2288,7 @@ function kit() {
 		 */
 		async writeBundle(_options, bundle) {
 			for (const file of manifest_data.components) {
-				const id = path.resolve(file);
+				const id = vite.normalizePath(path.resolve(file));
 				const node = this.getModuleInfo(id);
 
 				if (node) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sveltejs/kit",
-  "version": "1.0.0-next.397",
+  "version": "1.0.0-next.398",
   "repository": {
     "type": "git",
     "url": "https://github.com/sveltejs/kit",