From c51c9ebb17a34ead77bf4807322fb3ce32678916 Mon Sep 17 00:00:00 2001 From: Bryan Helmkamp Date: Fri, 20 Mar 2026 17:41:38 -0400 Subject: [PATCH] Show tool details for spawn_agent, use_skill, list_dir in `fabro logs` MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sync `tool_detail()` in logs.rs with `tool_display_name()` in run_progress.rs — the two had drifted, so `fabro logs -pf` was missing detail text for these tool types. Co-Authored-By: Claude Opus 4.6 (1M context) --- lib/crates/fabro-cli/src/commands/logs.rs | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/lib/crates/fabro-cli/src/commands/logs.rs b/lib/crates/fabro-cli/src/commands/logs.rs index 09fb2126c..96214d8f3 100644 --- a/lib/crates/fabro-cli/src/commands/logs.rs +++ b/lib/crates/fabro-cli/src/commands/logs.rs @@ -624,8 +624,13 @@ fn tool_detail(envelope: &serde_json::Value) -> Option { "edit_file" | "edit" => arg("path") .or_else(|| arg("file_path")) .map(|p| truncate(p, 60)), + "list_dir" => arg("path") + .or_else(|| arg("file_path")) + .map(|p| truncate(p, 60)), "web_search" => arg("query").map(|q| truncate(q, 60)), "web_fetch" => arg("url").map(|u| truncate(u, 60)), + "spawn_agent" => arg("task").map(|t| truncate(t, 60)), + "use_skill" => arg("skill_name").map(String::from), _ => None, } }