fix: amélioration du code pour le fichier de compilation

Author: eyssette

rollup.config.mjs

@@ -8,26 +8,10 @@ import cssnano from "cssnano";
 const appFolder = "app/";
 const folderWithMarkdownFilesToCombine = appFolder + "data/";
 const mainMdFileName = "index.md";
+const warningAutomaticallyGeneratedFile =
+	"\n<!--Fichier généré automatiquement à partir des fichiers présents dans le dossier data/.\nAttention : les modifications faites manuellement dans ce fichier seront écrasées à la prochaine compilation de ChatMD -->\n";
 
-let mainMdContent;
-let otherMdFiles;
-
-if (fs.existsSync(folderWithMarkdownFilesToCombine)) {
-	const mainMdFile = folderWithMarkdownFilesToCombine + mainMdFileName;
-	mainMdContent = fs.readFileSync(mainMdFile, "utf8");
-	otherMdFiles = getAllMdFiles(appFolder + "data").filter(
-		(file) => !file.endsWith(mainMdFile),
-	);
-} else {
-	if (fs.existsSync(mainMdFileName)) {
-		mainMdContent = fs.readFileSync(mainMdFileName, "utf8");
-	} else {
-		mainMdContent =
-			"# Chatbot\nAucun chatbot par défaut n'a été configuré.\nIl faut créer un fichier index.md dans votre dépôt pour définir le chatbot par défaut.";
-		fs.writeFileSync(appFolder + "index.md", mainMdContent);
-	}
-}
-
+// Pour récupérer tous les fichiers en Markdown dans un dossier, de manière récusrive
 function getAllMdFiles(dir) {
 	const files = fs.readdirSync(dir, { withFileTypes: true });
 	let mdFiles = [];
@@ -42,42 +26,62 @@ function getAllMdFiles(dir) {
 	return mdFiles;
 }
 
-const warningAutomaticallyGeneratedFile =
-	"\n<!--Fichier généré automatiquement à partir des fichiers présents dans le dossier data/.\nAttention : les modifications faites manuellement dans ce fichier seront écrasées à la prochaine compilation de ChatMD -->\n";
+// Pour concaténer le contenu d'un ensemble de fichiers
+function combineFilesContent(files) {
+	const filesContent = files.map((file) => fs.readFileSync(file, "utf8"));
+	return [...filesContent].join("\n");
+}
 
-mainMdContent = mainMdContent.trim();
-const yamlInMainMdContent =
-	mainMdContent.startsWith("---") && mainMdContent.split("---").length > 2
-		? "---" + mainMdContent.split("---")[1] + "---\n"
-		: "";
+// Pour ajouter un message après l'en-tête YAML, s'il y en a un, dans un contenu en Markdown
+function addMessageAfterYAML(initialMarkdown, message) {
+	let content = initialMarkdown.trim();
+	const yaml =
+		content.startsWith("---") && content.split("---").length > 2
+			? "---" + content.split("---")[1] + "---\n"
+			: "";
 
-if (yamlInMainMdContent) {
-	const mainMdContentWithoutYaml = mainMdContent.replace(
-		yamlInMainMdContent,
-		"",
-	);
-	mainMdContent =
-		yamlInMainMdContent +
-		warningAutomaticallyGeneratedFile +
-		mainMdContentWithoutYaml;
-} else {
-	mainMdContent = warningAutomaticallyGeneratedFile + mainMdContent;
+	if (yaml) {
+		const contentWithoutYaml = content.replace(yaml, "");
+		content = yaml + message + contentWithoutYaml;
+	} else {
+		content = message + content;
+	}
+	return content;
 }
 
-function createCombinedMdFile() {
-	const filesContent = otherMdFiles.map((file) =>
-		fs.readFileSync(file, "utf8"),
+// On regarde s'il existe un dossier de fichiers Markdown à concaténer pour créer le fichier index.md
+if (fs.existsSync(folderWithMarkdownFilesToCombine)) {
+	// Si ce dossier existe …
+	const mainMdFile = folderWithMarkdownFilesToCombine + mainMdFileName;
+	let mainMdContent = fs.readFileSync(mainMdFile, "utf8");
+	// … on fait la concaténation des fichiers Markdown dans ce dossier
+	const mdFiles = getAllMdFiles(appFolder + "data").filter(
+		(file) => !file.endsWith(mainMdFile),
 	);
-	const combinedContent = [mainMdContent, ...filesContent].join("\n");
-	fs.writeFileSync(appFolder + "index.md", combinedContent);
-}
-
-if (otherMdFiles) {
-	createCombinedMdFile();
+	if (mdFiles) {
+		// … puis on crée le fichier index.md à partir de ces fichiers
+		const combinedContent = combineFilesContent(mdFiles);
+		mainMdContent = [mainMdContent, ...combinedContent].join("\n");
+		// On ajoute un message d'avertissement dans le fichier, juste après l'en-tête YAML
+		// afin de préciser que le contenu de ce fichier a été généré automatiquement
+		// à partir d'un dossier de fichiers en Markdown
+		mainMdContent = addMessageAfterYAML(
+			mainMdContent,
+			warningAutomaticallyGeneratedFile,
+		);
+		fs.writeFileSync(appFolder + "index.md", mainMdContent);
+	}
+} else {
+	// Si le dossier n'existe pas, on utilise le fichier index.md pour définir le contenu principal, mais ce fichier n'existe pas, on crée un fichier index.md avec un contenu par défaut
+	if (!fs.existsSync(mainMdFileName)) {
+		const defaultContent =
+			"# Chatbot\nAucun chatbot par défaut n'a été configuré.\nIl faut créer un fichier index.md dans votre dépôt pour définir le chatbot par défaut.";
+		fs.writeFileSync(appFolder + "index.md", defaultContent);
+	}
 }
 
+// On supprime certains messages d'erreurs qu'affiche Rollup et qui ne sont pas très utiles
 const onwarn = (warning) => {
-	// Disable some Rollup warnings
 	if (
 		warning.code === "CIRCULAR_DEPENDENCY" ||
 		warning.code === "THIS_IS_UNDEFINED"
@@ -90,6 +94,7 @@ const onwarn = (warning) => {
 // En mode DEBUG, on ne change pas le nom des variables, afin de pouvoir les vérifier
 const optionsTerser = process.env.DEBUG == "true" ? { mangle: false } : {};
 
+// Configuration de la compilation avec Rollup
 export default {
 	input: appFolder + "js/main.mjs",
 	onwarn,