feat(cli): show terminal title status indicators - #10781
Conversation
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Incremental Changes ReviewedThe new commit (refactor: made this change configurable) adds Previously Reported Issues — ResolvedThe icon conversation resolved — the feature is now opt-in, keeping existing behaviour untouched while letting users who want emojis pick them explicitly. Unresolved (author disagreed)
Files Reviewed — This Increment (10 files)
All Files Reviewed (20 files)
Fix these issues in Kilo Cloud Previous Review Summaries (2 snapshots, latest commit 8ff1550)Current summary above is authoritative. Previous snapshots are kept for context only. Previous review (commit 8ff1550)Status: No Issues Found (incremental) | Recommendation: Merge Overview
Incremental Changes ReviewedThe new commit changes terminal title icons from emoji (💭🔶✅) to unicode symbols (◔⚠✓) — addressing reviewer feedback. Tests updated to match. No issues found. Previously Reported Issues — Resolved
Unresolved (author disagreed)
Files Reviewed — This Increment (2 files)
All Files Reviewed (6 files)
Fix these issues in Kilo Cloud Previous review (commit 66af690)Status: 1 Issue Found | Recommendation: Merge (suggestion only) Overview
Issue Details (click to expand)SUGGESTION
Previously Reported Issues — Resolved
Incremental Review NotesThe branch was force-pushed to Files Reviewed (6 files)
Reviewed by deepseek-v4-pro-20260423 · 1,636,735 tokens Review guidance: REVIEW.md from base branch |
48f689c to
4005b06
Compare
4005b06 to
914b35f
Compare
914b35f to
56ace1f
Compare
56ace1f to
66af690
Compare
|
|
||
| const icon = { | ||
| none: "", | ||
| working: "💭", |
There was a problem hiding this comment.
I am not sure if those icons are a good product fit. @catrielmuller any take on this?
There was a problem hiding this comment.
I went with icons because they are more easily noticeable than text, so they serve the purpose of getting a user's attention (especially the orange Attention Needed and green Done icons). However, I'm open to alternative options.
There was a problem hiding this comment.
How about: ◔ ⚠ ✓ ? Those are already used in the existing CLI
There was a problem hiding this comment.
I'm not sure if that draws attention enough, but we can see how it works. I'll push the change in a new commit.
|
Hey @shssoichiro ! I wanted to apologize for the delay with the pull requests. I’ve been busy working on the new Kilo Console and the config overlay layer with hot reload, which is super exciting because it’ll allow us to implement features without affecting our current users. I’ve updated the pull request to include both Unicode and Emoji versions, with the default option turned off for now. The great news is that moving forward, we’ll be able to roll out similar features much faster! We can create new configuration options and add them to the Kilo Console, which will really help improve visibility and make things easier to discover compared to the JSON files. Thanks for your patience, and I can’t wait to hear your thoughts! 2026-06-17_09-27-49.mp4 |
|
Awesome idea, thanks! |
feat(cli): show terminal title status indicators
Issue
Closes #10780
Context
Adds status indicators to the title bar of the current terminal tab when a session is active, indicating whether the session is working, needs user attention, or is finished.
Rationale: I wanted to know when Kilo needs my attention when my terminal tabs look like this:
Screenshots / Video
How to Test
Manual/local verification
Checklist
Get in Touch
ExpedientFalcon on Discord