updates schema

Author: dingfeli

crates/chat-cli/src/cli/chat/prompt.rs

@@ -745,10 +745,7 @@ mod tests {
             let key_event = KeyEvent(KeyCode::Char(key), Modifiers::CTRL);
 
             // Try to bind and get the previous handler
-            let previous_handler = test_editor.bind_sequence(
-                key_event,
-                EventHandler::Simple(Cmd::Noop)
-            );
+            let previous_handler = test_editor.bind_sequence(key_event, EventHandler::Simple(Cmd::Noop));
 
             // If there was a previous handler, it means the key was already bound
             // (which could be our custom binding overriding Emacs)

crates/chat-cli/src/cli/chat/tools/custom_tool.rs

@@ -46,7 +46,7 @@ impl Default for TransportType {
 }
 
 #[derive(Clone, Serialize, Deserialize, Debug, Eq, PartialEq, JsonSchema)]
-#[serde(tag = "type", rename_all = "camelCase", deny_unknown_fields)]
+#[serde(rename_all = "camelCase", deny_unknown_fields)]
 pub struct CustomToolConfig {
     /// The transport type to use for communication with the MCP server
     #[serde(default)]

crates/chat-cli/src/mcp_client/client.rs

@@ -60,7 +60,10 @@ use crate::cli::chat::tools::custom_tool::{
     TransportType,
 };
 use crate::os::Os;
-use crate::util::directories::DirectoryError;
+use crate::util::directories::{
+    DirectoryError,
+    canonicalizes_path,
+};
 
 /// Fetches all pages of specified resources from a server
 macro_rules! paginated_fetch {
@@ -524,7 +527,8 @@ impl McpClientService {
 
         match r#type {
             TransportType::Stdio => {
-                let command = Command::new(command_as_str).configure(|cmd| {
+                let expanded_cmd = canonicalizes_path(os, command_as_str)?;
+                let command = Command::new(expanded_cmd).configure(|cmd| {
                     if let Some(envs) = config_envs {
                         process_env_vars(envs, &os.env);
                         cmd.envs(envs);

crates/chat-cli/src/mcp_client/oauth_util.rs

@@ -176,7 +176,7 @@ pub async fn get_http_transport(
     os: &Os,
     url: &str,
     timeout: u64,
-    scopes: &Vec<String>,
+    scopes: &[String],
     headers: &HashMap<String, String>,
     auth_client: Option<AuthClient<Client>>,
     messenger: &dyn Messenger,
@@ -241,7 +241,7 @@ async fn get_auth_manager(
     url: Url,
     cred_full_path: PathBuf,
     reg_full_path: PathBuf,
-    scopes: &Vec<String>,
+    scopes: &[String],
     messenger: &dyn Messenger,
 ) -> Result<AuthorizationManager, OauthUtilError> {
     let cred_as_bytes = tokio::fs::read(&cred_full_path).await;
@@ -298,7 +298,7 @@ async fn get_auth_manager(
 
 async fn get_auth_manager_impl(
     mut oauth_state: OAuthState,
-    scopes: &Vec<String>,
+    scopes: &[String],
     messenger: &dyn Messenger,
 ) -> Result<(AuthorizationManager, String), OauthUtilError> {
     let socket_addr = SocketAddr::from(([127, 0, 0, 1], 0));

schemas/agent-v1.json

@@ -73,6 +73,27 @@
             "type": "string",
             "default": ""
           },
+          "headers": {
+            "description": "HTTP headers to include when communicating with HTTP-based MCP servers",
+            "type": "object",
+            "additionalProperties": {
+              "type": "string"
+            },
+            "default": {}
+          },
+          "oauthScopes": {
+            "description": "Scopes with which oauth is done",
+            "type": "array",
+            "items": {
+              "type": "string"
+            },
+            "default": [
+              "openid",
+              "email",
+              "profile",
+              "offline_access"
+            ]
+          },
           "command": {
             "description": "The command string used to initialize the mcp server",
             "type": "string",