fix(stats): report correct cache status for cold requests

[vipul674]

Related Issue

Fixes #5503

Type of Change

• Bug fix

Description

Check if cache had data before fetch using contributionsCache.has(). Now correctly reports:

• MISS for cold requests (no cache entry)
• HIT for requests served from cache
• MISS for bypass requests

Testing

• All existing tests pass (37/37)
• Added mocks for contributionsCache and cacheKey in test files

Checklist

• Code follows project style
• Tests pass
• No console.log statements
• Related issue linked

[vercel]

@vipul674 is attempting to deploy a commit to the jhasourav07's projects Team on Vercel.

A member of the Team first needs to authorize it.

[vipul674]

@JhaSourav07 Ready for review. All tests passing.

[github-actions]

📦 Next.js Bundle Size Report (Gzipped Sizes)

✨ No significant bundle size changes detected.

📊 Summary of Totals

Category	PR Size	Base Size	Difference
Total JS	3697.00 KB	3697.00 KB	0 B
Total CSS	296.58 KB	296.58 KB	0 B

[Aamod-Dev]

Fixing the cache reporting for cold requests is a nice quality-of-life improvement for debugging API performance. Adding the mock for contributionsCache.has in app/api/stats/route.empty-fallback.test.ts keeps the test suite robust. Approved!

[github-actions]

🎉 Congratulations @vipul674! Your PR has been successfully merged. 🚀

Thank you for contributing to CommitPulse. Your work helps us build a better tool for the community.

⚠️ Important for GSSoC Contributors:
You are strictly advised to join our Discord Server as it is mandatory for all GSSoC participants. All important announcements, point claims, and community discussions happen there.

Keep building! 💻✨