Skip to content

feat(skills): check ttlMs/cacheScope on resources/directory/read results - #559

Open
sambhav wants to merge 1 commit into
modelcontextprotocol:mainfrom
sambhav:skills-directory-read-cacheable
Open

sambhav wants to merge 1 commit into
modelcontextprotocol:mainfrom
sambhav:skills-directory-read-cacheable

Conversation

@sambhav

@sambhav sambhav commented Oct 15, 2026

Copy link
Copy Markdown
Member

ext-skills#157 made ReadResourceDirectoryResult extend CacheableResult, so from protocol 2026-07-28 a directory read result MUST carry ttlMs and cacheScope, matching resources/list and the existing skills/list and skills/get checks.

Adds sep-2640-directory-read-cache-attributes to the directory scenario, gated on the negotiated protocol version like its skills/list and skills/get siblings, the matching row in sep-2640.yaml, and negative controls covering each missing field.

Claude-Session: https://claude.ai/code/session_01QSiqKZYUECJYpKAY7qg4Cp

Motivation and Context

How Has This Been Tested?

Breaking Changes

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

ext-skills#157 made ReadResourceDirectoryResult extend CacheableResult, so
from protocol 2026-07-28 a directory read result MUST carry ttlMs and
cacheScope, matching resources/list and the existing skills/list and
skills/get checks.

Adds sep-2640-directory-read-cache-attributes to the directory scenario,
gated on the negotiated protocol version like its skills/list and skills/get
siblings, the matching row in sep-2640.yaml, and negative controls covering
each missing field.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QSiqKZYUECJYpKAY7qg4Cp
@pkg-pr-new

pkg-pr-new Bot commented Oct 15, 2026

Copy link
Copy Markdown

Open in StackBlitz

npx https://pkg.pr.new/@modelcontextprotocol/conformance@559

commit: 950c37e

This branch has not been deployed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant