feat: add --config-file flag for alternate configuration file (#430)

- Add --config-file global flag and REDISCTL_CONFIG_FILE environment variable
- Add Config::load_from_path() and Config::save_to_path() methods
- Update ConnectionManager to track and use alternate config path
- Update all profile commands to save to the correct location
- Tested with both flag and environment variable

Closes #428

Author: joshrotenberg

crates/redisctl-config/src/config.rs

@@ -9,7 +9,7 @@ use directories::ProjectDirs;
 use serde::{Deserialize, Serialize};
 use std::collections::HashMap;
 use std::fs;
-use std::path::PathBuf;
+use std::path::{Path, PathBuf};
 
 use crate::credential::CredentialStore;
 use crate::error::{ConfigError, Result};
@@ -306,12 +306,16 @@ impl Config {
     /// Load configuration from the standard location
     pub fn load() -> Result<Self> {
         let config_path = Self::config_path()?;
+        Self::load_from_path(&config_path)
+    }
 
+    /// Load configuration from a specific path
+    pub fn load_from_path(config_path: &Path) -> Result<Self> {
         if !config_path.exists() {
             return Ok(Config::default());
         }
 
-        let content = fs::read_to_string(&config_path).map_err(|e| ConfigError::LoadError {
+        let content = fs::read_to_string(config_path).map_err(|e| ConfigError::LoadError {
             path: config_path.display().to_string(),
             source: e,
         })?;
@@ -327,7 +331,11 @@ impl Config {
     /// Save configuration to the standard location
     pub fn save(&self) -> Result<()> {
         let config_path = Self::config_path()?;
+        self.save_to_path(&config_path)
+    }
 
+    /// Save configuration to a specific path
+    pub fn save_to_path(&self, config_path: &Path) -> Result<()> {
         // Create parent directories if they don't exist
         if let Some(parent) = config_path.parent() {
             fs::create_dir_all(parent).map_err(|e| ConfigError::SaveError {
@@ -338,7 +346,7 @@ impl Config {
 
         let content = toml::to_string_pretty(self)?;
 
-        fs::write(&config_path, content).map_err(|e| ConfigError::SaveError {
+        fs::write(config_path, content).map_err(|e| ConfigError::SaveError {
             path: config_path.display().to_string(),
             source: e,
         })?;

crates/redisctl/src/cli.rs

@@ -47,6 +47,10 @@ pub struct Cli {
     #[arg(long, short, global = true, env = "REDISCTL_PROFILE")]
     pub profile: Option<String>,
 
+    /// Path to alternate configuration file
+    #[arg(long, global = true, env = "REDISCTL_CONFIG_FILE")]
+    pub config_file: Option<String>,
+
     /// Output format
     #[arg(long, short = 'o', global = true, value_enum, default_value = "auto")]
     pub output: OutputFormat,

crates/redisctl/src/commands/profile.rs

@@ -68,9 +68,15 @@ async fn handle_list(
 
     match output_format {
         OutputFormat::Json | OutputFormat::Yaml => {
-            let config_path = Config::config_path()
-                .ok()
-                .and_then(|p| p.to_str().map(String::from));
+            let config_path = conn_mgr
+                .config_path
+                .as_ref()
+                .map(|p| p.to_string_lossy().to_string())
+                .or_else(|| {
+                    Config::config_path()
+                        .ok()
+                        .and_then(|p| p.to_str().map(String::from))
+                });
 
             let profile_list: Vec<serde_json::Value> = profiles
                 .iter()
@@ -123,7 +129,10 @@ async fn handle_list(
         }
         _ => {
             // Show config file path at the top
-            if let Ok(config_path) = Config::config_path() {
+            if let Some(ref path) = conn_mgr.config_path {
+                println!("Configuration file: {}", path.display());
+                println!();
+            } else if let Ok(config_path) = Config::config_path() {
                 println!("Configuration file: {}", config_path.display());
                 println!();
             }
@@ -443,14 +452,21 @@ async fn handle_set(
     let mut config = conn_mgr.config.clone();
     config.profiles.insert(name.to_string(), profile);
 
-    // Save the configuration
-    config.save().context("Failed to save configuration")?;
-
-    if let Ok(config_path) = Config::config_path() {
+    // Save the configuration to the appropriate location
+    if let Some(ref path) = conn_mgr.config_path {
+        config
+            .save_to_path(path)
+            .context("Failed to save configuration")?;
         println!("Profile '{}' saved successfully to:", name);
-        println!("  {}", config_path.display());
+        println!("  {}", path.display());
     } else {
-        println!("Profile '{}' saved successfully.", name);
+        config.save().context("Failed to save configuration")?;
+        if let Ok(config_path) = Config::config_path() {
+            println!("Profile '{}' saved successfully to:", name);
+            println!("  {}", config_path.display());
+        } else {
+            println!("Profile '{}' saved successfully.", name);
+        }
     }
 
     // Suggest setting as default if it's the only profile of its type
@@ -527,8 +543,14 @@ async fn handle_remove(conn_mgr: &ConnectionManager, name: &str) -> Result<(), R
         println!("Default cloud profile cleared.");
     }
 
-    // Save the configuration
-    config.save().context("Failed to save configuration")?;
+    // Save the configuration to the appropriate location
+    if let Some(ref path) = conn_mgr.config_path {
+        config
+            .save_to_path(path)
+            .context("Failed to save configuration")?;
+    } else {
+        config.save().context("Failed to save configuration")?;
+    }
 
     println!("Profile '{}' removed successfully.", name);
     Ok(())
@@ -558,8 +580,14 @@ async fn handle_default_enterprise(
     let mut config = conn_mgr.config.clone();
     config.default_enterprise = Some(name.to_string());
 
-    // Save the configuration
-    config.save().context("Failed to save configuration")?;
+    // Save the configuration to the appropriate location
+    if let Some(ref path) = conn_mgr.config_path {
+        config
+            .save_to_path(path)
+            .context("Failed to save configuration")?;
+    } else {
+        config.save().context("Failed to save configuration")?;
+    }
 
     println!("Default enterprise profile set to '{}'.", name);
     Ok(())
@@ -589,8 +617,14 @@ async fn handle_default_cloud(
     let mut config = conn_mgr.config.clone();
     config.default_cloud = Some(name.to_string());
 
-    // Save the configuration
-    config.save().context("Failed to save configuration")?;
+    // Save the configuration to the appropriate location
+    if let Some(ref path) = conn_mgr.config_path {
+        config
+            .save_to_path(path)
+            .context("Failed to save configuration")?;
+    } else {
+        config.save().context("Failed to save configuration")?;
+    }
 
     println!("Default cloud profile set to '{}'.", name);
     Ok(())

crates/redisctl/src/connection.rs

@@ -10,13 +10,39 @@ use tracing::{debug, info, trace};
 #[derive(Clone)]
 pub struct ConnectionManager {
     pub config: Config,
+    pub config_path: Option<std::path::PathBuf>,
 }
 
 impl ConnectionManager {
     /// Create a new connection manager with the given configuration
     #[allow(dead_code)] // Used by binary target
     pub fn new(config: Config) -> Self {
-        Self { config }
+        Self {
+            config,
+            config_path: None,
+        }
+    }
+
+    /// Create a new connection manager with a custom config path
+    #[allow(dead_code)] // Used by binary target
+    pub fn with_config_path(config: Config, config_path: Option<std::path::PathBuf>) -> Self {
+        Self {
+            config,
+            config_path,
+        }
+    }
+
+    /// Save the configuration to the appropriate location
+    #[allow(dead_code)] // Used by binary target
+    pub fn save_config(&self) -> CliResult<()> {
+        if let Some(ref path) = self.config_path {
+            self.config
+                .save_to_path(path)
+                .context("Failed to save configuration")?;
+        } else {
+            self.config.save().context("Failed to save configuration")?;
+        }
+        Ok(())
     }
 
     /// Create a Cloud client from profile credentials with environment variable override support

crates/redisctl/src/main.rs

@@ -23,9 +23,15 @@ async fn main() -> Result<()> {
     // Initialize tracing based on verbosity level
     init_tracing(cli.verbose);
 
-    // Load configuration
-    let config = Config::load()?;
-    let conn_mgr = ConnectionManager::new(config);
+    // Load configuration from specified path or default location
+    let (config, config_path) = if let Some(config_file) = &cli.config_file {
+        let path = std::path::PathBuf::from(config_file);
+        let config = Config::load_from_path(&path)?;
+        (config, Some(path))
+    } else {
+        (Config::load()?, None)
+    };
+    let conn_mgr = ConnectionManager::with_config_path(config, config_path);
 
     // Execute command
     if let Err(e) = execute_command(&cli, &conn_mgr).await {