Skip to content

Add MCP Toplist rank badge - #352

Open
chrstphe wants to merge 1 commit into
CursorTouch:mainfrom
chrstphe:mcp-toplist-badge
Open

Add MCP Toplist rank badge#352
chrstphe wants to merge 1 commit into
CursorTouch:mainfrom
chrstphe:mcp-toplist-badge

Conversation

@chrstphe

Copy link
Copy Markdown

Hi! MCP Toplist tracks 81,686 MCP servers across the major
registries, and Windows MCP is currently ranked #212.

This PR adds a small live badge to your README showing that rank — it updates
automatically as the leaderboard changes:

MCP Toplist

Totally fine to close if you'd rather not — and if you'd like your server's
data corrected or removed from the leaderboard, just say so here or open an
issue at the site. Thanks for building Windows MCP!

@qodo-code-review

Copy link
Copy Markdown

PR Summary by Qodo

Add MCP Toplist rank badge to README

📝 Documentation 🕐 Less than 5 minutes

Grey Divider

AI Description

• Add an MCP Toplist badge to the README to display the server’s live rank.
• Link the badge to the MCP Toplist server page for quick leaderboard access.
Diagram

graph TD
  A["README.md"] --> B{{"MCP Toplist"}} --> C["Badge SVG"]
  B --> D["Server page"]

  subgraph Legend
    direction LR
    _doc["Doc file"] ~~~ _ext{{"External site"}} ~~~ _res["Web resource"]
  end
Loading
High-Level Assessment

The following are alternative approaches to this PR:

1. Use a shields.io-style proxy badge
  • ➕ Reduces direct dependency on a single badge host URL structure
  • ➕ May offer caching and more consistent rendering across clients
  • ➖ Adds another third-party dependency
  • ➖ May not support this leaderboard directly without custom setup
2. Keep README free of live external badges
  • ➕ Avoids outbound requests to third-party services when viewing the README
  • ➕ Eliminates risk of badge downtime or unexpected content changes
  • ➖ Loses the discoverability and social-proof benefit of showing rank

Recommendation: The PR’s approach (a simple external badge + link) is appropriate given the minimal scope and zero runtime impact. If the project prefers fewer third-party calls from documentation, consider the “no live badges” alternative; otherwise, this is a low-risk documentation enhancement.

Files changed (1) +2 / -0

Documentation (1) +2 / -0
README.mdEmbed MCP Toplist rank badge +2/-0

Embed MCP Toplist rank badge

• Adds an MCP Toplist badge at the top of the README. The badge links to the MCP Toplist page for the Windows-MCP server entry.

README.md

@qodo-code-review

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (2) 📘 Rule violations (1) 📜 Skill insights (0)

Context used
✅ Compliance rules (platform): 15 rules

Grey Divider


Remediation recommended

1. README badge line too long 📘 Rule violation ✧ Quality
Description
The newly added badge Markdown line exceeds the 100-character maximum, reducing readability and
violating the repository's line-length policy. This should be wrapped/split without breaking the
rendered badge link.
Code

README.md[1]

