Skip to content

Cap remote compact overflow retries#78

Merged
cbusillo merged 1 commit into
local/cbusillo-overlayfrom
fix/remote-compact-overflow-cap
May 15, 2026
Merged

Cap remote compact overflow retries#78
cbusillo merged 1 commit into
local/cbusillo-overlayfrom
fix/remote-compact-overflow-cap

Conversation

@cbusillo
Copy link
Copy Markdown
Owner

Summary

  • replace the 32-step remote compaction context-overflow trim loop with one bounded retry
  • trim to a recent-history window before retrying, then fall back to emergency compaction if it still overflows
  • add focused tests for the bounded trim behavior

Why

The old path could issue one failed remote compaction request per dropped history item, up to 32 attempts. That is the likely source of the repeated remote compaction failures and avoidable rate-limit burn we saw during dogfood.

Validation

  • cargo test -p code-core compact_remote -- --nocapture
  • ./build-fast.sh

@cbusillo cbusillo merged commit bba9c3b into local/cbusillo-overlay May 15, 2026
1 check passed
@cbusillo cbusillo deleted the fix/remote-compact-overflow-cap branch May 15, 2026 00:34
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