* Add unit tests for create conversation executor
* Update indentation and comment typo.
* Added unit tests for declarative executor SetMultipleVariablesExecutor
* Updated comments and syntactic sugar
* Add unit test for declarative executor RetrieveConversationMessageExecutor
* Removed irrelevant code statements
* Updated based on copilot feedback.
Fixes#2219
Adds default=str to json.dumps() calls to handle non-JSON-serializable
types like datetime objects in tool function results.
Co-authored-by: kishikawa-hayato <84244732+HerBest-max@users.noreply.github.com>
* Potential fix for code scanning alert no. 18: Information exposure through an exception
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
* Fix test
---------
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
* First working version
* Simplify the implementations
* Remove unused env var
* Update Python syntax
* Address feedbacks
* Fix a typo
* Update names as review suggestions
* Citation for self-reflection
* Move to independent folder
* Update python/samples/getting_started/evaluation/azure_ai_foundry/evaluation/README.md
Co-authored-by: Eduard van Valkenburg <eavanvalkenburg@users.noreply.github.com>
* Updated from parquet to JSONL and hide the default environment variables
* As review feedback, remove the purpose of using `run_self_reflection_batch` as a library, only use it as sample code
* Update python/samples/getting_started/evaluation/azure_ai_foundry/evaluation/self_reflection.py
Co-authored-by: Eduard van Valkenburg <eavanvalkenburg@users.noreply.github.com>
---------
Co-authored-by: Eduard van Valkenburg <eavanvalkenburg@users.noreply.github.com>
* first work on declarative
* initial version of the declarative support
* fix tests and mypy
* fix parameters of functiontool
* slight logic improvement
* remove path until merge
* updates from comments
* create dispatcher and spec type, json_schema method
* fix mypy, skipping model
* updated lock
* fixed declarative tests and renamed some other test files
* refined loader
* updated lock
* fix mypy
* added readme to samples folder
* fixes from review
* undid test file rename
Eduard van Valkenburg
·
2025-11-19 16:33:02 +00:00
* fix: resolve string annotations in FunctionExecutor
Enhance type hint validation in FunctionExecutor by importing `typing` and
using `get_type_hints` to correctly resolve annotations.
This fixes validation failures when `from __future__ import annotations`
is enabled, which stores annotations as strings.
Fixes#1808
* Update python/packages/core/tests/workflow/test_function_executor_future.py
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* ran pre commit
---------
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Fix bug where ChatAgent system instructions were not captured in Langfuse
traces due to incorrect attribute access.
The observability code was attempting to retrieve instructions using
getattr(self, "instructions", None), but ChatAgent stores instructions
in self.chat_options.instructions. This caused system_instructions to
always be None in Langfuse traces.
Changed both _trace_agent_run and _trace_agent_run_stream functions
to correctly retrieve instructions from chat_options.instructions.
Fixes affect:
- Line 1123: _trace_agent_run (non-streaming)
- Line 1192: _trace_agent_run_stream (streaming)
Update XML documentation to clarify exception behavior.
See `ChatClientAgentThreadTests.SetConversationIdThrowsWhenMessageStoreIsSet` which already verifies this is the actual behavior.
Co-authored-by: Chris <66376200+crickman@users.noreply.github.com>