Skip to content

feat(client): capture conversation content by default - #213

Open
ccschmitz-launchdarkly wants to merge 1 commit into
ccschmitz/offline-eval-conversation-spans-17b3from
ccschmitz/capture-content-default-on-17b3
Open

ccschmitz-launchdarkly wants to merge 1 commit into
ccschmitz/offline-eval-conversation-spans-17b3from
ccschmitz/capture-content-default-on-17b3

Conversation

@ccschmitz-launchdarkly

Copy link
Copy Markdown
Contributor

Summary

Follow-up to #170. That PR stays capture-off. This branch starts from the current head of ccschmitz/offline-eval-conversation-spans-17b3, so the diff is only the default change.

SDKOptions.capture_content now defaults to true. Online handler spans and offline evaluation spans use that same gate. An explicit false on configure, a handler factory, or a runner-written span still strips prompts, output, tool payloads, and judge explanations, including on offline evaluation spans. The $ld:ai:offline-evals:* events still carry no transcript.

Test plan

  • uv run pytest — 3340 passed, 17 skipped
  • uv run mypy packages/*/src — 103 source files, no issues
  • uv run ruff check . and uv run ruff format --check . — clean
Open in Web Open in Cursor 

SDKOptions.capture_content now defaults to true for online and offline spans.
An explicit false still strips the transcript everywhere, including offline
evaluation spans.

Co-authored-by: christopher.schmitz <christopher.schmitz@launchdarkly.com>
@ccschmitz-launchdarkly
ccschmitz-launchdarkly requested a review from a team as a code owner October 23, 2026 19:32
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