Bugfix: don't mistake any embed for a status embed

When changing the channel status, the bot will update a previous embed
if it's in the bottom 5 messages, rather than sending a new message.
However, we were detecting *any* embed from the bot, rather than only
status messages, so it would also replace playground links, for example.

This detects status embeds by comparing their titles. In order to do
that, it adds a title to the dormant embed. This isn't *necessary*; the
following code would work instead:

const isStatusEmbed = (embed: MessageEmbed) => (
    embed.title === this.AVAILABLE_EMBED.title ||
    embed.title === this.OCCUPIED_EMBED_BASE.title ||
    embed.description === this.DORMANT_EMBED.description
);

However, this way is both cleaner code-wise, and I think it's nice for
all the status embeds to have the same format.

Author: smichel17

src/modules/helpchan.ts

@@ -92,6 +92,7 @@ export class HelpChanModule extends Module {
 	}
 
 	DORMANT_EMBED = new MessageEmbed()
+		.setTitle('💤 Dormant Help Channel')
 		.setColor(TS_BLUE)
 		.setDescription(DORMANT_MESSAGE);
 
@@ -336,16 +337,25 @@ export class HelpChanModule extends Module {
 	}
 
 	private async updateStatusEmbed(channel: TextChannel, embed: MessageEmbed) {
+		const isStatusEmbed = (embed: MessageEmbed) =>
+			[
+				this.AVAILABLE_EMBED.title,
+				this.OCCUPIED_EMBED_BASE.title,
+				this.DORMANT_EMBED.title,
+			].includes(embed.title);
+
 		let lastMessage = channel.messages.cache
 			.array()
 			.reverse()
-			.find(m => m.author.id === this.client.user?.id);
+			.filter(m => m.author.id === this.client.user?.id)
+			.find(m => m.embeds.some(isStatusEmbed));
 
 		if (!lastMessage)
 			lastMessage = (await channel.messages.fetch({ limit: 5 }))
 				.array()
 				.reverse()
-				.find(m => m.author.id === this.client.user?.id);
+				.filter(m => m.author.id === this.client.user?.id)
+				.find(m => m.embeds.some(isStatusEmbed));
 
 		if (lastMessage) {
 			// If there is a last message (the status message) by the bot, edit it