* Initial plan
* Add serialization support for ContextualFunctionProvider
- Added constructor overload that accepts JsonElement serializedState
- Implemented Serialize method to persist recent messages
- Added ContextualFunctionProviderState class
- Registered state class with AgentJsonUtilities JsonContext
- Added comprehensive unit tests for serialization/deserialization
Co-authored-by: westey-m <164392973+westey-m@users.noreply.github.com>
* Fix constructor parameter ordering and make default more explicit
- Reordered constructor parameters to maintain consistency (options before jsonSerializerOptions)
- Made default JsonElement explicit with default(JsonElement) in constructor chaining
- All tests continue to pass
Co-authored-by: westey-m <164392973+westey-m@users.noreply.github.com>
* Improve XML documentation comments for clarity
- Updated jsonSerializerOptions parameter comments to clarify actual usage
- Made it clear that AgentJsonUtilities.DefaultOptions is used
- All tests continue to pass
Co-authored-by: westey-m <164392973+westey-m@users.noreply.github.com>
* Change NumberOfRecentMessagesInContext to 2 to demonstrate truncation
- Updated test to use limit of 2 messages instead of 3
- Now test shows truncation in action (3 messages added, only 2 kept)
- Updated assertion to expect 2 messages in serialized state
- All tests pass
Co-authored-by: westey-m <164392973+westey-m@users.noreply.github.com>
* Apply suggestion from @westey-m
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: westey-m <164392973+westey-m@users.noreply.github.com>
westey ·
Copilot ·