From 516531677e771324dcd487b2db10c657bd8ac9b6 Mon Sep 17 00:00:00 2001 From: cliffhall Date: Sun, 13 Sep 2026 21:38:21 -0400 Subject: [PATCH] test: drop an unnecessary double cast in the relay progress test helper (#2028) The backendHoldingSend fetch mock is already declared with vi.fn(), so the trailing `as unknown as typeof fetch` was redundant and tripped the repo's no-double-cast rule. Return the typed mock directly; the test project typechecks without it. Co-Authored-By: Claude Opus 4.8 Claude-Session: https://claude.ai/code/session_01PsmUAZ6aLUAFqenjJKSSRp Signed-off-by: cliffhall --- .../web/src/test/core/mcp/remote/remoteClientTransport.test.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/clients/web/src/test/core/mcp/remote/remoteClientTransport.test.ts b/clients/web/src/test/core/mcp/remote/remoteClientTransport.test.ts index 90de46ef..e0807093 100644 --- a/clients/web/src/test/core/mcp/remote/remoteClientTransport.test.ts +++ b/clients/web/src/test/core/mcp/remote/remoteClientTransport.test.ts @@ -723,7 +723,7 @@ describe("RemoteClientTransport", () => { return new Response(JSON.stringify({ ok: true }), { status: 200 }); } return new Response("not found", { status: 404 }); - }) as unknown as typeof fetch; + }); return { fetchFn, getSse: () => sse, getSentId: () => sentId }; }