feat!: vite 6 support (#1020)

bluwy · web-flow · commit e3344b471004 · 2024-11-21T23:17:27.000+08:00
diff --git a/.changeset/itchy-lions-whisper.md b/.changeset/itchy-lions-whisper.md
@@ -0,0 +1,5 @@
+---
+'@sveltejs/vite-plugin-svelte': minor
+---
+
+Add `esm-env` to `ssr.noExternal` by default to resolve its conditions with Vite
diff --git a/.changeset/light-phones-happen.md b/.changeset/light-phones-happen.md
@@ -0,0 +1,5 @@
+---
+'@sveltejs/vite-plugin-svelte': major
+---
+
+Handle Vite 6 breaking change and remove Vite 5 handling
diff --git a/package.json b/package.json
@@ -53,7 +53,7 @@
     "svelte-eslint-parser": "^0.43.0",
     "typescript": "^5.6.3",
     "typescript-eslint": "^8.15.0",
-    "vite": "^5.4.11",
+    "vite": "^6.0.0-beta.10",
     "vitest": "^2.1.5"
   },
   "lint-staged": {
diff --git a/packages/e2e-tests/autoprefixer-browerslist/package.json b/packages/e2e-tests/autoprefixer-browerslist/package.json
@@ -17,7 +17,7 @@
     "postcss-load-config": "^6.0.1",
     "svelte": "^5.2.4",
     "svelte-preprocess": "^6.0.3",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   },
   "type": "module"
 }
diff --git a/packages/e2e-tests/configfile-custom/package.json b/packages/e2e-tests/configfile-custom/package.json
@@ -13,7 +13,7 @@
   "devDependencies": {
     "@sveltejs/vite-plugin-svelte": "workspace:^",
     "svelte": "^5.2.4",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   },
   "type": "module"
 }
diff --git a/packages/e2e-tests/configfile-esm/package.json b/packages/e2e-tests/configfile-esm/package.json
@@ -14,7 +14,7 @@
     "@sveltejs/vite-plugin-svelte": "workspace:^",
     "svelte": "^5.2.4",
     "svelte-preprocess": "^6.0.3",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   },
   "type": "module"
 }
diff --git a/packages/e2e-tests/css-dev-sourcemap/package.json b/packages/e2e-tests/css-dev-sourcemap/package.json
@@ -12,6 +12,6 @@
     "@sveltejs/vite-plugin-svelte": "workspace:^",
     "sass": "^1.81.0",
     "svelte": "^5.2.4",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   }
 }
diff --git a/packages/e2e-tests/custom-extensions/package.json b/packages/e2e-tests/custom-extensions/package.json
@@ -10,7 +10,7 @@
   "devDependencies": {
     "@sveltejs/vite-plugin-svelte": "workspace:^",
     "svelte": "^5.2.4",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   },
   "type": "module"
 }
diff --git a/packages/e2e-tests/dynamic-compile-options/package.json b/packages/e2e-tests/dynamic-compile-options/package.json
@@ -10,7 +10,7 @@
   "devDependencies": {
     "@sveltejs/vite-plugin-svelte": "workspace:^",
     "svelte": "^5.2.4",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   },
   "type": "module"
 }
diff --git a/packages/e2e-tests/env/package.json b/packages/e2e-tests/env/package.json
@@ -10,7 +10,7 @@
   "devDependencies": {
     "@sveltejs/vite-plugin-svelte": "workspace:^",
     "svelte": "^5.2.4",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   },
   "type": "module"
 }
diff --git a/packages/e2e-tests/hmr/package.json b/packages/e2e-tests/hmr/package.json
@@ -15,7 +15,7 @@
     "e2e-test-dep-vite-plugins": "file:../_test_dependencies/vite-plugins",
     "node-fetch": "^3.3.2",
     "svelte": "^5.2.4",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   },
   "type": "module"
 }
diff --git a/packages/e2e-tests/import-queries/package.json b/packages/e2e-tests/import-queries/package.json
@@ -12,6 +12,6 @@
     "@sveltejs/vite-plugin-svelte": "workspace:^",
     "sass": "^1.81.0",
     "svelte": "^5.2.4",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   }
 }
diff --git a/packages/e2e-tests/inspector-kit/package.json b/packages/e2e-tests/inspector-kit/package.json
@@ -11,7 +11,7 @@
     "@sveltejs/kit": "^2.8.1",
     "@sveltejs/vite-plugin-svelte": "workspace:^",
     "svelte": "^5.2.4",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   },
   "type": "module"
 }
