Skip to content

Conversation

@faridjaff
Copy link

@faridjaff faridjaff commented Jan 13, 2026

Motivation and Context

The CallToolRequest schema link in docs/specification/2025-11-25/server/tools.mdx was broken. It used a relative path (specification/2025-11-25/schema#calltoolrequest) which resolved incorrectly when published, producing:

  • Broken URL: https://modelcontextprotocol.io/specification/2025-11-25/server/specification/2025-11-25/schema#calltoolrequest
  • Expected URL: https://modelcontextprotocol.io/specification/2025-11-25/schema#calltoolrequest

How Has This Been Tested?

  • Ran npm run check:docs:links - no broken links found

Breaking Changes

None

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

This PR was assisted by Claude Code for research and planning.

Add missing leading slash to make the link absolute, fixing incorrect URL resolution that was producing a duplicate path segment.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@faridjaff faridjaff requested a review from a team as a code owner January 13, 2026 11:17
@localden localden self-assigned this Jan 14, 2026
@localden localden added the documentation Improvements or additions to documentation label Jan 14, 2026
@localden
Copy link
Contributor

This LGTM. @faridjaff can you please also make this change in the draft version of the spec? Once that's done, we can merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants