requests. No GraphQL client. python3 shop_mcp.py is the whole install.manifest.json (5 assertions) and README.md (1) are deliberately not shipped into site-packages — the manifest's entry_point names a bundle path that does not exist in an installed copy, so packaging it would make a correct install fail. Both assertions skip rather than fail when their file is absent, which is why the count moves and the verdict does not. Clone the repo to run all 189.print() corrupts the client's next parse. Nothing looks wrong when you run the server yourself. Every diagnostic here goes to stderr, and a test asserts stdout stays byte-empty across a full session.notifications/initialized has no id, so a reply to it is a message with no pending request. Strict clients treat that as a protocol violation and drop the connection.id: 0 read as a notification. if msg.get("id") is falsy for zero, so a client that numbers requests from zero has its first call silently dropped. Presence, not truthiness.isError: true and the reason as text — otherwise the model never sees the message and cannot correct its own arguments.protocolVersion. If a client asks for a revision the server does not know, agreeing to it leaves both sides believing a spec is in use that neither implements. This falls back to 2025-03-26, the spec's own default, and says so.search_products "what do we sell that matches X" — identity and total stock get_product one product in full, every variant with SKU, price, stock check_inventory stock for a SKU per location: available, committed, on-hand low_stock_report variants at or below a threshold, lowest firstsku:SH 1 is a different query from sku:"SH 1". The first silently matches the wrong variants and reports their stock as if it were yours. SKUs are quoted and internal quotes escaped.null for a variant that does not track inventory. Coerced to 0, it appears in every restock report forever. Untracked and out-of-stock are different facts and stay different.scan_exhausted. low_stock_report scans a bounded number of variants. If the scan hit its limit, "nothing is low" is indistinguishable from "I did not look far enough" — so the result says which it was, and the model can say so too.THROTTLED GraphQL response is a 200 and must be retried, not read as success; a 401 must not be retried, because waiting will not fix a bad token; backoff must actually grow.mcp Python SDK's types.py (LATEST_PROTOCOL_VERSION, CallToolResult, ServerCapabilities), not from memory.gpt-ads-feed. A README that blurs it is asking to be trusted on the wrong thing.mutation_test.sh injects known defects into copies of the source and asserts --self-test goes red for each, naming which assertion caught it. It also flags a NO-OP EDIT when a search pattern has gone stale — because a mutation that does not apply tests nothing while looking green, which is the failure mode that makes a suite worse than useless: trusted and empty.KeyError: 'result', from indexing a reply that the defect had turned into a JSON-RPC error. Fixed by routing result access through a shape guard, so the same defect now reports a tool crash returns a result, so the loop survives: reply is a JSON-RPC error {'code': -32603, ...} and the three following assertions each still report their own verdict.S.Tools(c).search_products(...), present only to make the assertion below it meaningful. When the throttle branch was disabled it raised, aborting the test before that assertion ran. Fixed with completes(), the exact inverse of raises(): the defect now reports a 200-with-THROTTLED is survivable, not a hard failure: raised ShopifyError: Throttled [THROTTLED], naming the rule and keeping the cause..mcpb bundle and launched the way a host launches it, which no test had ever done:SHOPIFY_SHOP; this README and the bundle manifest both told users to export SHOPIFY_SHOP_DOMAIN. Anyone following the docs got a permanently unconfigured server. Every one of the 180 assertions passed, because none of them compared the code against the docs.tools/list returned [] until credentials existed, so a host saw an empty server and reported it broken — and the readable no store is configured message on tools/call was unreachable, since nothing was listed to call. The docstring above that code stated the opposite requirement, and the test below it asserted the defect: eq(tools, [], ...). The list never depended on credentials; descriptors() touched no instance state at all, and is now a staticmethod.--self-test was advertised in the module docstring but crashed inside the bundle, which shipped only the server file. The bundle now ships the suite.README.md was absent, and the harness copied only two files, so it fired inside every mutant. The run still printed 17 caught, but six of those were credited to README.md is present instead of their own labels: six real assertions could have been dead with the suite still green. A missing README is a packaging fact, not a code defect. The load-bearing comparison now runs against the module docstring, which travels with the source, and the harness copies the README so the cross-check is real.tools/list, then returns isError with the missing variable named. A host that cannot read tools/list reports "broken server" and sends you looking in the wrong place.Posted Sep 8, 2026
Stdio MCP server for a live Shopify Admin API. One Python file, no SDK. 189 assertions; 23 injected defects each caught. shop-mcp 1.0.1 on PyPI.
0
0