From cd7bb9e272abe857d78fd984c84d90f475b17b1f Mon Sep 17 00:00:00 2001 From: Levi Neely <141506390+lneely@users.noreply.github.com> Date: Sat, 1 Aug 2026 21:53:01 +0200 Subject: [PATCH] gui: stop validating statewait via agent directory listing startStateStream now trusts the explicit session/agent selection and opens session//agent//statewait directly. It no longer calls getAgents(), which was repeatedly listing session//agent on every statewait reopen. --- gui/ollie9pclient.cpp | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/gui/ollie9pclient.cpp b/gui/ollie9pclient.cpp index 00f4256..f12b22a 100644 --- a/gui/ollie9pclient.cpp +++ b/gui/ollie9pclient.cpp @@ -444,14 +444,11 @@ void Ollie9pClient::startStateStream() qDebug() << "startStateStream active session" << m_activeSessionId << "active agent" << m_agentId; if (m_stateProc) return; - // Check if session still exists before starting + // Only explicit session+agent selection may start this stream. + // Do NOT preflight by listing session//agent; that was causing + // repeated agent-directory churn on every reopen. if (m_activeSessionId.isEmpty() || m_agentId.isEmpty()) return; - // The active agent must be explicitly selected. Do not re-resolve - // or auto-fallback here; just stop if the selection is no longer valid. - QStringList agents = getAgents(m_activeSessionId); - if (agents.isEmpty() || !agents.contains(m_agentId)) return; - m_stateProc = new QProcess(this); connect(m_stateProc, &QProcess::readyReadStandardOutput, this, &Ollie9pClient::onStateReadyRead); connect(m_stateProc, QOverload::of(&QProcess::finished),