chore: update mcp-science servers to pypi package #250
Conversation
|
Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. Warning Rate limit exceeded@JoJoJoJoJoJoJo has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 9 minutes and 16 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (7)
WalkthroughUpdated uvx installation arguments in two server registry JSON files to remove git-based --from sources and subdirectory references, leaving only the package names in args. No other fields changed. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
PR Reviewer Guide 🔍Here are some key observations to aid the review process:
|
PR Code Suggestions ✨Explore these optional code suggestions:
|
|||||||||||||
![qodo-merge-pro[bot]](https://avatars.githubusercontent.com/in/484649?s=60&v=4){kind=small}
Co-authored-by: qodo-merge-pro[bot] <151058649+qodo-merge-pro[bot]@users.noreply.github.com>
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
mcp-registry/servers/web-fetch.json (1)
31-35: Use the hyphenated console script name inweb-fetch.jsonThe PyPI package registers its console script as
mcp-web-fetch, notmcp_web_fetch, so the"args"entry should be updated to match.• File: mcp-registry/servers/web-fetch.json (lines 31–35)
• Change the single-token arg to the hyphenated name:- "args": [ - "mcp_web_fetch" - ], + "args": [ + "mcp-web-fetch" + ],Optionally, if you want to be explicit about the source, you can instead use:
- "args": [ - "mcp_web_fetch" - ], + "args": [ + "--from", + "mcp-web-fetch", + "mcp-web-fetch" + ],
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (3)
mcp-registry/servers/materials-project.json(1 hunks)mcp-registry/servers/txyz-search.json(0 hunks)mcp-registry/servers/web-fetch.json(1 hunks)
💤 Files with no reviewable changes (1)
- mcp-registry/servers/txyz-search.json
🔇 Additional comments (1)
mcp-registry/servers/materials-project.json (1)
181-185: Useuvxand the hyphenated console script nameThe PyPI package for the MCP “materials-project” server registers its console script as
mcp-materials-project(with hyphens), so the registry entry should:
- Use the
uvxcommand (matching the"uvx"installation type)- Invoke the hyphenated script name
Locations to update:
- File:
mcp-registry/servers/materials-project.json- Lines 181–185
Proposed diff:
--- a/mcp-registry/servers/materials-project.json +++ b/mcp-registry/servers/materials-project.json @@ -181,6 +181,6 @@ "env": { /* … */ }, - "command": "uv", + "command": "uvx", "args": [ - "mcp_materials_project" + "mcp-materials-project" ], /* … */This aligns with the package’s
console_scriptsentry (mcp-materials-project = mcp_materials_project.cli:main). Merging this change will ensure the server launches correctly.
| "args": [ | ||
| "--from", | ||
| "git+https://github.com/pathintegral-institute/mcp.science#subdirectory=servers/web-fetch", | ||
| "mcp-web-fetch" | ||
| "mcp_web_fetch" | ||
| ], |
There was a problem hiding this comment.
Suggestion: Fix incorrect CLI executable name
| "args": [ | |
| "--from", | |
| "git+https://github.com/pathintegral-institute/mcp.science#subdirectory=servers/web-fetch", | |
| "mcp-web-fetch" | |
| "mcp_web_fetch" | |
| ], | |
| "args": [ | |
| "mcp-web-fetch" | |
| ], |
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (1)
mcp-registry/servers/materials-project.json (1)
181-181: Good catch: switch runner to uvx (matches registry type and past feedback).Using "command": "uvx" aligns with the "type": "uvx" block and our learning to use uv tooling. Also reflects the prior reviewer’s suggestion that was marked as manually applied.
🧹 Nitpick comments (1)
mcp-registry/servers/materials-project.json (1)
183-184: Optional: consider version pinning strategy.Using the bare app name prevents pinning to a known-good version. If stability matters, prefer an explicit requirement with a version range and the console script name:
"command": "uvx", "args": [ - "mcp_materials_project" + "--from", "mcp_materials_project>=0.1,<1.0", + "mcp_materials_project" ],This stays PyPI-based while guarding against breaking changes.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
mcp-registry/servers/materials-project.json(1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2025-08-06T08:20:00.958Z
Learnt from: CR
PR: pathintegral-institute/mcpm.sh#0
File: CLAUDE.md:0-0
Timestamp: 2025-08-06T08:20:00.958Z
Learning: Use `uv` for all Python dependency management.
Applied to files:
mcp-registry/servers/materials-project.json
🔇 Additional comments (1)
mcp-registry/servers/materials-project.json (1)
183-184: Unable to verify console‐script name for “mcp_materials_project”The PyPI lookup for
mcp_materials_projectreturned a 404, so we couldn’t confirm whether the package actually exposes an underscore‐named entry point. Please manually verify:
That the package you’re depending on is indeed published under “mcp_materials_project” (and not a hyphenated name like “mcp-materials-project”), or that you’re pointing at a private index.
That its
setup.py/pyproject.tomllists a console_scripts entry exactly namedmcp_materials_project.If the actual script name differs, update the
argsaccordingly, or use the explicit--frompattern for robustness:"command": "uvx", "args": [ - "mcp_materials_project" + "--from", "mcp_materials_project", "mcp_materials_project" ],
|
🎉 This PR is included in version 2.7.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
User description
update to install from pypi
PR Type
Other
Description
Update MCP science servers installation from Git to PyPI packages
Simplify uvx command arguments by removing Git repository references
Standardize package names with underscore format
Diagram Walkthrough
File Walkthrough
materials-project.json
Switch materials-project server to PyPI installationmcp-registry/servers/materials-project.json
--fromand Git repository URL from uvx argsmcp-materials-projecttomcp_materials_projecttxyz-search.json
Switch txyz-search server to PyPI installationmcp-registry/servers/txyz-search.json
--fromand Git repository URL from uvx argsmcp-txyz-searchweb-fetch.json
Switch web-fetch server to PyPI installationmcp-registry/servers/web-fetch.json
--fromand Git repository URL from uvx argsmcp-web-fetchtomcp_web_fetchSummary by CodeRabbit