Repository navigation
fix(mcp): keep injected agent context on summaries - #1511
Closed
ScriptedAlchemy wants to merge 3 commits into
Closed
ScriptedAlchemy wants to merge 3 commits into
ScriptedAlchemy wants to merge 3 commits into
Conversation
The reused partition is a count and digest, not a chunk list, and demand admission now returns NotApplicable. Cover both so the scheduler builds. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
Status, skill view, and truncated-response recovery were putting raw payloads back into the prompt. Default those surfaces to summaries and enforce the session steering budget. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
|
…r/guard-context-window-5c20 Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
Owner
Author
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
3e6167b265639c17b7d1c174cd3b7baa3116adaeinto this branch (f88f92659f). Conflicts were only the compile-unblock copies of tip fixes; tip versions were kept.tracedecay_statusmatches the CLI compact contract.tracedecay_skill_viewreturns support-file summaries unlessinclude_support_files=true. Truncated responses no longer tell the model to rejoin the stored body. Session steering builders enforce their budgets.Does not merge #707 into master. Targets the #707 tip only.
Merge
3e6167b265639c17b7d1c174cd3b7baa3116adae(test(code-index): re-pin partitioned codec onto extractor.rust.v8)f88f92659f1c12f18b58aa64b43689da15cab072branch_publication.rs: kept the tip's typedNotApplicable→CODE_INDEX_IDENTITY_MISMATCH(non-retryable). Did not keep the earlier compile-only arm that treated it as queued.publication_store.rs: kept the tip's non-enumerated reuse comment (reused_count/reused_digest; per-chunk Reused rows areExtraChunkReceipt).Motivation
The context window is finite. Review of PR #707 through guard-the-context-window found agent-facing surfaces that either dump raw data into the main prompt or instruct the model to reassemble it.
Changes
tracedecay_statusdefaults the expensive sections off. Opt in for the full diagnostic list, storage health, session ingest, or Git staleness.tracedecay_skill_viewno longer inlines support-file bytes by default. Path andbyte_lensummaries stay.tracedecay_retrievetell the model to use the preview and request one omitted span, not concatenate every page.Findings not changed
tools/listregression ceiling is 640 KiB (measured ~627 KiB). Bounding the remaining canonical schemas is a separate deletion.tracedecay_readstill defaults tomode=full.include_subagentsto true.Test plan
hooks::steering(2),tools::render::tests(9), catalog always-load and summary-default tests,mcp_suitestatus_runtime_test(10) andmanaged_skill_mcp_tools_list_and_view_profile_store(1)tracedecay-code-index-runtimecompiled as a dependency of those runs after the conflict resolutioncargo nextestnot runChecklist
CHANGELOG.mdupdated (release-please owns this file; no Unreleased section).envfiles included