wrangler 2.0.16 → 2.0.19

package/src/__tests__/secret.test.ts

@@ -73,9 +73,9 @@ describe("wrangler secret", () => {
 				mockPutRequest({ name: `secret-name`, text: `hunter2` });
 				await runWrangler("secret put secret-name --name script-name");
 				expect(std.out).toMatchInlineSnapshot(`
-          "🌀 Creating the secret for script script-name
-          ✨ Success! Uploaded secret secret-name"
-        `);
+			"🌀 Creating the secret for the Worker \\"script-name\\"
+			✨ Success! Uploaded secret secret-name"
+		`);
 			});
 
 			it("should create a secret", async () => {
@@ -89,9 +89,9 @@ describe("wrangler secret", () => {
 				await runWrangler("secret put the-key --name script-name");
 
 				expect(std.out).toMatchInlineSnapshot(`
-                  "🌀 Creating the secret for script script-name
-                  ✨ Success! Uploaded secret the-key"
-              `);
+			"🌀 Creating the secret for the Worker \\"script-name\\"
+			✨ Success! Uploaded secret the-key"
+		`);
 				expect(std.err).toMatchInlineSnapshot(`""`);
 			});
 
@@ -112,9 +112,9 @@ describe("wrangler secret", () => {
 				);
 
 				expect(std.out).toMatchInlineSnapshot(`
-                  "🌀 Creating the secret for script script-name-some-env
-                  ✨ Success! Uploaded secret the-key"
-              `);
+			"🌀 Creating the secret for the Worker \\"script-name-some-env\\"
+			✨ Success! Uploaded secret the-key"
+		`);
 				expect(std.err).toMatchInlineSnapshot(`""`);
 			});
 
@@ -135,9 +135,9 @@ describe("wrangler secret", () => {
 				);
 
 				expect(std.out).toMatchInlineSnapshot(`
-                  "🌀 Creating the secret for script script-name (some-env)
-                  ✨ Success! Uploaded secret the-key"
-              `);
+			"🌀 Creating the secret for the Worker \\"script-name\\" (some-env)
+			✨ Success! Uploaded secret the-key"
+		`);
 				expect(std.err).toMatchInlineSnapshot(`""`);
 			});
 
@@ -149,15 +149,17 @@ describe("wrangler secret", () => {
 					error = e as Error;
 				}
 				expect(std.out).toMatchInlineSnapshot(`
-          "
-          If you think this is a bug then please create an issue at https://github.com/cloudflare/wrangler2/issues/new/choose"
-        `);
+			          "
+			          If you think this is a bug then please create an issue at https://github.com/cloudflare/wrangler2/issues/new/choose"
+		        `);
 				expect(std.err).toMatchInlineSnapshot(`
-          "X [ERROR] Missing script name
+			"X [ERROR] Required Worker name missing. Please specify the Worker name in wrangler.toml, or pass it as an argument with \`--name <worker-name>\`
 
-          "
-        `);
-				expect(error).toMatchInlineSnapshot(`[Error: Missing script name]`);
+			"
+		`);
+				expect(error).toMatchInlineSnapshot(
+					`[Error: Required Worker name missing. Please specify the Worker name in wrangler.toml, or pass it as an argument with \`--name <worker-name>\`]`
+				);
 			});
 		});
 
@@ -176,9 +178,9 @@ describe("wrangler secret", () => {
 				await runWrangler("secret put the-key --name script-name");
 
 				expect(std.out).toMatchInlineSnapshot(`
-          "🌀 Creating the secret for script script-name
-          ✨ Success! Uploaded secret the-key"
-        `);
+			"🌀 Creating the secret for the Worker \\"script-name\\"
+			✨ Success! Uploaded secret the-key"
+		`);
 				expect(std.warn).toMatchInlineSnapshot(`""`);
 				expect(std.err).toMatchInlineSnapshot(`""`);
 			});
@@ -190,9 +192,9 @@ describe("wrangler secret", () => {
 				await runWrangler("secret put the-key --name script-name");
 
 				expect(std.out).toMatchInlineSnapshot(`
-          "🌀 Creating the secret for script script-name
-          ✨ Success! Uploaded secret the-key"
-        `);
+			"🌀 Creating the secret for the Worker \\"script-name\\"
+			✨ Success! Uploaded secret the-key"
+		`);
 				expect(std.warn).toMatchInlineSnapshot(`""`);
 				expect(std.err).toMatchInlineSnapshot(`""`);
 			});
