Refactor: Clean up and improve changelog generation script

This commit refactors the `generate-changelog.js` script for improved clarity, functionality, and code style.

Key changes include:

*   **Filter Nightly Tags**: The script now filters out tags containing "nightly" to prevent them from being included in the official changelog.
*   **Code Cleanup**: Simplified comments, removed unnecessary logging (`log(...)` calls), and improved code formatting for better readability.
*   **Variable Renaming**: Renamed `previousTagInSlice` to a more direct name within its scope for clarity.

Author: CreativeCodeCat

generate-changelog.js

@@ -25,7 +25,7 @@ function log(...args) {
 
 // Commit parsing rules
 const commitParsers = [
-	// Skip some "noise" commits
+	// Skip noise
 	{ message: /^chore\(release\): prepare for/i, skip: true },
 	{ message: /^chore\(deps.*\)/i, skip: true },
 	{ message: /^chore\(change.*\)/i, skip: true },
@@ -34,52 +34,58 @@ const commitParsers = [
 	{ message: /^fixes/i, skip: true },
 	{ message: /^build/i, skip: true },
 
-	// Enhancements (new features, improvements, UX, performance)
+	// Enhancements
 	{ message: /^feat|^perf|^style|^ui|^ux/i, group: "### :sparkles: Enhancements:" },
 
-	// Bug fixes & hotfixes
+	// Bug fixes
 	{ message: /^fix|^bug|^hotfix|^emergency/i, group: "### :bug: Bug Fixes:" },
 
-	// Code quality (refactors, cleanup without changing behavior)
+	// Code quality
 	{ message: /^refactor/i, group: "### :wrench: Code Quality:" },
 
 	// Documentation
 	{ message: /^doc/i, group: "### :books: Documentation:" },
 
-	// Localization & internationalization
+	// Localization
 	{ message: /^(lang|i18n)/i, group: "### :globe_with_meridians: Localization:" },
 
 	// Security
 	{ message: /^security/i, group: "### :lock: Security:" },
 
-	// Feature removal / drops
+	// Feature removal
 	{ message: /^drop|^remove|^deprecated/i, group: "### :x: Feature Removals:" },
 
 	// Reverts
 	{ message: /^revert/i, group: "### :rewind: Reverts:" },
 
-	// Build-related
+	// Build
 	{ message: /^build/i, group: "### :building_construction: Build:" },
 
-	// Dependencies-related
+	// Dependencies
 	{ message: /^dependency|^deps/i, group: "### :package: Dependencies:" },
 
-	// Meta: configuration, CI/CD, versioning, releases
-	{ message: /^config|^configuration|^ci|^pipeline|^release|^version|^versioning/i, group: "### :gear: Meta:" },
+	// Meta
+	{
+		message: /^config|^configuration|^ci|^pipeline|^release|^version|^versioning/i,
+		group: "### :gear: Meta:",
+	},
 
 	// Tests
 	{ message: /^test/i, group: "### :test_tube: Tests:" },
 
-	// Infrastructure & Ops
+	// Infrastructure
 	{ message: /^infra|^infrastructure|^ops/i, group: "### :office: Infrastructure & Ops:" },
 
-	// Chore & cleanup
-	{ message: /^chore|^housekeeping|^cleanup|^clean\(up\)/i, group: "### :broom: Maintenance & Cleanup:" },
+	// Maintenance
+	{
+		message: /^chore|^housekeeping|^cleanup|^clean\(up\)/i,
+		group: "### :broom: Maintenance & Cleanup:",
+	},
 ];
 
 const GROUP_ORDER = commitParsers.filter((p) => !p.skip).map((p) => p.group);
 
-// Helper functions
+// Helpers
 function run(cmd) {
 	log("Running:", cmd);
 	return execSync(cmd, { encoding: "utf8" }).trim();
@@ -108,17 +114,20 @@ function cleanMessage(message) {
 }
 
 function linkPR(message) {
-	return message.replace(/\(#(\d+)\)/g, (_, num) => `([#${num}](${REPO_URL}/pull/${num}))`);
+	return message.replace(/\(#(\d+)\)/g, (_, num) => {
+		return `([#${num}](${REPO_URL}/pull/${num}))`;
+	});
 }
 
-// Get GitHub release title for a tag
+// GitHub release title
 function getReleaseTitle(tag) {
 	const options = {
 		hostname: "api.github.com",
 		path: `/repos/CodeWorksCreativeHub/mLauncher/releases/tags/${tag}`,
 		method: "GET",
 		headers: { "User-Agent": "Node.js" },
 	};
+
 	return new Promise((resolve) => {
 		const req = https.request(options, (res) => {
 			let data = "";
@@ -132,22 +141,28 @@ function getReleaseTitle(tag) {
 				}
 			});
 		});
+
 		req.on("error", () => resolve(tag));
 		req.end();
 	});
 }
 
-// Main async function
+// Main
 async function generateChangelog() {
-	const allTags = run("git tag --sort=-creatordate").split("\n");
+	// 🔥 Ignore nightly tags here
+	const allTags = run("git tag --sort=-creatordate")
+		.split("\n")
+		.filter((tag) => !/nightly/i.test(tag));
+
 	const tags = allTags.slice(0, TAGS_TO_INCLUDE);
 	log("Tags to include:", tags);
 
 	let changelog = HEADER;
 
-	// Coming Soon / Unreleased
+	// Unreleased
 	const latestTag = tags[0] || "unreleased";
 	const rawUnreleased = run(`git log ${latestTag}..HEAD --pretty=format:"%h|%s"`).split("\n");
+
 	const unreleasedCommits = rawUnreleased
 		.map((line) => {
 			const [hash, ...msgParts] = line.split("|");
@@ -159,38 +174,37 @@ async function generateChangelog() {
 		.filter(Boolean);
 
 	if (unreleasedCommits.length > 0) {
-		log("Unreleased commits found:", unreleasedCommits.length);
-		changelog += `## [${latestTag} → Unreleased](https://github.com/CodeWorksCreativeHub/mLauncher/tree/main) - In Development\n\n`;
+		changelog += `## [${latestTag} → Unreleased](${REPO_URL}/tree/main) - In Development\n\n`;
+
 		const groups = {};
 		for (const c of unreleasedCommits) {
 			groups[c.group] = groups[c.group] || [];
 			groups[c.group].push(`* ${linkPR(cleanMessage(c.message))} ([${c.hash}](${REPO_URL}/commit/${c.hash}))`);
 		}
+
 		for (const group of GROUP_ORDER) {
 			if (groups[group]) {
-				log(`Unreleased group: ${group}, commits: ${groups[group].length}`);
 				changelog += `${group}\n\n${groups[group].join("\n")}\n\n`;
 			}
 		}
 	}
 
-	// Generate changelog for each tag
+	// Tagged releases
 	for (let i = 0; i < tags.length; i++) {
 		const currentTag = tags[i];
-		const releaseTitle = await getReleaseTitle(currentTag); // GitHub release title
-		log(`Processing tag: ${currentTag} (${releaseTitle})`);
+		const releaseTitle = await getReleaseTitle(currentTag);
 
 		let range;
 		if (i === tags.length - 1) {
 			const oldestTagIndex = allTags.indexOf(currentTag);
 			const parentTag = allTags[oldestTagIndex - 1];
 			range = parentTag ? `${parentTag}..${currentTag}` : currentTag;
 		} else {
-			const previousTagInSlice = tags[i + 1];
-			range = `${previousTagInSlice}..${currentTag}`;
+			range = `${tags[i + 1]}..${currentTag}`;
 		}
 
 		const rawCommits = run(`git log ${range} --pretty=format:"%h|%s"`).split("\n");
+
 		const commits = rawCommits
 			.map((line) => {
 				const [hash, ...msgParts] = line.split("|");
@@ -226,5 +240,4 @@ async function generateChangelog() {
 	console.log(`✅ Generated ${OUTPUT_FILE}`);
 }
 
-// Run
 generateChangelog();