Repository navigation
fix: public profile not found, resume analysis 502, AI model errors, leaderboard rate limits - #3530
Merged
Merged
Conversation
…leaderboard rate limits - Public profile: the canonical-username redirect ran inside a try/catch. redirect() works by throwing, so the catch turned every mixed-case profile link (e.g. /u/Priyanshu-byte-coder) into "Profile Not Found". The page already redirects correctly after loading. - Resume: the contribution query asked for 50 repositories with per-commit diff stats in one request; GitHub answers 502 past its ~10s budget. Repositories are now paged 10 at a time and commits carry no diff stats (line totals come from merged pull requests). - AI features: Groq returned model_not_found for llama-3.3-70b-versatile on the production key, so personality, AI insights, resume wording and project tutor all failed (project tutor still used the long-retired llama3-8b-8192). A shared helper tries GROQ_MODEL, then falls back through available models and remembers the one that works. - Leaderboard: per-user stats paged up to 10 commit searches for streaks and hit GitHub's secondary rate limit. Stats now come from contribution data via GraphQL (streak from the contribution calendar, as on GitHub profiles), using each user's stored token. - SSE stream: connections now end at 4 minutes instead of being killed at the 300s function limit, which logged an error per connection.
github-actions
Bot
added
type:bug
github-actions
Bot
added
type:feature
|
🎉 Merged! Thanks for contributing to DevTrack. If the project has been useful to you, a ⭐ star on the repo is the easiest way to support it — it helps DevTrack get discovered by more developers. Keep an eye on open issues for your next contribution! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes the errors reported after the schema migration, found in production logs.
Public profile "Profile Not Found": the canonical-username
redirect()ran inside atry/catch.redirect()throws by design, so every mixed-case link (/u/Priyanshu-byte-coder) fell into the catch. Lowercase URLs already worked.Resume generation error:
/api/cv/analyzerequested 50 repos with per-commit diff stats in one GraphQL query, and GitHub returns 502 past its ~10s budget. Measured against GitHub:Repos are now paged 10 at a time, and line totals come from merged PRs.
AI features failing (personality, AI insights, resume wording, project tutor): Groq returned
model_not_foundforllama-3.3-70b-versatileon the production key. Project tutor still used the long-retiredllama3-8b-8192. The newsrc/lib/groq.ts:GROQ_MODEL;Leaderboard rate-limited: each user's streak paged up to 10 commit searches. Stats now come from GitHub contribution data via GraphQL (at most 2 requests per user, using their stored token).
SSE
/api/stream: connections now end at 4 min instead of being killed at the 300s function limit, which logged an error for every connection.Tests: 2488 pass (
jira-credentialsis an existing timing flake that passes when run alone); added tests for the Groq fallback.