Skip to content

Commit 5c4e478

Browse files
fix(slack): expose sender bot status in context (#97822)
* fix(slack): expose sender bot status in context * fix(slack): expose sender bot status in context --------- Co-authored-by: Vincent Koc <[email protected]>
1 parent 37341a7 commit 5c4e478

2 files changed

Lines changed: 21 additions & 0 deletions

File tree

extensions/slack/src/monitor/message-handler/prepare.test.ts

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1060,6 +1060,26 @@ Second paragraph should still reach the agent after Slack's preview cutoff.`;
10601060
expect(members).toHaveBeenCalledTimes(1);
10611061
});
10621062

1063+
it("forwards bot sender status to ctxPayload when allowBots admits the bot", async () => {
1064+
const { slackCtx } = createOwnerScopedBotRoomCtx({ members: ["UOWNER"] });
1065+
1066+
const prepared = await prepareMessageWith(
1067+
slackCtx,
1068+
createSlackAccount({ allowBots: true }),
1069+
createBotRoomMessage(),
1070+
);
1071+
1072+
assertPrepared(prepared);
1073+
expect(prepared.ctxPayload.SenderIsBot).toBe(true);
1074+
});
1075+
1076+
it("omits SenderIsBot for human messages", async () => {
1077+
const prepared = await prepareWithDefaultCtx(createSlackMessage({ text: "hello" }));
1078+
1079+
assertPrepared(prepared);
1080+
expect(prepared.ctxPayload.SenderIsBot).toBeUndefined();
1081+
});
1082+
10631083
it("allows bot-authored room messages when the bot is explicitly channel-allowlisted (#59284)", async () => {
10641084
const members = vi.fn();
10651085
const slackCtx = createInboundSlackCtx({

extensions/slack/src/monitor/message-handler/prepare.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1271,6 +1271,7 @@ export async function prepareSlackMessage(params: {
12711271
id: senderId,
12721272
name: senderName,
12731273
displayLabel: senderName,
1274+
isBot: isBotMessage || undefined,
12741275
},
12751276
conversation: {
12761277
kind: chatType,

0 commit comments

Comments
 (0)