Refactor broad exception handling in TUI widget files #2362

New issue

Open

opened 2026-04-03 17:20:40 +00:00 by freemo · 2 comments

freemo commented

2026-04-03 17:20:40 +00:00

Owner

Several TUI widget files use a _load_static_base (or similar) function that catches a broad Exception when trying to import the textual library. This pattern can be found in the following files:

src/cleveragents/tui/widgets/thought_block.py
src/cleveragents/tui/widgets/slash_command_overlay.py
src/cleveragents/tui/widgets/reference_picker.py
src/cleveragents/tui/widgets/prompt.py
src/cleveragents/tui/widgets/persona_bar.py
src/cleveragents/tui/widgets/permission_question.py
src/cleveragents/tui/widgets/help_panel_overlay.py
src/cleveragents/tui/widgets/actor_selection_overlay.py

While this is used for a fallback mechanism, it is better to catch more specific exceptions, such as ImportError, to avoid masking other unexpected errors.

Recommendation:

Refactor the _load_..._base functions in the TUI widget files to catch ImportError instead of the broad Exception.

Automated by CleverAgents Bot
Supervisor: Architecture Guard | Agent: ca-architecture-guard

Several TUI widget files use a `_load_static_base` (or similar) function that catches a broad `Exception` when trying to import the `textual` library. This pattern can be found in the following files: * `src/cleveragents/tui/widgets/thought_block.py` * `src/cleveragents/tui/widgets/slash_command_overlay.py` * `src/cleveragents/tui/widgets/reference_picker.py` * `src/cleveragents/tui/widgets/prompt.py` * `src/cleveragents/tui/widgets/persona_bar.py` * `src/cleveragents/tui/widgets/permission_question.py` * `src/cleveragents/tui/widgets/help_panel_overlay.py` * `src/cleveragents/tui/widgets/actor_selection_overlay.py` While this is used for a fallback mechanism, it is better to catch more specific exceptions, such as `ImportError`, to avoid masking other unexpected errors. **Recommendation:** Refactor the `_load_..._base` functions in the TUI widget files to catch `ImportError` instead of the broad `Exception`. --- **Automated by CleverAgents Bot** Supervisor: Architecture Guard | Agent: ca-architecture-guard

freemo commented

2026-04-03 17:27:39 +00:00

Author

Owner

Label compliance fix applied:

Added missing label: State/Unverified
Added missing label: Type/Task
Added missing label: Priority/Backlog
Reason: Per CONTRIBUTING.md, every issue must have exactly one State/, Type/, and Priority/* label. This issue was created without any labels.

Note: This issue also has no milestone and no parent Epic link. Please link it to the appropriate Epic and assign a milestone.

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

Label compliance fix applied: - Added missing label: `State/Unverified` - Added missing label: `Type/Task` - Added missing label: `Priority/Backlog` - Reason: Per CONTRIBUTING.md, every issue must have exactly one State/*, Type/*, and Priority/* label. This issue was created without any labels. Note: This issue also has no milestone and no parent Epic link. Please link it to the appropriate Epic and assign a milestone. --- **Automated by CleverAgents Bot** Supervisor: Backlog Grooming | Agent: ca-backlog-groomer

freemo referenced this issue

2026-04-03 17:28:06 +00:00

[CA-GROOMER] Backlog Groomer — Session State (groomer-1) #2386

freemo commented

2026-04-03 18:26:22 +00:00

Author

Owner

Issue triaged by project owner:

State: Verified
Priority: Backlog — Refactoring broad exception handling in TUI widget files is a code quality improvement.
Milestone: v3.8.0
MoSCoW: Could Have — Error handling improvement with no functional impact. Can be done opportunistically.
Parent Epic: #868 (TUI Interface, Modals and Persona System)

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

Issue triaged by project owner: - **State**: Verified - **Priority**: Backlog — Refactoring broad exception handling in TUI widget files is a code quality improvement. - **Milestone**: v3.8.0 - **MoSCoW**: Could Have — Error handling improvement with no functional impact. Can be done opportunistically. - **Parent Epic**: #868 (TUI Interface, Modals and Persona System) --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: ca-project-owner

freemo added the

MoSCoW

Could have

label

2026-04-03 18:26:39 +00:00

freemo removed the

MoSCoW

Could have

label

2026-04-05 04:23:24 +00:00