Skip to content

fix: intervals from API never reflected in on-screen analysis #29

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
MaximumTrainer merged 3 commits into from
Mar 23, 2026
Merged

fix: intervals from API never reflected in on-screen analysis #29

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
6cbae48
Initial plan
Copilot Mar 23, 2026
a8c56b0
fix: store allTrainingIntervals in state instead of parsedIntervals
Copilot Mar 23, 2026
f68ffec
fix: fetch interval data from all 60-day activities, not just 20
Copilot Mar 23, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 5 additions & 8 deletions src/hooks/useIntervalsData.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -154,11 +154,8 @@ export const useIntervalsData = (athleteId: string, accessToken: string, authTyp
?? wellnessEntries.find((w) => typeof w.weight === 'number' && w.weight > 0)?.weight
?? null;

// 3. Parse Latest Session for Sprint Metrics
// 3. Select latest session/activity for sprint metrics
const latestSession = activities[0];
const parsedIntervals = latestSession
? SprintParser.parseTrackSession(latestSession)
: [];

// 4. Calculate Neural Fatigue Index (NFI)
const todayVmax = latestSession?.max_speed || 0;
Expand Down Expand Up @@ -204,11 +201,11 @@ export const useIntervalsData = (athleteId: string, accessToken: string, authTyp
// 8. Race estimates based on best Vmax + training interval history
const bestVmax60d = activities.reduce((best, a) => Math.max(best, a.max_speed), 0);

// Fetch structured interval data from the Intervals.icu API for each activity.
// Fetch structured interval data from the Intervals.icu API for each activity
// in the 60-day window. Only sprint-range efforts (≤ 400m) are included by the parser.
// The /intervals endpoint provides accurate rep-level data (distance, max_speed,
// moving_time) that is not present in the activity list response.
// We limit to 20 most-recent activities to keep the number of parallel calls reasonable.
const activitiesForIntervals = activities.slice(0, 20);
const activitiesForIntervals = activities;
const intervalFetches = await Promise.allSettled(
activitiesForIntervals.map(async (a) => {
const res = await fetch(
Expand Down Expand Up @@ -370,7 +367,7 @@ export const useIntervalsData = (athleteId: string, accessToken: string, authTyp

setData({
activities,
intervals: parsedIntervals,
intervals: allTrainingIntervals,
wellness: latestWellness,
nfi: currentNFI,
nfiStatus,
Expand Down
Loading