v0.0.17: Sanitize tool names for Kimi API compatibility

Kimi's API requires tool names matching [a-zA-Z0-9_-] but AO MCP tools
use dots (ao.task.create). Added automatic sanitization: dots→underscores
when sending to Kimi, underscores→dots when processing tool call responses.
Also fixes pool_size overwrite bug and reasoning_content for multi-turn.

Author: Shooksie

Cargo.lock

@@ -135,6 +135,18 @@ pub async fn run_agent_loop(
         tool_call_id: None,
     });
 
+    let needs_tool_name_sanitization = model.contains("kimi");
+    let sanitized_tools: Vec<ToolDefinition> = if needs_tool_name_sanitization {
+        tools.iter().map(|t| {
+            let mut t = t.clone();
+            t.function.name = t.function.name.replace('.', "_");
+            t
+        }).collect()
+    } else {
+        tools.to_vec()
+    };
+    let api_tools = if needs_tool_name_sanitization { &sanitized_tools } else { tools };
+
     for turn in 0..max_turns {
         if cancel_token.is_cancelled() {
             eprintln!("[oai-runner] Cancelled by signal");
@@ -161,7 +173,7 @@ pub async fn run_agent_loop(
             model: model.to_string(),
             messages: messages.clone(),
             stream: true,
-            tools: Some(tools.to_vec()),
+            tools: Some(api_tools.to_vec()),
             max_tokens: Some(max_tokens as u32),
             response_format: format,
             stream_options: Some(StreamOptions { include_usage: true }),
@@ -237,32 +249,38 @@ pub async fn run_agent_loop(
                 break;
             }
 
+            let tool_name = if needs_tool_name_sanitization {
+                tc.function.name.replace('_', ".")
+            } else {
+                tc.function.name.clone()
+            };
+
             let args: serde_json::Value =
                 serde_json::from_str(&tc.function.arguments).unwrap_or(serde_json::Value::Null);
 
-            output.tool_call(&tc.function.name, &args);
+            output.tool_call(&tool_name, &args);
 
-            let result = if let Some(mcp) = mcp_client::find_client_for_tool(mcp_clients, &tc.function.name) {
-                match mcp_client::call_tool(mcp, &tc.function.name, &tc.function.arguments).await {
+            let result = if let Some(mcp) = mcp_client::find_client_for_tool(mcp_clients, &tool_name) {
+                match mcp_client::call_tool(mcp, &tool_name, &tc.function.arguments).await {
                     Ok(r) => {
-                        output.tool_result(&tc.function.name, &r);
+                        output.tool_result(&tool_name, &r);
                         r
                     }
                     Err(e) => {
                         let err_msg = e.to_string();
-                        output.tool_error(&tc.function.name, &err_msg);
+                        output.tool_error(&tool_name, &err_msg);
                         format!("Error: {}", err_msg)
                     }
                 }
             } else {
-                match executor::execute_tool(&tc.function.name, &tc.function.arguments, working_dir).await {
+                match executor::execute_tool(&tool_name, &tc.function.arguments, working_dir).await {
                     Ok(r) => {
-                        output.tool_result(&tc.function.name, &r);
+                        output.tool_result(&tool_name, &r);
                         r
                     }
                     Err(e) => {
                         let err_msg = e.to_string();
-                        output.tool_error(&tc.function.name, &err_msg);
+                        output.tool_error(&tool_name, &err_msg);
                         format!("Error: {}", err_msg)
                     }
                 }

crates/oai-runner/src/runner/agent_loop.rs

@@ -1,6 +1,6 @@
 [package]
 name = "orchestrator-cli"
-version = "0.0.16"
+version = "0.0.17"
 edition = "2021"
 
 [[bin]]