fix(agent): always includes dummy tool (#2433)

* always includes dummy tool

* eliminates version field

eliminates version field

Author: dingfeli

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

@@ -122,8 +122,6 @@ pub enum AgentConfigError {
 pub struct Agent {
     /// Name of the agent
     pub name: String,
-    /// Version of the agent config
-    pub version: String,
     /// This field is not model facing and is mostly here for users to discern between agents
     #[serde(default)]
     pub description: Option<String>,
@@ -169,7 +167,6 @@ impl Default for Agent {
     fn default() -> Self {
         Self {
             name: DEFAULT_AGENT_NAME.to_string(),
-            version: "0.1.0".to_string(),
             description: Some("Default agent".to_string()),
             prompt: Default::default(),
             mcp_servers: Default::default(),
@@ -794,7 +791,6 @@ mod tests {
     const INPUT: &str = r#"
             {
               "name": "some_agent",
-              "version": "0.1.0",
               "description": "My developer agent is used for small development tasks like solving open issues.",
               "prompt": "You are a principal developer who uses multiple agents to accomplish difficult engineering tasks",
               "mcpServers": {

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

@@ -63,6 +63,7 @@ use crate::cli::agent::{
     McpServerConfig,
 };
 use crate::cli::chat::cli::prompts::GetPromptError;
+use crate::cli::chat::consts::DUMMY_TOOL_NAME;
 use crate::cli::chat::message::AssistantToolUse;
 use crate::cli::chat::server_messenger::{
     ServerMessengerBuilder,
@@ -896,7 +897,8 @@ impl ToolManager {
                 serde_json::from_str::<HashMap<String, ToolSpec>>(include_str!("tools/tool_index.json"))?
                     .into_iter()
                     .filter(|(name, _)| {
-                        is_allow_all
+                        name == DUMMY_TOOL_NAME
+                            || is_allow_all
                             || is_allow_native
                             || tool_list.contains(name)
                             || tool_list.contains(&format!("@builtin/{name}"))

docs/agent-format.md

@@ -5,7 +5,6 @@ The agent configuration file for each agent is a JSON file. The filename (withou
 Every agent configuration file can include the following sections:
 
 - [`name`](#name-field) — The name of the agent (optional, derived from filename if not specified).
-- [`version`](#version-field) - The version of the agent config.
 - [`description`](#description-field) — A description of the agent.
 - [`prompt`](#prompt-field) — High-level context for the agent (not yet implemented).
 - [`mcpServers`](#mcpservers-field) — The MCP servers the agent has access to.
@@ -29,17 +28,6 @@ The `name` field specifies the name of the agent. This is used for identificatio
 
 Note: While this field can be included in the configuration file, it will be overridden by the filename when the agent is loaded.
 
-## Version Field
-
-The `version` field specifies the version of the agent config. This is currently not being utilized though you must specify it.
-Currently, the default version is "0.1.0"
-
-```json
-{
-  "version": "0.1.0"
-}
-```
-
 ## Description Field
 
 The `description` field provides a description of what the agent does. This is primarily for human readability and helps users distinguish between different agents.