fix(eval): make _EvalMetricResultWithInvocation.expected_invocation optional

[giulio-leone]

Summary

Fixes #5214 — AgentEvaluator.evaluate() crashes with a pydantic ValidationError when evaluating conversation_scenario eval cases because _EvalMetricResultWithInvocation.expected_invocation is typed as required Invocation, while local_eval_service intentionally passes None for scenario-based cases.

Changes

src/google/adk/evaluation/agent_evaluator.py

1. Make expected_invocation optional (line 93):

   expected_invocation: Optional[Invocation] = None

   This aligns with the public EvalMetricResultPerInvocation model in eval_metrics.py:323.

2. Guard downstream attribute access in _print_details (lines 439–456):
   Added if per_invocation_result.expected_invocation else None guards for user_content, final_response, and intermediate_data accesses. Both _convert_content_to_text and _convert_tool_calls_to_text already handle None gracefully.

Tests

Added tests/unittests/evaluation/test_eval_metric_result_with_invocation.py with 5 regression tests:

• Construction with None / omitted / real expected_invocation
• _get_eval_metric_results_with_invocation propagates None
• _print_details does not crash with None expected invocation

All 5 tests pass. Existing eval tests unaffected (2 pre-existing failures in test_custom_metric_evaluator.py confirmed on main).

[google-cla]

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

[adk-bot]

Response from ADK Triaging Agent

Hello @giulio-leone, thank you for your contribution!

Before we can merge this pull request, you'll need to sign the Contributor License Agreement (CLA). You can do so by following the instructions in the "cla/google" check at the bottom of the pull request.

Thanks!

[rohityan]

Hi @giulio-leone, Thank you for your contribution! We appreciate you taking the time to submit this pull request.
Can you fix the formatting errors.

[rohityan]

Hi @ankursharmas , can you please review this.

[giulio-leone]

Rebased onto the current main and pushed a formatting-only update for the test file (tests/unittests/evaluation/test_eval_metric_result_with_invocation.py).

Local revalidation on the rebased branch:

• isort --check src/google/adk/evaluation/agent_evaluator.py tests/unittests/evaluation/test_eval_metric_result_with_invocation.py
• pyink --check --diff --config pyproject.toml on the same two files
• PYTHONPATH=$PWD/src python -m pytest tests/unittests/evaluation/test_eval_metric_result_with_invocation.py → 5 passed

The production fix remains unchanged; this update only brings the PR back in line with the repo formatting checks.