Skip to main content

trustee_core/
session.rs

1//! Session state — core agent session without any UI concerns.
2//!
3//! This struct holds all the state shared between frontends (TUI, API, Web):
4//! output lines, input, workflow state, config, resume info, MCP servers, etc.
5
6use std::collections::HashMap;
7use std::sync::Arc;
8
9use tokio::sync::mpsc;
10use tokio_util::sync::CancellationToken;
11
12use abk::cli::ResumeInfo;
13use abk::context::RunContext;
14
15use std::sync::atomic::{AtomicU8, Ordering};
16
17use crate::types::{
18    AutoHandoffConfig, BuildInfo, CapturedText, HandoffCaptureSink, McpServerInfo, McpServerStatus,
19    TuiMessage, WorkflowState,
20};
21
22/// Core session state for the Trustee agent.
23///
24/// Holds all state that is independent of the presentation layer (TUI, API, Web).
25/// Frontend crates compose this struct and add their own UI-specific fields.
26pub struct Session {
27    /// Input buffer for user commands
28    pub input: String,
29    /// Output log lines
30    pub output_lines: Vec<String>,
31    /// Sender for messages from async workflows (clone and pass to workflow runners)
32    pub workflow_tx: mpsc::UnboundedSender<TuiMessage>,
33    /// Current workflow lifecycle state
34    pub workflow_state: WorkflowState,
35    /// Configuration TOML for ABK workflows
36    pub config_toml: Option<String>,
37    /// Secrets for ABK workflows
38    pub secrets: Option<HashMap<String, String>>,
39    /// Build info for ABK workflows
40    pub build_info: Option<BuildInfo>,
41    /// Resume info from the last completed task for session continuity
42    pub resume_info: Option<ResumeInfo>,
43    /// Saved resume_info before execute_command consumes it; restored if task
44    /// is cancelled before producing a real checkpoint (mistake-ENTER recovery).
45    pub backup_resume_info: Option<ResumeInfo>,
46    /// Latest todo list from LLM todowrite tool
47    pub todo_lines: Vec<String>,
48    /// Cancellation token for aborting the current workflow
49    pub cancel_token: CancellationToken,
50    /// Command buffered by user during cancellation wind-down.
51    pub pending_command: Option<String>,
52    /// Whether a session handoff (Ctrl+H) should fire once the current workflow cancels.
53    pub handoff_pending: bool,
54    /// In-flight spinner entries: (tool_name, output_lines_index, hint).
55    pub pending_tool_lines: Vec<(String, usize, Option<String>)>,
56    /// Current context token count (updated from ApiCallStarted events).
57    pub current_context_tokens: usize,
58    /// Auto-handoff configuration parsed from [tui.auto_handoff].
59    pub auto_handoff: AutoHandoffConfig,
60    /// MCP server statuses received from agent init
61    pub mcp_servers: Vec<McpServerInfo>,
62    /// Whether the session should quit
63    pub should_quit: bool,
64    /// Whether auto-scroll is enabled (follows new output)
65    pub auto_scroll: bool,
66
67    // --- TMU Phase 1: Stateless Core ---
68    /// Agent name for checkpoint/token paths (replaces ABK_AGENT_NAME env var).
69    /// Defaults to "trustee". Set from config at startup.
70    pub agent_name: String,
71    /// Per-session token store (None = FileTokenStore fallback).
72    /// When set, MCP credential flows use this instead of file-based storage.
73    pub token_store: Option<Arc<dyn pep::token_store::TokenStore>>,
74
75    // --- Project/Session Identity (backward compatible, all None = old behavior) ---
76    /// Storage partition key (replaces path hash). None = hash(working_dir)
77    pub project_id: Option<String>,
78    /// Human-readable project name. None = directory name
79    pub project_name: Option<String>,
80    /// Storage directory name (replaces timestamp slug). None = auto-generate
81    pub session_id: Option<String>,
82    /// Human-readable session name. None = no description
83    pub session_name: Option<String>,
84}
85
86impl Session {
87    /// Create a new Session with default state and a fresh message channel.
88    ///
89    /// Returns `(Session, Receiver)` so the caller can own the receiver
90    /// without locking the session (prevents deadlock in async drain loops).
91    pub fn new() -> (Self, mpsc::UnboundedReceiver<TuiMessage>) {
92        let (workflow_tx, workflow_rx) = mpsc::unbounded_channel();
93        let session = Self {
94            input: String::new(),
95            output_lines: Vec::new(),
96            workflow_tx,
97            workflow_state: WorkflowState::Idle,
98            config_toml: None,
99            secrets: None,
100            build_info: None,
101            resume_info: None,
102            backup_resume_info: None,
103            todo_lines: Vec::new(),
104            cancel_token: CancellationToken::new(),
105            pending_command: None,
106            handoff_pending: false,
107            pending_tool_lines: Vec::new(),
108            current_context_tokens: 0,
109            auto_handoff: AutoHandoffConfig::default(),
110            mcp_servers: Vec::new(),
111            should_quit: false,
112            auto_scroll: true,
113            agent_name: "trustee".to_string(),
114            token_store: None,
115            project_id: None,
116            project_name: None,
117            session_id: None,
118            session_name: None,
119        };
120        (session, workflow_rx)
121    }
122
123    /// Parse auto-handoff configuration from the stored config TOML.
124    pub fn parse_auto_handoff_config(&mut self) {
125        if let Some(ref config_toml) = self.config_toml {
126            self.auto_handoff = crate::config::parse_auto_handoff_config(config_toml);
127        }
128    }
129
130    /// Handle messages from async workflows.
131    ///
132    /// This processes all workflow lifecycle events, output updates, and state transitions.
133    /// Returns `true` if the caller should check for pending commands/handoffs after.
134    pub fn handle_workflow_message(&mut self, msg: TuiMessage) {
135        match msg {
136            TuiMessage::WorkflowCancelled => {
137                self.output_lines.push("⏹ Workflow cancelled".to_string());
138                self.output_lines.push("".to_string());
139                self.workflow_state = WorkflowState::Cancelling;
140            }
141            TuiMessage::OutputLine(line) => {
142                self.output_lines.push(line);
143            }
144            TuiMessage::StreamDelta(delta) => {
145                if let Some(last) = self.output_lines.last_mut() {
146                    last.push_str(&delta);
147                } else {
148                    self.output_lines.push(delta);
149                }
150            }
151            TuiMessage::ReasoningDelta(delta) => {
152                if let Some(last) = self.output_lines.last_mut() {
153                    if !last.starts_with('\x01') {
154                        last.insert(0, '\x01');
155                    }
156                    last.push_str(&delta);
157                } else {
158                    self.output_lines.push(format!("\x01{}", delta));
159                }
160            }
161            TuiMessage::WorkflowCompleted => {
162                self.output_lines.push("✓ Workflow completed".to_string());
163                self.output_lines.push("".to_string());
164                if self.workflow_state == WorkflowState::Running {
165                    self.workflow_state = WorkflowState::Cancelling;
166                }
167            }
168            TuiMessage::WorkflowError(err) => {
169                self.output_lines.push(format!("✗ Error: {}", err));
170                self.output_lines.push("".to_string());
171                if self.workflow_state == WorkflowState::Running {
172                    self.workflow_state = WorkflowState::Cancelling;
173                }
174            }
175            TuiMessage::TodoUpdate(content) => {
176                self.todo_lines = content.lines().map(|l| l.to_string()).collect();
177            }
178            TuiMessage::ToolPending { tool_name, hint } => {
179                let label = match &hint {
180                    Some(h) => format!("⠋ {} {}", tool_name, h),
181                    None => format!("⠋ {}", tool_name),
182                };
183                let idx = self.output_lines.len();
184                self.output_lines.push(label);
185                self.pending_tool_lines.push((tool_name, idx, hint));
186            }
187            TuiMessage::ToolDone { tool_name, success, hint } => {
188                let status = if success { "✓" } else { "✗" };
189                if let Some(pos) = self.pending_tool_lines.iter().position(|(n, _, _)| *n == tool_name) {
190                    let (_, idx, pending_hint) = self.pending_tool_lines.remove(pos);
191                    let h = hint.or(pending_hint);
192                    let label = match &h {
193                        Some(h) => format!("{} {} {}", status, tool_name, h),
194                        None => format!("{} {}", status, tool_name),
195                    };
196                    if idx < self.output_lines.len() {
197                        self.output_lines[idx] = label;
198                        return;
199                    }
200                    self.output_lines.push(label);
201                } else {
202                    let label = match &hint {
203                        Some(h) => format!("{} {} {}", status, tool_name, h),
204                        None => format!("{} {}", status, tool_name),
205                    };
206                    self.output_lines.push(label);
207                }
208            }
209            TuiMessage::ResumeInfo(info) => {
210                if self.workflow_state == WorkflowState::Cancelling && info.is_none() {
211                    self.resume_info = self.backup_resume_info.take();
212                } else if info.is_some() {
213                    // Only overwrite with a valid resume_info.
214                    // Don't let ResumeInfo(None) clobber a valid Some that was
215                    // set by an earlier incremental checkpoint message — this
216                    // happens when the error/cancel path fails to create a
217                    // final checkpoint but earlier checkpoints exist.
218                    self.resume_info = info;
219                    self.backup_resume_info = None;
220                }
221                // If info is None and we're not cancelling, keep existing resume_info.
222                if self.workflow_state == WorkflowState::Cancelling {
223                    self.workflow_state = WorkflowState::Idle;
224                }
225                if self.resume_info.is_some() {
226                    if std::env::var("RUST_LOG")
227                        .map(|v| v.to_lowercase().contains("debug"))
228                        .unwrap_or(false)
229                    {
230                        self.output_lines.push("🔄 Session preserved — next command will continue this session".to_string());
231                    }
232                }
233                if self.workflow_state == WorkflowState::Idle && self.handoff_pending {
234                    self.handoff_pending = false;
235                    self.trigger_handoff(String::new());
236                } else if let Some(cmd) = self.pending_command.take() {
237                    self.input = cmd;
238                    self.execute_command();
239                }
240            }
241            TuiMessage::ContextTokensUpdated(count) => {
242                self.current_context_tokens = count;
243                if self.auto_handoff.enabled
244                    && count >= self.auto_handoff.context_threshold
245                    && self.workflow_state == WorkflowState::Running
246                    && !self.handoff_pending
247                    && self.resume_info.is_some()
248                {
249                    self.handoff_pending = true;
250                    self.cancel_token.cancel();
251                    self.workflow_state = WorkflowState::Cancelling;
252                    self.output_lines.push(format!(
253                        "🔄 Auto-handoff: cancelling workflow, context tokens ({}) ≥ threshold ({})",
254                        count, self.auto_handoff.context_threshold
255                    ));
256                }
257            }
258            TuiMessage::McpServerStatus { name, connected, tool_count, error } => {
259                let status = if connected { McpServerStatus::Connected } else { McpServerStatus::Failed };
260                if let Some(existing) = self.mcp_servers.iter_mut().find(|s| s.name == name) {
261                    existing.status = status;
262                    existing.tool_count = tool_count;
263                    existing.error = error;
264                } else {
265                    self.mcp_servers.push(McpServerInfo { name, status, tool_count, error });
266                }
267            }
268            TuiMessage::HandoffReady(briefing) => {
269                self.workflow_state = WorkflowState::Idle;
270                self.resume_info = None;
271                self.input = briefing;
272                self.execute_command();
273            }
274        }
275        if self.auto_scroll {
276            // Signal to frontend that it should scroll to bottom.
277            // Frontend reads auto_scroll flag directly.
278        }
279    }
280
281    /// Execute the current command in the input buffer.
282    ///
283    /// Spawns an async ABK workflow task, clears the input buffer, and sets
284    /// workflow_state to Running.
285    pub fn execute_command(&mut self) {
286        let command = self.input.trim().to_string();
287
288        if self.workflow_state != WorkflowState::Idle {
289            self.pending_command = Some(command);
290            self.output_lines.push("⏳ Previous workflow finishing — command queued".to_string());
291            self.input.clear();
292            return;
293        }
294
295        let is_continuation = self.resume_info.is_some();
296
297        if !is_continuation {
298            self.output_lines.clear();
299            // Auto-derive session_name from the first command if not explicitly set.
300            // Truncate to 80 chars for a reasonable display name.
301            if self.session_name.is_none() {
302                let derived = if command.len() > 80 {
303                    format!("{}...", &command[..77])
304                } else {
305                    command.clone()
306                };
307                self.session_name = Some(derived);
308            }
309        }
310
311        self.output_lines.push(format!("> {}", command));
312
313        let config_toml = match &self.config_toml {
314            Some(c) => c.clone(),
315            None => {
316                self.output_lines.push("✗ Error: Configuration not loaded".to_string());
317                self.output_lines.push("".to_string());
318                return;
319            }
320        };
321
322        let secrets = self.secrets.clone().unwrap_or_default();
323        let build_info = self.build_info.clone();
324        let tx = self.workflow_tx.clone();
325
326        let agent_name = self.agent_name.clone();
327        let token_store = self.token_store.clone();
328        let project_id = self.project_id.clone();
329        let project_name = self.project_name.clone();
330        let session_id = self.session_id.clone();
331        let session_name = self.session_name.clone();
332
333        self.backup_resume_info = self.resume_info.clone();
334        let resume_info = self.resume_info.take();
335
336        self.workflow_state = WorkflowState::Running;
337        self.auto_scroll = true;
338
339        self.cancel_token = CancellationToken::new();
340        let child_token = self.cancel_token.clone();
341
342        let (resume_tx, mut resume_rx) = mpsc::unbounded_channel();
343
344        let resume_forward_tx = tx.clone();
345        tokio::spawn(async move {
346            while let Some(info) = resume_rx.recv().await {
347                resume_forward_tx.send(TuiMessage::ResumeInfo(info)).ok();
348            }
349        });
350
351        tokio::spawn(async move {
352            let tui_sink: abk::orchestration::output::SharedSink =
353                Arc::new(crate::session::TuiForwardSink::new(tx.clone()));
354
355            // Build RunContext from session fields for stateless operation
356            let mut run_ctx = RunContext::new()
357                .with_agent_name(agent_name.clone());
358
359            // Set project identity if any field is provided
360            if project_id.is_some() || project_name.is_some() {
361                run_ctx = run_ctx.with_project(abk::context::ProjectIdentity {
362                    id: project_id.unwrap_or_else(|| "default".to_string()),
363                    name: project_name,
364                });
365            }
366
367            // Set session identity if any field is provided
368            if session_id.is_some() || session_name.is_some() {
369                run_ctx = run_ctx.with_session(abk::context::SessionIdentity {
370                    id: session_id.unwrap_or_else(|| "default".to_string()),
371                    name: session_name,
372                });
373            }
374
375            #[cfg(feature = "registry-mcp-token")]
376            {
377                if let Some(ref ts) = token_store {
378                    run_ctx = run_ctx.with_token_store(ts.clone());
379                }
380            }
381
382            // Run the entire workflow inside a TUI-mode scope.
383            // This replaces the old set_tui_mode(true)/set_tui_mode(false)
384            // process-global mutations with a task-local scope.
385            let result = abk::observability::with_tui_mode(true, async {
386                abk::cli::run_task_from_raw_config(
387                    &config_toml,
388                    secrets,
389                    build_info,
390                    &command,
391                    Some(tui_sink),
392                    resume_info,
393                    Some(resume_tx),
394                    Some(child_token),
395                    Some(&run_ctx),
396                )
397                .await
398            })
399            .await;
400
401            let task_result = result.unwrap_or_else(|e| abk::cli::TaskResult {
402                success: false,
403                error: Some(e.to_string()),
404                // resume_info will be None here, but the on_checkpoint channel
405                // may have already delivered a valid ResumeInfo via TuiMessage.
406                // The ResumeInfo handler now ignores None when a valid Some exists,
407                // so this None won't clobber the earlier incremental checkpoint.
408                resume_info: None,
409            });
410
411            let msg = if task_result.success {
412                TuiMessage::WorkflowCompleted
413            } else {
414                TuiMessage::WorkflowError(task_result.error.unwrap_or_default())
415            };
416            tx.send(msg).ok();
417            tx.send(TuiMessage::ResumeInfo(task_result.resume_info)).ok();
418        });
419
420        self.input.clear();
421    }
422
423    /// Trigger a session handoff.
424    ///
425    /// Runs a single LLM call using the current session's resume_info to generate
426    /// a briefing. On completion, sends `TuiMessage::HandoffReady(briefing)`.
427    pub fn trigger_handoff(&mut self, hint: String) {
428        if self.resume_info.is_none() {
429            self.output_lines.push("ℹ Nothing to hand off — run a task first".to_string());
430            return;
431        }
432
433        let config_toml = match &self.config_toml {
434            Some(c) => c.clone(),
435            None => {
436                self.output_lines.push("✗ Error: Configuration not loaded".to_string());
437                return;
438            }
439        };
440
441        let secrets = self.secrets.clone().unwrap_or_default();
442        let build_info = self.build_info.clone();
443        let tx = self.workflow_tx.clone();
444
445        let agent_name = self.agent_name.clone();
446        let token_store = self.token_store.clone();
447
448        let resume_info = self.resume_info.take();
449
450        self.workflow_state = WorkflowState::Running;
451        self.auto_scroll = true;
452        self.cancel_token = CancellationToken::new();
453        let child_token = self.cancel_token.clone();
454
455        self.output_lines.push("🔀 Generating session handoff briefing...".to_string());
456
457        tokio::spawn(async move {
458            let (cap_tx, mut cap_rx) = mpsc::unbounded_channel::<CapturedText>();
459            let cap_sink: abk::orchestration::output::SharedSink =
460                Arc::new(HandoffCaptureSink::new(cap_tx, child_token.clone()));
461
462            let base = "Output a session handoff briefing in at most 300 lines. \
463                 Do NOT use any tools. Include: the FULL ABSOLUTE PATH of every \
464                 project/repository being worked on (e.g. /Projects/Foo/bar — never \
465                 omit the leading path), all project/task/workstream UUIDs referenced, \
466                 every file created or modified with its full absolute path, all \
467                 commands run and their outcomes, the current state of the work, any \
468                 blockers, and the exact next action to take. \
469                 Output ONLY the briefing text — no preamble, headers, or closing remarks.";
470            let prompt = if hint.is_empty() {
471                base.to_string()
472            } else {
473                format!("{base}\n\nIn the briefing also consider: {hint}")
474            };
475
476            let (dummy_tx, _dummy_rx) = mpsc::unbounded_channel();
477
478            // Build RunContext for stateless operation
479            let run_ctx = RunContext::new()
480                .with_agent_name(agent_name.clone());
481            #[cfg(feature = "registry-mcp-token")]
482            {
483                // Note: token_store not available in handoff — handoffs don't
484                // need MCP credentials, so we skip it here.
485            }
486
487            // Run inside TUI-mode scope (task-local, not process-global)
488            let _res = abk::observability::with_tui_mode(true, async {
489                abk::cli::run_task_from_raw_config(
490                    &config_toml,
491                    secrets,
492                    build_info,
493                    &prompt,
494                    Some(cap_sink),
495                    resume_info,
496                    Some(dummy_tx),
497                    Some(child_token),
498                    Some(&run_ctx),
499                )
500                .await
501            })
502            .await;
503
504            let mut text_parts = String::new();
505            let mut reasoning_parts = String::new();
506            while let Ok(captured) = cap_rx.try_recv() {
507                match captured {
508                    CapturedText::Text(s) => text_parts.push_str(&s),
509                    CapturedText::Reasoning(s) => reasoning_parts.push_str(&s),
510                }
511            }
512
513            let briefing = if !text_parts.trim().is_empty() {
514                text_parts.trim().to_string()
515            } else if !reasoning_parts.trim().is_empty() {
516                reasoning_parts.trim().to_string()
517            } else {
518                "Session handoff: briefing unavailable — continue from previous context.".to_string()
519            };
520
521            tx.send(TuiMessage::HandoffReady(briefing)).ok();
522        });
523    }
524}
525
526impl Default for Session {
527    fn default() -> Self {
528        Self::new().0
529    }
530}
531
532/// A sink that forwards ABK `OutputEvent`s to the message channel.
533///
534/// Includes a 3-state atomic state machine (IDLE/REASONING/CONTENT) that
535/// inserts blank separator lines when transitioning between reasoning and
536/// content streams, so the frontend can distinguish them visually.
537pub struct TuiForwardSink {
538    tx: mpsc::UnboundedSender<TuiMessage>,
539    stream_state: AtomicU8,
540}
541
542/// Stream state machine constants.
543const STREAM_IDLE: u8 = 0;
544const STREAM_REASONING: u8 = 1;
545const STREAM_CONTENT: u8 = 2;
546
547impl TuiForwardSink {
548    pub fn new(tx: mpsc::UnboundedSender<TuiMessage>) -> Self {
549        Self {
550            tx,
551            stream_state: AtomicU8::new(STREAM_IDLE),
552        }
553    }
554}
555
556impl abk::orchestration::output::OutputSink for TuiForwardSink {
557    fn emit(&self, event: abk::orchestration::output::OutputEvent) {
558        use abk::orchestration::output::OutputEvent;
559
560        let msg = match event {
561            OutputEvent::StreamingChunk { delta } => {
562                if delta.is_empty() {
563                    return;
564                }
565                let prev = self.stream_state.swap(STREAM_CONTENT, Ordering::Relaxed);
566                if prev != STREAM_CONTENT {
567                    let _ = self.tx.send(TuiMessage::OutputLine(String::new()));
568                }
569                let _ = self.tx.send(TuiMessage::StreamDelta(delta));
570                return;
571            }
572
573            OutputEvent::LlmResponse { text, model } => {
574                TuiMessage::OutputLine(format!("[{}] {}", model, text))
575            }
576
577            OutputEvent::Info { message } => {
578                // Suppress noisy/no-value messages from ABK
579                if message.contains("API call completed successfully") {
580                    return;
581                }
582                TuiMessage::OutputLine(message)
583            }
584
585            OutputEvent::WorkflowStarted { task_description } => {
586                TuiMessage::OutputLine(format!("🚀 Workflow started: {}", task_description))
587            }
588
589            OutputEvent::WorkflowCompleted { reason, iterations } => {
590                TuiMessage::OutputLine(format!(
591                    "✅ Workflow completed after {} iterations: {}",
592                    iterations, reason
593                ))
594            }
595
596            OutputEvent::IterationStarted { iteration, context_tokens } => {
597                let _ = self.tx.send(TuiMessage::ContextTokensUpdated(context_tokens));
598                TuiMessage::OutputLine(format!(
599                    "📡 Iteration {} | Context = {} tokens",
600                    iteration, context_tokens
601                ))
602            }
603
604            OutputEvent::ApiCallStarted {
605                call_number,
606                model,
607                tool_count,
608                streaming,
609                context_tokens,
610                tool_tokens,
611            } => {
612                let mode = if streaming { "Streaming" } else { "Non-streaming" };
613                let total = context_tokens + tool_tokens;
614                let _ = self.tx.send(TuiMessage::ContextTokensUpdated(total));
615                // Blank line separator before each API call for readability
616                let _ = self.tx.send(TuiMessage::OutputLine(String::new()));
617                TuiMessage::OutputLine(format!(
618                    "🔥 API Call {} | Ctx={}({}+{}) | {} | Model: {} | Tools: {}",
619                    call_number, total, context_tokens, tool_tokens, mode, model, tool_count
620                ))
621            }
622
623            OutputEvent::ToolsExecuting { tool_names, hints } => {
624                for (name, hint) in tool_names.into_iter().zip(hints.into_iter()) {
625                    let _ = self.tx.send(TuiMessage::ToolPending { tool_name: name, hint });
626                }
627                self.stream_state.store(STREAM_IDLE, Ordering::Relaxed);
628                return;
629            }
630
631            OutputEvent::ToolCompleted {
632                tool_name,
633                success,
634                content,
635                description,
636            } => {
637                if tool_name == "todowrite" && success {
638                    let _ = self.tx.send(TuiMessage::TodoUpdate(content.clone()));
639                }
640                let hint = description;
641                let _ = self.tx.send(TuiMessage::ToolDone { tool_name, success, hint });
642                self.stream_state.store(STREAM_IDLE, Ordering::Relaxed);
643                return;
644            }
645
646            OutputEvent::Error { message, context } => {
647                if let Some(ctx) = context {
648                    TuiMessage::OutputLine(format!("❌ Error: {} — {}", message, ctx))
649                } else {
650                    TuiMessage::OutputLine(format!("❌ Error: {}", message))
651                }
652            }
653
654            OutputEvent::ReasoningChunk { delta } => {
655                if delta.is_empty() {
656                    return;
657                }
658                let prev = self.stream_state.swap(STREAM_REASONING, Ordering::Relaxed);
659                if prev != STREAM_REASONING {
660                    let _ = self.tx.send(TuiMessage::OutputLine(String::new()));
661                }
662                let _ = self.tx.send(TuiMessage::ReasoningDelta(delta));
663                return;
664            }
665
666            OutputEvent::McpServerStatus { name, connected, tool_count, error } => {
667                let _ = self.tx.send(TuiMessage::McpServerStatus {
668                    name,
669                    connected,
670                    tool_count,
671                    error,
672                });
673                return;
674            }
675        };
676
677        self.stream_state.store(STREAM_IDLE, Ordering::Relaxed);
678        let _ = self.tx.send(msg);
679    }
680}