fix: improve message permission checks

src/classes/Detector.ts

@@ -1,6 +1,7 @@
 import { setTimeout } from 'node:timers/promises';
 import type { Application } from '#classes';
 import { env } from '#env';
+import { isSendable } from '#util';
 import {
   type APIEmbedField,
   type GuildMember,
@@ -64,8 +65,8 @@ export class Detector {
   }
 
   async reply(message: Message<true>, edited: boolean) {
-    const permissions = message.channel.permissionsFor(this.client.user);
-    if (edited || (permissions && !permissions.has(PermissionFlagsBits.SendMessages))) {
+    const sendable = await isSendable(message.channel);
+    if (edited || !sendable) {
       return;
     }
 
@@ -226,7 +227,7 @@ export class Detector {
       }
     }
 
-    if (channel?.permissionsFor(this.client.user)?.has(PermissionFlagsBits.SendMessages) && channel.viewable) {
+    if (channel && (await isSendable(channel))) {
       await channel.send({
         embeds: [
           {

src/events/messageCreate.ts

@@ -1,26 +1,25 @@
 import type { Application } from '#classes';
-import { type Message, MessageType, PermissionFlagsBits } from 'discord.js';
+import { isSendable } from '#util';
+import { type Message, MessageType } from 'discord.js';
 
 export async function run(client: Application<true>, message: Message<true>) {
-  const permissions = message.channel.permissionsFor(client.user);
   if (
     message.author.bot ||
     message.author.system ||
     (message.type !== MessageType.Default && message.type !== MessageType.Reply) ||
-    !message.content ||
-    (permissions && !permissions.has(PermissionFlagsBits.SendMessages))
+    !message.content
   ) {
     return;
   }
 
-  if (message.content === `<@${client.user.id}>`) {
+  if (message.content === `<@${client.user.id}>` && (await isSendable(message.channel))) {
     return message.channel.send('Hi! Type `/` to see my commands');
   }
 
   const [commandName] = message.content.slice(2).split(' ');
   const command = client.chatInputCommands.get(commandName);
 
-  if (message.content.startsWith('h!') && command && !command.dev) {
+  if (message.content.startsWith('h!') && command && !command.dev && (await isSendable(message.channel))) {
     const commands = client.application.commands.cache.size
       ? client.application.commands.cache
       : await client.application.commands.fetch();

src/events/messageUpdate.ts

@@ -1,5 +1,5 @@
 import type { Application } from '#classes';
-import { type Message, MessageType, PermissionFlagsBits } from 'discord.js';
+import { type Message, MessageType } from 'discord.js';
 
 export async function run(client: Application<true>, oldMessage: Message, message: Message<true>) {
   if (message.partial) {
@@ -10,13 +10,11 @@ export async function run(client: Application<true>, oldMessage: Message, messag
     }
   }
 
-  const permissions = message.channel.permissionsFor(client.user);
   if (
     message.author.bot ||
     message.author.system ||
     (message.type !== MessageType.Default && message.type !== MessageType.Reply) ||
-    !message.content ||
-    (permissions && !permissions.has(PermissionFlagsBits.SendMessages))
+    !message.content
   ) {
     return;
   }

src/util/index.ts

@@ -1,4 +1,5 @@
 export { default as changelog } from './changelog.json' assert { type: 'json' };
 export * from './fetchTags.js';
 export * from './files.js';
+export * from './sendable.js';
 export * from './types.js';

src/util/sendable.ts

@@ -0,0 +1,20 @@
+import { type GuildTextBasedChannel, PermissionFlagsBits } from 'discord.js';
+
+export async function isSendable(channel: GuildTextBasedChannel) {
+  if (channel.isThread()) {
+    return channel.sendable;
+  }
+
+  const permissions = channel.permissionsFor(channel.client.user);
+
+  if (!permissions || !channel.viewable) {
+    return false;
+  }
+
+  if (permissions.has(PermissionFlagsBits.Administrator, false)) {
+    return true;
+  }
+
+  const clientMember = await channel.guild.members.fetchMe();
+  return permissions.has(PermissionFlagsBits.SendMessages, false) && !clientMember.isCommunicationDisabled();
+}