Repository navigation
Python: Simplify Python memory sample to use session state - #4085
Merged
Eduard van Valkenburg (eavanvalkenburg) merged 1 commit intoFeb 19, 2026
Conversation
- Rename UserNameProvider → UserMemoryProvider - Use session state (state dict) instead of instance variables - Use context.extend_instructions() instead of context.instructions.append() - Use DEFAULT_SOURCE_ID class attribute - Fix imports to use public agent_framework API - Add session state inspection at end of sample Co-authored-by: Copilot <[email protected]>
Eduard van Valkenburg (eavanvalkenburg)
added
the
python
github-actions
Bot
changed the title
Copilot started reviewing on behalf of
Eduard van Valkenburg (eavanvalkenburg)
February 19, 2026 14:15
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the Python getting-started memory sample (04_memory.py) to demonstrate persisting user info via AgentSession.state using a context provider, aligning the sample with newer session-state guidance.
Changes:
- Switches imports to the public
agent_frameworkAPI surface. - Reworks the provider to store/read the user name from provider-scoped session state and uses
SessionContext.extend_instructions(). - Adds a final print showing how to inspect stored provider state from the session.
Eduard van Valkenburg (eavanvalkenburg)
enabled auto-merge
February 19, 2026 14:23
Mark Wallace (markwallace-microsoft)
approved these changes
Feb 19, 2026
westey (westey-m)
approved these changes
Feb 19, 2026
Eduard van Valkenburg (eavanvalkenburg)
deleted the
docs/update-memory-sample
branch
June 30, 2026 09:01
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.
Simplifies the
04_memory.pyget-started sample:UserNameProvider→UserMemoryProviderfor claritystatedict) instead of instance variables for persistencecontext.extend_instructions()instead ofcontext.instructions.append()DEFAULT_SOURCE_IDclass attribute (proper pattern)agent_frameworkAPIThis aligns the sample with the updated docs in MicrosoftDocs/semantic-kernel-pr#880.