From c5a876a7d4148be82cef3dbeb43061604b4b68e5 Mon Sep 17 00:00:00 2001
From: Ramon Bartl <rb@ridingbytes.com>
Date: Tue, 9 Jun 2026 21:04:59 +0200
Subject: [PATCH] Skip the Unreleased section in the What's New parser and
 split top-level bullets into separate items so each bullet renders with one
 dot instead of two

Two bugs surfaced together when opening the dialog against
a CHANGELOG.md that had an ## Unreleased section staged
above the latest release:

1. The dialog title read 'What's New Unreleased' because
   parseChangelog took the first version heading it saw.
   Unreleased is a staging slot, not a thing to show users.

2. Every bullet rendered with two dots: the outer <li>
   added one (the cta-coloured dot in the dialog markup),
   and the Markdown component added another because the
   entire release was concatenated into one item string
   that still started with '- ' lines, which markdown
   renders as a nested <ul>.

The parser now:

- Treats any version named 'Unreleased' (case-insensitive)
  as a skip: no entry pushed, lines inside ignored until
  the next ## heading.
- Recognises top-level '- ' lines as bullet starts:
  flushes the previous item, strips the leading dash, and
  begins a fresh item. Indented continuation lines still
  fold into the active item, so wrapped bullet bodies
  stay together.

Verified end-to-end against the current CHANGELOG.md:
top three parsed versions are 2.5.1, 2.5.0, 2.4.1
(Unreleased absent), and no parsed item starts with '- '
(no more double bullets).
---
 desktop/src-tauri/Cargo.lock    |  2 +-
 frontend/src/utils/changelog.ts | 28 ++++++++++++++++++++++++----
 2 files changed, 25 insertions(+), 5 deletions(-)

diff --git a/desktop/src-tauri/Cargo.lock b/desktop/src-tauri/Cargo.lock
index d828ecab..e95817be 100644
--- a/desktop/src-tauri/Cargo.lock
+++ b/desktop/src-tauri/Cargo.lock
@@ -1840,7 +1840,7 @@ dependencies = [
 
 [[package]]
 name = "kaisho-desktop"
-version = "2.5.0"
+version = "2.5.1"
 dependencies = [
  "ab_glyph",
  "embed_plist",
diff --git a/frontend/src/utils/changelog.ts b/frontend/src/utils/changelog.ts
index 0409050a..5249868a 100644
--- a/frontend/src/utils/changelog.ts
+++ b/frontend/src/utils/changelog.ts
@@ -66,10 +66,17 @@ export function parseChangelog(
     const versionMatch = line.match(/^## (.+)/);
     if (versionMatch) {
       flush();
-      current = {
-        version: versionMatch[1].trim(),
-        items: [],
-      };
+      const version = versionMatch[1].trim();
+      // ``Unreleased`` is a staging area in CHANGELOG.md
+      // for entries waiting on the next bump-version run;
+      // it must not surface in the What's New dialog (it
+      // would show up as "What's New Unreleased" and steal
+      // the slot the real latest release should occupy).
+      if (/^unreleased$/i.test(version)) {
+        current = null;
+        continue;
+      }
+      current = { version, items: [] };
       entries.push(current);
       continue;
     }
@@ -82,6 +89,19 @@ export function parseChangelog(
       continue;
     }
 
+    // A top-level ``- `` line starts a new item. The
+    // leading dash is dropped because the dialog renders
+    // its own bullet glyph in the outer ``<li>``; passing
+    // ``- text`` through to the Markdown component would
+    // produce a second nested bullet ("• •"). Indented
+    // continuation lines (``  more text``) are folded
+    // into the current item by the catch-all below.
+    if (/^- /.test(line)) {
+      flush();
+      buffer = line.replace(/^- /, "");
+      continue;
+    }
+
     if (buffer === null) {
       // Hold blank lines until real content appears so
       // the first item doesn't start with a blank line.