Skip to content

[WIP] Emit langfuse.session.id and langfuse.user.id for views#33839

Closed
Copilot wants to merge 2 commits into
mainfrom
copilot/emit-langfuse-session-user-ids
Closed

[WIP] Emit langfuse.session.id and langfuse.user.id for views#33839
Copilot wants to merge 2 commits into
mainfrom
copilot/emit-langfuse-session-user-ids

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 21, 2026

  • Add langfuse.session.id and session.id in buildEpisodeAttributesFromContext
  • Add langfuse.user.id and user.id in sendJobSetupSpan
  • Add langfuse.user.id and user.id in sendJobConclusionSpan
  • Update emit_outcome_spans.cjs to emit all four attributes
  • Add unit tests for the four new attributes in send_otlp_span.test.cjs
  • Add unit tests for the four new attributes in emit_outcome_spans.test.cjs

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@pelikhan pelikhan closed this May 21, 2026
Copilot stopped work on behalf of pelikhan due to an error May 21, 2026 18:57
Copilot AI requested a review from pelikhan May 21, 2026 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Emit langfuse.session.id and langfuse.user.id (with session.id/user.id aliases) so Langfuse populates Sessions and Users views

2 participants