-
Notifications
You must be signed in to change notification settings - Fork 136
chore: add MCP Trust Score link #1289
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
WalkthroughAdded a new centered badge block near the top of Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Suggested reviewers
Poem
Tip 🔌 Remote MCP (Model Context Protocol) integration is now available!Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats. ✨ 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 (
|
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.
Actionable comments posted: 1
🧹 Nitpick comments (2)
README.md (2)
111-111: Keep badge style and markup consistent with existing shields block.Current top badges are centered and use HTML
<a>+<img>witharia-label. Consider matching that pattern for consistency and accessibility (even if you keep it in the Community section).Suggested inline replacement:
-[](https://archestra.ai/mcp-catalog/Canner__wren-engine) +<a aria-label="Trust Score" href="https://archestra.ai/mcp-catalog/Canner__wren-engine"> + <img alt="Trust Score" src="https://archestra.ai/mcp-catalog/api/badge/quality/Canner/wren-engine"> +</a>
111-111: Optionally relocate the badge near the top badges.For discoverability and visual consistency, consider placing the Trust Score badge alongside the other badges under the main header. If you prefer to keep it in Community, adding a short sentence above it explaining what the score represents could help readers.
📜 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)
README.md(1 hunks)
🔇 Additional comments (1)
README.md (1)
111-111: LGTM — Trust Score badge added.The badge link and target URL look correct and non-breaking. Fine to merge as-is.
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.
Actionable comments posted: 1
♻️ Duplicate comments (1)
README.md (1)
12-12: Badge link previously returned 404 — verify endpoint still resolvesA prior review found the linked catalog page 404’d while the image endpoint was OK. With the corrected href, please re-verify both endpoints for 200 responses and cache headers.
#!/bin/bash set -euo pipefail badge_img="https://archestra.ai/mcp-catalog/api/badge/quality/Canner/wren-engine" badge_page="https://archestra.ai/mcp-catalog/canner__wren-engine" for url in "$badge_img" "$badge_page"; do echo echo "=== $url ===" echo "--- Initial HEAD (no redirects) ---" curl -sS -I --max-redirs 0 "$url" || true echo echo "--- Follow redirects (summary) ---" curl -sS -L -o /dev/null -w "Final URL: %{url_effective}\nHTTP %{http_code} | size=%{size_download} | time=%{time_total}s | type=%{content_type}\n" "$url" echo done
🧹 Nitpick comments (2)
README.md (2)
11-15: Optional: open external link in new tab and add rel for securityIf you want the catalog to open in a new tab, add target and rel.
- <a aria-label="Trust Score (MCP Catalog)" href="https://archestra.ai/mcp-catalog/canner__wren-engine"> + <a aria-label="Trust Score (MCP Catalog)" href="https://archestra.ai/mcp-catalog/canner__wren-engine" target="_blank" rel="noopener noreferrer">
11-15: Consider consolidating badges or order for consistencyYou now have two centered
blocks with badges. Consider placing this Trust Score badge into the existing badge row (or keeping both but deciding the intended order), to avoid layout jumps on different screens.
📜 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)
README.md(2 hunks)
| <a aria-label="Follow us" href="[https://x.com/getwrenai](https://archestra.ai/mcp-catalog/canner__wren-engine)"> | ||
| <img alt="" src="https://archestra.ai/mcp-catalog/api/badge/quality/Canner/wren-engine"> | ||
| </a> |
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.
Fix invalid href and missing alt; use the correct MCP catalog URL
The anchor’s href contains markdown literal [...](...), which results in an invalid link in HTML. Also, the image has an empty alt. Use the catalog URL directly and provide a meaningful alt.
Apply this diff:
- <a aria-label="Follow us" href="[https://x.com/getwrenai](https://archestra.ai/mcp-catalog/canner__wren-engine)">
- <img alt="" src="https://archestra.ai/mcp-catalog/api/badge/quality/Canner/wren-engine">
+ <a aria-label="Trust Score (MCP Catalog)" href="https://archestra.ai/mcp-catalog/canner__wren-engine">
+ <img alt="Trust Score" src="https://archestra.ai/mcp-catalog/api/badge/quality/Canner/wren-engine">📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| <a aria-label="Follow us" href="[https://x.com/getwrenai](https://archestra.ai/mcp-catalog/canner__wren-engine)"> | |
| <img alt="" src="https://archestra.ai/mcp-catalog/api/badge/quality/Canner/wren-engine"> | |
| </a> | |
| <a aria-label="Trust Score (MCP Catalog)" href="https://archestra.ai/mcp-catalog/canner__wren-engine"> | |
| <img alt="Trust Score" src="https://archestra.ai/mcp-catalog/api/badge/quality/Canner/wren-engine"> | |
| </a> |
🤖 Prompt for AI Agents
In README.md around lines 12 to 14, the anchor href incorrectly includes
markdown-style wrapper and the image alt is empty; replace the href with the
direct MCP catalog URL https://archestra.ai/mcp-catalog/canner__wren-engine
(remove the [text](url) format) and set a meaningful alt on the img such as
"Canner Wren Engine quality badge" so the link is valid and accessible.
Hi!
This PR adds the "Trust Score" badge from our new Open Source MCP catalog.
Our catalog evaluates MCP servers based on technical quality—like protocol feature implementation and dependency health—rather than vanity metrics like GitHub stars.
The scoring process is fully transparent and reproducible:
The badge is designed to be respectful to the structure of your readme, example:
Projects like Grafana MCP (https://github.com/grafana/mcp-grafana) are already participating.
We believe that transparent and truly open source MCP catalog should help the community to identify great MCP servers like yours 😊
We'd appreciate your support by merging this PR!
Summary by CodeRabbit