Skip to content

Move Fantasy caches to Redis - #293

Merged
sfreeman422 merged 1 commit into
masterfrom
feature/redis-db
Sep 12, 2026
Merged

Move Fantasy caches to Redis#293
sfreeman422 merged 1 commit into
masterfrom
feature/redis-db

Conversation

@sfreeman422

@sfreeman422 sfreeman422 commented Sep 12, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • move persistent Sleeper player, projection, FantasyCalc, waiver-market, and AI-analysis caches from the Node heap to Redis
  • compact the large Sleeper player payload and preserve cache TTLs, stale FantasyCalc fallback, failure backoff, and in-flight request deduplication
  • give the Redis test mock real TTL semantics and isolate Fantasy cache state between tests

Testing

  • full backend and frontend test suites
  • backend and frontend coverage gates
  • backend TypeScript build
  • repository lint and formatting checks

@sfreeman422
feat(fantasy): move persistent caches to Redis
f7d1b64 
Move player, projection, FantasyCalc, waiver market, and AI analysis cache values out of the Node heap while preserving TTLs, stale fallback, and in-flight request deduplication.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI lite review requested due to automatic review settings September 12, 2026 00:14
Copilot AI reviewed Sep 12, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@sfreeman422
sfreeman422 requested a lite review from Copilot September 12, 2026 00:15
Copilot AI reviewed Sep 12, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@sfreeman422
sfreeman422 merged commit b86f11b into master Sep 12, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort, defaults to Lite
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sfreeman422