Skip to content

Conversation

@kzu
Copy link
Member

@kzu kzu commented Oct 14, 2025

We already support those properties via extensions in our Extensions.AI project, but agent was not setting them properly.

This is otherwise quite cumbersome to set (see microsoft/agent-framework#1455).

We also turn on configuration binder source generation and review the emitted warnings for properties that wouldn't be deserialized (since they can't be easily mapped) and ignore the warnings/errors accordingly. We might want to offer custom converters for them in the future.

@kzu kzu added the enhancement New feature or request label Oct 14, 2025
@kzu kzu enabled auto-merge (rebase) October 14, 2025 20:00
@kzu
Copy link
Member Author

kzu commented Oct 14, 2025

16 passed 16 passed 17 skipped

🧪 Details on Ubuntu 24.04.3 LTS

from dotnet-retest v1.0.0 on .NET 9.0.10 with 💜 by @devlooped

@kzu kzu disabled auto-merge October 14, 2025 20:01
We already support those properties via extensions in our Extensions.AI project, but agent was not setting them properly.

This is otherwise quite cumbersome to set (see microsoft/agent-framework#1455).

We also turn on configuration binder source generation and review the emitted warnings for properties that wouldn't be deserialized (since they can't be easily mapped) and ignore the warnings/errors accordingly. We might want to offer custom converters for them in the future.
@kzu kzu enabled auto-merge (rebase) October 14, 2025 20:04
@kzu kzu merged commit faf6d74 into main Oct 14, 2025
7 checks passed
@kzu kzu deleted the dev/binder branch October 14, 2025 20:06
@devlooped devlooped locked and limited conversation to collaborators Nov 14, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants