CH-045: Document current Rumble rankings - #273
Merged
Merged
Conversation
flemming-n-larsen
marked this pull request as ready for review
September 13, 2026 12:56
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TumFGXBzSvL3FFd6SWiZt6
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
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.
Acceptance brief
CH-045; it remains wanted because robocode.dev needs the canonical explanation for the ranking behavior accepted in mergedrobocode-dev/rumble-data#12.RBC-005— “Hourly catalog polling publishes only changed reviewed content”:verifies-something-adjacent; external integration evidence exercises changed and unchanged synchronization and asserts the serialized workflow/skip contract, but does not execute the hosted no-op path end to end.RDA-006— “APS gives each distinct pairing equal weight within the active ranking epoch”:verifies; external integration evidence covers pairing-first averaging, counts, active-version and behavior-epoch exclusion, zero-sample entries, and stable ties.RDA-007— “Publication freshness changes only when current ranking data changes”:verifies-something-adjacent; external integration evidence covers ranking-tree freshness and snapshot stability, while dispatch/reconciliation checks are static workflow assertions.RDA-008— “The first writer after a UTC month boundary preserves immutable cumulative history”:verifies; external integration evidence covers missing months, byte-for-byte copies, first-deployment behavior, partial recovery, and overwrite refusal.RDA-009— “A dashboard viewer can distinguish current rankings from read-only monthly history”:verifies-something-adjacent; external E2E-tagged evidence checks the shipped page and selection paths statically rather than driving a browser./rumble/rankingscontract and alignedCAP-014/CAP-015meanings; all five new criteria remain@draftuntil external evidence can be registered locally. No decision record is added, invalidated, or superseded.Summary
rumble-data.CHANGELOG.md.Verification
npm run buildinweb: passed (VitePress; existing chunk-size warning only).clue validate --forbid-changes: passed.git diff --check origin/main HEAD: passed.python -m unittest discover -s tests -vinrumble-data: 41 tests passed.Cliewen proposal
CH-045changes/CH-045-document-rumble-rankings/proposal.mdat proposal commit562915672; removed by the digest as required.329af8c3bfb73d3307a4ccbe621e91d0350b45bd329af8c3bfb73d3307a4ccbe621e91d0350b45bdTraceability and decisions
RBC-005,RDA-006,RDA-007,RDA-008, andRDA-009;CAP-014andCAP-015documentation aligned.rumble-data#12.C-001passed viaclue validate;C-002is satisfied by this branch and human-merge boundary;C-003andC-004do not apply because no Bot API, physics, or compatibility behavior changes.Cliewen checklist
@draft, with their external scenario-resolution verdicts recorded above.329af8c3bfb73d3307a4ccbe621e91d0350b45bdreceived a clean second review pass.origin/main.main.