minor tweaks

Author: jsamuel1

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

@@ -1149,70 +1149,38 @@ impl Command {
             .await
     }
 
+    /// Returns a vector of all available commands for dynamic enumeration
+    pub fn all_commands() -> Vec<(&'static str, &'static dyn CommandHandler)> {
+        vec![
+            ("help", &HELP_HANDLER as &dyn CommandHandler),
+            ("quit", &QUIT_HANDLER as &dyn CommandHandler),
+            ("clear", &CLEAR_HANDLER as &dyn CommandHandler),
+            ("context", &CONTEXT_HANDLER as &dyn CommandHandler),
+            ("profile", &PROFILE_HANDLER as &dyn CommandHandler),
+            ("tools", &TOOLS_HANDLER as &dyn CommandHandler),
+            ("compact", &COMPACT_HANDLER as &dyn CommandHandler),
+            ("usage", &USAGE_HANDLER as &dyn CommandHandler),
+            ("editor", &EDITOR_HANDLER as &dyn CommandHandler),
+            ("issue", &ISSUE_HANDLER as &dyn CommandHandler),
+        ]
+    }
+
     /// Generate descriptions for all commands for LLM tool descriptions
+    ///
+    /// This method dynamically iterates through all available commands and collects
+    /// their descriptions for use in LLM integration. This ensures that all commands
+    /// are properly described and no commands are missed when new ones are added.
     pub fn generate_llm_descriptions() -> std::collections::HashMap<String, CommandDescription> {
         let mut descriptions = std::collections::HashMap::new();
 
-        // Add descriptions for all implemented commands
-        descriptions.insert("help".to_string(), CommandDescription {
-            short_description: HELP_HANDLER.description().to_string(),
-            full_description: HELP_HANDLER.llm_description(),
-            usage: HELP_HANDLER.usage().to_string(),
-        });
-
-        descriptions.insert("quit".to_string(), CommandDescription {
-            short_description: QUIT_HANDLER.description().to_string(),
-            full_description: QUIT_HANDLER.llm_description(),
-            usage: QUIT_HANDLER.usage().to_string(),
-        });
-
-        descriptions.insert("clear".to_string(), CommandDescription {
-            short_description: CLEAR_HANDLER.description().to_string(),
-            full_description: CLEAR_HANDLER.llm_description(),
-            usage: CLEAR_HANDLER.usage().to_string(),
-        });
-
-        descriptions.insert("context".to_string(), CommandDescription {
-            short_description: CONTEXT_HANDLER.description().to_string(),
-            full_description: CONTEXT_HANDLER.llm_description(),
-            usage: CONTEXT_HANDLER.usage().to_string(),
-        });
-
-        descriptions.insert("profile".to_string(), CommandDescription {
-            short_description: PROFILE_HANDLER.description().to_string(),
-            full_description: PROFILE_HANDLER.llm_description(),
-            usage: PROFILE_HANDLER.usage().to_string(),
-        });
-
-        descriptions.insert("tools".to_string(), CommandDescription {
-            short_description: TOOLS_HANDLER.description().to_string(),
-            full_description: TOOLS_HANDLER.llm_description(),
-            usage: TOOLS_HANDLER.usage().to_string(),
-        });
-
-        descriptions.insert("compact".to_string(), CommandDescription {
-            short_description: COMPACT_HANDLER.description().to_string(),
-            full_description: COMPACT_HANDLER.llm_description(),
-            usage: COMPACT_HANDLER.usage().to_string(),
-        });
-
-        descriptions.insert("usage".to_string(), CommandDescription {
-            short_description: USAGE_HANDLER.description().to_string(),
-            full_description: USAGE_HANDLER.llm_description(),
-            usage: USAGE_HANDLER.usage().to_string(),
-        });
-
-        descriptions.insert("editor".to_string(), CommandDescription {
-            short_description: EDITOR_HANDLER.description().to_string(),
-            full_description: EDITOR_HANDLER.llm_description(),
-            usage: EDITOR_HANDLER.usage().to_string(),
-        });
-
-        descriptions.insert("issue".to_string(), CommandDescription {
-            short_description: ISSUE_HANDLER.description().to_string(),
-            full_description: ISSUE_HANDLER.llm_description(),
-            usage: ISSUE_HANDLER.usage().to_string(),
-        });
+        // Dynamically iterate through all commands
+        for (name, handler) in Self::all_commands() {
+            descriptions.insert(name.to_string(), CommandDescription {
+                short_description: handler.description().to_string(),
+                full_description: handler.llm_description(),
+                usage: handler.usage().to_string(),
+            });
+        }
 
         descriptions
     }

