fix: apply clippy fixes for test files and CI compliance

Resolves all clippy warnings in test files and unexpected cfg checks.

## Changes

### Test Files Fixed
1. **tests/agent_chat_e2e_tests.rs**
   - Changed `config` to `_config` to mark as intentionally unused
   - Removed needless borrow on args array: `&["run", ...]` → `["run", ...]`
   - Fixed const assertion: `!example_usage.is_empty()` → `contains("osvm chat")`

2. **tests/test_100_microvms.rs**
   - Fixed redundant pattern matching: `while let Some(_) = ...` → `while (...).is_some()`

3. **tests/ui_test_utils.rs**
   - Added `#[allow(dead_code)]` to `cursive()` method (unused but useful for future tests)

### Global Fixes
4. **src/lib.rs**
   - Added `#![allow(unexpected_cfgs)]` to suppress warnings for `incomplete_tests` feature flag used in multiple test files

## Clippy Status
- ✅ All lib and bin targets pass with `-D warnings`
- ✅ No clippy warnings in production code
- ✅ Test files cleaned up for CI compliance

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Author: 0xrinegade

src/lib.rs

@@ -21,6 +21,7 @@
 // Allow clippy warnings for this codebase since it's under active development
 #![allow(clippy::all)]
 #![allow(unused)]
+#![allow(unexpected_cfgs)]
 
 pub mod clparse;
 pub mod commands;

tests/agent_chat_e2e_tests.rs

@@ -33,7 +33,7 @@ struct TuiInteractor {
     process: std::process::Child,
     stdin: std::process::ChildStdin,
     stdout_reader: std::thread::JoinHandle<Result<String>>,
-    config: TuiTestConfig,
+    _config: TuiTestConfig,
 }
 
 impl TuiInteractor {
@@ -43,7 +43,7 @@ impl TuiInteractor {
 
         // Set environment variables for testing
         let mut cmd = StdCommand::new("cargo");
-        cmd.args(&["run", "--", "chat"])
+        cmd.args(["run", "--", "chat"])
             .current_dir(std::env::current_dir()?)
             .env("TERM", "xterm-256color")
             .env("COLUMNS", config.terminal_size.0.to_string())
@@ -76,7 +76,7 @@ impl TuiInteractor {
             process,
             stdin,
             stdout_reader,
-            config,
+            _config: config,
         })
     }
 
@@ -633,7 +633,7 @@ fn test_documentation_examples() {
     "#;
 
     // Validate that our implementation matches the documented behavior
-    assert!(!example_usage.is_empty());
+    assert!(example_usage.contains("osvm chat"));
 }
 
 /// Benchmark test to ensure TUI performance is acceptable

tests/test_100_microvms.rs

@@ -164,7 +164,7 @@ async fn test_launch_100_concurrent_microvms() -> Result<()> {
         });
     }
 
-    while let Some(_) = shutdown_tasks.join_next().await {}
+    while (shutdown_tasks.join_next().await).is_some() {}
 
     let shutdown_duration = shutdown_start.elapsed();
     println!(

tests/ui_test_utils.rs

@@ -213,6 +213,7 @@ impl HeadlessUI {
     }
 
     /// Access cursive instance directly
+    #[allow(dead_code)]
     pub fn cursive(&mut self) -> &mut Cursive {
         &mut self.siv
     }