Skip to content

.Net: Moved IChatHistoryReducer from Agents to SK packages - #10285

Merged
Dmytro Struk (dmytrostruk) merged 13 commits into
microsoft:mainfrom
dmytrostruk:chat-history-reducer
Jan 27, 2025
Merged

Dmytro Struk (dmytrostruk) merged 13 commits into
microsoft:mainfrom
dmytrostruk:chat-history-reducer

Conversation

@dmytrostruk

@dmytrostruk Dmytro Struk (dmytrostruk) commented Jan 24, 2025 •

Copy link
Copy Markdown
Member

Motivation and Context

Resolves: #10203

This PR:

  • Moves IChatHistoryReducer from Agents.Core to Microsoft.SemanticKernel.Abstractions package.
  • Moves implementations ChatHistorySummarizationReducer and ChatHistoryTruncationReducer from Agents.Core to Microsoft.SemanticKernel.Core package.

The functionality is marked as experimental.

Contribution Checklist

@dmytrostruk
Dmytro Struk (dmytrostruk) requested a review from a team as a code owner January 24, 2025 00:58
@markwallace-microsoft Mark Wallace (markwallace-microsoft) added .NET Issue or Pull requests regarding .NET code kernel Issues or pull requests impacting the core kernel kernel.core labels Jan 24, 2025
@dmytrostruk Dmytro Struk (dmytrostruk) changed the title .Net: Moved IChatHistoryReducer from Agents to SK Abstractions package .Net: Moved IChatHistoryReducer from Agents to SK packages Jan 24, 2025
@dmytrostruk
Dmytro Struk (dmytrostruk) added this pull request to the merge queue Jan 27, 2025
Merged via the queue into microsoft:main with commit b592a4a Jan 27, 2025
@dmytrostruk
Dmytro Struk (dmytrostruk) deleted the chat-history-reducer branch January 27, 2025 20:10
jcruzmot-te pushed a commit to thousandeyes/aia-semantic-kernel that referenced this pull request Sep 15, 2025
…#10285)

### Motivation and Context

<!-- Thank you for your contribution to the semantic-kernel repo!
Please help reviewers and future users, providing the following
information:
  1. Why is this change required?
  2. What problem does it solve?
  3. What scenario does it contribute to?
  4. If it fixes an open issue, please link to the issue here.
-->

Resolves: microsoft#10203

This PR:
- Moves `IChatHistoryReducer` from `Agents.Core` to
`Microsoft.SemanticKernel.Abstractions` package.
- Moves implementations `ChatHistorySummarizationReducer` and
`ChatHistoryTruncationReducer` from `Agents.Core` to
`Microsoft.SemanticKernel.Core` package.

The functionality is marked as experimental.

### Contribution Checklist

<!-- Before submitting this PR, please make sure: -->

- [x] The code builds clean without any errors or warnings
- [x] The PR follows the [SK Contribution
Guidelines](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
and the [pre-submission formatting
script](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md#development-scripts)
raises no violations
- [x] All unit tests pass, and I have added new tests where possible
- [x] I didn't break anyone 😄

This branch was previously deployed

1 inactive deployment
integration — 48a4eff1 Deployed Jan 27, 2025 by dmytrostruk
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kernel.core kernel Issues or pull requests impacting the core kernel .NET Issue or Pull requests regarding .NET code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

.Net: Chat History Reducer is possibly in the wrong spot

5 participants