first commit

Author: ekang7

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

@@ -2166,7 +2166,7 @@ impl ChatSession {
                     }
                     tool_use.accepted = true;
 
-                    return Ok(ChatState::ExecuteTools);
+                    return Ok(ChatState::ExecuteTools); //REMOVEMEBUTIMPT
                 }
             } else if !self.pending_prompts.is_empty() {
                 let prompts = self.pending_prompts.drain(0..).collect();

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

@@ -130,7 +130,7 @@ pub const COMMANDS: &[&str] = &[
     "/context rm",
     "/context clear",
     "/hooks",
-    "/hooks help",
+    // "/hooks help", // currently not implemented
     "/hooks add",
     "/hooks rm",
     "/hooks enable",

crates/chat-cli/src/cli/chat/tools/execute/mod.rs

@@ -47,7 +47,7 @@ pub struct ExecuteCommand {
     pub summary: Option<String>,
 }
 
-impl ExecuteCommand {
+impl ExecuteCommand { //REMOVEMEBUTIMPT actual tool call verification 
     pub fn requires_acceptance(&self, allowed_commands: Option<&Vec<String>>, allow_read_only: bool) -> bool {
         // Always require acceptance for multi-line commands.
         if self.command.contains("\n") || self.command.contains("\r") {
@@ -294,7 +294,7 @@ mod tests {
     };
 
     #[test]
-    fn test_requires_acceptance_for_readonly_commands() {
+    fn test_requires_acceptance_for_readonly_commands() { //REMOVEMEBUTIMPT tests unit tests you should write some
         let cmds = &[
             // Safe commands
             ("ls ~", false),
@@ -442,7 +442,7 @@ mod tests {
     }
 
     #[tokio::test]
-    async fn test_eval_perm() {
+    async fn test_eval_perm() { 
         let tool_name = if cfg!(windows) { "execute_cmd" } else { "execute_bash" };
         let mut agent = Agent {
             name: "test_agent".to_string(),