Phase 7.2: CLI Command Implementation

Cargo.toml

@@ -121,16 +121,17 @@ unsafe_op_in_unsafe_fn = "deny"
 unused_lifetimes = "warn"
 
 [workspace.lints.clippy]
-# Note: Temporarily relaxed for Phase 7.1 - will be re-enabled in follow-up
-all = { level = "warn", priority = -1 }
-pedantic = { level = "warn", priority = -1 }
-cargo = { level = "warn", priority = -1 }
-nursery = { level = "warn", priority = -1 }
+# Deny all clippy warnings by default to enforce code quality in CI
+all = { level = "deny", priority = -1 }
+pedantic = { level = "deny", priority = -1 }
+cargo = { level = "deny", priority = -1 }
+nursery = { level = "deny", priority = -1 }
 
 # Allow specific lints that are too strict or have false positives
 needless_borrows_for_generic_args = { level = "allow", priority = 10 }
 multiple_crate_versions = { level = "allow", priority = 10 }           # Common with transitive dependencies
 cargo_common_metadata = { level = "allow", priority = 10 }             # Not required for internal workspace crates
+too_many_lines = { level = "allow", priority = 10 }                    # Allow comprehensive examples and tests
 
 [profile.release]
 opt-level = 3

crates/mcp-bridge/benches/cache_benchmark.rs

@@ -12,7 +12,7 @@ fn bench_cache_key_generation(c: &mut Criterion) {
     let mut group = c.benchmark_group("cache_key_generation");
 
     // Test with varying parameter sizes
-    for size in [10, 100, 1000, 10000].iter() {
+    for size in &[10, 100, 1000, 10000] {
         let params = "x".repeat(*size);
 
         group.throughput(Throughput::Bytes(*size as u64));
@@ -66,7 +66,7 @@ fn bench_cache_key_comparison(c: &mut Criterion) {
     });
 }
 
-/// Benchmarks cache key hashing for HashMap lookups
+/// Benchmarks cache key hashing for `HashMap` lookups
 fn bench_cache_key_hash(c: &mut Criterion) {
     use std::collections::hash_map::DefaultHasher;
     use std::hash::{Hash, Hasher};

crates/mcp-bridge/src/lib.rs

@@ -653,6 +653,7 @@ mod tests {
     }
 
     #[test]
+    #[allow(clippy::float_cmp)]
     fn test_cache_stats_usage_percent() {
         let stats = CacheStats {
             size: 50,
@@ -674,6 +675,7 @@ mod tests {
     }
 
     #[test]
+    #[allow(clippy::float_cmp)]
     fn test_cache_stats_zero_capacity() {
         let stats = CacheStats {
             size: 0,

crates/mcp-bridge/tests/integration_test.rs

@@ -23,6 +23,7 @@ async fn test_bridge_creation() {
 
 /// Tests cache statistics tracking
 #[tokio::test]
+#[allow(clippy::float_cmp)]
 async fn test_cache_statistics() {
     let bridge = Bridge::new(100);
 
@@ -92,8 +93,9 @@ async fn test_cache_clearing() {
     assert_eq!(stats.size, 0);
 }
 
-/// Tests CacheStats helper methods
+/// Tests `CacheStats` helper methods
 #[test]
+#[allow(clippy::float_cmp)]
 fn test_cache_stats_methods() {
     let empty = CacheStats {
         size: 0,

crates/mcp-cli/Cargo.toml

@@ -32,7 +32,7 @@ serde.workspace = true
 serde_json.workspace = true
 
 # Async runtime
-tokio = { workspace = true, features = ["rt-multi-thread", "macros"] }
+tokio = { workspace = true, features = ["rt-multi-thread", "macros", "fs"] }
 
 # Logging
 tracing.workspace = true
@@ -49,3 +49,7 @@ human-panic.workspace = true
 # Configuration (Phase 7.5 - for future use)
 toml.workspace = true
 dirs.workspace = true
+
+[dev-dependencies]
+wat.workspace = true
+tempfile.workspace = true