Skip to content

Commit 4fc5068

Browse files
committed
fix(discord): validate 2-100 message count before bulk delete
Discord's bulk-delete endpoint requires 2-100 message IDs; forwarding an out-of-range count produced an opaque Discord API error instead of a clear preflight message.
1 parent a6712fb commit 4fc5068

1 file changed

Lines changed: 8 additions & 3 deletions

File tree

apps/sim/tools/discord/bulk_delete_messages.ts

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -51,9 +51,14 @@ export const discordBulkDeleteMessagesTool: ToolConfig<
5151
Authorization: `Bot ${params.botToken.trim()}`,
5252
}),
5353
body: (params: DiscordBulkDeleteMessagesParams) => {
54-
const messages = (
55-
Array.isArray(params.messageIds) ? params.messageIds : [params.messageIds]
56-
).map((id) => String(id).trim())
54+
const messages = (Array.isArray(params.messageIds) ? params.messageIds : [params.messageIds])
55+
.map((id) => String(id).trim())
56+
.filter(Boolean)
57+
if (messages.length < 2 || messages.length > 100) {
58+
throw new Error(
59+
`Discord requires 2-100 message IDs for bulk delete, got ${messages.length}`
60+
)
61+
}
5762
return { messages }
5863
},
5964
},

0 commit comments

Comments
 (0)