Skip to content

fix(cli): surface keyboard controls on add-on multiselect prompt#441

Merged
tannerlinsley merged 1 commit intomainfrom
taren/bold-wilbur-a6f1e3
Apr 30, 2026
Merged

fix(cli): surface keyboard controls on add-on multiselect prompt#441
tannerlinsley merged 1 commit intomainfrom
taren/bold-wilbur-a6f1e3

Conversation

@tannerlinsley
Copy link
Copy Markdown
Member

@tannerlinsley tannerlinsley commented Apr 30, 2026

Summary

Users reported not realizing the add-on selector during tanstack create is a checkbox list (Space toggles) that needs Enter to confirm. The existing keyboard hint was shown only once per session and could appear before single-select prompts where it didn't apply.

  • Show the "Keyboard Shortcuts" note immediately above every multiselect prompt (no longer once-only, no longer before single-selects).
  • Append (Space to toggle, Enter to confirm) directly to the multiselect message so the cue is inseparable from the prompt itself.

Result:

```
┌ Keyboard Shortcuts
│ Use ↑/↓ to navigate • Space to select/deselect • Enter to confirm

◆ What add-ons would you like for your project? (Space to toggle, Enter to confirm)
│ ◯ Clerk
│ ◯ Drizzle
│ ...
```

Test plan

  • `vitest run tests/ui-prompts.test.ts` — 14/14 pass
  • `tsc --noEmit` — clean
  • Pre-commit hook (lint, type, unit, e2e blocking) — passed
  • Manual smoke: `tanstack create my-app` and confirm the hint shows above the add-on prompt

Summary by CodeRabbit

  • Bug Fixes
    • Keyboard shortcuts help text now displays consistently above every multiselect prompt instead of appearing only once
    • Added inline keyboard hint "(Space to toggle, Enter to confirm)" to multiselect prompts for clearer user guidance
    • Removed redundant keyboard shortcuts display from single-select prompts

Users reported not realizing the add-on selector uses checkboxes (Space
to toggle) and requires Enter to confirm. Show the keyboard shortcuts
note above every multiselect (not once per session, and no longer before
single-select prompts), and append an inline `(Space to toggle, Enter to
confirm)` hint to the prompt message itself so the cue is inseparable
from the prompt.
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 30, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: cf40fb5b-7664-4113-9748-a913899852c6

📥 Commits

Reviewing files that changed from the base of the PR and between c37953e and 6b8e1ad.

📒 Files selected for processing (2)
  • .changeset/multiselect-keyboard-hint.md
  • packages/cli/src/ui-prompts.ts

📝 Walkthrough

Walkthrough

The changes introduce a patch release for the CLI package that enhances the multiselect prompt user experience. The "Keyboard Shortcuts" note is now displayed every time a multiselect prompt appears, and the multiselect message includes inline keyboard instructions specifying "(Space to toggle, Enter to confirm)".

Changes

Cohort / File(s) Summary
Changeset Documentation
.changeset/multiselect-keyboard-hint.md
Patch release metadata documenting the multiselect keyboard shortcut improvements.
CLI Prompt Updates
packages/cli/src/ui-prompts.ts
Removes one-time keyboard shortcut tracking boolean; displays keyboard shortcuts note on every multiselect prompt invocation. Multiselect message updated with inline keyboard hint text.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A prompt so keen, with hints so bright,
Space and Enter, now in sight!
Keyboard secrets no more hidden,
Every select, with guidance bidden!

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: fixing the CLI to surface keyboard controls on the multiselect prompt, which is the core objective of improving UX discoverability.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ 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 taren/bold-wilbur-a6f1e3

Review rate limit: 4/5 reviews remaining, refill in 12 minutes.

Comment @coderabbitai help to get the list of available commands and usage tips.

@tannerlinsley tannerlinsley merged commit dcacf77 into main Apr 30, 2026
4 checks passed
@tannerlinsley tannerlinsley deleted the taren/bold-wilbur-a6f1e3 branch April 30, 2026 22:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant