Fix GetCallerBaggagePairs: resolve userId across all channels

userId was only set from aad_object_id, which is None on non-Teams
channels and A2A calls. Add fallback chain: aad_object_id → agentic_user_id → frm.id

Port of microsoft/Agent365-dotnet#246

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: jsl517

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

@@ -36,7 +36,10 @@ def get_caller_pairs(activity: Activity) -> Iterator[tuple[str, Any]]:
     frm = activity.from_property
     if not frm:
         return
-    yield USER_ID_KEY, frm.aad_object_id
+    # Fallback chain for user_id: AadObjectId → AgenticUserId → From.Id
+    # AadObjectId is null on non-Teams channels and A2A calls (see dotnet PR #246)
+    user_id = frm.aad_object_id or frm.agentic_user_id or frm.id
+    yield USER_ID_KEY, user_id
     yield USER_NAME_KEY, frm.name
     yield USER_EMAIL_KEY, frm.agentic_user_id
 

tests/observability/hosting/scope_helpers/test_scope_helper_utils.py

@@ -83,6 +83,53 @@ def test_get_channel_pairs():
     assert (CHANNEL_LINK_KEY, None) in result
 
 
+def test_get_caller_pairs_non_teams_fallback_to_from_id():
+    """Test userId falls back to from.id when aad_object_id is None (non-Teams channel)."""
+    from_account = ChannelAccount(
+        id="from-id-123",
+        name="Non-Teams User",
+    )
+    activity = Activity(type="message", from_property=from_account)
+
+    result = list(get_caller_pairs(activity))
+
+    assert (USER_ID_KEY, "from-id-123") in result
+    assert (USER_NAME_KEY, "Non-Teams User") in result
+    assert (USER_EMAIL_KEY, None) in result
+
+
+def test_get_caller_pairs_a2a_fallback_to_agentic_user_id():
+    """Test userId falls back to agentic_user_id for A2A calls (no aad_object_id)."""
+    from_account = ChannelAccount(
+        id="from-id-456",
+        name="Agent Caller",
+        agentic_user_id="a2a-agent-guid",
+    )
+    activity = Activity(type="message", from_property=from_account)
+
+    result = list(get_caller_pairs(activity))
+
+    assert (USER_ID_KEY, "a2a-agent-guid") in result
+    assert (USER_EMAIL_KEY, "a2a-agent-guid") in result
+
+
+def test_get_caller_pairs_aad_object_id_wins_when_all_set():
+    """Test aad_object_id takes precedence when all identifiers are present."""
+    from_account = ChannelAccount(
+        id="from-id-789",
+        aad_object_id="aad-wins",
+        name="Full User",
+        agentic_user_id="agent-upn",
+    )
+    activity = Activity(type="message", from_property=from_account)
+
+    result = list(get_caller_pairs(activity))
+
+    assert (USER_ID_KEY, "aad-wins") in result
+    assert (USER_NAME_KEY, "Full User") in result
+    assert (USER_EMAIL_KEY, "agent-upn") in result
+
+
 def test_get_conversation_pairs():
     """Test get_conversation_pairs extracts conversation information."""
     conversation = ConversationAccount(id="conversation-123")