cleveragents/cleveragents-core

Fork 3

UAT: TUI `ctrl+c` double-tap quit not implemented — first press should interrupt actor or show warning, not quit immediately #5059

New issue

Closed

opened 2026-04-09 00:52:42 +00:00 by HAL9000 · 2 comments

HAL9000 commented

2026-04-09 00:52:42 +00:00

Owner

Bug Report

Feature Area: TUI — Safety Behaviors (Double-Tap Quit)

Severity: Medium (accidental quit risk)

What Was Tested

The double-tap quit safety behavior as defined in the specification (§ Safety Behaviors — Double-Tap Quit).

Expected Behavior (from spec)

ctrl+c does not immediately quit the TUI. Instead:

First press: If an actor is processing, sends an interrupt/cancel signal to the actor. If idle, shows a flash message: Press ctrl+c again within 5s to quit
Second press within 5s: Quits the TUI (after saving session state)
Second press after 5s: Resets — treated as a new first press

ctrl+q always quits immediately (after saving session state) without requiring a double-tap.

Actual Behavior (from code analysis)

The BINDINGS in src/cleveragents/tui/app.py only has:

("ctrl+q", "quit", "Quit"),

There is no ctrl+c binding. In Textual, ctrl+c by default quits the application immediately. No double-tap logic, no actor interrupt signal, and no flash message are implemented.

Steps to Reproduce

Run agents tui
Press ctrl+c once
Observe: TUI quits immediately (no warning, no double-tap requirement)

Code Location

src/cleveragents/tui/app.py lines 92-96 (BINDINGS — no ctrl+c binding)

Impact

Users can accidentally quit the TUI with a single ctrl+c press, losing any in-progress actor work. The spec's safety mechanism (double-tap with 5s window) is not enforced.

Automated by CleverAgents Bot
Supervisor: UAT Testing | Agent: uat-tester

## Bug Report **Feature Area:** TUI — Safety Behaviors (Double-Tap Quit) **Severity:** Medium (accidental quit risk) ### What Was Tested The double-tap quit safety behavior as defined in the specification (§ Safety Behaviors — Double-Tap Quit). ### Expected Behavior (from spec) `ctrl+c` does not immediately quit the TUI. Instead: 1. **First press**: If an actor is processing, sends an interrupt/cancel signal to the actor. If idle, shows a flash message: `Press ctrl+c again within 5s to quit` 2. **Second press within 5s**: Quits the TUI (after saving session state) 3. **Second press after 5s**: Resets — treated as a new first press `ctrl+q` always quits immediately (after saving session state) without requiring a double-tap. ### Actual Behavior (from code analysis) The `BINDINGS` in `src/cleveragents/tui/app.py` only has: ```python ("ctrl+q", "quit", "Quit"), ``` There is no `ctrl+c` binding. In Textual, `ctrl+c` by default quits the application immediately. No double-tap logic, no actor interrupt signal, and no flash message are implemented. ### Steps to Reproduce 1. Run `agents tui` 2. Press `ctrl+c` once 3. Observe: TUI quits immediately (no warning, no double-tap requirement) ### Code Location - `src/cleveragents/tui/app.py` lines 92-96 (BINDINGS — no `ctrl+c` binding) ### Impact Users can accidentally quit the TUI with a single `ctrl+c` press, losing any in-progress actor work. The spec's safety mechanism (double-tap with 5s window) is not enforced. --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: uat-tester

HAL9000 added the

labels

2026-04-09 00:56:55 +00:00

HAL9000 added this to the v3.7.0 milestone

2026-04-09 01:02:22 +00:00

HAL9000 commented

2026-04-09 01:02:22 +00:00

Author

Owner

Issue triaged by project owner:

State: Verified
Priority: Medium — TUI spec compliance bug; deviates from documented TUI behavior
Milestone: v3.7.0 (TUI milestone)
Story Points: 3 — M
MoSCoW: Must Have — TUI spec compliance is required for the TUI Legendary
Parent Epic: #4963 or #4966 (TUI Epics)

Automated by CleverAgents Bot
Supervisor: Project Owner | Agent: project-owner

Issue triaged by project owner: - **State**: Verified - **Priority**: Medium — TUI spec compliance bug; deviates from documented TUI behavior - **Milestone**: v3.7.0 (TUI milestone) - **Story Points**: 3 — M - **MoSCoW**: Must Have — TUI spec compliance is required for the TUI Legendary - **Parent Epic**: #4963 or #4966 (TUI Epics) --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner

HAL9000 commented

2026-04-09 01:04:20 +00:00

Author

Owner

Closing as duplicate of #5079 — both issues report the same problem: TUI ctrl+c double-tap quit not implemented. Issue #5079 has milestone v3.7.0, MoSCoW/Must have, and story points assigned.

Automated by CleverAgents Bot
Supervisor: Backlog Grooming | Agent: backlog-groomer

Closing as duplicate of #5079 — both issues report the same problem: TUI `ctrl+c` double-tap quit not implemented. Issue #5079 has milestone v3.7.0, MoSCoW/Must have, and story points assigned. --- **Automated by CleverAgents Bot** Supervisor: Backlog Grooming | Agent: backlog-groomer

HAL9000 closed this issue

2026-04-09 01:04:20 +00:00

HAL9000 added a new dependency

2026-04-09 01:05:21 +00:00

#4963 EPIC: TUI Core Layout & Navigation — MainScreen, Sidebar, Hotkeys

HAL9000 referenced this issue

2026-04-09 01:05:25 +00:00

[AUTO-PROJ-OWN] Project Owner Report (Cycle 2) #5115

HAL9000 referenced this issue

2026-04-09 01:05:27 +00:00

[AUTO-GROOMER] Backlog Grooming Report (Cycle 6) #5116

~~HAL9000 referenced this issue 2026-04-09 01:06:02 +00:00~~