Repository navigation
feat: add an AI capture lane to AsyncPosthog - #1028
Merged
Merged
Conversation
3 of 20 tasks
eli-r-ph
force-pushed
the
v1-capture-async-ai-lane
branch
from
October 7, 2026 03:56
d5eb0bc to
ae647de
Compare
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-async-ai-lane
branch
from
October 7, 2026 17:43
42c1d07 to
691c8bd
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
dustinbyrne
left a comment
Contributor
There was a problem hiding this comment.
The async lane and both-lane lifecycle handling look consistent with the sync implementation. One minor, non-blocking malformed-input exception-boundary issue is noted inline. Consider ensuring the stack’s spec update and migration guide explicitly cover default async-wrapper routing to the AI endpoint.
AI-assisted review.
eli-r-ph
force-pushed
the
v1-capture-async-ai-lane
branch
from
October 8, 2026 16:25
691c8bd to
1f784bb
Compare
eli-r-ph
force-pushed
the
v1-capture-ai-wrappers
branch
2 times, most recently
from
October 8, 2026 18:13
054e194 to
d909586
Compare
eli-r-ph
force-pushed
the
v1-capture-async-ai-lane
branch
2 times, most recently
from
October 8, 2026 21:12
04507c0 to
adf5b16
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-async-ai-lane
branch
from
October 8, 2026 22:54
adf5b16 to
0cfa23c
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-async-ai-lane
branch
from
October 8, 2026 23:40
0cfa23c to
252b30d
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-async-ai-lane
branch
from
October 10, 2026 23:35
252b30d to
6e1a501
Compare
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-async-ai-lane
branch
from
October 11, 2026 20:17
6e1a501 to
2765967
Compare
eli-r-ph
force-pushed
the
v1-capture-ai-wrappers
branch
from
October 14, 2026 03:11
dde4812 to
d860568
Compare
eli-r-ph
force-pushed
the
v1-capture-async-ai-lane
branch
from
October 14, 2026 03:11
2765967 to
4cd3a28
Compare
eli-r-ph
force-pushed
the
v1-capture-ai-wrappers
branch
from
October 14, 2026 03:47
d860568 to
8232570
Compare
eli-r-ph
force-pushed
the
v1-capture-async-ai-lane
branch
from
October 14, 2026 03:47
4cd3a28 to
66b1e17
Compare
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 #1027. The sync
Clientsends$ai_*events on a separate AI lane, butAsyncPosthoghas only the analytics lane. This PR givesAsyncPosthogthe same AI lane, with the same options and defaults.Changes
AsyncPosthog.capture_ai(queued) andcapture_ai_immediate(awaits delivery), both posting to/i/v1/ai/events.capture_ai_compression,capture_ai_max_queue_size(1000),capture_ai_timeout(30 s) andcapture_ai_max_event_bytes(8 MiB, can only be lowered).capture_aicall.flushandshutdowndrain both lanes.privacy_modeandenable_full_ai_captureoptions, read by the AI wrappers as onClient.With #1027, a wrapper given an
AsyncPosthogclient now sends to the AI endpoint.💚 How did you test it?
flushandshutdowndeliver both lanes.🤖 Agent context
Autonomy: Human-driven (agent-assisted)
Written with Cursor (Claude Opus) under the direction of the assignee.
🤖 Generated with Claude Code