Skip to content

fix(tracing): capture Codex MCP tool calls - #105

Open
jstar0 wants to merge 1 commit into
langfuse:mainfrom
jstar0:codex/langfuse-codex-mcp-call-tracing
Open

jstar0 wants to merge 1 commit into
langfuse:mainfrom
jstar0:codex/langfuse-codex-mcp-call-tracing

Conversation

@jstar0

@jstar0 jstar0 commented Sep 26, 2026

Copy link
Copy Markdown

Summary

  • Capture Codex MCP tool calls emitted as lifecycle items so nested Code Mode calls create named tracing observations.
  • Merge matching model-visible calls by call ID to prevent duplicates.
  • Preserve arguments, results, errors, and call timing.

Verification

  • pnpm test (122 tests)
  • pnpm run lint

Related: langfuse/langfuse#17969

Signed-off-by: King Star <mcxin.y@gmail.com>

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

@CLAassistant

CLAassistant commented Sep 26, 2026 •

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

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.

2 participants