fix

Author: duruer

scripts/aggregate-ui-changelog.js

@@ -1,10 +1,13 @@
 // scripts/aggregate-ui-changelog.js
-// Node 18+ (or Bun). Requires GITHUB_TOKEN.
+// Node 18+ (or Bun). Requires GITHUB_TOKEN with read access to the UI repos.
 
 import {execSync} from 'node:child_process';
 import {readdirSync, writeFileSync} from 'node:fs';
 
+// Look for bundled UI zips inside the Pano module
 const UI_DIR = 'Pano/src/main/resources/UIFiles';
+
+// Organization / repository mapping
 const OWNER = 'PanoMC';
 const REPOS = {
     'panel-ui': 'panel-ui',
@@ -18,16 +21,23 @@ if (!token) {
     process.exit(1);
 }
 
+/**
+ * Parse zip file name: "<component>-v1.0.0-dev.34.zip" (the "v" is optional)
+ */
 function parseZip(name) {
     const m = name.match(/^([a-z0-9-]+)-((?:v)?\d+\.\d+\.\d+(?:-[a-z0-9.]+)?)\.zip$/i);
     if (!m) return null;
     return {comp: m[1], version: m[2].startsWith('v') ? m[2] : `v${m[2]}`};
 }
 
+/**
+ * Read current UI zip versions from working tree
+ */
 function listCurrentVersions() {
     const entries = readdirSync(UI_DIR, {withFileTypes: true})
         .filter((d) => d.isFile() && d.name.endsWith('.zip'))
         .map((d) => d.name);
+
     const map = {};
     for (const f of entries) {
         const p = parseZip(f);
@@ -36,6 +46,9 @@ function listCurrentVersions() {
     return map;
 }
 
+/**
+ * Get the previous tag (semantic-release last tag)
+ */
 function getPrevTag() {
     try {
         return execSync('git describe --tags --abbrev=0 HEAD^', {encoding: 'utf8'}).trim();
@@ -48,6 +61,9 @@ function getPrevTag() {
     }
 }
 
+/**
+ * Read UI zip versions from the previous tag’s tree without checkout
+ */
 function listPreviousVersionsFromTag(prevTag) {
     if (!prevTag) return {};
     let listing = '';
@@ -65,6 +81,9 @@ function listPreviousVersionsFromTag(prevTag) {
     return map;
 }
 
+/**
+ * Minimal GitHub API helper
+ */
 async function ghJson(path, params = {}) {
     const url = `https://api.github.com${path}`;
     const res = await fetch(url, {
@@ -82,10 +101,18 @@ async function ghJson(path, params = {}) {
     return await res.json();
 }
 
-// Normalize UI repo release body into clean bullet lines
+/**
+ * Normalize a UI repo release body to clean bullet lines:
+ * - drop any heading lines (start with #) and section names (Features, Bug Fixes, etc.)
+ * - drop version/date headings
+ * - drop code blocks
+ * - flatten to "- ..." bullets
+ * - de-duplicate
+ */
 function normalizeReleaseBody(body) {
     const out = [];
     let inCode = false;
+
     for (const raw of body.split('\n')) {
         const line = raw.replace(/\s+$/, ''); // rtrim
         const t = line.trim();
@@ -100,33 +127,28 @@ function normalizeReleaseBody(body) {
         // skip empty
         if (!t) continue;
 
-        // drop version/date headings & section headers
+        // drop any markdown headings
+        if (/^#{1,6}\s*/.test(t)) continue;
+
+        // drop common section headers and separators
         if (
-            /^#{1,6}\s*(v?\d+\.\d+\.\d+(?:-[^\s)]+)?)(\s*\(\d{4}-\d{2}-\d{2}\))?/i.test(t) ||
-            /^(v?\d+\.\d+\.\d+(?:-[^\s)]+)?)(\s*\(\d{4}-\d{2}-\d{2}\))?$/.test(t) ||
-            /^#{1,6}\s*(features|bug fixes|fixes|performance improvements|reverts|chores|chore|ci|build)\s*$/i.test(t) ||
-            /^(features|bug fixes|fixes|performance improvements|reverts|chores|chore|ci|build)\s*$/i.test(t) ||
-            /^-{3,}$/.test(t)
+            /^(features|bug fixes|fix(es)?|performance improvements|reverts|chores?|ci|build|changes)\s*:?$/i.test(t) ||
+            /^[-_]{3,}$/.test(t)
         ) continue;
 
-        // normalize list items / commit-like lines
-        let item = t.replace(/^[-*]\s+/, '');      // strip leading list marker
-        item = item.replace(/^\d+\.\s+/, '');      // strip ordered list
+        // drop version/date headings like "1.0.0 (2025-08-28)" or "v1.2.3"
+        if (/^(v?\d+\.\d+\.\d+(?:-[^\s)]+)?)(\s*\(\d{4}-\d{2}-\d{2}\))?$/i.test(t)) continue;
+
+        // normalize list items / ordered items
+        let item = t.replace(/^[-*]\s+/, '');   // strip leading list marker
+        item = item.replace(/^\d+\.\s+/, '');   // strip ordered list
         item = item.replace(/^\s+/, '');
 
-        // if it looks like a conventional header or a plain message, bullet it
-        if (!/^[-*]\s+/.test(t)) {
-            if (/^(feat|fix|perf|refactor|docs|chore|build|ci)(\(.+\))?:/i.test(item)) {
-                out.push(`- ${item}`);
-            } else {
-                out.push(`- ${item}`);
-            }
-        } else {
-            out.push(`- ${item}`);
-        }
+        // bulletize anything left (including conventional headers)
+        out.push(`- ${item}`);
     }
 
-    // dedupe (case-insensitive)
+    // de-duplicate (case-insensitive)
     const seen = new Set();
     const uniq = [];
     for (const l of out) {
@@ -139,11 +161,16 @@ function normalizeReleaseBody(body) {
     return uniq;
 }
 
+/**
+ * Collect notes between fromTag…toTag for a repo:
+ * 1) Conventional commit headers from compare API
+ * 2) Target tag’s normalized GitHub Release body (if exists)
+ */
 async function collectNotesForRange(repo, fromTag, toTag) {
-    const set = new Set(); // for dedupe
+    const set = new Set();
     const lines = [];
 
-    // 1) Conventional commit headers via compare API
+    // 1) Conventional commits via compare API
     try {
         const cmp = await ghJson(
             `/repos/${OWNER}/${repo}/compare/${encodeURIComponent(fromTag)}...${encodeURIComponent(toTag)}`
@@ -164,7 +191,7 @@ async function collectNotesForRange(repo, fromTag, toTag) {
         // ignore
     }
 
-    // 2) Append normalized release body (toTag), if any
+    // 2) Append normalized release body of toTag
     try {
         const releases = await ghJson(`/repos/${OWNER}/${repo}/releases?per_page=100`);
         for (const r of releases) {
@@ -186,6 +213,9 @@ async function collectNotesForRange(repo, fromTag, toTag) {
     return lines.length ? lines.join('\n') : null;
 }
 
+/**
+ * Main
+ */
 (async () => {
     const current = listCurrentVersions();
     const prevTag = getPrevTag();
@@ -201,9 +231,10 @@ async function collectNotesForRange(repo, fromTag, toTag) {
         const notes = await collectNotesForRange(repo, oldV, nowV);
         if (!notes) continue;
 
+        // Section title WITHOUT "###" — use bold line instead
         sections.push(
             [
-                `### ${comp}: ${oldV} → ${nowV}`,
+                `**${comp}: ${oldV} → ${nowV}**`,
                 '',
                 notes
             ].join('\n')
@@ -216,7 +247,7 @@ async function collectNotesForRange(repo, fromTag, toTag) {
         return;
     }
 
-    // Visible spacing in GitHub release body
+    // Visible spacing in GitHub release body (don’t rely on collapsed newlines)
     const SPACER = '\n<br/>\n<br/>\n';
     const output = `${SPACER}${sections.join(SPACER)}\n`;