From 4fa1cbd3a5bc230a8126f002cf5608c646255292 Mon Sep 17 00:00:00 2001
From: HonzysClawdbot <janclawdbotstratil@gmail.com>
Date: Sat, 14 Mar 2026 10:38:53 +0100
Subject: [PATCH] fix: show all agents in command select, disable those without
 active session
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Previously the agent select in the OrchestrationBar only listed agents
with a session_key set (agents.filter(a => a.session_key)), causing the
dropdown to appear completely empty when agents exist but none have an
active session. This was confusing — users thought no agents were
registered.

Fix: show all registered agents in the dropdown. Agents without an
active session_key are shown as disabled with a '— no session' suffix
and a tooltip explaining why they can't be messaged. The 'No agents
registered' placeholder is shown only when the agents array is truly
empty.

The send button remains correctly disabled when no valid agent is
selected.

Fixes #321
---
 messages/en.json                            | 2 ++
 src/components/panels/orchestration-bar.tsx | 9 ++++++---
 2 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/messages/en.json b/messages/en.json
index fcb57d7..30a603d 100644
--- a/messages/en.json
+++ b/messages/en.json
@@ -1830,6 +1830,8 @@
     "tabPipelines": "Pipelines",
     "tabFleet": "Fleet",
     "selectAgent": "Select agent...",
+    "noAgentsRegistered": "No agents registered",
+    "noSessionSuffix": "no session",
     "commandPlaceholder": "Send command or message to agent...",
     "send": "Send",
     "noTemplates": "No workflow templates yet",
diff --git a/src/components/panels/orchestration-bar.tsx b/src/components/panels/orchestration-bar.tsx
index 116605e..4044d0a 100644
--- a/src/components/panels/orchestration-bar.tsx
+++ b/src/components/panels/orchestration-bar.tsx
@@ -268,9 +268,12 @@ export function OrchestrationBar() {
               className="h-9 px-2 rounded-md bg-secondary border border-border text-sm text-foreground min-w-[140px]"
             >
               <option value="">{t('selectAgent')}</option>
-              {agents.filter(a => a.session_key).map(a => (
-                <option key={a.name} value={a.name}>
-                  {a.name} ({a.status})
+              {agents.length === 0 && (
+                <option value="" disabled>{t('noAgentsRegistered')}</option>
+              )}
+              {agents.map(a => (
+                <option key={a.name} value={a.name} disabled={!a.session_key} title={!a.session_key ? 'Agent has no active session' : undefined}>
+                  {a.name} ({a.status}){!a.session_key ? ` — ${t('noSessionSuffix')}` : ''}
                 </option>
               ))}
             </select>