Skip to content

fix: show contributor rank movement#28

Merged
rogerchappel merged 1 commit into
mainfrom
fix/codex/australia-rank-delta
May 13, 2026
Merged

fix: show contributor rank movement#28
rogerchappel merged 1 commit into
mainfrom
fix/codex/australia-rank-delta

Conversation

@rogerchappel
Copy link
Copy Markdown
Owner

@rogerchappel rogerchappel commented May 13, 2026

Summary

  • Fix previous-rank lookup so same-day reruns do not compare a leaderboard against itself.
  • Add a visible Movement column to contributor leaderboards.
  • Repair the latest Australia snapshot so rogerchappel shows the movement from rank 18 to rank 4.
  • Repair the latest global snapshot so repeated global contributors show movement against the 2026-05-07 snapshot instead of all appearing as new.

Verification

  • Tests or checks run: pnpm run check; pnpm run build; pnpm run validate:data; ad hoc writeSnapshots same-day rerun check returned { "rank": 4, "previous_rank": 18, "delta": 14 }
  • Manual review completed: inspected generated Australia leaderboard HTML for the Movement column and rogerchappel showing ↑14; inspected generated global leaderboard HTML for mixed New and movement deltas such as bocaletto-luca showing ↓2 and mP1 showing ↓7.

Risk Level

  • Low
  • Medium
  • High

Notes:

Rendering and snapshot metadata only. No production data mutation, secrets, auth, billing, or public API compatibility changes.

Rollback Plan

  • Revert commits 856c1fe and ffea7fc to restore the previous snapshot lookup, table layout, and latest snapshot values.

Human Decision Needed

  • None
  • Maintainer review
  • Product/design decision
  • Security/privacy review
  • Other:

@rogerchappel rogerchappel marked this pull request as ready for review May 13, 2026 05:51
Copy link
Copy Markdown
Owner Author

@rogerchappel rogerchappel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good

@rogerchappel rogerchappel merged commit 958667e into main May 13, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant