feat(enterprise): add DNS suffix management commands (#161) (#295)

- Implement suffix list command to list all DNS suffixes
- Add suffix get command to get specific suffix details
- Support JMESPath query filtering for output
- Add comprehensive mdBook documentation
- Include unit tests for command parsing

Author: joshrotenberg

crates/redisctl/src/cli.rs

@@ -1051,6 +1051,10 @@ pub enum EnterpriseCommands {
     #[command(subcommand)]
     Stats(EnterpriseStatsCommands),
 
+    /// DNS suffix management
+    #[command(subcommand)]
+    Suffix(crate::commands::enterprise::suffix::SuffixCommands),
+
     /// Usage report operations
     #[command(subcommand, name = "usage-report")]
     UsageReport(crate::commands::enterprise::usage_report::UsageReportCommands),

crates/redisctl/src/commands/enterprise/mod.rs

@@ -26,5 +26,6 @@ pub mod rbac;
 pub mod rbac_impl;
 pub mod shard;
 pub mod stats;
+pub mod suffix;
 pub mod usage_report;
 pub mod utils;

crates/redisctl/src/commands/enterprise/suffix.rs

@@ -0,0 +1,114 @@
+use anyhow::Context;
+use clap::Subcommand;
+
+use crate::{cli::OutputFormat, connection::ConnectionManager, error::Result as CliResult};
+
+#[allow(dead_code)]
+pub async fn handle_suffix_command(
+    conn_mgr: &ConnectionManager,
+    profile_name: Option<&str>,
+    suffix_cmd: SuffixCommands,
+    output_format: OutputFormat,
+    query: Option<&str>,
+) -> CliResult<()> {
+    suffix_cmd
+        .execute(conn_mgr, profile_name, output_format, query)
+        .await
+}
+
+#[derive(Debug, Clone, Subcommand)]
+pub enum SuffixCommands {
+    /// List all DNS suffixes
+    List,
+
+    /// Get a specific DNS suffix by name
+    Get {
+        /// DNS suffix name
+        name: String,
+    },
+}
+
+impl SuffixCommands {
+    #[allow(dead_code)]
+    pub async fn execute(
+        &self,
+        conn_mgr: &ConnectionManager,
+        profile_name: Option<&str>,
+        output_format: OutputFormat,
+        query: Option<&str>,
+    ) -> CliResult<()> {
+        handle_suffix_command_impl(conn_mgr, profile_name, self, output_format, query).await
+    }
+}
+
+#[allow(dead_code)]
+async fn handle_suffix_command_impl(
+    conn_mgr: &ConnectionManager,
+    profile_name: Option<&str>,
+    command: &SuffixCommands,
+    output_format: OutputFormat,
+    query: Option<&str>,
+) -> CliResult<()> {
+    let client = conn_mgr.create_enterprise_client(profile_name).await?;
+
+    match command {
+        SuffixCommands::List => {
+            let response: serde_json::Value = client
+                .get("/v1/suffixes")
+                .await
+                .context("Failed to list DNS suffixes")?;
+
+            let output_data = if let Some(q) = query {
+                super::utils::apply_jmespath(&response, q)?
+            } else {
+                response
+            };
+
+            super::utils::print_formatted_output(output_data, output_format)?;
+        }
+        SuffixCommands::Get { name } => {
+            let response: serde_json::Value = client
+                .get(&format!("/v1/suffix/{}", name))
+                .await
+                .context(format!("Failed to get DNS suffix '{}'", name))?;
+
+            let output_data = if let Some(q) = query {
+                super::utils::apply_jmespath(&response, q)?
+            } else {
+                response
+            };
+
+            super::utils::print_formatted_output(output_data, output_format)?;
+        }
+    }
+
+    Ok(())
+}
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+
+    #[test]
+    fn test_suffix_command_parsing() {
+        use clap::Parser;
+
+        #[derive(Parser)]
+        struct TestCli {
+            #[command(subcommand)]
+            cmd: SuffixCommands,
+        }
+
+        // Test list command
+        let cli = TestCli::parse_from(["test", "list"]);
+        assert!(matches!(cli.cmd, SuffixCommands::List));
+
+        // Test get command
+        let cli = TestCli::parse_from(["test", "get", "example.redis.local"]);
+        if let SuffixCommands::Get { name } = cli.cmd {
+            assert_eq!(name, "example.redis.local");
+        } else {
+            panic!("Expected Get command");
+        }
+    }
+}

crates/redisctl/src/main.rs

@@ -375,6 +375,16 @@ async fn execute_enterprise_command(
             )
             .await
         }
+        Suffix(suffix_cmd) => {
+            commands::enterprise::suffix::handle_suffix_command(
+                conn_mgr,
+                profile,
+                suffix_cmd.clone(),
+                output,
+                query,
+            )
+            .await
+        }
         UsageReport(usage_report_cmd) => {
             commands::enterprise::usage_report::handle_usage_report_command(
                 conn_mgr,

docs/src/SUMMARY.md

@@ -47,6 +47,7 @@
 - [Endpoints](./enterprise/endpoints.md)
 - [Job Scheduler](./enterprise/job-scheduler.md)
 - [JSON Schema](./enterprise/jsonschema.md)
+- [DNS Suffixes](./enterprise/suffix.md)
 - [Workflows](./enterprise/workflows.md)
 - [Raw API Access](./enterprise/api-access.md)