chrisp/brackt

Fork 0

claude/reprocess-bracket-scoring-mbkfa4 #128

Merged

chrisp merged 3 commits from claude/reprocess-bracket-scoring-mbkfa4 into main

2026-07-04 04:09:10 +00:00

chrisp commented

2026-07-04 03:37:46 +00:00

Owner

No description provided.

chrisp added 2 commits 2026-07-04 03:37:47 +00:00

Fix reprocess bracket to score mirror windows like the primary 0baef5a948

Reprocessing a tennis (qualifying) major recomputed the primary window's
QP correctly (R16 players = 1.5, the average of the 9th–16th values) but
left mirror/sibling windows showing 2 QP. Two defects:

1. The mirror fan-out ran through fanOutMajorIfPrimary, which swallows
   every error and returns void, so reprocess reported a green "success"
   even when mirrors were never re-scored. The reprocess qualifying path
   now calls syncMajorFromPrimaryEvent directly and folds the SyncReport
   (windows synced / failed) into the response, surfacing failures instead
   of hiding them.

2. Mirror windows split QP by counting canonical tournament_results rows
   at each placement, which only equals the round's structural tie span
   when placements are final. Mid-tournament, players floored at a tier
   make the row-count diverge from the tier size, so mirrors split
   differently than the primary. syncMajorFromPrimaryEvent now derives the
   structural span (R16 = 8, QF = 4, SF = 2, Final = 1) from the primary
   bracket via deriveBracketQualifyingStates and merges it over the
   canonical counts, so every window splits identically to the primary at
   every stage. Golf and CS2 Swiss-exit placements keep their canonical
   count via the merge.

Adds a fan-out test covering an R16-in-progress bracket where only 4 rows
sit at placement 9: the mirror is scored with the structural span (8), not
the live count (4).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013UkZVYgLmquWV2xDn349T2

Surface partial mirror fan-out failures as a warning, not a green success

🚀 Deploy / 🧪 Test (pull_request) Successful in 3m30s

Details

🚀 Deploy / ʦ🔍 Typecheck & Lint (pull_request) Failing after 49s

Details

🚀 Deploy / 🐳 Build (pull_request) Has been skipped

Details

🚀 Deploy / 🚀 Deploy (pull_request) Has been skipped

Details

12c165a481

When some mirror windows fail to sync during reprocess, return an error
result so it renders as a warning banner (with the synced/failed counts and
reasons) instead of a green success the admin might skim past while those
windows are left stale.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013UkZVYgLmquWV2xDn349T2

chrisp added 1 commit 2026-07-04 04:02:12 +00:00

Fix lint errors in PlayoffBracket and sync-tournament-results test

🚀 Deploy / ʦ🔍 Typecheck & Lint (pull_request) Successful in 1m25s

Details

🚀 Deploy / 🐳 Build (pull_request) Has been skipped

Details

🚀 Deploy / 🧪 Test (pull_request) Successful in 3m8s

Details

🚀 Deploy / 🚀 Deploy (pull_request) Has been skipped

Details

698c6a6509

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

chrisp merged commit 8416b54052 into main

2026-07-04 04:09:10 +00:00

chrisp deleted branch claude/reprocess-bracket-scoring-mbkfa4

2026-07-04 04:09:10 +00:00

chrisp referenced this pull request from a commit

2026-07-04 04:09:10 +00:00

Merge pull request 'claude/reprocess-bracket-scoring-mbkfa4' (#128) from claude/reprocess-bracket-scoring-mbkfa4 into main

No reviewers

No labels

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: chrisp/brackt#128

No description provided.

Rows
Columns