Skip to content

[DERCBOT-1746] Langfuse V3 Callback Fixes#1954

Merged
assouktim merged 1 commit intomasterfrom
upgrade/langfusev3/DERCBOT-1746
Feb 3, 2026
Merged

[DERCBOT-1746] Langfuse V3 Callback Fixes#1954
assouktim merged 1 commit intomasterfrom
upgrade/langfusev3/DERCBOT-1746

Conversation

@scezen
Copy link
Member

@scezen scezen commented Dec 2, 2025

Ticket : DERCBOT-1746

Details

  • Migrated from deprecated langfuse.callback API to the new langfuse.langchain module.
  • Updated the LangfuseCallbackHandlerFactory to align with the new Langfuse V3 client.
  • Added internal caching for the Langfuse client to avoid repeated initializations.
  • Updated the observability service to use the new get_client() API and revised trace extraction logic.
  • Adjusted the RAG chain execution to pass metadata (user, session, tags) using the new supported mechanism.
  • Cleaned up unused parameters and removed unsupported kwargs that Langfuse no longer handles.

@scezen scezen marked this pull request as ready for review December 2, 2025 10:19
Copy link
Member

@Benvii Benvii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks small issue in get_observability_info.

@assouktim assouktim force-pushed the upgrade/langfusev3/DERCBOT-1746 branch from b92135b to 960a12d Compare December 17, 2025 16:57
@assouktim assouktim force-pushed the upgrade/langfusev3/DERCBOT-1746 branch from 8174cad to 003eccb Compare January 30, 2026 08:30
@vsct-jburet vsct-jburet self-requested a review January 30, 2026 13:07
@assouktim assouktim marked this pull request as ready for review February 3, 2026 09:10
@assouktim
Copy link
Contributor

Thanks a lot for these changes.

@assouktim assouktim merged commit f3b5f18 into master Feb 3, 2026
7 checks passed
@zigzago zigzago added this to the 25.10.5 milestone Feb 3, 2026
@zigzago zigzago added the bug label Feb 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants