cleveragents/cleveragents-core

Fork 3

UAT: Dracula theme not set as default — CleverAgentsTuiApp missing THEME = "dracula" per ADR-044 #5954

New issue

Open

opened 2026-04-09 12:15:02 +00:00 by HAL9000 · 1 comment

HAL9000 commented

2026-04-09 12:15:02 +00:00

Owner

Bug Report

Feature Area: TUI Main Screen — Dracula Theme
Severity: Non-critical (cosmetic/configuration deviation)
Spec Reference: ADR-044 §Application Class and Screen Architecture, §Theme and Styling Architecture
Found by: UAT Testing Pool (uat-pool-1), worker: tui-main-screen

What Was Tested

Code-level analysis of src/cleveragents/tui/app.py for Dracula theme configuration.

Expected Behavior (from spec/ADR-044)

ADR-044 §Application Class and Screen Architecture specifies:

CleverAgentsApp(App)
└── Theme:
    └── Default: "dracula" (17+ Textual built-in themes supported)

ADR-044 §Theme and Styling Architecture states:

"Default theme is Dracula. All 17+ Textual built-in themes are supported and switchable via /theme <name> command or settings."

The Textual App class supports setting the default theme via THEME class variable or DEFAULT_CSS / theme property.

Actual Behavior

The _TextualCleverAgentsTuiApp class in app.py has no THEME class variable:

class _TextualCleverAgentsTuiApp(_TextualApp):
    """Main TUI app."""

    CSS_PATH: ClassVar[str] = "cleveragents.tcss"
    BINDINGS: ClassVar[list[tuple[str, str, str]]] = [
        ("ctrl+q", "quit", "Quit"),
        ("f1", "help", "Help"),
        ("ctrl+t", "cycle_preset", "Cycle Preset"),
    ]
    # No THEME = "dracula" here

Without THEME = "dracula", Textual will use its default theme (typically "textual-dark" or "textual-light"), not Dracula. The /theme slash command is registered in the catalog but not implemented in TuiCommandRouter.handle().

Steps to Reproduce

grep -r "THEME\|dracula\|DEFAULT_THEME" src/cleveragents/tui/ --include="*.py"
# Only finds slash_catalog.py with "theme" command description
# No THEME class variable in app.py

Impact

The TUI will launch with the wrong default theme. While this doesn't prevent functionality, it deviates from the spec and the Dracula theme is part of the v3.7.0 milestone scope:

"Dracula theme, block cursor navigation, notification system"

Fix

Add THEME: ClassVar[str] = "dracula" to _TextualCleverAgentsTuiApp in app.py.

Code Location

src/cleveragents/tui/app.py — line 91-96 (class definition, missing THEME)

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

## Bug Report **Feature Area:** TUI Main Screen — Dracula Theme **Severity:** Non-critical (cosmetic/configuration deviation) **Spec Reference:** ADR-044 §Application Class and Screen Architecture, §Theme and Styling Architecture **Found by:** UAT Testing Pool (uat-pool-1), worker: tui-main-screen --- ## What Was Tested Code-level analysis of `src/cleveragents/tui/app.py` for Dracula theme configuration. ## Expected Behavior (from spec/ADR-044) ADR-044 §Application Class and Screen Architecture specifies: ``` CleverAgentsApp(App) └── Theme: └── Default: "dracula" (17+ Textual built-in themes supported) ``` ADR-044 §Theme and Styling Architecture states: > "Default theme is **Dracula**. All 17+ Textual built-in themes are supported and switchable via `/theme <name>` command or settings." The Textual `App` class supports setting the default theme via `THEME` class variable or `DEFAULT_CSS` / `theme` property. ## Actual Behavior The `_TextualCleverAgentsTuiApp` class in `app.py` has no `THEME` class variable: ```python class _TextualCleverAgentsTuiApp(_TextualApp): """Main TUI app.""" CSS_PATH: ClassVar[str] = "cleveragents.tcss" BINDINGS: ClassVar[list[tuple[str, str, str]]] = [ ("ctrl+q", "quit", "Quit"), ("f1", "help", "Help"), ("ctrl+t", "cycle_preset", "Cycle Preset"), ] # No THEME = "dracula" here ``` Without `THEME = "dracula"`, Textual will use its default theme (typically "textual-dark" or "textual-light"), not Dracula. The `/theme` slash command is registered in the catalog but not implemented in `TuiCommandRouter.handle()`. ## Steps to Reproduce ```bash grep -r "THEME\|dracula\|DEFAULT_THEME" src/cleveragents/tui/ --include="*.py" # Only finds slash_catalog.py with "theme" command description # No THEME class variable in app.py ``` ## Impact The TUI will launch with the wrong default theme. While this doesn't prevent functionality, it deviates from the spec and the Dracula theme is part of the v3.7.0 milestone scope: > "Dracula theme, block cursor navigation, notification system" ## Fix Add `THEME: ClassVar[str] = "dracula"` to `_TextualCleverAgentsTuiApp` in `app.py`. ## Code Location - `src/cleveragents/tui/app.py` — line 91-96 (class definition, missing `THEME`) --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: uat-tester

HAL9000 added the

labels

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

HAL9000 referenced this issue

2026-04-09 12:20:53 +00:00

[AUTO-WATCHDOG] System Health Report (Cycle 21) #5915

~~HAL9000 referenced this issue 2026-04-09 12:21:23 +00:00~~

[AUTO-UAT-POOL] UAT Testing Report (Cycle 1) #5750

HAL9000 commented

2026-04-09 12:51:36 +00:00

Author

Owner

MoSCoW classification: MoSCoW/Could have

Rationale: The Dracula theme not being set as default is a cosmetic/configuration deviation from ADR-044. The TUI works correctly — it just uses a different default theme. This is a non-critical issue that can be fixed with a single line change (THEME = "dracula" in CleverAgentsTuiApp). It does not affect functionality and is not blocking any milestone.

Also adding Points/1 — XS — Setting a default theme is a trivial one-line change.

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

MoSCoW classification: **MoSCoW/Could have** Rationale: The Dracula theme not being set as default is a cosmetic/configuration deviation from ADR-044. The TUI works correctly — it just uses a different default theme. This is a non-critical issue that can be fixed with a single line change (`THEME = "dracula"` in `CleverAgentsTuiApp`). It does not affect functionality and is not blocking any milestone. Also adding `Points/1` — XS — Setting a default theme is a trivial one-line change. --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner

HAL9000 added the

MoSCoW

Could have

Points

labels

2026-04-09 13:12:57 +00:00

HAL9000 referenced this issue

2026-04-09 13:41:18 +00:00

[AUTO-ARCH] Architecture Review (Cycle 1) #5368

HAL9000 referenced this issue

2026-04-09 15:00:45 +00:00

[AUTO-ARCH] Architecture Review (Cycle 1) #5767