Address review comments: fix logic callback type, gate on is_agentic_request

- Fix `logic` type annotation from `Callable[[TurnContext], Awaitable]` to
  `Callable[[], Awaitable]` in both BaggageMiddleware and
  OutputLoggingMiddleware. At runtime, MiddlewareSet passes a zero-arg
  `call_next_middleware` closure, so the annotation now matches the callsite.

- Gate `_derive_agent_details` on `activity.is_agentic_request()` to avoid
  emitting spans with empty agent_id for non-agentic requests.

- Remove unused `logging`/`logger` from baggage_middleware.py.

- Update test recipient role from "assistant" to "agenticAppInstance" to
  exercise the intended agentic code path.

Co-authored-by: nikhilNava <211831449+nikhilNava@users.noreply.github.com>

Author: Copilot

libraries/microsoft-agents-a365-observability-hosting/microsoft_agents_a365/observability/hosting/middleware/baggage_middleware.py

@@ -5,7 +5,6 @@
 
 from __future__ import annotations
 
-import logging
 from collections.abc import Awaitable, Callable
 
 from microsoft_agents.activity import ActivityEventNames, ActivityTypes
@@ -14,8 +13,6 @@
 
 from ..scope_helpers.populate_baggage import populate
 
-logger = logging.getLogger(__name__)
-
 
 class BaggageMiddleware:
     """Middleware that propagates OpenTelemetry baggage context derived from TurnContext.
@@ -26,7 +23,7 @@ class BaggageMiddleware:
     async def on_turn(
         self,
         context: TurnContext,
-        logic: Callable[[TurnContext], Awaitable],
+        logic: Callable[[], Awaitable],
     ) -> None:
         activity = context.activity
         is_async_reply = (

libraries/microsoft-agents-a365-observability-hosting/microsoft_agents_a365/observability/hosting/middleware/output_logging_middleware.py

@@ -42,8 +42,15 @@
 
 
 def _derive_agent_details(context: TurnContext) -> AgentDetails | None:
-    """Derive target agent details from the activity recipient."""
+    """Derive target agent details from the activity recipient.
+
+    Returns ``None`` when the activity is not an agentic request or the
+    recipient is missing, so callers can short-circuit without emitting
+    spans with empty identifiers.
+    """
     activity = context.activity
+    if not activity.is_agentic_request():
+        return None
     recipient = getattr(activity, "recipient", None)
     if not recipient:
         return None
@@ -119,7 +126,7 @@ class OutputLoggingMiddleware:
     async def on_turn(
         self,
         context: TurnContext,
-        logic: Callable[[TurnContext], Awaitable],
+        logic: Callable[[], Awaitable],
     ) -> None:
         agent_details = _derive_agent_details(context)
         tenant_details = _derive_tenant_details(context)

tests/observability/hosting/middleware/test_output_logging_middleware.py

@@ -36,7 +36,7 @@ def _make_turn_context(
         ),
         "recipient": ChannelAccount(
             tenant_id=recipient_tenant_id,
-            role="assistant",
+            role="agenticAppInstance",
             name="Agent One",
             agentic_app_id=recipient_agentic_app_id,
             aad_object_id="agent-auid",