-
Notifications
You must be signed in to change notification settings - Fork 71
feat(mcp): add script to dump mcp tool list #1211
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
This change is part of the following stack: Change managed by git-spice. |
a421135 to
49c64b7
Compare
49c64b7 to
52465c1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note that in subsequent PRs this file is moved to internal/mcp
|
Just a random question: should the src-cli surface depend on the MCP specification, or should it just have similar tools such as (I genuinely don't know, just a conversation starter!) |
The later sounds more sustainable and allows more tools that can just be organically discovered. From the latest release of Claude and it's advanced skills I think it's moving a lot more towards selective disclosure of tools that the agent can see on the fly. I think MCP spec wise, you're stuck in a place where you either create loads of tools (+descriptions) or create "big" tools that does a lot of heavy lifting. |
|
yeah and since when you got So wondering if - inspired by what tools we saw useful in MCP - it would be interesting to design a slightly more human-friendly version of that for src-cli independently of MCP tool names/descriptions/params which would change the CLI surface if we decide to change the MCP(?) |
It will definitely be interesting since I wonder once you make the src-cli interface more appealing, surely the LLM would prefer to use it more over the MCP? I haven't used the GitHub MCP much but I wonder how often the LLM picks to use the MCP over |
|
good question, I think it's also a distribution question. Some people have |
Ah yeah that is valid. Even though I just interfaced with the SG MCP that is HTTP based, I seemingly forgot about it in my comment 🤦🏼 |
Add a script which curls s2 mcp endpoint and dumps the list-tools json
Test plan
executed script locally and committed the json