diff --git a/packages/e2e-tests/inspector-vite/package.json b/packages/e2e-tests/inspector-vite/package.json
@@ -11,6 +11,6 @@
   "devDependencies": {
     "@sveltejs/vite-plugin-svelte": "workspace:^",
     "svelte": "^5.2.4",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   }
 }
diff --git a/packages/e2e-tests/kit-node/__tests__/kit.spec.ts b/packages/e2e-tests/kit-node/__tests__/kit.spec.ts
@@ -16,6 +16,7 @@ import {
 
 import glob from 'tiny-glob';
 import path from 'node:path';
+import { defaultClientConditions, defaultClientMainFields } from 'vite';
 import { describe, expect, it } from 'vitest';
 
 describe('kit-node', () => {
@@ -285,7 +286,8 @@ describe('kit-node', () => {
 						'/^svelte\\//', // serialized with toString
 						'e2e-test-dep-svelte-api-only',
 						'svelte-i18n',
-						'esm-env',
+						'esm-env', // first added by svelte-kit
+						'esm-env', // second added by vite-plugin-svelte
 						'@sveltejs/kit'
 					],
 					`ssr.noExternal in ${filename}`
@@ -346,12 +348,12 @@ describe('kit-node', () => {
 				expectArrayEqual(config.resolve.dedupe, expectedDedupe, `resolve.dedupe in ${filename}`);
 				expectArrayEqual(
 					config.resolve.mainFields,
-					['svelte', 'browser', 'module', 'jsnext:main', 'jsnext'],
+					['svelte', ...defaultClientMainFields],
 					`resolve.mainFields in ${filename}`
 				);
 				expectArrayEqual(
 					config.resolve.conditions,
-					['svelte'],
+					[...defaultClientConditions, 'svelte'],
 					`resolve.conditions in ${filename}`
 				);
 			}
diff --git a/packages/e2e-tests/kit-node/package.json b/packages/e2e-tests/kit-node/package.json
@@ -22,7 +22,7 @@
     "svelte-i18n": "^4.0.1",
     "tiny-glob": "^0.2.9",
     "typescript": "^5.6.3",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   },
   "type": "module"
 }
diff --git a/packages/e2e-tests/package-json-svelte-field/package.json b/packages/e2e-tests/package-json-svelte-field/package.json
@@ -15,7 +15,7 @@
     "@sveltejs/vite-plugin-svelte": "workspace:^",
     "sass": "^1.81.0",
     "svelte": "^5.2.4",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   },
   "type": "module"
 }
diff --git a/packages/e2e-tests/prebundle-svelte-deps/package.json b/packages/e2e-tests/prebundle-svelte-deps/package.json
@@ -12,15 +12,15 @@
     "e2e-test-dep-svelte-api-only": "file:../_test_dependencies/svelte-api-only",
     "e2e-test-dep-svelte-exports-simple": "file:../_test_dependencies/svelte-exports-simple",
     "e2e-test-dep-svelte-hybrid": "file:../_test_dependencies/svelte-hybrid",
+    "e2e-test-dep-svelte-module": "file:../_test_dependencies/svelte-module",
     "e2e-test-dep-svelte-nested": "file:../_test_dependencies/svelte-nested",
-    "e2e-test-dep-svelte-simple": "file:../_test_dependencies/svelte-simple",
-    "e2e-test-dep-svelte-module": "file:../_test_dependencies/svelte-module"
+    "e2e-test-dep-svelte-simple": "file:../_test_dependencies/svelte-simple"
   },
   "devDependencies": {
     "@sveltejs/vite-plugin-svelte": "workspace:^",
     "sass": "^1.81.0",
     "svelte": "^5.2.4",
     "svelte-preprocess": "^6.0.3",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   }
 }
diff --git a/packages/e2e-tests/preprocess-with-vite/package.json b/packages/e2e-tests/preprocess-with-vite/package.json
@@ -12,7 +12,7 @@
     "sass": "^1.81.0",
     "stylus": "^0.64.0",
     "svelte": "^5.2.4",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   },
   "type": "module"
 }
diff --git a/packages/e2e-tests/resolve-exports-svelte/package.json b/packages/e2e-tests/resolve-exports-svelte/package.json
@@ -14,6 +14,6 @@
   "devDependencies": {
     "@sveltejs/vite-plugin-svelte": "workspace:^",
     "svelte": "^5.2.4",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   }
 }
