fix: strip hash from urls before comparing

package.json

@@ -9,9 +9,9 @@
   },
   "dependencies": {
     "@googleapis/drive": "8.4.0",
-    "compare-urls": "4.1.0",
     "consola": "3.2.3",
-    "discord.js": "14.14.1"
+    "discord.js": "14.14.1",
+    "normalize-url": "8.0.0"
   },
   "devDependencies": {
     "@biomejs/biome": "1.4.1",

src/embeds.ts

@@ -1,6 +1,5 @@
 import { drive_v3 } from "@googleapis/drive";
 import { deepMatch, sleep } from "bun";
-import compareUrls from "compare-urls";
 import {
 	EmbedBuilder,
 	Message,
@@ -12,6 +11,9 @@ import {
 	isJSONEncodable,
 } from "discord.js";
 import { GaxiosError } from "gaxios";
+import normalizeUrl, {
+	type Options as NormalizeUrlOptions,
+} from "normalize-url";
 import { driveClient, fileTypes } from "./gdrive";
 import { appendInvisible, decodeAppendedInvisible } from "./util";
 
@@ -146,13 +148,28 @@ const createEmbedsMessage = async (
  * @param fileUrls URLs of files to suppress embeds of
  */
 const suppressEmbeds = async (message: Message, fileUrls: string[]) => {
+	const normalizeOptions: NormalizeUrlOptions = {
+		defaultProtocol: "https",
+		normalizeProtocol: true,
+		forceHttps: true,
+		removeExplicitPort: true,
+		stripHash: true,
+		removeQueryParameters: true,
+	};
 	const embedsUrls = message.embeds.map(({ url }) => url);
 	const shouldSuppress =
 		!!embedsUrls.length &&
 		embedsUrls.every(
 			// return false if null, which means the embed is not a link
-			(embedUrl) =>
-				embedUrl && fileUrls.some((fileUrl) => compareUrls(fileUrl, embedUrl)),
+			(embedUrl) => {
+				if (!embedUrl) {
+					return false;
+				}
+				const _normalizedEmbedUrl = normalizeUrl(embedUrl, normalizeOptions);
+				return fileUrls.some((fileUrl) =>
+					normalizeUrl(fileUrl, normalizeOptions),
+				);
+			},
 		);
 
 	// do not send a request if no change is needed