Show tool details for remaining agent tools in fabro logs and fabro attach

Add detail text for wait, send_input, close_agent (agent_id),
apply_patch (ellipsis), and read_many_files (file count) — these
were falling through to the `_ => None` catch-all in both
`tool_detail()` and `tool_display_name()`.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Bryan Helmkamp 2026-03-20 17:56:58 -04:00
parent c51c9ebb17
commit f8151c8f89
No known key found for this signature in database
2 changed files with 12 additions and 0 deletions

View file

@ -630,7 +630,13 @@ fn tool_detail(envelope: &serde_json::Value) -> Option<String> {
"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)),
"wait" | "send_input" | "close_agent" => arg("agent_id").map(String::from),
"use_skill" => arg("skill_name").map(String::from),
"apply_patch" => Some("".into()),
"read_many_files" => arguments
.get("paths")
.and_then(|v| v.as_array())
.map(|a| format!("{} files", a.len())),
_ => None,
}
}

View file

@ -247,7 +247,13 @@ impl ProgressUI {
"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)),
"wait" | "send_input" | "close_agent" => arg("agent_id").map(String::from),
"use_skill" => arg("skill_name").map(String::from),
"apply_patch" => Some("".into()),
"read_many_files" => arguments
.get("paths")
.and_then(|v| v.as_array())
.map(|a| format!("{} files", a.len())),
_ => None,
};