Skip to content

perf: optimize weekly digest aggregation - #30

Merged
sarthakagrawal927 merged 1 commit into
mainfrom
agent/optimize-weekly-digest
Aug 10, 2026
Merged

perf: optimize weekly digest aggregation#30
sarthakagrawal927 merged 1 commit into
mainfrom
agent/optimize-weekly-digest

Conversation

@sarthakagrawal927

Copy link
Copy Markdown
Member

What

  • replace repeated sender/date/theme traversals with cached parsing and single-pass aggregation
  • add a deterministic 50,000-message performance guard
  • ignore CodeVetter's local performance-run artifacts

Why

CodeVetter repeatedly selected weekly digest generation as an application hotspot. The source experiment moved from a 16.181 ms/op observed baseline to 7.266 and 7.534 ms/op on later runs.

Evidence boundary

This is directional evidence, not a paired-confirmed percentage claim: the original baseline capsule was not retained for the identical paired verifier. The draft should merge only after normal repository review.

Checks

  • pnpm test — 49/49 passed
  • pnpm typecheck
  • Biome on the complete repository
  • digest golden fixture

@sarthakagrawal927
sarthakagrawal927 marked this pull request as ready for review August 10, 2026 07:04
@sarthakagrawal927
sarthakagrawal927 merged commit 0631c11 into main Aug 10, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant