fix(/tools reset): add existence check before resetting tool permissions (#1234)

Author: GoodluckH

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

@@ -1807,13 +1807,25 @@ where
                         )?;
                     },
                     Some(ToolsSubcommand::ResetSingle { tool_name }) => {
-                        self.tool_permissions.reset_tool(&tool_name);
-                        queue!(
-                            self.output,
-                            style::SetForegroundColor(Color::Green),
-                            style::Print(format!("\nReset tool '{}' to the default permission level.", tool_name)),
-                            style::SetForegroundColor(Color::Reset),
-                        )?;
+                        if self.tool_permissions.has(&tool_name) {
+                            self.tool_permissions.reset_tool(&tool_name);
+                            queue!(
+                                self.output,
+                                style::SetForegroundColor(Color::Green),
+                                style::Print(format!("\nReset tool '{}' to the default permission level.", tool_name)),
+                                style::SetForegroundColor(Color::Reset),
+                            )?;
+                        } else {
+                            queue!(
+                                self.output,
+                                style::SetForegroundColor(Color::Red),
+                                style::Print(format!(
+                                    "\nTool '{}' does not exist or is already in default settings.",
+                                    tool_name
+                                )),
+                                style::SetForegroundColor(Color::Reset),
+                            )?;
+                        }
                     },
                     Some(ToolsSubcommand::Help) => {
                         queue!(

crates/q_cli/src/cli/chat/tools/mod.rs

@@ -27,7 +27,6 @@ use fs_read::FsRead;
 use fs_write::FsWrite;
 use gh_issue::GhIssue;
 use serde::Deserialize;
-use tracing::warn;
 use use_aws::UseAws;
 
 use super::parser::ToolUse;
@@ -196,10 +195,6 @@ impl ToolPermissions {
     }
 
     pub fn reset_tool(&mut self, tool_name: &str) {
-        if !self.permissions.contains_key(tool_name) {
-            warn!("No custom permissions set for tool '{tool_name}' to reset");
-            return;
-        }
         self.permissions.remove(tool_name);
     }