make sure that remote binding errors are surfaced when using mixed (hybrid) mode (#9515)

Author: dario-piotrowicz

.changeset/pink-ties-relate.md

@@ -0,0 +1,5 @@
+---
+"wrangler": patch
+---
+
+make sure that remote binding errors are surfaced when using mixed (hybrid) mode

packages/wrangler/e2e/dev-mixed-mode.test.ts

@@ -3,7 +3,7 @@ import { resolve } from "node:path";
 import { setTimeout } from "node:timers/promises";
 import getPort from "get-port";
 import dedent from "ts-dedent";
-import { afterAll, beforeAll, describe, expect, it } from "vitest";
+import { afterAll, beforeAll, describe, expect, it, vi } from "vitest";
 import { WranglerE2ETestHelper } from "./helpers/e2e-wrangler-test";
 import { fetchText } from "./helpers/fetch-text";
 import { generateResourceName } from "./helpers/generate-resource-name";
@@ -194,6 +194,66 @@ describe("wrangler dev - mixed mode", () => {
 		`);
 	});
 
+	describe("shows helpful error logs", () => {
+		it("when a remote service binding is not properly configured", async () => {
+			await helper.seed({
+				"wrangler.json": JSON.stringify({
+					name: "mixed-mode-mixed-bindings-test",
+					main: "simple-service-binding.js",
+					compatibility_date: "2025-05-07",
+					services: [
+						{
+							binding: "REMOTE_WORKER",
+							service: "non-existent-service-binding",
+							remote: true,
+						},
+					],
+				}),
+			});
+
+			const worker = helper.runLongLived("wrangler dev --x-mixed-mode");
+
+			await worker.waitForReady();
+
+			await vi.waitFor(
+				() =>
+					expect(worker.currentOutput).toContain(
+						"Could not resolve service binding 'REMOTE_WORKER'. Target script 'non-existent-service-binding' not found."
+					),
+				5_000
+			);
+		});
+
+		it("when a remote KV binding is not properly configured", async () => {
+			await helper.seed({
+				"wrangler.json": JSON.stringify({
+					name: "mixed-mode-mixed-bindings-test",
+					main: "kv.js",
+					compatibility_date: "2025-05-07",
+					kv_namespaces: [
+						{
+							binding: "KV_BINDING",
+							id: "non-existent-kv",
+							remote: true,
+						},
+					],
+				}),
+			});
+
+			const worker = helper.runLongLived("wrangler dev --x-mixed-mode");
+
+			await worker.waitForReady();
+
+			await vi.waitFor(
+				() =>
+					expect(worker.currentOutput).toContain(
+						"KV namespace 'non-existent-kv' is not valid."
+					),
+				5_000
+			);
+		});
+	});
+
 	describe("multi-worker", () => {
 		it("handles both remote and local service bindings at the same time in all workers", async () => {
 			await helper.seed({

packages/wrangler/src/api/mixedMode/index.ts

@@ -54,7 +54,7 @@ export async function startMixedModeSession(
 			inspector: {
 				port: await getPort(),
 			},
-			logLevel: "none",
+			logLevel: "error",
 		},
 		bindings: rawBindings,
 	});

packages/wrangler/src/logger.ts

@@ -68,6 +68,18 @@ overrideLoggerLevel.getStore;
 
 export type TableRow<Keys extends string> = Record<Keys, string>;
 
+function consoleMethodToLoggerLevel(
+	method: Exclude<keyof Console, "Console">
+): LoggerLevel {
+	if (method in LOGGER_LEVELS) {
+		return method as LoggerLevel;
+	}
+	// categorize anything different from the common console methods as
+	// a standard log (in the future if need be we can add logic here to
+	// associate different methods to different log levels)
+	return "log";
+}
+
 export class Logger {
 	constructor() {}
 
@@ -127,7 +139,11 @@ export class Logger {
 		if (typeof console[method] !== "function") {
 			throw new Error(`console.${method}() is not a function`);
 		}
-		if (LOGGER_LEVELS[this.loggerLevel] !== LOGGER_LEVELS.none) {
+
+		if (
+			LOGGER_LEVELS[this.loggerLevel] >=
+			LOGGER_LEVELS[consoleMethodToLoggerLevel(method)]
+		) {
 			Logger.#beforeLogHook?.();
 			(console[method] as (...args: unknown[]) => unknown).apply(console, args);
 			Logger.#afterLogHook?.();