Skip to content

Commit ed2386c

Browse files
committed
fix(mcp): add example to server name validation error message
Fixes bounty issue #1604 The error message for invalid MCP server names now includes an example ('my-server') to help users understand the expected format.
1 parent 8f839ec commit ed2386c

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

cortex-cli/src/mcp_cmd.rs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,8 @@
22
33
use anyhow::{Context, Result, bail};
44
use clap::{ArgGroup, Parser};
5-
use cortex_common::dirs::get_cortex_home;
65
use cortex_common::CliConfigOverrides;
6+
use cortex_common::dirs::get_cortex_home;
77
use std::io::{self, BufRead, Write};
88

99
// ============================================================================
@@ -918,7 +918,9 @@ fn validate_server_name(name: &str) -> Result<()> {
918918
.all(|c| c.is_ascii_alphanumeric() || c == '-' || c == '_');
919919

920920
if !is_valid_chars {
921-
bail!("invalid server name '{name}' (use letters, numbers, '-', '_')");
921+
bail!(
922+
"invalid server name '{name}' (use letters, numbers, '-', '_'); example: 'my-server'"
923+
);
922924
}
923925

924926
// Must start with a letter or underscore

0 commit comments

Comments
 (0)