Repository navigation
fix: preserve tool icons on failed calls - #9606
UI Consistency: no issues found
All clear
Details
Note
Your check run agent prompt is: .macroscope/check-run-agents/ui-consistency.md
More information about how Check Run Agents work can be found in our Help Center.
Reviewed changed lines in apps/web/src/components/chat/MessagesTimeline.tsx and apps/web/src/index.css (test file excluded by scope).
- No raw controls rebuilt in place of
components/uiprimitives; changes are icon tint/conditional-rendering logic only. - New
index.cssdeclarations are theme role tokens (--color-tool-error-icon,--tool-error-iconin the base,@variant dark, and thedata-theme-idoverride block), which fall under the allowed runtime theme variable exception; they use@variant darkrather than a raw.darkselector. - No new inline
styleobjects, no duplicated primitive markup, no ad hoc one-off class strings on shared components. - The previously flagged hardcoded hex is now a theme token consumed via
text-tool-error-icon/40; that thread is resolved.
🔍 Agent activity — 2 tool calls
💭 I'm checking whether the new
--tool-error-icontoken was added inside the existing dark mode block in index.css, consistent with how other theme variables are scoped there.
1. browse_code — apps/web/src/index.css
2. browse_code — apps/web/src/index.css
💭 The CSS changes use runtime theme variables and dark variant, which is fine, and the TSX changes are just logic and classnames without primitive violations, so I don't see any issues to flag. The prior thread is already resolved, so there's nothing more to do there.
Agent Credits: 18 credits
Token usage: 4 uncached input · 17,679 cache-read input · 21,349 cache-write input · 990 output
Agent Credits may also include non-token charges from external tools such as web research.