From b8be51fe7ceba8943e5b14605d16ab493286ed17 Mon Sep 17 00:00:00 2001
From: Friel <friel@openai.com>
Date: Sat, 28 Mar 2026 19:24:31 -0700
Subject: [PATCH] fix(tui): annotate subagent panel test literals

---
 codex-rs/tui/src/app.rs              |  2 +-
 codex-rs/tui/src/chatwidget/tests.rs | 14 ++++++++++----
 2 files changed, 11 insertions(+), 5 deletions(-)

diff --git a/codex-rs/tui/src/app.rs b/codex-rs/tui/src/app.rs
index 798a57863f..70b845cf41 100644
--- a/codex-rs/tui/src/app.rs
+++ b/codex-rs/tui/src/app.rs
@@ -7464,7 +7464,7 @@ mod tests {
         app.subagents.set_root_thread(root_thread_id);
 
         let mut info = SubagentInfo::new(
-            1,
+            /*ordinal*/ 1,
             "watchdog-agent".to_string(),
             "watchdog idle".to_string(),
             CollabAgentSpawnMode::Watchdog,
diff --git a/codex-rs/tui/src/chatwidget/tests.rs b/codex-rs/tui/src/chatwidget/tests.rs
index b91109ee51..83786aec52 100644
--- a/codex-rs/tui/src/chatwidget/tests.rs
+++ b/codex-rs/tui/src/chatwidget/tests.rs
@@ -2437,7 +2437,7 @@ fn status_line_text(chat: &ChatWidget) -> Option<String> {
 }
 
 async fn subagent_panel_is_not_flushed_into_transcript_history() {
-    let (mut chat, mut rx, _op_rx) = make_chatwidget_manual(None).await;
+    let (mut chat, mut rx, _op_rx) = make_chatwidget_manual(/*model_override*/ None).await;
 
     let state = Arc::new(StdMutex::new(SubagentPanelState {
         started_at: Instant::now(),
@@ -2452,7 +2452,10 @@ async fn subagent_panel_is_not_flushed_into_transcript_history() {
             latest_update_at: Instant::now(),
         }],
     }));
-    chat.on_subagent_panel_updated(Arc::new(SubagentStatusCell::new(Arc::clone(&state), true)));
+    chat.on_subagent_panel_updated(Arc::new(SubagentStatusCell::new(
+        Arc::clone(&state),
+        /*animations_enabled*/ true,
+    )));
 
     chat.add_to_history(history_cell::new_error_event("follow-up cell".to_string()));
 
@@ -2475,7 +2478,7 @@ async fn subagent_panel_is_not_flushed_into_transcript_history() {
 
 #[tokio::test]
 async fn subagent_panel_mounts_while_placeholder_active_cell_exists_snapshot() {
-    let (mut chat, _rx, _op_rx) = make_chatwidget_manual(None).await;
+    let (mut chat, _rx, _op_rx) = make_chatwidget_manual(/*model_override*/ None).await;
     chat.active_cell = Some(ChatWidget::placeholder_session_header_cell(
         chat.config_ref(),
     ));
@@ -2498,7 +2501,10 @@ async fn subagent_panel_mounts_while_placeholder_active_cell_exists_snapshot() {
             latest_update_at: Instant::now(),
         }],
     }));
-    chat.on_subagent_panel_updated(Arc::new(SubagentStatusCell::new(Arc::clone(&state), false)));
+    chat.on_subagent_panel_updated(Arc::new(SubagentStatusCell::new(
+        Arc::clone(&state),
+        /*animations_enabled*/ false,
+    )));
 
     assert!(
         chat.active_cell