Align mcp-server-dev skills with claude.com/docs connector guidance#1418
Open
tobinsouth wants to merge 1 commit intomainfrom
Open
Align mcp-server-dev skills with claude.com/docs connector guidance#1418tobinsouth wants to merge 1 commit intomainfrom
tobinsouth wants to merge 1 commit intomainfrom
Conversation
- build-mcp-server: load llms-full.txt for Claude-specific context; add Phase 6 (test in Claude, review checklist, submit, ship plugin) - references/auth.md: add Claude auth-type table, callback URL, not-supported list - references/tool-design.md: add Anthropic Directory hard requirements (annotations, name length, read/write split, prompt-injection rule) - build-mcp-app: add Claude host specifics (prefersBorder, safeAreaInsets, CSP) and submission asset specs; testing via custom connector - build-mcpb: note remote servers are the recommended directory path
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
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.
Summary
The plugin's three skills currently reference only the generic MCP spec. This PR adds Claude-specific guidance and links to the new pages at https://claude.com/docs/connectors/building so the skill produces servers that pass directory review on the first try.
Changes
build-mcp-server/SKILL.md— instruct Claude to loadhttps://claude.com/docs/llms-full.txtbefore scaffolding; add Phase 6: Test in Claude and publish (custom-connector test path, review checklist, submission, "also ship a plugin")references/auth.md— add Claude auth-type table (oauth_dcr,oauth_cimd,oauth_anthropic_creds,custom_connection,none), callback URL, and not-supported list (nostatic_bearer, no pure m2m client_credentials)references/tool-design.md— add Anthropic Directory hard requirements:readOnlyHint/destructiveHint/titlerequired, ≤64-char names, read/write split, no behavioral directives in descriptions, freeform-API tools must reference target docsbuild-mcp-app/SKILL.md— add Claude host specifics (prefersBorder,safeAreaInsets,_meta.ui.csp), submission screenshot specs, and custom-connector testing pathbuild-mcpb/SKILL.md— note MCPB is secondary; remote is recommended for directory listingDoc links added
All point at
claude.com/docs/connectors/building/{authentication, testing, review-criteria, submission, what-to-build}plusllms-full.txt.Test plan
/mcp-server-dev:build-mcp-serverand confirm it fetches llms-full.txt and surfaces Phase 6