@sveltejs/vite-plugin-svelte 6.2.0 → 6.2.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sveltejs/vite-plugin-svelte",
-  "version": "6.2.0",
+  "version": "6.2.1",
   "license": "MIT",
   "author": "dominikg",
   "files": [

package/src/plugins/configure.js

@@ -12,7 +12,6 @@ import {
 } from '../utils/options.js';
 import { buildIdFilter, buildIdParser } from '../utils/id.js';
 import { createCompileSvelte } from '../utils/compile.js';
-import { gte } from '../utils/svelte-version.js';
 
 // @ts-ignore rolldownVersion
 const { version: viteVersion, rolldownVersion } = vite;
@@ -59,39 +58,7 @@ export function configure(api, inlineOptions) {
 				preOptions = await preResolveOptions(inlineOptions, config, configEnv);
 				// extra vite config
 				const extraViteConfig = await buildExtraViteConfig(preOptions, config);
-
-				if (
-					rolldownVersion &&
-					configEnv.command === 'build' &&
-					gte(rolldownVersion, '1.0.0-beta.35') // inlineConst received a critical bugfix in 1.0.0-beta.35
-				) {
-					extraViteConfig.build ??= {};
-					// rename rollupOptions to rolldownOptions
-					//@ts-ignore rolldownOptions only exists in rolldown-vite
-					extraViteConfig.build.rolldownOptions = extraViteConfig.build.rollupOptions || {};
-					delete extraViteConfig.build.rollupOptions;
-					// read user config inlineConst value
-					const inlineConst =
-						//@ts-ignore optimization only exists in rolldown-vite
-						config.build?.rolldownOptions?.optimization?.inlineConst ??
-						//@ts-ignore optimization only exists in rolldown-vite
-						config.build?.rollupOptions?.optimization?.inlineConst;
-
-					if (inlineConst == null) {
-						// set inlineConst build optimization for esm-env
-						//@ts-ignore rolldownOptions only exists in rolldown-vite
-						extraViteConfig.build.rolldownOptions.optimization ??= {};
-						//@ts-ignore rolldownOptions only exists in rolldown-vite
-						extraViteConfig.build.rolldownOptions.optimization.inlineConst = true;
-					} else if (inlineConst === false) {
-						log.warn(
-							'Your rolldown config contains `optimization.inlineConst: false`. This can lead to increased bundle size and leaked server code in client build.'
-						);
-					}
-				}
-
 				log.debug('additional vite config', extraViteConfig, 'config');
-
 				return extraViteConfig;
 			}
 		},

package/src/utils/log.js

@@ -1,5 +1,6 @@
 /* eslint-disable no-console */
 
+// eslint-disable-next-line n/no-unsupported-features/node-builtins
 import { styleText } from 'node:util';
 const cyan = (/** @type {string} */ txt) => styleText('cyan', txt);
 const yellow = (/** @type {string} */ txt) => styleText('yellow', txt);
@@ -145,9 +146,7 @@ export function logCompilerWarnings(svelteRequest, warnings, options) {
 	let warn = isBuild ? warnBuild : warnDev;
 	/** @type {import('svelte/compiler').Warning[]} */
 	const handledByDefaultWarn = [];
-	const notIgnored = warnings?.filter((w) => !ignoreCompilerWarning(w, isBuild, emitCss));
-	const extra = buildExtraWarnings(warnings, isBuild);
-	const allWarnings = [...notIgnored, ...extra];
+	const allWarnings = warnings?.filter((w) => !ignoreCompilerWarning(w, isBuild, emitCss));
 	if (sendViaWS) {
 		const _warn = warn;
 		/** @type {(w: import('svelte/compiler').Warning) => void} */
@@ -202,31 +201,6 @@ function isNoScopableElementWarning(warning) {
 	return warning.code === 'css_unused_selector' && warning.message.includes('"*"');
 }
 
-/**
- *
- * @param {import('svelte/compiler').Warning[]} warnings
- * @param {boolean} isBuild
- * @returns {import('svelte/compiler').Warning[]}
- */
-function buildExtraWarnings(warnings, isBuild) {
-	const extraWarnings = [];
-	if (!isBuild) {
-		const noScopableElementWarnings = warnings.filter((w) => isNoScopableElementWarning(w));
-		if (noScopableElementWarnings.length > 0) {
-			// in case there are multiple, use last one as that is the one caused by our *{} rule
-			const noScopableElementWarning =
-				noScopableElementWarnings[noScopableElementWarnings.length - 1];
-			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."
-			});
-		}
-	}
-	return extraWarnings;
-}
-
 /**
  * @param {import('svelte/compiler').Warning} w
  */

package/src/utils/options.js

@@ -6,7 +6,9 @@ const {
 	defaultClientConditions,
 	defaultServerConditions,
 	normalizePath,
-	searchForWorkspaceRoot
+	searchForWorkspaceRoot,
+	// @ts-ignore
+	rolldownVersion
 } = vite;
 import { log } from './log.js';
 import { loadSvelteConfig } from './load-svelte-config.js';
@@ -418,6 +420,20 @@ function validateViteConfig(extraViteConfig, config, options) {
 			);
 		}
 	}
+	if (rolldownVersion && isBuild) {
+		// read user config inlineConst value
+		const inlineConst =
+			//@ts-ignore optimization only exists in rolldown-vite
+			config.build?.rolldownOptions?.optimization?.inlineConst ??
+			//@ts-ignore optimization only exists in rolldown-vite
+			config.build?.rollupOptions?.optimization?.inlineConst;
+
+		if (inlineConst === false) {
+			log.warn(
+				'Your rolldown config contains `optimization.inlineConst: false`. This can lead to increased bundle size and leaked server code in client build.'
+			);
+		}
+	}
 }
 
 /**