docs: correct the Zapier mcp-registry partition counts (58, not 60) - #39
Closed
hamchowderr wants to merge 1 commit into
Closed
hamchowderr wants to merge 1 commit into
hamchowderr wants to merge 1 commit into
Conversation
READ_ONLY holds 25 methods, not 27 — the previous figure counted the
strings "waiting" and "error" from the getActionRun comment. The registry
itself returns 58 methods, and is byte-identical at 0.103.0 and 0.111.1,
so the pending SDK bump needs no reclassification.
Verified by enumerating sdk.getRegistry({ package: "mcp" }) at both
versions and diffing against the three classification sets.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PVrtz6XV3M8zxBttQUYKBz
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
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.
What
Corrects one line in
CLAUDE.md. The Zapier SDK classification protocol claimed the three sets partition the mcp registry as 10 + 27 + 23 = 60. The real numbers are 10 + 25 + 23 = 58.Why
READ_ONLYholds 25 methods, not 27. The 27 figure counted two string literals from thegetActionRuncomment (status: "waiting"/status: "error") as if they were method names.The number matters because it is the documented checklist item after every SDK bump — "confirm the registry-method count still equals the sum of the three sets." Anyone following it against a real registry would find 58, not 60, and reasonably conclude two methods had gone missing.
How it was verified
Enumerated
sdk.getRegistry({ package: "mcp" })in a clean scratch install at both 0.103.0 (the pinned version) and 0.111.1 (latest), then diffed each name list against the three sets parsed out ofzapier-sdk-tools.tswith comments stripped:So the partition is exact; only the documented count was wrong. This also means the pending 0.103.0 → 0.111.1 bump needs no reclassification work — everything added across those 15 releases (VFS, agentic management,
runWorkflowDraft) landed on the experimental client, which is outside the mcp registry.Risk
Documentation only. No code, no test, no behavior change.
tests/unit/zapier-sdk-tools.test.tsalready asserts the invariant correctly and was never keyed to the printed number.🤖 Generated with Claude Code
https://claude.ai/code/session_01PVrtz6XV3M8zxBttQUYKBz
Generated by Claude Code