Allow spaces or more symbols in MCP server names

[oprypkhantt]

What variant of Codex are you using?

CLI

What feature would you like to see?

Hey. Currently the validation for MCP servers warns if name contains anything other than letters, numbers, _ or -. My assumption is this limitation stems from TOML spec, used for Codex config:

Bare keys may only contain ASCII letters, ASCII digits, underscores, and dashes (A-Za-z0-9_-)

But TOML also supports quoted keys, like so:

[mcp_servers."Postgres MCP - Production - Read Only"]
...

[mcp_servers."Postgres MCP - Development"]
...

This is useful for a case like shown above - to give more easily readable names to MCP servers. Would you consider expanding the validation to at least support all ASCII characters?

Additional information

codex/codex-rs/cli/src/mcp_cmd.rs

Lines 912 to 918 in f1923a3

	fn validate_server_name(name: &str) -> Result<()> {
	let is_valid = !name.is_empty()
	&& name
	.chars()
	.all(|c| c.is_ascii_alphanumeric() || c == '-' || c == '_');
	if is_valid {

[autaut03]

I do not have permission to open a PR, but here's a branch with a fix: main...autaut03:codex:mcp-server-names-ascii

Needs to be checked by someone that actually knows what they're doing though 😅