feat(code): clean up onboarding UX

[adboio]

Problem

onboarding UX is a little unintuitive

Changes

• puts "quick tasks" and discovery run in two columns side by side
• makes the skip button more prominent

How did you test this?

manually

Publish to changelog?

no

[adboio]

• feat(code): clean up onboarding UX #2096 👈 (View in Graphite)
• feat(code): enricher run #2016
• feat: Add setup wizard integration and data-less task discovery #1745
• feat: Revamped product tours #1661
• feat: Revamped onboarding #1657
• chore: Delete the tutorial #1656
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[greptile-apps]

Prompt To Fix All With AI

Fix the following 1 code review issue. Work through them one at a time, proposing concise fixes.

---

### Issue 1 of 1
apps/code/src/renderer/features/setup/components/SetupView.tsx:45-46
`isEnricherDone` is `true` whenever the enricher is not actively running — including the `"idle"` state before it starts. If a user has persisted enricher tasks from a previous session and returns to the onboarding view before the enricher fires, `QuickWinsColumn` briefly renders with `isDone={true}` (showing "Ready"), then flips to `isDone={false}` ("Checking your PostHog setup…") once `startEnrichment()` is called. The fix requires exposing `enricherStatus` (or a dedicated `isEnricherDone`) from `useSetupRun` so the check can be `enricherStatus === "done" || enricherStatus === "error"` rather than simply `!isEnricherRunning`.

```suggestion
  const showQuickWins = enricherTasks.length > 0 || isEnricherRunning;
  const isEnricherDone = !isEnricherRunning && enricherStatus !== "idle";
```

_{Reviews (1): Last reviewed commit: "feat(code): clean up onboarindg UX" | Re-trigger Greptile}

[greptile-apps]

isEnricherDone is true whenever the enricher is not actively running — including the "idle" state before it starts. If a user has persisted enricher tasks from a previous session and returns to the onboarding view before the enricher fires, QuickWinsColumn briefly renders with isDone={true} (showing "Ready"), then flips to isDone={false} ("Checking your PostHog setup…") once startEnrichment() is called. The fix requires exposing enricherStatus (or a dedicated isEnricherDone) from useSetupRun so the check can be enricherStatus === "done" || enricherStatus === "error" rather than simply !isEnricherRunning.

Suggested change

	const showQuickWins = enricherTasks.length > 0 || isEnricherRunning;
	const isEnricherDone = !isEnricherRunning;
	const showQuickWins = enricherTasks.length > 0 || isEnricherRunning;
	const isEnricherDone = !isEnricherRunning && enricherStatus !== "idle";

Prompt To Fix With AI

This is a comment left during a code review.
Path: apps/code/src/renderer/features/setup/components/SetupView.tsx
Line: 45-46

Comment:
`isEnricherDone` is `true` whenever the enricher is not actively running — including the `"idle"` state before it starts. If a user has persisted enricher tasks from a previous session and returns to the onboarding view before the enricher fires, `QuickWinsColumn` briefly renders with `isDone={true}` (showing "Ready"), then flips to `isDone={false}` ("Checking your PostHog setup…") once `startEnrichment()` is called. The fix requires exposing `enricherStatus` (or a dedicated `isEnricherDone`) from `useSetupRun` so the check can be `enricherStatus === "done" || enricherStatus === "error"` rather than simply `!isEnricherRunning`.

```suggestion
  const showQuickWins = enricherTasks.length > 0 || isEnricherRunning;
  const isEnricherDone = !isEnricherRunning && enricherStatus !== "idle";
```

How can I resolve this? If you propose a fix, please make it concise.

[charlesvien]

TYVM

[adboio]

@charlesvien tyyyy can i get a stamp downstream too pls 🙏

[adboio]

Merge activity

• May 7, 8:44 PM UTC: A user started a stack merge that includes this pull request via Graphite.
• May 7, 8:45 PM UTC: Graphite rebased this pull request as part of a merge.
• May 7, 8:54 PM UTC: @adboio merged this pull request with Graphite.