[INFO] cloning repository https://github.com/kowalskyyy999/mcp-code-reviewer
[INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/kowalskyyy999/mcp-code-reviewer" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fkowalskyyy999%2Fmcp-code-reviewer", kill_on_drop: false }`
[INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fkowalskyyy999%2Fmcp-code-reviewer'...
[INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }`
[INFO] [stdout] 69ac3d1e3eadfe5256e028b41a2781fc7670a407
[INFO] checking kowalskyyy999/mcp-code-reviewer against eca445e5ae4a6679cc27d3a09106ce245e13a5a6 for pr-162478
[INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fkowalskyyy999%2Fmcp-code-reviewer" "/workspace/builds/worker-2-tc1/source", kill_on_drop: false }`
[INFO] [stderr] Cloning into '/workspace/builds/worker-2-tc1/source'...
[INFO] [stderr] done.
[INFO] started tweaking git repo https://github.com/kowalskyyy999/mcp-code-reviewer
[INFO] finished tweaking git repo https://github.com/kowalskyyy999/mcp-code-reviewer
[INFO] tweaked toml for git repo https://github.com/kowalskyyy999/mcp-code-reviewer written to /workspace/builds/worker-2-tc1/source/Cargo.toml
[INFO] validating manifest of git repo https://github.com/kowalskyyy999/mcp-code-reviewer on toolchain eca445e5ae4a6679cc27d3a09106ce245e13a5a6
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+eca445e5ae4a6679cc27d3a09106ce245e13a5a6" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] crate git repo https://github.com/kowalskyyy999/mcp-code-reviewer already has a lockfile, it will not be regenerated
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+eca445e5ae4a6679cc27d3a09106ce245e13a5a6" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] [stderr] warning: binary `mcp_client` should have a kebab-case name
[INFO] [stderr]    |
[INFO] [stderr]  1 | /workspace/builds/worker-2-tc1/source/target/.../mcp_client
[INFO] [stderr]    |                                                  ^^^^^^^^^^
[INFO] [stderr]    |
[INFO] [stderr]    = note: `cargo::non_kebab_case_bins` is set to `warn` by default
[INFO] [stderr] help: to change the binary name to `mcp-client`, convert `package.name`
[INFO] [stderr]   --> mcp_client/Cargo.toml:2:8
[INFO] [stderr]    |
[INFO] [stderr]  2 - name = "mcp_client"
[INFO] [stderr]  2 + name = "mcp-client"
[INFO] [stderr]    |
[INFO] [stderr] help: to change the binary name to `mcp-client`, specify `bin.name`
[INFO] [stderr]   --> mcp_client/Cargo.toml:14:72
[INFO] [stderr]    |
[INFO] [stderr] 14 ~ tracing-subscriber = { version = "0.3.23", features = ["env-filter"] }
[INFO] [stderr] 15 + [[bin]]
[INFO] [stderr] 16 + name = "mcp-client"
[INFO] [stderr] 17 + path = "src/main.rs"
[INFO] [stderr]    |
[INFO] [stderr] warning: `mcp_client` (manifest) generated 1 warning
[INFO] [stderr] warning: binary `mcp_code_reviewer` should have a kebab-case name
[INFO] [stderr]    |
[INFO] [stderr]  1 | /workspace/builds/worker-2-tc1/source/target/.../mcp_code_reviewer
[INFO] [stderr]    |                                                  ^^^^^^^^^^^^^^^^^
[INFO] [stderr]    |
[INFO] [stderr]    = note: `cargo::non_kebab_case_bins` is set to `warn` by default
[INFO] [stderr] help: to change the binary name to `mcp-code-reviewer`, convert `package.name`
[INFO] [stderr]   --> Cargo.toml:6:8
[INFO] [stderr]    |
[INFO] [stderr]  6 - name = "mcp_code_reviewer"
[INFO] [stderr]  6 + name = "mcp-code-reviewer"
[INFO] [stderr]    |
[INFO] [stderr] help: to change the binary name to `mcp-code-reviewer`, specify `bin.name`
[INFO] [stderr]   --> Cargo.toml:11:16
[INFO] [stderr]    |
[INFO] [stderr] 11 ~ resolver = "2"
[INFO] [stderr] 12 + [[bin]]
[INFO] [stderr] 13 + name = "mcp-code-reviewer"
[INFO] [stderr] 14 + path = "src/main.rs"
[INFO] [stderr]    |
[INFO] [stderr] warning: `mcp_code_reviewer` (manifest) generated 1 warning
[INFO] [stderr] warning: binary `mcp_server` should have a kebab-case name
[INFO] [stderr]    |
[INFO] [stderr]  1 | /workspace/builds/worker-2-tc1/source/target/.../mcp_server
[INFO] [stderr]    |                                                  ^^^^^^^^^^
[INFO] [stderr]    |
[INFO] [stderr]    = note: `cargo::non_kebab_case_bins` is set to `warn` by default
[INFO] [stderr] help: to change the binary name to `mcp-server`, convert `package.name`
[INFO] [stderr]   --> mcp_server/Cargo.toml:2:8
[INFO] [stderr]    |
[INFO] [stderr]  2 - name = "mcp_server"
[INFO] [stderr]  2 + name = "mcp-server"
[INFO] [stderr]    |
[INFO] [stderr] help: to change the binary name to `mcp-server`, specify `bin.name`
[INFO] [stderr]   --> mcp_server/Cargo.toml:12:72
[INFO] [stderr]    |
[INFO] [stderr] 12 ~ tracing-subscriber = { version = "0.3.23", features = ["env-filter"] }
[INFO] [stderr] 13 + [[bin]]
[INFO] [stderr] 14 + name = "mcp-server"
[INFO] [stderr] 15 + path = "src/main.rs"
[INFO] [stderr]    |
[INFO] [stderr] warning: `mcp_server` (manifest) generated 1 warning
[INFO] [stderr]  Downloading crates ...
[INFO] [stderr]   Downloaded rmcp-macros v3.1.4
[INFO] [stderr]   Downloaded rmcp v3.1.4
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-2-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-2-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-m" "1610612736" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:8683fc1fc2eb5c9ac98e0d076ab094b2ffac7f99da555d2b6a2e27f346de2ec7" "sleep" "infinity", kill_on_drop: false }`
[INFO] [stdout] 14fe8ec08f806478bf9b99ff95f03e999ea6ec6e8f5cdcd1e01f590a51f87ddf
[INFO] running `Command { std: "docker" "start" "14fe8ec08f806478bf9b99ff95f03e999ea6ec6e8f5cdcd1e01f590a51f87ddf", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "14fe8ec08f806478bf9b99ff95f03e999ea6ec6e8f5cdcd1e01f590a51f87ddf", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "exec" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-w" "/opt/rustwide/workdir" "--user" "0:0" "14fe8ec08f806478bf9b99ff95f03e999ea6ec6e8f5cdcd1e01f590a51f87ddf" "/opt/rustwide/cargo-home/bin/cargo" "+eca445e5ae4a6679cc27d3a09106ce245e13a5a6" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "14fe8ec08f806478bf9b99ff95f03e999ea6ec6e8f5cdcd1e01f590a51f87ddf", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "exec" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-w" "/opt/rustwide/workdir" "--user" "0:0" "14fe8ec08f806478bf9b99ff95f03e999ea6ec6e8f5cdcd1e01f590a51f87ddf" "/opt/rustwide/cargo-home/bin/cargo" "+eca445e5ae4a6679cc27d3a09106ce245e13a5a6" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr] warning: binary `mcp_client` should have a kebab-case name
[INFO] [stderr]    |
[INFO] [stderr]  1 | /opt/rustwide/target/.../mcp_client
[INFO] [stderr]    |                          ^^^^^^^^^^
[INFO] [stderr]    |
[INFO] [stderr]    = note: `cargo::non_kebab_case_bins` is set to `warn` by default
[INFO] [stderr] help: to change the binary name to `mcp-client`, convert `package.name`
[INFO] [stderr]   --> mcp_client/Cargo.toml:2:8
[INFO] [stderr]    |
[INFO] [stderr]  2 - name = "mcp_client"
[INFO] [stderr]  2 + name = "mcp-client"
[INFO] [stderr]    |
[INFO] [stderr] help: to change the binary name to `mcp-client`, specify `bin.name`
[INFO] [stderr]   --> mcp_client/Cargo.toml:14:72
[INFO] [stderr]    |
[INFO] [stderr] 14 ~ tracing-subscriber = { version = "0.3.23", features = ["env-filter"] }
[INFO] [stderr] 15 + [[bin]]
[INFO] [stderr] 16 + name = "mcp-client"
[INFO] [stderr] 17 + path = "src/main.rs"
[INFO] [stderr]    |
[INFO] [stderr] warning: `mcp_client` (manifest) generated 1 warning
[INFO] [stderr] warning: binary `mcp_server` should have a kebab-case name
[INFO] [stderr]    |
[INFO] [stderr]  1 | /opt/rustwide/target/.../mcp_server
[INFO] [stderr]    |                          ^^^^^^^^^^
[INFO] [stderr]    |
[INFO] [stderr]    = note: `cargo::non_kebab_case_bins` is set to `warn` by default
[INFO] [stderr] help: to change the binary name to `mcp-server`, convert `package.name`
[INFO] [stderr]   --> mcp_server/Cargo.toml:2:8
[INFO] [stderr]    |
[INFO] [stderr]  2 - name = "mcp_server"
[INFO] [stderr]  2 + name = "mcp-server"
[INFO] [stderr]    |
[INFO] [stderr] help: to change the binary name to `mcp-server`, specify `bin.name`
[INFO] [stderr]   --> mcp_server/Cargo.toml:12:72
[INFO] [stderr]    |
[INFO] [stderr] 12 ~ tracing-subscriber = { version = "0.3.23", features = ["env-filter"] }
[INFO] [stderr] 13 + [[bin]]
[INFO] [stderr] 14 + name = "mcp-server"
[INFO] [stderr] 15 + path = "src/main.rs"
[INFO] [stderr]    |
[INFO] [stderr] warning: `mcp_server` (manifest) generated 1 warning
[INFO] [stderr] warning: binary `mcp_code_reviewer` should have a kebab-case name
[INFO] [stderr]    |
[INFO] [stderr]  1 | /opt/rustwide/target/.../mcp_code_reviewer
[INFO] [stderr]    |                          ^^^^^^^^^^^^^^^^^
[INFO] [stderr]    |
[INFO] [stderr]    = note: `cargo::non_kebab_case_bins` is set to `warn` by default
[INFO] [stderr] help: to change the binary name to `mcp-code-reviewer`, convert `package.name`
[INFO] [stderr]   --> Cargo.toml:6:8
[INFO] [stderr]    |
[INFO] [stderr]  6 - name = "mcp_code_reviewer"
[INFO] [stderr]  6 + name = "mcp-code-reviewer"
[INFO] [stderr]    |
[INFO] [stderr] help: to change the binary name to `mcp-code-reviewer`, specify `bin.name`
[INFO] [stderr]   --> Cargo.toml:11:16
[INFO] [stderr]    |
[INFO] [stderr] 11 ~ resolver = "2"
[INFO] [stderr] 12 + [[bin]]
[INFO] [stderr] 13 + name = "mcp-code-reviewer"
[INFO] [stderr] 14 + path = "src/main.rs"
[INFO] [stderr]    |
[INFO] [stderr] warning: `mcp_code_reviewer` (manifest) generated 1 warning
[INFO] [stderr]    Compiling libc v0.2.189
[INFO] [stderr]     Checking futures-channel v0.3.34
[INFO] [stderr]    Compiling syn v3.0.3
[INFO] [stderr]     Checking indexmap v2.14.0
[INFO] [stderr]    Compiling serde_core v1.0.229
[INFO] [stderr]    Compiling cfg_aliases v0.2.2
[INFO] [stderr]    Compiling serde_json v1.0.151
[INFO] [stderr]     Checking tracing v0.1.44
[INFO] [stderr]    Compiling ref-cast v1.0.27
[INFO] [stderr]    Compiling rustls v0.23.43
[INFO] [stderr]    Compiling nix v0.31.3
[INFO] [stderr]    Compiling rmcp v3.1.4
[INFO] [stderr]     Checking matchers v0.2.0
[INFO] [stderr]     Checking ipnet v2.12.1
[INFO] [stderr]    Compiling pastey v0.2.3
[INFO] [stderr]     Checking base64 v0.23.1
[INFO] [stderr]     Checking mcp_code_reviewer v0.1.0 (/opt/rustwide/workdir)
[INFO] [stderr]     Checking tracing-subscriber v0.3.23
[INFO] [stderr]    Compiling jobserver v0.1.35
[INFO] [stderr]    Compiling cc v1.4.4
[INFO] [stderr]     Checking errno v0.3.14
[INFO] [stderr]     Checking parking_lot_core v0.9.12
[INFO] [stderr]     Checking mio v1.2.2
[INFO] [stderr]     Checking socket2 v0.6.5
[INFO] [stderr]     Checking getrandom v0.4.3
[INFO] [stderr]     Checking signal-hook-registry v1.4.8
[INFO] [stderr]     Checking parking_lot v0.12.5
[INFO] [stderr]     Checking uuid v1.24.1
[INFO] [stderr]    Compiling cmake v0.1.58
[INFO] [stderr]    Compiling aws-lc-sys v0.44.0
[INFO] [stderr]    Compiling darling_core v0.24.1
[INFO] [stderr]    Compiling serde_derive_internals v0.30.0
[INFO] [stderr]    Compiling tokio-macros v2.7.2
[INFO] [stderr]    Compiling zerovec-derive v0.11.6
[INFO] [stderr]    Compiling displaydoc v0.2.7
[INFO] [stderr]    Compiling futures-macro v0.3.34
[INFO] [stderr]    Compiling serde_derive v1.0.229
[INFO] [stderr]    Compiling ref-cast-impl v1.0.27
[INFO] [stderr]    Compiling thiserror-impl v2.0.20
[INFO] [stderr]    Compiling schemars_derive v1.2.2
[INFO] [stderr]     Checking tokio v1.53.1
[INFO] [stderr]     Checking futures-util v0.3.34
[INFO] [stderr]     Checking zerotrie v0.2.5
[INFO] [stderr]     Checking thiserror v2.0.20
[INFO] [stderr]     Checking zerovec v0.11.8
[INFO] [stderr]    Compiling darling_macro v0.24.1
[INFO] [stderr]    Compiling darling v0.24.1
[INFO] [stderr]    Compiling rmcp-macros v3.1.4
[INFO] [stderr]     Checking tinystr v0.8.4
[INFO] [stderr]     Checking potential_utf v0.1.6
[INFO] [stderr]     Checking icu_collections v2.3.0
[INFO] [stderr]     Checking icu_locale_core v2.3.0
[INFO] [stderr]     Checking icu_provider v2.3.1
[INFO] [stderr]     Checking icu_properties v2.3.0
[INFO] [stderr]     Checking icu_normalizer v2.3.0
[INFO] [stderr]     Checking serde v1.0.229
[INFO] [stderr]     Checking futures-executor v0.3.34
[INFO] [stderr]     Checking futures v0.3.34
[INFO] [stderr]     Checking chrono v0.4.45
[INFO] [stderr]     Checking idna_adapter v1.2.2
[INFO] [stderr]     Checking idna v1.1.0
[INFO] [stderr]     Checking url v2.5.8
[INFO] [stderr]     Checking schemars v1.2.2
[INFO] [stderr]     Checking tokio-util v0.7.19
[INFO] [stderr]     Checking tokio-stream v0.1.19
[INFO] [stderr]     Checking process-wrap v9.1.0
[INFO] [stderr]     Checking tower v0.5.3
[INFO] [stderr]     Checking h2 v0.4.18
[INFO] [stderr]     Checking tower-http v0.6.11
[INFO] [stderr]    Compiling aws-lc-rs v1.18.0
[INFO] [stderr]     Checking hyper v1.11.0
[INFO] [stderr]     Checking hyper-util v0.1.20
[INFO] [stderr]     Checking rustls-webpki v0.103.14
[INFO] [stderr]     Checking tokio-rustls v0.26.4
[INFO] [stderr]     Checking rustls-platform-verifier v0.7.0
[INFO] [stderr]     Checking hyper-rustls v0.27.9
[INFO] [stderr]     Checking reqwest v0.13.4
[INFO] [stderr]     Checking mcp_server v0.1.0 (/opt/rustwide/workdir/mcp_server)
[INFO] [stderr]     Checking mcp_client v0.1.0 (/opt/rustwide/workdir/mcp_client)
[INFO] [stdout] warning: unused imports: `ConfigureCommandExt` and `serde`
[INFO] [stdout]  --> mcp_client/src/main.rs:4:13
[INFO] [stdout]   |
[INFO] [stdout] 4 |     object, serde,
[INFO] [stdout]   |             ^^^^^
[INFO] [stdout] 5 |     service::{RequestContext, RoleClient, RunningService},
[INFO] [stdout] 6 |     transport::{ConfigureCommandExt, TokioChildProcess},
[INFO] [stdout]   |                 ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `ServerHandler`
[INFO] [stdout]  --> mcp_server/src/main.rs:1:12
[INFO] [stdout]   |
[INFO] [stdout] 1 | use rmcp::{ServerHandler, tool};
[INFO] [stdout]   |            ^^^^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::fs`
[INFO] [stdout]  --> mcp_server/src/main.rs:4:5
[INFO] [stdout]   |
[INFO] [stdout] 4 | use std::fs;
[INFO] [stdout]   |     ^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated struct `rmcp::model::SamplingMessage`: Sampling is deprecated by SEP-2577 and will be removed in a future release. See https://github.com/modelcontextprotocol/modelcontextprotocol/pull/2577
[INFO] [stdout]   --> mcp_client/src/main.rs:21:22
[INFO] [stdout]    |
[INFO] [stdout] 21 |         _messages: &[SamplingMessage],
[INFO] [stdout]    |                      ^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(deprecated)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated struct `rmcp::model::CreateMessageRequestParams`: Sampling is deprecated by SEP-2577 and will be removed in a future release. See https://github.com/modelcontextprotocol/modelcontextprotocol/pull/2577
[INFO] [stdout]   --> mcp_client/src/main.rs:31:17
[INFO] [stdout]    |
[INFO] [stdout] 31 |         params: CreateMessageRequestParams,
[INFO] [stdout]    |                 ^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated struct `rmcp::model::CreateMessageResult`: Sampling is deprecated by SEP-2577 and will be removed in a future release. See https://github.com/modelcontextprotocol/modelcontextprotocol/pull/2577
[INFO] [stdout]   --> mcp_client/src/main.rs:33:17
[INFO] [stdout]    |
[INFO] [stdout] 33 |     ) -> Result<CreateMessageResult, ErrorData> {
[INFO] [stdout]    |                 ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated struct `rmcp::model::CreateMessageResult`: Sampling is deprecated by SEP-2577 and will be removed in a future release. See https://github.com/modelcontextprotocol/modelcontextprotocol/pull/2577
[INFO] [stdout]   --> mcp_client/src/main.rs:40:12
[INFO] [stdout]    |
[INFO] [stdout] 40 |         Ok(CreateMessageResult::new(
[INFO] [stdout]    |            ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated struct `rmcp::model::SamplingMessage`: Sampling is deprecated by SEP-2577 and will be removed in a future release. See https://github.com/modelcontextprotocol/modelcontextprotocol/pull/2577
[INFO] [stdout]   --> mcp_client/src/main.rs:41:13
[INFO] [stdout]    |
[INFO] [stdout] 41 |             SamplingMessage::assistant_text(response_text),
[INFO] [stdout]    |             ^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated struct `rmcp::model::CreateMessageResult`: Sampling is deprecated by SEP-2577 and will be removed in a future release. See https://github.com/modelcontextprotocol/modelcontextprotocol/pull/2577
[INFO] [stdout]   --> mcp_client/src/main.rs:44:27
[INFO] [stdout]    |
[INFO] [stdout] 44 |         .with_stop_reason(CreateMessageResult::STOP_REASON_END_TURN))
[INFO] [stdout]    |                           ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated field `rmcp::model::CreateMessageRequestParams::messages`: Sampling is deprecated by SEP-2577 and will be removed in a future release. See https://github.com/modelcontextprotocol/modelcontextprotocol/pull/2577
[INFO] [stdout]   --> mcp_client/src/main.rs:38:37
[INFO] [stdout]    |
[INFO] [stdout] 38 |             self.mock_llm_response(&params.messages, params.system_prompt.as_deref());
[INFO] [stdout]    |                                     ^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated field `rmcp::model::CreateMessageRequestParams::system_prompt`: Sampling is deprecated by SEP-2577 and will be removed in a future release. See https://github.com/modelcontextprotocol/modelcontextprotocol/pull/2577
[INFO] [stdout]   --> mcp_client/src/main.rs:38:54
[INFO] [stdout]    |
[INFO] [stdout] 38 |             self.mock_llm_response(&params.messages, params.system_prompt.as_deref());
[INFO] [stdout]    |                                                      ^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused imports: `ConfigureCommandExt` and `serde`
[INFO] [stdout]  --> mcp_client/src/main.rs:4:13
[INFO] [stdout]   |
[INFO] [stdout] 4 |     object, serde,
[INFO] [stdout]   |             ^^^^^
[INFO] [stdout] 5 |     service::{RequestContext, RoleClient, RunningService},
[INFO] [stdout] 6 |     transport::{ConfigureCommandExt, TokioChildProcess},
[INFO] [stdout]   |                 ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `ServerHandler`
[INFO] [stdout]  --> mcp_server/src/main.rs:1:12
[INFO] [stdout]   |
[INFO] [stdout] 1 | use rmcp::{ServerHandler, tool};
[INFO] [stdout]   |            ^^^^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::fs`
[INFO] [stdout]  --> mcp_server/src/main.rs:4:5
[INFO] [stdout]   |
[INFO] [stdout] 4 | use std::fs;
[INFO] [stdout]   |     ^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated struct `rmcp::model::SamplingMessage`: Sampling is deprecated by SEP-2577 and will be removed in a future release. See https://github.com/modelcontextprotocol/modelcontextprotocol/pull/2577
[INFO] [stdout]   --> mcp_client/src/main.rs:21:22
[INFO] [stdout]    |
[INFO] [stdout] 21 |         _messages: &[SamplingMessage],
[INFO] [stdout]    |                      ^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(deprecated)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated struct `rmcp::model::CreateMessageRequestParams`: Sampling is deprecated by SEP-2577 and will be removed in a future release. See https://github.com/modelcontextprotocol/modelcontextprotocol/pull/2577
[INFO] [stdout]   --> mcp_client/src/main.rs:31:17
[INFO] [stdout]    |
[INFO] [stdout] 31 |         params: CreateMessageRequestParams,
[INFO] [stdout]    |                 ^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated struct `rmcp::model::CreateMessageResult`: Sampling is deprecated by SEP-2577 and will be removed in a future release. See https://github.com/modelcontextprotocol/modelcontextprotocol/pull/2577
[INFO] [stdout]   --> mcp_client/src/main.rs:33:17
[INFO] [stdout]    |
[INFO] [stdout] 33 |     ) -> Result<CreateMessageResult, ErrorData> {
[INFO] [stdout]    |                 ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated struct `rmcp::model::CreateMessageResult`: Sampling is deprecated by SEP-2577 and will be removed in a future release. See https://github.com/modelcontextprotocol/modelcontextprotocol/pull/2577
[INFO] [stdout]   --> mcp_client/src/main.rs:40:12
[INFO] [stdout]    |
[INFO] [stdout] 40 |         Ok(CreateMessageResult::new(
[INFO] [stdout]    |            ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated struct `rmcp::model::SamplingMessage`: Sampling is deprecated by SEP-2577 and will be removed in a future release. See https://github.com/modelcontextprotocol/modelcontextprotocol/pull/2577
[INFO] [stdout]   --> mcp_client/src/main.rs:41:13
[INFO] [stdout]    |
[INFO] [stdout] 41 |             SamplingMessage::assistant_text(response_text),
[INFO] [stdout]    |             ^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated struct `rmcp::model::CreateMessageResult`: Sampling is deprecated by SEP-2577 and will be removed in a future release. See https://github.com/modelcontextprotocol/modelcontextprotocol/pull/2577
[INFO] [stdout]   --> mcp_client/src/main.rs:44:27
[INFO] [stdout]    |
[INFO] [stdout] 44 |         .with_stop_reason(CreateMessageResult::STOP_REASON_END_TURN))
[INFO] [stdout]    |                           ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> mcp_client/src/main.rs:95:9
[INFO] [stdout]    |
[INFO] [stdout] 95 |     let mut mcp_client = setup_mcp().await;
[INFO] [stdout]    |         ----^^^^^^^^^^
[INFO] [stdout]    |         |
[INFO] [stdout]    |         help: remove this `mut`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `tools`
[INFO] [stdout]    --> mcp_client/src/main.rs:106:9
[INFO] [stdout]     |
[INFO] [stdout] 106 |     let tools = mcp_client.list_all_tools().await.unwrap();
[INFO] [stdout]     |         ^^^^^ help: if this is intentional, prefix it with an underscore: `_tools`
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated field `rmcp::model::CreateMessageRequestParams::messages`: Sampling is deprecated by SEP-2577 and will be removed in a future release. See https://github.com/modelcontextprotocol/modelcontextprotocol/pull/2577
[INFO] [stdout]   --> mcp_client/src/main.rs:38:37
[INFO] [stdout]    |
[INFO] [stdout] 38 |             self.mock_llm_response(&params.messages, params.system_prompt.as_deref());
[INFO] [stdout]    |                                     ^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated field `rmcp::model::CreateMessageRequestParams::system_prompt`: Sampling is deprecated by SEP-2577 and will be removed in a future release. See https://github.com/modelcontextprotocol/modelcontextprotocol/pull/2577
[INFO] [stdout]   --> mcp_client/src/main.rs:38:54
[INFO] [stdout]    |
[INFO] [stdout] 38 |             self.mock_llm_response(&params.messages, params.system_prompt.as_deref());
[INFO] [stdout]    |                                                      ^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> mcp_client/src/main.rs:95:9
[INFO] [stdout]    |
[INFO] [stdout] 95 |     let mut mcp_client = setup_mcp().await;
[INFO] [stdout]    |         ----^^^^^^^^^^
[INFO] [stdout]    |         |
[INFO] [stdout]    |         help: remove this `mut`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `tools`
[INFO] [stdout]    --> mcp_client/src/main.rs:106:9
[INFO] [stdout]     |
[INFO] [stdout] 106 |     let tools = mcp_client.list_all_tools().await.unwrap();
[INFO] [stdout]     |         ^^^^^ help: if this is intentional, prefix it with an underscore: `_tools`
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] warning: unused dependency `dotenvy`
[INFO] [stderr]  --> Cargo.toml:2:1
[INFO] [stderr]   |
[INFO] [stderr] 2 | dotenvy = "0.15.7"
[INFO] [stderr]   | ^^^^^^^^^^^^^^^^^^
[INFO] [stderr]   |
[INFO] [stderr]   = note: `cargo::unused_dependencies` is set to `warn` by default
[INFO] [stderr] help: consider removing the dependency on `dotenvy`
[INFO] [stderr] warning: `mcp_code_reviewer` (manifest) generated 1 warning
[INFO] [stderr]     Finished `dev` profile [unoptimized + debuginfo] target(s) in 1m 06s
[INFO] running `Command { std: "docker" "inspect" "14fe8ec08f806478bf9b99ff95f03e999ea6ec6e8f5cdcd1e01f590a51f87ddf", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "14fe8ec08f806478bf9b99ff95f03e999ea6ec6e8f5cdcd1e01f590a51f87ddf", kill_on_drop: false }`
[INFO] [stdout] 14fe8ec08f806478bf9b99ff95f03e999ea6ec6e8f5cdcd1e01f590a51f87ddf
