Repository navigation
aggregator: anchor OpenCode Go resets to each workspace paragraph (#1626) - #1717
Conversation
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 5 minutes. View limit details
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Fixes #1626 (per triage task 1746 ruling: paragraph anchoring).
Change
pm-stack/aggregator/src/quota-extract.ts—goWorkspaceno longer pairs monthly resets by section-wide index. Each workspace's reset is now the firstmonthly reset … UTCmatch in its own paragraph — the text from its percentage match to the next blank line / end of section. A paragraph with no reset yieldsresetsAt: nullfor that workspace; a reset is never borrowed from another workspace or from a stray earlier mention. Percentages (already name-anchored),monthDayTime's roll-forward policy, and other pools' extractors are unchanged.Tests
New regression test in
pm-stack/aggregator/test/quota.test.tsfor the three probe shapes:null, Lab 2026-11-09;bun test test/quota.test.ts— 12 passed ( incl. the existing combined-paragraph fixture).actions.test.tssnooze-strip failure (aggregator: actions.test.ts fails on main — applied snooze never lands in the Snoozed strip ('Snoozed by you (1)' missing) #1682), confirmed on clean base earlier this shift.Queue task 1882 · branch
queue/1882-1626-go-reset-anchorNeed help on this PR? Tag
@codesmith-botwith what you need.