Sanitize server stack trace on errors

Author: hamboomger

.changeset/sharp-experts-unite.md

@@ -74,8 +74,6 @@ jobs:
       - run: pnpm playwright install ${{ matrix.e2e-browser }}
       - run: pnpm run sync-all
       - run: pnpm test:kit
-        env:
-          NODE_OPTIONS: ${{matrix.node-version == 22 && '--experimental-strip-types' || ''}} # allows loading svelte.config.ts
       - name: Print flaky test report
         run: node scripts/print-flaky-test-report.js
       - name: Archive test results
@@ -183,19 +181,14 @@ jobs:
           path: test-results-server-side-route-resolution-${{ matrix.mode }}.tar.gz
   test-others:
     runs-on: ubuntu-latest
-    strategy:
-      matrix:
-        node-version: [18, 20, 22, 24]
     steps:
       - uses: actions/checkout@v4
       - uses: pnpm/[email protected]
       - uses: actions/setup-node@v4
         with:
-          node-version: ${{matrix.node-version}}
+          node-version: 18
           cache: pnpm
       - run: pnpm install --frozen-lockfile
       - run: pnpm playwright install chromium
       - run: cd packages/kit && pnpm prepublishOnly
       - run: pnpm run test:others
-        env:
-          NODE_OPTIONS: ${{matrix.node-version == 22 && '--experimental-strip-types' || ''}} # allows loading svelte.config.ts

.changeset/twenty-pants-fetch.md

@@ -14,9 +14,7 @@ export default [
 			'**/test-results',
 			'**/build',
 			'**/.custom-out-dir',
-			'packages/adapter-*/files',
-			'packages/kit/src/core/config/fixtures/multiple', // dir contains svelte config with multiple extensions tripping eslint
-			'packages/package/test/fixtures/typescript-svelte-config/expected'
+			'packages/adapter-*/files'
 		]
 	},
 	{

.github/workflows/ci.yml

@@ -1,4 +1,5 @@
 import fs from 'node:fs';
+import path from 'node:path';
 import process from 'node:process';
 import colors from 'kleur';
 import sade from 'sade';
@@ -27,11 +28,8 @@ prog
 	.describe('Synchronise generated type definitions')
 	.option('--mode', 'Specify a mode for loading environment variables', 'development')
 	.action(async ({ mode }) => {
-		const config_files = ['js', 'ts']
-			.map((ext) => `svelte.config.${ext}`)
-			.filter((f) => fs.existsSync(f));
-		if (config_files.length === 0) {
-			console.warn(`Missing Svelte config file in ${process.cwd()} — skipping`);
+		if (!fs.existsSync('svelte.config.js')) {
+			console.warn(`Missing ${path.resolve('svelte.config.js')} — skipping`);
 			return;
 		}
 

eslint.config.js

@@ -57,24 +57,17 @@ export function load_error_page(config) {
 }
 
 /**
- * Loads and validates Svelte config file
+ * Loads and validates svelte.config.js
  * @param {{ cwd?: string }} options
  * @returns {Promise<import('types').ValidatedConfig>}
  */
 export async function load_config({ cwd = process.cwd() } = {}) {
-	const config_files = ['js', 'ts']
-		.map((ext) => path.join(cwd, `svelte.config.${ext}`))
-		.filter((f) => fs.existsSync(f));
+	const config_file = path.join(cwd, 'svelte.config.js');
 
-	if (config_files.length === 0) {
+	if (!fs.existsSync(config_file)) {
 		return process_config({}, { cwd });
 	}
-	const config_file = config_files[0];
-	if (config_files.length > 1) {
-		console.log(
-			`Found multiple Svelte config files in ${cwd}: ${config_files.map((f) => path.basename(f)).join(', ')}. Using ${path.basename(config_file)}`
-		);
-	}
+
 	const config = await import(`${url.pathToFileURL(config_file).href}?ts=${Date.now()}`);
 
 	try {
@@ -83,7 +76,7 @@ export async function load_config({ cwd = process.cwd() } = {}) {
 		const error = /** @type {Error} */ (e);
 
 		// redact the stack trace — it's not helpful to users
-		error.stack = `Could not load ${config_file}: ${error.message}\n`;
+		error.stack = `Could not load svelte.config.js: ${error.message}\n`;
 		throw error;
 	}
 }
@@ -118,7 +111,7 @@ function process_config(config, { cwd = process.cwd() } = {}) {
 export function validate_config(config) {
 	if (typeof config !== 'object') {
 		throw new Error(
-			'The Svelte config file must have a configuration object as its default export. See https://svelte.dev/docs/kit/configuration'
+			'svelte.config.js must have a configuration object as its default export. See https://svelte.dev/docs/kit/configuration'
 		);
 	}
 

packages/kit/src/cli.js

@@ -365,30 +365,6 @@ test('load default config (esm)', async () => {
 	expect(config).toEqual(defaults);
 });
 
-test('load default config (esm) with .ts extensions', async () => {
-	const cwd = join(__dirname, 'fixtures/typescript');
-
-	const config = await load_config({ cwd });
-	remove_keys(config, ([, v]) => typeof v === 'function');
-
-	const defaults = get_defaults(cwd + '/');
-	defaults.kit.version.name = config.kit.version.name;
-
-	expect(config).toEqual(defaults);
-});
-
-test('load .js config when both .js and .ts configs are present', async () => {
-	const cwd = join(__dirname, 'fixtures/multiple');
-
-	const config = await load_config({ cwd });
-	remove_keys(config, ([, v]) => typeof v === 'function');
-
-	const defaults = get_defaults(cwd + '/');
-	defaults.kit.version.name = config.kit.version.name;
-
-	expect(config).toEqual(defaults);
-});
-
 test('errors on loading config with incorrect default export', async () => {
 	let message = null;
 
@@ -401,6 +377,6 @@ test('errors on loading config with incorrect default export', async () => {
 
 	assert.equal(
 		message,
-		'The Svelte config file must have a configuration object as its default export. See https://svelte.dev/docs/kit/configuration'
+		'svelte.config.js must have a configuration object as its default export. See https://svelte.dev/docs/kit/configuration'
 	);
 });