Skip to content

🐛 🔒 Instructions requirements for MCP#41

Merged
HanXHX merged 1 commit into
masterfrom
mcp_description
May 15, 2026
Merged

🐛 🔒 Instructions requirements for MCP#41
HanXHX merged 1 commit into
masterfrom
mcp_description

Conversation

@HanXHX
Copy link
Copy Markdown
Contributor

@HanXHX HanXHX commented May 15, 2026

  • It runs locally (prevent some issue when running in Windows while Claude thinks this MCP is a remote one)
  • Never read local files

- It runs locally (prevent some issue when running in Windows while
  Claude thinks this MCP is a remote one)
- Never read local files
Copilot AI review requested due to automatic review settings May 15, 2026 16:07
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Strengthens MCP tool descriptions to make explicit that the RETYC MCP server runs locally, performs in-process E2EE encryption/decryption, and that the calling LLM must not read or relay file contents. This addresses an issue where Claude treated the server as remote (causing path issues on Windows) and helps enforce zero-knowledge handling of files.

Changes:

  • Expanded the MCP server's top-level description to specify local execution, post-quantum AGE encryption, and a strict zero-knowledge directive.
  • Updated transfer_send, transfer_download, dataroom_upload, and dataroom_download tool descriptions to reiterate local encryption/decryption and forbid the LLM from previewing files.
  • Augmented the files array description for transfer_send to include the platform-specific absolute-path hint.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@HanXHX HanXHX merged commit 3c8e052 into master May 15, 2026
8 checks passed
@HanXHX HanXHX deleted the mcp_description branch May 15, 2026 16:09
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