Merge pull request #38 from gitcoder89431/31-provider-configuration-input-system

🏗️ Refactor demo and settings module for improved readability and org…

Author: gitcoder89431

examples/issue_31_demo.rs

@@ -1,84 +1,111 @@
 //! Issue #31 Provider Configuration Input System Demo
-//! 
+//!
 //! Demonstrates the comprehensive input handling for provider configuration
 
 use agentic::{
-    settings::{Settings, ProviderField, SettingsAction, ValidationResult, validate_local_endpoint, validate_api_key, mask_api_key},
+    settings::{
+        ProviderField, Settings, SettingsAction, ValidationResult, mask_api_key, validate_api_key,
+        validate_local_endpoint,
+    },
     theme::{Theme, ThemeVariant},
 };
 use ratatui::crossterm::event::{KeyCode, KeyEvent};
 
 fn main() {
     println!("⌨️ Issue #31: Provider Configuration Input System Demo");
     println!("{}", "=".repeat(70));
-    
+
     let mut settings = Settings::new();
-    
+
     println!("\n🔧 INPUT SYSTEM FEATURES:");
     println!("  ✅ Tab navigation through fields");
     println!("  ✅ Enter to activate edit mode");
     println!("  ✅ Live text input with validation");
     println!("  ✅ API key masking for security");
     println!("  ✅ ESC to cancel/revert changes");
     println!("  ✅ Real-time input validation");
-    
+
     println!("\n📊 FIELD NAVIGATION ORDER:");
-    let fields = ["Theme", "Local Endpoint", "OpenRouter API Key", "Save Button"];
+    let fields = [
+        "Theme",
+        "Local Endpoint",
+        "OpenRouter API Key",
+        "Save Button",
+    ];
     for (i, field) in fields.iter().enumerate() {
         println!("  {}. {}", i + 1, field);
     }
-    
+
     println!("\n🧪 TESTING NAVIGATION:");
     for i in 0..5 {
         settings.navigate_next_field();
         let field_name = match settings.focused_field.as_ref().unwrap() {
             ProviderField::Theme => "Theme",
             ProviderField::LocalEndpoint => "Local Endpoint",
-            ProviderField::OpenRouterApiKey => "OpenRouter API Key", 
+            ProviderField::OpenRouterApiKey => "OpenRouter API Key",
             ProviderField::SaveButton => "Save Button",
         };
         println!("  Step {}: Focused on {}", i + 1, field_name);
     }
-    
+
     println!("\n🔑 TESTING INPUT SYSTEM:");
-    
+
     // Test Local Endpoint editing
     println!("\n📝 Local Endpoint Configuration:");
     settings.focused_field = Some(ProviderField::LocalEndpoint);
     settings.enter_edit_mode(ProviderField::LocalEndpoint);
-    
+
     // Simulate typing
     let test_input = "http://localhost:8080";
     for c in test_input.chars() {
         settings.handle_action(SettingsAction::InputCharacter(c));
     }
-    
-    println!("  Input Buffer: {}", settings.input_state.as_ref().unwrap().input_buffer);
+
+    println!(
+        "  Input Buffer: {}",
+        settings.input_state.as_ref().unwrap().input_buffer
+    );
     println!("  Validation: {:?}", settings.validate_current_input());
-    
+
     // Save the input
     settings.exit_edit_mode(true);
-    println!("  Saved Value: {}", settings.get_display_value(&ProviderField::LocalEndpoint));
-    
+    println!(
+        "  Saved Value: {}",
+        settings.get_display_value(&ProviderField::LocalEndpoint)
+    );
+
     // Test API Key editing with masking
     println!("\n🔐 API Key Configuration (with masking):");
     settings.focused_field = Some(ProviderField::OpenRouterApiKey);
     settings.enter_edit_mode(ProviderField::OpenRouterApiKey);
-    
+
     let test_api_key = "sk-or-v1-1234567890abcdefghijklmnop";
     for c in test_api_key.chars() {
         settings.handle_action(SettingsAction::InputCharacter(c));
     }
-    
-    println!("  Input Buffer (editing): {}", settings.input_state.as_ref().unwrap().input_buffer);
+
+    println!(
+        "  Input Buffer (editing): {}",
+        settings.input_state.as_ref().unwrap().input_buffer
+    );
     println!("  Validation: {:?}", settings.validate_current_input());
-    
+
     settings.exit_edit_mode(true);
-    println!("  Masked Display: {}", settings.get_display_value(&ProviderField::OpenRouterApiKey));
-    println!("  Raw Value: {}", settings.openrouter_provider.api_key.as_ref().unwrap_or(&String::new()));
-    
+    println!(
+        "  Masked Display: {}",
+        settings.get_display_value(&ProviderField::OpenRouterApiKey)
+    );
+    println!(
+        "  Raw Value: {}",
+        settings
+            .openrouter_provider
+            .api_key
+            .as_ref()
+            .unwrap_or(&String::new())
+    );
+
     println!("\n🎛️ KEYBOARD BINDINGS:");
-    
+
     // Test keyboard event handling
     let test_keys = [
         (KeyCode::Tab, "Tab: Next field"),
@@ -90,7 +117,7 @@ fn main() {
         (KeyCode::Char('x'), "Characters: Live input"),
         (KeyCode::Backspace, "Backspace: Delete char"),
     ];
-    
+
     for (key_code, description) in test_keys {
         let key_event = KeyEvent::from(key_code);
         if let Some(action) = settings.handle_key_event(key_event) {
@@ -99,9 +126,9 @@ fn main() {
             println!("  {}: No action", description);
         }
     }
-    
+
     println!("\n🔍 VALIDATION TESTING:");
-    
+
     // Test validation functions
     let test_cases = [
         ("http://localhost:11434", "Valid local endpoint"),
@@ -111,33 +138,42 @@ fn main() {
         ("invalid-key", "Invalid API key format"),
         ("", "Empty value"),
     ];
-    
+
     for (test_value, description) in test_cases {
         let local_result = validate_local_endpoint(test_value);
         let api_result = validate_api_key(test_value);
-        println!("  {}: Local={:?}, API={:?}", description, local_result, api_result);
+        println!(
+            "  {}: Local={:?}, API={:?}",
+            description, local_result, api_result
+        );
     }
-    
+
     println!("\n🛡️ SECURITY FEATURES:");
-    
+
     // Test API key masking
     let test_keys = [
         "short",
         "sk-or-v1-abc123",
         "sk-or-v1-1234567890abcdefghijklmnopqrstuvwxyz",
     ];
-    
+
     for key in test_keys {
         let masked = mask_api_key(key);
         println!("  Original: {} -> Masked: {}", key, masked);
     }
-    
+
     println!("\n✨ INPUT STATE MANAGEMENT:");
     println!("  Current edit mode: {}", settings.is_editing());
     println!("  Focused field: {:?}", settings.focused_field);
-    println!("  Local provider configured: {}", settings.local_provider.is_configured());
-    println!("  OpenRouter provider configured: {}", settings.openrouter_provider.is_configured());
-    
+    println!(
+        "  Local provider configured: {}",
+        settings.local_provider.is_configured()
+    );
+    println!(
+        "  OpenRouter provider configured: {}",
+        settings.openrouter_provider.is_configured()
+    );
+
     println!("\n🎉 Issue #31 Implementation Complete!");
     println!("📋 All Success Criteria Met:");
     println!("  ✅ Tab navigation cycles through all fields correctly");