fix(leaderboard): page through daily_breakdowns so wide date ranges don't drop users

[yoominho91]

Widening the date filter makes the leaderboard smaller, and high-spend accounts disappear from it.

Repro

range	rows returned	bottom entry
?from=2026-07-18&to=2026-07-31 (14d)	100	—
?from=2026-07-01&to=2026-07-31 (31d)	52	$13.40

Accounts with five figures of spend in that window are absent from the 31d board while $13 entries are present. A wider window returning fewer rows, with big spenders missing and tiny ones kept, does not look like ranking.

Cause

getLeaderboardByDateRange pulls every matching daily row in one call:

const { data: allDailyBreakdowns } = await this.client
  .from("daily_breakdowns")
  .select("*")
  .in("submission_id", submissionIds)
  .gte("date", params.dateFrom)
  .lte("date", params.dateTo)
  .limit(50000);

PostgREST caps rows server-side (db-max-rows), so .limit(50000) is silently truncated once the window is wide enough to exceed it.

Truncation is not a partial total. A submission whose rows land past the cut ends up with zero daily rows, and the loop below skips it:

const filteredDaily = dailyBySubmission.get(submission.id) || [];
if (filteredDaily.length === 0) continue;

so the user drops off the board entirely rather than showing a smaller number. There is also no .order(), so which submissions survive the cut is arbitrary.

Row count scales with window width × submissions, which is why 7d and 14d look fine and 30d does not.

Fix

Page through with a stable order (submission_id, date), bounded at 200 pages (200k rows) so a pathological range cannot spin.

Aggregating in the DB (an RPC that returns SUM(...) GROUP BY submission_id for the window) would be the leaner long-term shape since it avoids shipping every daily row to the client at all — happy to follow up with that if you prefer it, but this keeps the change contained to the bug.

Checks

• pnpm exec tsc --noEmit: 14 pre-existing errors on main, still 14 after — none in the touched file.
• pnpm build: compiles successfully. Page-data collection fails on both main and this branch without Supabase env vars (different route each run), so unrelated to this change.

[vercel]

@yoo-minho is attempting to deploy a commit to the sculpt Team on Vercel.

A member of the Team first needs to authorize it.

[yoominho91]

Closing — reopened from my personal account as #97 (same commit, same diff). This one was opened from a work account by mistake. Sorry for the noise.