Fix(lesson 02): Replaced deprecated AgentThread with AgentSession for context management#588
Open
marietta-a wants to merge 2 commits into
Open
Fix(lesson 02): Replaced deprecated AgentThread with AgentSession for context management#588marietta-a wants to merge 2 commits into
marietta-a wants to merge 2 commits into
Conversation
Contributor
|
👋 Thanks for contributing @marietta-a! We will review the pull request and get back to you soon. |
Contributor
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 .NET agentic framework code sample to use the newer agent/session APIs instead of the older agent/thread pattern for maintaining conversational context.
Changes:
- Replaced
CreateAIAgent(...)withAsAIAgent(...)in the chat client pipeline. - Replaced
AgentThreadusage with an async-createdAgentSession. - Updated context-management comments to refer to sessions (partially).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
108
to
111
| // Execute Agent: First Travel Planning Request | ||
| // Run the agent with an initial request that will likely trigger the random destination tool | ||
| // The agent will analyze the request, use the GetRandomDestination tool, and create an itinerary | ||
| // Using the thread parameter maintains conversation context for subsequent interactions |
Comment on lines
121
to
123
| // Demonstrate contextual conversation by referencing the previous response | ||
| // The agent remembers the previous destination suggestion and will provide an alternative | ||
| // This showcases the power of conversation threads and contextual understanding in .NET agents |
marietta-a
changed the title
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.
Description
Fixes
CS1061andCS0246compilation errors in 02-dotnet-agent-framework.cs by updating to the correct agent API methods and types.Addresses: #587
Root Cause
During the creation of an agent with outdated methods, the code generated:
.CreateAIAgent(...)is not defined forIChatClient. The supported method is.AsAIAgent(...).AgentThreadis not part of the current framework; the correct type isAgentSession.GetNewThread()has been replaced byCreateSessionAsync()for session managementReference: Multi-turn Conversations, C#
Changes
.CreateAIAgent(...)with.AsAIAgent(...)AgentThreadwithAgentSession.GetNewThread()with .CreateSessionAsync()Testing
Verified code compiles and runs without errors on: