refactor: extract admin keys validation into helper function

Author: AndreaDiazCorreia

src/cli/take_dispute.rs

@@ -1,5 +1,6 @@
 use anyhow::Result;
 use mostro_core::prelude::*;
+use nostr_sdk::prelude::Keys;
 use uuid::Uuid;
 
 use crate::{
@@ -9,6 +10,19 @@ use crate::{
     util::{admin_send_dm, send_dm, wait_for_dm},
 };
 
+/// Helper function to retrieve and validate admin keys from context
+fn get_admin_keys(ctx: &Context) -> Result<&Keys> {
+    let admin_keys = ctx.context_keys.as_ref()
+        .ok_or_else(|| anyhow::anyhow!("Admin keys not available. ADMIN_NSEC must be set for admin commands."))?;
+    
+    // Only log admin public key in verbose mode
+    if std::env::var("RUST_LOG").is_ok() {
+        println!("🔑 Admin Keys: {}", admin_keys.public_key);
+    }
+    
+    Ok(admin_keys)
+}
+
 pub async fn execute_admin_add_solver(npubkey: &str, ctx: &Context) -> Result<()> {
     println!("👑 Admin Add Solver");
     println!("═══════════════════════════════════════");
@@ -23,9 +37,7 @@ pub async fn execute_admin_add_solver(npubkey: &str, ctx: &Context) -> Result<()
     println!("{table}");
     println!("💡 Adding new solver to Mostro...\n");
 
-    let admin_keys = ctx.context_keys.as_ref()
-        .ok_or_else(|| anyhow::anyhow!("Admin keys not available. ADMIN_NSEC must be set for admin commands."))?;
-    println!("🔑 Admin Keys: {}", admin_keys.public_key);
+    let _admin_keys = get_admin_keys(ctx)?;
 
     // Create takebuy message
     let take_dispute_message = Message::new_dispute(
@@ -62,16 +74,15 @@ pub async fn execute_admin_cancel_dispute(dispute_id: &Uuid, ctx: &Context) -> R
     ));
     println!("{table}");
     println!("💡 Canceling dispute...\n");
+
+    let _admin_keys = get_admin_keys(ctx)?;
+
     // Create takebuy message
     let take_dispute_message =
         Message::new_dispute(Some(*dispute_id), None, None, Action::AdminCancel, None)
             .as_json()
             .map_err(|_| anyhow::anyhow!("Failed to serialize message"))?;
 
-    let admin_keys = ctx.context_keys.as_ref()
-        .ok_or_else(|| anyhow::anyhow!("Admin keys not available. ADMIN_NSEC must be set for admin commands."))?;
-    println!("🔑 Admin Keys: {}", admin_keys.public_key);
-
     admin_send_dm(ctx, take_dispute_message).await?;
 
     println!("✅ Dispute canceled successfully!");
@@ -96,15 +107,14 @@ pub async fn execute_admin_settle_dispute(dispute_id: &Uuid, ctx: &Context) -> R
     ));
     println!("{table}");
     println!("💡 Settling dispute...\n");
+
+    let _admin_keys = get_admin_keys(ctx)?;
+
     // Create takebuy message
     let take_dispute_message =
         Message::new_dispute(Some(*dispute_id), None, None, Action::AdminSettle, None)
             .as_json()
             .map_err(|_| anyhow::anyhow!("Failed to serialize message"))?;
-
-    let admin_keys = ctx.context_keys.as_ref()
-        .ok_or_else(|| anyhow::anyhow!("Admin keys not available. ADMIN_NSEC must be set for admin commands."))?;
-    println!("🔑 Admin Keys: {}", admin_keys.public_key);
     admin_send_dm(ctx, take_dispute_message).await?;
 
     println!("✅ Dispute settled successfully!");
@@ -128,6 +138,9 @@ pub async fn execute_take_dispute(dispute_id: &Uuid, ctx: &Context) -> Result<()
     ));
     println!("{table}");
     println!("💡 Taking dispute...\n");
+
+    let admin_keys = get_admin_keys(ctx)?;
+
     // Create takebuy message
     let take_dispute_message = Message::new_dispute(
         Some(*dispute_id),
@@ -139,10 +152,6 @@ pub async fn execute_take_dispute(dispute_id: &Uuid, ctx: &Context) -> Result<()
     .as_json()
     .map_err(|_| anyhow::anyhow!("Failed to serialize message"))?;
 
-    let admin_keys = ctx.context_keys.as_ref()
-        .ok_or_else(|| anyhow::anyhow!("Admin keys not available. ADMIN_NSEC must be set for admin commands."))?;
-    println!("🔑 Admin Keys: {}", admin_keys.public_key);
-
     // Send the dispute message and wait for response
     let sent_message = send_dm(
         &ctx.client,