Ogham Research
ogham-mcp/ogham-mcp
Structured memory capture for Ogham shared memory. An agent skill from ogham-mcp/ogham-mcp.
Design the tool surface, resources, and service layer for a new MCP server.
$ npx skills add cyanheads/pubmed-mcp-server --skill design-mcp-server -a claude-codeProject install by default; add -g for ~/.claude/skills/.
$ gh skill install cyanheads/pubmed-mcp-server design-mcp-server --agent claude-codeProject scope by default; add --scope user for a personal install. Needs GitHub CLI 2.90.0 or later (public preview).
$ git clone --depth 1 https://github.com/cyanheads/pubmed-mcp-server.git skills-src && mkdir -p .claude/skills && cp -r skills-src/framework-skills/design-mcp-server .claude/skills/design-mcp-server && rm -rf skills-srcUse ~/.claude/skills/ instead of .claude/skills for a personal install. The folder must contain SKILL.md.
Claude Code skills documentation · loads skills from .claude/skills/
Install the "design-mcp-server" agent skill from https://github.com/cyanheads/pubmed-mcp-server/tree/main/framework-skills/design-mcp-server into .claude/skills/design-mcp-server/ in this project. Copy the whole folder (SKILL.md and every file beside it), keep the folder name "design-mcp-server", then confirm the skill loads.Claude Code copies the folder itself, the same result as the manual copy. Check what it changed before you commit it.
$skill-installer install https://github.com/cyanheads/pubmed-mcp-server/tree/main/framework-skills/design-mcp-serverType this inside Codex. $skill-installer <name> installs a curated skill from openai/skills. The installer writes to $CODEX_HOME/skills (default ~/.codex/skills). Restart Codex if the skill does not show up.
$ npx skills add cyanheads/pubmed-mcp-server --skill design-mcp-server -a codexProject install goes to .agents/skills/; add -g for ~/.codex/skills/.
$ gh skill install cyanheads/pubmed-mcp-server design-mcp-server --agent codexProject scope by default (.agents/skills/); add --scope user for a personal install.
$ git clone --depth 1 https://github.com/cyanheads/pubmed-mcp-server.git skills-src && mkdir -p .agents/skills && cp -r skills-src/framework-skills/design-mcp-server .agents/skills/design-mcp-server && rm -rf skills-srcUse ~/.agents/skills/ instead of .agents/skills for a personal install.
Codex skills documentation · loads skills from .agents/skills/
Install the "design-mcp-server" agent skill from https://github.com/cyanheads/pubmed-mcp-server/tree/main/framework-skills/design-mcp-server into .agents/skills/design-mcp-server/ in this project. Copy the whole folder (SKILL.md and every file beside it), keep the folder name "design-mcp-server", then confirm the skill loads.Codex copies the folder itself, the same result as the manual copy. Check what it changed before you commit it.
$ npx skills add cyanheads/pubmed-mcp-server --skill design-mcp-server -a cursorProject install goes to .agents/skills/; add -g for ~/.cursor/skills/.
$ gh skill install cyanheads/pubmed-mcp-server design-mcp-server --agent cursorProject scope by default (.agents/skills/); add --scope user for a personal install.
$ git clone --depth 1 https://github.com/cyanheads/pubmed-mcp-server.git skills-src && mkdir -p .cursor/skills && cp -r skills-src/framework-skills/design-mcp-server .cursor/skills/design-mcp-server && rm -rf skills-srcUse ~/.cursor/skills/ instead of .cursor/skills for a personal install.
Cursor skills documentation · loads skills from .cursor/skills/, .agents/skills/, .claude/skills/, .codex/skills/
Install the "design-mcp-server" agent skill from https://github.com/cyanheads/pubmed-mcp-server/tree/main/framework-skills/design-mcp-server into .cursor/skills/design-mcp-server/ in this project. Copy the whole folder (SKILL.md and every file beside it), keep the folder name "design-mcp-server", then confirm the skill loads.Cursor copies the folder itself, the same result as the manual copy. Check what it changed before you commit it.
$ gemini skills install https://github.com/cyanheads/pubmed-mcp-server.git --path framework-skills/design-mcp-server--scope user (default) or --scope workspace; --path is the subfolder of the repo that holds the skill; --consent skips the security confirmation prompt.
$ npx skills add cyanheads/pubmed-mcp-server --skill design-mcp-server -a gemini-cliProject install goes to .agents/skills/; add -g for ~/.gemini/skills/.
$ gh skill install cyanheads/pubmed-mcp-server design-mcp-server --agent gemini-cliProject scope by default (.agents/skills/); add --scope user for a personal install.
$ git clone --depth 1 https://github.com/cyanheads/pubmed-mcp-server.git skills-src && mkdir -p .gemini/skills && cp -r skills-src/framework-skills/design-mcp-server .gemini/skills/design-mcp-server && rm -rf skills-srcUse ~/.gemini/skills/ instead of .gemini/skills for a personal install, then run /skills reload.
Gemini CLI skills documentation · loads skills from .gemini/skills/, .agents/skills/
Install the "design-mcp-server" agent skill from https://github.com/cyanheads/pubmed-mcp-server/tree/main/framework-skills/design-mcp-server into .gemini/skills/design-mcp-server/ in this project. Copy the whole folder (SKILL.md and every file beside it), keep the folder name "design-mcp-server", then confirm the skill loads.Gemini CLI copies the folder itself, the same result as the manual copy. Check what it changed before you commit it.
$ gh skill install cyanheads/pubmed-mcp-server design-mcp-serverInstalls for Copilot at project scope by default; add --scope user for a personal install. Preview a skill first with gh skill preview. Needs GitHub CLI 2.90.0 or later (public preview).
$ npx skills add cyanheads/pubmed-mcp-server --skill design-mcp-server -a github-copilotProject install goes to .agents/skills/; add -g for ~/.copilot/skills/.
$ git clone --depth 1 https://github.com/cyanheads/pubmed-mcp-server.git skills-src && mkdir -p .github/skills && cp -r skills-src/framework-skills/design-mcp-server .github/skills/design-mcp-server && rm -rf skills-srcUse ~/.copilot/skills/ instead of .github/skills for a personal install. Commit .github/skills so cloud agent and code review can use it.
GitHub Copilot skills documentation · loads skills from .github/skills/, .claude/skills/, .agents/skills/
Install the "design-mcp-server" agent skill from https://github.com/cyanheads/pubmed-mcp-server/tree/main/framework-skills/design-mcp-server into .github/skills/design-mcp-server/ in this project. Copy the whole folder (SKILL.md and every file beside it), keep the folder name "design-mcp-server", then confirm the skill loads.GitHub Copilot copies the folder itself, the same result as the manual copy. Check what it changed before you commit it.
$ npx skills add cyanheads/pubmed-mcp-server --skill design-mcp-server -a opencodeOpenCode documents no install command of its own. Project install goes to .agents/skills/; add -g for ~/.config/opencode/skills/.
$ gh skill install cyanheads/pubmed-mcp-server design-mcp-server --agent opencodeProject scope by default (.agents/skills/); add --scope user for a personal install.
$ git clone --depth 1 https://github.com/cyanheads/pubmed-mcp-server.git skills-src && mkdir -p .opencode/skills && cp -r skills-src/framework-skills/design-mcp-server .opencode/skills/design-mcp-server && rm -rf skills-srcUse ~/.config/opencode/skills/ instead of .opencode/skills for a personal install.
OpenCode skills documentation · loads skills from .opencode/skills/, .claude/skills/, .agents/skills/
Install the "design-mcp-server" agent skill from https://github.com/cyanheads/pubmed-mcp-server/tree/main/framework-skills/design-mcp-server into .opencode/skills/design-mcp-server/ in this project. Copy the whole folder (SKILL.md and every file beside it), keep the folder name "design-mcp-server", then confirm the skill loads.OpenCode copies the folder itself, the same result as the manual copy. Check what it changed before you commit it.
design-mcp-serverDesign the tool surface, resources, and service layer for a new MCP server.
Design MCP Server is an agent skill from cyanheads/pubmed-mcp-server. Design the tool surface, resources, and service layer for a new MCP server. Use when starting a new server, planning a major feature expansion, or when the user describes a domain/API they want to expose via MCP. Produces a design doc at docs/design.md that drives implementation.
Its SKILL.md is about 22k tokens, which your agent loads only when the skill is triggered. It is a single SKILL.md file with no bundled scripts.
It sits in Agent Workflows, covering MCP servers, Architecture decision records and Design tokens. It works with Model Context Protocol. The repository describes itself as: Search PubMed/Europe PMC, fetch articles and full text (PMC/EPMC/Unpaywall), citations, MeSH terms via MCP. STDIO or Streamable HTTP. The licence is Apache-2.0.
9 steps, taken from the step headings in SKILL.md.
Read from SKILL.md and the folder at commit 79145a6. It shows what the files ask for, not the result of running them.
Pre-approves nothing: there is no allowed-tools line, so your agent's usual permission prompts apply.
From allowed-tools in the SKILL.md frontmatter.
No scripts in the folder and no shell commands in SKILL.md (its code samples are typescript and markdown).
From the folder's file list and the shell code blocks in SKILL.md.
Links to these hosts (documentation or services it may open):
github.comFrom URLs in SKILL.md, links to its own repository left out.
Names these keys or tokens, usually read from environment variables:
MCP_REQUEST_STATE_KEYFrom names ending in _API_KEY, _TOKEN, _SECRET, _KEY or _PASSWORD in SKILL.md.
Design MCP Server loads about 22k tokens when it runs. Until then it costs about 75 tokens; SKILL.md has 11,367 words of instructions outside code blocks.
Estimates: characters ÷ 4, the usual rule of thumb; real counts depend on the model's tokenizer. Scripts and assets cost tokens only if the agent reads them.
The automated check found no risky patterns in SKILL.md.
Automated static check — not a guarantee. Review scripts before installing. It scans the text of SKILL.md for risky patterns (piping downloads into a shell, reading credential files, hidden Unicode, destructive commands); files beside SKILL.md are not scanned.
The full file from cyanheads/pubmed-mcp-server at commit 79145a6, republished under its Apache-2.0 licence (© cyanheads). 11,367 words, ~22,185 tokens.
.claude/skills/design-mcp-server/SKILL.md (or your agent's skills folder).Do NOT use for a single tool on an existing noun — use add-tool directly.
Gather before designing. Ask the user if not obvious from context:
MirrorService don't run on Workers, and a tool that asks the caller for input mid-call needs a stateful HTTP session (see the Client round-trip row in Step 3).If the domain has a public API, read its docs before designing. For internal-only servers, skip API research and go straight to user goals. Don't design from vibes either way.
Before committing to a server boundary, answer: what workflow does this server serve, and who is the audience?
The unit of a server is a user workflow, not an API. A single rich API can earn its own server when the audience is large and the API surface supports a full workflow (PubMed for literature research, SEC EDGAR for financial analysis, Shodan for internet-wide device intelligence). Multiple APIs should collapse into one server when they serve the same workflow from different angles — a "threat intelligence" server that aggregates VirusTotal, AbuseIPDB, and GreyNoise is more useful than three separate servers because the user's goal is "assess this indicator," not "query VirusTotal."
Don't default to one-API-one-server. That's the right call when the API is deep enough and the audience is large enough, but it's not the starting point. The starting point is the workflow:
| Signal | Server boundary |
|---|---|
| Single API with rich surface, large audience | Standalone server named for the platform (pubmed-mcp-server, secedgar-mcp-server) |
| Multiple APIs serving the same workflow | One server named for the workflow (threat-intel-mcp-server), APIs are internal sources |
| Domain with distinct sub-audiences | Consider splitting — a pentester and a SOC analyst have different workflows even in the same domain |
| Pure computation, no external deps | Standalone server named for the capability (calculator-mcp-server, pentest-mcp-server) |
When multiple APIs collapse into one server, the tool surface is organized around what the user is doing, not which API gets called. The agent says "investigate this domain" and the server routes to the best available source internally. Individual APIs become service-layer implementation details, not tool-surface identities.
Usually settled before this skill runs; confirm it passes one test before designing against it. A name is earned when someone reading only the name — an npm result, a marketplace grid — forms no wrong expectation about scope. Name a wrapper for its source (pubmed-mcp-server, secedgar-mcp-server); a generic domain name is earned only by aggregating independent sources (threat-intel-mcp-server), and a single-jurisdiction scope goes in the name (uk-legislation-mcp-server). Spell out an acronym that reads as something else (libofcongress, not loc), or pair it with its domain (eia-energy, bls-labor). The tool prefix is a separate, stable identifier (see the Design table) — it need not move when the package is renamed, and renaming it is a breaking change for every client.
Applies when: the server wraps an external API or service. Skip for internal-only servers (computation, local file ops, in-memory state, code analysis utilities) and jump to Step 2.
Before designing, verify the APIs and services the server will wrap. Read the docs, then hit the API — real requests reveal what docs omit.
Research inline by default — fetch docs, read SDK readmes, confirm assumptions before committing them to the design. For each external dependency:
When research is genuinely parallelizable (multiple independent APIs, several SDKs to evaluate), spawn background agents for the independent legs while you proceed with domain mapping. Skip the overhead for a single API — just read it yourself.
Live API probing. After reading docs, make real requests against the API to verify assumptions:
filter.ids, bulk GET, or query-by-multiple-IDs patterns. A single batch request replaces N individual fetches and eliminates serial-request bottlenecks and rate-limit accumulation.fields or select parameters to request only the data you need. This reduces payload size dramatically for large objects.Stopping condition: at minimum, probe one list/search endpoint, one single-item GET, one error case (force a 404 or 400), and one unknown-param request. For large APIs with many resource types, add one probe per major noun. Stop when the response shapes and error envelope are confirmed.
This step prevents building a service layer against assumed response shapes that don't match reality.
Probe with real data; write the design with made-up data. Every example value that reaches docs/design.md — names, emails, phone numbers, hosts, IP addresses, identifiers tied to a person — is synthetic, and no key, token, or private hostname appears in it. The doc is tracked, and git history keeps whatever its first commit carried.
Start with user goals, not endpoints. Enumerate the outcomes an agent (and its human) will actually try to accomplish with this server — usually 3–10, scaled to domain size. These drive the workflow tools that form the spine of the surface. Endpoint-inventory-first design produces 1:1 API mirrors; goal-first design produces tools agents reach for. For internal-only servers, goals map to capabilities rather than endpoints — e.g., "format markdown to GFM," "tokenize text by model," "compute file hash."
Example user goals for a project management server:
Then enumerate the underlying domain operations the system supports, grouped by noun. These are the raw material workflow tools compose and single-action tools back-fill where workflows don't cover an edge case.
| Noun | Operations |
|---|---|
| Project | list, get, create, archive |
| Task | list (by project), get, create, update status, assign, comment |
| User | list, get current |
The user-goal list shapes the tool surface; the operation list fills in the gaps. Not every operation becomes a tool — an operation stays as raw material (not its own tool) when it's already fully covered by an existing tool's output, or when the only agents who'd use it are in scenarios outside this server's stated purpose.
Tools are the primary interface. Not all MCP clients expose resources, and those that do rarely surface one to the model without a human selecting it. Design the tool surface to be self-sufficient: an agent with only tool access should be able to do everything the server is built for. Resources add convenience for clients that support them (injectable context, stable URIs), but are not a reliable access path.
| Primitive | Use when | Examples |
|---|---|---|
| Tool | The default. Any operation or data access an agent needs to accomplish the server's purpose. | Search, create, update, analyze, fetch-by-ID, list reference data |
| App Tool | Rare — default to a standard tool. Only when a human will actively interact with the result in real time and the target client supports MCP Apps. Most clients are tool-only and most agent workflows are read-by-LLM, not viewed-by-human. App tools add an iframe + CSP, app.ontoolresult/callServerTool plumbing, host-context wiring, and a format() text twin that still has to be content-complete (since most clients only see that). Two surfaces to keep in sync, two failure modes per change. | Dense tabular state a human scrubs through; form-based human approval in an MCP Apps-capable client |
| Resource | Additionally expose as a resource when the data is addressable by stable URI, read-only, and useful as injectable context. | Config, schemas, status, entity-by-ID lookups |
| Prompt | Reusable message template that structures how the LLM approaches a task | Analysis framework, report template, review checklist |
| Client round-trip | Not a registered primitive — a handler returns ctx.requestInput(...) to ask the client for what only it has, and is re-entered with the answer: a confirmation or form (inputRequired.elicit), an authorization or hosted-form URL (inputRequired.elicitUrl), the client model's judgment (inputRequired.createMessage — borrow the caller's model rather than bundling one), filesystem roots (inputRequired.listRoots). Design it into the tool that needs it; see Workflow tool safety and api-context. A server with any such tool declares createApp({ sessionMode: { default: 'stateful', require: 'stateful' } }) — under stateless HTTP a 2025-era client's round trip is refused, so the tool is unusable rather than guarded. | Destructive-arm confirmation, OAuth consent, summarize-with-the-client's-model |
| Neither | Internal detail, admin-only, not useful to an LLM | Token refresh, webhook setup, migrations |
What the tool surface needs to cover depends on the server: a read-only research server has different economics than a CRUD project management server. Consider the domain, the expected agent workflows, whether it wraps one API or many, and what data relationships exist.
Common traps:
operation/mode parameter (see Step 4).Irreversible operations stay in the UI. The "Neither" bucket above covers operations that aren't useful to an LLM. There's a second, sharper reason to exclude something from the tool surface: operations whose failure mode is catastrophic and unrecoverable. Examples span domains — dropping a production database table (data loss across every row), force-emptying a versioned cloud-storage bucket (no recovery once the lifecycle policy fires), revoking the workspace's last admin role (locks everyone out, recovery requires vendor support), GDPR permanent-delete on a customer profile (un-restorable by design), purging an analytics warehouse partition older than the retention window (auditable history gone), or deleting the single audience on a free-plan email platform (nukes every subscriber and historical report in one call). These are useful to an LLM in principle, but the blast radius of a mis-call is disproportionate to any agent workflow. Humans do these in the vendor UI, where confirmation dialogs and undo paths exist. Agents shouldn't have the tool at all.
This is distinct from destructiveHint — that annotation is for operations that are destructive but recoverable (deleting a task, reverting a commit) and agents should still have them. The "stays in the UI" line applies only to operations whose failure is both catastrophic and irreversible.
This is the highest-leverage step. Tool definitions — names, descriptions, parameters, output schemas — are the entire interface contract the LLM reads to decide whether and how to call a tool. Every field is context. Design accordingly.
Most tools follow the {server}_{verb}_{noun} default — one focused responsibility, one clear verb, often (but not always) one upstream call. API-wrapping examples: pubmed_search_articles, pubmed_fetch_articles. Internal-only examples: markdown_format_text, regex_test_pattern, tokens_count_text — same naming convention, no external dep. Three variants warrant explicit design pressures of their own:
| Shape | Purpose | Typical form | Examples |
|---|---|---|---|
| Workflow | Multi-step orchestration that replaces a common agent chain | N upstream calls (often parallelized); may request confirmation; may need mid-flow cleanup | clinicaltrials_find_eligible (search → filter → rank) |
| Instruction | State-aware procedural guidance — advice, not action | Static markdown + a few live-state fetches, readOnlyHint: true, outputs nextToolSuggestions pre-filling the recommended follow-up. No writes. | git_wrapup_instructions |
| Reference | Decode opaque domain vocabulary — codes, enums, identifier formats, coverage windows — so agents can build valid inputs for the rest of the surface | Static tables or one cached fetch (often zero upstream calls); consolidate N lists under one topic enum; readOnlyHint: true, openWorldHint: false when offline | medcode_list_systems, osv_list_ecosystems |
These aren't boxes every tool must fit into — some blend shapes — but the design pressures differ enough that naming them helps avoid re-discovering the patterns per server.
The unit of a tool is a useful action, not an API call. Ask: "What is the agent trying to accomplish?" — not "What endpoints does the API have?"
A single tool can call multiple APIs internally, apply local filtering, reshape data, and return enriched results. The LLM doesn't know or care about the underlying calls.
// Workflow tool — search + local filter pipeline, not a raw API proxy
const findEligible = tool('clinicaltrials_find_eligible', {
description: 'Match a patient profile to eligible clinical trials, filtering by age, sex, conditions, location, and healthy volunteer status. Results are ranked and carry a per-study eligibility explanation.',
// handler: listStudies() → filter by eligibility → rank by location proximity → slice
});Tip — mode consolidation. When a tool has several related operations on the same noun, you can consolidate them under one tool with a
mode/operationenum. This affects both naming (noun-led, e.g.,github_pull_request) and handler design (dispatch by mode). Use when it tightens the surface; skip when ops diverge enough to warrant separate tools. When the arms need different required fields (look up by ID vs. search by name), declare the input asz.discriminatedUnion('mode', [...])rather than making every field optional and checking the combination by hand — each arm advertises its ownrequired, the handler narrows on the discriminator, and mixed arguments are rejected. Two constraints:outputstays a flatz.object, and a union root rules outheaderParam. Seeadd-tool§ Multi-mode tools.
Applies when: a server aggregates multiple data sources for the same workflow, and the "best" source varies by input type, availability, or coverage. Skip for single-API servers.
When a tool's goal can be served by multiple sources, design it as a multi-source tool — the agent calls one tool, the handler routes to the best source (or fans out to several) internally. This is the difference between a "PubMed wrapper" and a "literature research server": a hypothetical literature_search_articles tries PubMed first, falls back to EuropePMC for broader coverage, then Unpaywall for open access. The agent doesn't choose which API to hit — the server makes that decision based on what works.
Two patterns:
Source fallback chains — try sources in priority order, fall through on failure or empty results. Best when sources cover the same corpus with different depth or availability. The output should indicate which source provided the data so the agent (and human) can assess provenance. When the fallback changes what is being searched — a different corpus, different identifiers, different licensing — don't chain: expose the second source as a sibling tool so the agent chooses the corpus knowingly (the shipped pubmed-mcp-server keeps pubmed_europepmc_search separate for exactly this reason).
Multi-source fan-out — query multiple sources in parallel, merge results. Best when sources provide complementary data about the same entity. Use Promise.allSettled so one unavailable source doesn't tank the whole call — but a rejected input is wrong for every source. Rethrow an input-class rejection (ValidationError, InvalidParams) from any source instead of folding it into a per-source failure: a source that ignores the bad value would otherwise answer for it, and the agent reads a caller mistake as an outage to retry.
// Handler pseudocode — indicator enrichment across threat intel sources
async handler(input, ctx) {
const settled = await Promise.allSettled([
vtService.lookup(input.indicator),
abuseIpService.check(input.indicator),
greynoiseService.query(input.indicator),
]);
for (const r of settled) {
if (r.status === 'rejected' && isInputError(r.reason)) throw r.reason; // the caller's mistake, not an outage
}
const [vt, abuse, greynoise] = settled;
return {
indicator: input.indicator,
sources: {
// { status: 'ok', data } | { status: 'unavailable', error } — provenance per source
virustotal: toSourceResult(vt),
abuseipdb: toSourceResult(abuse),
greynoise: toSourceResult(greynoise),
},
// Server synthesizes a verdict from the sources that answered — the agent gets a conclusion, not raw API dumps
assessment: synthesizeVerdict(vt, abuse, greynoise),
};
}In both patterns, the tool surface is organized around what the user is doing. Sources are service-layer details — the agent sees threat_enrich_indicator, not virustotal_lookup + abuseipdb_check + greynoise_query. Mode-based dispatch by input type (e.g., indicator_type: 'ip' | 'domain' | 'hash') naturally routes to different source chains per mode, since different sources cover different indicator types.
There is no fixed ceiling on tool count and no target either. The ceiling is workflow coverage — if the domain genuinely has 20 distinct workflows, expose 20 tools; the cut is per-tool overlap and reach. After mapping tools, review the full list critically. A tool that covers a niche use case, serves a tiny fraction of agents, or duplicates what another tool already handles is a candidate for deferral. Drop it from the design and note it as a future addition if demand warrants. Every tool in the surface is cognitive load for tool selection — a tight surface outperforms a comprehensive one.
Applies when: the domain has recurring "how do I do X well given my current state" questions worth merging with static procedural content. Skip otherwise.
Some domains benefit from a tool whose output is guidance, not data — a markdown playbook tailored by live account state, with pre-filled next-step tool calls. These sit between Prompts (static templates, client-invokable) and action tools (do work, return data): they return advice, but the advice is worth more than static text because it merges procedural content with the agent's actual situation.
Characteristics:
guidance, diagnostics, and nextToolSuggestions)orders.customer_id — add the index before tuning the planner" (database advisor), "Error rate spiked 4× at 14:32 UTC, 4 minutes after the web@a3f9c2 deploy — roll back before chasing the upstream provider" (incident triage).readOnlyHint: true; openWorldHint follows where the diagnostics come from — false when the live state is local (a repo on disk), true when it is fetched from an external API. No writes either way.nextToolSuggestions — an array of recommended follow-up tool calls with arguments pre-filled from the diagnostics, not just tool names. The agent consumes the playbook, then executes steps with other tools.topic enum — what could be N separate per-topic tools collapses into oneconst wrapupInstructions = tool('git_wrapup_instructions', {
description: 'Get procedural guidance tailored to the current repo state: best-practice markdown merged with live diagnostics (staged/unstaged files, branch info, recent commits) and pre-filled follow-up tool calls. Read-only; execute the steps with other tools.',
annotations: { readOnlyHint: true, openWorldHint: false },
input: z.object({
topic: z.enum(['review-changes', 'stage-and-commit', 'push-to-remote'])
.describe('Playbook topic. Determines which static guidance is returned and which live state is fetched for tailoring.'),
}),
output: z.object({
guidance: z.string()
.describe('Markdown playbook content, tailored to current account state.'),
diagnostics: z.record(z.string(), z.unknown())
.describe('Live state used to tailor the guidance (e.g., staged file count, branch divergence, recent commit cadence).'),
nextToolSuggestions: z.array(z.object({
toolName: z.string().describe('Tool to call next.'),
reason: z.string().describe('Why this step is recommended given current state.'),
args: z.record(z.string(), z.unknown()).describe('Arguments pre-filled from diagnostics; {} when the tool takes none.'),
})).describe('Recommended follow-up calls with arguments already populated.'),
}),
});Prior art: git_wrapup_instructions walks through staging, commit, and push with repo state inspected. If a server has recurring "how do I do X well given my state" questions, an instruction tool typically beats N topic-specific tools and duplicating guidance in tool descriptions.
One suggestion shape, every server. Each entry is exactly { toolName, reason, args } — no per-server renames (tool, rationale, suggestedArgs, arguments), which force every client to special-case every server. args is always present ({} for a tool that takes none), and the array is always present, empty when nothing is worth suggesting. A tool on another server is never an entry: this server cannot know it is installed, so name it in the guidance prose instead.
Data tools qualify on the same terms. A data tool carries nextToolSuggestions when the right next call depends on its own result and the arguments come from that result — a status sweep that finds a degraded vendor, a connect call that learns which services the upstream supports. A fixed chain (search → get by the returned ID) does not: the IDs in the result and the server instructions already carry it, and a suggestion on every call is token noise.
Suggestions are scoped to what this deployment registers. A nextToolSuggestions entry is an executable call, so it is only correct when its target is enabled under the same configuration — a tool wrapped in disabledTool() is absent from tools/list, and a suggestion naming it hands the agent a call that fails on dispatch. Build the array from the same config the registration reads, and when the target is off, drop the entry rather than the explanation: guidance can still say the capability is unavailable in this deployment and what to do instead. The audit and a worked example live under Feature-flagged tools in add-tool/SKILL.md.
Applies when: the domain speaks in opaque vocabulary — enum codes, classification systems, identifier formats, per-source coverage windows — that agents must supply as inputs elsewhere. Skip when inputs are self-evident (free text, ISO dates, well-known formats).
A reference tool is the surface's decoder ring: which codes exist, what they mean, what format each identifier takes, what each source covers. Consolidate what could be N per-list tools under one topic enum, and serve it from static tables or a cached fetch so it stays cheap to call. Its second job is structural: it is the standing target of recovery routing — error recovery strings, zero-hit notices, and resolver guidance across the surface end with "consult the reference tool, topic X" (see Error design), which only works when the tool exists. Implement it first — it usually has no service dependency, and it grounds field-testing for every other tool.
Applies when: a tool performs multi-step mutations with destructive modes (send/apply/promote) that benefit from human confirmation before the irreversible step fires. Skip for read-only or idempotent workflows.
Tools that perform multi-step mutations (the Workflow shape) have two safety considerations beyond single-call tools. Both are about giving the agent — and the human behind it — a chance to catch a bad invocation before it commits.
Confirmation-gated destructive modes, with an annotation fallback. When a workflow's mode parameter switches between safe and destructive arms (draft vs send, plan vs apply), gate the destructive arm on a confirmation the handler asks for via ctx.requestInput(...), so a human approves before the irreversible step fires. The handler is re-entered with the answer on ctx.inputs; it does not await mid-call. The answer alone does not prove the prompt was shown — a client can send one unasked, and any requestState replays within its lifetime — so the gate stores the operation, the caller, the confirmed target, and a hash of what it holds in a ctx.state record keyed by a random id, sends only that id as requestState, and redeems the record before the destructive arm runs (api-context § Consent gates). Plan shared storage for that record when a 2026-07-28 retry can reach another instance (filesystem, supabase, or cloudflare-d1 — not cloudflare-kv), and set MCP_REQUEST_STATE_KEY so a retry can only carry state the server minted. Redeeming is not atomic — concurrent retries on one id can each pass the gate until ctx.state gains an atomic take (#593) — so an arm that must not run twice is idempotent per record.
The gate is always reachable — ctx.requestInput is present on every transport and both protocol revisions (2025-11-25 legacy, 2026-07-28 current) — but it is not always answerable: a client that never fulfils the input_required result simply doesn't retry, and — with the record pattern, which proceeds only on a record it redeemed — the destructive step never runs. The same holds for a 2025-11-25 HTTP client when the server runs MCP_SESSION_MODE=stateless: the legacy round-trip shim still runs, but its capability gate refuses because the serving instance never processed initialize — the destructive step never fires. That is the safe outcome, but it makes the tool unusable for those clients, so a server built around such a gate declares createApp({ sessionMode: { default: 'stateful', require: 'stateful' } }) and refuses to start stateless rather than degrading (api-context § ctx.requestInput). Keep destructiveHint: true in annotations so those clients' own approval flows still surface the risk. A decline is terminal — the handler fails the call rather than re-asking, which would loop until the round budget runs out. ctx.clientCapabilities never decides whether to ask: a gate that skips its prompt when elicitation is undeclared is the bypass the gate exists to prevent.
Safe defaults on parameters that determine blast radius. When a workflow accepts a parameter that controls how far-reaching a mutation is, default to the safer value. A bulk file-update tool defaulting mode: 'preview' (no writes) means a sloppy agent call shows a diff rather than blasting changes; an apply-plan tool defaulting dryRun: true means a misread plan previews rather than executes; an object-delete tool requiring an explicit confirmCount matching the result-set size means an unscoped query can't silently nuke a million rows. Agents that genuinely want the destructive behavior have to name it explicitly, which surfaces intent in the tool call and in logs.
Make retried writes safe. Agents re-issue a call that timed out. A write that can double-apply (create, send, charge, enqueue) takes a caller-supplied idempotency key or resolves to upsert semantics — and only then earns idempotentHint: true.
The description is the LLM's primary signal for tool selection. It must answer: what does this do, and when should I use it?
tool-defs-analysis skill audits this across a finished surface.operation enum's .describe().// Good — describes a prerequisite the LLM must know
description: 'Set the session working directory for all git operations. This allows subsequent git commands to omit the path parameter.'
// Good — self-explanatory, no workflow hints needed
description: 'Show the working tree status including staged, unstaged, and untracked files.'
// Good — warns about constraints
description: 'Fetch trial results data for completed studies. Only available for studies where hasResults is true.'Descriptions should be as long as needed — concise but complete. Don't artificially truncate, and don't pad with filler.
Every .describe() is prompt text the LLM reads. Parameters should convey: what the value is, what it affects, and (where non-obvious) how to use it well.
tool() applies .strict() at the root and advertises additionalProperties: false, so an unknown top-level key is rejected by name instead of silently stripped; nested objects still strip unless made strict themselves. Open the root with .passthrough() (Zod 4 also spells it .loose(); tool() honors either) only on a tool that deliberately proxies arbitrary upstream parameters (a raw-query tool), and say so in its description."". Treat the blank as omitted — left off the upstream request, never forwarded as param=, which some APIs read differently from omission — and never design a .min(1) onto an optional field to catch it. add-tool has the schema pattern that keeps a validator on the field..max(). An output-only ceiling the content already bounds (maxSections, a character budget) takes none: a larger value means no limit. A workload cap (an ID batch, limit) keeps a hard ceiling against absurd input. Between the per-call cap and that ceiling, a read — or an operation whose contract allows partial completion — processes up to the cap and names the remainder, disclosed like a capped list (below), since a rejection for a value with one obvious reading only costs the caller a round trip. A write, destructive, or otherwise atomic batch over its cap rejects before doing anything, naming the cap, so its .max() is the cap itself: doing the first items and naming the rest splits one intended operation in two and leaves the caller to work out which items changed.tools/list. Types like z.custom(), z.date(), z.transform(), z.bigint(), z.symbol(), z.void(), z.map(), z.set() throw at runtime. Use structural equivalents (e.g., z.string().describe('ISO 8601 date') instead of z.date()).// Good — explains cost, recommends action, names the alternative
fields: z.array(z.string()).optional()
.describe('Specific fields to return (reduces payload size). Without this, the full study record (~70KB each) is returned. Use full data only when you need detailed eligibility criteria, locations, or results.'),
// Good — explains what the flag does AND how to override
autoExclude: z.boolean().default(true)
.describe('Automatically exclude lock files and generated files from diff output to reduce context bloat. Set to false if you need to inspect these files.'),
// Good — names the format and gives one example
nctIds: z.union([z.string(), z.array(z.string()).max(5)])
.describe('A single NCT ID (e.g., "NCT12345678") or an array of up to 5 NCT IDs to fetch.'),Input-edge normalization. For every identifier, code, or enum-ish input, enumerate at design time the variants a caller will plausibly send, and decide per variant: normalize, or error. The rule is normalize what is certain, error on what is ambiguous. A variant is certain when the mapping is unambiguous, one-to-one, and preserves the submitted meaning exactly — then the call succeeds instead of returning a miss for a value the server could resolve. Everything short of that is an error naming the expected shape, with a recovery routing to the reference tool. Never fuzzy-match a value, broaden a query, swap one entity for another, or drop a filter to make a call succeed: plausible-looking rows from a guessed input are worse than a rejection the agent can act on.
| Class | Example | Handling |
|---|---|---|
| Case or bare-leaf shorthand of a code | ACS5, acs5 → acs/acs5 | Normalize before lookup |
| Domain value alias | mph → m/h, kph → km/h | Alias table at the input edge |
| Composite identifier completed by context | part: "52" + section: "21" → 52.21 | Try as-given first, retry the composed form on a miss — never rewrite unconditionally, since the bare form can be legitimate |
| Delimiter-joined list where an array is accepted | "US,JP,KR" → ["US","JP","KR"] | Split on the documented separator |
| Spelled-out vs. abbreviated name | "Houston, Texas" → "Houston, TX" | Normalize against the bundled name table |
These are value-level, and the mappings are domain knowledge — settle them per input in the design doc's param table. Argument key names are not: the framework rewrites declared and case-style key aliases and drops client-added root keys before the schema sees the arguments, and, against the tool's own schema after a failed parse, repairs a JSON-stringified array or object, an integer sent for a string, a number or boolean sent as a string, and a lone string sent for a list, and deletes null sent for an optional field — so an ID field stays z.string(), never a string | number union, a list field stays z.array(), never a string | string[] union, and an optional field never needs .nullish() to absorb a client that sends null for unset. A delimiter-joined list is still yours to split: the framework never wraps a string holding a comma or a line break. Don't re-implement any of that per server — see add-tool § Three things the framework fixes before the schema sees the arguments.
This resolves one submitted value to one canonical value, and does not loosen the strict token match in MCP-side list filtering, which scores a query against many candidate names.
The output schema and format function control what the LLM reads back. Design for the agent's next decision, not for a UI or an API consumer. See the add-tool skill's Tool Response Design section for implementation-level patterns (partial success, empty results, metadata, context budget).
Principles:
file shrunk could be deliberate truncation or a bug; only the agent knows. See add-tool skill's Mutator response design..optional() and render "Not available" in format() rather than coercing to false, 0, or "" — a fabricated value is worse than a gap.format() marks it. When a field carries text other people wrote — posts, reviews, bios, comments, summaries, a user-named place or project — list those fields in the design and plan how format() sets them apart: a blockquote or fence for free text, and CR/LF flattened to a space wherever a value is interpolated inline (a heading, a bolded name, a matched on "…" line), so a newline in the value can't forge structure in content[]. structuredContent keeps the value verbatim. Say in the server instructions that this content is data, never instructions; security-pass audits the result.ctx.content, never output. ctx.content.image(data, mimeType) / .audio(...) emit a content[] block once; output keeps the metadata the agent reasons over (dimensions, duration, a reference). Base64 in a typed output field ships the bytes twice.wait_for_s: 30), where 0 means act and return — not a boolean plus a timeout, which is two parameters for one decision and an awkward default. Every terminal outcome is a result, never a throw: already_<state>, <state> (with elapsed time), not_<state> within the window (with guidance naming the read-only check tool to re-poll), and unverified (window 0, or nothing to probe). Size the default to cover the common cases while staying inside client tool timeouts, and pair the action with a read-only sibling that probes the same state so an agent can re-check without re-acting.Promise.allSettled so a single failure degrades to a warning rather than tanking the outer call.excludedFiles list whose description says "call again with autoExclude=false". The agent can't act on what it doesn't know about.enrichment notice, never an error, and the notice is composed from condition → fragment pairs (too-narrow filter, a defaulted date window, a syntax trap), each routing to a concrete next call — relax a named filter, switch to the named sibling tool, or consult the reference tool. Relatedly, when the server applies a default that changes result semantics (an as-of date, an implicit status filter), echo the applied value in the output — an agent can't reason about a filter it can't see.limit, per_page, page_size, max_results, max_items) and returns an array, the handler must disclose when the cap was hit — truncated: true, shown, cap in the enrichment block via ctx.enrich.truncated({ shown, cap }), and ctx.enrich.total(n) for the full count. The same applies to a display cut in format(): show the top N and say "...and X more". Silent caps leave the agent treating a partial set as complete; the capped-list-no-truncation lint rule enforces the input-cap case.cursor plus has_more (via extractCursor/paginateArray for local sets), and never invent page numbers over a cursor-based upstream — a page the agent can't ask for is a page it will never see.DataCanvas table holding the full set (spillover() in api-canvas), and compute distributions or refinement hints across the full result, not the preview, so aggregate signal stays honest. The gates on when a canvas earns its keep are in Step 7.sections: [...] to pull only what it needs. outlineOnOverflow() (@cyanheads/mcp-ts-core/utils) returns a full | outline result; pure measure + key-slice, so Cloudflare Workers-portable, unlike canvas-bound spillover(). Distinct from spillover on shape: spillover splits a row collection, this outlines one fat record. Schema shape and format() parity are in the techniques skill's outline-on-overflow reference.MirrorService (embedded SQLite + FTS5; declare a schema + a sync ingester via defineMirror/sqliteMirrorStore, then runSync/query, see api-mirror); above ~10⁷ → an external store. Distinct lifecycle from DataCanvas: a mirror is long-lived and cross-session, refreshed on a schedule; canvas is ephemeral and per-session.structuredContent from output, others (e.g., Claude Desktop) read content[] from format(). format() is the markdown twin of structuredContent, not a summary — a thin format() that returns only a count or title leaves content[]-only clients blind (the format-parity lint catches this). Agent-facing context that is not domain payload — empty-result notices, the query as the server parsed it, echoed defaults, totals — goes in the enrichment block via ctx.enrich(...), which reaches both surfaces automatically; hand-authored into format() text alone it reaches only one. Field-by-field rendering of that block is in the Design table's Enrichment row.Applies when: the upstream API supports batch requests (filter-by-IDs, bulk GET) OR agents commonly need multiple items per call. Skip for inherently single-target operations.
Some tools naturally operate on multiple items — fetching several entities, updating a set of records, running checks across a list. Decide during design whether a tool accepts single items, arrays, or both.
When to accept array input:
| Accept array | Keep single-item | Separate batch tool |
|---|---|---|
| The upstream API supports batch requests (fetch-by-IDs, bulk update) | The operation is inherently single-target (read a file, run a query) | Batch has fundamentally different output shape or error semantics |
| Reduces N+1 round trips for a common workflow | Array input adds complexity with no backend efficiency gain | Single-item tool is simple; batch version needs progress, partial failure handling |
| Agent commonly needs multiple items in one step | The tool already returns a collection (search results) |
If a tool accepts arrays, design for partial success. When 3 of 5 items succeed, the agent needs to know which succeeded, which failed, and why — not just a success/failure boolean. Plan the output schema to report per-item results:
output: z.object({
succeeded: z.array(ItemResultSchema).describe('Items that completed successfully.'),
failed: z.array(z.object({
id: z.string().describe('Item ID that failed.'),
error: z.string().describe('What went wrong and how to resolve it.'),
})).describe('Items that failed with per-item error details.'),
}),Single-item tools don't need this — they either succeed or throw. The partial success question only arises when the tool can partially complete.
Applies when: a tool wraps a structured query language or filter system where the 80% case is a simple string. Skip when the primary input is already simple.
When a tool wraps a complex query language or filter system, provide a simple shortcut parameter for the 80% case alongside the full-power escape hatch. This keeps simple queries simple while preserving full expressiveness.
// text_search handles the common case; query handles everything else
text_search: z.string().optional()
.describe('Convenience shortcut: full-text search across title and abstract. For structured filters or field-specific matching, use the query parameter instead.'),
query: z.record(z.string(), z.unknown()).optional()
.describe('Full query object for structured filters. Supports operators: _eq, _gt, _and, _or, ...'),The pattern: name the shortcut for what it does (text_search, name_search), document what it expands to, and point to the full parameter for advanced use. Validate that at least one of the two is provided.
Applies when: an upstream API has no native search, the relevant set is bounded (fits one or a few fetches), and an agent needs to resolve a name → opaque ID. Skip when the API already searches, or when the set is unbounded (bills, votes, filings) — that belongs in the DataCanvas dataframe layer (*_dataframe_query), not an in-memory filter.
Two params, two behaviors — keep them named distinctly:
query → upstream full-text search. The API does the work; it may honor operators and ranking.filter or nameContains (the latter self-documents the local, name-keyed half of the split). Don't overload query for it — the two have different semantics and different cost.Earns-its-keep gate — all must hold: bounded set; no native upstream search; real scan pain (opaque IDs, a large/unordered list, or a default page that hides relevant rows); and it filters the natural lookup key (name/title). When any fails, skip it — paginate, or send the agent to upstream query.
Correctness: filter the complete bounded set, not the current page. Fetch up to the cap (or page through) before filtering — filtering one page returns a misleading partial slice.
Matching: strict token match is the default. Normalize (lowercase, strip punctuation/diacritics) and require every query token to appear, so word order and missing interior words still match. That strict core is the ~90% case and needs no fuzzy library. Add a fuzzy fallback only when a caller genuinely needs typo tolerance (an LLM caller rarely does): fire it only when the strict match is empty, score against the best-matching token in each name (not the whole string) and cap the results — or one short query clears the threshold against dozens of long multi-word names — and label its hits approximate. Often a bare "no match — call the unfiltered list to browse" beats an approximate guess: it lets the model self-correct instead of committing to the wrong record. See add-tool for the param + handler implementation.
Errors are part of the tool's interface — design them during the design phase, not as an afterthought. Three aspects: the contract (which failures are public), classification (what error code), and messaging (what the LLM reads).
Declare a typed contract for domain failures. When a tool has known failure modes the agent should plan around (no_match, queue_full, vendor_down), enumerate them as errors: [{ reason, code, when, recovery, retryable? }] on the definition. recovery is required metadata — the agent's next move when this failure fires (≥ 5 words, lint-validated; the framework sends it on the wire as data.recovery.hint with any failure carrying that reason and no hint of its own). The framework types ctx.fail(reason, …) against the declared reason union (typos become TS errors) and auto-populates data.reason on the thrown error for stable observability. The error reaches clients with parity across both surfaces — structuredContent.error (Claude Code) and content[] text (Claude Desktop). Baseline codes (InternalError, ServiceUnavailable, Timeout, ValidationError, SerializationError, RequestCancelled) bubble from anywhere and don't need to be enumerated. Mark an entry the service layer throws, rather than the handler, with thrownBy: 'service' so the conformance lint doesn't report it as a reason the handler never raises. See api-errors skill for the full pattern.
Classify errors by origin. Different error sources need different codes and different recovery guidance. Map the failure modes for each tool during design:
| Origin | Examples | Error code | Agent can recover? |
|---|---|---|---|
| Client input | Bad ID format, invalid params, missing required field, out-of-range value | ValidationError | Yes — fix the input and retry |
| Upstream API | 5xx, network error | ServiceUnavailable | Maybe — retry later, or the upstream is down |
| Timeout | Upstream 408/504, a fetch that ran out its timeout, an exhausted retry deadline | Timeout | Maybe — retry, or narrow the request so it finishes sooner |
| Rate limit | 429, quota exhausted, queue full | RateLimited (retryable: true; withRetry honors Retry-After) | Yes — wait, then retry or reduce frequency |
| Not found | Valid ID format but entity doesn't exist | NotFound (or ValidationError if ambiguous) | Yes — check the ID, try a search |
| Conflict | Duplicate key, version mismatch, concurrent modification on a write | Conflict | Yes — re-read current state, then retry with it |
| Caller auth | Insufficient scopes, expired token, a rejected key the caller supplied | Forbidden / Unauthorized | Maybe — escalate or re-auth |
| Server credential | The server's own upstream key is missing, or the upstream rejects it (401/403) | ConfigurationError — translated in the service, since the automatic status mapping yields Unauthorized/Forbidden, which read as the caller's credentials | No — the operator fixes it; the recovery names the env var |
| Server internal | Parse failure, missing config, unexpected state | InternalError | No — server-side issue |
(InvalidParams also exists — the framework's parseToolArguments emits it when input fails Zod schema validation before the handler runs. Anything the handler itself throws about inputs uses ValidationError.)
The framework auto-classifies many of these at runtime (HTTP status codes, JS error types, common patterns), but explicit classification in the handler gives better error messages. For declared contract failures, throw via ctx.fail('reason', …). For ad-hoc throws outside the contract, use error factories (notFound(), validationError(), etc.) when the code matters; plain throw new Error() when the framework's auto-classification is good enough.
Expected misses are results, not errors. When a tool's whole job is resolving one identifier — a citation, a code, a name → ID — a no-match is an expected outcome the agent must reason about, not a failure. Return { found: false, guidance } instead of throwing, and treat guidance as a first-class recovery surface: per miss outcome, say what didn't parse or resolve and route to the named tool that can recover (the broader search tool, the reference tool). Agents self-correct better from a structured miss than from a throw — a throw reads as "something broke," a miss result reads as "adjust and retry." The split with search tools: a search's empty result is a valid empty collection, so its notice rides enrichment; a resolver's miss is the primary result, so found and its guidance live in output.
Write error messages as recovery instructions. The message is the agent's only signal for what to do next — and the strongest recovery instruction ends in a named tool call, never a bare "check your input."
// Bad — dead end, no recovery path
throw new Error('Not found');
// Good — names both resolution options
"No session working directory set. Please specify a 'path' or use 'git_set_working_dir' first."
// Good — structured hint in error data using the canonical `data.recovery.hint` shape.
// The framework mirrors `data.recovery.hint` into the content[] text as
// `Recovery: <hint>` so format()-only clients (Claude Desktop) see the same
// guidance structuredContent clients (Claude Code) read from `error.data.recovery.hint`.
// A hint the message already contains verbatim is dropped from the text rather
// than stated twice, and stays on structuredContent either way.
throw forbidden(
"Cannot perform 'reset --hard' on protected branch 'main' without explicit confirmation.",
{
branch: 'main',
operation: 'reset --hard',
recovery: { hint: 'Set the confirmed parameter to true to proceed.' },
},
);
// Good — upstream error with actionable context
throw notFound(`Paper '${id}' not found on arXiv. Verify the ID format (e.g., '2401.12345' or '2401.12345v2').`);During design, settle the full contract for each tool — reason, code, when-clause, and the verbatim recovery string — in the tool's section of the design doc; they become the literal errors: [...] entries during scaffolding. Hold every recovery string (and zero-hit notice, and resolver guidance) to the no-dead-ends rule: it names the concrete next tool call, with the reference tool as the most common routing target. Settled at design time these stay sharp; left to implementation they degrade into "check your input." Not every failure needs a contract entry; baseline infrastructure errors (5xx, timeouts, validation) are fine to let bubble.
A routing target must be callable in the deployment doing the routing. A recovery string, notice, or guidance line that names a config-gated tool is a dead end wherever that gate is off — the agent is sent to a tool absent from tools/list, at the moment it is already recovering from a failure. Prefer routing to ungated tools (the reference tool is a good target precisely because nothing gates it). Where the target genuinely is gated, resolve the text from the same config that decides registration, and say the capability is unavailable in this deployment rather than naming a call that cannot be made. Structured follow-ups are stricter still — see Instruction tools above.
Summarize each tool:
| Aspect | Decision |
|---|---|
| Name | Lowercase snake_case with a canonical server prefix. 3 segments is the strong default ({server}_{verb}_{noun} — e.g., pubmed_search_articles, clinicaltrials_find_eligible). 2 is fine only when the verb is a complete action whose object the domain implies (git_pull, git_push, git_status, git_commit — the remote, working tree, or repo is implicit); don't invent a word to pad those to 3. A verb that takes a caller-specified object — search, find, get, list, query, fetch, connect, create, update — always carries its noun (ontology_search → ontology_search_terms, geofeatures_connect → geofeatures_connect_endpoint). Smell test: if {server}_{verb} leaves "…what?" unanswered, the noun is missing. 4 is fine when the noun is inherently two words (openfda_search_device_clearances) or the prefix is multi-part. The prefix is judged on clarity, not length: the brand name or the plain well-known word for the domain both pass (pubmed_, patents_, earthquake_); an abbreviation fails only when it reads as something else out of context (loc_ → lines of code, ct_ → CT scan). The verb+noun pair should be unambiguous within the server — if two tools could plausibly share a name, the noun isn't specific enough (read_fulltext not read_text when structured metadata is a separate concept). Treat name length as a scope smell only when the extra segment is the verb overreaching (e.g., foo_create_and_send_notification → split or use modes). |
| Granularity | Scope each tool to one coherent agent action. The implementation can be a single API call (pubmed_search_articles), a multi-step workflow, or internal-only — match the unit to the work, don't constrain by call count. |
| Description | Concrete capability statement. Add operational guidance (prerequisites, constraints, gotchas) when non-obvious. |
| Input schema | .describe() on every field. Constrained types (enums, literals, regex). Explain costs/tradeoffs of parameter choices. |
| Output schema | Designed for the LLM's next action. Include chaining IDs. Communicate filtering. Post-write state where useful. |
| Errors | Declare domain failure modes as a typed contract (errors: [{ reason, code, when, recovery, retryable? }]) so ctx.fail is type-checked and capable clients can preview failures via tools/list. Every recovery string follows the no-dead-ends rule — it names the next tool call. |
| Enrichment | The success-path counterpart to errors: declare the agent-facing context fields the handler populates via ctx.enrich(...) — zero-hit notice, echoed defaults, totals, truncation — with a kind-tag (notice/total/echo/delta) where one fits and an enrichmentTrailer.render for any structured field. Keys stay disjoint from output. Declare a field required only when every path writes it — a required field one branch skips fails the output parse on every call that takes another branch. |
| Annotations | readOnlyHint, destructiveHint, idempotentHint, openWorldHint. Helps clients auto-approve safely. destructiveHint defaults to true on any tool that isn't read-only, so a benign write must set destructiveHint: false explicitly; a read-only tool omits it entirely (annotation-coherence lint). |
| Auth scopes | tool:<snake_tool_name>:<verb> or resource:<kebab-resource-name>:<verb> (e.g., tool:inventory_search:read, resource:echo-app-ui:read). Domain-led <domain>:<verb> (e.g., inventory:read) is an acceptable alternative — pick one convention per server and stay consistent. Skip when no deployment will run MCP_AUTH_MODE=jwt or oauth — under none (stdio, or single-tenant HTTP) scope checks never run. |
Resources are supplementary — a convenience for clients that support injectable context via stable URIs. Since many clients are tool-only, verify that any data exposed via resources is also reachable through the tool surface. This doesn't require a 1:1 resource-to-tool mapping — the data might be covered by an existing tool's output, bundled into a broader tool, or warrant its own dedicated tool, depending on the server's purpose and how agents will use it.
For each resource:
| Aspect | Decision |
|---|---|
| URI template | scheme://{param}/path. Server domain as scheme. Keep shallow. |
| Params | Minimal — typically just an identifier. Complex queries belong in tools. |
| Pagination | Needed if lists exceed ~50 items. Opaque cursors via extractCursor/paginateArray. |
| list() | Provide if discoverable. Top-level categories or recent items, not exhaustive dumps. |
| Cache hint | cacheHint: { ttlMs, cacheScope } on 2026-07-28 connections. Static reference data is public with a long TTL; per-tenant or per-session data is private or uncached. |
| Completion | Template params with a bounded vocabulary (a species list, a dataset code) get argument completion so clients can offer valid values; the same completable() wrapper applies to prompt args. |
| Tool coverage | Verify the data is reachable via tools — either a dedicated tool, included in another tool's output, or not needed for tool-only agents. |
Optional. Use when the server has recurring interaction patterns worth structuring:
Wrap enum-like prompt args in completable() so clients can offer valid values. Skip prompts entirely for purely data/action-oriented servers.
Services — one per external dependency (or per source, for multi-source servers). Init/accessor pattern. Skip if all tools are thin wrappers with no shared state. For multi-source servers, each upstream API gets its own service with its own auth, rate limits, and retry config — tools compose across services internally, agents never see the service boundary.
Server-as-service. When the server IS the source of truth (knowledge graph, in-memory task tracker, local scratchpad, embedded inference wrapper), the resilience table below doesn't apply — there's no upstream to retry. The design questions shift to state management: what's tenant-scoped vs. global, what TTLs apply, what survives a restart, what the storage backend is. Plan persistence via ctx.state for tenant-scoped KV (auto-namespaced by tenantId), or use a StorageService provider directly when data must cross tenants. Service init still happens in setup(), accessed via getMyService() at request time. Calls within the server are local and synchronous-ish — the API-efficiency table below also doesn't apply.
Analytical API servers: DataCanvas is one option. For servers that fetch analytical data — result sets an agent runs SQL over (aggregate, group, join, time-series) — and want to expose a SQL workspace, the framework's optional DataCanvas primitive (Tier 3, opt-in via CANVAS_PROVIDER_TYPE=duckdb) handles lifecycle, ID generation, eviction, and export wiring so you don't design your own. It earns its keep on shape, not size: a discovery/search surface returning categorical metadata (titles, IDs, types) — where the workflow is find-the-record-then-drill-in — does not qualify even when the result is large; resolve names over a bounded set with MCP-side list filtering instead. If you opt in, the consumer tools are mandatory: a tool that emits a canvas_id MUST be paired with a dataframe_query (and dataframe_describe) tool in the same surface — a canvas_id with no query tool is dead output the agent can't reach. Surface canvas_id as an optional input on register/query/export tools; the framework mints on omit and resolves on match. The accessor is wired once in setup() via setCanvas(core.canvas) (undefined when disabled or running on Cloudflare Workers — DuckDB has no V8-isolate build). See api-canvas for the full reference.
For services wrapping external APIs, plan the resilience layer.
| Concern | Decision |
|---|---|
| Retry boundary | Service method wraps full pipeline (fetch + parse), not just the network call. Use withRetry from /utils. |
| Backoff calibration | Match base delay to upstream recovery time: 200–500ms (ephemeral), 1–2s (rate-limited), 2–5s (degraded). |
| HTTP status check | fetchWithTimeout already handles this — a non-2xx throws an McpError whose code is mapped from the status (400 → InvalidParams, 401 → Unauthorized, 403 → Forbidden, 404 → NotFound, 409 → Conflict, 422 → ValidationError, 429 → RateLimited, 408/504 → Timeout, other 5xx → ServiceUnavailable; the full table is api-errors § HTTP Response → McpError), with status and body on error.data, plus retryAfter when the upstream sent one. Plan error contracts around those codes, not a blanket ServiceUnavailable. |
| Parse failure classification | Response handler detects HTML error pages and throws transient errors, not SerializationError. |
| Exhausted retry messaging | withRetry enriches the final error with attempt count automatically. |
| Total deadline | Retries multiply a per-attempt timeout: four 30s attempts plus backoff outlast a client's 60s request timeout, and the caller gets a transport timeout instead of this server's classified error. Size withRetry's deadlineMs to fit inside the client's timeout and thread attempt.signal into each fetch (api-utils). |
| Pacing | When the upstream mandates a rate (one request per second, N per minute, N concurrent), put a createPacer from /utils in front of that service — one pacer per upstream budget, composed as withRetry outside and the pacer inside — and note the resulting ceiling in the server instructions when it shapes how an agent should batch work. |
| Caller-supplied URLs or hosts | Route through fetchWithTimeout with rejectPrivateIPs: true — the SSRF guard is off by default. It blocks private, loopback, link-local, and metadata ranges, and is best-effort: DNS rebinding still gets past it, so a deployment that needs hard isolation adds egress controls. Never a bare fetch on a caller-controlled destination; security-pass audits this sink. |
For API efficiency, design the service methods to minimize upstream calls:
| Concern | Decision |
|---|---|
| Batch over N+1 | If the API supports filter-by-IDs or bulk-GET endpoints, use a single batch request instead of N individual fetches. Cross-reference the response against requested IDs to detect missing items. |
| Field selection | If the API supports fields/select parameters, request only the fields the tool needs. A full study record might be 70KB; selecting 4 fields might be 5KB. |
| Request consolidation | When a tool needs data from multiple related endpoints, check if a single endpoint with broader field selection can serve the same data in one round trip. |
| Pagination awareness | If a batch request might exceed the API's page size, either paginate internally or assert/throw when results are truncated so callers aren't silently missing data. |
Config — list env vars (API keys, base URLs). Goes in src/config/server-config.ts as a separate Zod schema.
Create docs/design.md with the structure below. The MCP surface (tools, resources, prompts) goes first — it's what matters most and what the developer will reference during implementation.
# {{Server Name}} — Design
## MCP Surface
### Tools
| Name | Description | Key Inputs | Annotations |
|:-----|:------------|:-----------|:------------|
### Resources
| URI Template | Description | Pagination |
|:-------------|:------------|:-----------|
### Prompts
| Name | Description | Args |
|:-----|:------------|:-----|
## Overview
What this server does, what system it wraps, who it's for.
## Requirements
- Bullet list of capabilities and constraints
- Auth requirements, rate limits, data access scope
- Deployment targets (stdio / HTTP / Workers), session mode, upstream credential model, and the terms-of-use constraints from Step 1
## User Goals
1. Numbered outcomes agents will accomplish (from Step 2) — every tool in the surface traces back to one
## Tools — detail
One subsection per tool: a param table (param | type | maps-to | notes), the output field
list, the error contract table (reason | code | when | recovery — verbatim strings), and
zero-hit notice fragments for search tools. This is the section implementation reads
tool-by-tool.
## Resources — detail / ## Prompts — detail
Same treatment when the server has them: one subsection per resource (URI template, params,
cache hint, the tool that also covers the data) and per prompt (args, completions, the
message shape).
## Services
| Service | Wraps | Used By |
|:--------|:------|:--------|
## Config
| Env Var | Required | Description |
|:--------|:---------|:------------|
## Server Instructions
Draft `instructions` string for `createApp()` — the orientation every client sees at
initialize: canonical workflow chain, identifier semantics, rate-limit posture. A short
paragraph under 2,048 characters, essentials first — Claude Code truncates the string at that
length, mid-sentence. Tool descriptions carry the rest.
## Implementation Order
1. Config and server setup
2. Reference tool (static, no service dependency — grounds field-testing for everything else)
3. Services (external API clients)
4. Read-only tools
5. Write tools
6. Resources
7. Prompts
Each step is independently testable.
<!-- Optional sections — include when the trigger fires: -->
## Domain Mapping <!-- nouns × operations → API endpoints; include when ≥3 nouns each with ≥3 operations -->
## Workflow Analysis <!-- how tools chain for real tasks; include when any tool makes ≥3 upstream calls -->
## Design Decisions <!-- rationale for consolidation, naming, tradeoffs; include when a choice would otherwise be opaque -->
## Known Limitations <!-- inherent API/data constraints the server can't solve; include when a constraint visibly caps utility -->
## API Reference <!-- query language, pagination, rate limits; include when worth documenting -->Keep it concise. The design doc is a working reference, not a spec document — enough to orient a developer (or agent) implementing the server, not more. It stays true after the build: when the implementation diverges — a renamed field, a dropped method, a changed error code — the doc changes in the same commit, with the why under Design Decisions. The next agent reads it as the spec.
Workflow Analysis example. For multi-step workflow tools, document the upstream call sequence in a table — it drives several downstream decisions during implementation: the service-layer method shape, retry boundaries, where cleanup or the confirmation round belongs, and what post-action state to fetch for the response.
deploy_release (5–8 upstream calls, plus a confirmation round):
| # | Call | Purpose | Mode gate |
|---|---|---|---|
| 0 | ctx.requestInput confirmation | Human approval before promote | promote |
| 1 | POST /releases | Create release record | always |
| 2 | PUT /releases/{id}/artifacts | Attach build artifacts | always |
| 3 | GET /releases/{id}/preflight | Health checks, smoke tests | always |
| 4 | POST /releases/{id}/canary | Deploy to 5% of traffic | canary |
| 5 | POST /releases/{id}/promote | Roll out to 100% | promote |
| 6 | POST /releases/{id}/rollback | Restore previous version | rollback |
| 7 | GET /releases/{id} | Post-action state for response | always |
| — | DELETE /releases/{id}/canary-traffic | Cleanup canary if mid-flow error | on error + cleanupOnError |
The table surfaces design questions early: should the confirmation round happen before or after the artifacts are attached? Does cleanup drop the canary on any failure, or only failures past the promote step? What does the response body need from the final GET — version, traffic percentage, health summary? Answering these during design is far cheaper than mid-implementation.
If the user has already authorized implementation — any message that contains both a design request and a build/implement verb in the same clause (e.g., "build me a ___ server", "design and implement a ___") — proceed directly to scaffolding using the design doc as the plan. Otherwise, present the design doc to the user for review before implementing.
Execute the plan using the scaffolding skills:
add-service for each serviceadd-tool for each standard tooladd-resource for each standalone resourceadd-prompt for each promptadd-app-tool only if any app tools survived the design step (rare — see the App Tool row in Step 3)add-test alongside each definition, declared error contracts includeddevcheck after each additionOnce the surface is built, tool-defs-analysis audits the definition language and field-test exercises the tools against the live upstream.
Items without an If …: prefix apply to every design. Conditional items only apply when the trigger fires — otherwise skip them.
destructiveHint.describe() text explains what the value is, what it affects, and tradeoffsrecovery naming the expected shape; any composite form tried as-given before a composed retryformat() renders all data the LLM needs — different clients forward different surfaces (Claude Code → structuredContent, Claude Desktop → content[]); both must carry the same data, not just a count or titleerrors: [{ reason, code, when, recovery, retryable? }]) with verbatim recovery strings settled in the design doc, each naming the next tool call{ found: false, guidance } — a result, not a throw — with guidance routing per miss outcometopic enum), implemented first, and used as the routing target in recovery strings and noticesreadOnlyHint, destructiveHint, idempotentHint, openWorldHint) — benign writes set destructiveHint: false explicitly, read-only tools omit itinstructions string drafted — workflow chain, identifier semantics, rate-limit posture, under 2,048 characters (ships via createApp() on every initialize)mode/operation enum — as a z.discriminatedUnion input when the arms need different required fieldsfilter/nameContains, not query), filtering the full set, strict token match (fuzzy only when a caller needs typo tolerance)nextToolSuggestions pre-filled from diagnosticsguidance name a callable target or state the capability is unavailable, and structured follow-ups naming it are emitted only under the config that registers itctx.requestInput confirmation that redeems a ctx.state consent record bound to the operation, caller, and target (shared storage when retries can reach another instance, MCP_REQUEST_STATE_KEY set, an arm that must not run twice idempotent per record), with destructiveHint annotation so clients that never fulfil the round still surface the riskctx.requestInput: createApp() declares sessionMode with require: 'stateful'format() quotes or fences free text and flattens CR/LF in inline slots, and the server instructions say the content is datamode: 'preview', dryRun: true, confirmCount required)0 = act and return), pre-probe then poll with early return, every outcome a result rather than a throw, and a read-only sibling tool that probes the same stateformat()-twin maintenance cost — otherwise dropped in favor of a standard tool{param} templates, pagination planned for large listsrejectPrivateIPs on caller-supplied URLs)api-canvas skill), and it earns its keep on analytical fit (an agent would SQL it), not row count — a discovery/search surface of categorical metadata doesn't qualify. Any tool emitting a canvas_id is paired with a dataframe_query (+ dataframe_describe) tool in the same surface — a token with no query tool is dead outputserver-config.tsdocs/design.md — every example value synthetic, no keys, tokens, or private hosts© cyanheads, Apache-2.0. Rendered from Markdown: HTML in the file is shown as text, images as links, and headings moved down two levels. Raw file
Just SKILL.md in framework-skills/design-mcp-server of cyanheads/pubmed-mcp-server.
Open the folder on GitHubat commit 79145a6
Design MCP Server next to the 5 skills that share the most tags, products or categories with it. Stars are the repository's; “used in” counts other GitHub owners with a copy.
| Skill | Stars | Used in | Tokens | Auto-check | Licence | Repo updated |
|---|---|---|---|---|---|---|
| Design MCP Server this skillcyanheads/pubmed-mcp-server | 158 | — | ~22k | Automated safety check: Pass | Apache-2.0 | |
| Ogham Researchogham-mcp/ogham-mcp | 115 | — | ~1.4k | Automated safety check: Pass | MIT | |
| MCP Google Map Projectcablate/mcp-google-map | 469 | — | ~774 | Automated safety check: Pass | MIT | |
| Memory686f6c61/alfred-dev | 117 | — | ~601 | Automated safety check: Pass | MIT | |
| Oia Manifestruvnet/metaharness | 696 | — | ~910 | Automated safety check: Pass | MIT | |
| MCP Developmentcoollabsio/coolify | 63k | 1 repos | ~949 | Automated safety check: Pass | MIT |
ogham-mcp/ogham-mcp
Structured memory capture for Ogham shared memory. An agent skill from ogham-mcp/ogham-mcp.
cablate/mcp-google-map
Project knowledge for developing and maintaining @cablate/mcp-google-map.
686f6c61/alfred-dev
This skill should be used when the user asks to record a design decision, search past project decisions, inspect the Alfred memory timeline, or work with the alfred-memory MCP server.
ruvnet/metaharness
Emit .harness/oia-manifest.json declaring layer alignment with the OIA v0.1 9-layer reference architecture.
coollabsio/coolify
A skill your agent uses for Laravel MCP development. An agent skill from coollabsio/coolify.
ruvnet/metaharness
7-section readiness scorecard for a LOCAL repo. An agent skill from ruvnet/metaharness.
cyanheads/pubmed-mcp-server
Scaffold an MCP App tool + UI resource pair. An agent skill from cyanheads/pubmed-mcp-server.
cyanheads/pubmed-mcp-server
Scaffold a new MCP prompt template. An agent skill from cyanheads/pubmed-mcp-server.
cyanheads/pubmed-mcp-server
Scaffold a new MCP resource definition. An agent skill from cyanheads/pubmed-mcp-server.
cyanheads/pubmed-mcp-server
Scaffold a new service integration. An agent skill from cyanheads/pubmed-mcp-server.
cyanheads/pubmed-mcp-server
Scaffold a test file for an existing tool, resource, or service.
cyanheads/pubmed-mcp-server
Authentication, authorization, and multi-tenancy patterns for @cyanheads/mcp-ts-core.
Works with
Categories
Design the tool surface, resources, and service layer for a new MCP server. Design MCP Server is an agent skill from cyanheads/pubmed-mcp-server. Design the tool surface, resources, and service layer for a new MCP server.
Design MCP Server fits situations like: starting a new server; planning a major feature expansion; the user describes a domain/API they want to expose via MCP.
Run `npx skills add cyanheads/pubmed-mcp-server --skill design-mcp-server -a claude-code`. Or copy the skill folder (framework-skills/design-mcp-server in cyanheads/pubmed-mcp-server) into .claude/skills/design-mcp-server in your project. Claude Code loads it when a task matches its description.
Run `npx skills add cyanheads/pubmed-mcp-server --skill design-mcp-server -a codex`. Or copy the skill folder (framework-skills/design-mcp-server in cyanheads/pubmed-mcp-server) into .agents/skills/design-mcp-server in your project. Codex loads it when a task matches its description.
Cursor, Gemini CLI, GitHub Copilot and OpenCode also load SKILL.md folders. With the skills CLI, run `npx skills add cyanheads/pubmed-mcp-server --skill design-mcp-server -a cursor` (or -a gemini-cli, github-copilot or opencode for the others). To copy it by hand, put the folder in .cursor/skills/design-mcp-server, .gemini/skills/design-mcp-server, .github/skills/design-mcp-server and .opencode/skills/design-mcp-server in your project.
Going by SKILL.md and its folder, Design MCP Server needs credentials named MCP_REQUEST_STATE_KEY.
SKILL.md names 1 domain. As links in the text: github.com. This is read from the text; nothing was executed.
Our automated static check of SKILL.md found no risky patterns, such as piping downloads into a shell, reading credential files or hidden Unicode. It is not a guarantee. Review the folder before installing.
Design MCP Server is published under the Apache-2.0 licence (the repository's licence). It allows redistribution, so the full SKILL.md is shown on this page.
About 22k tokens (SKILL.md is roughly 89k characters). Agents keep only the skill's name and description in context until a task matches; then they load SKILL.md in full.
Skills that share tags, products or a category with Design MCP Server: Ogham Research (ogham-mcp/ogham-mcp, 115 stars), MCP Google Map Project (cablate/mcp-google-map, 469 stars), Memory (686f6c61/alfred-dev, 117 stars) and Oia Manifest (ruvnet/metaharness, 696 stars). The comparison table on this page puts their stars, adoption, token cost, safety result and licence side by side.
cyanheads (a GitHub user) maintains it in cyanheads/pubmed-mcp-server, which has 158 GitHub stars. The repository holds 30 skills in this directory. The repository was last updated on October 9, 2026.
Source: cyanheads/pubmed-mcp-server on GitHub. Facts on this page come from the repository at the commit we read; the author's words are quoted as theirs.