-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Add unsupported tools validation and new content types to evaluators validators #45175
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This pull request enhances the Azure AI Evaluation SDK's validator infrastructure to support new content types and adds validation for unsupported tools in agentic evaluators. The changes enable the SDK to handle emerging tool call patterns (function calls, MCP approval workflows, OpenAPI calls) while preventing usage of known unsupported tools.
Changes:
- Added six new content types to support function calls, MCP approval requests/responses, and OpenAPI calls with their outputs
- Implemented unsupported tools validation via
check_for_unsupported_toolsflag in conversation validators - Fixed typo in task navigation efficiency evaluator docstring
- Relaxed validation for tool result fields from requiring dictionary type to only requiring field existence
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| _validation_constants.py | Added six new ContentType enum values for function_call, function_call_output, mcp_approval_request, mcp_approval_response, openapi_call, and openapi_call_output |
| _conversation_validator.py | Added UNSUPPORTED_TOOLS list, check_for_unsupported_tools parameter, _validate_field_exists method, and validation logic for new content types |
| _tool_definitions_validator.py | Added check_for_unsupported_tools parameter and propagated it to parent class |
| _tool_calls_validator.py | Added check_for_unsupported_tools parameter and propagated it to parent class |
| _tool_output_utilization.py | Enabled unsupported tools validation by setting check_for_unsupported_tools=True |
| _tool_input_accuracy.py | Enabled unsupported tools validation by setting check_for_unsupported_tools=True |
| _tool_call_success.py | Enabled unsupported tools validation by setting check_for_unsupported_tools=True |
| _tool_call_accuracy.py | Enabled unsupported tools validation by setting check_for_unsupported_tools=True |
| _groundedness.py | Enabled unsupported tools validation by setting check_for_unsupported_tools=True |
| _task_navigation_efficiency.py | Fixed typo in docstring (removed extra quote before "call_tool_A") |
Description
All SDK Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines