Skip to content
This repository was archived by the owner on Mar 14, 2026. It is now read-only.

fix: decay poll retry level per-sweep not per-batch#139

Merged
cdot65 merged 1 commit intomainfrom
cdot65/fix-sweep-level-decay
Mar 12, 2026
Merged

fix: decay poll retry level per-sweep not per-batch#139
cdot65 merged 1 commit intomainfrom
cdot65/fix-sweep-level-decay

Conversation

@cdot65
Copy link
Owner

@cdot65 cdot65 commented Mar 12, 2026

Summary

  • Retry level was decaying per-batch-success within a sweep, causing it to oscillate 0↔1 forever when early batches succeeded but later ones hit rate limits
  • Moved decay to after a full successful sweep (no rate limit errors)
  • Updated existing decay test to match sweep-level semantics
  • Added new test: multi-batch sweep where early batches succeed but later ones fail — verifies level escalates

Closes #135

Test plan

  • New test: multi-batch sweep with partial rate limit — level must escalate
  • Updated decay test for sweep-level semantics
  • All 496 tests pass
  • tsc, biome lint, biome format pass

🤖 Generated with Claude Code

Moves retry level decay from per-batch-success to after a full
sweep completes with no rate limit errors. Prevents early batches
in a sweep from decaying the level before later batches fail.

Closes #135

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@cdot65 cdot65 merged commit d22650b into main Mar 12, 2026
4 checks passed
@cdot65 cdot65 deleted the cdot65/fix-sweep-level-decay branch March 12, 2026 00:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: pollResults retry level decays too fast within sweep (still stuck at retry 1)

1 participant