fix for tool args and missing msgs
This commit is contained in:
@@ -71,7 +71,7 @@ impl UiWriter for ConsoleUiWriter {
|
|||||||
.find(|(k, _)| k == "command" || k == "file_path" || k == "path" || k == "diff")
|
.find(|(k, _)| k == "command" || k == "file_path" || k == "path" || k == "diff")
|
||||||
.or_else(|| args.first());
|
.or_else(|| args.first());
|
||||||
|
|
||||||
if let Some((_, value)) = important_arg {
|
if let Some((key, value)) = important_arg {
|
||||||
// Truncate long values for display
|
// Truncate long values for display
|
||||||
let display_value = if value.len() > 80 {
|
let display_value = if value.len() > 80 {
|
||||||
format!("{}...", &value[..77])
|
format!("{}...", &value[..77])
|
||||||
@@ -79,7 +79,11 @@ impl UiWriter for ConsoleUiWriter {
|
|||||||
value.clone()
|
value.clone()
|
||||||
};
|
};
|
||||||
// Print with bold green formatting using ANSI escape codes
|
// Print with bold green formatting using ANSI escape codes
|
||||||
println!("┌─\x1b[1;32m {} | {}\x1b[0m", tool_name, display_value);
|
// Always show the key name to make it clear what the argument is
|
||||||
|
println!("┌─\x1b[1;32m {} | {}: {}\x1b[0m",
|
||||||
|
tool_name,
|
||||||
|
key,
|
||||||
|
display_value);
|
||||||
} else {
|
} else {
|
||||||
// Print with bold green formatting using ANSI escape codes
|
// Print with bold green formatting using ANSI escape codes
|
||||||
println!("┌─\x1b[1;32m {}\x1b[0m", tool_name);
|
println!("┌─\x1b[1;32m {}\x1b[0m", tool_name);
|
||||||
|
|||||||
@@ -1403,7 +1403,10 @@ The tool will execute immediately and you'll receive the result (success or erro
|
|||||||
}
|
}
|
||||||
|
|
||||||
// Execute the tool with formatted output
|
// Execute the tool with formatted output
|
||||||
self.ui_writer.println(""); // New line before tool execution
|
// Only add newline if we printed some text content
|
||||||
|
if !new_content.trim().is_empty() {
|
||||||
|
self.ui_writer.println(""); // New line before tool execution
|
||||||
|
}
|
||||||
|
|
||||||
// Skip printing tool call details for final_output
|
// Skip printing tool call details for final_output
|
||||||
if tool_call.tool != "final_output" {
|
if tool_call.tool != "final_output" {
|
||||||
|
|||||||
Reference in New Issue
Block a user