@sveltejs/vite-plugin-svelte 2.4.5 → 3.0.0-next.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sveltejs/vite-plugin-svelte",
-  "version": "2.4.5",
+  "version": "3.0.0-next.0",
   "license": "MIT",
   "author": "dominikg",
   "files": [
@@ -16,7 +16,7 @@
     "./package.json": "./package.json"
   },
   "engines": {
-    "node": "^14.18.0 || >= 16"
+    "node": "^18.0.0 || >=20"
   },
   "repository": {
     "type": "git",
@@ -37,20 +37,20 @@
     "debug": "^4.3.4",
     "deepmerge": "^4.3.1",
     "kleur": "^4.1.5",
-    "magic-string": "^0.30.2",
+    "magic-string": "^0.30.3",
     "svelte-hmr": "^0.15.3",
     "vitefu": "^0.2.4",
-    "@sveltejs/vite-plugin-svelte-inspector": "^1.0.3"
+    "@sveltejs/vite-plugin-svelte-inspector": "^2.0.0-next.0"
   },
   "peerDependencies": {
-    "svelte": "^3.54.0 || ^4.0.0",
-    "vite": "^4.0.0"
+    "svelte": "^4.0.0",
+    "vite": "^5.0.0-beta.1 || ^5.0.0"
   },
   "devDependencies": {
     "@types/debug": "^4.1.8",
-    "esbuild": "^0.19.0",
-    "svelte": "^4.1.2",
-    "vite": "^4.4.9"
+    "esbuild": "^0.19.3",
+    "svelte": "^4.2.0",
+    "vite": "^5.0.0-beta.1"
   },
   "scripts": {
     "check:publint": "publint --strict",

package/src/handle-hot-update.js

@@ -14,7 +14,11 @@ import { toRollupError } from './utils/error.js';
 export async function handleHotUpdate(compileSvelte, ctx, svelteRequest, cache, options) {
 	if (!cache.has(svelteRequest)) {
 		// file hasn't been requested yet (e.g. async component)
-		log.debug(`handleHotUpdate called before initial transform for ${svelteRequest.id}`);
+		log.debug(
+			`handleHotUpdate called before initial transform for ${svelteRequest.id}`,
+			undefined,
+			'hmr'
+		);
 		return;
 	}
 	const { read, server, modules } = ctx;
@@ -39,7 +43,7 @@ export async function handleHotUpdate(compileSvelte, ctx, svelteRequest, cache,
 	if (cssIdx > -1) {
 		const cssUpdated = cssChanged(cachedCss, compileData.compiled.css);
 		if (!cssUpdated) {
-			log.debug(`skipping unchanged css for ${svelteRequest.cssId}`);
+			log.debug(`skipping unchanged css for ${svelteRequest.cssId}`, undefined, 'hmr');
 			affectedModules.splice(cssIdx, 1);
 		}
 	}
@@ -47,7 +51,7 @@ export async function handleHotUpdate(compileSvelte, ctx, svelteRequest, cache,
 	if (jsIdx > -1) {
 		const jsUpdated = jsChanged(cachedJS, compileData.compiled.js, svelteRequest.filename);
 		if (!jsUpdated) {
-			log.debug(`skipping unchanged js for ${svelteRequest.id}`);
+			log.debug(`skipping unchanged js for ${svelteRequest.id}`, undefined, 'hmr');
 			affectedModules.splice(jsIdx, 1);
 			// transform won't be called, log warnings here
 			logCompilerWarnings(svelteRequest, compileData.compiled.warnings, options);
@@ -57,14 +61,20 @@ export async function handleHotUpdate(compileSvelte, ctx, svelteRequest, cache,
 	// TODO is this enough? see also: https://github.com/vitejs/vite/issues/2274
 	const ssrModulesToInvalidate = affectedModules.filter((m) => !!m.ssrTransformResult);
 	if (ssrModulesToInvalidate.length > 0) {
-		log.debug(`invalidating modules ${ssrModulesToInvalidate.map((m) => m.id).join(', ')}`);
+		log.debug(
+			`invalidating modules ${ssrModulesToInvalidate.map((m) => m.id).join(', ')}`,
+			undefined,
+			'hmr'
+		);
 		ssrModulesToInvalidate.forEach((moduleNode) => server.moduleGraph.invalidateModule(moduleNode));
 	}
 	if (affectedModules.length > 0) {
 		log.debug(
 			`handleHotUpdate for ${svelteRequest.id} result: ${affectedModules
 				.map((m) => m.id)
-				.join(', ')}`
+				.join(', ')}`,
+			undefined,
+			'hmr'
 		);
 	}
 	return affectedModules;

package/src/index.js

@@ -1,9 +1,7 @@
-import fs from 'fs';
-import { version as viteVersion } from 'vite';
+import fs from 'node:fs';
 
 import { svelteInspector } from '@sveltejs/vite-plugin-svelte-inspector';
 
-import { isDepExcluded } from 'vitefu';
 import { handleHotUpdate } from './handle-hot-update.js';
 import { log, logCompilerWarnings } from './utils/log.js';
 import { createCompileSvelte } from './utils/compile.js';
@@ -17,16 +15,10 @@ import {
 } from './utils/options.js';
 
 import { ensureWatchedFile, setupWatchers } from './utils/watch.js';
-import { resolveViaPackageJsonSvelte } from './utils/resolve.js';
-
 import { toRollupError } from './utils/error.js';
 import { saveSvelteMetadata } from './utils/optimizer.js';
 import { VitePluginSvelteCache } from './utils/vite-plugin-svelte-cache.js';
 import { loadRaw } from './utils/load-raw.js';
-import { FAQ_LINK_CONFLICTS_IN_SVELTE_RESOLVE } from './utils/constants.js';
-import { isSvelte3 } from './utils/svelte-version.js';
-
-const isVite4_0 = viteVersion.startsWith('4.0');
 
 /** @type {import('./index.d.ts').svelte} */
 export function svelte(inlineOptions) {
@@ -42,15 +34,9 @@ export function svelte(inlineOptions) {
 	let options;
 	/** @type {import('vite').ResolvedConfig} */
 	let viteConfig;
-
 	/** @type {import('./types/compile.d.ts').CompileSvelte} */
 	let compileSvelte;
 	/* eslint-enable no-unused-vars */
-
-	/** @type {Promise<import('vite').Rollup.PartialResolvedId | null>} */
-	let resolvedSvelteSSR;
-	/** @type {Set<string>} */
-	let packagesWithResolveWarnings;
 	/** @type {import('./types/plugin-api.d.ts').PluginAPI} */
 	const api = {};
 	/** @type {import('vite').Plugin[]} */
@@ -71,7 +57,7 @@ export function svelte(inlineOptions) {
 				options = await preResolveOptions(inlineOptions, config, configEnv);
 				// extra vite config
 				const extraViteConfig = await buildExtraViteConfig(options, config);
-				log.debug('additional vite config', extraViteConfig);
+				log.debug('additional vite config', extraViteConfig, 'config');
 				return extraViteConfig;
 			},
 
@@ -83,11 +69,10 @@ export function svelte(inlineOptions) {
 				viteConfig = config;
 				// TODO deep clone to avoid mutability from outside?
 				api.options = options;
-				log.debug('resolved options', options);
+				log.debug('resolved options', options, 'config');
 			},
 
 			async buildStart() {
-				packagesWithResolveWarnings = new Set();
 				if (!options.prebundleSvelteLibraries) return;
 				const isSvelteMetadataChanged = await saveSvelteMetadata(viteConfig.cacheDir, options);
 				if (isSvelteMetadataChanged) {
@@ -108,18 +93,24 @@ export function svelte(inlineOptions) {
 				if (svelteRequest) {
 					const { filename, query, raw } = svelteRequest;
 					if (raw) {
-						return loadRaw(svelteRequest, compileSvelte, options);
+						const code = await loadRaw(svelteRequest, compileSvelte, options);
+						// prevent vite from injecting sourcemaps in the results.
+						return {
+							code,
+							map: {
+								mappings: ''
+							}
+						};
 					} else {
 						if (query.svelte && query.type === 'style') {
 							const css = cache.getCSS(svelteRequest);
 							if (css) {
-								log.debug(`load returns css for ${filename}`);
 								return css;
 							}
 						}
 						// prevent vite asset plugin from loading files as url that should be compiled in transform
 						if (viteConfig.assetsInclude(filename)) {
-							log.debug(`load returns raw content for ${filename}`);
+							log.debug(`load returns raw content for ${filename}`, undefined, 'load');
 							return fs.readFileSync(filename, 'utf-8');
 						}
 					}
@@ -133,78 +124,12 @@ export function svelte(inlineOptions) {
 					if (svelteRequest.query.type === 'style' && !svelteRequest.raw) {
 						// return cssId with root prefix so postcss pipeline of vite finds the directory correctly
 						// see https://github.com/sveltejs/vite-plugin-svelte/issues/14
-						log.debug(`resolveId resolved virtual css module ${svelteRequest.cssId}`);
-						return svelteRequest.cssId;
-					}
-				}
-
-				// TODO: remove this after bumping peerDep on Vite to 4.1+ or Svelte to 4.0+
-				if (isVite4_0 && isSvelte3 && ssr && importee === 'svelte') {
-					if (!resolvedSvelteSSR) {
-						resolvedSvelteSSR = this.resolve('svelte/ssr', undefined, { skipSelf: true }).then(
-							(svelteSSR) => {
-								log.debug('resolved svelte to svelte/ssr');
-								return svelteSSR;
-							},
-							(err) => {
-								log.debug(
-									'failed to resolve svelte to svelte/ssr. Update svelte to a version that exports it',
-									err
-								);
-								return null; // returning null here leads to svelte getting resolved regularly
-							}
+						log.debug(
+							`resolveId resolved virtual css module ${svelteRequest.cssId}`,
+							undefined,
+							'resolve'
 						);
-					}
-					return resolvedSvelteSSR;
-				}
-				//@ts-expect-error scan
-				const scan = !!opts?.scan; // scanner phase of optimizeDeps
-				const isPrebundled =
-					options.prebundleSvelteLibraries &&
-					viteConfig.optimizeDeps?.disabled !== true &&
-					viteConfig.optimizeDeps?.disabled !== (options.isBuild ? 'build' : 'dev') &&
-					!isDepExcluded(importee, viteConfig.optimizeDeps?.exclude ?? []);
-				// for prebundled libraries we let vite resolve the prebundling result
-				// for ssr, during scanning and non-prebundled, we do it
-				if (ssr || scan || !isPrebundled) {
-					try {
-						const isFirstResolve = !cache.hasResolvedSvelteField(importee, importer);
-						const resolved = await resolveViaPackageJsonSvelte(importee, importer, cache);
-						if (isFirstResolve && resolved) {
-							const packageInfo = await cache.getPackageInfo(resolved);
-							const packageVersion = `${packageInfo.name}@${packageInfo.version}`;
-							log.debug.once(
-								`resolveId resolved ${importee} to ${resolved} via package.json svelte field of ${packageVersion}`
-							);
-
-							try {
-								const viteResolved = (
-									await this.resolve(importee, importer, { ...opts, skipSelf: true })
-								)?.id;
-								if (resolved !== viteResolved) {
-									packagesWithResolveWarnings.add(packageVersion);
-									log.debug.enabled &&
-										log.debug.once(
-											`resolve difference for ${packageVersion} ${importee} - svelte: "${resolved}", vite: "${viteResolved}"`
-										);
-								}
-							} catch (e) {
-								packagesWithResolveWarnings.add(packageVersion);
-								log.debug.enabled &&
-									log.debug.once(
-										`resolve error for ${packageVersion} ${importee} - svelte: "${resolved}", vite: ERROR`,
-										e
-									);
-							}
-						}
-						return resolved;
-					} catch (e) {
-						log.debug.once(
-							`error trying to resolve ${importee} from ${importer} via package.json svelte field `,
-							e
-						);
-						// this error most likely happens due to non-svelte related importee/importers so swallow it here
-						// in case it really way a svelte library, users will notice anyway. (lib not working due to failed resolve)
+						return svelteRequest.cssId;
 					}
 				}
 			},
@@ -235,7 +160,6 @@ export function svelte(inlineOptions) {
 						}
 					}
 				}
-				log.debug(`transform returns compiled js for ${svelteRequest.filename}`);
 				return {
 					...compileData.compiled.js,
 					meta: {
@@ -257,16 +181,6 @@ export function svelte(inlineOptions) {
 			},
 			async buildEnd() {
 				await options.stats?.finishAll();
-				if (
-					!options.experimental?.disableSvelteResolveWarnings &&
-					packagesWithResolveWarnings?.size > 0
-				) {
-					log.warn(
-						`WARNING: The following packages use a svelte resolve configuration in package.json that has conflicting results and is going to cause problems future.\n\n${[
-							...packagesWithResolveWarnings
-						].join('\n')}\n\nPlease see ${FAQ_LINK_CONFLICTS_IN_SVELTE_RESOLVE} for details.`
-					);
-				}
 			}
 		},
 		svelteInspector()

package/src/types/compile.d.ts

@@ -19,7 +19,7 @@ export interface Compiled {
 	css: Code;
 	ast: any; // TODO type
 	warnings: any[]; // TODO type
-	vars: {
+	vars: Array<{
 		name: string;
 		export_name: string;
 		injected: boolean;
@@ -29,7 +29,7 @@ export interface Compiled {
 		referenced: boolean;
 		writable: boolean;
 		referenced_from_script: boolean;
-	}[];
+	}>;
 	stats: {
 		timings: {
 			total: number;

package/src/utils/compile.js

@@ -9,8 +9,6 @@ import { mapToRelative } from './sourcemaps.js';
 
 const scriptLangRE = /<script [^>]*lang=["']?([^"' >]+)["']?[^>]*>/;
 
-import { isSvelte3 } from './svelte-version.js';
-
 /**
  * @param {Function} [makeHot]
  * @returns {import('../types/compile.d.ts').CompileSvelte}
@@ -55,13 +53,9 @@ export const _createCompileSvelte = (makeHot) => {
 			filename,
 			generate: ssr ? 'ssr' : 'dom'
 		};
-		if (isSvelte3) {
-			// @ts-ignore
-			compileOptions.format = 'esm';
-		}
+
 		if (options.hot && options.emitCss) {
 			const hash = `s-${safeBase64Hash(normalizedFilename)}`;
-			log.debug(`setting cssHash ${hash} for ${normalizedFilename}`);
 			compileOptions.cssHash = () => hash;
 		}
 		if (ssr && compileOptions.enableSourcemap !== false) {
@@ -112,7 +106,9 @@ export const _createCompileSvelte = (makeHot) => {
 		});
 		if (dynamicCompileOptions && log.debug.enabled) {
 			log.debug(
-				`dynamic compile options for  ${filename}: ${JSON.stringify(dynamicCompileOptions)}`
+				`dynamic compile options for  ${filename}: ${JSON.stringify(dynamicCompileOptions)}`,
+				undefined,
+				'compile'
 			);
 		}
 		const finalCompileOptions = dynamicCompileOptions
@@ -125,15 +121,6 @@ export const _createCompileSvelte = (makeHot) => {
 		const endStat = stats?.start(filename);
 		const compiled = compile(finalCode, finalCompileOptions);
 
-		if (isSvelte3) {
-			// prevent dangling pure comments
-			// see https://github.com/sveltejs/kit/issues/9492#issuecomment-1487704985
-			// uses regex replace with whitespace to keep sourcemap/character count unmodified
-			compiled.js.code = compiled.js.code.replace(
-				/\/\* [@#]__PURE__ \*\/(\s*)$/gm,
-				'               $1'
-			);
-		}
 		if (endStat) {
 			endStat();
 		}

package/src/utils/constants.js

@@ -1,5 +1,3 @@
-import { isSvelte3 } from './svelte-version.js';
-
 export const VITE_RESOLVE_MAIN_FIELDS = ['module', 'jsnext:main', 'jsnext'];
 
 export const SVELTE_RESOLVE_MAIN_FIELDS = ['svelte'];
@@ -8,16 +6,13 @@ export const SVELTE_IMPORTS = [
 	'svelte/animate',
 	'svelte/easing',
 	'svelte/internal',
+	'svelte/internal/disclose-version',
 	'svelte/motion',
 	'svelte/ssr',
 	'svelte/store',
 	'svelte/transition',
 	'svelte'
 ];
-// TODO add to global list after dropping svelte 3
-if (!isSvelte3) {
-	SVELTE_IMPORTS.push('svelte/internal/disclose-version');
-}
 
 export const SVELTE_HMR_IMPORTS = [
 	'svelte-hmr/runtime/hot-api-esm.js',
@@ -27,5 +22,5 @@ export const SVELTE_HMR_IMPORTS = [
 
 export const SVELTE_EXPORT_CONDITIONS = ['svelte'];
 
-export const FAQ_LINK_CONFLICTS_IN_SVELTE_RESOLVE =
-	'https://github.com/sveltejs/vite-plugin-svelte/blob/main/docs/faq.md#conflicts-in-svelte-resolve';
+export const FAQ_LINK_MISSING_EXPORTS_CONDITION =
+	'https://github.com/sveltejs/vite-plugin-svelte/blob/main/docs/faq.md#missing-exports-condition';

package/src/utils/dependencies.js

@@ -1,5 +1,5 @@
-import path from 'path';
-import fs from 'fs/promises';
+import path from 'node:path';
+import fs from 'node:fs/promises';
 import { findDepPkgJsonPath } from 'vitefu';
 
 /**

package/src/utils/esbuild.js

@@ -1,8 +1,7 @@
-import { readFileSync } from 'fs';
+import { readFileSync } from 'node:fs';
 import { compile, preprocess } from 'svelte/compiler';
 import { log } from './log.js';
 import { toESBuildError } from './error.js';
-import { isSvelte3 } from './svelte-version.js';
 
 /**
  * @typedef {NonNullable<import('vite').DepOptimizationOptions['esbuildOptions']>} EsbuildOptions
@@ -24,7 +23,7 @@ export function esbuildSveltePlugin(options) {
 			if (build.initialOptions.plugins?.some((v) => v.name === 'vite:dep-scan')) return;
 
 			const svelteExtensions = (options.extensions ?? ['.svelte']).map((ext) => ext.slice(1));
-			const svelteFilter = new RegExp(`\\.(` + svelteExtensions.join('|') + `)(\\?.*)?$`);
+			const svelteFilter = new RegExp('\\.(' + svelteExtensions.join('|') + ')(\\?.*)?$');
 			/** @type {import('../types/vite-plugin-svelte-stats.d.ts').StatCollection | undefined} */
 			let statsCollection;
 			build.onStart(() => {
@@ -67,10 +66,7 @@ async function compileSvelte(options, { filename, code }, statsCollection) {
 		filename,
 		generate: 'dom'
 	};
-	if (isSvelte3) {
-		// @ts-ignore
-		compileOptions.format = 'esm';
-	}
+
 	let preprocessed;
 
 	if (options.preprocess) {
@@ -106,5 +102,7 @@ async function compileSvelte(options, { filename, code }, statsCollection) {
 	if (endStat) {
 		endStat();
 	}
-	return compiled.js.code + '//# sourceMappingURL=' + compiled.js.map.toUrl();
+	return compiled.js.map
+		? compiled.js.code + '//# sourceMappingURL=' + compiled.js.map.toUrl()
+		: compiled.js.code;
 }

package/src/utils/hash.js

@@ -1,4 +1,4 @@
-import * as crypto from 'crypto';
+import * as crypto from 'node:crypto';
 
 const hashes = Object.create(null);
 

package/src/utils/id.js

@@ -1,5 +1,5 @@
 import { createFilter, normalizePath } from 'vite';
-import * as fs from 'fs';
+import * as fs from 'node:fs';
 import { log } from './log.js';
 
 const VITE_FS_PREFIX = '/@fs/';
@@ -21,7 +21,7 @@ const TYPES_WITH_COMPILER_OPTIONS = ['style', 'script', 'all'];
  * @returns {{ filename: string, rawQuery: string }}
  */
 function splitId(id) {
-	const parts = id.split(`?`, 2);
+	const parts = id.split('?', 2);
 	const filename = parts[0];
 	const rawQuery = parts[1];
 	return { filename, rawQuery };

package/src/utils/load-raw.js

@@ -1,4 +1,4 @@
-import fs from 'fs';
+import fs from 'node:fs';
 import { toRollupError } from './error.js';
 import { log } from './log.js';
 
@@ -64,7 +64,7 @@ export async function loadRaw(svelteRequest, compileSvelte, options) {
 				}" combined with direct in ${id}. supported are: ${supportedDirectTypes.join(', ')}`
 			);
 		}
-		log.debug(`load returns direct result for ${id}`);
+		log.debug(`load returns direct result for ${id}`, undefined, 'load');
 		let directOutput = result.code;
 		if (query.sourcemap && result.map?.toUrl) {
 			const map = `sourceMappingURL=${result.map.toUrl()}`;
@@ -76,7 +76,7 @@ export async function loadRaw(svelteRequest, compileSvelte, options) {
 		}
 		return directOutput;
 	} else if (query.raw) {
-		log.debug(`load returns raw result for ${id}`);
+		log.debug(`load returns raw result for ${id}`, undefined, 'load');
 		return toRawExports(result);
 	} else {
 		throw new Error(`invalid raw mode in ${id}, supported are raw, direct`);
@@ -126,7 +126,7 @@ function toRawExports(object) {
 			.map(([key, value]) => `export const ${key}=${JSON.stringify(value)}`)
 			.join('\n') + '\n';
 	if (Object.prototype.hasOwnProperty.call(object, 'code')) {
-		exports += `export default code\n`;
+		exports += 'export default code\n';
 	}
 	return exports;
 }

package/src/utils/load-svelte-config.js

@@ -1,7 +1,7 @@
-import { createRequire } from 'module';
-import path from 'path';
-import fs from 'fs';
-import { pathToFileURL } from 'url';
+import { createRequire } from 'node:module';
+import path from 'node:path';
+import fs from 'node:fs';
+import { pathToFileURL } from 'node:url';
 import { log } from './log.js';
 
 // used to require cjs config in esm.
@@ -104,7 +104,7 @@ function findConfigToLoad(viteConfig, inlineOptions) {
 			.map((candidate) => path.resolve(root, candidate))
 			.filter((file) => fs.existsSync(file));
 		if (existingKnownConfigFiles.length === 0) {
-			log.debug(`no svelte config found at ${root}`);
+			log.debug(`no svelte config found at ${root}`, undefined, 'config');
 			return;
 		} else if (existingKnownConfigFiles.length > 1) {
 			log.warn(

package/src/utils/log.js

@@ -8,7 +8,7 @@ const prefix = 'vite-plugin-svelte';
 /** @type {Record<import('../types/log.d.ts').LogLevel, any>} */
 const loggers = {
 	debug: {
-		log: debug(`vite:${prefix}`),
+		log: debug(`${prefix}`),
 		enabled: false,
 		isDebug: true
 	},
@@ -65,7 +65,13 @@ function _log(logger, message, payload, namespace) {
 		return;
 	}
 	if (logger.isDebug) {
-		const log = namespace ? logger.log.extend(namespace) : logger.log;
+		let log = logger.log;
+		if (namespace) {
+			if (!isDebugNamespaceEnabled(namespace)) {
+				return;
+			}
+			log = logger.log.extend(namespace);
+		}
 		payload !== undefined ? log(message, payload) : log(message);
 	} else {
 		logger.log(
@@ -204,7 +210,8 @@ function buildExtraWarnings(warnings, isBuild) {
 			extraWarnings.push({
 				...noScopableElementWarning,
 				code: 'vite-plugin-svelte-css-no-scopable-elements',
-				message: `No scopable elements found in template. If you're using global styles in the style tag, you should move it into an external stylesheet file and import it in JS. See https://github.com/sveltejs/vite-plugin-svelte/blob/main/docs/faq.md#where-should-i-put-my-global-styles.`
+				message:
+					"No scopable elements found in template. If you're using global styles in the style tag, you should move it into an external stylesheet file and import it in JS. See https://github.com/sveltejs/vite-plugin-svelte/blob/main/docs/faq.md#where-should-i-put-my-global-styles."
 			});
 		}
 	}
@@ -250,5 +257,5 @@ export function buildExtendedLogMessage(w) {
  * @returns {boolean}
  */
 export function isDebugNamespaceEnabled(namespace) {
-	return debug.enabled(`vite:${prefix}:${namespace}`);
+	return debug.enabled(`${prefix}:${namespace}`);
 }

package/src/utils/optimizer.js

@@ -1,5 +1,5 @@
-import { promises as fs } from 'fs';
-import path from 'path';
+import { promises as fs } from 'node:fs';
+import path from 'node:path';
 
 // List of options that changes the prebundling result
 /** @type {(keyof import('../types/options.d.ts').ResolvedOptions)[]} */

package/src/utils/options.js

@@ -3,6 +3,7 @@ import { normalizePath } from 'vite';
 import { isDebugNamespaceEnabled, log } from './log.js';
 import { loadSvelteConfig } from './load-svelte-config.js';
 import {
+	FAQ_LINK_MISSING_EXPORTS_CONDITION,
 	SVELTE_EXPORT_CONDITIONS,
 	SVELTE_HMR_IMPORTS,
 	SVELTE_IMPORTS,
@@ -10,7 +11,7 @@ import {
 	VITE_RESOLVE_MAIN_FIELDS
 } from './constants.js';
 
-import path from 'path';
+import path from 'node:path';
 import { esbuildSveltePlugin, facadeEsbuildSveltePluginName } from './esbuild.js';
 import { addExtraPreprocessors } from './preprocess.js';
 import deepmerge from 'deepmerge';
@@ -428,7 +429,7 @@ export async function buildExtraViteConfig(options, config) {
 			(options.hot && options.hot.partialAccept !== false)) && // deviate from svelte-hmr, default to true
 		config.experimental?.hmrPartialAccept !== false
 	) {
-		log.debug('enabling "experimental.hmrPartialAccept" in vite config');
+		log.debug('enabling "experimental.hmrPartialAccept" in vite config', undefined, 'config');
 		extraViteConfig.experimental = { hmrPartialAccept: true };
 	}
 	validateViteConfig(extraViteConfig, config, options);
@@ -481,6 +482,7 @@ function validateViteConfig(extraViteConfig, config, options) {
  */
 async function buildExtraConfigForDependencies(options, config) {
 	// extra handling for svelte dependencies in the project
+	const packagesWithoutSvelteExportsCondition = new Set();
 	const depsConfig = await crawlFrameworkPkgs({
 		root: options.root,
 		isBuild: options.isBuild,
@@ -496,7 +498,11 @@ async function buildExtraConfigForDependencies(options, config) {
 					return value;
 				});
 			}
-			return hasSvelteCondition || !!pkgJson.svelte;
+			const hasSvelteField = !!pkgJson.svelte;
+			if (hasSvelteField && !hasSvelteCondition) {
+				packagesWithoutSvelteExportsCondition.add(`${pkgJson.name}@${pkgJson.version}`);
+			}
+			return hasSvelteCondition || hasSvelteField;
 		},
 		isSemiFrameworkPkgByJson(pkgJson) {
 			return !!pkgJson.dependencies?.svelte || !!pkgJson.peerDependencies?.svelte;
@@ -510,8 +516,17 @@ async function buildExtraConfigForDependencies(options, config) {
 			}
 		}
 	});
-
-	log.debug('extra config for dependencies generated by vitefu', depsConfig);
+	if (
+		!options.experimental?.disableSvelteResolveWarnings &&
+		packagesWithoutSvelteExportsCondition?.size > 0
+	) {
+		log.warn(
+			`WARNING: The following packages have a svelte field in their package.json but no exports condition for svelte.\n\n${[
+				...packagesWithoutSvelteExportsCondition
+			].join('\n')}\n\nPlease see ${FAQ_LINK_MISSING_EXPORTS_CONDITION} for details.`
+		);
+	}
+	log.debug('extra config for dependencies generated by vitefu', depsConfig, 'config');
 
 	if (options.prebundleSvelteLibraries) {
 		// prebundling enabled, so we don't need extra dependency excludes
@@ -545,7 +560,7 @@ async function buildExtraConfigForDependencies(options, config) {
 		});
 	}
 
-	log.debug('post-processed extra config for dependencies', depsConfig);
+	log.debug('post-processed extra config for dependencies', depsConfig, 'config');
 
 	return depsConfig;
 }
@@ -562,11 +577,17 @@ function buildExtraConfigForSvelte(config) {
 	if (!isDepExcluded('svelte', config.optimizeDeps?.exclude ?? [])) {
 		const svelteImportsToInclude = SVELTE_IMPORTS.filter((x) => x !== 'svelte/ssr'); // not used on clientside
 		log.debug(
-			`adding bare svelte packages to optimizeDeps.include: ${svelteImportsToInclude.join(', ')} `
+			`adding bare svelte packages to optimizeDeps.include: ${svelteImportsToInclude.join(', ')} `,
+			undefined,
+			'config'
 		);
 		include.push(...svelteImportsToInclude);
 	} else {
-		log.debug('"svelte" is excluded in optimizeDeps.exclude, skipped adding it to include.');
+		log.debug(
+			'"svelte" is excluded in optimizeDeps.exclude, skipped adding it to include.',
+			undefined,
+			'config'
+		);
 	}
 	/** @type {(string | RegExp)[]} */
 	const noExternal = [];

package/src/utils/preprocess.js

@@ -1,6 +1,6 @@
 import MagicString from 'magic-string';
 import { log } from './log.js';
-import path from 'path';
+import path from 'node:path';
 
 /**
  * this appends a *{} rule to component styles to force the svelte compiler to add style classes to all nodes
@@ -84,14 +84,18 @@ function buildExtraPreprocessors(options, config) {
 		log.debug(
 			`Ignoring svelte preprocessors defined by these vite plugins: ${ignored
 				.map((p) => p.name)
-				.join(', ')}`
+				.join(', ')}`,
+			undefined,
+			'preprocess'
 		);
 	}
 	if (included.length > 0) {
 		log.debug(
 			`Adding svelte preprocessors defined by these vite plugins: ${included
 				.map((p) => p.name)
-				.join(', ')}`
+				.join(', ')}`,
+			undefined,
+			'preprocess'
 		);
 		appendPreprocessors.push(...pluginsWithPreprocessors.map((p) => p.api.sveltePreprocess));
 	}

package/src/utils/sourcemaps.js

@@ -1,4 +1,4 @@
-import path from 'path';
+import path from 'node:path';
 
 const IS_WINDOWS = process.platform === 'win32';
 

package/src/utils/vite-plugin-svelte-cache.js

@@ -1,5 +1,5 @@
-import { readFileSync } from 'fs';
-import { dirname } from 'path';
+import { readFileSync } from 'node:fs';
+import { dirname } from 'node:path';
 import { findClosestPkgJsonPath } from 'vitefu';
 import { normalizePath } from 'vite';
 
@@ -24,8 +24,6 @@ export class VitePluginSvelteCache {
 	#dependencies = new Map();
 	/** @type {Map<string, Set<string>>} */
 	#dependants = new Map();
-	/** @type {Map<string, string>} */
-	#resolvedSvelteFields = new Map();
 	/** @type {Map<string, any>} */
 	#errors = new Map();
 	/** @type {PackageInfo[]} */
@@ -168,45 +166,6 @@ export class VitePluginSvelteCache {
 		return dependants ? [...dependants] : [];
 	}
 
-	/**
-	 * @param {string} name
-	 * @param {string} [importer]
-	 * @returns {string|void}
-	 */
-	getResolvedSvelteField(name, importer) {
-		return this.#resolvedSvelteFields.get(this.#getResolvedSvelteFieldKey(name, importer));
-	}
-
-	/**
-	 * @param {string} name
-	 * @param {string} [importer]
-	 * @returns {boolean}
-	 */
-	hasResolvedSvelteField(name, importer) {
-		return this.#resolvedSvelteFields.has(this.#getResolvedSvelteFieldKey(name, importer));
-	}
-	/**
-	 *
-	 * @param {string} importee
-	 * @param {string | undefined} importer
-	 * @param {string} resolvedSvelte
-	 */
-	setResolvedSvelteField(importee, importer, resolvedSvelte) {
-		this.#resolvedSvelteFields.set(
-			this.#getResolvedSvelteFieldKey(importee, importer),
-			resolvedSvelte
-		);
-	}
-
-	/**
-	 * @param {string} importee
-	 * @param {string | undefined} importer
-	 * @returns {string}
-	 */
-	#getResolvedSvelteFieldKey(importee, importer) {
-		return importer ? `${importer} > ${importee}` : importee;
-	}
-
 	/**
 	 * @param {string} file
 	 * @returns {Promise<PackageInfo>}

package/src/utils/vite-plugin-svelte-stats.js

@@ -1,5 +1,5 @@
 import { log } from './log.js';
-import { performance } from 'perf_hooks';
+import { performance } from 'node:perf_hooks';
 import { normalizePath } from 'vite';
 
 /** @type {import('../types/vite-plugin-svelte-stats.d.ts').CollectionOptions} */

package/src/utils/watch.js

@@ -1,7 +1,7 @@
-import fs from 'fs';
+import fs from 'node:fs';
 import { log } from './log.js';
 import { knownSvelteConfigNames } from './load-svelte-config.js';
-import path from 'path';
+import path from 'node:path';
 
 /**
  * @param {import('../types/options.d.ts').ResolvedOptions} options
@@ -22,7 +22,9 @@ export function setupWatchers(options, cache, requestParser) {
 		dependants.forEach((dependant) => {
 			if (fs.existsSync(dependant)) {
 				log.debug(
-					`emitting virtual change event for "${dependant}" because depdendency "${filename}" changed`
+					`emitting virtual change event for "${dependant}" because depdendency "${filename}" changed`,
+					undefined,
+					'hmr'
 				);
 				watcher.emit('change', dependant);
 			}
@@ -34,7 +36,7 @@ export function setupWatchers(options, cache, requestParser) {
 		if (svelteRequest) {
 			const removedFromCache = cache.remove(svelteRequest);
 			if (removedFromCache) {
-				log.debug(`cleared VitePluginSvelteCache for deleted file ${filename}`);
+				log.debug(`cleared VitePluginSvelteCache for deleted file ${filename}`, undefined, 'hmr');
 			}
 		}
 	};

package/src/utils/resolve.js

@@ -1,66 +0,0 @@
-import path from 'path';
-import { builtinModules } from 'module';
-import { resolveDependencyData, isCommonDepWithoutSvelteField } from './dependencies.js';
-import { normalizePath } from 'vite';
-
-/**
- * @param {string} importee
- * @param {string | undefined} importer
- * @param {import('./vite-plugin-svelte-cache').VitePluginSvelteCache} cache
- * @returns {Promise<string | void>}
- */
-export async function resolveViaPackageJsonSvelte(importee, importer, cache) {
-	if (
-		importer &&
-		isBareImport(importee) &&
-		!isNodeInternal(importee) &&
-		!isCommonDepWithoutSvelteField(importee)
-	) {
-		const cached = cache.getResolvedSvelteField(importee, importer);
-		if (cached) {
-			return cached;
-		}
-		const pkgData = await resolveDependencyData(importee, importer);
-		if (pkgData) {
-			const { pkg, dir } = pkgData;
-			if (pkg.svelte) {
-				const result = normalizePath(path.resolve(dir, pkg.svelte));
-				cache.setResolvedSvelteField(importee, importer, result);
-				return result;
-			}
-		}
-	}
-}
-
-/**
- * @param {string} importee
- * @returns {boolean}
- */
-function isNodeInternal(importee) {
-	return importee.startsWith('node:') || builtinModules.includes(importee);
-}
-
-/**
- * @param {string} importee
- * @returns {boolean}
- */
-function isBareImport(importee) {
-	if (
-		!importee ||
-		importee[0] === '.' ||
-		importee[0] === '\0' ||
-		importee.includes(':') ||
-		path.isAbsolute(importee)
-	) {
-		return false;
-	}
-	const parts = importee.split('/');
-	switch (parts.length) {
-		case 1:
-			return true;
-		case 2:
-			return parts[0].startsWith('@');
-		default:
-			return false;
-	}
-}