Skip to content

Document Armorer Guard MCP proxy#691

Open
armorer-labs wants to merge 1 commit into
lastmile-ai:mainfrom
armorer-labs:feature/add-armorer-guard-mcp-proxy
Open

Document Armorer Guard MCP proxy#691
armorer-labs wants to merge 1 commit into
lastmile-ai:mainfrom
armorer-labs:feature/add-armorer-guard-mcp-proxy

Conversation

@armorer-labs

@armorer-labs armorer-labs commented May 19, 2026

Copy link
Copy Markdown

Adds a short optional configuration example showing how to wrap a stdio MCP server with armorer-guard mcp-proxy -- ....

This gives mcp-agent users a local guardrail path for checking MCP tool-call arguments for prompt injection, credentials, exfiltration risk, and dangerous actions before forwarding the call to the underlying server.

@coderabbitai

coderabbitai Bot commented May 19, 2026

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

README.md is updated with a configuration example demonstrating how to wrap the filesystem MCP server command using Armorer Guard. The addition shows replacing the direct npx @modelcontextprotocol/server-filesystem`` invocation with an armorer-guard mcp-proxy -- wrapper to route tool calls through the guard mechanism.

Changes

Armorer Guard Filesystem Integration Documentation

Layer / File(s) Summary
Filesystem server command wrapping with Armorer Guard
README.md
Configuration example added showing how to wrap the filesystem MCP server using armorer-guard mcp-proxy -- wrapper in mcp_agent.config.yaml.

Estimated Code Review Effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested Reviewers

  • saqadri
  • StreetLamb

Poem

A rabbit reads the docs with glee,
Guard wraps the files, now safe and free,
mcp-proxy shields with armor bright,
Configuration guides see the light,
One line of change, simplicity's might! 🐰✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Document Armorer Guard MCP proxy' directly matches the PR's main objective of adding documentation for wrapping MCP servers with Armorer Guard.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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