Skip to content

docs(server): disclose play_audio's file-folder walking behavior in tool description (#78)#83

Open
TumCucTom wants to merge 1 commit into
MiniMax-AI:mainfrom
TumCucTom:fix/mcp-78-info-disclosure
Open

docs(server): disclose play_audio's file-folder walking behavior in tool description (#78)#83
TumCucTom wants to merge 1 commit into
MiniMax-AI:mainfrom
TumCucTom:fix/mcp-78-info-disclosure

Conversation

@TumCucTom
Copy link
Copy Markdown

Summary

  • Update play_audio tool description to disclose that process_input_file walks the parent directory for similar filenames when the exact path is not found.
  • (Optionally) add a matching note in README.md / README-CN.md if either already documents the tool.

Why

Issue #78: clients and users currently have no way to know that the server may look beyond the literal path they passed. Disclosing the behavior in the tool description lets MCP clients make an informed trust decision.

Changes

  • minimax_mcp/server.py: extended the play_audio tool description= string with a sentence about the parent-directory filename walking.
  • README.md (and/or README-CN.md): one-line disclosure note in any existing play_audio section, if present.

Testing

  • Docs change only; no new tests.
  • pytest tests/ still green.

Fixes #78.

Generated with Claude Code

…ool description

play_audio relies on process_input_file(), which automatically searches the
parent directory for similar filenames when the exact path is not found.
This behavior was not mentioned in the tool description, so MCP clients
had no way to know up front that the server would look beyond the
literal path they provided.

Disclose the behavior in the tool description so clients/users can make
an informed trust decision. Fixes MiniMax-AI#78.

Co-authored-by: Zippy AI <tomkinsbale@icloud.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
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.

Information disclosure for play_audio

1 participant