Repository navigation
feat(coding-agent): exclude plans from compaction - #251
Merged
Merged
Conversation
- Let the latest attached snapshot supersede retained plan history without changing persisted entries. - Exclude compaction and branch snapshots before tree summarization token budgeting.
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.
Summary
Exclude historical plan data from compaction and branch summarization while attaching the latest authoritative plan state afterward.
Remove plan data superseded by that snapshot from the resulting model context, including retained messages and earlier summary snapshots.
Reasoning
The plan is persisted separately, so summarizing its history adds duplicate tokens and can preserve obsolete checklists.
An attached snapshot should supersede earlier plan data when restoring context after compaction or tree navigation.
Keep the existing summarizer prompts and their goal, constraints and preferences, progress, key decisions, next steps, and critical context sections unchanged.
Changes
Testing
AgentSessionwith the faux provider before fixing them.npm run checkpassed../test.shpassed with local socket access and isolated credentials.