Add a dependency-free reference client - #1
Merged
Conversation
The repo was manifest-only: server.json, README, LICENSE, glama.json. That is a real problem, not a cosmetic one. Awesome-MCP-ZH rejected the listing on 2026-08-30 with 'no verifiable MCP server implementation (only a few config files)', and Glama cannot produce a quality score for a repo with no code, which is what blocks the score badge that punkpeye/awesome-mcp-servers requires before merging. This is not padding to satisfy a checker. It is the thing someone evaluating a remote MCP server actually wants: proof it works, and the exact bytes on the wire. src/client.js is ~50 lines of plain fetch, no SDK, so it ports to any language and doubles as a spec. It handles the two details that trip a hand-rolled MCP client: Accept must list BOTH application/json and text/event-stream (omitting the SSE type is the usual cause of a 406), and a single response may come back SSE-framed. src/cli.js runs the server end to end and prints real results. Verified against production: handshake, 3 tools, search_gaps, get_top_gaps, and validate_idea returning a strong verdict on 10 matches. 9 tests via node:test, no dependencies, offline by default; LIVE=1 also hits the real server. CI runs them on every push.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The repo was manifest-only. That has now cost a listing and is blocking another:
punkpeye/awesome-mcp-servers(93k stars) requires before merging #12145This is not padding to satisfy a checker — it is what someone evaluating a remote MCP server actually wants: proof it works, and the exact bytes on the wire.
src/client.js— ~50 lines of plainfetch, no SDK. Ports to any language, doubles as a spec. Handles the two things that trip hand-rolled MCP clients:Acceptmust list bothapplication/jsonandtext/event-stream, and a single response may arrive SSE-framed.src/cli.js— runs the server end to end. Verified against production: handshake, 3 tools,search_gaps,get_top_gaps,validate_ideareturningstrongon 10 matches.node:test, zero dependencies, offline by default.LIVE=1 npm testhits the real server. CI on every push.🤖 Generated with Claude Code
https://claude.ai/code/session_01Ch4Z1okMGerjZGdWryxeQ6