[fix] don't polyfill undici if using Deno or Bun

.changeset/wise-adults-pull.md

@@ -0,0 +1,5 @@
+---
+'@sveltejs/kit': patch
+---
+
+[fix] conditionally polyfill but better to support Deno

packages/kit/src/core/prerender/prerender.js

@@ -1,8 +1,9 @@
 import { readFileSync, writeFileSync } from 'fs';
 import { dirname, join } from 'path';
 import { pathToFileURL, URL } from 'url';
-import { mkdirp, posixify, walk } from '../../utils/filesystem.js';
 import { installPolyfills } from '../../exports/node/polyfills.js';
+import { mkdirp, posixify, walk } from '../../utils/filesystem.js';
+import { non_node_runtime } from '../../utils/platform.js';
 import { is_root_relative, resolve } from '../../utils/url.js';
 import { queue } from './queue.js';
 import { crawl } from './crawl.js';
@@ -89,7 +90,9 @@ export async function prerender() {
 		verbose: verbose === 'true'
 	});
 
-	installPolyfills();
+	if (!non_node_runtime()) {
+		installPolyfills();
+	}
 
 	const server_root = join(config.outDir, 'output');
 

packages/kit/src/exports/node/polyfills.js

@@ -16,6 +16,7 @@ const globals = {
 };
 
 // exported for dev/preview and node environments
+// TODO: remove this once we only support Node 18.11+ (the version multipart/form-data was added)
 export function installPolyfills() {
 	for (const name in globals) {
 		Object.defineProperty(globalThis, name, {

packages/kit/src/exports/vite/dev/index.js

@@ -8,6 +8,7 @@ import { getRequest, setResponse } from '../../../exports/node/index.js';
 import { installPolyfills } from '../../../exports/node/polyfills.js';
 import { coalesce_to_error } from '../../../utils/error.js';
 import { posixify, resolve_entry, to_fs } from '../../../utils/filesystem.js';
+import { non_node_runtime } from '../../../utils/platform.js';
 import { load_error_page, load_template } from '../../../core/config/index.js';
 import { SVELTE_KIT_ASSETS } from '../../../constants.js';
 import * as sync from '../../../core/sync/sync.js';
@@ -24,7 +25,9 @@ const cwd = process.cwd();
  * @return {Promise<Promise<() => void>>}
  */
 export async function dev(vite, vite_config, svelte_config) {
-	installPolyfills();
+	if (!non_node_runtime()) {
+		installPolyfills();
+	}
 
 	sync.init(svelte_config, vite_config.mode);
 

packages/kit/src/exports/vite/preview/index.js

@@ -2,10 +2,11 @@ import fs from 'fs';
 import { join } from 'path';
 import sirv from 'sirv';
 import { pathToFileURL } from 'url';
+import { loadEnv, normalizePath } from 'vite';
 import { getRequest, setResponse } from '../../../exports/node/index.js';
 import { installPolyfills } from '../../../exports/node/polyfills.js';
 import { SVELTE_KIT_ASSETS } from '../../../constants.js';
-import { loadEnv, normalizePath } from 'vite';
+import { non_node_runtime } from '../../../utils/platform.js';
 import { not_found } from '../utils.js';
 
 /** @typedef {import('http').IncomingMessage} Req */
@@ -21,7 +22,9 @@ import { not_found } from '../utils.js';
  * @param {import('types').ValidatedConfig} svelte_config
  */
 export async function preview(vite, vite_config, svelte_config) {
-	installPolyfills();
+	if (!non_node_runtime()) {
+		installPolyfills();
+	}
 
 	const { paths } = svelte_config.kit;
 	const base = paths.base;

packages/kit/src/utils/platform.js

@@ -0,0 +1,3 @@
+export function non_node_runtime() {
+	return typeof globalThis.Deno !== 'undefined' || typeof globalThis.Bun !== 'undefined';
+}

packages/kit/types/internal.d.ts

@@ -389,4 +389,6 @@ declare global {
 	const __SVELTEKIT_BROWSER__: boolean;
 	const __SVELTEKIT_DEV__: boolean;
 	const __SVELTEKIT_EMBEDDED__: boolean;
+	var Bun: object;
+	var Deno: object;
 }