Zhichao Li a4a3e6c1bd
otel: address PR review feedback on session-id correlation
- Rewrite BYOK session-id tests to exercise the real Anthropic/Gemini
  providers with a capturing OTel service and a correlated CapturingToken,
  instead of reproducing the attribute spread (which could not catch a
  production regression). Removes the synthetic tests from
  byokProviderSpans.spec.ts.
- agent_monitoring.md: add copilot_chat.session_id rows to the foreground
  execute_tool, Claude execute_tool and execute_hook span tables (the code
  emits it); mark chat-span gen_ai.conversation.id as conditional rather
  than required.
- agent_monitoring_arch.md: narrow the Session Correlation claim — the
  bridge only enriches the debug-panel copy, not the SDK's own OTLP export;
  document that CLI SDK-native OTLP correlation depends on the runtime
  (execute_tool covered once copilot-agent-runtime#12383 lands).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
2026-07-09 16:37:18 -07:00
..