Skip to content

Ingest node_repl stderr telemetry spans#19878

Draft
charley-openai wants to merge 1 commit intomainfrom
dev/charley/node-repl-stderr-otel-spans
Draft

Ingest node_repl stderr telemetry spans#19878
charley-openai wants to merge 1 commit intomainfrom
dev/charley/node-repl-stderr-otel-spans

Conversation

@charley-openai
Copy link
Copy Markdown

@charley-openai charley-openai commented Apr 27, 2026

Summary

  • add node_repl/js MCP _meta trace context for stderr span telemetry
  • parse @codex-telemetry stderr lines from local stdio MCP servers and route node_repl telemetry into OTel
  • reconstruct sanitized Browser Use/node_repl spans with explicit trace/span/parent ids so emitted child spans nest under node_repl.js
  • keep traceparent fallback for older records and add js_repl tool-call span coverage

Tests

  • cargo test -p codex-otel stderr_span_telemetry
  • cargo test -p codex-rmcp-client stderr_telemetry_parser
  • cargo test -p codex-core mcp_tool_call
  • cargo test -p codex-core otel_stderr_spans
  • cargo test -p codex-mcp
  • just fix -p codex-otel
  • just fix -p codex-rmcp-client -p codex-mcp -p codex-core
  • just fmt
  • git diff --check

@chatgpt-codex-connector
Copy link
Copy Markdown
Contributor

Summary

Testing

  • just fmt
  • ⚠️ cargo test -p codex-otel stderr_span_telemetry (environment limitation: crates.io download blocked with CONNECT tunnel failed, response 403)
  • ⚠️ cargo test -p codex-otel stderr_span_telemetry --offline (environment limitation: required crate rustls-webpki not available locally)
  • git diff --check

View task →

@charley-openai charley-openai force-pushed the dev/charley/node-repl-stderr-otel-spans branch from fcaaec5 to 963fcb8 Compare April 27, 2026 21:08
Bojun-Vvibe added a commit to Bojun-Vvibe/oss-contributions that referenced this pull request Apr 27, 2026
@charley-openai charley-openai force-pushed the dev/charley/node-repl-stderr-otel-spans branch from 963fcb8 to 5d0fdbc Compare April 28, 2026 00:35
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.

1 participant