Lane: work-2 · Category: improvement (docs polish) · Source: claude-review bot finding on PR #652 (#504), 2026-07-20.
Triggering example
SKILL.md step 9 reads: 'Schedule the next wake from the snapshot's recommended_cadence (reference/cadence.md); the static ladder in reference/loop.md §5.3 is the Python-free degrade.'
After #504's fix landed (PR #652), loop.md §5.3 now houses BOTH the engine-backed recommended_cadence→delaySeconds mapping table AND the pre-existing Python-free degrade ladder. Step 9's phrasing still technically works (it correctly points to loop.md §5.3 for the degrade ladder) but doesn't surface that §5.3 is also where the engine-backed mapping now lives — a reader following step 9 might go to cadence.md first looking for the seconds mapping, when it's actually in loop.md §5.3.
Observed vs expected
Observed: step 9's cross-reference phrasing predates #504's change and doesn't reflect that both the mapping table and the degrade ladder now live in the same section.
Expected (reviewer's suggested wording): 'Schedule the next wake using the mapping table in loop.md §5.3 (engine-backed) or the static degrade ladder in the same section (Python-free).'
Category
improvement — cosmetic/clarity only, not a functional bug (the existing cross-reference is still navigable, per the reviewer's own assessment). Deferred out of PR #652 to keep that PR scoped to the mapping-table fix itself.
Related
Found during the post-green review pass on PR #652 (closes #504).
Work-class: C2 (mechanical) — attended triage 2026-07-23, operator-ratified. 🤖
Lane: work-2 · Category: improvement (docs polish) · Source: claude-review bot finding on PR #652 (#504), 2026-07-20.
Triggering example
SKILL.mdstep 9 reads: 'Schedule the next wake from the snapshot'srecommended_cadence(reference/cadence.md); the static ladder inreference/loop.md§5.3 is the Python-free degrade.'After #504's fix landed (PR #652),
loop.md§5.3 now houses BOTH the engine-backedrecommended_cadence→delaySecondsmapping table AND the pre-existing Python-free degrade ladder. Step 9's phrasing still technically works (it correctly points to loop.md §5.3 for the degrade ladder) but doesn't surface that §5.3 is also where the engine-backed mapping now lives — a reader following step 9 might go tocadence.mdfirst looking for the seconds mapping, when it's actually inloop.md§5.3.Observed vs expected
Observed: step 9's cross-reference phrasing predates #504's change and doesn't reflect that both the mapping table and the degrade ladder now live in the same section.
Expected (reviewer's suggested wording): 'Schedule the next wake using the mapping table in
loop.md§5.3 (engine-backed) or the static degrade ladder in the same section (Python-free).'Category
improvement — cosmetic/clarity only, not a functional bug (the existing cross-reference is still navigable, per the reviewer's own assessment). Deferred out of PR #652 to keep that PR scoped to the mapping-table fix itself.
Related
Found during the post-green review pass on PR #652 (closes #504).
Work-class: C2 (mechanical) — attended triage 2026-07-23, operator-ratified. 🤖