feat(rounds): polish single-elim tournament — pilots/advance per heat, whole-bracket viz, own container#311
Merged
Merged
Conversation
…, whole-bracket viz, own container Refining the single-elim Build-tournament UI from testing: - Pilots per heat: a dropdown (capped at the primary timer's nodes); the bracket's level geometry follows it. `bracketLevelFields(field, heatSize, advance)` computes the field entering each level. - Advance per heat: a configurable "how many move on" (engine: single_elim gains an `advance` param, default heat_size/2; UI: a selector 1..heatSize-1, defaulting to the top half — so 4-up → top 2 advance, the rest eliminated, and you can set top-1-of-4). Threaded through the level math + seeding. - Whole bracket built out: buildBracketView now synthesizes the full tree — every level rendered with the right number of matches and TBD placeholder slots, before any heats exist. - Own container: brackets are no longer folded into the Heats list (each level shows as a normal heat card); a new "Tournaments" card below Heats holds the bracket tree(s) and the Build-tournament button. Rounds + heats appear in the normal Rounds/Heats sections. cargo xtask ci green (single_elim advance param + 17 unit tests); frontend check 0 errors; 501 console + 35 component tests; verified the full TBD tree + container live via screenshots. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Polishing the single-elim Build-tournament UI from testing:
single_elimadvanceparam, defaultheat_size/2; UI selector 1..heatSize-1). 4-up → top 2 advance by default; you can set top-1-of-4.buildBracketViewsynthesizes the full tree with TBD placeholder matches for every level, before any heats exist.cargo xtask cigreen; check 0 errors; 501 console + 35 component tests; full tree + container verified live via screenshots.🤖 Generated with Claude Code