Skip to content

Commit 1888f73

Browse files
committed
fix: only warn when warnings exist
Prevent empty warnings array from triggering warning output. Signed-off-by: tanish111 <[email protected]>
1 parent 0e27026 commit 1888f73

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

crates/rmcp/src/handler/server/tool_name_validation.rs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -176,8 +176,9 @@ pub fn issue_tool_name_warning(name: &str, warnings: &[String]) {
176176
pub fn validate_and_warn_tool_name(name: &str) -> bool {
177177
let result = validate_tool_name(name);
178178

179-
// Always issue warnings for any validation issues (both invalid names and warnings)
180-
issue_tool_name_warning(name, &result.warnings);
179+
if !result.warnings.is_empty() {
180+
issue_tool_name_warning(name, &result.warnings);
181+
}
181182

182183
result.is_valid
183184
}

0 commit comments

Comments
 (0)