Problem
In src/managers/summary-manager.js around line 384-389, the week-to-month overlap logic uses both Sunday and Monday boundaries, which can cause a week (e.g., 2026-W09) to be included in two different months.
Expected behavior
A week should be assigned to exactly one month. The src/utils/summary-detector.js already uses Monday-based assignment — summary-manager.js should match.
Suggested fix
Replace the condition that checks sunday >= firstDay && monday <= lastDay with a check that monday >= firstDay && monday <= lastDay, so week ownership is based solely on the week's Monday.
Found by CodeRabbit review on wiggitywhitney/spinybacked-orbweaver-eval#7.
Problem
In
src/managers/summary-manager.jsaround line 384-389, the week-to-month overlap logic uses both Sunday and Monday boundaries, which can cause a week (e.g., 2026-W09) to be included in two different months.Expected behavior
A week should be assigned to exactly one month. The
src/utils/summary-detector.jsalready uses Monday-based assignment —summary-manager.jsshould match.Suggested fix
Replace the condition that checks
sunday >= firstDay && monday <= lastDaywith a check thatmonday >= firstDay && monday <= lastDay, so week ownership is based solely on the week's Monday.Found by CodeRabbit review on wiggitywhitney/spinybacked-orbweaver-eval#7.