wrangler 0.0.0-eab9542 → 0.0.0-eaf71b86c

package/src/config/validation.ts

@@ -1,1745 +0,0 @@
-import path from "node:path";
-import TOML from "@iarna/toml";
-import { Diagnostics } from "./diagnostics";
-import {
-	deprecated,
-	experimental,
-	hasProperty,
-	inheritable,
-	isBoolean,
-	isObjectWith,
-	isOneOf,
-	isOptionalProperty,
-	isRequiredProperty,
-	isString,
-	isStringArray,
-	validateAdditionalProperties,
-	notInheritable,
-	validateOptionalProperty,
-	validateOptionalTypedArray,
-	validateRequiredProperty,
-	validateTypedArray,
-	all,
-	isMutuallyExclusiveWith,
-	inheritableInLegacyEnvironments,
-	appendEnvName,
-	getBindingNames,
-	isValidName,
-} from "./validation-helpers";
-import type { Config, DevConfig, RawConfig, RawDevConfig } from "./config";
-import type {
-	RawEnvironment,
-	DeprecatedUpload,
-	Environment,
-	Rule,
-} from "./environment";
-import type { ValidatorFn } from "./validation-helpers";
-
-const ENGLISH = new Intl.ListFormat("en");
-
-/**
- * Validate the given `rawConfig` object that was loaded from `configPath`.
- *
- * The configuration is normalized, which includes using default values for missing field,
- * and copying over inheritable fields into named environments.
- *
- * Any errors or warnings from the validation are available in the returned `diagnostics` object.
- */
-export function normalizeAndValidateConfig(
-	rawConfig: RawConfig,
-	configPath: string | undefined,
-	args: unknown
-): {
-	config: Config;
-	diagnostics: Diagnostics;
-} {
-	const diagnostics = new Diagnostics(
-		`Processing ${
-			configPath ? path.relative(process.cwd(), configPath) : "wrangler"
-		} configuration:`
-	);
-
-	deprecated(
-		diagnostics,
-		rawConfig,
-		"miniflare",
-		"Wrangler does not use configuration in the `miniflare` section. Unless you are using Miniflare directly you can remove this section.",
-		true,
-		"😶 Ignored"
-	);
-
-	deprecated(
-		diagnostics,
-		rawConfig,
-		"type",
-		"Most common features now work out of the box with wrangler, including modules, jsx, typescript, etc. If you need anything more, use a custom build.",
-		true,
-		"😶 Ignored"
-	);
-
-	deprecated(
-		diagnostics,
-		rawConfig,
-		"webpack_config",
-		"Most common features now work out of the box with wrangler, including modules, jsx, typescript, etc. If you need anything more, use a custom build.",
-		true,
-		"😶 Ignored"
-	);
-
-	validateOptionalProperty(
-		diagnostics,
-		"",
-		"legacy_env",
-		rawConfig.legacy_env,
-		"boolean"
-	);
-
-	// TODO: set the default to false to turn on service environments as the default
-	const isLegacyEnv =
-		(args as { "legacy-env": boolean | undefined })["legacy-env"] ??
-		rawConfig.legacy_env ??
-		true;
-
-	// TODO: remove this once service environments goes GA.
-	if (!isLegacyEnv) {
-		diagnostics.warnings.push(
-			"Experimental: Service environments are in beta, and their behaviour is guaranteed to change in the future. DO NOT USE IN PRODUCTION."
-		);
-	}
-
-	const topLevelEnv = normalizeAndValidateEnvironment(
-		diagnostics,
-		configPath,
-		rawConfig
-	);
-
-	//TODO: find a better way to define the type of Args that can be passed to the normalizeAndValidateConfig()
-	const envName = (args as { env: string | undefined }).env;
-
-	let activeEnv = topLevelEnv;
-	if (envName !== undefined) {
-		const envDiagnostics = new Diagnostics(
-			`"env.${envName}" environment configuration`
-		);
-		const rawEnv = rawConfig.env?.[envName];
-		if (rawEnv !== undefined) {
-			activeEnv = normalizeAndValidateEnvironment(
-				envDiagnostics,
-				configPath,
-				rawEnv,
-				envName,
-				topLevelEnv,
-				isLegacyEnv,
-				rawConfig
-			);
-			diagnostics.addChild(envDiagnostics);
-		} else {
-			// An environment was specified, but no configuration for it was found.
-			// To cover any legacy environment cases, where the `envName` is used,
-			// Let's create a fake active environment with the specified `envName`.
-			activeEnv = normalizeAndValidateEnvironment(
-				envDiagnostics,
-				configPath,
-				{},
-				envName,
-				topLevelEnv,
-				isLegacyEnv,
-				rawConfig
-			);
-			const envNames = rawConfig.env
-				? `The available configured environment names are: ${JSON.stringify(
-						Object.keys(rawConfig.env)
-				  )}\n`
-				: "";
-			const message =
-				`No environment found in configuration with name "${envName}".\n` +
-				`Before using \`--env=${envName}\` there should be an equivalent environment section in the configuration.\n` +
-				`${envNames}\n` +
-				`Consider adding an environment configuration section to the wrangler.toml file:\n` +
-				"```\n[env." +
-				envName +
-				"]\n```\n";
-
-			if (envNames.length > 0) {
-				diagnostics.errors.push(message);
-			} else {
-				// Only warn (rather than error) if there are not actually any environments configured in wrangler.toml.
-				diagnostics.warnings.push(message);
-			}
-		}
-	}
-
-	// Process the top-level default environment configuration.
-	const config: Config = {
-		configPath,
-		legacy_env: isLegacyEnv,
-		...activeEnv,
-		dev: normalizeAndValidateDev(diagnostics, rawConfig.dev ?? {}),
-		migrations: normalizeAndValidateMigrations(
-			diagnostics,
-			rawConfig.migrations ?? [],
-			activeEnv.durable_objects
-		),
-		site: normalizeAndValidateSite(
-			diagnostics,
-			configPath,
-			rawConfig,
-			activeEnv.main
-		),
-		assets: normalizeAndValidateAssets(diagnostics, configPath, rawConfig),
-		wasm_modules: normalizeAndValidateModulePaths(
-			diagnostics,
-			configPath,
-			"wasm_modules",
-			rawConfig.wasm_modules
-		),
-		text_blobs: normalizeAndValidateModulePaths(
-			diagnostics,
-			configPath,
-			"text_blobs",
-			rawConfig.text_blobs
-		),
-		data_blobs: normalizeAndValidateModulePaths(
-			diagnostics,
-			configPath,
-			"data_blobs",
-			rawConfig.data_blobs
-		),
-	};
-
-	validateBindingsHaveUniqueNames(diagnostics, config);
-
-	validateAdditionalProperties(
-		diagnostics,
-		"top-level",
-		Object.keys(rawConfig),
-		[...Object.keys(config), "env"]
-	);
-
-	experimental(diagnostics, rawConfig, "assets");
-
-	return { config, diagnostics };
-}
-
-/**
- * Validate the `build` configuration and return the normalized values.
- */
-function normalizeAndValidateBuild(
-	diagnostics: Diagnostics,
-	rawEnv: RawEnvironment,
-	rawBuild: Config["build"],
-	configPath: string | undefined
-): Config["build"] & { deprecatedUpload: DeprecatedUpload } {
-	const { command, cwd, watch_dir = "./src", upload, ...rest } = rawBuild;
-	const deprecatedUpload: DeprecatedUpload = { ...upload };
-	validateAdditionalProperties(diagnostics, "build", Object.keys(rest), []);
-
-	validateOptionalProperty(diagnostics, "build", "command", command, "string");
-	validateOptionalProperty(diagnostics, "build", "cwd", cwd, "string");
-	if (Array.isArray(watch_dir)) {
-		validateTypedArray(diagnostics, "build.watch_dir", watch_dir, "string");
-	} else {
-		validateOptionalProperty(
-			diagnostics,
-			"build",
-			"watch_dir",
-			watch_dir,
-			"string"
-		);
-	}
-
-	deprecated(
-		diagnostics,
-		rawEnv,
-		"build.upload.format",
-		"The format is inferred automatically from the code.",
-		true
-	);
-
-	if (rawEnv.main !== undefined && rawBuild.upload?.main) {
-		diagnostics.errors.push(
-			`Don't define both the \`main\` and \`build.upload.main\` fields in your configuration.\n` +
-				`They serve the same purpose: to point to the entry-point of your worker.\n` +
-				`Delete the \`build.upload.main\` and \`build.upload.dir\` field from your config.`
-		);
-	} else {
-		deprecated(
-			diagnostics,
-			rawEnv,
-			"build.upload.main",
-			`Delete the \`build.upload.main\` and \`build.upload.dir\` fields.\n` +
-				`Then add the top level \`main\` field to your configuration file:\n` +
-				`\`\`\`\n` +
-				`main = "${path.join(
-					rawBuild.upload?.dir ?? "./dist",
-					rawBuild.upload?.main ?? "."
-				)}"\n` +
-				`\`\`\``,
-			true
-		);
-
-		deprecated(
-			diagnostics,
-			rawEnv,
-			"build.upload.dir",
-			`Use the top level "main" field or a command-line argument to specify the entry-point for the Worker.`,
-			true
-		);
-	}
-
-	return {
-		command,
-		watch_dir:
-			// - `watch_dir` only matters when `command` is defined, so we apply
-			// a default only when `command` is defined
-			// - `configPath` will always be defined since `build` can only
-			// be configured in `wrangler.toml`, but who knows, that may
-			// change in the future, so we do a check anyway
-			command && configPath
-				? Array.isArray(watch_dir)
-					? watch_dir.map((dir) =>
-							path.relative(
-								process.cwd(),
-								path.join(path.dirname(configPath), `${dir}`)
-							)
-					  )
-					: path.relative(
-							process.cwd(),
-							path.join(path.dirname(configPath), `${watch_dir}`)
-					  )
-				: watch_dir,
-		cwd,
-		deprecatedUpload,
-	};
-}
-
-/**
- * Validate the `main` field and return the normalized values.
- */
-function normalizeAndValidateMainField(
-	configPath: string | undefined,
-	rawMain: string | undefined,
-	deprecatedUpload: DeprecatedUpload | undefined
-): string | undefined {
-	const configDir = path.dirname(configPath ?? "wrangler.toml");
-	if (rawMain !== undefined) {
-		if (typeof rawMain === "string") {
-			const directory = path.resolve(configDir);
-			return path.resolve(directory, rawMain);
-		} else {
-			return rawMain;
-		}
-	} else if (deprecatedUpload?.main !== undefined) {
-		const directory = path.resolve(
-			configDir,
-			deprecatedUpload?.dir || "./dist"
-		);
-		return path.resolve(directory, deprecatedUpload.main);
-	} else {
-		return;
-	}
-}
-
-/**
- * Validate the `dev` configuration and return the normalized values.
- */
-function normalizeAndValidateDev(
-	diagnostics: Diagnostics,
-	rawDev: RawDevConfig
-): DevConfig {
-	const {
-		ip = "localhost",
-		port,
-		local_protocol = "http",
-		upstream_protocol = "https",
-		host,
-		...rest
-	} = rawDev;
-	validateAdditionalProperties(diagnostics, "dev", Object.keys(rest), []);
-
-	validateOptionalProperty(diagnostics, "dev", "ip", ip, "string");
-	validateOptionalProperty(diagnostics, "dev", "port", port, "number");
-	validateOptionalProperty(
-		diagnostics,
-		"dev",
-		"local_protocol",
-		local_protocol,
-		"string",
-		["http", "https"]
-	);
-	validateOptionalProperty(
-		diagnostics,
-		"dev",
-		"upstream_protocol",
-		upstream_protocol,
-		"string",
-		["http", "https"]
-	);
-	validateOptionalProperty(diagnostics, "dev", "host", host, "string");
-	return { ip, port, local_protocol, upstream_protocol, host };
-}
-
-/**
- * Validate the `migrations` configuration and return the normalized values.
- */
-function normalizeAndValidateMigrations(
-	diagnostics: Diagnostics,
-	rawMigrations: Config["migrations"],
-	durableObjects: Config["durable_objects"]
-): Config["migrations"] {
-	if (!Array.isArray(rawMigrations)) {
-		diagnostics.errors.push(
-			`The optional "migrations" field should be an array, but got ${JSON.stringify(
-				rawMigrations
-			)}`
-		);
-		return [];
-	} else {
-		for (let i = 0; i < rawMigrations.length; i++) {
-			const { tag, new_classes, renamed_classes, deleted_classes, ...rest } =
-				rawMigrations[i];
-
-			validateAdditionalProperties(
-				diagnostics,
-				"migrations",
-				Object.keys(rest),
-				[]
-			);
-
-			validateRequiredProperty(
-				diagnostics,
-				`migrations[${i}]`,
-				`tag`,
-				tag,
-				"string"
-			);
-			validateOptionalTypedArray(
-				diagnostics,
-				`migrations[${i}].new_classes`,
-				new_classes,
-				"string"
-			);
-			if (renamed_classes !== undefined) {
-				if (!Array.isArray(renamed_classes)) {
-					diagnostics.errors.push(
-						`Expected "migrations[${i}].renamed_classes" to be an array of "{from: string, to: string}" objects but got ${JSON.stringify(
-							renamed_classes
-						)}.`
-					);
-				} else if (
-					renamed_classes.some(
-						(c) =>
-							typeof c !== "object" ||
-							!isRequiredProperty(c, "from", "string") ||
-							!isRequiredProperty(c, "to", "string")
-					)
-				) {
-					diagnostics.errors.push(
-						`Expected "migrations[${i}].renamed_classes" to be an array of "{from: string, to: string}" objects but got ${JSON.stringify(
-							renamed_classes
-						)}.`
-					);
-				}
-			}
-			validateOptionalTypedArray(
-				diagnostics,
-				`migrations[${i}].deleted_classes`,
-				deleted_classes,
-				"string"
-			);
-		}
-
-		if (
-			Array.isArray(durableObjects?.bindings) &&
-			durableObjects.bindings.length > 0
-		) {
-			// intrinsic [durable_objects] implies [migrations]
-			const exportedDurableObjects = (durableObjects.bindings || []).filter(
-				(binding) => !binding.script_name
-			);
-			if (exportedDurableObjects.length > 0 && rawMigrations.length === 0) {
-				if (
-					!exportedDurableObjects.some(
-						(exportedDurableObject) =>
-							typeof exportedDurableObject.class_name !== "string"
-					)
-				) {
-					const durableObjectClassnames = exportedDurableObjects.map(
-						(durable) => durable.class_name
-					);
-
-					diagnostics.warnings.push(
-						`In wrangler.toml, you have configured [durable_objects] exported by this Worker (${durableObjectClassnames.join(
-							", "
-						)}), but no [migrations] for them. This may not work as expected until you add a [migrations] section to your wrangler.toml. Add this configuration to your wrangler.toml:
-
-  \`\`\`
-  [[migrations]]
-  tag = "v1" # Should be unique for each entry
-  new_classes = [${durableObjectClassnames
-		.map((name) => `"${name}"`)
-		.join(", ")}]
-  \`\`\`
-
-Refer to https://developers.cloudflare.com/workers/learning/using-durable-objects/#durable-object-migrations-in-wranglertoml for more details.`
-					);
-				}
-			}
-		}
-
-		return rawMigrations;
-	}
-}
-
-/**
- * Validate the `site` configuration and return the normalized values.
- */
-function normalizeAndValidateSite(
-	diagnostics: Diagnostics,
-	configPath: string | undefined,
-	rawConfig: RawConfig,
-	mainEntryPoint: string | undefined
-): Config["site"] {
-	if (rawConfig?.site !== undefined) {
-		const { bucket, include = [], exclude = [], ...rest } = rawConfig.site;
-
-		validateAdditionalProperties(diagnostics, "site", Object.keys(rest), [
-			"entry-point",
-		]);
-		validateRequiredProperty(diagnostics, "site", "bucket", bucket, "string");
-		validateTypedArray(diagnostics, "sites.include", include, "string");
-		validateTypedArray(diagnostics, "sites.exclude", exclude, "string");
-		validateOptionalProperty(
-			diagnostics,
-			"site",
-			"entry-point",
-			rawConfig.site["entry-point"],
-			"string"
-		);
-
-		deprecated(
-			diagnostics,
-			rawConfig,
-			`site.entry-point`,
-			`Delete the \`site.entry-point\` field, then add the top level \`main\` field to your configuration file:\n` +
-				`\`\`\`\n` +
-				`main = "${path.join(
-					String(rawConfig.site["entry-point"]) || "workers-site",
-					path.extname(String(rawConfig.site["entry-point"]) || "workers-site")
-						? ""
-						: "index.js"
-				)}"\n` +
-				`\`\`\``,
-			false,
-			undefined,
-			"warning"
-		);
-
-		let siteEntryPoint = rawConfig.site["entry-point"];
-
-		if (!mainEntryPoint && !siteEntryPoint) {
-			// this means that we're defaulting to "workers-site"
-			// so let's add the deprecation warning
-			diagnostics.warnings.push(
-				`Because you've defined a [site] configuration, we're defaulting to "workers-site" for the deprecated \`site.entry-point\`field.\n` +
-					`Add the top level \`main\` field to your configuration file:\n` +
-					`\`\`\`\n` +
-					`main = "workers-site/index.js"\n` +
-					`\`\`\``
-			);
-			siteEntryPoint = "workers-site";
-		} else if (mainEntryPoint && siteEntryPoint) {
-			diagnostics.errors.push(
-				`Don't define both the \`main\` and \`site.entry-point\` fields in your configuration.\n` +
-					`They serve the same purpose: to point to the entry-point of your worker.\n` +
-					`Delete the deprecated \`site.entry-point\` field from your config.`
-			);
-		}
-
-		if (configPath && siteEntryPoint) {
-			// rewrite the path to be relative to the working directory
-			siteEntryPoint = path.relative(
-				process.cwd(),
-				path.join(path.dirname(configPath), siteEntryPoint)
-			);
-		}
-
-		return {
-			bucket,
-			"entry-point": siteEntryPoint,
-			include,
-			exclude,
-		};
-	}
-	return undefined;
-}
-
-/**
- * Validate the `assets` configuration and return normalized values.
- */
-function normalizeAndValidateAssets(
-	diagnostics: Diagnostics,
-	configPath: string | undefined,
-	rawConfig: RawConfig
-) {
-	if (
-		typeof rawConfig?.assets === "string" ||
-		rawConfig?.assets === undefined
-	) {
-		return rawConfig?.assets;
-	}
-
-	const { bucket, include = [], exclude = [], ...rest } = rawConfig.assets;
-
-	validateAdditionalProperties(diagnostics, "assets", Object.keys(rest), []);
-	validateRequiredProperty(diagnostics, "assets", "bucket", bucket, "string");
-	validateTypedArray(diagnostics, "assets.include", include, "string");
-	validateTypedArray(diagnostics, "assets.exclude", exclude, "string");
-
-	return {
-		bucket,
-		include,
-		exclude,
-	};
-}
-
-/**
- * Map the paths of the `wasm_modules`, `text_blobs` or `data_blobs` configuration to be relative to the current working directory.
- */
-function normalizeAndValidateModulePaths(
-	diagnostics: Diagnostics,
-	configPath: string | undefined,
-	field: "wasm_modules" | "text_blobs" | "data_blobs",
-	rawMapping: Record<string, string> | undefined
-): Record<string, string> | undefined {
-	if (rawMapping === undefined) {
-		return undefined;
-	}
-	const mapping: Record<string, string> = {};
-	// Rewrite paths to be relative to the cwd, rather than the config path.
-	for (const [name, filePath] of Object.entries(rawMapping)) {
-		if (isString(diagnostics, `${field}['${name}']`, filePath, undefined)) {
-			if (configPath) {
-				mapping[name] = configPath
-					? path.relative(
-							process.cwd(),
-							path.join(path.dirname(configPath), filePath)
-					  )
-					: filePath;
-			}
-		}
-	}
-	return mapping;
-}
-
-/**
- * Check whether a value has the shape of a route, which can be a string
- * or an object that looks like {pattern: string, zone_id: string }
- */
-function isValidRouteValue(item: unknown): boolean {
-	if (!item) {
-		return false;
-	}
-	if (typeof item === "string") {
-		return true;
-	}
-	if (typeof item === "object") {
-		if (!hasProperty(item, "pattern") || typeof item.pattern !== "string") {
-			return false;
-		}
-
-		const otherKeys = Object.keys(item).length - 1; // minus one to subtract "pattern"
-
-		const hasZoneId =
-			hasProperty(item, "zone_id") && typeof item.zone_id === "string";
-		const hasZoneName =
-			hasProperty(item, "zone_name") && typeof item.zone_name === "string";
-		const hasCustomDomainFlag =
-			hasProperty(item, "custom_domain") &&
-			typeof item.custom_domain === "boolean";
-
-		if (otherKeys === 2 && hasCustomDomainFlag && (hasZoneId || hasZoneName)) {
-			return true;
-		} else if (
-			otherKeys === 1 &&
-			(hasZoneId || hasZoneName || hasCustomDomainFlag)
-		) {
-			return true;
-		}
-	}
-	return false;
-}
-
-/**
- * If account_id has been passed as an empty string, normalise it to undefined.
- * This is to workaround older wrangler1-era templates that have account_id = '',
- * which isn't a valid value anyway
- */
-function mutateEmptyStringAccountIDValue(
-	diagnostics: Diagnostics,
-	rawEnv: RawEnvironment
-) {
-	if (rawEnv.account_id === "") {
-		diagnostics.warnings.push(
-			`The "account_id" field in your configuration is an empty string and will be ignored.\n` +
-				`Please remove the "account_id" field from your configuration.`
-		);
-		rawEnv.account_id = undefined;
-	}
-	return rawEnv;
-}
-
-/**
- * Normalize empty string to `undefined` by mutating rawEnv.route value.
- * As part of backward compatibility with Wrangler1 converting empty string to `undefined`
- */
-function mutateEmptyStringRouteValue(
-	diagnostics: Diagnostics,
-	rawEnv: RawEnvironment
-): RawEnvironment {
-	if (rawEnv["route"] === "") {
-		diagnostics.warnings.push(
-			`The "route" field in your configuration is an empty string and will be ignored.\n` +
-				`Please remove the "route" field from your configuration.`
-		);
-		rawEnv["route"] = undefined;
-	}
-
-	return rawEnv;
-}
-
-/**
- * Validate that the field is a route.
- */
-const isRoute: ValidatorFn = (diagnostics, field, value) => {
-	if (value !== undefined && !isValidRouteValue(value)) {
-		diagnostics.errors.push(
-			`Expected "${field}" to be either a string, or an object with shape { pattern, custom_domain, zone_id | zone_name }, but got ${JSON.stringify(
-				value
-			)}.`
-		);
-		return false;
-	}
-	return true;
-};
-
-/**
- * Validate that the field is an array of routes.
- */
-const isRouteArray: ValidatorFn = (diagnostics, field, value) => {
-	if (value === undefined) {
-		return true;
-	}
-	if (!Array.isArray(value)) {
-		diagnostics.errors.push(
-			`Expected "${field}" to be an array but got ${JSON.stringify(value)}.`
-		);
-		return false;
-	}
-	const invalidRoutes = [];
-	for (const item of value) {
-		if (!isValidRouteValue(item)) {
-			invalidRoutes.push(item);
-		}
-	}
-	if (invalidRoutes.length > 0) {
-		diagnostics.errors.push(
-			`Expected "${field}" to be an array of either strings or objects with the shape { pattern, custom_domain, zone_id | zone_name }, but these weren't valid: ${JSON.stringify(
-				invalidRoutes,
-				null,
-				2
-			)}.`
-		);
-	}
-	return invalidRoutes.length === 0;
-};
-
-function normalizeAndValidateRoute(
-	diagnostics: Diagnostics,
-	topLevelEnv: Environment | undefined,
-	rawEnv: RawEnvironment
-): Config["route"] {
-	return inheritable(
-		diagnostics,
-		topLevelEnv,
-		mutateEmptyStringRouteValue(diagnostics, rawEnv),
-		"route",
-		isRoute,
-		undefined
-	);
-}
-
-function validateRoutes(
-	diagnostics: Diagnostics,
-	topLevelEnv: Environment | undefined,
-	rawEnv: RawEnvironment
-): Config["routes"] {
-	return inheritable(
-		diagnostics,
-		topLevelEnv,
-		rawEnv,
-		"routes",
-		all(isRouteArray, isMutuallyExclusiveWith(rawEnv, "route")),
-		undefined
-	);
-}
-
-/**
- * Validate top-level environment configuration and return the normalized values.
- */
-function normalizeAndValidateEnvironment(
-	diagnostics: Diagnostics,
-	configPath: string | undefined,
-	topLevelEnv: RawEnvironment
-): Environment;
-/**
- * Validate the named environment configuration and return the normalized values.
- */
-function normalizeAndValidateEnvironment(
-	diagnostics: Diagnostics,
-	configPath: string | undefined,
-	rawEnv: RawEnvironment,
-	envName: string,
-	topLevelEnv: Environment,
-	isLegacyEnv: boolean,
-	rawConfig: RawConfig
-): Environment;
-function normalizeAndValidateEnvironment(
-	diagnostics: Diagnostics,
-	configPath: string | undefined,
-	rawEnv: RawEnvironment,
-	envName = "top level",
-	topLevelEnv?: Environment | undefined,
-	isLegacyEnv?: boolean,
-	rawConfig?: RawConfig | undefined
-): Environment {
-	deprecated(
-		diagnostics,
-		rawEnv,
-		"zone_id",
-		"This is unnecessary since we can deduce this from routes directly.",
-		false // We need to leave this in-place for the moment since `route` commands might use it.
-	);
-
-	// The field "experimental_services" doesn't exist anymore in the config, but we still want to error about any older usage.
-	deprecated(
-		diagnostics,
-		rawEnv,
-		"experimental_services",
-		`The "experimental_services" field is no longer supported. Simply rename the [experimental_services] field to [services].`,
-		true
-	);
-
-	experimental(diagnostics, rawEnv, "unsafe");
-	experimental(diagnostics, rawEnv, "services");
-	experimental(diagnostics, rawEnv, "worker_namespaces");
-
-	const route = normalizeAndValidateRoute(diagnostics, topLevelEnv, rawEnv);
-
-	const account_id = inheritableInLegacyEnvironments(
-		diagnostics,
-		isLegacyEnv,
-		topLevelEnv,
-		mutateEmptyStringAccountIDValue(diagnostics, rawEnv),
-		"account_id",
-		isString,
-		undefined,
-		undefined
-	);
-
-	const routes = validateRoutes(diagnostics, topLevelEnv, rawEnv);
-
-	const workers_dev = inheritable(
-		diagnostics,
-		topLevelEnv,
-		rawEnv,
-		"workers_dev",
-		isBoolean,
-		undefined
-	);
-
-	const { deprecatedUpload, ...build } = normalizeAndValidateBuild(
-		diagnostics,
-		rawEnv,
-		rawEnv.build ?? topLevelEnv?.build ?? {},
-		configPath
-	);
-
-	const environment: Environment = {
-		// Inherited fields
-		account_id,
-		compatibility_date: inheritable(
-			diagnostics,
-			topLevelEnv,
-			rawEnv,
-			"compatibility_date",
-			isString,
-			undefined
-		),
-		compatibility_flags: inheritable(
-			diagnostics,
-			topLevelEnv,
-			rawEnv,
-			"compatibility_flags",
-			isStringArray,
-			[]
-		),
-		jsx_factory: inheritable(
-			diagnostics,
-			topLevelEnv,
-			rawEnv,
-			"jsx_factory",
-			isString,
-			"React.createElement"
-		),
-		jsx_fragment: inheritable(
-			diagnostics,
-			topLevelEnv,
-			rawEnv,
-			"jsx_fragment",
-			isString,
-			"React.Fragment"
-		),
-		tsconfig: validateAndNormalizeTsconfig(
-			diagnostics,
-			topLevelEnv,
-			rawEnv,
-			configPath
-		),
-		rules: validateAndNormalizeRules(
-			diagnostics,
-			topLevelEnv,
-			rawEnv,
-			deprecatedUpload?.rules,
-			envName
-		),
-		name: inheritableInLegacyEnvironments(
-			diagnostics,
-			isLegacyEnv,
-			topLevelEnv,
-			rawEnv,
-			"name",
-			isValidName,
-			appendEnvName(envName),
-			undefined
-		),
-		main: normalizeAndValidateMainField(
-			configPath,
-			inheritable(
-				diagnostics,
-				topLevelEnv,
-				rawEnv,
-				"main",
-				isString,
-				undefined
-			),
-			deprecatedUpload
-		),
-		route,
-		routes,
-		triggers: inheritable(
-			diagnostics,
-			topLevelEnv,
-			rawEnv,
-			"triggers",
-			isObjectWith("crons"),
-			{ crons: [] }
-		),
-		usage_model: inheritable(
-			diagnostics,
-			topLevelEnv,
-			rawEnv,
-			"usage_model",
-			isOneOf("bundled", "unbound"),
-			undefined
-		),
-		build,
-		workers_dev,
-		// Not inherited fields
-		vars: notInheritable(
-			diagnostics,
-			topLevelEnv,
-			rawConfig,
-			rawEnv,
-			envName,
-			"vars",
-			validateVars(envName),
-			{}
-		),
-		define: notInheritable(
-			diagnostics,
-			topLevelEnv,
-			rawConfig,
-			rawEnv,
-			envName,
-			"define",
-			validateDefines(envName),
-			{}
-		),
-		durable_objects: notInheritable(
-			diagnostics,
-			topLevelEnv,
-			rawConfig,
-			rawEnv,
-			envName,
-			"durable_objects",
-			validateBindingsProperty(envName, validateDurableObjectBinding),
-			{
-				bindings: [],
-			}
-		),
-		kv_namespaces: notInheritable(
-			diagnostics,
-			topLevelEnv,
-			rawConfig,
-			rawEnv,
-			envName,
-			"kv_namespaces",
-			validateBindingArray(envName, validateKVBinding),
-			[]
-		),
-		r2_buckets: notInheritable(
-			diagnostics,
-			topLevelEnv,
-			rawConfig,
-			rawEnv,
-			envName,
-			"r2_buckets",
-			validateBindingArray(envName, validateR2Binding),
-			[]
-		),
-		services: notInheritable(
-			diagnostics,
-			topLevelEnv,
-			rawConfig,
-			rawEnv,
-			envName,
-			"services",
-			validateBindingArray(envName, validateServiceBinding),
-			[]
-		),
-		worker_namespaces: notInheritable(
-			diagnostics,
-			topLevelEnv,
-			rawConfig,
-			rawEnv,
-			envName,
-			"worker_namespaces",
-			validateBindingArray(envName, validateWorkerNamespaceBinding),
-			[]
-		),
-		unsafe: notInheritable(
-			diagnostics,
-			topLevelEnv,
-			rawConfig,
-			rawEnv,
-			envName,
-			"unsafe",
-			validateBindingsProperty(envName, validateUnsafeBinding),
-			{
-				bindings: [],
-			}
-		),
-		zone_id: rawEnv.zone_id,
-		no_bundle: inheritable(
-			diagnostics,
-			topLevelEnv,
-			rawEnv,
-			"no_bundle",
-			isBoolean,
-			undefined
-		),
-		minify: inheritable(
-			diagnostics,
-			topLevelEnv,
-			rawEnv,
-			"minify",
-			isBoolean,
-			undefined
-		),
-		node_compat: inheritable(
-			diagnostics,
-			topLevelEnv,
-			rawEnv,
-			"node_compat",
-			isBoolean,
-			undefined
-		),
-	};
-
-	return environment;
-}
-
-function validateAndNormalizeTsconfig(
-	diagnostics: Diagnostics,
-	topLevelEnv: Environment | undefined,
-	rawEnv: RawEnvironment,
-	configPath: string | undefined
-) {
-	const tsconfig = inheritable(
-		diagnostics,
-		topLevelEnv,
-		rawEnv,
-		"tsconfig",
-		isString,
-		undefined
-	);
-
-	return configPath && tsconfig
-		? path.relative(
-				process.cwd(),
-				path.join(path.dirname(configPath), tsconfig)
-		  )
-		: tsconfig;
-}
-
-const validateAndNormalizeRules = (
-	diagnostics: Diagnostics,
-	topLevelEnv: Environment | undefined,
-	rawEnv: RawEnvironment,
-	deprecatedRules: Rule[] | undefined,
-	envName: string
-): Rule[] => {
-	if (topLevelEnv === undefined) {
-		// Only create errors/warnings for the top-level environment
-		if (rawEnv.rules && deprecatedRules) {
-			diagnostics.errors.push(
-				`You cannot configure both [rules] and [build.upload.rules] in your wrangler.toml. Delete the \`build.upload\` section.`
-			);
-		} else if (deprecatedRules) {
-			diagnostics.warnings.push(
-				`Deprecation: The \`build.upload.rules\` config field is no longer used, the rules should be specified via the \`rules\` config field. Delete the \`build.upload\` field from the configuration file, and add this:\n` +
-					"```\n" +
-					TOML.stringify({ rules: deprecatedRules }) +
-					"```"
-			);
-		}
-	}
-
-	return inheritable(
-		diagnostics,
-		topLevelEnv,
-		rawEnv,
-		"rules",
-		validateRules(envName),
-		deprecatedRules ?? []
-	);
-};
-
-const validateRules =
-	(envName: string): ValidatorFn =>
-	(diagnostics, field, envValue, config) => {
-		if (!envValue) {
-			return true;
-		}
-		const fieldPath =
-			config === undefined ? `${field}` : `env.${envName}.${field}`;
-		if (!Array.isArray(envValue)) {
-			diagnostics.errors.push(
-				`The field "${fieldPath}" should be an array but got ${JSON.stringify(
-					envValue
-				)}.`
-			);
-			return false;
-		}
-
-		let isValid = true;
-		for (let i = 0; i < envValue.length; i++) {
-			isValid =
-				validateRule(diagnostics, `${fieldPath}[${i}]`, envValue[i], config) &&
-				isValid;
-		}
-		return isValid;
-	};
-
-const validateRule: ValidatorFn = (diagnostics, field, value) => {
-	if (typeof value !== "object" || value === null) {
-		diagnostics.errors.push(
-			`"${field}" should be an object but got ${JSON.stringify(value)}.`
-		);
-		return false;
-	}
-	// Rules must have a type string and glob string array, and optionally a fallthrough boolean.
-	let isValid = true;
-	const rule = value as Rule;
-
-	if (
-		!isRequiredProperty(rule, "type", "string", [
-			"ESModule",
-			"CommonJS",
-			"CompiledWasm",
-			"Text",
-			"Data",
-		])
-	) {
-		diagnostics.errors.push(
-			`bindings should have a string "type" field, which contains one of "ESModule", "CommonJS", "CompiledWasm", "Text", or "Data".`
-		);
-		isValid = false;
-	}
-
-	isValid =
-		validateTypedArray(diagnostics, `${field}.globs`, rule.globs, "string") &&
-		isValid;
-
-	if (!isOptionalProperty(rule, "fallthrough", "boolean")) {
-		diagnostics.errors.push(
-			`the field "fallthrough", when present, should be a boolean.`
-		);
-		isValid = false;
-	}
-
-	return isValid;
-};
-
-const validateDefines =
-	(envName: string): ValidatorFn =>
-	(diagnostics, field, value, config) => {
-		let isValid = true;
-		const fieldPath =
-			config === undefined ? `${field}` : `env.${envName}.${field}`;
-
-		if (typeof value === "object" && value !== null) {
-			for (const varName in value) {
-				// some casting here to appease typescript
-				// even though the value might not match the type
-				if (typeof (value as Record<string, string>)[varName] !== "string") {
-					diagnostics.errors.push(
-						`The field "${fieldPath}.${varName}" should be a string but got ${JSON.stringify(
-							(value as Record<string, string>)[varName]
-						)}.`
-					);
-					isValid = false;
-				}
-			}
-		} else {
-			if (value !== undefined) {
-				diagnostics.errors.push(
-					`The field "${fieldPath}" should be an object but got ${JSON.stringify(
-						value
-					)}.\n`
-				);
-				isValid = false;
-			}
-		}
-
-		const configDefines = Object.keys(config?.define ?? {});
-
-		// If there are no top level vars then there is nothing to do here.
-		if (configDefines.length > 0) {
-			if (typeof value === "object" && value !== null) {
-				const configEnvDefines = config === undefined ? [] : Object.keys(value);
-
-				for (const varName of configDefines) {
-					if (!(varName in value)) {
-						diagnostics.warnings.push(
-							`"define.${varName}" exists at the top level, but not on "${fieldPath}".\n` +
-								`This is not what you probably want, since "define" configuration is not inherited by environments.\n` +
-								`Please add "define.${varName}" to "env.${envName}".`
-						);
-					}
-				}
-				for (const varName of configEnvDefines) {
-					if (!configDefines.includes(varName)) {
-						diagnostics.warnings.push(
-							`"${varName}" exists on "env.${envName}", but not on the top level.\n` +
-								`This is not what you probably want, since "define" configuration within environments can only override existing top level "define" configuration\n` +
-								`Please remove "${fieldPath}.${varName}", or add "define.${varName}".`
-						);
-					}
-				}
-			}
-		}
-
-		return isValid;
-	};
-
-const validateVars =
-	(envName: string): ValidatorFn =>
-	(diagnostics, field, value, config) => {
-		let isValid = true;
-		const fieldPath =
-			config === undefined ? `${field}` : `env.${envName}.${field}`;
-		const configVars = Object.keys(config?.vars ?? {});
-		// If there are no top level vars then there is nothing to do here.
-		if (configVars.length > 0) {
-			if (typeof value !== "object" || value === null) {
-				diagnostics.errors.push(
-					`The field "${fieldPath}" should be an object but got ${JSON.stringify(
-						value
-					)}.\n`
-				);
-				isValid = false;
-			} else {
-				for (const varName of configVars) {
-					if (!(varName in value)) {
-						diagnostics.warnings.push(
-							`"vars.${varName}" exists at the top level, but not on "${fieldPath}".\n` +
-								`This is not what you probably want, since "vars" configuration is not inherited by environments.\n` +
-								`Please add "vars.${varName}" to "env.${envName}".`
-						);
-					}
-				}
-			}
-		}
-		return isValid;
-	};
-
-const validateBindingsProperty =
-	(envName: string, validateBinding: ValidatorFn): ValidatorFn =>
-	(diagnostics, field, value, config) => {
-		let isValid = true;
-		const fieldPath =
-			config === undefined ? `${field}` : `env.${envName}.${field}`;
-
-		if (value !== undefined) {
-			// Check the validity of the `value` as a bindings container.
-			if (typeof value !== "object" || value === null || Array.isArray(value)) {
-				diagnostics.errors.push(
-					`The field "${fieldPath}" should be an object but got ${JSON.stringify(
-						value
-					)}.`
-				);
-				isValid = false;
-			} else if (!hasProperty(value, "bindings")) {
-				diagnostics.errors.push(
-					`The field "${fieldPath}" is missing the required "bindings" property.`
-				);
-				isValid = false;
-			} else if (!Array.isArray(value.bindings)) {
-				diagnostics.errors.push(
-					`The field "${fieldPath}.bindings" should be an array but got ${JSON.stringify(
-						value.bindings
-					)}.`
-				);
-				isValid = false;
-			} else {
-				for (let i = 0; i < value.bindings.length; i++) {
-					const binding = value.bindings[i];
-					const bindingDiagnostics = new Diagnostics(
-						`"${fieldPath}.bindings[${i}]": ${JSON.stringify(binding)}`
-					);
-					isValid =
-						validateBinding(
-							bindingDiagnostics,
-							`${fieldPath}.bindings[${i}]`,
-							binding,
-							config
-						) && isValid;
-					diagnostics.addChild(bindingDiagnostics);
-				}
-			}
-
-			const configBindingNames = getBindingNames(
-				config?.[field as keyof Environment]
-			);
-			if (isValid && configBindingNames.length > 0) {
-				// If there are top level bindings then check that they all appear in the environment.
-				const envBindingNames = new Set(getBindingNames(value));
-				const missingBindings = configBindingNames.filter(
-					(name) => !envBindingNames.has(name)
-				);
-				if (missingBindings.length > 0) {
-					diagnostics.warnings.push(
-						`The following bindings are at the top level, but not on "env.${envName}".\n` +
-							`This is not what you probably want, since "${field}" configuration is not inherited by environments.\n` +
-							`Please add a binding for each to "${fieldPath}.bindings":\n` +
-							missingBindings.map((name) => `- ${name}`).join("\n")
-					);
-				}
-			}
-		}
-		return isValid;
-	};
-
-/**
- * Check that the given field is a valid "durable_object" binding object.
- */
-const validateDurableObjectBinding: ValidatorFn = (
-	diagnostics,
-	field,
-	value
-) => {
-	if (typeof value !== "object" || value === null) {
-		diagnostics.errors.push(
-			`Expected "${field}" to be an object but got ${JSON.stringify(value)}`
-		);
-		return false;
-	}
-
-	// Durable Object bindings must have a name and class_name, and optionally a script_name and an environment.
-	let isValid = true;
-	if (!isRequiredProperty(value, "name", "string")) {
-		diagnostics.errors.push(`binding should have a string "name" field.`);
-		isValid = false;
-	}
-	if (!isRequiredProperty(value, "class_name", "string")) {
-		diagnostics.errors.push(`binding should have a string "class_name" field.`);
-		isValid = false;
-	}
-	if (!isOptionalProperty(value, "script_name", "string")) {
-		diagnostics.errors.push(
-			`the field "script_name", when present, should be a string.`
-		);
-		isValid = false;
-	}
-	// environment requires a script_name
-	if (!isOptionalProperty(value, "environment", "string")) {
-		diagnostics.errors.push(
-			`the field "environment", when present, should be a string.`
-		);
-		isValid = false;
-	}
-
-	if ("environment" in value && !("script_name" in value)) {
-		diagnostics.errors.push(
-			`binding should have a "script_name" field if "environment" is present.`
-		);
-		isValid = false;
-	}
-
-	return isValid;
-};
-
-/**
- * Check that the given field is a valid "unsafe" binding object.
- *
- * TODO: further validation of known unsafe bindings.
- */
-const validateUnsafeBinding: ValidatorFn = (diagnostics, field, value) => {
-	if (typeof value !== "object" || value === null) {
-		diagnostics.errors.push(
-			`Expected ${field} to be an object but got ${JSON.stringify(value)}.`
-		);
-		return false;
-	}
-
-	let isValid = true;
-	// Unsafe bindings must have a name and type.
-	if (!isRequiredProperty(value, "name", "string")) {
-		diagnostics.errors.push(`binding should have a string "name" field.`);
-		isValid = false;
-	}
-	if (isRequiredProperty(value, "type", "string")) {
-		const safeBindings = [
-			"plain_text",
-			"json",
-			"wasm_module",
-			"data_blob",
-			"text_blob",
-			"kv_namespace",
-			"durable_object_namespace",
-			"r2_bucket",
-			"service",
-		];
-
-		if (safeBindings.includes(value.type)) {
-			diagnostics.warnings.push(
-				`The binding type "${value.type}" is directly supported by wrangler.\n` +
-					`Consider migrating this unsafe binding to a format for '${value.type}' bindings that is supported by wrangler for optimal support.\n` +
-					"For more details, see https://developers.cloudflare.com/workers/cli-wrangler/configuration"
-			);
-		}
-	} else {
-		diagnostics.errors.push(`binding should have a string "type" field.`);
-		isValid = false;
-	}
-	return isValid;
-};
-
-/**
- * Check that the given environment field is a valid array of bindings.
- */
-const validateBindingArray =
-	(envName: string, validateBinding: ValidatorFn): ValidatorFn =>
-	(diagnostics, field, envValue, config) => {
-		if (envValue === undefined) {
-			return true;
-		}
-
-		const fieldPath =
-			config === undefined ? `${field}` : `env.${envName}.${field}`;
-		if (!Array.isArray(envValue)) {
-			diagnostics.errors.push(
-				`The field "${fieldPath}" should be an array but got ${JSON.stringify(
-					envValue
-				)}.`
-			);
-			return false;
-		}
-
-		let isValid = true;
-		for (let i = 0; i < envValue.length; i++) {
-			isValid =
-				validateBinding(
-					diagnostics,
-					`${fieldPath}[${i}]`,
-					envValue[i],
-					config
-				) && isValid;
-		}
-		const configValue = config?.[field as keyof Environment] as {
-			binding: unknown;
-		}[];
-		if (Array.isArray(configValue)) {
-			const configBindingNames = configValue.map((value) => value.binding);
-			// If there are no top level bindings then there is nothing to do here.
-			if (configBindingNames.length > 0) {
-				const envBindingNames = new Set(envValue.map((value) => value.binding));
-				for (const configBindingName of configBindingNames) {
-					if (!envBindingNames.has(configBindingName)) {
-						diagnostics.warnings.push(
-							`There is a ${field} binding with name "${configBindingName}" at the top level, but not on "env.${envName}".\n` +
-								`This is not what you probably want, since "${field}" configuration is not inherited by environments.\n` +
-								`Please add a binding for "${configBindingName}" to "env.${envName}.${field}.bindings".`
-						);
-					}
-				}
-			}
-		}
-		return isValid;
-	};
-
-const validateKVBinding: ValidatorFn = (diagnostics, field, value) => {
-	if (typeof value !== "object" || value === null) {
-		diagnostics.errors.push(
-			`"kv_namespaces" bindings should be objects, but got ${JSON.stringify(
-				value
-			)}`
-		);
-		return false;
-	}
-	let isValid = true;
-	// KV bindings must have a binding and id.
-	if (!isRequiredProperty(value, "binding", "string")) {
-		diagnostics.errors.push(
-			`"${field}" bindings should have a string "binding" field but got ${JSON.stringify(
-				value
-			)}.`
-		);
-		isValid = false;
-	}
-	if (!isRequiredProperty(value, "id", "string")) {
-		diagnostics.errors.push(
-			`"${field}" bindings should have a string "id" field but got ${JSON.stringify(
-				value
-			)}.`
-		);
-		isValid = false;
-	}
-	if (!isOptionalProperty(value, "preview_id", "string")) {
-		diagnostics.errors.push(
-			`"${field}" bindings should, optionally, have a string "preview_id" field but got ${JSON.stringify(
-				value
-			)}.`
-		);
-		isValid = false;
-	}
-	return isValid;
-};
-
-const validateR2Binding: ValidatorFn = (diagnostics, field, value) => {
-	if (typeof value !== "object" || value === null) {
-		diagnostics.errors.push(
-			`"r2_buckets" bindings should be objects, but got ${JSON.stringify(
-				value
-			)}`
-		);
-		return false;
-	}
-	let isValid = true;
-	// R2 bindings must have a binding and bucket_name.
-	if (!isRequiredProperty(value, "binding", "string")) {
-		diagnostics.errors.push(
-			`"${field}" bindings should have a string "binding" field but got ${JSON.stringify(
-				value
-			)}.`
-		);
-		isValid = false;
-	}
-	if (!isRequiredProperty(value, "bucket_name", "string")) {
-		diagnostics.errors.push(
-			`"${field}" bindings should have a string "bucket_name" field but got ${JSON.stringify(
-				value
-			)}.`
-		);
-		isValid = false;
-	}
-	if (!isOptionalProperty(value, "preview_bucket_name", "string")) {
-		diagnostics.errors.push(
-			`"${field}" bindings should, optionally, have a string "preview_bucket_name" field but got ${JSON.stringify(
-				value
-			)}.`
-		);
-		isValid = false;
-	}
-	return isValid;
-};
-
-/**
- * Check that bindings whose names might conflict, don't.
- *
- * We don't want to have, for example, a KV namespace named "DATA"
- * and a Durable Object also named "DATA". Then it would be ambiguous
- * what exactly would live at `env.DATA` (or in the case of service-workers,
- * the `DATA` global).
- */
-const validateBindingsHaveUniqueNames = (
-	diagnostics: Diagnostics,
-	{
-		durable_objects,
-		kv_namespaces,
-		r2_buckets,
-		text_blobs,
-		unsafe,
-		vars,
-		define,
-		wasm_modules,
-		data_blobs,
-	}: Partial<Config>
-): boolean => {
-	let hasDuplicates = false;
-
-	const bindingsGroupedByType = {
-		"Durable Object": getBindingNames(durable_objects),
-		"KV Namespace": getBindingNames(kv_namespaces),
-		"R2 Bucket": getBindingNames(r2_buckets),
-		"Text Blob": getBindingNames(text_blobs),
-		Unsafe: getBindingNames(unsafe),
-		"Environment Variable": getBindingNames(vars),
-		Definition: getBindingNames(define),
-		"WASM Module": getBindingNames(wasm_modules),
-		"Data Blob": getBindingNames(data_blobs),
-	} as Record<string, string[]>;
-
-	const bindingsGroupedByName: Record<string, string[]> = {};
-
-	for (const bindingType in bindingsGroupedByType) {
-		const bindingNames = bindingsGroupedByType[bindingType];
-
-		for (const bindingName of bindingNames) {
-			if (!(bindingName in bindingsGroupedByName)) {
-				bindingsGroupedByName[bindingName] = [];
-			}
-
-			bindingsGroupedByName[bindingName].push(bindingType);
-		}
-	}
-
-	for (const bindingName in bindingsGroupedByName) {
-		const bindingTypes = bindingsGroupedByName[bindingName];
-		if (bindingTypes.length < 2) {
-			// there's only one (or zero) binding(s) with this name, which is fine, actually
-			continue;
-		}
-
-		hasDuplicates = true;
-
-		// there's two types of duplicates we want to look for:
-		// - bindings with the same name of the same type (e.g. two Durable Objects both named "OBJ")
-		// - bindings with the same name of different types (a KV namespace and DO both named "DATA")
-
-		const sameType = bindingTypes
-			// filter once to find duplicate binding types
-			.filter((type, i) => bindingTypes.indexOf(type) !== i)
-			// filter twice to only get _unique_ duplicate binding types
-			.filter(
-				(type, i, duplicateBindingTypes) =>
-					duplicateBindingTypes.indexOf(type) === i
-			);
-
-		const differentTypes = bindingTypes.filter(
-			(type, i) => bindingTypes.indexOf(type) === i
-		);
-
-		if (differentTypes.length > 1) {
-			// we have multiple different types using the same name
-			diagnostics.errors.push(
-				`${bindingName} assigned to ${ENGLISH.format(differentTypes)} bindings.`
-			);
-		}
-
-		sameType.forEach((bindingType) => {
-			diagnostics.errors.push(
-				`${bindingName} assigned to multiple ${bindingType} bindings.`
-			);
-		});
-	}
-
-	if (hasDuplicates) {
-		const problem =
-			"Bindings must have unique names, so that they can all be referenced in the worker.";
-		const resolution = "Please change your bindings to have unique names.";
-		diagnostics.errors.push(`${problem}\n${resolution}`);
-	}
-
-	return !hasDuplicates;
-};
-
-const validateServiceBinding: ValidatorFn = (diagnostics, field, value) => {
-	if (typeof value !== "object" || value === null) {
-		diagnostics.errors.push(
-			`"services" bindings should be objects, but got ${JSON.stringify(value)}`
-		);
-		return false;
-	}
-	let isValid = true;
-	// Service bindings must have a binding, service, and environment.
-	if (!isRequiredProperty(value, "binding", "string")) {
-		diagnostics.errors.push(
-			`"${field}" bindings should have a string "binding" field but got ${JSON.stringify(
-				value
-			)}.`
-		);
-		isValid = false;
-	}
-	if (!isRequiredProperty(value, "service", "string")) {
-		diagnostics.errors.push(
-			`"${field}" bindings should have a string "service" field but got ${JSON.stringify(
-				value
-			)}.`
-		);
-		isValid = false;
-	}
-	if (!isOptionalProperty(value, "environment", "string")) {
-		diagnostics.errors.push(
-			`"${field}" bindings should have a string "environment" field but got ${JSON.stringify(
-				value
-			)}.`
-		);
-		isValid = false;
-	}
-	return isValid;
-};
-
-const validateWorkerNamespaceBinding: ValidatorFn = (
-	diagnostics,
-	field,
-	value
-) => {
-	if (typeof value !== "object" || value === null) {
-		diagnostics.errors.push(
-			`"${field}" binding should be objects, but got ${JSON.stringify(value)}`
-		);
-		return false;
-	}
-	let isValid = true;
-	// Worker namespace bindings must have a binding, and a namespace.
-	if (!isRequiredProperty(value, "binding", "string")) {
-		diagnostics.errors.push(
-			`"${field}" should have a string "binding" field but got ${JSON.stringify(
-				value
-			)}.`
-		);
-		isValid = false;
-	}
-	if (!isRequiredProperty(value, "namespace", "string")) {
-		diagnostics.errors.push(
-			`"${field}" should have a string "namespace" field but got ${JSON.stringify(
-				value
-			)}.`
-		);
-		isValid = false;
-	}
-	return isValid;
-};