aljaz/goose
1
0
Fork 0
mirror of https://github.com/aljazceru/goose.git synced 2025-12-19 07:04:21 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix: write to config yaml if provider test succeeds (#1178)

Browse Source
This commit is contained in:
Salman Mohammed
2025-02-11 09:52:45 -05:00
committed by GitHub
parent 9bb3f8de05
commit fdcf374462
1 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
crates/goose-cli/src/commands/configure.rs
View File

@@ -283,10 +283,6 @@ pub async fn configure_provider_dialog() -> Result<bool, Box<dyn Error>> {
.default_input(&default_model) .default_input(&default_model)
.interact()?; .interact()?;
// Update config with new values
config.set("GOOSE_PROVIDER", Value::String(provider_name.to_string()))?;
config.set("GOOSE_MODEL", Value::String(model.clone()))?;
// Test the configuration // Test the configuration
let spin = spinner(); let spin = spinner();
spin.start("Checking your configuration..."); spin.start("Checking your configuration...");
@@ -319,6 +315,9 @@ pub async fn configure_provider_dialog() -> Result<bool, Box<dyn Error>> {
match result { match result {
Ok((_message, _usage)) => { Ok((_message, _usage)) => {
// Update config with new values only if the test succeeds
config.set("GOOSE_PROVIDER", Value::String(provider_name.to_string()))?;
config.set("GOOSE_MODEL", Value::String(model.clone()))?;
cliclack::outro("Configuration saved successfully")?; cliclack::outro("Configuration saved successfully")?;
Ok(true) Ok(true)
} }