From 4e4f7196703db9de2e07d502ec9cdcf986c86333 Mon Sep 17 00:00:00 2001 From: Chris Farhood Date: Mon, 20 Jul 2026 19:11:39 -0400 Subject: [PATCH] fix: resolve code-review correctness bugs in README + build template MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Three CONFIRMED defects, all from template/README text written before the vo2max conflict section was revised, now reconciled with the corrected docs: - README taper example said TSB '−5 to +5' (under-recovered) — corrected to the doctrinal +5 to +15 that periodization.md and both plan templates use. - plan-template-build.md VO2 prescription said 'bias toward longer reps (3–5 min)... accumulate more time >90% VO2max than 30/30s' — contradicted its own cited source and was self-contradictory (140s ≠ 3–5 min). Replaced with the neutral menu: default medium (~2–4 min, near the ~140s optimum), short/long by goal, 'no rep length proven superior in cycling.' - plan-template-build.md cited the phantom 'Yu et al.' (marked ❌ NOT FOUND in citations.md) — re-grounded the frequency heuristic in Seiler 2024. DRY findings (inline PMID/DOI duplication, restated rhythm/field-test tables) left as-is: inline citations and self-contained, independently-loadable docs were explicit design requirements for this skill. Co-Authored-By: Claude Opus 4.8 (1M context) Claude-Session: https://claude.ai/code/session_01TqrhBhC3GEcKyaw8RTk8G6 --- README.md | 4 ++-- assets/plan-template-build.md | 10 ++++++---- 2 files changed, 8 insertions(+), 6 deletions(-) diff --git a/README.md b/README.md index 5471147..e9a423c 100644 --- a/README.md +++ b/README.md @@ -23,8 +23,8 @@ number; ask this skill what to do about it. 1. **MCP** returns the data — e.g. `get_athlete_summary` (CTL/ATL/TSB), `get_wellness_data` (HRV, RHR, sleep), `get_athlete_power_curves`, `get_activities`. 2. **Skill** supplies the reasoning — e.g. "TSB is −25 eight days out from an A-race; the taper - doc says pull it toward −5 to +5 by race day," or "this VO2 session accumulated little time - ≥90% VO2max; the interval doc says lengthen the reps." + doc targets +5 to +15 by race day, so shed fatigue," or "this VO2 session banked little time + ≥90% VO2max; the interval doc says check whether rep length / work:rest kept power in the band." 3. **You (Claude)** combine them into a recommendation, always citing which layer supplied what. ## Repository layout diff --git a/assets/plan-template-build.md b/assets/plan-template-build.md index 0d81fe8..42d53ae 100644 --- a/assets/plan-template-build.md +++ b/assets/plan-template-build.md @@ -25,8 +25,9 @@ carry more *intensity*, so recovery weeks matter more, not less. ## Weekly shape (build phase) -Fewer but sharper hard sessions. **Frequency of hard sessions matters more than the exact -polarized-vs-pyramidal label** (Yu et al. — see `references/periodization.md`). A common shape: +Fewer but sharper hard sessions. **Getting enough quality hard sessions in tends to matter more +than the exact polarized-vs-pyramidal label** — a heuristic grounded in Seiler 2024, not a +quantified law (see `references/periodization.md`). A common shape: ``` VO2max session: 1–2 × /week (see references/vo2max-intervals.md for rep length / work:rest) @@ -39,8 +40,9 @@ Recovery / off: as needed ### VO2max prescription (from `references/vo2max-intervals.md`) - Goal is **maximizing time ≥90% VO2max**, not just "going hard." -- Bias toward **longer reps (~3–5 min, ≈140 s optimum region)** at ~0.85 work:rest for rebuilding - sustained aerobic power; they accumulate more time >90% VO2max than 30/30s at matched intensity. +- **No rep length is proven superior in cycling.** Default to **medium reps (~2–4 min, near the + pooled ~140 s optimum)** at ~0.85 work:rest; use short (30/15 s) or long (4–5 min) by athlete and + goal. Do not assume "longer is better" — see the modality conflict in `references/vo2max-intervals.md`. - Prescribe from **power**; for MTB, HR will lag and mislead (`references/data-confounds.md`). ## Keep durability alive