fix: Fix landing GitHub stats showing zeros - #3505
VedantMadane wants to merge 1 commit into
Conversation
|
Thanks for your first PR on DevTrack! 🎉 A maintainer will review it within 48 hours. While you wait:
If you find DevTrack useful, a ⭐ star on the repo is always appreciated — it helps the project grow and attract more contributors! |
GSSoC Label Checklist 🏷️@Priyanshu-byte-coder — please apply the appropriate labels before merging: Difficulty (pick one):
Quality (optional):
Validation (required to score):
|
|
|
Overall Grade |
Security Reliability Complexity Hygiene |
Code Review Summary
| Analyzer | Status | Updated (UTC) | Details |
|---|---|---|---|
| Secrets | Sep 15, 2026 9:42a.m. | Review ↗ |
Important
AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.
8ba80b3 to
0eea905
Compare
|
I dug into this one properly because the diagnosis is testable, and it splits into one change I want and one built on a premise that doesn't hold. The User-Agent part doesn't fix anything. The claim is that unauthenticated fetches 403 without a I can reproduce a 403 only by explicitly blanking the header ( The good-first-issue change is a real improvement — keep it. What I think is actually happening. Running the page's exact four-call sequence unauthenticated: Core API is 60/hr; Search is 10/min unauthenticated, and this page makes two search calls per render. That's the quota that runs out first, and when either search 403s the values fall back to zero. Note this PR adds a second search call where there was one, so it makes that pressure slightly worse, not better. The fix for #3493 is almost certainly setting What I'd like: keep the good-first-issue counting change, keep the Verified against current |
- Count good-first-issues via search API with is:issue to exclude PRs and use total_count - Send User-Agent header on GitHub API calls as recommended hygiene Fixes Priyanshu-byte-coder#3493 Signed-off-by: Vedant Madane <6527493+VedantMadane@users.noreply.github.com>
0eea905 to
8107f42
Compare
|



Summary
Count open good-first-issues using GitHub Search API with
is:issueto exclude pull requests and readtotal_count, and include a standardUser-Agentheader on GitHub API requests.Changes
repo:Priyanshu-byte-coder/devtrack label:"good first issue" is:issue is:open) readingtotal_countrather than listing up to 100 issues where pull requests are mixed inUser-Agentheader (devtrack-landing-stats) on GitHub API calls as standard client hygieneFixes #3493