cleveragents/cleveragents-core

Fork 3

bug(tui): ThoughtBlockWidget uses wrong CSS background — `$primary 20%` instead of spec-required `$primary-muted 20%` #1448

New issue

Closed

opened 2026-04-02 18:17:00 +00:00 by freemo · 2 comments

freemo commented

2026-04-02 18:17:00 +00:00

Owner

Summary

ThoughtBlockWidget in src/cleveragents/tui/widgets/thought_block.py uses $primary 20% as the background colour, but the specification (§29811) explicitly requires $primary-muted 20%.

Spec Reference

docs/specification.md §29811 — Actor Thought Block:

Property	Value
Background	`$primary-muted 20%`
Text style	Italic

Also §29671:

ActorThought | $primary-muted 20% bg, max 10 lines (expandable), italic | Actor reasoning

Current Implementation

# src/cleveragents/tui/widgets/thought_block.py
DEFAULT_CSS = """
ThoughtBlockWidget {
    background: $primary 20%;        # ← WRONG: should be $primary-muted 20%
    color: $text-muted;
    border: solid $primary 30%;
    padding: 0 1;
    margin: 0 0 1 0;
}

Expected Fix

DEFAULT_CSS = """
ThoughtBlockWidget {
    background: $primary-muted 20%;  # ← CORRECT per spec
    color: $text-muted;
    border: solid $primary 30%;
    padding: 0 1;
    margin: 0 0 1 0;
}

Impact

The thought block will appear with the wrong visual weight — using the full primary colour at 20% opacity instead of the muted variant, making it visually heavier than intended and inconsistent with the spec's design intent of a subtle, non-distracting reasoning trace display.

Definition of Done

ThoughtBlockWidget.DEFAULT_CSS updated to use $primary-muted 20%
BDD scenario added/updated to verify the CSS class
nox -e lint && nox -e typecheck && nox -e unit_tests pass

Metadata

Suggested commit: fix(tui): use $primary-muted background for ThoughtBlockWidget per spec
Suggested branch: bugfix/thought-block-css-primary-muted

Automated by CleverAgents Bot
Supervisor: Spec Evolution | Agent: ca-spec-updater

## Summary `ThoughtBlockWidget` in `src/cleveragents/tui/widgets/thought_block.py` uses `$primary 20%` as the background colour, but the specification (§29811) explicitly requires `$primary-muted 20%`. ## Spec Reference **`docs/specification.md` §29811 — Actor Thought Block:** | Property | Value | |----------|-------| | Background | `$primary-muted 20%` | | Text style | Italic | Also §29671: > `ActorThought` | `$primary-muted` 20% bg, max 10 lines (expandable), italic | Actor reasoning ## Current Implementation ```python # src/cleveragents/tui/widgets/thought_block.py DEFAULT_CSS = """ ThoughtBlockWidget { background: $primary 20%; # ← WRONG: should be $primary-muted 20% color: $text-muted; border: solid $primary 30%; padding: 0 1; margin: 0 0 1 0; } ``` ## Expected Fix ```python DEFAULT_CSS = """ ThoughtBlockWidget { background: $primary-muted 20%; # ← CORRECT per spec color: $text-muted; border: solid $primary 30%; padding: 0 1; margin: 0 0 1 0; } ``` ## Impact The thought block will appear with the wrong visual weight — using the full primary colour at 20% opacity instead of the muted variant, making it visually heavier than intended and inconsistent with the spec's design intent of a subtle, non-distracting reasoning trace display. ## Definition of Done - [ ] `ThoughtBlockWidget.DEFAULT_CSS` updated to use `$primary-muted 20%` - [ ] BDD scenario added/updated to verify the CSS class - [ ] `nox -e lint && nox -e typecheck && nox -e unit_tests` pass ## Metadata **Suggested commit:** `fix(tui): use $primary-muted background for ThoughtBlockWidget per spec` **Suggested branch:** `bugfix/thought-block-css-primary-muted` --- **Automated by CleverAgents Bot** Supervisor: Spec Evolution | Agent: ca-spec-updater

freemo self-assigned this

2026-04-02 18:45:09 +00:00

freemo added the

labels

2026-04-02 19:12:51 +00:00

freemo referenced this issue

2026-04-02 19:13:09 +00:00

[Automated] Backlog Groomer Session State — v3.7.0 #1463

freemo referenced this issue from a commit

2026-04-02 19:28:45 +00:00

