Skip to content

fix(filesystem): fix structuredContent tests and update dependencies#3270

Open
forbiddancherry wants to merge 1 commit into
modelcontextprotocol:mainfrom
forbiddancherry:fix/structuredContent-tests
Open

fix(filesystem): fix structuredContent tests and update dependencies#3270
forbiddancherry wants to merge 1 commit into
modelcontextprotocol:mainfrom
forbiddancherry:fix/structuredContent-tests

Conversation

@forbiddancherry

Copy link
Copy Markdown

Summary

  • Add tools.structuredContent capability to test client to receive structured content from server (MCP spec 2025-06-18)
  • Use fs.realpath() for temp directory to resolve macOS symlinks (/var -> /private/var)
  • Update vitest to v4.0.18 to fix security vulnerabilities
  • Update @vitest/coverage-v8 to v4.0.18

Test plan

  • All 195 tests pass
  • npm audit shows 0 vulnerabilities
  • Build succeeds

🤖 Generated with Claude Code

- Add tools.structuredContent capability to test client to receive
  structured content from server (MCP spec 2025-06-18)
- Use fs.realpath() for temp directory to resolve macOS symlinks
  (/var -> /private/var)
- Update vitest to v4.0.18 to fix security vulnerabilities
- Update @vitest/coverage-v8 to v4.0.18

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@cliffhall cliffhall added bug Something isn't working server-filesystem Reference implementation for the Filesystem MCP server - src/filesystem labels Apr 20, 2026
@BossChaos

This comment was marked as abuse.

@LuuOW LuuOW left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Technical audit: Verified MCP server implementation for consistency with current SDK patterns.

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

Labels

bug Something isn't working server-filesystem Reference implementation for the Filesystem MCP server - src/filesystem

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants