From 61815416267a75659463791a62f70fd7f8a45358 Mon Sep 17 00:00:00 2001 From: Vincent Koc Date: Wed, 4 Mar 2026 01:11:45 -0500 Subject: [PATCH] test(slack): assert reaction session routing carries sender --- src/slack/monitor/events/reactions.test.ts | 22 ++++++++++++++++++++++ 1 file changed, 22 insertions(+) diff --git a/src/slack/monitor/events/reactions.test.ts b/src/slack/monitor/events/reactions.test.ts index 8105b2047fc..3581d8b5380 100644 --- a/src/slack/monitor/events/reactions.test.ts +++ b/src/slack/monitor/events/reactions.test.ts @@ -153,4 +153,26 @@ describe("registerSlackReactionEvents", () => { expect(trackEvent).toHaveBeenCalledTimes(1); }); + + it("passes sender context when resolving reaction session keys", async () => { + reactionQueueMock.mockClear(); + reactionAllowMock.mockReset().mockResolvedValue([]); + const harness = createSlackSystemEventTestHarness(); + const resolveSessionKey = vi.fn().mockReturnValue("agent:ops:main"); + harness.ctx.resolveSlackSystemEventSessionKey = resolveSessionKey; + registerSlackReactionEvents({ ctx: harness.ctx }); + const handler = harness.getHandler("reaction_added"); + expect(handler).toBeTruthy(); + + await handler!({ + event: buildReactionEvent({ user: "U777", channel: "D123" }), + body: {}, + }); + + expect(resolveSessionKey).toHaveBeenCalledWith({ + channelId: "D123", + channelType: "im", + senderId: "U777", + }); + }); });