chore(docs): add OpenAI Agents SDK LLM analytics installation#45071
chore(docs): add OpenAI Agents SDK LLM analytics installation#45071andrewm4894 merged 2 commits intomasterfrom
Conversation
Add onboarding content for OpenAI Agents SDK integration with PostHog LLM analytics. Covers basic setup, multi-agent workflows, and tool usage.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e87695f2ef
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| Step, | ||
| CodeBlock, | ||
| CalloutBox, | ||
| ProductScreenshot, |
There was a problem hiding this comment.
Remove unused CalloutBox destructure
The CalloutBox entry is destructured from useMDXComponents() but never used in this file. Because tsconfig.json enables noUnusedLocals and includes docs/onboarding/**/*, this will fail the TypeScript check (e.g., pnpm --filter=@posthog/frontend typescript:check) and block the build. Either remove CalloutBox from the destructuring or add a usage.
Useful? React with 👍 / 👎.
| Steps, | ||
| Step, | ||
| CodeBlock, | ||
| CalloutBox, |
There was a problem hiding this comment.
style: CalloutBox is imported but never used. Consider removing it or adding a "Proxy note" callout similar to other LLM integration docs to clarify that PostHog doesn't proxy calls.
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Prompt To Fix With AI
This is a comment left during a code review.
Path: docs/onboarding/llm-analytics/openai-agents.tsx
Line: 8:8
Comment:
**style:** `CalloutBox` is imported but never used. Consider removing it or adding a "Proxy note" callout similar to other LLM integration docs to clarify that PostHog doesn't proxy calls.
<sub>Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!</sub>
How can I resolve this? If you propose a fix, please make it concise.|
This PR hasn't seen activity in a week! Should it be merged, closed, or further worked on? If you want to keep it open, post a comment or remove the |
|
Docs from this PR will be published at posthog.com
Preview will be ready in ~10 minutes. Click Preview link above to access docs at |
Summary
Add onboarding content for OpenAI Agents SDK integration with PostHog LLM analytics.
Changes
docs/onboarding/llm-analytics/openai-agents.tsx- installation guideCovers:
instrument()helperRelated