From 6277698f8686164ed292397c75b54709c926c17e Mon Sep 17 00:00:00 2001
From: Sebastian <19554889+sebslight@users.noreply.github.com>
Date: Sun, 15 Feb 2026 11:44:39 -0500
Subject: [PATCH] test(discord): fix updated test harness mocks

---
 src/channels/plugins/actions/discord.test.ts |  1 -
 src/discord/monitor/exec-approvals.test.ts   | 24 ++++++++++++--------
 src/slack/monitor/slash.test-harness.ts      |  9 +++++++-
 3 files changed, 22 insertions(+), 12 deletions(-)

diff --git a/src/channels/plugins/actions/discord.test.ts b/src/channels/plugins/actions/discord.test.ts
index 640df817a1e..63426c89c26 100644
--- a/src/channels/plugins/actions/discord.test.ts
+++ b/src/channels/plugins/actions/discord.test.ts
@@ -69,7 +69,6 @@ describe("handleDiscordMessageAction", () => {
 
   it("forwards legacy embeds for send", async () => {
     sendMessageDiscord.mockClear();
-    const handleDiscordMessageAction = await loadHandleDiscordMessageAction();
 
     const embeds = [{ title: "Legacy", description: "Use components v2." }];
 
diff --git a/src/discord/monitor/exec-approvals.test.ts b/src/discord/monitor/exec-approvals.test.ts
index 896e4a8c48a..9d2685874fd 100644
--- a/src/discord/monitor/exec-approvals.test.ts
+++ b/src/discord/monitor/exec-approvals.test.ts
@@ -29,16 +29,20 @@ const mockRestPost = vi.hoisted(() => vi.fn());
 const mockRestPatch = vi.hoisted(() => vi.fn());
 const mockRestDelete = vi.hoisted(() => vi.fn());
 
-vi.mock("../send.shared.js", () => ({
-  createDiscordClient: () => ({
-    rest: {
-      post: mockRestPost,
-      patch: mockRestPatch,
-      delete: mockRestDelete,
-    },
-    request: (_fn: () => Promise<unknown>, _label: string) => _fn(),
-  }),
-}));
+vi.mock("../send.shared.js", async (importOriginal) => {
+  const actual = await importOriginal<typeof import("../send.shared.js")>();
+  return {
+    ...actual,
+    createDiscordClient: () => ({
+      rest: {
+        post: mockRestPost,
+        patch: mockRestPatch,
+        delete: mockRestDelete,
+      },
+      request: (_fn: () => Promise<unknown>, _label: string) => _fn(),
+    }),
+  };
+});
 
 vi.mock("../../gateway/client.js", () => ({
   GatewayClient: class {
diff --git a/src/slack/monitor/slash.test-harness.ts b/src/slack/monitor/slash.test-harness.ts
index 20f4e65afee..c81f8be25a5 100644
--- a/src/slack/monitor/slash.test-harness.ts
+++ b/src/slack/monitor/slash.test-harness.ts
@@ -28,7 +28,14 @@ vi.mock("../../agents/identity.js", async (importOriginal) => {
   };
 });
 
-export function getSlackSlashMocks() {
+type SlashHarnessMocks = {
+  dispatchMock: ReturnType<typeof vi.fn>;
+  readAllowFromStoreMock: ReturnType<typeof vi.fn>;
+  upsertPairingRequestMock: ReturnType<typeof vi.fn>;
+  resolveAgentRouteMock: ReturnType<typeof vi.fn>;
+};
+
+export function getSlackSlashMocks(): SlashHarnessMocks {
   return mocks;
 }