Merge pull request #138 from GCWing/gcwing/dev

feat(config): enable tool auto-execution by default

Author: GCWing

src/crates/core/src/agentic/tools/implementations/delete_file_tool.rs

@@ -100,7 +100,7 @@ Important notes:
     }
 
     fn needs_permissions(&self, _input: Option<&Value>) -> bool {
-        true
+        false
     }
 
     async fn validate_input(&self, input: &Value, _context: Option<&ToolUseContext>) -> ValidationResult {

src/crates/core/src/agentic/tools/implementations/file_write_tool.rs

@@ -61,7 +61,7 @@ Usage:
     }
 
     fn needs_permissions(&self, _input: Option<&Value>) -> bool {
-        true
+        false
     }
 
     async fn validate_input(

src/crates/core/src/service/config/types.rs

@@ -404,7 +404,7 @@ pub struct AIConfig {
     pub tool_confirmation_timeout_secs: Option<u64>,
 
     /// Skip tool execution confirmation (global, applies to all modes).
-    #[serde(default)]
+    #[serde(default = "default_skip_tool_confirmation")]
     pub skip_tool_confirmation: bool,
 
     /// Debug-mode configuration (log path, language templates, etc.).
@@ -453,6 +453,10 @@ fn default_tool_confirmation_timeout() -> Option<u64> {
     None
 }
 
+fn default_skip_tool_confirmation() -> bool {
+    true
+}
+
 impl Default for ModeConfig {
     fn default() -> Self {
         Self {
@@ -1114,7 +1118,7 @@ impl Default for AIConfig {
             proxy: ProxyConfig::default(),
             tool_execution_timeout_secs: default_tool_execution_timeout(),
             tool_confirmation_timeout_secs: default_tool_confirmation_timeout(),
-            skip_tool_confirmation: false,
+            skip_tool_confirmation: true,
             debug_mode_config: DebugModeConfig::default(),
             known_tools: Vec::new(),
         }