Skip to content

fix(tui): separate autocomplete descriptions from labels - #13047

Merged
lambertjosh merged 2 commits into
mainfrom
fix/tui-autocomplete-description-spacing
Aug 10, 2026
Merged

fix(tui): separate autocomplete descriptions from labels#13047
lambertjosh merged 2 commits into
mainfrom
fix/tui-autocomplete-description-spacing

Conversation

@lambertjosh

Copy link
Copy Markdown
Contributor

What changed

Use an explicit one-column flex gap between autocomplete labels and descriptions instead of relying on leading whitespace inside a sibling text node.

Why

OpenTUI can collapse that leading whitespace, causing entries such as Past chats and Search previous sessions to run together. A layout gap preserves the separation consistently for every autocomplete item with a description.

@lambertjosh

Copy link
Copy Markdown
Contributor Author

Before

image

After

image

@kilo-code-bot

kilo-code-bot Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • packages/tui/src/component/prompt/autocomplete.tsx
  • .changeset/tidy-autocomplete-labels.md

lgtm — replacing the leading-space text node with an explicit gap={1} on the existing flexDirection="row" box is the right fix for OpenTUI whitespace collapsing, and matches a pattern used widely across the TUI. Keeping trimStart() is correct since some descriptions carry a leading space in source data (e.g. line 458), so the gap plus trim yields the same one-column separation. Edge cases (missing or whitespace-only descriptions) behave equivalently to before, no new resource/subscription lifecycle is introduced (no leak risk), kilocode_change markers are present on both changed lines of this shared upstream file, and the changeset is present with user-facing wording.


Reviewed by kimi-k3 · Input: 65.2K · Output: 4.9K · Cached: 198.5K

Review guidance: REVIEW.md from base branch main

@lambertjosh
lambertjosh merged commit ed313c3 into main Aug 10, 2026
28 checks passed
@lambertjosh
lambertjosh deleted the fix/tui-autocomplete-description-spacing branch August 10, 2026 20:39
t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
…description-spacing

fix(tui): separate autocomplete descriptions from labels
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.

2 participants