fix(frontend): address review suggestions from embed card PRs

[junhoyeo]

Summary

Follow-up to #384 and #386 — addresses all code review suggestions from Copilot and cubic.

Fixes

From #384 (embed card redesign):

• Rank accent bar uses amber for non-medal ranks — rankAccent was computed identically to rankColor via getRankColor(), so non-medal ranks rendered blue (rankDefault) for the accent bar instead of amber. Now uses palette.accentRank (#D29922 dark / #9A6700 light) for the default accent, while medal ranks (1–3) keep their specific colors. (Identified by cubic)

From #386 (contribution graph):

• Skip contribution fetch in compact mode — ?compact=1&graph=1 no longer triggers an unnecessary backend call since the renderer ignores contributions in compact mode anyway. (Identified by cubic)
• Graceful degradation on graph fetch errors — Contribution fetch failures now fall back to null via .catch(() => null) instead of triggering a 500 response, so the stats-only card still renders. (Identified by cubic)
• Cutoff date alignment buffer — Added 7-day buffer before the 1-year cutoff using UTC-based dates, so contribution data for the first visible week of the grid isn't missing. (Identified by Copilot)
• SQL-level aggregation — Replaced JS-side Map aggregation with SUM(cost) GROUP BY date in the query, reducing data transfer for users with many submissions. (Identified by Copilot)

Tests

• Added test: uses amber accent bar for non-medal ranks instead of brand blue
• All 23 tests pass

---

---

Summary by cubic

Addresses review feedback from #384 (embed card redesign) and #386 (contribution graph) to polish the profile embed. Fixes rank accents, avoids unnecessary graph fetches in compact mode, adds graceful fallbacks, aligns UTC cutoff dates with a 7‑day buffer, and moves contribution aggregation to SQL.

• Bug Fixes

  • Use amber accent bar for non‑medal ranks; medal ranks keep their colors.
  • Skip contribution fetch when compact=1 and graph=1.
  • On graph fetch failure, return null so the stats‑only card still renders.
  • Use UTC cutoff of one year ago minus 7 days to include the first visible week.

• Refactors

  • Aggregate contributions with SUM(cost) GROUP BY date in SQL to reduce data transfer.

^{Written for commit 2fc5c83. Summary will update on new commits.}

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
tokscale	Ready	Preview, Comment	Apr 2, 2026 2:47am

[cubic-dev-ai]

No issues found across 4 files

[devin-ai-integration]

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 4 additional findings.