Skip to content

Announce top-three leaderboard changes#512

Merged
msaroufim merged 2 commits into
mainfrom
agent/top-three-trashtalk
Jul 12, 2026
Merged

Announce top-three leaderboard changes#512
msaroufim merged 2 commits into
mainfrom
agent/top-three-trashtalk

Conversation

@msaroufim

Copy link
Copy Markdown
Member

What changed

  • watch every leaderboard/GPU podium through the shared database, including API-originated submissions
  • seed current standings silently when Kernelbot starts, then poll every 15 seconds
  • congratulate new winners and top-three entrants with randomized GPU-themed copy
  • roast displaced and dethroned users with randomized copy while using their public leaderboard username and never pinging Discord accounts
  • retry unacknowledged changes after transient Discord failures

Why

Most submissions now arrive through the API while people still gather in Discord and read rankings on gpumode.com. Discord-only submission hooks therefore miss the leaderboard moments worth celebrating.

Behavior

  • the first three submissions fill the podium without trash talk
  • existing standings are not replayed after bot restarts
  • personal bests that do not change podium membership or first place remain silent
  • new leaderboards and all configured GPU types are discovered automatically

Validation

  • ruff check src/kernelbot/top_three.py src/kernelbot/cogs/top_three_cog.py tests/test_top_three.py src/kernelbot/main.py
  • PYTHONPATH=/Users/mark/Dev/kernelbot-top-three-pr/src pytest -q tests/test_top_three.py (10 passed)
  • python -m compileall -q src/kernelbot
  • end-to-end in-process test covers startup seed, simulated API database update, and Discord channel send

An actual staging Discord connection could not be performed because DISCORD_DEBUG_TOKEN is absent from the local environment, macOS Keychain, and the connected Infisical dev/staging/prod environments. The production token was intentionally not reused for staging.

@github-actions

Copy link
Copy Markdown

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  src/libkernelbot
  utils.py
Project Total  

This report was generated by python-coverage-comment-action

@msaroufim msaroufim marked this pull request as ready for review July 12, 2026 03:37
@msaroufim msaroufim merged commit 6eb55f7 into main Jul 12, 2026
4 checks 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