Use MiddlewareSet type for adapter and Activity helper methods instead of direct attribute access

- Change adapter param type from ChannelAdapter to MiddlewareSet (the actual
  middleware registration object)
- Use activity.get_agentic_instance_id() instead of recipient.agentic_app_id
- Use activity.get_agentic_user() instead of recipient.agentic_user_id
- Update test to use agentic role for proper helper method behavior

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

Author: Copilot

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

@@ -8,7 +8,7 @@
 import logging
 from dataclasses import dataclass
 
-from microsoft_agents.hosting.core import ChannelAdapter
+from microsoft_agents.hosting.core.middleware_set import MiddlewareSet
 
 from .baggage_middleware import BaggageMiddleware
 from .output_logging_middleware import OutputLoggingMiddleware
@@ -46,15 +46,15 @@ def __init__(self) -> None:
     @classmethod
     def configure(
         cls,
-        adapter: ChannelAdapter,
+        adapter: MiddlewareSet,
         options: ObservabilityHostingOptions,
     ) -> ObservabilityHostingManager:
         """Configure the singleton instance and register middleware on the adapter.
 
         Subsequent calls after the first are no-ops and return the existing instance.
 
         Args:
-            adapter: The channel adapter to register middleware on.
+            adapter: The middleware set to register middleware on.
             options: Configuration options controlling which middleware to enable.
 
         Returns:

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

@@ -43,14 +43,15 @@
 
 def _derive_agent_details(context: TurnContext) -> AgentDetails | None:
     """Derive target agent details from the activity recipient."""
-    recipient = getattr(context.activity, "recipient", None)
+    activity = context.activity
+    recipient = getattr(activity, "recipient", None)
     if not recipient:
         return None
     return AgentDetails(
-        agent_id=getattr(recipient, "agentic_app_id", None) or "",
+        agent_id=activity.get_agentic_instance_id() or "",
         agent_name=getattr(recipient, "name", None),
         agent_auid=getattr(recipient, "aad_object_id", None),
-        agent_upn=getattr(recipient, "agentic_user_id", None),
+        agent_upn=activity.get_agentic_user(),
         agent_description=getattr(recipient, "role", None),
         tenant_id=getattr(recipient, "tenant_id", None),
     )

libraries/microsoft-agents-a365-observability-hosting/microsoft_agents_a365/observability/hosting/scope_helpers/utils.py

@@ -62,10 +62,10 @@ def get_target_agent_pairs(activity: Activity) -> Iterator[tuple[str, Any]]:
     rec = activity.recipient
     if not rec:
         return
-    yield GEN_AI_AGENT_ID_KEY, rec.agentic_app_id
+    yield GEN_AI_AGENT_ID_KEY, activity.get_agentic_instance_id()
     yield GEN_AI_AGENT_NAME_KEY, rec.name
     yield GEN_AI_AGENT_AUID_KEY, rec.aad_object_id
-    yield GEN_AI_AGENT_UPN_KEY, rec.agentic_user_id
+    yield GEN_AI_AGENT_UPN_KEY, activity.get_agentic_user()
     yield (
         GEN_AI_AGENT_DESCRIPTION_KEY,
         rec.role,

tests/observability/hosting/scope_helpers/test_scope_helper_utils.py

@@ -66,7 +66,7 @@ def test_get_target_agent_pairs():
         name="Test Agent",
         aad_object_id="agent-auid",
         agentic_user_id="agent-upn",
-        role="Assistant",
+        role="agenticAppInstance",
     )
     activity = Activity(type="message", recipient=recipient)
 
@@ -76,7 +76,7 @@ def test_get_target_agent_pairs():
     assert (GEN_AI_AGENT_NAME_KEY, "Test Agent") in result
     assert (GEN_AI_AGENT_AUID_KEY, "agent-auid") in result
     assert (GEN_AI_AGENT_UPN_KEY, "agent-upn") in result
-    assert (GEN_AI_AGENT_DESCRIPTION_KEY, "Assistant") in result
+    assert (GEN_AI_AGENT_DESCRIPTION_KEY, "agenticAppInstance") in result
 
 
 def test_get_tenant_id_pair():