UI: Claude-like warm beige theme + System Health engine-status fix#43
Conversation
Flip the dark palette to a warm cream/beige light theme: background #f4f1ea, warm near-black text #33312b, surface #fbf9f3, border #e3ded2, terracotta accent #c15f3c, warmed status colors. Driven by globals.css :root vars + theme.ts tokens; drop the html.dark class; convert TaskGraph and code-block hardcoded dark hex to theme vars. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
The /api/settings/health engine loop referenced container.engine_registry, which does not exist (drivers live in container.agent_drivers keyed by AgentEngineType). Every engine fell to status=skip, so the UI only showed ollama + database. Read agent_drivers and map name->AgentEngineType. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
Caution Review failedPull request was closed or merged during review 📝 WalkthroughWalkthroughThe PR contains two independent changes: a backend health endpoint refactor and a frontend theme migration. The engine availability check now uses ChangesHealth Endpoint Engine Availability
UI Theme Migration to Light Palette
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Two small changes.
1. style(ui): Claude-like warm beige light theme
Flip the dark palette to a warm cream/beige light theme — background `#f4f1ea`, warm near-black text `#33312b`, surface `#fbf9f3`, border `#e3ded2`, terracotta accent `#c15f3c`, warmed status colors. Variable-driven via `globals.css :root` + `theme.ts`; `html.dark` removed; TaskGraph and code-block hardcoded dark hex converted to theme vars.
2. fix(api): System Health shows real engine status
`/api/settings/health` referenced a non-existent `container.engine_registry`, so every agent engine fell to `status=skip` and the UI only showed ollama + database. Now reads `container.agent_drivers` keyed by `AgentEngineType`.
Note: health still reflects `is_available()` (binary + key presence), which is shallow — e.g. a present-but-broken CLI can still report `ok`. Verified separately via real calls: claude_code ✓, gemini_cli ✓, ollama ✓, codex_cli ✗ (host CLI issue, not app).
🤖 Generated with Claude Code
Summary by CodeRabbit
Style
Refactor