From 81d431363113d9a21ec252b016ff41210833e97c Mon Sep 17 00:00:00 2001 From: Zane Helton Date: Sat, 28 Jun 2025 10:55:56 -0400 Subject: [PATCH] Properly filter existing embed links In a previous PR, I was checking if the `.url` property existed, the correct fix is to remove any embed that isn't a rich embed type (non-link). See also: #1296 --- src/api/util/handlers/Message.ts | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/src/api/util/handlers/Message.ts b/src/api/util/handlers/Message.ts index db368d55..30fa5491 100644 --- a/src/api/util/handlers/Message.ts +++ b/src/api/util/handlers/Message.ts @@ -290,14 +290,8 @@ export async function postHandleMessage(message: Message) { } } - // Remove ALL embeds that have URLs when processing links (start fresh) - data.embeds = data.embeds.filter((embed) => { - if (!embed.url) { - return true; - } - - return false; - }); + // Filter out embeds that could be links, start from scratch + data.embeds = data.embeds.filter((embed) => embed.type === "rich"); const seenNormalizedUrls = new Set(); const uniqueLinks: string[] = [];