Python: remove function_invocation_configuration from BaseChatClient.as_agent

[Andrew Barnes (Bortlesboat)]

Motivation and Context

Fixes #5180.

BaseChatClient.as_agent() currently accepts function_invocation_configuration but forwards it into Agent(**agent_kwargs), where Agent.__init__() does not accept that keyword. In practice this raises a TypeError instead of applying the configuration to function-invoking clients.

Description

This change keeps the override on the client side, which is where FunctionInvocationLayer consumes it:

• stop passing function_invocation_configuration into Agent()
• when the client supports FunctionInvocationLayer, normalize and apply the override directly to self.function_invocation_configuration
• add a regression test covering as_agent(function_invocation_configuration=...) so the issue stays fixed

Contribution Checklist

• The code builds clean without any errors or warnings
• The PR follows the Contribution Guidelines
• All unit tests pass, and I have added new tests where possible
• Is this a breaking change? If yes, add "[BREAKING]" prefix to the title of the PR.

[Eduard van Valkenburg (eavanvalkenburg)]

just added a note on the original issue, that this is not the way to fix that issue, if you are up for it, you can change this PR, open a new PR or we can pick it up with the team

[Andrew Barnes (Bortlesboat)]

Rescoped this draft to match the issue note: the unction_invocation_configuration kwarg is removed from BaseChatClient.as_agent() instead of being applied there. Verified with python -m pytest python/packages/core/tests/core/test_clients.py -q.

[Evan Mattson (moonbox3)]

Python Test Coverage Report •

File	Stmts	Miss	Cover	Missing
packages/core/agent_framework
_clients.py	135	6	95%	324, 375, 531–534
TOTAL	27198	3183	88%

Python Unit Test Overview

Tests	Skipped	Failures	Errors	Time
5440	20 💤	0 ❌	0 🔥	1m 30s ⏱️