Fix CI coverage: exclude cargo-rustapi from tarpaulin, gate mcp_tools example

Author: Tuntii

.github/workflows/ci.yml

@@ -211,7 +211,14 @@ jobs:
         uses: taiki-e/install-action@cargo-tarpaulin
 
       - name: Run coverage
-        run: cargo tarpaulin --workspace --out Html --output-dir coverage
+        run: |
+          # cargo-rustapi spawns nested `cargo` builds in integration tests;
+          # tarpaulin cannot instrument those subprocesses reliably.
+          cargo tarpaulin --workspace \
+            --exclude cargo-rustapi \
+            --timeout 900 \
+            --out Html \
+            --output-dir coverage
 
       - name: Upload coverage report
         uses: actions/upload-artifact@v4

crates/cargo-rustapi/tests/cli_tests.rs

@@ -10,6 +10,11 @@ fn cargo_rustapi() -> Command {
     assert_cmd::cargo::cargo_bin_cmd!("cargo-rustapi")
 }
 
+/// Tarpaulin sets `TARPAULIN` when instrumenting tests; nested `cargo` builds are unreliable there.
+fn running_under_tarpaulin() -> bool {
+    std::env::var_os("TARPAULIN").is_some()
+}
+
 mod new_command {
     use super::*;
 
@@ -320,6 +325,11 @@ mod generate_command {
 
     #[test]
     fn test_generate_crud_sqlx_compiles() {
+        if running_under_tarpaulin() {
+            eprintln!("skipping nested cargo e2e under tarpaulin");
+            return;
+        }
+
         let dir = tempdir().expect("Failed to create temp dir");
         let project_name = "test-crud-sqlx";
         let project_path = dir.path().join(project_name);

crates/rustapi-rs/Cargo.toml

@@ -37,6 +37,11 @@ rustapi-openapi = { workspace = true, default-features = false }
 name = "file_upload"
 path = "examples/file_upload.rs"
 
+[[example]]
+name = "mcp_tools"
+path = "examples/mcp_tools.rs"
+required-features = ["protocol-mcp"]
+
 [dev-dependencies]
 rustapi-core = { workspace = true }
 rustapi-macros = { workspace = true }

crates/rustapi-rs/examples/mcp_tools.rs

@@ -28,31 +28,45 @@
 //!     -H 'content-type: application/json' \
 //!     -d '{"jsonrpc":"2.0","id":3,"method":"tools/call","params":{"name":"get_agent_weather_city","arguments":{"city":"Istanbul"}}}'
 
+#[cfg(not(any(feature = "protocol-mcp", feature = "mcp")))]
+fn main() {
+    eprintln!(
+        "Run this example with MCP support enabled:\n  cargo run -p rustapi-rs --example mcp_tools --features protocol-mcp"
+    );
+}
+
+#[cfg(any(feature = "protocol-mcp", feature = "mcp"))]
 use rustapi_rs::prelude::*;
+#[cfg(any(feature = "protocol-mcp", feature = "mcp"))]
 use rustapi_rs::protocol::mcp::{
     run_rustapi_and_mcp_with_shutdown, InvocationMode, McpConfig, McpServer,
 };
+#[cfg(any(feature = "protocol-mcp", feature = "mcp"))]
 use serde::{Deserialize, Serialize};
 
+#[cfg(any(feature = "protocol-mcp", feature = "mcp"))]
 #[derive(Serialize, Schema)]
 struct Weather {
     city: String,
     temperature: i32,
     unit: &'static str,
 }
 
+#[cfg(any(feature = "protocol-mcp", feature = "mcp"))]
 #[derive(Deserialize, Serialize, Schema)]
 struct SumRequest {
     a: i32,
     b: i32,
 }
 
+#[cfg(any(feature = "protocol-mcp", feature = "mcp"))]
 #[derive(Serialize, Schema)]
 struct SumResponse {
     sum: i32,
 }
 
 /// This route will be exposed as an MCP tool because of the "agent" tag.
+#[cfg(any(feature = "protocol-mcp", feature = "mcp"))]
 #[rustapi_rs::get("/agent/weather/{city}")]
 #[rustapi_rs::tag("agent")]
 #[rustapi_rs::summary("Get weather information for a city")]
@@ -65,6 +79,7 @@ async fn get_weather(Path(city): Path<String>) -> Json<Weather> {
 }
 
 /// Another exposed tool (POST with JSON body).
+#[cfg(any(feature = "protocol-mcp", feature = "mcp"))]
 #[rustapi_rs::post("/agent/sum")]
 #[rustapi_rs::tag("agent")]
 #[rustapi_rs::summary("Add two integers and return the result")]
@@ -73,11 +88,13 @@ async fn sum(Json(req): Json<SumRequest>) -> Json<SumResponse> {
 }
 
 /// This route is deliberately NOT tagged — it will NOT appear in MCP discovery.
+#[cfg(any(feature = "protocol-mcp", feature = "mcp"))]
 #[rustapi_rs::get("/admin/internal-config")]
 async fn internal_only() -> &'static str {
     "this-should-never-be-visible-to-agents"
 }
 
+#[cfg(any(feature = "protocol-mcp", feature = "mcp"))]
 #[tokio::main]
 async fn main() -> Result<(), Box<dyn std::error::Error + Send + Sync>> {
     let app = RustApi::auto();