fix(miner-ui): type a chat-dispatch mock against the union #9815 closed (#9846)

main is red on ui:typecheck, which blocks every open PR.

#9659/#9815 correctly tightened ChatActionDispatchResult from a loose
`{ ok: boolean; status: string; ... }` into a union discriminated on `ok`, so the
miner MCP server's refusal -> error-code mapping could finally be checked. But
this miner-UI test mocks the dispatcher with a bare object literal, whose
`ok`/`status` widen to boolean/string on inference -- and a widened literal no
longer satisfies the closed union.

Annotate the mock's return type (and import the real type) so it is validated
against the shipped contract instead of inference. That also stops the mock
drifting from the union the next time it changes -- the widened version would
have kept compiling against any shape at all.

Author: JSONbored

apps/loopover-miner-ui/src/chat-conversation.test.tsx

@@ -1,6 +1,7 @@
 import { fireEvent, render, screen, waitFor } from "@testing-library/react";
 import { describe, expect, it, vi } from "vitest";
 
+import type { ChatActionDispatchResult } from "../../../packages/loopover-miner/lib/chat-action-dispatch.js";
 import { createChatActionRegistry } from "../../../packages/loopover-miner/lib/chat-action-registry.js";
 import { ChatConversation } from "./components/chat/conversation";
 import { GOVERNOR_CHAT_ACTION_PENDING_MESSAGE } from "./lib/chat-governor-action-copy";
@@ -288,7 +289,11 @@ describe("ChatConversation governor pause/resume chat actions (#8670)", () => {
   });
 
   it("surfaces a non-executed dispatch (flag off / gated) as a system note instead of an empty turn", async () => {
-    const runGovernorChatActionImpl = vi.fn(async () => ({ ok: false, status: "disabled", action: null }));
+    const runGovernorChatActionImpl = vi.fn(async (): Promise<ChatActionDispatchResult> => ({
+      ok: false,
+      status: "disabled",
+      action: null,
+    }));
     render(
       <ChatConversation
         streamChatImpl={async function* () {