fix(eve): improve compaction checkpoints#771
Open
ruiconti wants to merge 1 commit into
Open
Conversation
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
ctgowrie
approved these changes
Jul 14, 2026
16afb63 to
55cefc0
Compare
De-ASI-INTERFACE
approved these changes
Jul 14, 2026
58b8a46 to
dd027db
Compare
Signed-off-by: Rui Conti <[email protected]>
dd027db to
4a925e0
Compare
De-ASI-INTERFACE
approved these changes
Jul 14, 2026
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.
Why
#749 records a production task that lost completed investigation state across compaction and continued for 442 model turns, 2,026 tool calls, and $59.0342 before cancellation. The free-form checkpoint prompt did not tell the compactor how to update an earlier checkpoint or keep completed work separate from pending work.
The first version of stacked test PR #755 required pi's exact headings. Its 60-call result measured formatting compliance, not semantic progress loss. After removing that requirement, base run 29303880312 passed the original six one-compaction cases on eve's existing prompt. Those results are retained as evidence that the earlier claim was invalid, not as proof for this change.
Review then exposed a separate repeated-compaction bug: eve stored its checkpoint as ordinary conversation messages. A later compaction could truncate the checkpoint to 280 characters or split its marker from its content and retain the stale checkpoint beside the replacement. Two focused regressions failed on both paths before the fix.
What changed
eve.On the final head, all six repeated-compaction cases passed in both the local run and the Vercel run. Each case observed exactly 2 compactions, 1 original work call, 1 checkpoint-advance call, and both checkpoint markers in the final response.
This PR is stacked on #755. Repeated-call containment remains a separate follow-up.
Addresses #749.