Skip to content

fix(cache): drain workout range pagination - #163

Merged
garrettladley merged 2 commits into
mainfrom
gml/fix-workout-range-pagination
May 22, 2026
Merged

fix(cache): drain workout range pagination#163
garrettladley merged 2 commits into
mainfrom
gml/fix-workout-range-pagination

Conversation

@garrettladley

Copy link
Copy Markdown
Owner
  • drains whoop next_token pages before returning workout range results
  • drains repository cursors before treating cached range data as complete
  • applies cursor draining to workout, cycle, sleep, calendar, and historical cache reads
  • adds regression coverage for api pagination and cache cursor draining with t.Context() and t.Helper()

@codecov

codecov Bot commented May 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 33.80282% with 47 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
internal/cache/service.go 33.80% 42 Missing and 5 partials ⚠️

📢 Thoughts on this report? Let us know!

@garrettladley
garrettladley enabled auto-merge (squash) May 22, 2026 18:53
@garrettladley
garrettladley merged commit bb51051 into main May 22, 2026
5 checks passed
@garrettladley
garrettladley deleted the gml/fix-workout-range-pagination branch May 22, 2026 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant