fix(sidebar): highlight active agent in collapsed sidebar

[ianks]

Current behavior

In the collapsed sidebar, every agent row in the bottom section renders with the same muted style, so there is no way to tell which agent is active. The workspace list in the section directly above does mark its active entry, which makes the two lists look inconsistent at a glance.

Expected behavior

The active agent should be visually distinguishable in the collapsed sidebar, the same way the active workspace already is.

Cause

render_sidebar_collapsed styles the agent position unconditionally:

let position_style = Style::default().fg(p.overlay0);

There is no active/selected branch, unlike the collapsed workspace rows above it and render_agent_detail in the expanded sidebar, which both resolve and style the active entry.

Change

Resolve the active entry with the same app.is_active_pane(detail.ws_idx, detail.tab_idx, detail.pane_id) call the expanded agent panel already uses, then apply the treatment the collapsed workspace rows already use: surface_dim row background and the position number in text.

No new tokens, styles, or affordances. state_icon returns a foreground-only Style, so ratatui's patch semantics leave the row background intact on the icon cell, matching how the workspace rows behave.

Tests

Two tests in src/ui/sidebar.rs:

• collapsed_sidebar_highlights_active_agent asserts exactly one row is marked active and the rest stay muted.
• collapsed_sidebar_does_not_highlight_agents_without_active_workspace asserts nothing is highlighted when there is no active workspace.

Both assert on styles rather than row order, since agent_panel_entries ordering is not guaranteed.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 9e0f3677-5d8a-4bd8-8f52-a17f20120867

📥 Commits

Reviewing files that changed from the base of the PR and between 2863b71 and 1ad5a90.

📒 Files selected for processing (1)

• src/ui/sidebar.rs

🚧 Files skipped from review as they are similar to previous changes (1)

• src/ui/sidebar.rs

---

📝 Walkthrough

Walkthrough

The collapsed sidebar now highlights the agent row for the active pane. Active rows use surface_dim and primary text styling. Tests cover focused, sibling, other-workspace, and unset active states.

Changes

Collapsed sidebar highlighting

Layer / File(s)	Summary
Active agent row behavior src/ui/sidebar.rs	Collapsed agent rows compare workspace, tab, and pane identity. The active row receives full-width surface_dim highlighting and primary position text. Inactive rows remain muted. Tests cover multi-pane and no-active-workspace states.

Estimated code review effort: 2 (Simple) | ~10 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely describes the main change: highlighting the active agent in the collapsed sidebar.
Description check	✅ Passed	The description directly explains the current behavior, expected behavior, implementation, and tests for the sidebar change.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

🧹 Nitpick comments (1)

src/ui/sidebar.rs (1)

2202-2222: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win

Verify pane-specific full-row styling.

The tests inspect only the position cell. The active test also has one agent pane per workspace. It can pass if code highlights by workspace instead of focused pane, or styles only the position cell.

Add two agent panes in the active workspace. Assert that only the focused pane has surface_dim across every cell in its detail row. When app.active is None, assert that every cell in every agent row is not highlighted.

Based on supplied context, AppState::is_active_pane requires the focused pane to match.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: f944bd15-c772-40a0-87f7-4dd444eba789

📥 Commits

Reviewing files that changed from the base of the PR and between 2863b71 and 97b3d9b.

📒 Files selected for processing (1)

• src/ui/sidebar.rs

[coderabbitai]

Note

GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer.

[greptile-apps]

Greptile Summary

The PR updates the collapsed sidebar to visually distinguish the active agent pane and adds focused rendering tests.

• Applies the active row background and text color consistently across the agent row.
• Verifies that only the focused pane is highlighted and that no agent is highlighted without an active workspace.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename	Overview
src/ui/sidebar.rs	Adds active-agent row styling to the collapsed sidebar and tests active, inactive, and absent-workspace states.

_{Reviews (2): Last reviewed commit: "fix(sidebar): highlight active agent in ..." | Re-trigger Greptile}

[coderabbitai]

Note

GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer.

[ogulcancelik]

ty!