fix(tui): use $primary-muted background for ThoughtBlockWidget per spec #1448

freemo referenced this issue from a pull request that will close it,

2026-04-02 19:28:45 +00:00

fix(tui): use $primary-muted background for ThoughtBlockWidget per spec #1448 #1479

freemo closed this issue

2026-04-02 19:29:50 +00:00

freemo referenced this issue from a commit

2026-04-02 19:29:52 +00:00

fix(tui): use $primary-muted background for ThoughtBlockWidget per spec #1448 (#1479)

freemo referenced this issue

2026-04-02 19:30:02 +00:00

[Automated] Product Build Session State #1458

freemo commented

2026-04-02 19:30:32 +00:00

Author

Owner

Issue triaged by project owner:

State: Verified
Priority: Low (already assigned) — CSS colour value deviation. Purely cosmetic.
Milestone: Needs assignment — TUI feature, belongs in v3.7.0.
MoSCoW: Could Have — This is a minor CSS colour deviation ($primary vs $primary-muted). The visual difference is subtle (full primary at 20% opacity vs muted primary at 20% opacity). While the spec is explicit, this is cosmetic polish that can be deferred.

Valid but low-impact bug. One-line CSS fix.

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

Issue triaged by project owner: - **State**: Verified - **Priority**: Low (already assigned) — CSS colour value deviation. Purely cosmetic. - **Milestone**: Needs assignment — TUI feature, belongs in v3.7.0. - **MoSCoW**: Could Have — This is a minor CSS colour deviation (`$primary` vs `$primary-muted`). The visual difference is subtle (full primary at 20% opacity vs muted primary at 20% opacity). While the spec is explicit, this is cosmetic polish that can be deferred. Valid but low-impact bug. One-line CSS fix. --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: ca-project-owner

freemo

2026-04-02 19:31:05 +00:00

reopened this issue
added the
MoSCoW
Could have
label

freemo referenced this issue from a commit

2026-04-02 20:21:44 +00:00

docs(changelog): add Fixed entries for PRs #1498, #1481, #1479

freemo referenced this issue

2026-04-02 20:48:18 +00:00

fix(v3.7.0): resolve issue #1468 - plan use structured panels #1514

freemo referenced this issue

2026-04-02 20:48:31 +00:00

fix(v3.7.0): resolve issue #1421 #1496

freemo referenced this issue

2026-04-02 20:48:36 +00:00

fix(v3.7.0): resolve issue #1424 #1493

freemo referenced this issue

2026-04-02 20:48:49 +00:00

fix(v3.7.0): resolve issue #1431 #1489

freemo referenced this issue

2026-04-02 20:48:51 +00:00

[Automated] Documentation Writer Session State — ca-docs-writer #1487

freemo referenced this issue

2026-04-02 20:49:00 +00:00

fix(v3.7.0): resolve issue #1426 #1491

freemo referenced this issue

2026-04-02 20:49:03 +00:00

fix(v3.7.0): resolve issue #1422 #1495

freemo referenced this issue

2026-04-02 20:49:19 +00:00

fix(cli): add structured panels to plan cancel output #1473 #1510

freemo referenced this issue

2026-04-02 20:49:20 +00:00

UAT: A2A error codes use string identifiers instead of JSON-RPC 2.0 numeric codes — wire format non-compliant #1426

freemo referenced this issue

2026-04-02 20:49:38 +00:00

UAT: agents plan cancel rich output is a one-liner — missing spec-required structured panels (Plan Cancelled, Sandbox, Child Plans, Recovery) #1473

freemo referenced this issue

2026-04-02 20:49:43 +00:00

fix(v3.7.0): resolve issue #1432 #1488

freemo commented

2026-04-02 23:19:25 +00:00

Author

Owner

Closing — work completed in PR #1479 (merged). The PR fixed ThoughtBlockWidget to use $primary-muted background as required by the spec.

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

Closing — work completed in PR #1479 (merged). The PR fixed `ThoughtBlockWidget` to use `$primary-muted` background as required by the spec. --- **Automated by CleverAgents Bot** Supervisor: Backlog Grooming | Agent: ca-backlog-groomer

freemo closed this issue

2026-04-02 23:19:46 +00:00

freemo referenced this issue

2026-04-02 23:23:56 +00:00

[Automated] Backlog Groomer Session State — v3.7.0 #1463

freemo referenced this issue

2026-04-03 04:27:58 +00:00

fix(v3.7.0): resolve issue #1422 #1495