Skip to content

Commit 3c8e1f7

Browse files
CopilotMossaka
andcommitted
Fix linting and test failures: remove unused import and update config field references
Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
1 parent 6a3660f commit 3c8e1f7

2 files changed

Lines changed: 15 additions & 10 deletions

File tree

src/config.rs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -76,15 +76,17 @@ mod tests {
7676
crate::Serve {
7777
plugin_dir: Some(PathBuf::from("/test/plugin/dir")),
7878
stdio: true,
79-
http: false,
79+
sse: false,
80+
streamable_http: false,
8081
}
8182
}
8283

8384
fn empty_test_cli_config() -> crate::Serve {
8485
crate::Serve {
8586
plugin_dir: None,
8687
stdio: false,
87-
http: false,
88+
sse: false,
89+
streamable_http: false,
8890
}
8991
}
9092

src/main.rs

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -20,11 +20,10 @@ use rmcp::model::{
2020
ListToolsResult, PaginatedRequestParam, ServerCapabilities, ServerInfo, ToolsCapability,
2121
};
2222
use rmcp::service::{serve_server, RequestContext, RoleServer};
23-
use rmcp::transport::{stdio as stdio_transport, SseServer};
2423
use rmcp::transport::streamable_http_server::{
25-
StreamableHttpService, session::local::LocalSessionManager,
24+
session::local::LocalSessionManager, StreamableHttpService,
2625
};
27-
use axum;
26+
use rmcp::transport::{stdio as stdio_transport, SseServer};
2827
use rmcp::ServerHandler;
2928
use serde::{Deserialize, Serialize};
3029
use tracing_subscriber::layer::SubscriberExt as _;
@@ -186,11 +185,15 @@ async fn main() -> Result<()> {
186185
match &cli.command {
187186
Commands::Serve(cfg) => {
188187
// Initialize logging based on transport type
189-
let (use_stdio_transport, use_streamable_http) = match (cfg.stdio, cfg.sse, cfg.streamable_http) {
190-
(false, false, false) => (true, false), // Default case: use stdio transport
191-
(true, false, false) => (true, false), // Stdio transport only
192-
(false, true, false) => (false, false), // SSE transport only
193-
(false, false, true) => (false, true), // Streamable HTTP transport only
188+
let (use_stdio_transport, use_streamable_http) = match (
189+
cfg.stdio,
190+
cfg.sse,
191+
cfg.streamable_http,
192+
) {
193+
(false, false, false) => (true, false), // Default case: use stdio transport
194+
(true, false, false) => (true, false), // Stdio transport only
195+
(false, true, false) => (false, false), // SSE transport only
196+
(false, false, true) => (false, true), // Streamable HTTP transport only
194197
_ => {
195198
return Err(anyhow::anyhow!(
196199
"Running multiple transports simultaneously is not supported. Please choose one of: --stdio, --sse, or --streamable-http."

0 commit comments

Comments
 (0)