-
Notifications
You must be signed in to change notification settings - Fork 6.1k
Add 124 Community MCP Servers #2007
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. < 8000 /p>
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
Add 124 Community MCP Servers #2007
Conversation
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.
@olaservo Looks good, but there are conflicts to be resolved. Should we coordinate getting this up to date and merged?
Also, I assume you have a repeatable process that we can use going forward, so we could plan a weekly update on Fridays or some such?
Thanks @olaservo and @cliffhall for maintaining this! |
Conflicts should now be resolved, plus some non-server resource links were being added that got resolved. Also grabbed a few of the latest ones and added to this PR plus description. There's still some PRs that will need to get reviewed/added manually, for transparency this PR was batched together using a couple simple scripts here: https://github.com/olaservo/mcp-misc/tree/main/server-readme-reviewer (additions were reviewed by me after a first pass by Claude). |
@olaservo thanks for adding the scripts to the mcp-misc project. The server validation prompt looks great. I notice several competing servers such as for fabric, ms 365, and SSH. I suppose that's fine for the current state of affairs. No reason to favor one over the other. One that stands out here, that I think we should ask for an update to is: I think the servers should be by server name, not repo/server name. Also, what is the strategy for closing all the associated PRs that are rolled up into this one after it is merged? |
I think I fixed that in the actual Readme diff, just not in the PR description?
Without getting too fancy I just added this, which will grab the aggregated PR description which includes the original PR links, then close the original PRs with a comment that links to the aggregated PR. I have a dry run + confirmation step here ^^ which should reduce the chances of things going awry, until we have something a little more streamlined and less brittle. |
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.
LGTM! 👍
Hi @olaservo Actually I just noticed my PR to add MCP Review wasn't added in the final version. Was there any problem with it? Shall I create a new PR? |
Hi @henroger it looks like it should have been added to the Resources section instead, so I was going to add it to a separate PR so it could go in that section. I did have one clarifying question on the PR which is still open here: #1981 |
This PR adds 124 new MCP servers to the README, sourced from community pull requests.
Added Servers