diff --git a/packages/e2e-tests/scan-deps/package.json b/packages/e2e-tests/scan-deps/package.json
@@ -11,6 +11,6 @@
   "devDependencies": {
     "@sveltejs/vite-plugin-svelte": "workspace:^",
     "svelte": "^5.2.4",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   }
 }
diff --git a/packages/e2e-tests/svelte-preprocess/package.json b/packages/e2e-tests/svelte-preprocess/package.json
@@ -13,7 +13,7 @@
     "svelte": "^5.2.4",
     "svelte-preprocess": "^6.0.3",
     "typescript": "^5.6.3",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   },
   "type": "module"
 }
diff --git a/packages/e2e-tests/ts-type-import/package.json b/packages/e2e-tests/ts-type-import/package.json
@@ -13,7 +13,7 @@
     "@types/node": "^22.9.1",
     "svelte": "^5.2.4",
     "svelte-preprocess": "^6.0.3",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   },
   "type": "module"
 }
diff --git a/packages/e2e-tests/vite-ssr-esm/package.json b/packages/e2e-tests/vite-ssr-esm/package.json
@@ -21,6 +21,6 @@
     "npm-run-all2": "^7.0.1",
     "serve-static": "^1.16.2",
     "svelte": "^5.2.4",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   }
 }
diff --git a/packages/playground/basic/package.json b/packages/playground/basic/package.json
@@ -12,6 +12,6 @@
     "@sveltejs/vite-plugin-svelte": "workspace:^",
     "e2e-test-dep-svelte-module": "workspace:^",
     "svelte": "^5.2.4",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   }
 }
diff --git a/packages/playground/big-component-library-kit/package.json b/packages/playground/big-component-library-kit/package.json
@@ -21,7 +21,7 @@
     "svelte-check": "^4.0.9",
     "svelte-preprocess": "^6.0.3",
     "typescript": "^5.6.3",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   },
   "type": "module"
 }
diff --git a/packages/playground/big-component-library-vite-ssr/package.json b/packages/playground/big-component-library-vite-ssr/package.json
@@ -24,6 +24,6 @@
     "lodash-es": "^4.17.21",
     "svelte": "^5.2.4",
     "svelte-preprocess": "^6.0.3",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   }
 }
diff --git a/packages/playground/big-component-library/package.json b/packages/playground/big-component-library/package.json
@@ -15,7 +15,7 @@
     "carbon-preprocess-svelte": "^0.11.7",
     "svelte": "^5.2.4",
     "svelte-preprocess": "^6.0.3",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   },
   "dependencies": {
     "lodash-es": "^4.17.21"
diff --git a/packages/playground/big/package.json b/packages/playground/big/package.json
@@ -11,6 +11,6 @@
   "devDependencies": {
     "@sveltejs/vite-plugin-svelte": "workspace:^",
     "svelte": "^5.2.4",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   }
 }
diff --git a/packages/playground/kit-demo-app/package.json b/packages/playground/kit-demo-app/package.json
@@ -18,7 +18,7 @@
     "svelte": "^5.2.4",
     "svelte-check": "^4.0.9",
     "typescript": "^5.6.3",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   },
   "type": "module"
 }
diff --git a/packages/playground/optimizedeps-include/package.json b/packages/playground/optimizedeps-include/package.json
@@ -12,6 +12,6 @@
     "@sveltejs/vite-plugin-svelte": "workspace:^",
     "svelte": "^5.2.4",
     "tinro": "^0.6.12",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   }
 }
diff --git a/packages/vite-plugin-svelte-inspector/package.json b/packages/vite-plugin-svelte-inspector/package.json
@@ -51,6 +51,6 @@
   "devDependencies": {
     "@types/debug": "^4.1.12",
     "svelte": "^5.2.4",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   }
 }
diff --git a/packages/vite-plugin-svelte/package.json b/packages/vite-plugin-svelte/package.json
@@ -57,6 +57,6 @@
     "esbuild": "^0.24.0",
     "sass": "^1.81.0",
     "svelte": "^5.2.4",
-    "vite": "^5.4.11"
+    "vite": "^6.0.0-beta.10"
   }
 }
