Skip to content

feat(renderer): pin orchestrator row in sidebar session tree#3032

Open
AllBeingsFuture wants to merge 2 commits into
AgentWrapper:mainfrom
AllBeingsFuture:ao/agent-orchestrator-24/sidebar-orchestrator-row
Open

feat(renderer): pin orchestrator row in sidebar session tree#3032
AllBeingsFuture wants to merge 2 commits into
AgentWrapper:mainfrom
AllBeingsFuture:ao/agent-orchestrator-24/sidebar-orchestrator-row

Conversation

@AllBeingsFuture

Copy link
Copy Markdown

Closes #2804

Supersedes #2810 (same change, rebased onto current main after merge conflicts). The original PR branch lives on AgentWrapper/agent-orchestrator and this worker could not force-push there (Permission denied to AllBeingsFuture), so the rebased tip is on AllBeingsFuture:ao/agent-orchestrator-24/sidebar-orchestrator-row.

What

  • Adds a pinned "Orchestrator" row at the top of every expanded project session group in the Electron sidebar.
  • The row shows the same live status dot as worker sessions and navigates to the orchestrator session on click.
  • If no live orchestrator exists, clicking the row spawns one and then navigates.

Shared helper / targeting fix

Rebase conflict resolution (ShellTopbar.tsx)

Main had grown past the PR base (new shell-terminal control, board spawn error UI, "Board" root crumb, richer open-orchestrator telemetry). Resolution kept those main behaviors and still routed open/spawn through useOpenOrchestrator. The hook rethrows spawn failures so the board topbar can surface boardSpawnError.

Tests

  • cd frontend && npm test -- --run src/renderer/components/Sidebar.test.tsx ✅ (38 tests)
  • Component tests for the pinned row: presence/order, navigation, spawn fallback, status dot, and active selection.

Visual verification

ao preview is for previewing URLs in the desktop browser panel, not the app chrome, so it is not applicable for this sidebar UI change.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(nav): give the orchestrator a home in the sidebar session tree

2 participants