-
Notifications
You must be signed in to change notification settings - Fork 2.2k
chore(docs): add OpenAI Agents SDK LLM analytics installation #45071
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Add onboarding content for OpenAI Agents SDK integration with PostHog LLM analytics. Covers basic setup, multi-agent workflows, and tool usage.
|
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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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 👍 / 👎.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1 file reviewed, 1 comment
| Steps, | ||
| Step, | ||
| CodeBlock, | ||
| CalloutBox, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
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