@@ -205,9 +207,9 @@ describe("wrangler secret", () => {
 				).rejects.toThrowErrorMatchingInlineSnapshot(`"Error in stdin stream"`);
 
 				expect(std.out).toMatchInlineSnapshot(`
-          "
-          If you think this is a bug then please create an issue at https://github.com/cloudflare/wrangler2/issues/new/choose"
-        `);
+			          "
+			          If you think this is a bug then please create an issue at https://github.com/cloudflare/wrangler2/issues/new/choose"
+		        `);
 				expect(std.warn).toMatchInlineSnapshot(`""`);
 			});
 
@@ -244,9 +246,9 @@ describe("wrangler secret", () => {
 					mockPutRequest({ name: "the-key", text: "the-secret" });
 					await runWrangler("secret put the-key --name script-name");
 					expect(std.out).toMatchInlineSnapshot(`
-            "🌀 Creating the secret for script script-name
-            ✨ Success! Uploaded secret the-key"
-          `);
+				"🌀 Creating the secret for the Worker \\"script-name\\"
+				✨ Success! Uploaded secret the-key"
+			`);
 					expect(std.warn).toMatchInlineSnapshot(`""`);
 					expect(std.err).toMatchInlineSnapshot(`""`);
 				});
@@ -317,14 +319,14 @@ describe("wrangler secret", () => {
 		it("should delete a secret", async () => {
 			mockDeleteRequest({ scriptName: "script-name", secretName: "the-key" });
 			mockConfirm({
-				text: "Are you sure you want to permanently delete the variable the-key on the script script-name?",
+				text: "Are you sure you want to permanently delete the secret the-key on the Worker script-name?",
 				result: true,
 			});
 			await runWrangler("secret delete the-key --name script-name");
 			expect(std.out).toMatchInlineSnapshot(`
-        "🌀 Deleting the secret the-key on script script-name
-        ✨ Success! Deleted secret the-key"
-      `);
+			"🌀 Deleting the secret the-key on the Worker script-name
+			✨ Success! Deleted secret the-key"
+		`);
 			expect(std.err).toMatchInlineSnapshot(`""`);
 		});
 
@@ -335,16 +337,16 @@ describe("wrangler secret", () => {
 				true
 			);
 			mockConfirm({
-				text: "Are you sure you want to permanently delete the variable the-key on the script script-name-some-env?",
+				text: "Are you sure you want to permanently delete the secret the-key on the Worker script-name-some-env?",
 				result: true,
 			});
 			await runWrangler(
 				"secret delete the-key --name script-name --env some-env --legacy-env"
 			);
 			expect(std.out).toMatchInlineSnapshot(`
-        "🌀 Deleting the secret the-key on script script-name-some-env
-        ✨ Success! Deleted secret the-key"
-      `);
+			"🌀 Deleting the secret the-key on the Worker script-name-some-env
+			✨ Success! Deleted secret the-key"
+		`);
 			expect(std.err).toMatchInlineSnapshot(`""`);
 		});
 
@@ -354,16 +356,16 @@ describe("wrangler secret", () => {
 				"some-env"
 			);
 			mockConfirm({
-				text: "Are you sure you want to permanently delete the variable the-key on the script script-name (some-env)?",
+				text: "Are you sure you want to permanently delete the secret the-key on the Worker script-name (some-env)?",
 				result: true,
 			});
 			await runWrangler(
 				"secret delete the-key --name script-name --env some-env --legacy-env false"
 			);
 			expect(std.out).toMatchInlineSnapshot(`
-        "🌀 Deleting the secret the-key on script script-name (some-env)
-        ✨ Success! Deleted secret the-key"
-      `);
+			"🌀 Deleting the secret the-key on the Worker script-name (some-env)
+			✨ Success! Deleted secret the-key"
+		`);
 			expect(std.err).toMatchInlineSnapshot(`""`);
 		});
 
@@ -376,15 +378,17 @@ describe("wrangler secret", () => {
 				error = e as Error;
 			}
 			expect(std.out).toMatchInlineSnapshot(`
-        "
-        If you think this is a bug then please create an issue at https://github.com/cloudflare/wrangler2/issues/new/choose"
-      `);
+			        "
+			        If you think this is a bug then please create an issue at https://github.com/cloudflare/wrangler2/issues/new/choose"
+		      `);
 			expect(std.err).toMatchInlineSnapshot(`
-        "X [ERROR] Missing script name
+			"X [ERROR] Required Worker name missing. Please specify the Worker name in wrangler.toml, or pass it as an argument with \`--name <worker-name>\`
 
-        "
-      `);
-			expect(error).toMatchInlineSnapshot(`[Error: Missing script name]`);
+			"
+		`);
+			expect(error).toMatchInlineSnapshot(
+				`[Error: Required Worker name missing. Please specify the Worker name in wrangler.toml, or pass it as an argument with \`--name <worker-name>\`]`
+			);
 		});
 	});
 
