Skip to content

Commit b0997eb

Browse files
committed
refactor(router): simplify disable_route and notifier call
1 parent 30ac65c commit b0997eb

1 file changed

Lines changed: 3 additions & 4 deletions

File tree

  • crates/rmcp/src/handler/server/router

crates/rmcp/src/handler/server/router/tool.rs

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -449,11 +449,10 @@ where
449449
pub fn disable_route(&mut self, name: impl Into<Cow<'static, str>>) -> bool {
450450
let name = name.into();
451451
let was_visible = self.map.contains_key(&name) && !self.disabled.contains(&name);
452-
let newly_disabled = self.disabled.insert(name.clone());
453-
if was_visible && newly_disabled {
452+
if was_visible {
454453
self.notify_if_visible(&name);
455454
}
456-
newly_disabled
455+
self.disabled.insert(name)
457456
}
458457

459458
/// Re-enable a previously disabled tool. Returns `true` if the name
@@ -530,7 +529,7 @@ where
530529
fn notify_if_visible(&self, name: &str) {
531530
if self.map.contains_key(name) {
532531
if let Some(notifier) = &self.notifier {
533-
(notifier)();
532+
notifier();
534533
}
535534
}
536535
}

0 commit comments

Comments
 (0)