Fix: make sure that remote proxy sessions's logs can be silenced when the wrangler log level is set to "none" (#12498)

Author: dario-piotrowicz

.changeset/every-dolls-type.md

@@ -0,0 +1,5 @@
+---
+"wrangler": patch
+---
+
+Fix: make sure that remote proxy sessions's logs can be silenced when the wrangler log level is set to "none"

packages/wrangler/src/api/remoteBindings/start-remote-proxy-session.ts

@@ -5,6 +5,7 @@ import remoteBindingsWorkerPath from "worker:remoteBindings/ProxyServerWorker";
 import { logger } from "../../logger";
 import { getBasePath } from "../../paths";
 import { startWorker } from "../startDevWorker";
+import type { LoggerLevel } from "../../logger";
 import type { StartDevWorkerInput, Worker } from "../startDevWorker";
 import type { Config } from "@cloudflare/workers-utils";
 import type { RemoteProxyConnectionString } from "miniflare";
@@ -45,11 +46,7 @@ export async function startRemoteProxySession(
 				port: await getPort(),
 			},
 			inspector: false,
-			logLevel:
-				// If the logger has a logLevel of "debug" it means that the user is likely trying to debug some issue,
-				// so we should respect that here as well for the remote proxy session. In any other case, to avoid noisy
-				// logs, we just simply fall back to "error"
-				logger.loggerLevel === "debug" ? "debug" : "error",
+			logLevel: getStartWorkerLogLevel(logger.loggerLevel),
 		},
 		bindings: rawBindings,
 	}).catch((startWorkerError) => {
@@ -114,3 +111,27 @@ export type RemoteProxySession = Pick<Worker, "ready" | "dispose"> & {
 	updateBindings: (bindings: StartDevWorkerInput["bindings"]) => Promise<void>;
 	remoteProxyConnectionString: RemoteProxyConnectionString;
 };
+
+/**
+ * Gets the log level to use for the remote worker.
+ *
+ * @param wranglerLogLevel The log level set for the Wrangler process.
+ * @returns The log level to use for the remove worker.
+ */
+function getStartWorkerLogLevel(wranglerLogLevel: LoggerLevel): LoggerLevel {
+	switch (wranglerLogLevel) {
+		case "debug":
+			// If the `logLevel` is "debug" it means that the user is likely trying to debug some issue,
+			// so we should respect that here as well for the remote proxy session.
+			return "debug";
+
+		case "none":
+			// If the `logLevel` is "none" it means that the user is trying to silence all output,
+			// so we should respect that here as well for the remote proxy session.
+			return "none";
+
+		default:
+			// In any other case we want to default to "error" to avoid noisy logs
+			return "error";
+	}
+}