chore(init): accept trailing commas in JSON configs via serde_json_lenient (#89)

Only parse_json_config() uses lenient parsing. All other JSON handling
stays on serde_json for type compatibility (axum, MCP, hooks).

Co-authored-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>
Co-authored-by: KuSh <kush@users.noreply.github.com>

Author: KuSh

Cargo.lock

@@ -26,6 +26,7 @@ fastembed = "4"
 # Serialization
 serde = { version = "1", features = ["derive"] }
 serde_json = { version = "1", features = ["preserve_order"] }
+serde_json_lenient = { version = "0.2", features = ["preserve_order"] }
 toml = "0.8"
 
 # Error handling

Cargo.toml

@@ -43,6 +43,7 @@ tower-http = { workspace = true, optional = true }
 rust-embed = { workspace = true, optional = true }
 mime_guess = { workspace = true, optional = true }
 getrandom = { workspace = true, optional = true }
+serde_json_lenient = { workspace = true }
 
 [dev-dependencies]
 tempfile = "3"

crates/icm-cli/Cargo.toml

@@ -2604,12 +2604,20 @@ fn strip_jsonc_comments(content: &str) -> String {
 }
 
 /// Parse a JSON/JSONC config file, handling comments and empty files gracefully.
+/// Parse a JSON config file with lenient parsing: accepts trailing commas
+/// and JSONC comments (// and /* */). This is the only place we use lenient
+/// parsing — all other JSON handling uses strict serde_json.
 fn parse_json_config(config_path: &std::path::Path) -> Result<Value> {
     let content = std::fs::read_to_string(config_path)
         .with_context(|| format!("cannot read {}", config_path.display()))?;
     let clean = strip_jsonc_comments(&content);
-    serde_json::from_str(&clean)
-        .with_context(|| format!("invalid JSON in {}", config_path.display()))
+    // Use serde_json_lenient to accept trailing commas in user-edited configs,
+    // then round-trip to serde_json::Value for compatibility with the rest of the codebase.
+    let lenient: serde_json_lenient::Value = serde_json_lenient::from_str(&clean)
+        .with_context(|| format!("invalid JSON in {}", config_path.display()))?;
+    let strict: Value = serde_json::from_str(&lenient.to_string())
+        .with_context(|| format!("JSON conversion error in {}", config_path.display()))?;
+    Ok(strict)
 }
 
 /// Inject ICM MCP server into a JSON config file. Returns a status string.