Repository navigation
Conversation
This was referenced Oct 7, 2026
eli-r-ph
force-pushed
the
v1-capture-ai-wrappers
branch
from
October 7, 2026 03:56
fde9c61 to
31c13ba
Compare
eli-r-ph
force-pushed
the
v1-capture-harness
branch
from
October 7, 2026 17:43
f2faf4d to
743d8ca
Compare
eli-r-ph
force-pushed
the
v1-capture-ai-wrappers
branch
from
October 7, 2026 17:43
31c13ba to
17eeb3b
Compare
Contributor
Author
|
@greptileai review |
eli-r-ph
marked this pull request as ready for review
October 7, 2026 20:44
dustinbyrne
approved these changes
Oct 8, 2026
eli-r-ph
force-pushed
the
v1-capture-harness
branch
from
October 8, 2026 16:25
743d8ca to
ca9c26c
Compare
eli-r-ph
force-pushed
the
v1-capture-ai-wrappers
branch
from
October 8, 2026 16:25
17eeb3b to
054e194
Compare
eli-r-ph
force-pushed
the
v1-capture-harness
branch
from
October 8, 2026 18:13
ca9c26c to
e743d1e
Compare
eli-r-ph
force-pushed
the
v1-capture-ai-wrappers
branch
from
October 8, 2026 18:13
054e194 to
d909586
Compare
eli-r-ph
force-pushed
the
v1-capture-harness
branch
from
October 8, 2026 21:12
e743d1e to
78973e7
Compare
eli-r-ph
force-pushed
the
v1-capture-ai-wrappers
branch
2 times, most recently
from
October 8, 2026 22:54
19134a6 to
d569aca
Compare
eli-r-ph
force-pushed
the
v1-capture-harness
branch
from
October 8, 2026 22:54
78973e7 to
9dcf4df
Compare
eli-r-ph
force-pushed
the
v1-capture-ai-wrappers
branch
from
October 8, 2026 23:40
d569aca to
a473b99
Compare
eli-r-ph
force-pushed
the
v1-capture-harness
branch
from
October 8, 2026 23:40
9dcf4df to
d31608d
Compare
eli-r-ph
force-pushed
the
v1-capture-ai-wrappers
branch
from
October 10, 2026 23:35
a473b99 to
fd66496
Compare
eli-r-ph
force-pushed
the
v1-capture-harness
branch
from
October 10, 2026 23:35
d31608d to
5c2b03e
Compare
enable_full_ai_capture now controls only truncation and media redaction. A client object without capture_ai still gets capture calls.
eli-r-ph
force-pushed
the
v1-capture-ai-wrappers
branch
from
October 11, 2026 20:17
fd66496 to
dde4812
Compare
eli-r-ph
force-pushed
the
v1-capture-harness
branch
from
October 11, 2026 20:17
5c2b03e to
41336ea
Compare
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
💡 Motivation and Context
Stacked on #1026. AI wrapper events went to the analytics endpoint unless
enable_full_ai_capturewas set, and that endpoint caps an event near 1 MB. The AI endpoint accepts up to 8 MiB, but reaching it also meant turning off truncation and media redaction. This PR always sends wrapper events throughcapture_ai, andenable_full_ai_capturenow controls content only.Changes
capture_ai, through one helper.capture_aistill receivescapturecalls._use_ai_laneand_enable_multimodal_capturealiases keep working.capture, because they are not$ai_*events.Warning
mock.capturefor a wrapper now seemock.capture_ai./i/v1/ai/eventsnow drops wrapper events.This differs from the sdk-specs
capture-aispec, which ties routing to the flag; a spec change is still to be proposed.💚 How did you test it?
capture_aiwith the flag off.🤖 Agent context
Autonomy: Human-driven (agent-assisted)
Written with Cursor (Claude Opus) under the direction of the assignee.
🤖 Generated with Claude Code