-
Notifications
You must be signed in to change notification settings - Fork 17
Open
Description
Playing around with this gem (great work!) - and noticed that I had to dip into the gem code to mess with RubyLLM::MCP::Tool#format_name in order to remove a forward slash in my tool name, since that's unacceptable to both ChatGPT + Claude.
I noticed that Tool#name does a lot of gsubbing to avoid this, but this gem does not use this:
https://github.com/crmne/ruby_llm/blob/main/lib/ruby_llm/tool.rb#L69-L71
Curious if this was an oversight or a choice...happy to whip up a patch to call super#name, or bring in this regexp but unclear if this was done intentionally. Thanks!
Metadata
Metadata
Assignees
Labels
No labels