fix(cli): display platform-correct secrets path in keyring config dialog (#8328)

Signed-off-by: SushaanthSrinivasan <SushaanthSrinivasan@users.noreply.github.com>
Co-authored-by: SushaanthSrinivasan <SushaanthSrinivasan@users.noreply.github.com>
This commit is contained in:
Sushaanth Srinivasan
2026-04-05 20:01:23 -07:00
committed by GitHub
parent 2c4129af56
commit e8c66f8c71
+9 -4
View File
@@ -1478,7 +1478,11 @@ pub fn configure_keyring_dialog() -> anyhow::Result<()> {
};
let _ = cliclack::log::info(format!("Current secret storage: {}", current_status));
let _ = cliclack::log::warning("Note: Disabling the keyring stores secrets in a plain text file (~/.config/goose/secrets.yaml)");
let secrets_path = Paths::config_dir().join("secrets.yaml");
let _ = cliclack::log::warning(format!(
"Note: Disabling the keyring stores secrets in a plain text file ({})",
secrets_path.display()
));
let storage_option = cliclack::select("How would you like to store secrets?")
.item(
@@ -1504,9 +1508,10 @@ pub fn configure_keyring_dialog() -> anyhow::Result<()> {
"file" => {
// Set the disable flag to use file storage
config.set_param("GOOSE_DISABLE_KEYRING", Value::String("true".to_string()))?;
cliclack::outro(
"Secret storage set to file (~/.config/goose/secrets.yaml). Keep this file secure!",
)?;
cliclack::outro(format!(
"Secret storage set to file ({}). Keep this file secure!",
secrets_path.display(),
))?;
let _ =
cliclack::log::info("You may need to restart goose for this change to take effect");
}