don't ban users that are already banned

2 files changed, 5 insertions, 2 deletions

diff --git a/src/lib/extensions/discord.js/BushGuild.ts b/src/lib/extensions/discord.js/BushGuild.ts
index 0011ce6..33ee3fc 100644
--- a/src/lib/extensions/discord.js/BushGuild.ts
+++ b/src/lib/extensions/discord.js/BushGuild.ts
@@ -163,6 +163,8 @@ export class BushGuild extends Guild {
 		const user = (await util.resolveNonCachedUser(options.user))!;
 		const moderator = client.users.resolve(options.moderator ?? client.user!)!;
 
+		if ((await this.bans.fetch()).has(user.id)) return banResponse.ALREADY_BANNED;
+
 		const ret = await (async () => {
 			// dm user
 			dmSuccessEvent = await Moderation.punishDM({
diff --git a/src/lib/extensions/discord.js/BushGuildMember.ts b/src/lib/extensions/discord.js/BushGuildMember.ts
index 6a95ebf..54fb3f0 100644
--- a/src/lib/extensions/discord.js/BushGuildMember.ts
+++ b/src/lib/extensions/discord.js/BushGuildMember.ts
@@ -465,7 +465,7 @@ export class BushGuildMember extends GuildMember {
 	 * @returns A status message for banning the user.
 	 * @emits {@link BushClientEvents.bushBan}
 	 */
-	public async bushBan(options: BushBanOptions): Promise<BanResponse> {
+	public async bushBan(options: BushBanOptions): Promise<Exclude<BanResponse, typeof banResponse['ALREADY_BANNED']>> {
 		// checks
 		if (!this.guild.me!.permissions.has('BAN_MEMBERS') || !this.bannable) return banResponse.MISSING_PERMISSIONS;
 
@@ -1025,7 +1025,8 @@ export const kickResponse = Object.freeze({
 export const banResponse = Object.freeze({
 	...dmResponse,
 	...permissionsResponse,
-	...punishmentEntryAdd
+	...punishmentEntryAdd,
+	ALREADY_BANNED: 'already banned'
 } as const);
 
 export const blockResponse = Object.freeze({