Mcp server best practices

[kentcdodds]

Adds MCP Server Best Practices documentation and updates the example server to follow these guidelines for improved AI interaction and discoverability.

---

 

---

Note

Low Risk
Primarily documentation and metadata/UX improvements for an example tool; behavior changes are small and covered by updated E2E assertions.

Overview
Adds docs/mcp-server-best-practices.md, capturing recommended MCP patterns (server instructions, tool descriptions/annotations, schemas, responses, etc.).

Refactors the example MCP server to consume centralized server-metadata.ts for server identity/instructions and do_math tool metadata/annotations, and upgrades do_math to return markdown content plus structuredContent, validate inputs (finite numbers, precision), and emit actionable error responses (e.g., division by zero/non-finite results). E2E tests are updated to assert presence of onboarding instructions and verify structuredContent.result alongside text output.

^{Written by Cursor Bugbot for commit 2ff2860. This will update automatically on new commits. Configure here.}

[cursor]

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
_{Learn more about Cursor Agents}

[github-actions]

🔎 Preview deployed: https://epicflare-pr-32.kentcdodds.workers.dev

Worker: epicflare-pr-32
D1: epicflare-pr-32-db
KV: epicflare-pr-32-oauth-kv

Mocks:

• resend: https://epicflare-pr-32-mock-resend.kentcdodds.workers.dev/__mocks (epicflare-pr-32-mock-resend)

[cursor]

Bugbot Autofix prepared fixes for 1 of the 1 bugs found in the latest run.

• ✅ Fixed: Arithmetic overflow silently corrupts structured result to null
  • Added a non-finite result guard that returns an explicit error instead of emitting an unrepresentable value in structuredContent.

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{Bugbot Autofix is ON. A Cloud Agent has been kicked off to fix the reported issue.}