8000 Add 124 Community MCP Servers by olaservo · Pull Request #2007 · modelcontextprotocol/servers · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

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

Conversation

olaservo
Copy link
Member
@olaservo olaservo commented Jun 4, 2025

This PR adds 124 new MCP servers to the README, sourced from community pull requests.

Added Servers

@olaservo olaservo added the add-community-server This pull request adds a link to a community-created server. label Jun 5, 2025
cliffhall
cliffhall previously approved these changes Jun 5, 2025
Copy link
Contributor
@cliffhall cliffhall left a 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?

@henroger
Copy link
henroger commented Jun 7, 2025

Thanks @olaservo and @cliffhall for maintaining this!

@olaservo olaservo changed the title Add 105 Community MCP Servers Add 124 Community MCP Servers Jun 9, 2025
@olaservo olaservo requested a review from cliffhall June 9, 2025 04:18
@olaservo
Copy link
Member Author
olaservo commented Jun 9, 2025

@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?

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 olaservo requested a review from tadasant June 9, 2025 12:24
@cliffhall
Copy link
Contributor
cliffhall commented Jun 9, 2025

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?

@olaservo
Copy link
Member Author
olaservo commented Jun 9, 2025

One that stands out here, that I think we should ask for an update to is: TimLukaHorstmann/mcp-weather

I think I fixed that in the actual Readme diff, just not in the PR description?

Also, what is the strategy for closing all the associated PRs that are rolled up into this one after it is merged?

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.

Copy link
Contributor
@cliffhall cliffhall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 👍

@henroger
Copy link

Thanks @olaservo and @cliffhall for maintaining this!

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?

It was originally in this PR

@olaservo
Copy link
Member Author
olaservo commented Jun 11, 2025

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add-community-server This pull request adds a link to a community-created server.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0