fix(agent): name not being made optional (#2410)

Author: dingfeli

crates/chat-cli/src/cli/agent/mod.rs

@@ -120,9 +120,8 @@ pub enum AgentConfigError {
 #[serde(rename_all = "camelCase", deny_unknown_fields)]
 #[schemars(description = "An Agent is a declarative way of configuring a given instance of q chat.")]
 pub struct Agent {
-    /// Agent names are derived from the file name. Thus they are skipped for
-    /// serializing
-    #[serde(skip)]
+    /// Agent names are optional. If they are not provided they are derived from the file name
+    #[serde(default)]
     pub name: String,
     /// This field is not model facing and is mostly here for users to discern between agents
     #[serde(default)]
@@ -211,13 +210,16 @@ impl Agent {
     fn thaw(&mut self, path: &Path, global_mcp_config: Option<&McpServerConfig>) -> Result<(), AgentConfigError> {
         let Self { mcp_servers, .. } = self;
 
-        let name = path
-            .file_stem()
-            .ok_or(AgentConfigError::MissingFilename)?
-            .to_string_lossy()
-            .to_string();
+        if self.name.is_empty() {
+            let name = path
+                .file_stem()
+                .ok_or(AgentConfigError::MissingFilename)?
+                .to_string_lossy()
+                .to_string();
+
+            self.name = name;
+        }
 
-        self.name = name.clone();
         self.path = Some(path.to_path_buf());
 
         if let (true, Some(global_mcp_config)) = (self.use_legacy_mcp_json, global_mcp_config) {