refactor: move projectRoot computation to config validation

packages/wrangler/src/__tests__/api/startDevWorker/LocalRuntimeController.test.ts

@@ -88,7 +88,6 @@ function makeEsbuildBundle(testBundle: TestBundle): Bundle {
 		entrypointSource: "",
 		entry: {
 			file: "index.mjs",
-			projectRoot: "/virtual/",
 			format: "modules",
 			moduleRoot: "/virtual",
 			name: undefined,
@@ -233,7 +232,6 @@ describe("LocalRuntimeController", () => {
 			`,
 				entry: {
 					file: "esm/index.mjs",
-					projectRoot: "/virtual/",
 					format: "modules",
 					moduleRoot: "/virtual",
 					name: undefined,
@@ -347,7 +345,6 @@ describe("LocalRuntimeController", () => {
 				path: "/virtual/index.js",
 				entry: {
 					file: "index.js",
-					projectRoot: "/virtual/",
 					format: "service-worker",
 					moduleRoot: "/virtual",
 					name: undefined,

packages/wrangler/src/__tests__/configuration.test.ts

@@ -63,6 +63,7 @@ describe("normalizeAndValidateConfig()", () => {
 			compatibility_date: undefined,
 			compatibility_flags: [],
 			configPath: undefined,
+			projectRoot: process.cwd(),
 			d1_databases: [],
 			vectorize: [],
 			hyperdrive: [],

packages/wrangler/src/__tests__/find-additional-modules.test.ts

@@ -37,9 +37,9 @@ describe("traverse module graph", () => {
 		);
 
 		const modules = await findAdditionalModules(
+			process.cwd(),
 			{
 				file: path.join(process.cwd(), "./index.js"),
-				projectRoot: process.cwd(),
 				format: "modules",
 				moduleRoot: process.cwd(),
 				exports: [],
@@ -73,9 +73,9 @@ describe("traverse module graph", () => {
 		);
 
 		const modules = await findAdditionalModules(
+			process.cwd(),
 			{
 				file: path.join(process.cwd(), "./index.js"),
-				projectRoot: process.cwd(),
 				format: "modules",
 				moduleRoot: process.cwd(),
 				exports: [],
@@ -107,9 +107,9 @@ describe("traverse module graph", () => {
 		);
 
 		const modules = await findAdditionalModules(
+			path.join(process.cwd(), "./src/nested"),
 			{
 				file: path.join(process.cwd(), "./src/nested/index.js"),
-				projectRoot: path.join(process.cwd(), "./src/nested"),
 				format: "modules",
 				// The default module root is dirname(file)
 				moduleRoot: path.join(process.cwd(), "./src/nested"),
@@ -142,9 +142,9 @@ describe("traverse module graph", () => {
 		);
 
 		const modules = await findAdditionalModules(
+			path.join(process.cwd(), "./src/nested"),
 			{
 				file: path.join(process.cwd(), "./src/nested/index.js"),
-				projectRoot: path.join(process.cwd(), "./src/nested"),
 				format: "modules",
 				// The default module root is dirname(file)
 				moduleRoot: path.join(process.cwd(), "./src"),
@@ -177,9 +177,9 @@ describe("traverse module graph", () => {
 		);
 
 		const modules = await findAdditionalModules(
+			path.join(process.cwd(), "./src/nested"),
 			{
 				file: path.join(process.cwd(), "./src/nested/index.js"),
-				projectRoot: path.join(process.cwd(), "./src/nested"),
 				format: "modules",
 				// The default module root is dirname(file)
 				moduleRoot: path.join(process.cwd(), "./src"),
@@ -212,9 +212,9 @@ describe("traverse module graph", () => {
 		);
 
 		const modules = await findAdditionalModules(
+			path.join(process.cwd(), "./src"),
 			{
 				file: path.join(process.cwd(), "./src/index.js"),
-				projectRoot: path.join(process.cwd(), "./src"),
 				format: "modules",
 				// The default module root is dirname(file)
 				moduleRoot: path.join(process.cwd(), "./src"),
@@ -247,9 +247,9 @@ describe("traverse module graph", () => {
 
 		await expect(
 			findAdditionalModules(
+				path.join(process.cwd(), "./src"),
 				{
 					file: path.join(process.cwd(), "./src/index.js"),
-					projectRoot: path.join(process.cwd(), "./src"),
 					format: "modules",
 					// The default module root is dirname(file)
 					moduleRoot: path.join(process.cwd(), "./src"),

packages/wrangler/src/__tests__/get-entry.test.ts

@@ -5,8 +5,16 @@ import { getEntry } from "../deployment-bundle/entry";
 import { mockConsoleMethods } from "./helpers/mock-console";
 import { runInTempDir } from "./helpers/run-in-tmp";
 import { seed } from "./helpers/seed";
+import type { Config } from "../config/config";
 import type { Entry } from "../deployment-bundle/entry";
 
+function getConfig(): Config {
+	return {
+		...defaultWranglerConfig,
+		projectRoot: process.cwd(),
+	};
+}
+
 function normalize(entry: Entry): Entry {
 	const tmpDir = process.cwd();
 	const tmpDirName = path.basename(tmpDir);
@@ -37,13 +45,8 @@ describe("getEntry()", () => {
 							}
 						`,
 		});
-		const entry = await getEntry(
-			{ script: "index.ts" },
-			defaultWranglerConfig,
-			"deploy"
-		);
+		const entry = await getEntry({ script: "index.ts" }, getConfig(), "deploy");
 		expect(normalize(entry)).toMatchObject({
-			projectRoot: "/tmp/dir",
 			file: "/tmp/dir/index.ts",
 			moduleRoot: "/tmp/dir",
 		});
@@ -61,11 +64,10 @@ describe("getEntry()", () => {
 		});
 		const entry = await getEntry(
 			{ script: "src/index.ts" },
-			defaultWranglerConfig,
+			getConfig(),
 			"deploy"
 		);
 		expect(normalize(entry)).toMatchObject({
-			projectRoot: "/tmp/dir",
 			file: "/tmp/dir/src/index.ts",
 			moduleRoot: "/tmp/dir/src",
 		});
@@ -83,11 +85,10 @@ describe("getEntry()", () => {
 		});
 		const entry = await getEntry(
 			{},
-			{ ...defaultWranglerConfig, main: "index.ts" },
+			{ ...getConfig(), main: "index.ts" },
 			"deploy"
 		);
 		expect(normalize(entry)).toMatchObject({
-			projectRoot: "/tmp/dir",
 			file: "/tmp/dir/index.ts",
 			moduleRoot: "/tmp/dir",
 		});
@@ -105,11 +106,10 @@ describe("getEntry()", () => {
 		});
 		const entry = await getEntry(
 			{},
-			{ ...defaultWranglerConfig, main: "src/index.ts" },
+			{ ...getConfig(), main: "src/index.ts" },
 			"deploy"
 		);
 		expect(normalize(entry)).toMatchObject({
-			projectRoot: "/tmp/dir",
 			file: "/tmp/dir/src/index.ts",
 			moduleRoot: "/tmp/dir/src",
 		});
@@ -128,14 +128,14 @@ describe("getEntry()", () => {
 		const entry = await getEntry(
 			{},
 			{
-				...defaultWranglerConfig,
+				...getConfig(),
 				main: "src/index.ts",
 				configPath: "other-worker/wrangler.toml",
+				projectRoot: "other-worker",
 			},
 			"deploy"
 		);
 		expect(normalize(entry)).toMatchObject({
-			projectRoot: "/tmp/dir/other-worker",
 			file: "/tmp/dir/other-worker/src/index.ts",
 			moduleRoot: "/tmp/dir/other-worker/src",
 		});

packages/wrangler/src/__tests__/navigator-user-agent.test.ts

@@ -104,7 +104,6 @@ describe("defineNavigatorUserAgent is respected", () => {
 		await bundleWorker(
 			{
 				file: path.resolve("src/index.js"),
-				projectRoot: process.cwd(),
 				format: "modules",
 				moduleRoot: path.dirname(path.resolve("src/index.js")),
 				exports: [],
@@ -167,7 +166,6 @@ describe("defineNavigatorUserAgent is respected", () => {
 		await bundleWorker(
 			{
 				file: path.resolve("src/index.js"),
-				projectRoot: process.cwd(),
 				format: "modules",
 				moduleRoot: path.dirname(path.resolve("src/index.js")),
 				exports: [],

packages/wrangler/src/__tests__/pages/deploy.test.ts

@@ -2855,7 +2855,7 @@ Failed to publish your Function. Got error: Uncaught TypeError: a is not a funct
 		});
 	});
 
-	describe("in Advanced Mode [_worker,js]", () => {
+	describe("in Advanced Mode [_worker.js]", () => {
 		it("should upload an Advanced Mode project", async () => {
 			// set up the directory of static files to upload.
 			mkdirSync("public");

packages/wrangler/src/api/startDevWorker/BundlerController.ts

@@ -79,14 +79,14 @@ export class BundlerController extends Controller<BundlerControllerEventMap> {
 
 			const entry: Entry = {
 				file: config.entrypoint,
-				projectRoot: config.projectRoot,
 				format: config.build.format,
 				moduleRoot: config.build.moduleRoot,
 				exports: config.build.exports,
 			};
 
 			const entryDirectory = path.dirname(config.entrypoint);
 			const moduleCollector = createModuleCollector({
+				projectRoot: config.projectRoot,
 				wrangler1xLegacyModuleReferences: getWrangler1xLegacyModuleReferences(
 					entryDirectory,
 					config.entrypoint
@@ -103,6 +103,7 @@ export class BundlerController extends Controller<BundlerControllerEventMap> {
 			).bindings;
 			const bundleResult: Omit<BundleResult, "stop"> = !config.build?.bundle
 				? await noBundleWorker(
+						config.projectRoot,
 						entry,
 						config.build.moduleRules,
 						this.#tmpDir.path
@@ -234,7 +235,6 @@ export class BundlerController extends Controller<BundlerControllerEventMap> {
 		assert(this.#tmpDir);
 		const entry: Entry = {
 			file: config.entrypoint,
-			projectRoot: config.projectRoot,
 			format: config.build.format,
 			moduleRoot: config.build.moduleRoot,
 			exports: config.build.exports,

packages/wrangler/src/api/startDevWorker/ConfigController.ts

@@ -263,7 +263,7 @@ async function resolveConfig(
 		compatibilityDate: getDevCompatibilityDate(config, input.compatibilityDate),
 		compatibilityFlags: input.compatibilityFlags ?? config.compatibility_flags,
 		entrypoint: entry.file,
-		projectRoot: entry.projectRoot,
+		projectRoot: config.projectRoot,
 		bindings,
 		migrations: input.migrations ?? config.migrations,
 		sendMetrics: input.sendMetrics ?? config.send_metrics,

packages/wrangler/src/config/config.ts

@@ -22,17 +22,25 @@ import type { CamelCaseKey } from "yargs";
  * - `@breaking`: the deprecation/optionality is a breaking change from Wrangler v1.
  * - `@todo`: there's more work to be done (with details attached).
  */
-export type Config = ConfigFields<DevConfig> & PagesConfigFields & Environment;
+export type Config = ComputedConfigFields &
+	ConfigFields<DevConfig> &
+	PagesConfigFields &
+	Environment;
 
 export type RawConfig = Partial<ConfigFields<RawDevConfig>> &
 	PagesConfigFields &
 	RawEnvironment &
 	DeprecatedConfigFields &
 	EnvironmentMap & { $schema?: string };
 
-export interface ConfigFields<Dev extends RawDevConfig> {
+export interface ComputedConfigFields {
+	/** Path to the configuration file (e.g. wrangler.toml/json), if one was provided. */
 	configPath: string | undefined;
 
+	/** A worker's directory. Usually where the Wrangler configuration file is located */
+	projectRoot: string;
+}
+export interface ConfigFields<Dev extends RawDevConfig> {
 	/**
 	 * A boolean to enable "legacy" style wrangler environments (from Wrangler v1).
 	 * These have been superseded by Services, but there may be projects that won't
@@ -325,8 +333,11 @@ export const defaultWranglerConfig: Config = {
 	/*====================================================*/
 	/*           Fields supported by Workers only         */
 	/*====================================================*/
-	/* TOP-LEVEL ONLY FIELDS */
+	/* COMPUTED CONFIG FIELDS */
 	configPath: undefined,
+	projectRoot: process.cwd(),
+
+	/* TOP-LEVEL ONLY FIELDS */
 	legacy_env: true,
 	site: undefined,
 	legacy_assets: undefined,

packages/wrangler/src/config/validation-pages.ts

@@ -36,9 +36,10 @@ const supportedPagesConfigFields = [
 	"dev",
 	"mtls_certificates",
 	"browser",
-	// normalizeAndValidateConfig() sets this value
-	"configPath",
 	"upload_source_maps",
+	// normalizeAndValidateConfig() sets the following values
+	"configPath",
+	"projectRoot",
 ] as const;
 
 export function validatePagesConfig(

packages/wrangler/src/config/validation.ts

@@ -260,6 +260,9 @@ export function normalizeAndValidateConfig(
 	// Process the top-level default environment configuration.
 	const config: Config = {
 		configPath,
+		projectRoot: path.resolve(
+			configPath !== undefined ? path.dirname(configPath) : process.cwd()
+		),
 		pages_build_output_dir: normalizeAndValidatePagesBuildOutputDir(
 			configPath,
 			rawConfig.pages_build_output_dir

packages/wrangler/src/d1/execute.ts

@@ -19,7 +19,7 @@ import { requireAuth } from "../user";
 import * as options from "./options";
 import splitSqlQuery from "./splitter";
 import { getDatabaseByNameOrBinding, getDatabaseInfoFromConfig } from "./utils";
-import type { Config, ConfigFields, DevConfig, Environment } from "../config";
+import type { Config } from "../config";
 import type {
 	CommonYargsArgv,
 	StrictYargsOptionsToInterface,
@@ -197,7 +197,7 @@ export async function executeSql({
 }: {
 	local: boolean | undefined;
 	remote: boolean | undefined;
-	config: ConfigFields<DevConfig> & Environment;
+	config: Config;
 	name: string;
 	shouldPrompt: boolean | undefined;
 	persistTo: string | undefined;

packages/wrangler/src/d1/migrations/helpers.ts

@@ -7,7 +7,7 @@ import { isNonInteractiveOrCI } from "../../is-interactive";
 import { logger } from "../../logger";
 import { DEFAULT_MIGRATION_PATH } from "../constants";
 import { executeSql } from "../execute";
-import type { ConfigFields, DevConfig, Environment } from "../../config";
+import type { Config } from "../../config";
 import type { QueryResult } from "../execute";
 import type { Migration } from "../types";
 
@@ -57,7 +57,7 @@ export async function getUnappliedMigrations({
 	migrationsPath: string;
 	local: boolean | undefined;
 	remote: boolean | undefined;
-	config: ConfigFields<DevConfig> & Environment;
+	config: Config;
 	name: string;
 	persistTo: string | undefined;
 	preview: boolean | undefined;
@@ -92,7 +92,7 @@ type ListAppliedMigrationsProps = {
 	migrationsTableName: string;
 	local: boolean | undefined;
 	remote: boolean | undefined;
-	config: ConfigFields<DevConfig> & Environment;
+	config: Config;
 	name: string;
 	persistTo: string | undefined;
 	preview: boolean | undefined;
@@ -170,7 +170,7 @@ export const initMigrationsTable = async ({
 	migrationsTableName: string;
 	local: boolean | undefined;
 	remote: boolean | undefined;
-	config: ConfigFields<DevConfig> & Environment;
+	config: Config;
 	name: string;
 	persistTo: string | undefined;
 	preview: boolean | undefined;