fix(markdown): pas de prise en compte des parties dans un message avec de l'aléatoire qui ne contiennent que des commentaires HTML

Author: eyssette

app/js/markdown/custom/random.mjs

@@ -3,10 +3,10 @@ import { getRandomElement } from "../../utils/arrays.mjs";
 // Gestion du cas où il y a plusieurs messages possibles de réponse, séparés par "---"
 export function processRandomMessage(message) {
 	const SEPARATOR = "\n---\n";
-	// Découpe en variantes et ignore les parties vides (espaces / retours à la ligne)
+	// Découpe en variantes et ignore les parties vides (espaces / retours à la ligne) ou seulement avec des commentaires
 	const messageVariants = message
 		.split(SEPARATOR)
-		.map((part) => part.trim())
+		.map((part) => part.replace(/<!--[\s\S]*?-->/g, "").trim())
 		.filter((part) => part.length > 0);
 
 	if (messageVariants.length > 1) {
@@ -17,7 +17,7 @@ export function processRandomMessage(message) {
 			// On découpe la partie 'baseMessage' en variantes et on ignore les parties vides
 			const baseMessageVariants = baseMessage
 				.split("---")
-				.map((part) => part.trim())
+				.map((part) => part.replace(/<!--[\s\S]*?-->/g, "").trim())
 				.filter((part) => part.length > 0);
 			if (baseMessageVariants.length > 0) {
 				message = getRandomElement(baseMessageVariants) + choiceOptionsHtml;

app/script.min.js

@@ -48,6 +48,36 @@ ${choiceOptionsHTML}`;
 		expect(output).toContain(`${choiceOptionsHTML}`);
 	});
 
+	it("ignores parts that contain only multi-line HTML comments", () => {
+		const variants = ["Real message 1", "Real message 2"];
+		const input = `
+<!--
+  This is a multi-line comment
+  that should be ignored
+-->
+---
+${variants[0]}
+---
+<!-- Another comment -->
+<!-- Another comment -->
+
+<!-- Another comment -->
+---
+${variants[1]}
+
+${choiceOptionsHTML}
+`;
+
+		const output = processRandomMessage(input).trim();
+		const bareOutputWithoutChoiceOptions = output
+			.replace(choiceOptionsHTML, "")
+			.trim();
+
+		// Vérifie que la sortie ne contient pas de commentaire
+		expect(bareOutputWithoutChoiceOptions.startsWith("<!--")).toBeFalse();
+		expect(bareOutputWithoutChoiceOptions.endsWith("-->")).toBeFalse();
+	});
+
 	it("ignores empty or whitespace-only parts between separators or before choiceOptions", () => {
 		const variants = [`Real message 1`, `Real message 2`];
 		const input = `