@@ -422,13 +426,13 @@ describe("wrangler secret", () => {
 			mockListRequest({ scriptName: "script-name" });
 			await runWrangler("secret list --name script-name");
 			expect(std.out).toMatchInlineSnapshot(`
-        "[
-          {
-            \\"name\\": \\"the-secret-name\\",
-            \\"type\\": \\"secret_text\\"
-          }
-        ]"
-      `);
+			        "[
+			          {
+			            \\"name\\": \\"the-secret-name\\",
+			            \\"type\\": \\"secret_text\\"
+			          }
+			        ]"
+		      `);
 			expect(std.err).toMatchInlineSnapshot(`""`);
 		});
 
@@ -438,13 +442,13 @@ describe("wrangler secret", () => {
 				"secret list --name script-name --env some-env --legacy-env"
 			);
 			expect(std.out).toMatchInlineSnapshot(`
-        "[
-          {
-            \\"name\\": \\"the-secret-name\\",
-            \\"type\\": \\"secret_text\\"
-          }
-        ]"
-      `);
+			        "[
+			          {
+			            \\"name\\": \\"the-secret-name\\",
+			            \\"type\\": \\"secret_text\\"
+			          }
+			        ]"
+		      `);
 			expect(std.err).toMatchInlineSnapshot(`""`);
 		});
 
@@ -454,13 +458,13 @@ describe("wrangler secret", () => {
 				"secret list --name script-name --env some-env --legacy-env false"
 			);
 			expect(std.out).toMatchInlineSnapshot(`
-        "[
-          {
-            \\"name\\": \\"the-secret-name\\",
-            \\"type\\": \\"secret_text\\"
-          }
-        ]"
-      `);
+			        "[
+			          {
+			            \\"name\\": \\"the-secret-name\\",
+			            \\"type\\": \\"secret_text\\"
+			          }
+			        ]"
+		      `);
 			expect(std.err).toMatchInlineSnapshot(`""`);
 		});
 
@@ -472,15 +476,17 @@ describe("wrangler secret", () => {
 				error = e as Error;
 			}
 			expect(std.out).toMatchInlineSnapshot(`
-        "
-        If you think this is a bug then please create an issue at https://github.com/cloudflare/wrangler2/issues/new/choose"
-      `);
+			        "
+			        If you think this is a bug then please create an issue at https://github.com/cloudflare/wrangler2/issues/new/choose"
+		      `);
 			expect(std.err).toMatchInlineSnapshot(`
-        "X [ERROR] Missing script name
+			"X [ERROR] Required Worker name missing. Please specify the Worker name in wrangler.toml, or pass it as an argument with \`--name <worker-name>\`
 
-        "
-      `);
-			expect(error).toMatchInlineSnapshot(`[Error: Missing script name]`);
+			"
+		`);
+			expect(error).toMatchInlineSnapshot(
+				`[Error: Required Worker name missing. Please specify the Worker name in wrangler.toml, or pass it as an argument with \`--name <worker-name>\`]`
+			);
 		});
 	});
 });

package/src/__tests__/tail.test.ts

