fix(metrics): include GraphQL discussion activity in engagement score - #3461
fix(metrics): include GraphQL discussion activity in engagement score#3461nyxsky404 wants to merge 4 commits into
Conversation
Co-authored-by: Cursor <cursoragent@cursor.com>
GSSoC Label Checklist 🏷️@Priyanshu-byte-coder — please apply the appropriate labels before merging: Difficulty (pick one):
Quality (optional):
Validation (required to score):
|
|
@Priyanshu-byte-coder please review pr and merge it, i have also fixed the ci failures |
|
Hi @Priyanshu-byte-coder — gentle nudge. This PR is ready for review from my side. I've rebased/kept it current where possible. If CI is red due to unrelated upstream/main issues or deploy previews, the code change itself should still be reviewable. Please review and merge when you can — and add Thank you! |
Summary
/api/metrics/devtrack-badgesalways returneddiscussions.count/points: 0because the score used a hardcoded placeholder. This wires up the same GraphQLcontributionsCollectionfields used by/api/metrics/discussions(discussion + discussion-comment counts for the last 30 days) and folds them into the weighted breakdown (* 2, capped at 15). GraphQL failures soft-fail to 0 so the rest of the score still returns.Closes #3352
Type of Change
What Changed
src/app/api/metrics/devtrack-badges/route.tsHow to Test
GET /api/metrics/devtrack-badgesbreakdown.discussions.countandpointsare non-zero andtotalincludes those pointsExpected result:
Discussions reflect real GitHub contribution data instead of a permanent zero placeholder.
Checklist
console.log, debug code, or commented-out blocksnpm run lintpasses locallynpm run type-check)Additional Context
Weight matches
issuesOpened(count * 2, max 15). Overall score still clamps to 100.Made with Cursor