Repository navigation
Workflows first run: any delivered test email completes the quick start task #290
Description
Map: Map: build the Workflows first run, from template gallery to first send
Spec: Spec: Workflows first run
Cast: GPT-6.1 Sol (logic wiring)
Question
Once a team has a workflow, can a test email still complete the quick start's "Send yourself a test email"?
Delivers end to end. Follow-up of quick start follows the first run (PostHog#112068). Decision (Michael, 2026-10-06): no open test-email PR completes the task, so this slice adds it.
- Any test email that is actually delivered completes the task: the email step's test send in the workflow editor, the broadcast test send, and the template library test send. A skipped or failed send does not.
- Completion goes through the same setup-task completion the first run's "Send me a test" uses, in one shared place, not once per surface.
- Once a workflow exists, the task leads to the workflow list instead of the gallery.
Test seams. A kea logic test per test-send surface (or one test of the shared helper they call) with the API mocked at the HTTP layer: delivered completes the task, skipped or failed does not.
Write scope. The test-send logics under products/workflows/frontend/ and the quick start entry in frontend/src/lib/components/ProductSetup/productSetupRegistry.ts, and tests.
Overlap: PostHog#105308 (email step test send), PostHog#111755 (real test emails to your own address) and PostHog#112065 (shared test email recipient rule) change the same test-send logics. Build on what is on your base; keep the completion call small so those PRs rebase cleanly, and name the overlap on the PR.
Acceptance criteria
- A delivered test send from a workflow email step, a broadcast, or a template completes "Send yourself a test email".
- A skipped or failed send does not.
- With a workflow present, the task leads to the workflow list.
Activity
Silthus commentedon Oct 6, 2026
Runtime: T3 delegated task from conductor thread
mcp:53aaf922-c8de-490b-8df2-687a523ef55f(devbox-michaelr-3)Worker:
impl-290, GPT-6.1 Sol (high reasoning), full accessTask:
node:delegated-task:command%3Amcp%3A2750404e-79ed-4374-a7ce-9a8c4fb2184b%3Adelegate-task%3Amap253-impl-290-r1Worktree:
/home/coder/.t3/worktrees/posthog/fr-290, branchfeat/workflows-first-run-test-send-taskfromfeat/workflows-first-run-quick-startat0bf72dfc40f(stacked on PostHog#112068)Review: one cross-family review (read-only
claude -p --model claude-opus-5-5)Cap: 4 hours from 03:55 UTC (deadline 07:55 UTC)
CodeRabbit slot: 1 of 3 (the CLI is signed out, so expect a skip)
Silthus commentedon Oct 6, 2026
Note
🤖 GPT-6.1 Sol responding on behalf of Michael
Implemented in draft PostHog/posthog#112299, stacked on PostHog#112068. Final reviewed source head:
dd8707653eb19b78ac2f94bc0bd9cc65926e71d7.Claude Opus 5.5 read-only review found one defect: removing the gallery highlight lost guidance for teams without a workflow. Fixed in
dd8707653eb19; the same review session confirmed the final fix. No rejected or unresolved findings. CodeRabbit CLI was skipped because it is signed out.Decisions: retain the gallery highlight and existing route selection; capture the tested workflow action before awaiting the invocation; leave sender/recipient rules unchanged. The specified stack parent stays unchanged. Preflight's fork-master age advisory concerns fork-only agent docs; inherited Python/OpenAPI advisories do not originate in this frontend increment.
Conductor follow-ups: account for overlap with PostHog#105308, PostHog#111755, and PostHog#112065 during graduation. PostHog#105308 introduces a separate stepTestSendLogic absent from this base, so its delivered-email path should reuse this helper when combined. The upstream API denied assignment to the verified handle Silthus; a collaborator can assign the PR. PostHog's image uploader also denied access, so the images use the authorized fork asset branch. The PR remains a draft.