Skip to content

Commit f9ba2bd

Browse files
authored
Merge pull request #598 from odilitime/patch-4
fix: discord crash on sending message to null channel
2 parents 0355ab6 + ebc5fdd commit f9ba2bd

File tree

1 file changed

+10
-2
lines changed

1 file changed

+10
-2
lines changed

packages/client-discord/src/messages.ts

+10-2
Original file line numberDiff line numberDiff line change
@@ -261,6 +261,13 @@ function splitMessage(content: string): string[] {
261261
}
262262

263263
function canSendMessage(channel) {
264+
// validate input
265+
if (!channel) {
266+
return {
267+
canSend: false,
268+
reason: "No channel given",
269+
}
270+
}
264271
// if it is a DM channel, we can always send messages
265272
if (channel.type === ChannelType.DM) {
266273
return {
@@ -438,10 +445,11 @@ export class MessageManager {
438445
this.client.user?.displayName,
439446
});
440447

441-
if (!canSendMessage(message.channel).canSend) {
448+
const canSendResult = canSendMessage(message.channel)
449+
if (!canSendResult.canSend) {
442450
return elizaLogger.warn(
443451
`Cannot send message to channel ${message.channel}`,
444-
canSendMessage(message.channel)
452+
canSendResult
445453
);
446454
}
447455

0 commit comments

Comments
 (0)