Skip to content

Conversation

@arthurdedeus
Copy link
Contributor

@arthurdedeus arthurdedeus commented Oct 24, 2025

Problem

When users have not yet sent any LLM generation events, they don't receive clear guidance on how to set up LLM Analytics in person feed.

Part of #40328

Changes

  • Added a setup prompt component (LLMAnalyticsSetupPrompt) that displays when no LLM generation events have been detected
  • Wrapped the LLM Trace query component in notebook node with this new setup prompt
  • Added LLM_ANALYTICS to the ProductKey enum to properly identify this feature
  • The setup prompt includes documentation links to guide users on how to instrument their LLM calls

image.png

How did you test this code?

  • Tested with both states: when LLM generation events exist and when they don't
  • Verified the loading state displays correctly while checking event status
  • Confirmed the setup prompt appears with proper styling and links to documentation
  • Tested that the original query component renders correctly when events exist

Copy link
Contributor Author

arthurdedeus commented Oct 24, 2025

@arthurdedeus arthurdedeus requested review from a team and rafaeelaudibert and removed request for a team October 24, 2025 18:27
@arthurdedeus arthurdedeus self-assigned this Oct 24, 2025
@arthurdedeus arthurdedeus marked this pull request as ready for review October 24, 2025 18:28
@posthog-bot posthog-bot requested a review from daibhin October 24, 2025 18:28
@github-actions
Copy link
Contributor

github-actions bot commented Oct 24, 2025

Size Change: +32 B (0%)

Total Size: 3.32 MB

ℹ️ View Unchanged
Filename Size Change
frontend/dist/toolbar.js 3.32 MB +32 B (0%)

compressed-size-action

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

@arthurdedeus arthurdedeus changed the base branch from feat/notebook-node-cta-error-tracking to master October 24, 2025 18:38
@arthurdedeus arthurdedeus changed the base branch from master to feat/notebook-node-cta-error-tracking October 24, 2025 18:39
Copy link
Contributor Author

arthurdedeus commented Oct 27, 2025

Merge activity

  • Oct 27, 12:25 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Oct 27, 12:28 PM UTC: Graphite rebased this pull request as part of a merge.
  • Oct 27, 12:36 PM UTC: @arthurdedeus merged this pull request with Graphite.

@arthurdedeus arthurdedeus changed the base branch from feat/notebook-node-cta-error-tracking to graphite-base/40334 October 27, 2025 12:26
@arthurdedeus arthurdedeus changed the base branch from graphite-base/40334 to master October 27, 2025 12:26
@arthurdedeus arthurdedeus force-pushed the chore/notebook-node-traces-setup-prompt branch from 0dfcd4d to 27a0691 Compare October 27, 2025 12:27
@arthurdedeus arthurdedeus merged commit e83cea1 into master Oct 27, 2025
125 of 126 checks passed
@arthurdedeus arthurdedeus deleted the chore/notebook-node-traces-setup-prompt branch October 27, 2025 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants