style(clippy): fix test file lints (zombie_processes, empty_line_after_doc_comment, useless_format, unused_variable) (#674)

Author: github-actions[bot]

tests/compiler_tests.rs

@@ -43,7 +43,7 @@ This is a test agent for integration testing.
     fs::create_dir_all(temp_dir.join(".github/agents")).expect("Failed to create .github/agents");
 
     // Run the compilation
-    let output_yaml = temp_dir.join("test-agent.yml");
+    let _output_yaml = temp_dir.join("test-agent.yml");
 
     // Note: We can't directly call compile_pipeline from here since it's not a library function
     // This test verifies the output structure when compile runs
@@ -3730,7 +3730,7 @@ fn assert_marker_step_present(
         "{fixture_name}: marker line missing schema=1: {line}"
     );
     assert!(
-        line.contains(&format!("\"source\":\"")) && line.contains(expected_source_suffix),
+        line.contains("\"source\":\"") && line.contains(expected_source_suffix),
         "{fixture_name}: marker line does not include source suffix {expected_source_suffix}: {line}"
     );
     // The runtime echo on the next line should mirror the same data

tests/mcp_http_tests.rs

@@ -2,11 +2,11 @@ use std::io::BufRead;
 use std::process::{Child, Command, Stdio};
 use std::time::Duration;
 
-/// Integration tests for the SafeOutputs HTTP server (`mcp-http` subcommand).
-///
-/// These tests validate the HTTP transport layer that MCPG uses to reach
-/// SafeOutputs. They do NOT require Docker or the MCPG gateway — they test
-/// the ado-aw HTTP server directly.
+// Integration tests for the SafeOutputs HTTP server (`mcp-http` subcommand).
+//
+// These tests validate the HTTP transport layer that MCPG uses to reach
+// SafeOutputs. They do NOT require Docker or the MCPG gateway — they test
+// the ado-aw HTTP server directly.
 
 /// Guard that kills the child process on drop (even on panic).
 struct ServerGuard {
@@ -94,6 +94,7 @@ fn start_server() -> ServerGuard {
     }
     // Kill and panic if not ready
     child.kill().ok();
+    child.wait().ok();
     panic!("SafeOutputs HTTP server did not become ready within 5 s");
 }