feat(agent): adds agent config validation (#2357)

* adds schema validation

* adds better error surfacing

* better formmatting

* fixes typo

* fixes tests

* adds MIT-0 to allowlist

* only performs post parse validation on active agent

* denies unknown fields from config

* adds root command to for config validation

* adds doc comment to function

* fixes typo

Author: dingfeli

Cargo.lock

@@ -128,6 +128,7 @@ windows = { version = "0.61.1", features = ["Foundation", "Win32_System_ProcessS
 winnow = "=0.6.2"
 winreg = "0.55.0"
 schemars = "1.0.4"
+jsonschema = "0.30.0"
 
 [workspace.lints.rust]
 future_incompatible = "warn"

Cargo.toml

@@ -121,6 +121,7 @@ webpki-roots.workspace = true
 whoami.workspace = true
 winnow.workspace = true
 schemars.workspace = true
+jsonschema.workspace = true
 
 [target.'cfg(unix)'.dependencies]
 nix.workspace = true

crates/chat-cli/Cargo.toml

@@ -138,7 +138,7 @@ pub async fn migrate(os: &mut Os) -> eyre::Result<Vec<Agent>> {
             name: LEGACY_GLOBAL_AGENT_NAME.to_string(),
             description: Some(DEFAULT_DESC.to_string()),
             path: Some(directories::chat_global_agent_path(os)?.join(format!("{LEGACY_GLOBAL_AGENT_NAME}.json"))),
-            resources: context.paths.iter().map(|p| format!("file://{p}")).collect(),
+            resources: context.paths.iter().map(|p| format!("file://{p}").into()).collect(),
             hooks: HashMap::from([
                 (
                     super::HookTrigger::AgentSpawn,
@@ -175,7 +175,7 @@ pub async fn migrate(os: &mut Os) -> eyre::Result<Vec<Agent>> {
             path: Some(global_agent_path.join(format!("{profile_name}.json"))),
             name: profile_name,
             description: Some(PROFILE_DESC.to_string()),
-            resources: context.paths.iter().map(|p| format!("file://{p}")).collect(),
+            resources: context.paths.iter().map(|p| format!("file://{p}").into()).collect(),
             hooks: HashMap::from([
                 (
                     super::HookTrigger::AgentSpawn,