crates/chat-cli/src/cli/chat/commands/context/mod.rs

@@ -215,7 +215,7 @@ To see the full content of context files, use "/context show --expand"."#
                             Ok(ChatState::PromptUser {
                                 tool_uses,
                                 pending_tool_index,
-                                skip_printing_tools: false,
+                                skip_printing_tools: true,
                             })
                         },
                         // For other subcommands, delegate to the appropriate handler

crates/chat-cli/src/cli/chat/commands/editor.rs

@@ -167,7 +167,7 @@ Examples:
                         return Ok(ChatState::PromptUser {
                             tool_uses,
                             pending_tool_index,
-                            skip_printing_tools: false,
+                            skip_printing_tools: true,
                         });
                     },
                 };
@@ -185,7 +185,7 @@ Examples:
                         return Ok(ChatState::PromptUser {
                             tool_uses,
                             pending_tool_index,
-                            skip_printing_tools: false,
+                            skip_printing_tools: true,
                         });
                     }
                     // Flush to ensure content is written before editor opens
@@ -232,7 +232,7 @@ Examples:
                                         return Ok(ChatState::PromptUser {
                                             tool_uses,
                                             pending_tool_index,
-                                            skip_printing_tools: false,
+                                            skip_printing_tools: true,
                                         });
                                     }
 
@@ -276,7 +276,7 @@ Examples:
                 Ok(ChatState::PromptUser {
                     tool_uses,
                     pending_tool_index,
-                    skip_printing_tools: false,
+                    skip_printing_tools: true,
                 })
             } else {
                 Err(ChatError::Custom(

crates/chat-cli/src/cli/chat/commands/execute.rs

@@ -77,7 +77,7 @@ This command allows you to run any shell command without leaving the chat interf
                 Ok(ChatState::PromptUser {
                     tool_uses,
                     pending_tool_index,
-                    skip_printing_tools: false,
+                    skip_printing_tools: true,
                 })
             } else {
                 Err(ChatError::Custom(

crates/chat-cli/src/cli/chat/commands/profile/create.rs

@@ -128,7 +128,7 @@ impl CommandHandler for CreateProfileCommand {
             Ok(ChatState::PromptUser {
                 tool_uses,
                 pending_tool_index,
-                skip_printing_tools: false,
+                skip_printing_tools: true,
             })
         })
     }

crates/chat-cli/src/cli/chat/commands/profile/delete.rs

@@ -108,7 +108,7 @@ impl CommandHandler for DeleteProfileCommand {
             Ok(ChatState::PromptUser {
                 tool_uses,
                 pending_tool_index,
-                skip_printing_tools: false,
+                skip_printing_tools: true,
             })
         })
     }

crates/chat-cli/src/cli/chat/commands/profile/handler.rs

@@ -293,7 +293,7 @@ To get the current profiles, use the command "/profile list" which will display
             Ok(ChatState::PromptUser {
                 tool_uses,
                 pending_tool_index,
-                skip_printing_tools: false,
+                skip_printing_tools: true,
             })
         })
     }

crates/chat-cli/src/cli/chat/commands/profile/list.rs

@@ -138,7 +138,7 @@ impl CommandHandler for ListProfileCommand {
                 Ok(ChatState::PromptUser {
                     tool_uses,
                     pending_tool_index,
-                    skip_printing_tools: false,
+                    skip_printing_tools: true,
                 })
             } else {
                 Err(ChatError::Custom(

crates/chat-cli/src/cli/chat/commands/profile/rename.rs

@@ -125,7 +125,7 @@ impl CommandHandler for RenameProfileCommand {
             Ok(ChatState::PromptUser {
                 tool_uses,
                 pending_tool_index,
-                skip_printing_tools: false,
+                skip_printing_tools: true,
             })
         })
     }

crates/chat-cli/src/cli/chat/commands/profile/set.rs

@@ -108,7 +108,7 @@ impl CommandHandler for SetProfileCommand {
             Ok(ChatState::PromptUser {
                 tool_uses,
                 pending_tool_index,
-                skip_printing_tools: false,
+                skip_printing_tools: true,
             })
         })
     }