allow comments in config files

the following styles of comments are supported:
- C style block comments /* ... */
- C style line comments // ...
- Shell style line comments # ...

Signed-off-by: Piotr Ginał <[email protected]>

Author: piotr-ginal

Cargo.lock

@@ -16,6 +16,7 @@ futures = "0.3"
 termimad = "0.34"
 tree-sitter = "0.25"
 tree-sitter-highlight = "0.25"
+json_comments = "0.2.2"
 
 [target.'cfg(unix)'.dependencies]
 libc = "0.2"

shai-core/Cargo.toml

@@ -1,5 +1,6 @@
 use std::collections::HashMap;
 use std::path::PathBuf;
+use json_comments::StripComments;
 use serde::{Serialize, Deserialize};
 use shai_llm::ToolCallMethod;
 use crate::tools::mcp::McpConfig;
@@ -121,8 +122,9 @@ impl AgentConfig {
             return Err(format!("Agent config '{}' does not exist", agent_name).into());
         }
 
-        let content = std::fs::read_to_string(config_path)?;
-        let config: AgentConfig = serde_json::from_str(&content)?;
+        let content_bytes = std::fs::read(config_path)?;
+        let content_stripped = StripComments::new(&content_bytes[..]);
+        let config: AgentConfig = serde_json::from_reader(content_stripped)?;
         Ok(config)
     }
 

shai-core/src/config/agent.rs

@@ -3,6 +3,7 @@ use std::fs;
 #[cfg(unix)]
 use std::os::unix::fs::PermissionsExt;
 use reqwest::Url;
+use json_comments::StripComments;
 use serde::{Serialize, Deserialize};
 use shai_llm::{LlmClient, ToolCallMethod};
 use crate::tools::mcp::McpConfig;
@@ -90,9 +91,10 @@ impl ShaiConfig {
             return Err("config file does not exist".into());
         }
 
-        let content = fs::read_to_string(config_path)?;
-        let mut config: ShaiConfig = serde_json::from_str(&content)?;
-        
+        let content_bytes = fs::read(config_path)?;
+        let content_stripped = StripComments::new(&content_bytes[..]);
+        let mut config: ShaiConfig = serde_json::from_reader(content_stripped)?;
+
         // Validate selected_provider index
         if config.providers.is_empty() {
             config.selected_provider = 0;