Skip to content

Add VINdata MCP remote server#1243

Open
bhuvanabala wants to merge 1 commit into
stacklok:mainfrom
bhuvanabala:patch-1
Open

Add VINdata MCP remote server#1243
bhuvanabala wants to merge 1 commit into
stacklok:mainfrom
bhuvanabala:patch-1

Conversation

@bhuvanabala

Copy link
Copy Markdown

Adds VINdata MCP, a remote Streamable HTTP MCP server for VIN decoding and European vehicle data.

Repository:
https://github.com/bhuvanabala/vindata-mcp

Endpoint:
https://vindata-mcp.vindata.workers.dev/mcp

Tool:
decode_vin

VINdata MCP is already published in the Official MCP Registry and indexed on Glama, PulseMCP, and Smithery.

@bhuvanabala

Copy link
Copy Markdown
Author

VINdata MCP is already published in the Official MCP Registry and is live as a remote Streamable HTTP MCP server.

Repository:
https://github.com/bhuvanabala/vindata-mcp

Endpoint:
https://vindata-mcp.vindata.workers.dev/mcp

Tool:
decode_vin

It has also been indexed on Glama and PulseMCP.

@JAORMX

JAORMX commented Jun 3, 2026

Copy link
Copy Markdown
Collaborator

Thanks for the submission, and for building on MCP! I took VINdata MCP for a spin — the endpoint is live and decode_vin responds correctly to a real initialize / tools/list handshake, so the server itself works. 👍

That said, I'm going to decline this for now. The entry doesn't yet meet our registry inclusion criteria, and I'd rather see the project mature a bit before it lands in the catalog. The main blockers:

  • No license. The upstream repo has no LICENSE file, so the code is "all rights reserved" by default. We require a permissive license (Apache-2.0 / MIT / BSD-2 / BSD-3) — this is a hard gate.
  • Documentation doesn't describe the server. The README is the unmodified Cloudflare remote-mcp-github-oauth demo template — it documents GitHub OAuth and demo tools (add, userInfoOctokit, generateImage), with nothing about VIN decoding or the decode_vin tool. We need a README that actually documents this server (setup, the tool, auth requirements).
  • Project maturity / adoption. The repo was created only a few days ago, has no CI workflows, no tests, no releases or version tags, and no community traction yet. We'd like to see more real-world usage and a track record before adding it.
  • No security/supply-chain signals. No SECURITY.md, no dependency automation (Dependabot/Renovate), and no build provenance.

One gentle clarification: being published in the upstream Official MCP Registry and indexed on Glama / PulseMCP / Smithery is great, but it isn't part of our inclusion criteria — we evaluate each entry independently against our own criteria.

If you'd like to resubmit once the above are addressed, a few catalog-specific notes to get it right next time:

  • Location & namespace: new entries go under registries/toolhive/servers/<name>/, normalized under the io.github.stacklok namespace (see the README and docs/adding-entries-llm.md) — not directly under registries/official/ with a third-party namespace.
  • Tier: third-party servers are "tier": "Community", not "Official" (Official is reserved for the ToolHive team, MCP spec authors, or the platform owner of the service).
  • Icon: use a repo-hosted icon.svg (image/svg+xml) in the server directory rather than an external PNG.
  • Add the repository field, and drop the hand-set metadata block — metadata.* is auto-populated by CI.

Appreciate the contribution — please do come back once the project has a license, real docs, and a bit more mileage.

@JAORMX JAORMX left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

commented

rdimitrov pushed a commit that referenced this pull request Jun 3, 2026
…1244)

Reviewing PR #1243 (a third-party remote server submitted into
registries/official/ under a non-stacklok namespace, claiming Official
tier, with a boilerplate README) surfaced gaps in the review skills.

mcp-review:
- Add "Registry Layout" section: toolhive/ is the contribution path,
  official/ is a curated subset, both normalize to io.github.stacklok.
  Note that task catalog:validate does not catch placement/namespace/
  tier/icon convention issues.
- Step 2: add location & namespace, tier-matches-maintainer, and icon
  convention (repo-hosted SVG + file present) checks.
- Add live remote-endpoint check (initialize/tools/list handshake) so
  remotes are verified against their advertised tools, not the PR text.
- Step 4: add repo/author maturity and documentation-relevance checks.
- Add a severity rubric (Required / Expected / Weighed).

add-mcp-server:
- Flip remote template tier default Official -> Community and clarify
  in Critical Rules that Official is reserved for first-party servers.

mcp-review + skill-review:
- Add "Submitting the Verdict" section mapping APPROVE/REQUEST_CHANGES
  to gh pr review (which gates merge); note gh pr comment does not block.

Signed-off-by: Juan Antonio Osorio <ozz@stacklok.com>
Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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.

2 participants