Skip to content

feat(leaderboard): limit ranking queries to top 20#49

Open
KurosawaGeeker wants to merge 1 commit into
hikariming:mainfrom
KurosawaGeeker:feat/leaderboard-floating-comments
Open

feat(leaderboard): limit ranking queries to top 20#49
KurosawaGeeker wants to merge 1 commit into
hikariming:mainfrom
KurosawaGeeker:feat/leaderboard-floating-comments

Conversation

@KurosawaGeeker

Copy link
Copy Markdown
Contributor

Summary

  • Limit leaderboard page, API, and leaderboard JSON-LD queries to the shared top-20 leaderboard cap.
  • Remove client-side pagination so leaderboard views cannot enumerate entries beyond the capped result set.
  • Add 100-entry development preview data while only returning the requested capped subset to the page.
  • Stabilize profile comment ordering when comments are created within the same millisecond.

Validation

  • pnpm typecheck
  • pnpm test (14 files / 128 tests passed)
  • pnpm lint
  • curl http://localhost:3001/leaderboard?view=heat only includes mock-dev-001 through mock-dev-020.

Generated Files

No generated files, GraphQL codegen, or DB baseline updates included.

@vercel

vercel Bot commented Jun 29, 2026

Copy link
Copy Markdown

@KurosawaGeeker is attempting to deploy a commit to the hikariming's projects Team on Vercel.

A member of the Team first needs to authorize it.

@KurosawaGeeker KurosawaGeeker marked this pull request as ready for review June 29, 2026 11:49
@KurosawaGeeker KurosawaGeeker force-pushed the feat/leaderboard-floating-comments branch from de3ebb3 to 453fbc1 Compare June 30, 2026 12:25
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