fix(web): show provider icons for active and completed sidebar statuses - #1888
fix(web): show provider icons for active and completed sidebar statuses#1888Marve10s wants to merge 2 commits into
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 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 |
ApprovabilityVerdict: Approved This PR adds provider-specific icons to sidebar status indicators, replacing generic dots with branded icons for Claude, OpenAI, Cursor, and OpenCode providers. The changes are purely visual, self-contained to UI components, and include comprehensive test coverage. You can customize Macroscope's approvability policy. Learn more. |
Dismissing prior approval to re-evaluate 5b7e11d
5b7e11d to
2ac0638
Compare
Dismissing prior approval to re-evaluate 2ac0638
Dismissing prior approval to re-evaluate 5e33ffd
|
CLEAN ASFFF! |
Dismissing prior approval to re-evaluate 8a70a76
Dismissing prior approval to re-evaluate 181008a
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 181008a. Configure here.
181008a to
108d873
Compare
Dismissing prior approval to re-evaluate 108d873
|
Yo @juliusmarminge should I close this? PR is older than me |
|
V2 has icons so yes |


Summary
Why
The sidebar already knows which provider is attached to a running thread, but it still renders the same generic pulse for every active session. Showing the active provider directly makes it easier to scan mixed-provider thread lists.
Validation
bun fmtbun lintbun typecheckNote
Low Risk
UI-only sidebar status rendering and pure logic helpers; no auth, data, or API changes in this diff.
Overview
Sidebar thread status pills now carry an optional
workingProviderfrom the session when a thread is Working, Connecting, or Completed with an unseen completion, so the UI can distinguish mixed-provider lists.ThreadStatusLabelswaps the generic colored dot for provider-specific icons (Claude, Codex/OpenAI, Cursor, OpenCode) when that field is set, keeps brand tinting via a newproviderBrandClassNameshelper (including alias normalization for cursor/opencode CLI variants), and still falls back to the dot for unrecognized or missing providers. Compact and full layouts both use the same icon-or-dot behavior, with pulse on the icon for active sessions.Tests cover
resolveThreadStatusPillprovider propagation and static markup for Cursor/OpenCode status icons.Reviewed by Cursor Bugbot for commit 108d873. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Show provider-specific icons for active and completed thread statuses in the sidebar
ThreadStatusLabelnow renders a provider-specific icon (Claude, OpenAI, Cursor, OpenCode) instead of a generic colored dot whenstatus.workingProvidermaps to a known provider; the icon pulses whenstatus.pulseis true.resolveThreadStatusPillis extended to includeworkingProvideron pills for running/connecting sessions, and for completed threads with unseen completions when a provider is available.providerBrandClassNames.tsmodule addsnormalizeProviderBrandKey(normalizes aliases likeopencodevariants) andproviderIconClassName(returns brand color classes) utilities.Macroscope summarized 108d873.
Refs #1813