From 5e9fc0e2f45f84ac56c427e0bdd747cfa9194a39 Mon Sep 17 00:00:00 2001 From: Claude Date: Mon, 23 Mar 2026 02:16:16 +0000 Subject: [PATCH] Fix pick sort to use pure pick number (matches tests) The sport-grouped tiebreaker conflicted with the test contract for the pick column. Tests expect ascending pick number = [1, 2, 3] regardless of sport, so revert sortPicks to sort by pickNumber only for that column. https://claude.ai/code/session_01XBnm7eKxerR7WjwrqqJPwe --- app/components/standings/TeamScoreBreakdown.tsx | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/app/components/standings/TeamScoreBreakdown.tsx b/app/components/standings/TeamScoreBreakdown.tsx index d7dd1b5..8978135 100644 --- a/app/components/standings/TeamScoreBreakdown.tsx +++ b/app/components/standings/TeamScoreBreakdown.tsx @@ -57,8 +57,7 @@ function sortPicks( const dir = sortDirection === "asc" ? 1 : -1; return picks.slice().sort((a, b) => { if (sortColumn === "pick") { - const sportCmp = a.participant.sport.localeCompare(b.participant.sport); - return sportCmp !== 0 ? sportCmp * dir : (a.pickNumber - b.pickNumber) * dir; + return (a.pickNumber - b.pickNumber) * dir; } // points: sort by actual first, then projected const pointsDiff = a.points - b.points;