chore(context): remove stale/duplicated always-on context (validated deletion pass) - #106
Merged
Merged
Conversation
…ion-navigator T1: drop all <commentary> blocks from agents/graph-analyst.md (2 examples, commentary only) and agents/session-navigator.md (1 example, commentary only -- already well under the 2-example cap, so nothing is dropped there beyond commentary). Both files carry two separate top-level description fields in frontmatter -- one under a bundle: mapping (a short, one-line, no-examples summary used for bundle listings) and one under the meta: mapping (the real, description: | block-scalar one with the <example>/<commentary> content this item targets). Edited only the meta: description via git apply; verified the bundle: description was untouched and the meta: description now has commentary=0 (graph-analyst.md: examples=2, session-navigator.md: examples=1) via yaml.safe_load post-patch. Full module test suite green: modules/tool-context-intelligence-query/tests/ 195 passed, 0 failed. One test, test_tool_dependencies.py, references graph-analyst.md by name but asserts a module-entry-point-naming invariant unrelated to the <example>/<commentary> content this item touches -- confirmed it still passes. Validated by the 1e2 context-hygiene DTU A/B treatment: 12/12 runs task-correct on both providers at n=3/arm; wire audit confirmed every deleted string absent in all treatment runs and present in all controls; measured -5,860 chars/turn system prompt (-11.3%) and -1,277 chars/turn tool catalog (-4.5%) reduction across the full treatment set; no YAML corruption in mounted agent catalogs. (openai arm showed -45% cost but is noise-inflated at this sample size; the defensible claim is the wire-byte reduction.) 🤖 Generated with [Amplifier](https://github.com/microsoft/amplifier) Co-Authored-By: Amplifier <240397093+microsoft-amplifier@users.noreply.github.com>
Collaborator
Author
|
Merging with |
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.
What lands here
T1 (trim, from the 1e2 treatment MANIFEST): drop all
<commentary>blocks fromagents/graph-analyst.md(2 examples, commentary only) andagents/session-navigator.md(1 example, commentary only — already well under the 2-example cap, so nothing is dropped there beyond commentary).Both files carry two separate top-level
descriptionfields in frontmatter — one under abundle:mapping (a short, one-line, no-examples summary used for bundle listings) and one under themeta:mapping (the real,description: |block-scalar one with the<example>/<commentary>content this item targets). Edited only themeta:description viagit apply; verified thebundle:description was untouched and themeta:description now hascommentary=0(graph-analyst.md:examples=2,session-navigator.md:examples=1) viayaml.safe_loadpost-patch.Net: -479 chars / ~-129 tokens per turn for any session with this bundle mounted.
Test status
Full module test suite green:
modules/tool-context-intelligence-query/tests/195 passed, 0 failed. One test,test_tool_dependencies.py, referencesgraph-analyst.mdby name but asserts a module-entry-point-naming invariant unrelated to the<example>/<commentary>content this item touches — confirmed it still passes.Validation evidence (1e2 context-hygiene DTU A/B treatment)
This treatment PASSED its pre-registered DTU A/B gates: 12/12 runs task-correct on both providers at n=3/arm; wire audit confirmed every deleted string absent in all treatment runs and present in all controls; measured -5,860 chars/turn system prompt (-11.3%) and -1,277 chars/turn tool catalog (-4.5%) reduction across the full treatment set; no YAML corruption in mounted agent catalogs. The openai arm showed a -45% cost delta, but that is noise-inflated at this sample size — the defensible, evidence-backed claim is the wire-byte reduction, not the cost delta.
Source of truth for the full 12-repo treatment:
treatments/1e2/MANIFEST.md(per-repo files/items/tokens/risk table, recorded HEAD shas, APPLY.md/VERIFY.md per repo).🤖 Generated with Amplifier