From f928d7594bd9fde954138a3d6399ca9a1357cf7d Mon Sep 17 00:00:00 2001
From: Alexander Lanin <Alexander.Lanin@etas.com>
Date: Thu, 4 Jun 2026 14:41:50 +0200
Subject: [PATCH] fix: sort overview table numerically for PRs, issues, and
 merged columns
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Decorated cells (fire badge, em-dash for zero, red badge for >5 PRs)
had no data-sort-value, so the JS sort fell back to textContent and
localeCompare. parseFloat("🔥 12") and parseFloat("—") both return NaN,
causing repos with fire badges or zero counts to sort incorrectly.

Add data-sort-value with the raw integer on the three affected <td>
elements so the JS numeric sort path is always taken.
---
 src/generate_repo_overview/_html_index.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/generate_repo_overview/_html_index.py b/src/generate_repo_overview/_html_index.py
index 8e2e24f..2a7fe17 100644
--- a/src/generate_repo_overview/_html_index.py
+++ b/src/generate_repo_overview/_html_index.py
@@ -196,9 +196,9 @@ def _overview_row(entry: RepoEntry, org_name: str) -> str:
         f'    <tr data-name="{e(entry.name)}" data-merged="{entry.volatile.merged_prs_30_days}"'
         f' data-issues="{entry.volatile.open_issues}" data-stars="{entry.stars}">\n'
         f"      <td>{name_cell}</td>\n"
-        f'      <td class="text-right" data-tooltip="{e(merged_tip)}">{merged}</td>\n'
-        f'      <td class="text-right" data-tooltip="{e(issues_tip)}">{issues_cell}</td>\n'
-        f'      <td class="text-right" data-tooltip="{e(prs_tip)}">{prs_cell}</td>\n'
+        f'      <td class="text-right" data-sort-value="{entry.volatile.merged_prs_30_days}" data-tooltip="{e(merged_tip)}">{merged}</td>\n'
+        f'      <td class="text-right" data-sort-value="{entry.volatile.open_issues}" data-tooltip="{e(issues_tip)}">{issues_cell}</td>\n'
+        f'      <td class="text-right" data-sort-value="{total_prs}" data-tooltip="{e(prs_tip)}">{prs_cell}</td>\n'
         f'      <td data-tooltip="{e(release_tip)}">{release}</td>\n'
         f'      <td class="text-right" data-tooltip="{e(stars_tip)}">{stars_forks}</td>\n'
         f"    </tr>"