feat: reviewers / issues / opened-PR leaderboards#231
Conversation
Add three new "Top" leaderboards (reviewers, issue reporters, PR authors) alongside the existing contributors/companies sections, driven by a shared ranking data contract. - types: RankingEntry / Ranking + reviews / issuesOpened / pullRequestsOpened counters on Contributor - TopRankingView: reusable leaderboard card built on the existing TopCard (rank medal, avatar, name, count column, GitHub profile link) - app.vue / TopSectionView: fetch the three ranking files and render a card per metric, hidden when its ranking is empty - deterministic mock-data generator (scripts/generate-mock-stats.mjs) so the UI can be reviewed before the data pipeline lands - vitest coverage for the new components Note: ranking data is currently mocked. Real data comes from the Traces pipeline — see PrestaShop/traces#232. 🤖 Generated with Claude Code
|
@PrestaEdit Traces is Merged and released. Could you check your PR and give us screenshots or vidéo for checking the behavior ? Thanks |
Add traces:fetch:pullrequests:all / traces:fetch:issues / traces:generate:topstats to the scheduled Build & Deploy pipeline, and publish top_reviewers.json, top_issues.json and top_pullrequests.json to the site. Requires prestashop/traces >= v6.3.0 (pulled by composer create-project). 🤖 Generated with Claude Code
| @@ -0,0 +1,62 @@ | |||
| // Deterministic mock generator for the Top stats leaderboards. | |||
There was a problem hiding this comment.
Removed in 8ddaf39 — real ranking data now comes from prestashop/traces v6.3.0 via the Build & Deploy workflow, so the temporary mock generator is no longer needed.
| # Superpowers brainstorm (transient) | ||
| .superpowers/ | ||
| # Superpowers workflow docs (kept local only) | ||
| docs/superpowers/ |
There was a problem hiding this comment.
| # Superpowers brainstorm (transient) | |
| .superpowers/ | |
| # Superpowers workflow docs (kept local only) | |
| docs/superpowers/ |
There was a problem hiding this comment.
Removed in 508a043 (both superpowers .gitignore entries).
Real ranking data is now produced by prestashop/traces v6.3.0 via the Build & Deploy workflow, so the temporary mock generator is no longer needed. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
The three new leaderboards render as expected — screenshots below.
Each card reuses the existing Screenshots: Top reviewers / Top issue reporters / Top PR authors ⬇️ |
github-project-automation
Bot
moved this from Ready for review
to Merged
in PR Dashboard
|
Follow-up moved to its own PR (this one was already merged): #233 — full-population Wall-of-Fame tabs (Reviewers / Issue reporters / PR authors) + Top-50 cap. Depends on PrestaShop/traces#233. |
3 participants
Summary
Adds three new "Top" leaderboards next to the existing Contributors/Companies sections, to surface contribution types that are invisible today:
How it works
TopRankingView.vuerenders each leaderboard through the existing genericTopCard(rank medal, avatar, name, count column, GitHub profile link).app.vuefetches three pre-sorted ranking files (top_reviewers.json,top_issues.json,top_pullrequests.json) and passes them toTopSectionView; a section whose ranking is empty/missing is hidden (no crash).RankingEntry/Rankingand three optional counters (reviews,issuesOpened,pullRequestsOpened) onContributorform the data contract.scripts/generate-mock-stats.mjs) produces mock ranking files so the UI can be reviewed now. These files are gitignored (generated by Traces in production), consistent with the existingpublic/*.jsondatasets.Data dependency
Real data is produced by the Traces CLI — see PrestaShop/traces#232 (adds
traces:fetch:issues,traces:fetch:pullrequests:all,traces:generate:topstats). Once merged and deployed, the three ranking files join the published dataset and these sections show real figures.Test Plan
vitest run— green (TopRankingView + TopSectionView component tests, incl. empty-ranking-hidden case)eslint .— clean