Onboarding checklist polish: final-step placement, font-sized provider icons, "Hey Kody" prompt prefixes

[kentcdodds]

Summary

Follow-up polish to #1266 from design review of the live pages:

• Checklist placement: the "Your setup checklist" card no longer renders between the stepper and every panel (it distracted from the current step); it now lives only inside the final "Install a starter package" panel, where it doubles as the what's-left recap.
• Provider icons: ProviderIcon gains an optional size prop; the checklist's "Try Google, GitHub, Spotify, or Notion" row passes 1em so the marks sit at text height instead of oversized. Other surfaces keep the existing 1.25em default.
• Prompts: both first-win prompts now start with "Hey Kody, " so the receiving agent immediately knows the request is for its Kody MCP connection.

Testing

• npm run typecheck, lint, and client node tests green; full pre-push suite (unit + workers + e2e) green on push.
• Verified in the browser: step 1 shows no checklist; step 4 shows it below the starter cards with text-height icons; /onboarding.json serves the prefixed prompts.

System recap — composes existing primitives (low risk)

Mode: recap · Base: main @ fcbe1fdc · Head: 9fc8e2f6

Classification: composes — client-only presentation changes inside app-ui (checklist placement, icon sizing prop) plus two prompt-string edits in onboarding-data.ts. No routes, storage, or capability changes.

Primitives touched

Primitive	Group	Impact
app-ui	surfaces	composes — moves the checklist card into step 4; ProviderIcon size prop
app-sessions	auth	composes — "Hey Kody, " prefixes on the two first-win prompts

  

Summary by CodeRabbit

• UI Improvements
  • Provider icons now support flexible sizing for better alignment across the interface.
  • Updated onboarding links with appropriately sized provider icons.
• Onboarding
  • Moved the onboarding checklist to the final starter-packages step, alongside community-package links and navigation.
  • Updated onboarding instructions with clearer, more conversational guidance for sending the welcome email and starting memory collection.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: d13f7679-7a85-4fa1-a49c-d05a245c117d

📥 Commits

Reviewing files that changed from the base of the PR and between fcbe1fd and 9fc8e2f.

📒 Files selected for processing (4)

• packages/worker/client/provider-icons.tsx
• packages/worker/client/routes/onboarding-checklist.tsx
• packages/worker/client/routes/onboarding.tsx
• packages/worker/src/app/onboarding-data.ts

---

📝 Walkthrough

Walkthrough

Provider icons now support caller-defined sizes. The onboarding checklist moved to the starter-packages step. Onboarding prompts now address Kody directly and use conversational requests.

Changes

Onboarding updates

Layer / File(s)	Summary
Configurable provider icon sizing packages/worker/client/provider-icons.tsx	Provider icon renderers now accept a size value. ProviderIcon supports an optional size prop and defaults to 1.25em.
Starter-step checklist placement packages/worker/client/routes/onboarding.tsx, packages/worker/client/routes/onboarding-checklist.tsx	The checklist now renders in the starter-packages step. Integration links set provider icon size to 1em.
Onboarding prompt wording packages/worker/src/app/onboarding-data.ts	The introductory email and memory prompts address Kody directly and frame both actions as requests.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

• kentcdodds/kody#1184: Both changes update onboarding prompt behavior.
• kentcdodds/kody#1186: Both changes reposition checklist content in the onboarding wizard.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly summarizes the checklist placement, provider icon sizing, and prompt prefix changes.
Description check	✅ Passed	The description provides a clear summary and detailed testing evidence, but omits the optional Intent heading.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch cursor/onboarding-checklist-polish-d7c2

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[github-actions]

🔎 Preview deployed: https://kody-pr-1269.kody-a99.workers.dev

Worker: kody-pr-1269
D1: kody-pr-1269-db
KV: kody-pr-1269-oauth-kv

Mocks:

• cloudflare: https://kody-pr-1269-mock-cloudflare.kody-a99.workers.dev/__mocks (kody-pr-1269-mock-cloudflare)