Skip to content

Conversation

@robertyates
Copy link

added a read_binary_file tool to the file system MCP Server

Description

Server Details

  • Server: filesystem
  • Changes to: added read_binary_file tool

Motivation and Context

I was trying to use the existing server to fetch an excel file and it fails

How Has This Been Tested?

I have a custom agent that this works in and also include a unit test

Breaking Changes

pure additive, although read_file should likely now be removed vs. just being marked deprecated

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 Protocol Documentation
  • My changes follows MCP security best practices
  • I have updated the server's README accordingly
  • I have tested this with an LLM client
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have documented all environment variables and configuration options

Additional context

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.

2 participants