fix(tui): set default THEME to dracula on TUI app class #10917

Merged
HAL9000 merged 1 commits from bugfix/m8-tui-dracula-theme into master 2026-04-30 15:54:17 +00:00
Owner

Summary

  • Adds THEME: ClassVar[str] = "dracula" to _TextualCleverAgentsTuiApp in src/cleveragents/tui/app.py
  • Without this class variable, Textual uses its default textual-dark theme instead of the Dracula theme required by the spec
  • Adds a BDD scenario tagged @tdd_issue @tdd_issue_4742 to verify the fix

Changes

src/cleveragents/tui/app.py: Added THEME: ClassVar[str] = "dracula" class variable to _TextualCleverAgentsTuiApp.

features/tui_app_coverage.feature: Added scenario The Textual TUI app has THEME set to dracula by default tagged with @tdd_issue @tdd_issue_4742.

features/steps/tui_app_coverage_steps.py: Added step definition the app class should have THEME set to "{theme}".

Closes #4742

This PR blocks issue #4742


Automated by CleverAgents Bot
Supervisor: Implementation | Agent: task-implementor

## Summary - Adds `THEME: ClassVar[str] = "dracula"` to `_TextualCleverAgentsTuiApp` in `src/cleveragents/tui/app.py` - Without this class variable, Textual uses its default `textual-dark` theme instead of the Dracula theme required by the spec - Adds a BDD scenario tagged `@tdd_issue @tdd_issue_4742` to verify the fix ## Changes **`src/cleveragents/tui/app.py`**: Added `THEME: ClassVar[str] = "dracula"` class variable to `_TextualCleverAgentsTuiApp`. **`features/tui_app_coverage.feature`**: Added scenario `The Textual TUI app has THEME set to dracula by default` tagged with `@tdd_issue @tdd_issue_4742`. **`features/steps/tui_app_coverage_steps.py`**: Added step definition `the app class should have THEME set to "{theme}"`. Closes #4742 This PR blocks issue #4742 --- Automated by CleverAgents Bot Supervisor: Implementation | Agent: task-implementor
HAL9000 added this to the v3.7.0 milestone 2026-04-28 12:37:01 +00:00
HAL9001 approved these changes 2026-04-28 16:30:09 +00:00
HAL9001 left a comment
Owner

Review Summary

PR: fix(tui): set default THEME to dracula on TUI app class
Linked Issue: #4742
Branch: bugfix/m8-tui-dracula-theme

10-Category Checklist Results

  1. CORRECTNESS --pass
  2. SPECIFICATION ALIGNMENT --pass
  3. TEST QUALITY --pass
  4. TYPE SAFETY --pass
  5. READABILITY --pass
  6. PERFORMANCE --pass
  7. SECURITY --pass
  8. CODE STYLE --pass
  9. DOCUMENTATION --pass
  10. COMMIT/PR QUALITY --pass with note

CI Status

All 14 required checks passing: lint, typecheck, security, unit_tests, integration_tests, e2e_tests, coverage, build, docker, helm, status-check, push-validation.

Non-Blocking Observations

  • Labels: The PR has zero labels. Per contributing guidelines, needs exactly one Type/ label (e.g., Type/Bug) and Priority/ label.

Automated by CleverAgents Bot
Supervisor: PR Review | Agent: pr-review-worker

## Review Summary PR: fix(tui): set default THEME to dracula on TUI app class Linked Issue: #4742 Branch: bugfix/m8-tui-dracula-theme ### 10-Category Checklist Results 1. CORRECTNESS --pass 2. SPECIFICATION ALIGNMENT --pass 3. TEST QUALITY --pass 4. TYPE SAFETY --pass 5. READABILITY --pass 6. PERFORMANCE --pass 7. SECURITY --pass 8. CODE STYLE --pass 9. DOCUMENTATION --pass 10. COMMIT/PR QUALITY --pass with note ### CI Status All 14 required checks passing: lint, typecheck, security, unit_tests, integration_tests, e2e_tests, coverage, build, docker, helm, status-check, push-validation. ### Non-Blocking Observations - Labels: The PR has zero labels. Per contributing guidelines, needs exactly one Type/ label (e.g., Type/Bug) and Priority/ label. --- Automated by CleverAgents Bot Supervisor: PR Review | Agent: pr-review-worker
Owner

Automated by CleverAgents Bot
Supervisor: PR Review | Agent: pr-review-worker

Automated by CleverAgents Bot Supervisor: PR Review | Agent: pr-review-worker
HAL9000 scheduled this pull request to auto merge when all checks succeed 2026-04-28 19:50:24 +00:00
HAL9000 force-pushed bugfix/m8-tui-dracula-theme from 4a5fe026c9 to 18ea10779b 2026-04-29 17:09:18 +00:00 Compare
HAL9000 force-pushed bugfix/m8-tui-dracula-theme from 18ea10779b to 66bd3bf0cf 2026-04-30 15:36:27 +00:00 Compare
HAL9000 merged commit 66bd3bf0cf into master 2026-04-30 15:54:17 +00:00
Sign in to join this conversation.
No Reviewers
No Label
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: cleveragents/cleveragents-core#10917