mirrors/hermes-agent
1
0
Fork 0
mirror of https://github.com/NousResearch/hermes-agent.git synced 2026-05-06 02:41:48 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 6

fix(tui): require double-tap d to confirm session delete

Browse source 
Single-key confirm matches how the picker already accepts 1-9 to
resume — no separate y/n keymap to learn — and "press d again" is
self-documenting next to the cursor.
This commit is contained in:
Brooklyn Nicholson 2026-04-29 19:17:58 -05:00 • committed by Teknium
parent 24b5279f43
commit 49fcad8cf8
1 changed files with 4 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

8
ui-tui/src/components/sessionPicker.tsx
View file

@ -32,7 +32,7 @@ export function SessionPicker({ gw, onCancel, onSelect, t }: SessionPickerProps)
const [sel, setSel] = useState(0)
const [loading, setLoading] = useState(true)
// When non-null, the user pressed `d` on this index and we're waiting for
// `y`/`Y` to confirm deletion. Any other key cancels the prompt.
// a second `d`/`D` to confirm deletion. Any other key cancels the prompt.
const [confirmDelete, setConfirmDelete] = useState<null | number>(null)
const [deleting, setDeleting] = useState(false)
@ -103,7 +103,7 @@ export function SessionPicker({ gw, onCancel, onSelect, t }: SessionPickerProps)
}
if (confirmDelete !== null) {
if (ch === 'y' || ch === 'Y') {
if (ch?.toLowerCase() === 'd') {
const idx = confirmDelete
setConfirmDelete(null)
performDelete(idx)
@ -128,7 +128,7 @@ export function SessionPicker({ gw, onCancel, onSelect, t }: SessionPickerProps)
return
}
if ((ch === 'd' || ch === 'D') && items[sel]) {
if (ch?.toLowerCase() === 'd' && items[sel]) {
setConfirmDelete(sel)
return
@ -202,7 +202,7 @@ export function SessionPicker({ gw, onCancel, onSelect, t }: SessionPickerProps)
inverse={selected}
wrap="truncate-end"
>
{pendingDelete ? 'delete? y/n' : s.title || s.preview || '(untitled)'}
{pendingDelete ? 'press d again to delete' : s.title || s.preview || '(untitled)'}
</Text>
</Box>
)