Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -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 {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ Usage:
}

fn needs_permissions(&self, _input: Option<&Value>) -> bool {
true
false
}

async fn validate_input(
Expand Down
8 changes: 6 additions & 2 deletions src/crates/core/src/service/config/types.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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.).
Expand Down Expand Up @@ -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 {
Expand Down Expand Up @@ -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(),
}
Expand Down
Loading