fix: tolerate context-likes without user_content or session in record_agent_invocation

Use getattr for ctx.user_content and ctx.session.events when calling
_record_agent_metrics so that test doubles, partial migrations, and
external embedders that pass minimal InvocationContext-like objects
don't crash with AttributeError on the metrics call.

Change-Id: I7d54252dd222f3796e6ccdc13d4844a177b65e7d

Author: GWeale

src/google/adk/telemetry/_instrumentation.py

@@ -114,8 +114,8 @@ async def record_agent_invocation(
     _record_agent_metrics(
         agent.name,
         elapsed_ms,
-        ctx.user_content,
-        ctx.session.events,
+        getattr(ctx, "user_content", None),
+        getattr(getattr(ctx, "session", None), "events", []),
         caught_error,
     )
 

tests/unittests/telemetry/test_instrumentation.py

@@ -19,6 +19,7 @@
 
 from google.adk.telemetry import _instrumentation
 from opentelemetry import trace
+import pytest
 
 
 def test_get_elapsed_ms_span_none():
@@ -80,3 +81,37 @@ def test_get_elapsed_ms_span_non_int_end():
   with mock.patch("time.monotonic", return_value=12.0):
     elapsed = _instrumentation._get_elapsed_ms(mock_span, start_time)
   assert elapsed == 2000.0
+
+
+@pytest.mark.asyncio
+async def test_record_agent_invocation_tolerates_minimal_context():
+  """Tolerates context-likes that lack user_content or session.
+
+  Test doubles, partial migrations, and external embedders can pass an
+  InvocationContext-like object without `user_content` or with a `session`
+  that has no `events` attribute. The telemetry path must not raise
+  AttributeError on the metrics call in those cases.
+  """
+  agent = mock.MagicMock()
+  agent.name = "test_agent"
+  # Bare object without `user_content` and without `session`.
+  bare_ctx = object()
+
+  with (
+      mock.patch.object(
+          _instrumentation, "_record_agent_metrics"
+      ) as mock_record,
+      mock.patch.object(_instrumentation, "tracing") as mock_tracing,
+  ):
+    mock_tracing.tracer.start_as_current_span.return_value.__enter__.return_value = mock.MagicMock(
+        spec=trace.Span
+    )
+    async with _instrumentation.record_agent_invocation(bare_ctx, agent):
+      pass
+
+  mock_record.assert_called_once()
+  call_args = mock_record.call_args
+  # positional: (agent_name, elapsed_ms, user_content, events, caught_error)
+  assert call_args.args[0] == "test_agent"
+  assert call_args.args[2] is None  # user_content default
+  assert call_args.args[3] == []  # events default