Skip to content

fix(web): show provider icons for active and completed sidebar statuses - #1888

Closed
Marve10s wants to merge 2 commits into
pingdotgg:mainfrom
Marve10s:sidebar-provider-status-icons
Closed

fix(web): show provider icons for active and completed sidebar statuses#1888
Marve10s wants to merge 2 commits into
pingdotgg:mainfrom
Marve10s:sidebar-provider-status-icons

Conversation

@Marve10s

@Marve10s Marve10s commented Apr 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • replace the generic working pulse in the sidebar with provider-specific icons for active threads
  • preserve provider branding for the working icon instead of tinting it with the status color
  • cover the provider mapping in sidebar status tests

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 fmt
  • bun lint
  • bun typecheck

Note

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 workingProvider from the session when a thread is Working, Connecting, or Completed with an unseen completion, so the UI can distinguish mixed-provider lists.

ThreadStatusLabel swaps the generic colored dot for provider-specific icons (Claude, Codex/OpenAI, Cursor, OpenCode) when that field is set, keeps brand tinting via a new providerBrandClassNames helper (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 resolveThreadStatusPill provider 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

  • ThreadStatusLabel now renders a provider-specific icon (Claude, OpenAI, Cursor, OpenCode) instead of a generic colored dot when status.workingProvider maps to a known provider; the icon pulses when status.pulse is true.
  • resolveThreadStatusPill is extended to include workingProvider on pills for running/connecting sessions, and for completed threads with unseen completions when a provider is available.
  • A new providerBrandClassNames.ts module adds normalizeProviderBrandKey (normalizes aliases like opencode variants) and providerIconClassName (returns brand color classes) utilities.

Macroscope summarized 108d873.

Refs #1813

@coderabbitai

coderabbitai Bot commented Apr 10, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 79b4064b-8a2c-4954-a34b-30585e768bc7

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added size:M 30-99 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list. labels Apr 10, 2026
@Marve10s Marve10s changed the title Show provider icons for active sidebar threads Add provider icons to active sidebar thread statuses Apr 10, 2026
@Marve10s

Marve10s commented Apr 10, 2026

Copy link
Copy Markdown
Contributor Author

Easier to identify own running sessions. Works with different states. Also easier for me identify my threads while they working. I do use rename but not as much,personally this would help me more and generally it's better than just pulse icon imo

Знімок екрана 2026-04-10 о 20 55 43

@Marve10s Marve10s changed the title Add provider icons to active sidebar thread statuses fix(web): show provider icons for active and completed sidebar statuses Apr 10, 2026
@macroscopeapp

macroscopeapp Bot commented Apr 10, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: 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.

macroscopeapp[bot]
macroscopeapp Bot previously approved these changes Apr 10, 2026
Comment thread apps/web/src/components/Sidebar.tsx Outdated
@macroscopeapp
macroscopeapp Bot dismissed their stale review April 16, 2026 09:32

Dismissing prior approval to re-evaluate 5b7e11d

macroscopeapp[bot]
macroscopeapp Bot previously approved these changes Apr 16, 2026
@Marve10s
Marve10s force-pushed the sidebar-provider-status-icons branch from 5b7e11d to 2ac0638 Compare April 17, 2026 09:55
@macroscopeapp
macroscopeapp Bot dismissed their stale review April 17, 2026 09:55

Dismissing prior approval to re-evaluate 2ac0638

macroscopeapp[bot]
macroscopeapp Bot previously approved these changes Apr 17, 2026
@macroscopeapp
macroscopeapp Bot dismissed their stale review April 18, 2026 10:58

Dismissing prior approval to re-evaluate 5e33ffd

@github-actions github-actions Bot added size:L 100-499 changed lines (additions + deletions). and removed size:M 30-99 changed lines (additions + deletions). labels Apr 18, 2026
macroscopeapp[bot]
macroscopeapp Bot previously approved these changes Apr 18, 2026
@brrock

brrock commented Apr 18, 2026

Copy link
Copy Markdown

CLEAN ASFFF!

@github-actions github-actions Bot added size:M 30-99 changed lines (additions + deletions). and removed size:L 100-499 changed lines (additions + deletions). labels Apr 20, 2026
@macroscopeapp
macroscopeapp Bot dismissed their stale review April 20, 2026 07:55

Dismissing prior approval to re-evaluate 8a70a76

macroscopeapp[bot]
macroscopeapp Bot previously approved these changes Apr 20, 2026
@macroscopeapp
macroscopeapp Bot dismissed their stale review May 11, 2026 19:01

Dismissing prior approval to re-evaluate 181008a

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ 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.

Comment thread apps/web/src/components/ThreadStatusIndicators.tsx
Comment thread apps/web/src/components/Sidebar.logic.ts Outdated
macroscopeapp[bot]
macroscopeapp Bot previously approved these changes May 11, 2026
@juliusmarminge
juliusmarminge force-pushed the sidebar-provider-status-icons branch from 181008a to 108d873 Compare June 16, 2026 18:18
@macroscopeapp
macroscopeapp Bot dismissed their stale review June 16, 2026 18:18

Dismissing prior approval to re-evaluate 108d873

@Marve10s

Copy link
Copy Markdown
Contributor Author

Yo @juliusmarminge should I close this? PR is older than me

@juliusmarminge

Copy link
Copy Markdown
Member

V2 has icons so yes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M 30-99 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants