fix: ensure console methods are bound to instance (#6633)

* fix: ensure console methods are bound to instance

* chore: add changeset

* chore: better description

---------

Co-authored-by: Andy Jessop <[email protected]>

Author: andyjessop

.changeset/real-cooks-argue.md

@@ -0,0 +1,5 @@
+---
+"@cloudflare/vitest-pool-workers": patch
+---
+
+Ensures console methods are bound to their instance in Vitest Pool Worker tests

packages/vitest-pool-workers/src/worker/lib/node/console.ts

@@ -33,6 +33,17 @@ export class Console {
 		const colors =
 			typeof opts.colorMode === "string" ? false : opts.colorMode ?? false;
 		this.#inspectOptions = opts.inspectOptions ?? { colors };
+
+		// Ensure methods are bound to the instance
+		return new Proxy(this, {
+			get(target, prop) {
+				const value = target[prop as keyof Console];
+				if (typeof value === "function") {
+					return value.bind(target);
+				}
+				return value;
+			},
+		});
 	}
 
 	// Vitest expects this function to be called `value`:

packages/vitest-pool-workers/test/console.test.ts

@@ -53,6 +53,27 @@ test.skipIf(process.platform === "win32")(
 	}
 );
 
+test("handles detatched console methods", async ({
+	expect,
+	seed,
+	vitestDev,
+}) => {
+	await seed({
+		"vitest.config.mts": minimalVitestConfig,
+		"index.test.ts": dedent`
+			import { SELF } from "cloudflare:test";
+			import { expect, it } from "vitest";
+			it("does not crash when using a detached console method", async () => {
+				const fn = console["debug"];
+				fn("Does not crash");
+				expect(true).toBe(true);
+			});
+	`,
+	});
+	const result = vitestDev();
+	expect(result.stderr).toMatch("");
+});
+
 test("console.logs() inside `export default`ed handlers with SELF", async ({
 	expect,
 	seed,