Skip to content

fix: use unified metrics cache in compare endpoint (#811)#1219

Merged
Priyanshu-byte-coder merged 1 commit into
Priyanshu-byte-coder:mainfrom
Vaibhavi-14shetty:fix-compare-metrics-cache
May 27, 2026
Merged

fix: use unified metrics cache in compare endpoint (#811)#1219
Priyanshu-byte-coder merged 1 commit into
Priyanshu-byte-coder:mainfrom
Vaibhavi-14shetty:fix-compare-metrics-cache

Conversation

@Vaibhavi-14shetty
Copy link
Copy Markdown
Contributor

Summary

This PR refactors the compare metrics endpoint to use the unified caching implementation from metrics-cache.ts, ensuring consistent cache usage across metric-related APIs and eliminating redundant or endpoint-specific caching logic.

Closes #811

Changes Made

  • Updated src/app/api/metrics/compare/route.ts to use the shared metrics cache instead of local or duplicated caching logic
  • Refactored src/lib/metrics-cache.ts to support unified cache access for compare endpoint usage
  • Ensured consistent cache key usage and retrieval flow across metric requests
  • Removed redundant caching implementation in the compare endpoint

How to Test

  1. Run the development server
  2. Call the compare metrics API with identical inputs multiple times
  3. Verify that subsequent requests return cached responses
  4. Confirm no duplicate external metric fetches occur

Checklist

@vercel
Copy link
Copy Markdown

vercel Bot commented May 26, 2026

@Vaibhavi-14shetty is attempting to deploy a commit to the PRIYANSHU DOSHI's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot added gssoc26 GSSoC 2026 contribution type:bug GSSoC type bonus: bug fix labels May 26, 2026
@github-actions
Copy link
Copy Markdown

GSSoC Label Checklist 🏷️

@Priyanshu-byte-coder — please apply the appropriate labels before merging:

Difficulty (pick one):

  • level:beginner — 20 pts
  • level:intermediate — 35 pts
  • level:advanced — 55 pts
  • level:critical — 80 pts

Quality (optional):

  • quality:clean — ×1.2 multiplier
  • quality:exceptional — ×1.5 multiplier

Validation (required to score):

  • gssoc:approved — counts for points
  • gssoc:invalid / gssoc:spam / gssoc:ai-slop — does not score

Type labels (type:*) are auto-detected from files and title. Review and adjust if needed.
Points formula: (difficulty × quality_multiplier) + type_bonus

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your first PR on DevTrack! 🎉

A maintainer will review it within 48 hours. While you wait:

  • Make sure CI is passing (type-check + lint)
  • Double-check the PR description is filled out and the issue is linked
  • Feel free to ask questions in Discussions if you need help

If you find DevTrack useful, a ⭐ star on the repo is always appreciated — it helps the project grow and attract more contributors!

@Vaibhavi-14shetty
Copy link
Copy Markdown
Contributor Author

Hello @Priyanshu-byte-coder, I’ve completed the changes for this issue #811
Please review my PR when you get time. Let me know if any changes are needed.

@Priyanshu-byte-coder Priyanshu-byte-coder added gssoc:approved GSSoC: PR approved for scoring level:intermediate GSSoC: Intermediate difficulty (35 pts) labels May 27, 2026
@Priyanshu-byte-coder Priyanshu-byte-coder merged commit a1193d8 into Priyanshu-byte-coder:main May 27, 2026
4 checks passed
@github-actions
Copy link
Copy Markdown

🎉 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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gssoc:approved GSSoC: PR approved for scoring gssoc26 GSSoC 2026 contribution level:intermediate GSSoC: Intermediate difficulty (35 pts) type:bug GSSoC type bonus: bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix : use unified metrics cache in compare endpoint

2 participants