diff --git a/packages/vite-plugin-svelte/src/index.js b/packages/vite-plugin-svelte/src/index.js
@@ -10,7 +10,9 @@ import {
 	validateInlineOptions,
 	resolveOptions,
 	patchResolvedViteConfig,
-	preResolveOptions
+	preResolveOptions,
+	ensureConfigEnvironmentMainFields,
+	ensureConfigEnvironmentConditions
 } from './utils/options.js';
 import { ensureWatchedFile, setupWatchers } from './utils/watch.js';
 import { toRollupError } from './utils/error.js';
@@ -64,6 +66,16 @@ export function svelte(inlineOptions) {
 				return extraViteConfig;
 			},
 
+			configEnvironment(name, config, opts) {
+				ensureConfigEnvironmentMainFields(name, config, opts);
+				// @ts-expect-error the function above should make `resolve.mainFields` non-nullable
+				config.resolve.mainFields.unshift('svelte');
+
+				ensureConfigEnvironmentConditions(name, config, opts);
+				// @ts-expect-error the function above should make `resolve.conditions` non-nullable
+				config.resolve.conditions.push('svelte');
+			},
+
 			async configResolved(config) {
 				options = resolveOptions(options, config, cache);
 				patchResolvedViteConfig(config, options);
diff --git a/packages/vite-plugin-svelte/src/utils/constants.js b/packages/vite-plugin-svelte/src/utils/constants.js
@@ -1,9 +1,5 @@
 import { createRequire } from 'node:module';
 
-export const VITE_RESOLVE_MAIN_FIELDS = ['browser', 'module', 'jsnext:main', 'jsnext'];
-
-export const SVELTE_RESOLVE_MAIN_FIELDS = ['svelte'];
-
 export const SVELTE_IMPORTS = Object.entries(
 	createRequire(import.meta.url)('svelte/package.json').exports
 )
diff --git a/packages/vite-plugin-svelte/src/utils/options.js b/packages/vite-plugin-svelte/src/utils/options.js
diff --git a/packages/vite-plugin-svelte/types/index.d.ts.map b/packages/vite-plugin-svelte/types/index.d.ts.map
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +---
 +'@sveltejs/vite-plugin-svelte': minor
 +---
++
 +Add `esm-env` to `ssr.noExternal` by default to resolve its conditions with Vite
Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@`
`17`	`17`	`"postcss-load-config": "^6.0.1",`
`18`	`18`	`"svelte": "^5.2.4",`
`19`	`19`	`"svelte-preprocess": "^6.0.3",`
`20`		`- "vite": "^5.4.11"`
	`20`	`+ "vite": "^6.0.0-beta.10"`
`21`	`21`	`},`
`22`	`22`	`"type": "module"`
`23`	`23`	`}`
Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@`
`13`	`13`	`"devDependencies": {`
`14`	`14`	`"@sveltejs/vite-plugin-svelte": "workspace:^",`
`15`	`15`	`"svelte": "^5.2.4",`
`16`		`- "vite": "^5.4.11"`
	`16`	`+ "vite": "^6.0.0-beta.10"`
`17`	`17`	`},`
`18`	`18`	`"type": "module"`
`19`	`19`	`}`
Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@`
`14`	`14`	`"@sveltejs/vite-plugin-svelte": "workspace:^",`
`15`	`15`	`"svelte": "^5.2.4",`
`16`	`16`	`"svelte-preprocess": "^6.0.3",`
`17`		`- "vite": "^5.4.11"`
	`17`	`+ "vite": "^6.0.0-beta.10"`
`18`	`18`	`},`
`19`	`19`	`"type": "module"`
`20`	`20`	`}`
Original file line number	Diff line number	Diff line change
`@@ -12,6 +12,6 @@`
`12`	`12`	`"@sveltejs/vite-plugin-svelte": "workspace:^",`
`13`	`13`	`"sass": "^1.81.0",`
`14`	`14`	`"svelte": "^5.2.4",`
`15`		`- "vite": "^5.4.11"`
	`15`	`+ "vite": "^6.0.0-beta.10"`
`16`	`16`	`}`
`17`	`17`	`}`
Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@`
`10`	`10`	`"devDependencies": {`
`11`	`11`	`"@sveltejs/vite-plugin-svelte": "workspace:^",`
`12`	`12`	`"svelte": "^5.2.4",`
`13`		`- "vite": "^5.4.11"`
	`13`	`+ "vite": "^6.0.0-beta.10"`
`14`	`14`	`},`
`15`	`15`	`"type": "module"`
`16`	`16`	`}`