From e8c66f8c7158c081f24c241f9b3eb00ff36d67e5 Mon Sep 17 00:00:00 2001 From: Sushaanth Srinivasan <80441330+SushaanthSrinivasan@users.noreply.github.com> Date: Sun, 5 Apr 2026 20:01:23 -0700 Subject: [PATCH] fix(cli): display platform-correct secrets path in keyring config dialog (#8328) Signed-off-by: SushaanthSrinivasan Co-authored-by: SushaanthSrinivasan --- crates/goose-cli/src/commands/configure.rs | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/crates/goose-cli/src/commands/configure.rs b/crates/goose-cli/src/commands/configure.rs index 585427c9..5788baba 100644 --- a/crates/goose-cli/src/commands/configure.rs +++ b/crates/goose-cli/src/commands/configure.rs @@ -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"); }