Add neo-x402-mcp: AI-powered web intelligence toolkit with x402 monetization#1024
Add neo-x402-mcp: AI-powered web intelligence toolkit with x402 monetization#1024atakanelik34 wants to merge 2 commits intostacklok:mainfrom
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c2cf94c807
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| "io.github.atakanelik34": { | ||
| "metadata": { |
There was a problem hiding this comment.
Nest publisher metadata under the remote URL key
ToolHive’s extension readers expect _meta["io.modelcontextprotocol.registry/publisher-provided"]["io.github.stacklok"][<extension-key>], where for remote servers <extension-key> is remotes[0].url (see internal/serverjson/extensions.go ExtensionKey()/GetExtensions()). Here the metadata is placed directly under the namespace object, so the catalog tooling will treat this server as having no extensions (e.g., last_updated/stars won’t be read and tools/tags won’t flow through extension-aware paths).
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
The feedback given by chatgpt is correct, the nesting is not what we expect, let's update and fix that 👍
JAORMX
left a comment
There was a problem hiding this comment.
Hey @atakanelik34, thanks for the submission! I took a look and unfortunately there are quite a few things that need to be addressed before we can consider this for the catalog.
The big blocker
The source repository (https://github.com/atakanelik34/neo-x402-api-server) is completely empty... zero commits, no code, no license. We need to be able to verify what a server actually does before listing it. This is a hard requirement per our registry inclusion criteria — source code must be publicly available.
server.json spec issues
Running task catalog:validate fails because the description exceeds the 100-character limit (yours is ~155 chars). But there's more:
namemust use our namespace:io.github.stacklok/neo-x402-mcp, notio.github.atakanelik34/neo-x402-mcp_metanesting is wrong (as @rdimitrov already pointed out). The publisher key must beio.github.stacklok, and the extension key must matchremotes[0].urlexactly. So the structure should look like:Check out"_meta": { "io.modelcontextprotocol.registry/publisher-provided": { "io.github.stacklok": { "https://neo.docly.work/mcp": { "tier": "Community", "status": "Active", "overview": "## Neo x402 MCP\n\n...", ... } } } }
registries/toolhive/servers/stripe-remote/server.jsonfor a working example of a remote server entry.metadata.*fields must be removed —metadata.last_updatedandmetadata.starsare auto-populated by CI. Never add them manually.iconsarray is missing from the top-level fieldsicon.svgfile is missing from the PR entirelyoverviewfield is missing inside the extension block (should start with## Title\n\n+ 3-5 sentences)- Tags should include
"remote"since this is a remote server packages: []— just remove this field for remote servers- Missing newline at end of file
Concerns that need discussion
Beyond the spec issues, there are a couple of things the maintainer team would need to think through:
- x402 pay-per-use over crypto: Users connecting to this server could incur real financial costs on Base mainnet without realizing it. That's... not great for discoverability in a catalog where people expect to try things out. At minimum this would need very clear disclosure.
- "Cloudflare bypass" scraping: The PR description mentions anti-bot scraping with Cloudflare bypass. This could put users in a tricky spot with ToS compliance.
- No source to audit: With an empty repo, there's no way to verify what the remote endpoint actually does with requests.
What to do
The empty repo is the biggest issue. Please push your source code, add a license (we accept Apache-2.0, MIT, BSD-2-Clause, BSD-3-Clause), and then rework the server.json following the spec. The stripe-remote entry is a good template to follow.
Feel free to ping if you have questions about the schema!
|
Hey @atakanelik34, thanks again for your interest in contributing to the catalog! After reviewing this against our registry inclusion criteria, we're going to close this one out. The core blocker is that the source repository is empty (zero commits, no code, no license). Our criteria require publicly available source code with a permissive license... that's a hard requirement we can't waive. If you publish the source code and address the schema issues mentioned in the review, you're welcome to open a new PR. The Thanks for your time! |
Neo x402 MCP Server
AI-powered web intelligence toolkit with pay-per-use monetization via x402 protocol.
Tools
Details
Repository
https://github.com/atakanelik34/neo-x402-api-server