[CI] (4c70a79) rails/fizzy - #3870
Closed
wizard-ci-bot[bot] wants to merge 1 commit into
Closed
Conversation
Author
PR Evaluation ReportSummaryThis PR integrates PostHog into a Ruby on Rails app ("fizzy") by adding the
Confidence score: 5/5 🧙
File changes
App sanity check ✅
Issues
|
| Criteria | Result | Description |
|---|---|---|
| PostHog SDKs installed | Yes | Both posthog-ruby (~> 3.0) and posthog-rails gems added to Gemfile |
| PostHog client initialized | Yes | PostHog.init with api_key and host, plus PostHog::Rails.configure with exception tracking, ActiveJob instrumentation, and user context |
| capture() | Yes | 13 capture_posthog_event calls across 10 controllers using PostHog.capture class-level method |
| identify() | N/A | Server-only app |
| Error tracking | Yes | auto_capture_exceptions: true, report_rescued_exceptions: true, auto_instrument_active_job: true configured |
| Reverse proxy | N/A | Server-only app |
Issues
- Missing
distinct_idin pre-auth events: Three capture calls (magic_link_requested×2 insessions_controller.rbandsignup_startedinsignups_controller.rb) omitdistinct_id. Without frontend tracing headers configured, these events become personless with auto-generated UUIDs. Fix: pass the email identity's user ID where available, or accept the limitation for pre-auth flows and document it. [MEDIUM]
Other completed criteria
- API key loaded from
ENV["POSTHOG_PROJECT_TOKEN"]— not hardcoded - Host loaded from
ENV["POSTHOG_HOST"]— correctly configured posthog_distinct_iddefined on User model returningid.to_s— proper distinct ID sourceposthog_current_userhelper configured viacurrent_user_methodfor automatic user contextuser_id_method: :posthog_distinct_idconfigured for user ID extraction- Graceful degradation when env vars missing (raises in development, silently disables in production)
PostHog insights and events ✅
| Filename | PostHog events | Description |
|---|---|---|
sessions_controller.rb |
magic_link_requested |
Tracks sign-in and sign-up magic link requests with flow property |
signups_controller.rb |
signup_started |
Tracks initial signup form submission |
signups/completions_controller.rb |
signup_completed |
Tracks successful signup completion |
account/cancellations_controller.rb |
account_cancelled |
Tracks account cancellation with account_id |
boards_controller.rb |
board_created |
Tracks board creation with access level |
boards/publications_controller.rb |
board_published, board_unpublished |
Tracks board publish/unpublish lifecycle |
cards_controller.rb |
card_created |
Tracks card creation for both draft and published flows |
cards/closures_controller.rb |
card_closed, card_reopened |
Tracks card close/reopen lifecycle |
cards/comments_controller.rb |
card_comment_created |
Tracks comment creation on cards |
cards/steps_controller.rb |
card_step_created |
Tracks step creation on cards |
| Automatic (posthog-rails) | capturedException |
Auto-captures controller exceptions, rescued exceptions, and ActiveJob failures |
Issues
No critical or medium issues with event quality.
Other completed criteria
- Events represent real user actions across signup, board, and card workflows
- Events enable product insights — signup funnel (
signup_started→signup_completed), board lifecycle, card management funnel - Events include relevant contextual properties (board_id, card_id, account_id, status, flow, signup_method)
- No PII in event properties — only IDs and metadata
- Event names are descriptive and consistently use snake_case convention
Reviewed by wizard workbench PR evaluator
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.
Automated wizard CI run
Source: wizard-pr
Trigger ID:
4c70a79App:
rails/fizzyApp directory:
apps/rails/fizzyWorkbench branch:
wizard-ci-4c70a79-rails-fizzyWizard branch:
release-please--branches--main--components--wizardContext Mill branch:
mainPostHog (MCP) branch:
masterTimestamp: 2026-09-09T22:30:47.485Z
Duration: 448.9s
YARA Scanner