+[![MCP Toplist](https://mcptoplist.com/badge/io.github.CursorTouch%2FWindows-MCP.svg)](https://mcptoplist.com/server/io.github.CursorTouch%2FWindows-MCP)
Relevance

⭐⭐ Medium

100-char rule enforcement is mixed: wrap request accepted in PR307 but similar long-line request
rejected in PR329.

PR-#307
PR-#329

ⓘ Recommendations generated based on similar findings in past PRs

Evidence
PR Compliance ID 222796 requires all non-whitespace lines in changed files to be <= 100 characters.
The added badge line in README.md is ~153 characters long due to two full URLs in a single
Markdown construct.

Rule 222796: Enforce maximum line length of 100 characters
README.md[1-1]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
A newly added line in `README.md` exceeds the 100-character maximum due to the inline badge image + link URLs.

## Issue Context
The badge can be rendered equivalently using multi-line HTML so each line stays under 100 characters (e.g., put the `<a href=...>` and `<img src=...>` on separate lines).

## Fix Focus Areas
- README.md[1-1]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools



Informational

2. Badge outside centered header 🐞 Bug ⚙ Maintainability
Description
The MCP Toplist badge is placed above the existing <div align="center"> header container, so it
will render separately (typically left-aligned) while the other badges/title are centered, creating
inconsistent README formatting.
Code

README.md[R1-4]

+[![MCP Toplist](https://mcptoplist.com/badge/io.github.CursorTouch%2FWindows-MCP.svg)](https://mcptoplist.com/server/io.github.CursorTouch%2FWindows-MCP)
+
<div align="center">
  <h1>🪟 Windows-MCP</h1>
Relevance

⭐ Low

Repo previously merged a badge placed above the centered <div align="center"> block (PR12) without
alignment concerns.

PR-#12

ⓘ Recommendations generated based on similar findings in past PRs

Evidence
The new badge is now the very first line of the README, and the centered header block starts on the
next lines; existing badges are inside that centered block, so the new badge is not part of the same
alignment/layout group.

README.md[1-22]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The new MCP Toplist badge is inserted before the README’s centered header `<div align="center">`, which causes it to render outside the centered layout and look inconsistent with the other header badges.

## Issue Context
The README already uses a centered header block that contains the project title and multiple badges.

## Fix Focus Areas
- README.md[1-4]

## Suggested fix
Move the new badge markdown line to immediately after `<div align="center">` (or wrap it in its own `<div align="center">...</div>`), so it is centered consistently with the rest of the header.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


3. External badge host dependency 🐞 Bug ⛨ Security
Description
The README now embeds an externally hosted SVG from mcptoplist.com, adding a new third-party
dependency for README rendering and exposing README viewers to that host
(availability/privacy/trust-boundary considerations) without any repository-controlled pinning.
Code

README.md[1]

+[![MCP Toplist](https://mcptoplist.com/badge/io.github.CursorTouch%2FWindows-MCP.svg)](https://mcptoplist.com/server/io.github.CursorTouch%2FWindows-MCP)
Relevance

⭐ Low

Team previously accepted externally hosted README badge image/link from a third-party domain (PR12),
indicating this dependency is tolerated.

PR-#12

ⓘ Recommendations generated based on similar findings in past PRs

Evidence
The added markdown links to and loads an SVG badge from https://mcptoplist.com/...svg, introducing
a new external image host beyond the other badge hosts already used in the README header.

README.md[1-21]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The README embeds a live SVG badge from a new third-party domain (`mcptoplist.com`). This makes README rendering depend on an external service and shares viewer request metadata with that domain.

## Issue Context
The README already includes other external badges, but this PR introduces an additional new host and a dynamically served SVG.

## Fix Focus Areas
- README.md[1-1]

## Suggested fix
If you want to reduce external dependency risk, replace the live external SVG with one of:
- A static badge image committed under `assets/` and referenced locally in the README.
- A badge served via an existing, already-accepted badge provider used in this README.
- A rasterized (PNG) badge if you must keep a remote badge but want to avoid dynamic SVG content.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Grey Divider

Qodo Logo

Comment thread README.md
@@ -1,3 +1,5 @@
[![MCP Toplist](https://mcptoplist.com/badge/io.github.CursorTouch%2FWindows-MCP.svg)](https://mcptoplist.com/server/io.github.CursorTouch%2FWindows-MCP)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Remediation recommended

1. Readme badge line too long 📘 Rule violation ✧ Quality

The newly added badge Markdown line exceeds the 100-character maximum, reducing readability and
violating the repository's line-length policy. This should be wrapped/split without breaking the
rendered badge link.
Agent Prompt
## Issue description
A newly added line in `README.md` exceeds the 100-character maximum due to the inline badge image + link URLs.

## Issue Context
The badge can be rendered equivalently using multi-line HTML so each line stays under 100 characters (e.g., put the `<a href=...>` and `<img src=...>` on separate lines).

## Fix Focus Areas
- README.md[1-1]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant