Uh oh!
There was an error while loading. Please reload this page.
.NET: Clarify compaction provider and chat reducer choices - #7678
Conversation
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates the Step 18 compaction pipeline documentation to clarify where to apply compaction vs. reduction, and refreshes the sample index entry to match the expanded guidance.
Changes:
- Expanded Step 18 README with a comparison of
CompactionProvidervsIChatReducer, including adapter examples and guidance. - Updated the Agents sample index description for the compaction pipeline step to reflect the new guidance.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| dotnet/samples/02-agents/Agents/README.md | Updates the Step 18 row description to mention provider vs reducer integration. |
| dotnet/samples/02-agents/Agents/Agent_Step18_CompactionPipeline/README.md | Adds detailed guidance, a comparison table, and adapter examples for compaction vs reduction. |
💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
westey (westey-m)
commented
Aug 20, 2026
Ravi Kiran Pagidi (@ravikiranpagidi), thanks for contributing the additional docs! Much appreciated. |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Motivation & Context
CompactionProviderand Microsoft.Extensions.AI'sIChatReducercan both reduce the messages supplied to a model, but they run at different lifecycle points and have different persistence behavior. The existing compaction sample did not give users a clear way to choose between them or explain the two adapters, which led to confusion about whether they should be composed together.This documentation makes that decision explicit and addresses #4608 without changing runtime behavior or public API.
Description & Review Guide
CompactionProviderregistration withIChatReduceronInMemoryChatHistoryProvider; document reducer trigger events; show each supported adapter direction; and update the sample index so the guidance is discoverable.Related Issue
#4608
Contribution Checklist
breaking changelabel (or add "[BREAKING]" to the title prefix, before or after a language prefix such as.NET:) — workflows keep the label and title prefix in sync automatically.