v0.6.29: login improvements, posthog telemetry#4026
Conversation
TheodoreSpeaks
commented
Apr 7, 2026
- feat(posthog): Add tracking on mothership abort (feat(posthog): Add tracking on mothership abort #4023)
- **fix(login): fix captcha headers for manual login (fix(login): fix captcha headers for manual login #4025) **
Co-authored-by: Theodore Li <[email protected]>
* fix(signup): fix turnstile key loading * fix(login): fix captcha header passing * Catch user already exists, remove login form captcha --------- Co-authored-by: Theodore Li <[email protected]>
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryMedium Risk Overview Adds PostHog telemetry for aborted generation by introducing a new Reviewed by Cursor Bugbot for commit 0f602f7. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryThis PR bundles two focused changes: (1) PostHog telemetry for mothership/copilot generation abort (
Confidence Score: 5/5Safe to merge — all remaining findings are P2 style/analytics consistency suggestions with no correctness or data integrity impact. Both commits are small and focused. The captcha fix is correct and straightforward. The telemetry additions follow the existing posthogRef pattern properly. The only gap is a missing task_message_sent event in the copilot submit handler, which is a non-blocking analytics omission. panel.tsx — the missing task_message_sent capture in handleCopilotSubmit should be addressed for complete telemetry parity.
|
| Filename | Overview |
|---|---|
| apps/sim/lib/posthog/events.ts | Adds two new well-typed event entries (task_generation_aborted, task_message_sent) — schemas match all call sites exactly |
| apps/sim/app/workspace/[workspaceId]/home/home.tsx | Adds posthogRef to avoid stale closures, tracks abort and message-send events correctly in mothership view |
| apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/panel.tsx | Adds abort telemetry for copilot view; missing task_message_sent capture in handleCopilotSubmit creates inconsistent analytics coverage vs mothership |
| apps/sim/app/(auth)/signup/signup-form.tsx | Fixes captcha token forwarding — correctly gates x-captcha-response header on token presence; all error branches and loading state resets are intact |
Sequence Diagram
sequenceDiagram
participant U as User
participant Home as Home (Mothership)
participant Panel as Panel (Copilot)
participant PH as PostHog
participant API as Chat API
U->>Home: Submit message
Home->>PH: task_message_sent {view: mothership}
Home->>API: sendMessage()
U->>Home: Click Stop
Home->>PH: task_generation_aborted {view: mothership}
Home->>API: stopGeneration()
U->>Panel: Submit message
Note over Panel,PH: ⚠ task_message_sent NOT captured
Panel->>API: copilotSendMessage()
U->>Panel: Click Stop
Panel->>PH: task_generation_aborted {view: copilot}
Panel->>API: copilotStopGeneration()
U->>Home: Sign up (with captcha)
Home->>Home: widget.execute() gets token
Home->>API: signUp.email() + x-captcha-response header
Comments Outside Diff (1)
-
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/panel.tsx, line 414-421 (link)Missing
task_message_senttelemetry in copilot submithandleCopilotSubmitsends a message without capturing thetask_message_sentevent, while the equivalenthandleSubmitinhome.tsx(line 239) does capture it. This creates an incomplete analytics picture — copilot message sends won't appear in PostHog alongside mothership sends.Consider adding the capture here for parity:
Reviews (1): Last reviewed commit: "fix(login): fix captcha headers for manu..." | Re-trigger Greptile