Fix Claude Code provider to default to Auto mode (#5638) (#5642)

This commit is contained in:
Simon Sickle
2025-11-08 16:58:54 -05:00
committed by GitHub
parent e5a1474b4c
commit bb80d76289
+7 -8
View File
@@ -112,14 +112,16 @@ impl ClaudeCodeProvider {
/// Parse the JSON response from claude CLI /// Parse the JSON response from claude CLI
fn apply_permission_flags(cmd: &mut Command) -> Result<(), ProviderError> { fn apply_permission_flags(cmd: &mut Command) -> Result<(), ProviderError> {
let config = Config::global(); let config = Config::global();
match config.get_goose_mode() { let goose_mode = config.get_goose_mode().unwrap_or(GooseMode::Auto);
Ok(GooseMode::Auto) => {
match goose_mode {
GooseMode::Auto => {
cmd.arg("--dangerously-skip-permissions"); cmd.arg("--dangerously-skip-permissions");
} }
Ok(GooseMode::SmartApprove) => { GooseMode::SmartApprove => {
cmd.arg("--permission-mode").arg("acceptEdits"); cmd.arg("--permission-mode").arg("acceptEdits");
} }
Ok(GooseMode::Approve) => { GooseMode::Approve => {
return Err(ProviderError::RequestFailed( return Err(ProviderError::RequestFailed(
"\n\n\n### NOTE\n\n\n \ "\n\n\n### NOTE\n\n\n \
Claude Code CLI provider does not support Approve mode.\n \ Claude Code CLI provider does not support Approve mode.\n \
@@ -128,12 +130,9 @@ impl ClaudeCodeProvider {
.to_string(), .to_string(),
)); ));
} }
Ok(GooseMode::Chat) => { GooseMode::Chat => {
// Chat mode doesn't need permission flags // Chat mode doesn't need permission flags
} }
Err(_) => {
// Default behavior if mode is not set
}
} }
Ok(()) Ok(())
} }