@@ -27,6 +27,14 @@ describe("tail", () => {
 	 * deletion, and connection.
 	 */
 	describe("API interaction", () => {
+		it("should throw an error if name isn't provided", async () => {
+			await expect(
+				runWrangler("tail")
+			).rejects.toThrowErrorMatchingInlineSnapshot(
+				`"Required Worker name missing. Please specify the Worker name in wrangler.toml, or pass it as an argument with \`wrangler tail <worker-name>\`"`
+			);
+		});
+
 		it("creates and then delete tails", async () => {
 			const api = mockWebsocketAPIs();
 			expect(api.requests.creation.count).toStrictEqual(0);

package/src/__tests__/test-old-node-version.js

@@ -0,0 +1,31 @@
+// this test has to be run with a version of node.js older than 16.7 to pass
+
+const { spawn } = require("child_process");
+const path = require("path");
+
+const assert = require("assert");
+
+const wranglerProcess = spawn(
+	"node",
+	[path.join(__dirname, "../../bin/wrangler.js")],
+	{ stdio: "pipe" }
+);
+
+const messageToMatch = "Wrangler requires at least node.js v16.7.0";
+
+wranglerProcess.once("exit", (code) => {
+	try {
+		const errorMessage = wranglerProcess.stderr.read().toString();
+
+		assert(code === 1, "Expected exit code 1");
+		assert(
+			errorMessage.includes(messageToMatch),
+			`Expected error message to include "${messageToMatch}"`
+		);
+	} catch (err) {
+		console.error("Error:", err);
+		throw new Error(
+			"This test has to be run with a version of node.js under 16.7 to pass"
+		);
+	}
+});

package/src/__tests__/user.test.ts

@@ -1,7 +1,7 @@
 import fs from "node:fs";
-import os from "node:os";
 import path from "node:path";
 import fetchMock from "jest-fetch-mock";
+import { getGlobalWranglerConfigPath } from "../global-wrangler-config-path";
 import {
 	loginOrRefreshIfRequired,
 	readAuthConfigFile,
@@ -18,7 +18,7 @@ import type { Config } from "../config";
 import type { UserAuthConfig } from "../user";
 
 describe("User", () => {
-	runInTempDir({ homedir: "./home" });
+	runInTempDir();
 	const std = mockConsoleMethods();
 	const {
 		mockOAuthServerCallback,
@@ -86,7 +86,10 @@ describe("User", () => {
 			expect(fetchMock).toHaveBeenCalledTimes(1);
 
 			// Make sure that logout removed the config file containing the auth tokens.
-			const config = path.join(os.homedir(), USER_AUTH_CONFIG_FILE);
+			const config = path.join(
+				getGlobalWranglerConfigPath(),
+				USER_AUTH_CONFIG_FILE
+			);
 			expect(fs.existsSync(config)).toBeFalsy();
 		});
 	});
@@ -94,13 +97,10 @@ describe("User", () => {
 	// TODO: Improve OAuth mocking to handle `/token` endpoints from different calls
 	it("should handle errors for failed token refresh", async () => {
 		setIsTTY(false);
-		mockOAuthServerCallback();
 		writeAuthConfigFile({
 			oauth_token: "hunter2",
 			refresh_token: "Order 66",
 		});
-		mockGrantAuthorization({ respondWith: "success" });
-
 		mockExchangeRefreshTokenForAccessToken({
 			respondWith: "badResponse",
 		});
@@ -109,7 +109,7 @@ describe("User", () => {
 		await expect(
 			requireAuth({} as Config)
 		).rejects.toThrowErrorMatchingInlineSnapshot(
-			`"Did not login, quitting..."`
+			`"In a non-interactive environment, it's necessary to set a CLOUDFLARE_API_TOKEN environment variable for wrangler to work. Please go to https://developers.cloudflare.com/api/tokens/create/ for instructions on how to create an api token, and assign its value to CLOUDFLARE_API_TOKEN."`
 		);
 	});
 

package/src/__tests__/whoami.test.tsx

@@ -16,7 +16,7 @@ import type { UserInfo } from "../whoami";
 describe("getUserInfo()", () => {
 	const ENV_COPY = process.env;
 
-	runInTempDir({ homedir: "./home" });
+	runInTempDir();
 	const std = mockConsoleMethods();
 	const { setIsTTY } = useMockIsTTY();
 
@@ -173,6 +173,16 @@ describe("getUserInfo()", () => {
 
 	it("should display a warning message if the config file contains a legacy api_token field", async () => {
 		writeAuthConfigFile({ api_token: "API_TOKEN" });
+		setMockResponse("/user", () => {
+			return { email: "user@example.com" };
+		});
+		setMockResponse("/accounts", () => {
+			return [
+				{ name: "Account One", id: "account-1" },
+				{ name: "Account Two", id: "account-2" },
+				{ name: "Account Three", id: "account-3" },
+			];
+		});
 		await getUserInfo();
 		expect(std.warn).toMatchInlineSnapshot(`
       "▲ [WARNING] It looks like you have used Wrangler 1's \`config\` command to login with an API token.

package/src/api/dev.ts

@@ -43,7 +43,10 @@ export async function unstable_dev(script: string, options: DevOptions) {
 				showInteractiveDevSession: false,
 			});
 		}).then((devServer) => {
-			resolve({ stop: devServer.stop, fetch: devServer.fetch });
+			resolve({
+				stop: devServer.stop,
+				fetch: devServer.fetch,
+			});
 		});
 	});
 }

package/src/cli.ts

@@ -24,4 +24,4 @@ if (typeof jest === "undefined" && require.main) {
  * It makes it possible to import wrangler from 'wrangler',
  * and call wrangler.unstable_dev().
  */
-module.exports = { unstable_dev };
+export default { unstable_dev };

package/src/config/config.ts

@@ -38,6 +38,14 @@ export interface ConfigFields<Dev extends RawDevConfig> {
 	 */
 	legacy_env: boolean;
 
+	/**
+	 * Whether Wrangler should send usage metrics to Cloudflare for this project.
+	 *
+	 * When defined this will override any user settings.
+	 * Otherwise, Wrangler will use the user's preference.
+	 */
+	send_metrics: boolean | undefined;
+
 	/**
 	 * Options to configure the development server that your worker will use.
 	 */

package/src/config/validation.ts

@@ -94,6 +94,14 @@ export function normalizeAndValidateConfig(
 		"boolean"
 	);
 
+	validateOptionalProperty(
+		diagnostics,
+		"",
+		"send_metrics",
+		rawConfig.send_metrics,
+		"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"] ??
@@ -173,6 +181,7 @@ export function normalizeAndValidateConfig(
 	const config: Config = {
 		configPath,
 		legacy_env: isLegacyEnv,
+		send_metrics: rawConfig.send_metrics,
 		...activeEnv,
 		dev: normalizeAndValidateDev(diagnostics, rawConfig.dev ?? {}),
 		migrations: normalizeAndValidateMigrations(

package/src/config-cache.ts

@@ -6,7 +6,7 @@ import { logger } from "./logger";
 
 let cacheMessageShown = false;
 
-let __cacheFolder: string | null;
+let __cacheFolder: string | null | undefined;
 function getCacheFolder() {
 	if (__cacheFolder || __cacheFolder === null) return __cacheFolder;
 
@@ -80,4 +80,5 @@ export function purgeConfigCaches() {
 	if (cacheFolder) {
 		rmSync(cacheFolder, { recursive: true, force: true });
 	}
+	__cacheFolder = undefined;
 }

package/src/dev/local.tsx

@@ -1,4 +1,4 @@
-import { spawn } from "node:child_process";
+import { fork } from "node:child_process";
 import { realpathSync } from "node:fs";
 import { writeFile } from "node:fs/promises";
 import path from "node:path";
@@ -13,7 +13,7 @@ import type { AssetPaths } from "../sites";
 import type { CfWorkerInit, CfScriptFormat } from "../worker";
 import type { EsbuildBundle } from "./use-esbuild";
 import type { MiniflareOptions } from "miniflare";
-
+import type { ChildProcess } from "node:child_process";
 interface LocalProps {
 	name: string | undefined;
 	bundle: EsbuildBundle | undefined;
@@ -67,7 +67,7 @@ function useLocalWorker({
 	logLevel,
 }: LocalProps) {
 	// TODO: pass vars via command line
-	const local = useRef<ReturnType<typeof spawn>>();
+	const local = useRef<ChildProcess>();
 	const removeSignalExitListener = useRef<() => void>();
 	const [inspectorUrl, setInspectorUrl] = useState<string | undefined>();
 	// if we're using local persistence for data, we should use the cwd
@@ -187,6 +187,7 @@ function useLocalWorker({
 				compatibilityDate,
 				compatibilityFlags,
 				kvNamespaces: bindings.kv_namespaces?.map((kv) => kv.binding),
+				r2Buckets: bindings.r2_buckets?.map((r2) => r2.binding),
 				durableObjects: Object.fromEntries(
 					(bindings.durable_objects?.bindings ?? []).map<[string, string]>(
 						(value) => [value.name, value.class_name]
@@ -194,9 +195,10 @@ function useLocalWorker({
 				),
 				...(localPersistencePath
 					? {
-							kvPersist: path.join(localPersistencePath, "kv"),
-							durableObjectsPersist: path.join(localPersistencePath, "do"),
 							cachePersist: path.join(localPersistencePath, "cache"),
+							durableObjectsPersist: path.join(localPersistencePath, "do"),
+							kvPersist: path.join(localPersistencePath, "kv"),
+							r2Persist: path.join(localPersistencePath, "r2"),
 					  }
 					: {
 							// We mark these as true, so that they'll
@@ -204,9 +206,10 @@ function useLocalWorker({
 							// This means they'll persist across a dev session,
 							// even if we change source and reload,
 							// and be deleted when the dev session ends
-							durableObjectsPersist: true,
 							cachePersist: true,
+							durableObjectsPersist: true,
 							kvPersist: true,
+							r2Persist: true,
 					  }),
 
 				sitePath: assetPaths?.assetDirectory
@@ -236,43 +239,45 @@ function useLocalWorker({
 				__dirname,
 				"../miniflare-dist/index.mjs"
 			);
-			const optionsArg = JSON.stringify(options, null);
+			const miniflareOptions = JSON.stringify(options, null);
 
 			logger.log("⎔ Starting a local server...");
-			const localServerOptions = [
+			const nodeOptions = [
 				"--experimental-vm-modules", // ensures that Miniflare can run ESM Workers
 				"--no-warnings", // hide annoying Node warnings
-				miniflareCLIPath,
-				optionsArg,
 				// "--log=VERBOSE", // uncomment this to Miniflare to log "everything"!
 			];
 			if (inspect) {
-				localServerOptions.push("--inspect"); // start Miniflare listening for a debugger to attach
+				nodeOptions.push("--inspect"); // start Miniflare listening for a debugger to attach
 			}
-			// spawn isn't technically synchronous here
-			local.current = spawn("node", localServerOptions, {
-				cwd: path.dirname(scriptPath),
+			const child = (local.current = fork(
+				miniflareCLIPath,
+				[miniflareOptions],
+				{
+					cwd: path.dirname(scriptPath),
+					execArgv: nodeOptions,
+				}
+			));
+			child.on("message", (message) => {
+				if (message === "ready") {
+					onReady?.();
+				}
 			});
-			//TODO: instead of being lucky with spawn's timing, have miniflare-cli notify wrangler that it's ready in packages/wrangler/src/miniflare-cli/index.ts, after the mf.startScheduler promise resolves
-			if (onReady) {
-				await new Promise((resolve) => setTimeout(resolve, 500));
-				onReady();
-			}
 
-			local.current.on("close", (code) => {
+			child.on("close", (code) => {
 				if (code) {
 					logger.log(`Miniflare process exited with code ${code}`);
 				}
 			});
 
-			local.current.stdout?.on("data", (data: Buffer) => {
+			child.stdout?.on("data", (data: Buffer) => {
 				process.stdout.write(data);
 			});
 
 			// parse the node inspector url (which may be received in chunks) from stderr
 			let stderrData = "";
 			let inspectorUrlFound = false;
-			local.current.stderr?.on("data", (data: Buffer) => {
+			child.stderr?.on("data", (data: Buffer) => {
 				if (!inspectorUrlFound) {
 					stderrData += data.toString();
 					const matches =
@@ -288,20 +293,20 @@ function useLocalWorker({
 				process.stderr.write(data);
 			});
 
-			local.current.on("exit", (code) => {
+			child.on("exit", (code) => {
 				if (code) {
 					logger.error(`Miniflare process exited with code ${code}`);
 				}
 			});
 
-			local.current.on("error", (error: Error) => {
+			child.on("error", (error: Error) => {
 				logger.error(`Miniflare process failed to spawn`);
 				logger.error(error);
 			});
 
 			removeSignalExitListener.current = onExit((_code, _signal) => {
 				logger.log("⎔ Shutting down local server.");
-				local.current?.kill();
+				child.kill();
 				local.current = undefined;
 			});
 		}
@@ -328,6 +333,7 @@ function useLocalWorker({
 		ip,
 		bindings.durable_objects?.bindings,
 		bindings.kv_namespaces,
+		bindings.r2_buckets,
 		bindings.vars,
 		bindings.services,
 		compatibilityDate,

package/src/dev/remote.tsx

@@ -53,7 +53,7 @@ export function Remote(props: {
 	zone: string | undefined;
 	host: string | undefined;
 	routes: Route[] | undefined;
-	onReady?: () => void;
+	onReady?: (() => void) | undefined;
 }) {
 	const [accountId, setAccountId] = useState(props.accountId);
 	const accountChoicesRef = useRef<Promise<ChooseAccountItem[]>>();
@@ -158,7 +158,7 @@ export function useWorker(props: {
 	zone: string | undefined;
 	host: string | undefined;
 	routes: Route[] | undefined;
-	onReady?: () => void;
+	onReady: (() => void) | undefined;
 }): CfPreviewToken | undefined {
 	const {
 		name,