feat(changelog): color-code and reorder summary badges

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: davidb-tada

datacontract/output/text_changelog_results.py

@@ -1,30 +1,36 @@
 import io
+from collections import Counter
 
 from rich import box
 from rich.console import Console
 from rich.table import Table
 
-from datacontract.model.changelog import ChangelogResult, ChangelogType
+from datacontract.model.changelog import ChangelogEntry, ChangelogResult, ChangelogType
 
 _VAL_W = 30
 
+_CHANGE_COLOR = {
+    ChangelogType.added: "green",
+    ChangelogType.changed: "yellow",
+    ChangelogType.removed: "red",
+}
+
+_BADGE_ORDER = [ChangelogType.added, ChangelogType.changed, ChangelogType.removed]
+
 
 def write_text_changelog_results(result: ChangelogResult, console: Console):
     _print_summary(result, console)
     _print_table(result, console)
 
 
-def _badges(entries: list) -> str:
-    removed = sum(1 for e in entries if e.type == ChangelogType.removed)
-    changed = sum(1 for e in entries if e.type == ChangelogType.changed)
-    added = sum(1 for e in entries if e.type == ChangelogType.added)
+def _badges(entries: list[ChangelogEntry]) -> str:
+    counts = Counter(e.type for e in entries)
     parts = []
-    if removed:
-        parts.append(f"[ {removed} Removed ]")
-    if changed:
-        parts.append(f"[ {changed} Changed ]")
-    if added:
-        parts.append(f"[ {added} Added ]")
+    for ct in _BADGE_ORDER:
+        n = counts[ct]
+        if n:
+            color = _CHANGE_COLOR[ct]
+            parts.append(f"[ [{color}]{n} {ct.value.capitalize()}[/{color}] ]")
     return "  ".join(parts)
 
 
@@ -66,12 +72,9 @@ def _print_table(result: ChangelogResult, console: Console):
 
 
 def _with_markup(changelog_type: ChangelogType) -> str:
-    if changelog_type == ChangelogType.added:
-        return "[green]added[/green]"
-    if changelog_type == ChangelogType.removed:
-        return "[red]removed[/red]"
-    if changelog_type == ChangelogType.changed:
-        return "[yellow]changed[/yellow]"
+    color = _CHANGE_COLOR.get(changelog_type)
+    if color:
+        return f"[{color}]{changelog_type.value}[/{color}]"
     return changelog_type.value
 
 

tests/fixtures/changelog/golden_changelog_text.txt

@@ -1,5 +1,5 @@
 Summary
-[ 1 Removed ]  [ 4 Changed ]  [ 2 Added ]
+[ 2 Added ]  [ 4 Changed ]  [ 1 Removed ]
 ╭─────────┬───────────────────────────────────────╮
 │ Change  │ Field                                 │
 ├─────────┼───────────────────────────────────────┤

tests/test_changelog_output_text.py

@@ -44,9 +44,21 @@ def test_all_types(self):
         assert "3 Changed" in result
         assert "2 Added" in result
 
-    def test_ordering_removed_changed_added(self):
+    def test_ordering_added_changed_removed(self):
         result = _badges(_make_entries(added=1, removed=1, changed=1))
-        assert result.index("Removed") < result.index("Changed") < result.index("Added")
+        assert result.index("Added") < result.index("Changed") < result.index("Removed")
+
+    def test_added_badge_green(self):
+        result = _badges(_make_entries(added=1))
+        assert "[ [green]1 Added[/green] ]" == result
+
+    def test_changed_badge_yellow(self):
+        result = _badges(_make_entries(changed=1))
+        assert "[ [yellow]1 Changed[/yellow] ]" == result
+
+    def test_removed_badge_red(self):
+        result = _badges(_make_entries(removed=1))
+        assert "[ [red]1 Removed[/red] ]" == result
 
     def test_zero_count_omitted(self):
         result = _badges(_make_entries(added=3))