Skip to content

Conversation

@fhoering
Copy link
Contributor

@fhoering fhoering commented Oct 23, 2025

Important

Add trace_context parameter to LangchainCallbackHandler to enable trace context injection for observations.

  • Behavior:
    • Add trace_context parameter to LangchainCallbackHandler constructor for injecting trace context.
    • Use trace_context in on_chain_start() to start observations with context when obs is an instance of Langfuse.
  • Imports:
    • Import TraceContext from langfuse.types.

This description was created by Ellipsis for 5982ac7. You can customize this summary. It will automatically update as commits are pushed.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Oct 23, 2025

Greptile encountered an error while reviewing this PR. Please reach out to support@greptile.com for assistance.

@hassiebp hassiebp changed the base branch from main to trace-context-callback-handler December 1, 2025 16:14
@hassiebp hassiebp merged commit da8e1be into langfuse:trace-context-callback-handler Dec 1, 2025
1 check was pending
@hassiebp
Copy link
Contributor

hassiebp commented Dec 1, 2025

Thanks a lot for your contribution @fhoering - will push this over the line in #1458

hassiebp added a commit that referenced this pull request Dec 1, 2025
…Handler (#1419) (#1458)

Be able to inject trace_context into LangchainCallbackHandler (#1419)

Co-authored-by: Fabian Höring <fhoring@gmail.com>
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.

3 participants