[CI] (e2f7ab0) javascript-web/saas-dashboard#2423
Closed
wizard-ci-bot[bot] wants to merge 1 commit into
Closed
Conversation
Author
PR Evaluation ReportSummaryThis PR integrates PostHog into a vanilla JavaScript + Vite SaaS dashboard app ("TrackFlow"). It installs
Confidence score: 5/5 🧙
File changes
App sanity check ✅
IssuesNo issues.
|
| Criteria | Result | Description |
|---|---|---|
| PostHog SDKs installed | Yes | posthog-js: ^1.398.0 added to package.json dependencies |
| PostHog client initialized | Yes | posthog.init() called in src/main.js with defaults: '2026-05-30' and loaded callback for returning-user identification |
| capture() | Yes | 12 meaningful custom events across 5 files |
| identify() | Yes | Called on login (login.js) and on page load for returning users (main.js loaded callback); uses user.id as distinct_id with person properties (name, email, role) |
| Error tracking | Yes | posthog.captureException() called in login error, project detail error, add task error, and create project error paths with contextual properties |
| Reverse proxy | No | No reverse proxy configured; api_host points directly to PostHog via env var |
Issues
- No reverse proxy: Browser requests to PostHog will be sent directly, making them susceptible to ad blockers. Configure a reverse proxy (e.g., Vite proxy in dev, Cloudflare/Vercel rewrites in production) and update
VITE_POSTHOG_HOSTto point to it. [MEDIUM]
Other completed criteria
- API key loaded from
import.meta.env.VITE_POSTHOG_KEYenvironment variable (not hardcoded) - API host loaded from
import.meta.env.VITE_POSTHOG_HOSTenvironment variable posthog.reset()correctly called on logout inshell.js- Identification uses
user.id(database ID) as distinct_id — proper practice avoiding PII as identifiers
PostHog insights and events ✅
| Filename | PostHog events | Description |
|---|---|---|
src/pages/login.js |
user_signed_in, captureException |
Tracks successful sign-in with role property; captures login errors |
src/components/shell.js |
user_signed_out |
Tracks logout, followed by posthog.reset() |
src/pages/projects.js |
project_created, project_deleted, captureException |
Tracks project lifecycle with project_id and project_name |
src/pages/project-detail.js |
project_viewed, task_created, task_completed, task_status_changed, task_assigned, task_deleted, captureException |
Full task lifecycle tracking with project_id, task_id, priority, and assignment status |
src/pages/settings.js |
settings_updated, data_reset |
Tracks individual setting changes with setting name and value |
Issues
No issues.
Other completed criteria
- Events represent real user actions mapping to actual product flows (auth → project management → task operations → settings)
- Events enable product insights: sign-in → project viewed → task created → task completed funnel is fully trackable
- Events include relevant properties (project_id, task_id, priority, status, setting name/value)
- No PII in event properties — email and name are only in
identify()person properties, not incapture()calls - Event names are descriptive, consistent 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:
e2f7ab0App:
javascript-web/saas-dashboardApp directory:
apps/javascript-web/saas-dashboardWorkbench branch:
wizard-ci-e2f7ab0-javascript-web-saas-dashboardWizard branch:
release-please--branches--main--components--wizardContext Mill branch:
mainPostHog (MCP) branch:
masterTimestamp: 2026-07-06T22:05:58.401Z
Duration: 408.4s
YARA Scanner