release: 0.18.1 - #157
Merged
Merged
Conversation
Two commits since v0.18.0. Patch — no API change. Fixes a debug-build panic on summarization retries: delay_for_attempt documents a 1-indexed attempt and computes attempt - 1, and llm_compaction.rs passed the raw 0-indexed loop variable. The panic landed on a detached task, so the summarization vanished silently and compaction fell back deterministically. Adds a one-shot warning when LlmCompaction briefings keep losing the race to the budget, gated on a request having actually been issued and on the briefing surviving into the result, and reports the cost of a briefing rejected on fingerprint mismatch. 627 tests, clippy clean under -Dwarnings, live price audit 26 compared / 0 drifted. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two commits since
v0.18.0. Patch — no API change.Fixed
Summarization retries panicked in debug builds.
RetryConfig::delay_for_attemptdocuments a 1-indexed attempt and computesattempt - 1;llm_compaction.rspassed the raw0..=max_retriesloop variable, so the first retry underflowedusize.agent_loop.rsincrements before calling and was correct — this was confined to compaction.It landed on a detached task, so nothing surfaced: the summarization vanished, no briefing arrived, and compaction fell back deterministically. That is one of the behaviours #150 was filed about, so some of those fallbacks were this rather than summarizer speed.
A briefing rejected on fingerprint mismatch now reports what it cost. The event carried
summary: None, so cost accounting offSummaryStatsunder-counted the most wasteful failure — contradicting the contracttypes.rsstates explicitly.Added
LlmCompactionsays so when briefings keep losing the race. After five consecutive deterministic fallbacks it warns once, naming the likely cause; a splice that lands resets the streak and clears the latch. Gated on a request having actually been issued, so it cannot claim a cost in the inert configuration.The module docs now lead with the summarizer choice, which is the actual cause: reusing the loop's
ModelConfigis the obvious call and, for a slow loop model, the worst one.Verification
-DwarningsRelease commit is exactly two files —
Cargo.lockis gitignored.Still open: #150 —
compact_headroom_turnsdefaults toSome(30), pinning the target ratio to its 0.15 floor for tool-heavy work. That is a defaults change for every user and wants measurement across growth rates first.🤖 Generated with Claude Code