Skip to content

Enable GenAI message content capture by default - #195

Merged
rite2hhh merged 5 commits into
masterfrom
otel-capture-content-default
Aug 5, 2026
Merged

rite2hhh merged 5 commits into
masterfrom
otel-capture-content-default

Conversation

@rite2hhh

@rite2hhh rite2hhh commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Enable message content capture by default for OpenAI, Anthropic, Google, and Bedrock tracing
  • Preserve opt-out support through OTEL_INSTRUMENTATION_GENAI_CAPTURE_MESSAGE_CONTENT=false or captureContent: false
  • Update tracing documentation and examples, with test coverage for default and explicit settings

Testing

  • npm test
  • npm run lint
  • npm run build

Default all provider instrumentations to content capture unless users opt out.

Document the environment and configureTracing opt-out paths in examples.

Cover default and explicit capture settings with focused tracing tests.

@hasaan21 hasaan21 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Version update needed if previous SDK version is not already released

T3XRD and others added 4 commits August 5, 2026 13:08
## Summary
- always scaffold and populate the Trace column for evaluations
- remove scorer-dependent trace gating and the unused batch-row path
- remove obsolete trace-reference helpers and bump the SDK to 2.0.7

## Test plan
- [x] `npm test -- --run src/evals.test.ts
src/evaluations/scorers/scorers.test.ts src/evaluations/columns.test.ts`
- [x] `npm run lint`
- [x] `npm run build`

Made with [Cursor](https://cursor.com)

Co-authored-by: Cursor <cursoragent@cursor.com>
Update the package manifest and lockfile metadata for the next patch release.
@rite2hhh
rite2hhh merged commit e63ab23 into master Aug 5, 2026
5 checks passed
@rite2hhh
rite2hhh deleted the otel-capture-content-default branch August 5, 2026 18:48
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