possible issue with `/` as separator in tool names (when using `.group()`)

[brettimus]

historically, having slashes in tool names (title was fine!) could trip up some mcp clients.

it seems like this issue was resolved in claude code:

anthropics/claude-code#2257
anthropics/claude-code#985

there is an open SEP to specify the characters allowed in tool names but (as of writing) this has not landed in the MCP spec

modelcontextprotocol/modelcontextprotocol#986

we use / for prefixing tool names when they are added with .group

if this causes issues in practice, we may need to allow specification of the separator. __ is a good alternative