Repository navigation
Conversation
eli-r-ph
marked this pull request as ready for review
October 10, 2026 21:51
eli-r-ph
force-pushed
the
v1-ai-require-capture-ai
branch
from
October 10, 2026 23:35
dfa7894 to
585ce8b
Compare
eli-r-ph
force-pushed
the
v1-capture-api-fixes
branch
from
October 10, 2026 23:35
6982671 to
0c99129
Compare
eli-r-ph
force-pushed
the
v1-ai-require-capture-ai
branch
from
October 11, 2026 20:17
585ce8b to
9a89fd0
Compare
eli-r-ph
force-pushed
the
v1-capture-api-fixes
branch
from
October 11, 2026 20:17
0c99129 to
161a339
Compare
eli-r-ph
changed the title
dustinbyrne
reviewed
Oct 13, 2026
eli-r-ph
force-pushed
the
v1-ai-require-capture-ai
branch
from
October 14, 2026 03:11
9a89fd0 to
5ea139e
Compare
eli-r-ph
force-pushed
the
v1-capture-api-fixes
branch
from
October 14, 2026 03:11
161a339 to
5c57a86
Compare
Raising TypeError at construction could stop the host application over an analytics problem. The integration now logs one error at setup and captures no AI events; provider calls work and return normally. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
eli-r-ph
force-pushed
the
v1-capture-api-fixes
branch
from
October 14, 2026 03:48
5c57a86 to
d57b40f
Compare
eli-r-ph
force-pushed
the
v1-ai-require-capture-ai
branch
from
October 14, 2026 03:48
5ea139e to
81db481
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
2 participants
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 #1046. An AI integration given a client without
capture_aifell back tocapture. That skipped the AI endpoint and its 8 MiB limit, and the failure showed up late or silently, often after a paid provider call. This PR reports it at setup instead, without stopping the app.Changes
capture_ai, and then captures no AI events; provider calls still work.CallbackHandler, and the OpenAI Agents and Claude Agent SDK processors.Client,AsyncPosthogand the default global client work as before.setup; patchposthog.ai.utils.setupin tests instead.💚 How did you test it?
capture_ai; each logs one error and never callscapture.🤖 Agent context
Autonomy: Human-driven (agent-assisted)
Written with Cursor (Claude Opus 5.5) under the direction of the assignee, who chose failing at setup over failing at send time.
🤖 Generated with Claude Code