Skip to content

Phase 1: MCP firewall adapter, audit, AgentBOM, CI action, API - #3

Merged
new-world-coder merged 1 commit into
mainfrom
cursor/phase1-mcp-firewall-f5d6
Aug 11, 2026
Merged

new-world-coder merged 1 commit into
mainfrom
cursor/phase1-mcp-firewall-f5d6

Conversation

@new-world-coder

Copy link
Copy Markdown
Owner

Summary

Completes Phase 1 — MCP & Tool Firewall per docs/PHASE1_HANDOFF.md, reusing PR #2 primitives (pin/scan/allowlist) without rewriting them.

Deliverable Location
D1 Adapter (file/stdio/HTTP) python/agentshield/mcp/adapter.py
D2 Pin registry + rug-pull drift python/agentshield/mcp/registry.py
D3 Audit pipeline python/agentshield/mcp/audit.py + CLI mcp audit
D4 Policy YAML/JSON python/agentshield/mcp/policy.py
D5 JWT API server/routes/mcp.js, server/services/mcp/*
D6 AgentBOM v1 python/agentshield/bom/agentbom.py
D7 CI action .github/actions/mcp-audit, example workflow
D8 Docs docs/MCP_SECURITY.md, API/README/STRATEGY
Gaps (no GitHub issues) docs/PHASE1_GAPS.md

Test plan

  • cd python && pip install -e ".[dev]" && pytest — 29 passed
  • cd server && npm run test:assure — 13 passed (Assure + MCP)
  • agentshield mcp audit --tools examples/tools_poisoned.json --policy examples/policy.yaml --ignore-unpinned --sarif /tmp/mcp.sarif --fail-on-severity high — exits non-zero (Critical), SARIF + AgentBOM produced
  • Rug-pull + allowlist unit coverage in Python and Node tests

Notes

  • Human approval UX, CycloneDX, Mongo pin store, full MCP SDK lifecycle → deferred in docs/PHASE1_GAPS.md for maintainers to issue after merge.
  • Example workflow is workflow_dispatch only so poisoned fixtures do not break default CI.
Open in Web Open in Cursor 

Wire PR #2 primitives into adapter, pin registry, policy, audit pipeline,
AgentBOM, JWT /api/mcp routes, GitHub Action, and docs. Ship PHASE1_GAPS.md
for maintainer follow-ups (no GitHub issues created).

Co-authored-by: New World Coder <new-world-coder@users.noreply.github.com>
@new-world-coder
new-world-coder marked this pull request as ready for review August 11, 2026 11:48
@new-world-coder
new-world-coder merged commit ff73f23 into main Aug 11, 2026
7 checks passed
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